*** yamamoto has joined #openstack-fwaas | 00:03 | |
*** yamamoto has quit IRC | 00:08 | |
*** hoangcx has joined #openstack-fwaas | 01:16 | |
*** yushiro has joined #openstack-fwaas | 01:40 | |
*** yamamoto has joined #openstack-fwaas | 02:06 | |
*** yamamoto has quit IRC | 02:11 | |
*** yamamoto has joined #openstack-fwaas | 02:45 | |
*** hoangcx has quit IRC | 03:20 | |
*** hoangcx has joined #openstack-fwaas | 03:22 | |
openstackgerrit | Merged openstack/neutron-fwaas master: Updated from global requirements https://review.openstack.org/494854 | 03:34 |
---|---|---|
openstackgerrit | Merged openstack/neutron-fwaas master: Update links in README https://review.openstack.org/490709 | 03:40 |
*** ying_zuo has joined #openstack-fwaas | 04:50 | |
reedip | eezhova : got it ... | 05:41 |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal https://review.openstack.org/494742 | 06:22 |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Add new protocols in Firewalls https://review.openstack.org/440331 | 06:49 |
openstackgerrit | Merged openstack/neutron-fwaas master: Add firewall_policy_id in FWaaS v2 https://review.openstack.org/370731 | 07:22 |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Add validation for Firewall Rules and Policies https://review.openstack.org/488438 | 07:22 |
openstackgerrit | Merged openstack/neutron-fwaas master: [DevStack] Configure iptables_v2 firewall driver for FWaaS V2. https://review.openstack.org/494167 | 07:28 |
*** eezhova has joined #openstack-fwaas | 08:18 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal https://review.openstack.org/494742 | 09:09 |
yushiro | eezhova, ping | 09:23 |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal https://review.openstack.org/494742 | 09:23 |
eezhova | yushiro, pong | 09:23 |
yushiro | eezhova, Hi. Thanks for your patch https://review.openstack.org/#/c/494742 | 09:23 |
yushiro | eezhova, let me check about https://review.openstack.org/#/c/494742/3/neutron_fwaas/db/firewall/v2/firewall_db_v2.py@474 | 09:25 |
yushiro | if rule_info['firewall_rule_id'] is None | 09:25 |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Move linux_utils to oslo_concurrency https://review.openstack.org/442587 | 09:27 |
eezhova | yushiro, 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 |
yushiro | eezhova, OK. I agree with you. | 09:28 |
eezhova | yushiro, I'll be afk for about an hour but please feel free to ping me is you have other concerns :) | 09:30 |
yushiro | eezhova, sure, will comment on your patch. | 09:30 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata https://review.openstack.org/495786 | 09:35 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: FWaaS v2 extension for L2 agent https://review.openstack.org/323971 | 10:07 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: Generate default firewall group via project https://review.openstack.org/425769 | 10:07 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: OVS based l2 Firewall driver for FWaaS v2 https://review.openstack.org/447251 | 10:07 |
openstackgerrit | Akihiro Motoki proposed openstack/neutron-fwaas-dashboard master: doc: remove symlink for releasenotes https://review.openstack.org/495803 | 10:07 |
openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Move linux_utils to oslo_concurrency https://review.openstack.org/442587 | 10:08 |
openstackgerrit | Merged openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata https://review.openstack.org/495786 | 10:13 |
yushiro | eezhova, Thanks for your reply. I just commented to you but please let me know if you confuse my 'unstable' English. | 10:20 |
reedip | guys , https://etherpad.openstack.org/p/neutron_lib_fwaas_punchlist has been updated. | 10:57 |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal https://review.openstack.org/494742 | 11:01 |
reedip | yushiro , xgerman_ : Need to update the scorecard for Pike . Ocata is https://specs.openstack.org/openstack/neutron-specs/specs/stadium/ocata/neutron-fwaas.html#c7 | 11:03 |
reedip | It gives an idea about the stadium criteria | 11:03 |
reedip | L1 and C7 were NO in Ocata but YES in PIKE | 11:03 |
reedip | Similarly D2 is YES as well | 11:04 |
eezhova | yushiro, Thanks for comments, I enhanced validation in the patch | 11:05 |
reedip | JFYI : https://review.openstack.org/#/c/461657/ | 11:06 |
yushiro | eezhova, Thanks for your update. | 11:09 |
yushiro | eezhova, just 1 nit! | 11:10 |
yushiro | eezhova, Please check it. | 11:10 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: FWaaS v2 extension for L2 agent https://review.openstack.org/323971 | 11:12 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: Generate default firewall group via project https://review.openstack.org/425769 | 11:12 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas master: OVS based l2 Firewall driver for FWaaS v2 https://review.openstack.org/447251 | 11:12 |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Update policy on rule addition/removal https://review.openstack.org/494742 | 11:16 |
*** openstackgerrit has quit IRC | 11:17 | |
eezhova | yushiro, good catch! fixed :) | 11:24 |
yushiro | eezhova, LGTM :) waiting Jenkins result | 11:40 |
*** doude has joined #openstack-fwaas | 12:18 | |
*** yamamoto has quit IRC | 13:31 | |
*** yamamoto has joined #openstack-fwaas | 13:57 | |
*** reedip_ has joined #openstack-fwaas | 13:57 | |
*** yamamoto has quit IRC | 13:58 | |
*** yamamoto has joined #openstack-fwaas | 13:59 | |
*** yamamoto has quit IRC | 13:59 | |
*** eezhova has quit IRC | 14:28 | |
*** reedip_ has quit IRC | 14:52 | |
*** eezhova has joined #openstack-fwaas | 14:59 | |
*** yamamoto has joined #openstack-fwaas | 14:59 | |
*** yamamoto has quit IRC | 15:05 | |
*** eezhova has quit IRC | 15:40 | |
*** vishwanathj has joined #openstack-fwaas | 15:58 | |
*** eezhova has joined #openstack-fwaas | 16:28 | |
*** reedip_ has joined #openstack-fwaas | 16:52 | |
*** yamamoto has joined #openstack-fwaas | 17:03 | |
*** yamamoto has quit IRC | 17:09 | |
*** reedip_ has quit IRC | 17:23 | |
*** yamamoto has joined #openstack-fwaas | 18:05 | |
*** yamamoto has quit IRC | 18:11 | |
*** openstackgerrit has joined #openstack-fwaas | 19:25 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron-fwaas master: Fix router update on L3 agent restart https://review.openstack.org/495657 | 19:25 |
*** vishwanathj has quit IRC | 19:35 | |
*** vishwanathj has joined #openstack-fwaas | 19:39 | |
*** eezhova has quit IRC | 19:39 | |
*** eezhova has joined #openstack-fwaas | 19:41 | |
*** yamamoto has joined #openstack-fwaas | 20:04 | |
*** yamamoto has quit IRC | 20:11 | |
*** eezhova has quit IRC | 20:23 | |
*** yamamoto_ has joined #openstack-fwaas | 21:12 | |
*** yamamoto_ has quit IRC | 21:18 | |
*** yamamoto_ has joined #openstack-fwaas | 22:14 | |
*** yamamoto_ has quit IRC | 22:19 | |
*** yamamoto_ has joined #openstack-fwaas | 23:16 | |
*** yamamoto_ has quit IRC | 23:21 | |
*** yushiro is now known as yushiro_kaiouken | 23:37 | |
openstackgerrit | Merged openstack/neutron-fwaas-dashboard master: doc: remove symlink for releasenotes https://review.openstack.org/495803 | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!