Monday, 2017-08-21

*** yamamoto has joined #openstack-fwaas00:03
*** yamamoto has quit IRC00:08
*** hoangcx has joined #openstack-fwaas01:16
*** yushiro has joined #openstack-fwaas01:40
*** yamamoto has joined #openstack-fwaas02:06
*** yamamoto has quit IRC02:11
*** yamamoto has joined #openstack-fwaas02:45
*** hoangcx has quit IRC03:20
*** hoangcx has joined #openstack-fwaas03:22
openstackgerritMerged openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/49485403:34
openstackgerritMerged openstack/neutron-fwaas master: Update links in README  https://review.openstack.org/49070903:40
*** ying_zuo has joined #openstack-fwaas04:50
reedipeezhova : got it ...05:41
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal  https://review.openstack.org/49474206:22
openstackgerritReedip proposed openstack/neutron-fwaas master: Add new protocols in Firewalls  https://review.openstack.org/44033106:49
openstackgerritMerged openstack/neutron-fwaas master: Add firewall_policy_id in FWaaS v2  https://review.openstack.org/37073107:22
openstackgerritReedip proposed openstack/neutron-fwaas master: Add validation for Firewall Rules and Policies  https://review.openstack.org/48843807:22
openstackgerritMerged openstack/neutron-fwaas master: [DevStack] Configure iptables_v2 firewall driver for FWaaS V2.  https://review.openstack.org/49416707:28
*** eezhova has joined #openstack-fwaas08:18
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal  https://review.openstack.org/49474209:09
yushiroeezhova, ping09:23
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal  https://review.openstack.org/49474209:23
eezhovayushiro, pong09:23
yushiroeezhova, Hi. Thanks for your patch https://review.openstack.org/#/c/49474209:23
yushiroeezhova, let me check about https://review.openstack.org/#/c/494742/3/neutron_fwaas/db/firewall/v2/firewall_db_v2.py@47409:25
yushiroif rule_info['firewall_rule_id'] is None09:25
openstackgerritReedip proposed openstack/neutron-fwaas master: Move linux_utils to oslo_concurrency  https://review.openstack.org/44258709:27
eezhovayushiro, sure. my point here is that there is no need to query DB (especially outside of a transaction) when we exclude incorrect rule_info by a simple dict get.09:28
yushiroeezhova, OK. I agree with you.09:28
eezhovayushiro, I'll be afk for about an hour but please feel free to ping me is you have other concerns :)09:30
yushiroeezhova, sure, will comment on your patch.09:30
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata  https://review.openstack.org/49578609:35
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: FWaaS v2 extension for L2 agent  https://review.openstack.org/32397110:07
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Generate default firewall group via project  https://review.openstack.org/42576910:07
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: OVS based l2 Firewall driver for FWaaS v2  https://review.openstack.org/44725110:07
openstackgerritAkihiro Motoki proposed openstack/neutron-fwaas-dashboard master: doc: remove symlink for releasenotes  https://review.openstack.org/49580310:07
openstackgerritReedip proposed openstack/neutron-fwaas master: Move linux_utils to oslo_concurrency  https://review.openstack.org/44258710:08
openstackgerritMerged openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata  https://review.openstack.org/49578610:13
yushiroeezhova, Thanks for your reply.  I just commented to you but please let me know if you confuse my 'unstable' English.10:20
reedipguys , https://etherpad.openstack.org/p/neutron_lib_fwaas_punchlist has been updated.10:57
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal  https://review.openstack.org/49474211:01
reedipyushiro , xgerman_ : Need to update the scorecard for Pike . Ocata is https://specs.openstack.org/openstack/neutron-specs/specs/stadium/ocata/neutron-fwaas.html#c711:03
reedipIt gives an idea about the stadium criteria11:03
reedipL1 and C7 were NO in Ocata but YES in PIKE11:03
reedipSimilarly D2 is YES as well11:04
eezhovayushiro, Thanks for comments, I enhanced validation in the patch11:05
reedipJFYI : https://review.openstack.org/#/c/461657/11:06
yushiroeezhova, Thanks for your update.11:09
yushiroeezhova, just 1 nit!11:10
yushiroeezhova, Please check it.11:10
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: FWaaS v2 extension for L2 agent  https://review.openstack.org/32397111:12
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Generate default firewall group via project  https://review.openstack.org/42576911:12
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: OVS based l2 Firewall driver for FWaaS v2  https://review.openstack.org/44725111:12
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal  https://review.openstack.org/49474211:16
*** openstackgerrit has quit IRC11:17
eezhovayushiro, good catch! fixed :)11:24
yushiroeezhova, LGTM :)  waiting Jenkins result11:40
*** doude has joined #openstack-fwaas12:18
*** yamamoto has quit IRC13:31
*** yamamoto has joined #openstack-fwaas13:57
*** reedip_ has joined #openstack-fwaas13:57
*** yamamoto has quit IRC13:58
*** yamamoto has joined #openstack-fwaas13:59
*** yamamoto has quit IRC13:59
*** eezhova has quit IRC14:28
*** reedip_ has quit IRC14:52
*** eezhova has joined #openstack-fwaas14:59
*** yamamoto has joined #openstack-fwaas14:59
*** yamamoto has quit IRC15:05
*** eezhova has quit IRC15:40
*** vishwanathj has joined #openstack-fwaas15:58
*** eezhova has joined #openstack-fwaas16:28
*** reedip_ has joined #openstack-fwaas16:52
*** yamamoto has joined #openstack-fwaas17:03
*** yamamoto has quit IRC17:09
*** reedip_ has quit IRC17:23
*** yamamoto has joined #openstack-fwaas18:05
*** yamamoto has quit IRC18:11
*** openstackgerrit has joined #openstack-fwaas19:25
openstackgerritElena Ezhova proposed openstack/neutron-fwaas master: Fix router update on L3 agent restart  https://review.openstack.org/49565719:25
*** vishwanathj has quit IRC19:35
*** vishwanathj has joined #openstack-fwaas19:39
*** eezhova has quit IRC19:39
*** eezhova has joined #openstack-fwaas19:41
*** yamamoto has joined #openstack-fwaas20:04
*** yamamoto has quit IRC20:11
*** eezhova has quit IRC20:23
*** yamamoto_ has joined #openstack-fwaas21:12
*** yamamoto_ has quit IRC21:18
*** yamamoto_ has joined #openstack-fwaas22:14
*** yamamoto_ has quit IRC22:19
*** yamamoto_ has joined #openstack-fwaas23:16
*** yamamoto_ has quit IRC23:21
*** yushiro is now known as yushiro_kaiouken23:37
openstackgerritMerged openstack/neutron-fwaas-dashboard master: doc: remove symlink for releasenotes  https://review.openstack.org/49580323:45

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!