Tuesday, 2017-09-26

*** SumitNaiksatam has quit IRC00:05
*** yamamoto has quit IRC00:16
*** yamamoto has joined #openstack-fwaas00:21
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731901:03
reediphi doude01:40
*** yamamoto has quit IRC01:50
*** yamamoto has joined #openstack-fwaas01:50
*** yamamoto_ has joined #openstack-fwaas01:51
*** yamamoto has quit IRC01:55
*** mlavalle has quit IRC02:14
reedipivasilevskaya1 : IIUC , the max depth recursion error started coming from PS#42 , but there was no proper execution of py27 tests from PS#38, so to be sure, I think we should try to see what are the changes from PS#38 to 4302:49
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731902:53
*** SumitNaiksatam has joined #openstack-fwaas02:56
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731903:09
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731903:30
reedipivasilevskaya1 : maybe I found it. In PS#38, the https://review.openstack.org/#/c/425769/38/neutron_fwaas/db/firewall/v2/firewall_db_v2.py@968  has a check for is_admin. That check has been removed in PS#43 by logic03:47
reedipivasilevskaya1 : The test which fails actually mocks  the plugin's get_firewall_group with a side-effect of getdelete03:48
reedipivasilevskaya1 : The mock in turn calls the firewall_db_v2 's delete_firewall_group, which calls the _get_firewall_group again (??)03:48
reedipivasilevskaya1 : this is where the cycle seems to form. Correct me if I am wrong03:49
reediphttp://logs.openstack.org/69/425769/43/check/gate-neutron-fwaas-python27-ubuntu-xenial/225c4df/console.html#_2017-09-25_20_39_23_69156003:56
*** hoangcx has joined #openstack-fwaas04:27
*** yamamoto_ has quit IRC05:37
*** yamamoto has joined #openstack-fwaas05:37
*** eezhova has joined #openstack-fwaas06:03
reedipivasilevskaya1 : I updated the code, if you feel its incorrect, let me know06:32
reedipP.S. : I wont be attending today's meeting06:33
openstackgerritReedip proposed openstack/neutron-fwaas master: Introduce default firewall groups  https://review.openstack.org/42576906:39
*** eezhova has quit IRC06:54
*** annp has joined #openstack-fwaas07:34
annpreedip: ping07:35
*** yamamoto has quit IRC08:31
*** yamamoto has joined #openstack-fwaas08:34
*** yamamoto has quit IRC08:39
*** yamamoto has joined #openstack-fwaas08:47
*** yamamoto has quit IRC08:47
*** eezhova has joined #openstack-fwaas08:58
*** yamamoto has joined #openstack-fwaas09:10
*** yamamoto has quit IRC09:12
*** yamamoto has joined #openstack-fwaas09:14
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas master: Refactor _get_routers_in_project method  https://review.openstack.org/50744009:15
*** yamamoto has quit IRC09:16
*** yamamoto has joined #openstack-fwaas09:16
openstackgerritSarath Chandra Mekala proposed openstack/neutron-fwaas-dashboard master: FWaaS V2 Horizon Dashboard  https://review.openstack.org/47584009:29
*** yamamoto has quit IRC09:31
*** yamamoto has joined #openstack-fwaas09:35
openstackgerritNguyen Phuong An proposed openstack/neutron-fwaas master: FWaaS v2 extension for L2 agent  https://review.openstack.org/32397110:00
*** eezhova has quit IRC11:04
*** eezhova has joined #openstack-fwaas11:10
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731912:27
*** yamamoto has quit IRC12:38
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731912:43
*** yamamoto has joined #openstack-fwaas12:55
*** yamamoto has quit IRC13:06
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas master: Updated from global requirements  https://review.openstack.org/50731913:27
xgerman_Will be in meetings - so if someone else can run today’s meeting... I will be attending but distracted. Sridhar yushiro13:55
*** SarathMekala has joined #openstack-fwaas13:56
*** yushiro has joined #openstack-fwaas13:59
*** yamamoto has joined #openstack-fwaas14:07
*** yamamoto has quit IRC14:14
*** eezhova has quit IRC14:37
*** eezhova has joined #openstack-fwaas15:20
*** SarathMekala has quit IRC15:25
ivasilevskaya1reedip, thanks for fixing the test!15:27
ivasilevskaya1Actually, I personally think default fwg patch looks good enough and deserves some attention. Guys xgerman_, yushiro, annp - maybe you could take a look? https://review.openstack.org/#/c/42576915:29
ivasilevskaya1in the meanwhile I'll give it a try on devstack :)15:30
*** yushiro has quit IRC15:40
*** SumitNaiksatam has quit IRC16:49
*** SumitNaiksatam has joined #openstack-fwaas17:21
*** SridarK has joined #openstack-fwaas18:05
openstackgerritInessa Vasilevskaya proposed openstack/neutron-fwaas master: Introduce firewall l2 driver base  https://review.openstack.org/50432418:06
*** eezhova has quit IRC18:09
*** eezhova has joined #openstack-fwaas18:14
*** eezhova has quit IRC18:21
-openstackstatus- NOTICE: The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.18:24
*** eezhova has joined #openstack-fwaas20:59
*** eezhova has quit IRC21:46
*** yamamoto has joined #openstack-fwaas23:05
*** SumitNaiksatam has quit IRC23:58

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