Friday, 2015-08-28

*** achanda has quit IRC00:00
*** aranjan_ has joined #openstack-neutron00:00
*** gildub has quit IRC00:01
*** achanda_ has quit IRC00:01
*** vivek-ebay has joined #openstack-neutron00:01
*** breitz has quit IRC00:01
*** vivek-ebay has quit IRC00:02
*** vivek-ebay has joined #openstack-neutron00:02
*** dims has joined #openstack-neutron00:03
*** oomichi has joined #openstack-neutron00:03
*** pai15 has joined #openstack-neutron00:04
*** salv-orlando has quit IRC00:04
*** dims has quit IRC00:05
*** dims has joined #openstack-neutron00:05
*** enikanorov has quit IRC00:05
*** dave-mccowan has joined #openstack-neutron00:06
*** mickeys has quit IRC00:06
openstackgerritJames Arendt proposed openstack/neutron-fwaas: FWaaS: Fix 500 INTERNAL error on too long name or description  https://review.openstack.org/21736300:07
*** enikanorov has joined #openstack-neutron00:07
*** arosen has joined #openstack-neutron00:07
*** arosen is now known as Guest9992200:08
*** dims has quit IRC00:10
*** thorst1 has quit IRC00:10
*** dims has joined #openstack-neutron00:17
*** johnbelamaric has joined #openstack-neutron00:19
*** johnbelamaric has quit IRC00:21
*** pai15 has quit IRC00:22
*** beagles has quit IRC00:22
*** padkrish has quit IRC00:22
*** pai15 has joined #openstack-neutron00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-neutron00:23
*** gildub has joined #openstack-neutron00:25
*** pai15 has quit IRC00:26
*** pai15 has joined #openstack-neutron00:26
*** clev-away is now known as clev00:26
*** pai15 has quit IRC00:37
*** pai15 has joined #openstack-neutron00:37
*** sridhar_ram has joined #openstack-neutron00:38
*** markvoelker has joined #openstack-neutron00:39
*** pai15 has quit IRC00:42
*** Guest99922 has quit IRC00:43
*** hitalia has quit IRC00:44
*** _cjones_ has quit IRC00:44
*** britthou_ has joined #openstack-neutron00:48
*** abhiraut has joined #openstack-neutron00:49
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378100:50
*** britthouser has quit IRC00:51
*** britthou_ has quit IRC00:52
*** shivrao has quit IRC00:53
*** s3wong has quit IRC00:54
*** EinstCrazy has joined #openstack-neutron00:56
*** zhangjn has joined #openstack-neutron00:57
*** aranjan_ has quit IRC00:57
*** zhenguo has joined #openstack-neutron00:57
*** aranjan has joined #openstack-neutron00:57
openstackgerritxin wu proposed openstack/neutron: new version scheme.  https://review.openstack.org/21800001:03
*** dannywilson has quit IRC01:03
*** minwang2 has quit IRC01:03
openstackgerritZhu Zhu proposed openstack/neutron: Changed filter field to router_id  https://review.openstack.org/21497401:05
*** aranjan has quit IRC01:05
*** vivek-ebay has quit IRC01:08
*** hoangcx has joined #openstack-neutron01:09
*** zzzeek has joined #openstack-neutron01:14
*** thorst1 has joined #openstack-neutron01:15
*** annp has joined #openstack-neutron01:15
*** carl_baldwin has quit IRC01:15
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378101:17
*** iwamoto has joined #openstack-neutron01:19
*** bharathm has quit IRC01:19
*** jwy has quit IRC01:19
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity  https://review.openstack.org/21381901:20
*** carl_baldwin has joined #openstack-neutron01:21
*** thorst1 has quit IRC01:23
*** sridhar_ram has quit IRC01:26
*** erkules_ has joined #openstack-neutron01:27
*** erkules has quit IRC01:30
*** pai15 has joined #openstack-neutron01:32
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Kill contention between update and delete  https://review.openstack.org/21803401:32
armaxsc68cal: ^^01:32
armaxsc68cal: I finally got the head out my ass01:33
* armax thinks01:33
*** stanzgy has joined #openstack-neutron01:34
sc68calarmax: wha?01:34
armaxlooks like the bug 1483875 still bugging01:34
openstackbug 1483875 in neutron "FWaaS - firewall deleted unexpectedly by agent" [Critical,In progress] https://launchpad.net/bugs/1483875 - Assigned to Armando Migliaccio (armando-migliaccio)01:34
*** Guest62749 has joined #openstack-neutron01:34
armaxso I tried somethign different01:34
sc68calarmax: Damn.01:34
*** y_sawai has joined #openstack-neutron01:35
*** signed8bit has quit IRC01:35
*** Guest62749 has quit IRC01:36
*** yfried_ has quit IRC01:36
openstackgerritArmando Migliaccio proposed openstack/neutron: Testing if we can claim the bug killed once and for all  https://review.openstack.org/21803501:36
armaxsc68cal: ^^^01:36
sc68calarmax: OK, I'll take a look. Don't beat yourself up, seems like it's a knotty issue01:36
armaxwell01:37
armaxI only needed some piece and quite to think about this a tad harder01:37
* armax lazy01:37
sc68calwell I have peace and quiet constantly, so I think even at your worst you have a leg up on me :)01:39
armax:)01:39
armaxthat’s funny01:39
sc68calgreenlet and eventlet are not my strong points01:40
armaxmine neither01:42
*** pai15 has quit IRC01:44
*** jwy has joined #openstack-neutron01:44
*** pai15 has joined #openstack-neutron01:44
bharath_sc68cal: i have sent additional info to the mailing list. Hope additional info gives more clarity01:45
sc68calbharath_: thanks. It's a case of I know the parts of the code you're talking about, but I just don't know the side effect of router_info being empty01:46
*** spandhe has quit IRC01:47
*** pai15 has quit IRC01:48
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB  https://review.openstack.org/18336901:48
bharath_sc68cal: Do you know in what conditions router_info will get updated?01:49
*** shz has quit IRC01:50
*** abhiraut has quit IRC01:50
*** hanzhang has joined #openstack-neutron01:50
*** tiswanso has joined #openstack-neutron01:50
*** lyanchih has joined #openstack-neutron01:51
*** liangbo has joined #openstack-neutron01:51
*** madhu_ak has quit IRC01:53
sc68calbharath_: I think it's sent via RPC from the neutron plugin to the l3 agent when something gets changed via the API01:54
bharath_sc68cal:    _process_routers_loop() should take care of updating router info by ultimately calling _process_added_router ,01:54
sc68calbharath_: yeah that sounds familiar01:54
*** tiswanso_ has joined #openstack-neutron01:55
bharath_sc68cal: is firewall agent working in local setup?01:55
sc68calbharath_: do you think it was a change to the fwaas repo, or neutron ? I replied on the ML but I had only done a quick look in both fwaas and the neutron repo - nothing obvious jumped out at me01:56
*** pai15 has joined #openstack-neutron01:56
bharath_yes01:56
sc68calbharath_: yes to which?01:57
*** tiswanso has quit IRC01:57
*** tobias- has quit IRC01:58
sc68calbharath_: I just got my lab environment at home set up, I have the firewall plugin configured in neutron but haven't had a chance to exercise it.02:00
*** yamamoto has joined #openstack-neutron02:03
openstackgerritZhu Zhu proposed openstack/neutron: Changed filter field to router_id  https://review.openstack.org/21497402:03
*** huazhihao has joined #openstack-neutron02:03
bharath_sc68cal: Reason i am suspecting neutron repo changes is because we haven't made any changes to it in last 2 months. Our CI is fine  and  tempest fw cases are passed.  later  i found  that our tempest is not running fw cases anymore as these test case was move out of neutron_fwaas to neutron. CI is simply getting passed with 0 testcases.02:03
*** sdake_ is now known as sdake02:04
bharath_sc68cal: Due to breakage in CI , i dont have exact commit which broken it02:04
*** boltR has quit IRC02:05
*** dims has quit IRC02:06
bharath_sc68cal:  *Reason i am suspecting neutron repo changes is because we haven't made any changes to fw or l3 agents in last 2 months02:06
*** JoshNang has quit IRC02:06
sc68calbharath_: agree - that sounds like a fair assumption since fwaas just got restarted with the new core team.02:06
sc68calbharath_: let's do a little more digging - are you in asia-pacific timezone? If so let's put it on the agenda for next week's meeting02:07
bharath_sc68cal: yeah02:07
*** lyanchih has quit IRC02:08
bharath_sc68cal: yeah asia pacific,02:08
sc68calbharath_: ok - cool. I'll try and see if I can catch up to you by doing some debugging of the fwaas code in my new devstack lab.02:09
*** tflynn has joined #openstack-neutron02:09
*** JoshNang has joined #openstack-neutron02:09
bharath_sc68cal: ok thanks, can i contact on you private mail address to discuss ?02:10
*** shivrao_ has joined #openstack-neutron02:10
*** boris-42 has quit IRC02:10
*** lyanchih has joined #openstack-neutron02:12
openstackgerritZhaoBo proposed openstack/python-neutronclient: Fix return inappropriate message when run net-list -F {fields}  https://review.openstack.org/21307902:13
*** boltR has joined #openstack-neutron02:13
*** SourabhP has joined #openstack-neutron02:13
*** minwang2 has joined #openstack-neutron02:15
*** SourabhP_ has joined #openstack-neutron02:16
*** yamamoto has quit IRC02:17
*** SourabhP has quit IRC02:19
*** SourabhP_ is now known as SourabhP02:19
*** yamahata has quit IRC02:19
*** raginbajin has quit IRC02:23
openstackgerritZhangqi Chen proposed openstack/neutron: Fix algorithm error in _modify_rules  https://review.openstack.org/21708702:26
*** tflynn has quit IRC02:26
*** carl_baldwin has quit IRC02:27
*** raginbajin has joined #openstack-neutron02:28
*** saksham has quit IRC02:28
*** achanda has joined #openstack-neutron02:28
openstackgerritShaoHe Feng proposed openstack/neutron: Notify specific agent when create or update port  https://review.openstack.org/21545702:28
*** carl_baldwin has joined #openstack-neutron02:30
*** nlahouti has joined #openstack-neutron02:30
*** spandhe has joined #openstack-neutron02:32
*** bharath_ has quit IRC02:32
*** n00btube has joined #openstack-neutron02:32
*** dims has joined #openstack-neutron02:33
*** n00btube has left #openstack-neutron02:33
*** nlahouti has quit IRC02:35
*** carl_baldwin has quit IRC02:37
*** sthillma has quit IRC02:38
*** rossigee_ has joined #openstack-neutron02:40
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513602:41
*** dave-mccowan has quit IRC02:43
*** azbiswas has joined #openstack-neutron02:46
*** agireud has joined #openstack-neutron02:46
*** mlavalle has quit IRC02:47
*** azbiswas has quit IRC02:51
*** SourabhP has quit IRC02:52
*** hakimo has joined #openstack-neutron02:52
*** glavni_ninja has quit IRC02:53
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Allow enabling detailed logging for OpenSwan  https://review.openstack.org/21470102:54
*** shoutm has quit IRC02:54
*** hakimo_ has quit IRC02:54
*** yamamoto has joined #openstack-neutron02:56
*** glavni_ninja has joined #openstack-neutron02:57
*** insequent has joined #openstack-neutron02:59
*** baoli has joined #openstack-neutron03:01
*** spandhe has quit IRC03:01
*** jwy has quit IRC03:01
*** shivrao_ has quit IRC03:03
openstackgerritYi Zhao proposed openstack/neutron: Delete gateway conntrack state when remove external gateway.  https://review.openstack.org/21756403:06
*** lyanchih has quit IRC03:08
*** gberginc has quit IRC03:08
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources  https://review.openstack.org/21358603:09
*** lyanchih has joined #openstack-neutron03:11
*** pai15 has quit IRC03:17
*** pai15 has joined #openstack-neutron03:18
*** minwang2 has quit IRC03:18
*** shivrao has joined #openstack-neutron03:20
*** dims has quit IRC03:23
*** tflynn has joined #openstack-neutron03:23
*** srilatha has joined #openstack-neutron03:24
*** Samos123 has quit IRC03:26
*** rm_work has quit IRC03:27
*** daneyon has joined #openstack-neutron03:27
*** comstud has quit IRC03:28
*** daneyon_ has quit IRC03:28
*** jroll has quit IRC03:28
*** mgagne has quit IRC03:28
*** rstarr has quit IRC03:28
*** clev has quit IRC03:28
*** lbragstad has quit IRC03:28
*** zacksh has quit IRC03:28
*** wendar has quit IRC03:28
*** gus has quit IRC03:28
*** Nakato has quit IRC03:28
*** wendar_ has joined #openstack-neutron03:28
*** sbalukoff has quit IRC03:29
*** cdl has joined #openstack-neutron03:29
*** cdl has left #openstack-neutron03:29
*** minwang2 has joined #openstack-neutron03:29
*** salv-orlando has joined #openstack-neutron03:30
*** shwetaap has joined #openstack-neutron03:30
*** zacksh has joined #openstack-neutron03:31
*** Nakato has joined #openstack-neutron03:31
*** gus has joined #openstack-neutron03:31
*** rm_work has joined #openstack-neutron03:31
*** rstarr has joined #openstack-neutron03:31
*** Samos123 has joined #openstack-neutron03:32
*** jroll has joined #openstack-neutron03:32
*** clev has joined #openstack-neutron03:33
*** sbalukoff has joined #openstack-neutron03:33
*** agireud has quit IRC03:33
*** fawadkhaliq has joined #openstack-neutron03:34
*** mgagne has joined #openstack-neutron03:34
*** gberginc has joined #openstack-neutron03:36
*** salv-orlando has quit IRC03:36
*** comstud has joined #openstack-neutron03:37
*** lbragstad has joined #openstack-neutron03:37
*** hoangcx has quit IRC03:38
*** shwetaap1 has joined #openstack-neutron03:41
*** achanda has quit IRC03:41
*** shivrao has quit IRC03:42
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add enable_new_agents to neutron server  https://review.openstack.org/21698003:44
*** sdake_ has joined #openstack-neutron03:44
*** shwetaap has quit IRC03:44
*** tiswanso_ has quit IRC03:45
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513603:45
*** sdake has quit IRC03:48
*** azbiswas has joined #openstack-neutron03:53
*** sdake has joined #openstack-neutron03:57
*** baoli has quit IRC03:57
*** zzzeek has quit IRC03:59
*** sdake_ has quit IRC04:01
*** ajmiller_ has joined #openstack-neutron04:09
*** armax has quit IRC04:09
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513604:09
*** links has joined #openstack-neutron04:09
*** sdake_ has joined #openstack-neutron04:14
*** armax has joined #openstack-neutron04:16
*** bharath_ has joined #openstack-neutron04:17
*** sdake has quit IRC04:18
*** baojg has joined #openstack-neutron04:19
*** bharathm has joined #openstack-neutron04:20
*** sdake_ is now known as sdake04:22
*** gildub has quit IRC04:24
*** bharathm has quit IRC04:24
*** sdake_ has joined #openstack-neutron04:25
*** nlahouti has joined #openstack-neutron04:26
*** pai15 has quit IRC04:26
*** srilatha has quit IRC04:27
*** srilatha has joined #openstack-neutron04:27
*** nlahouti1 has joined #openstack-neutron04:28
*** sdake has quit IRC04:29
*** rha has quit IRC04:30
*** frickler has quit IRC04:30
*** nlahouti has quit IRC04:31
*** srilatha has quit IRC04:32
*** lyanchih has quit IRC04:33
*** amotoki has joined #openstack-neutron04:33
*** fitoduarte has joined #openstack-neutron04:33
*** sthillma has joined #openstack-neutron04:35
*** tflynn_ has joined #openstack-neutron04:35
*** sthillma_ has joined #openstack-neutron04:36
*** tflynn has quit IRC04:38
*** sthillma has quit IRC04:40
*** sthillma_ is now known as sthillma04:40
*** ajmiller_ has quit IRC04:50
*** boltR has quit IRC04:51
*** fedexo has joined #openstack-neutron04:51
*** huazhihao has quit IRC04:53
*** oomichi has quit IRC04:53
ajokevinbenton amotoki : https://review.openstack.org/#/c/215608/   if you have time to re-review, it could possibly be ready now :)04:54
amotokiajo: ack, let me check in the next hour.04:55
ajothanks amotoki, very appreciated04:56
*** bharath_ has quit IRC05:03
*** rossella_s has quit IRC05:05
*** lyanchih has joined #openstack-neutron05:06
*** rossella_s has joined #openstack-neutron05:06
amotokiajo: have you seen this kind of python27 failure? http://logs.openstack.org/48/217048/1/gate/gate-neutron-python27/dc83518/testr_results.html.gz05:08
ajoamotoki, readin05:08
ajoamotoki, no, it could be related to a recent patch05:09
ajoamotoki, is it happening in unrelated patches?05:09
*** huazhihao has joined #openstack-neutron05:09
amotokiajo: it happened in https://review.openstack.org/#/c/217048/05:09
*** azbiswas has quit IRC05:09
ajohmm05:10
*** azbiswas has joined #openstack-neutron05:10
ajoamotoki: I'm going to try with the latest master05:11
ajoseems cleanup related05:11
*** nyechiel_ has joined #openstack-neutron05:11
*** ijw has quit IRC05:12
*** ijw has joined #openstack-neutron05:12
*** ijw has quit IRC05:14
*** ijw has joined #openstack-neutron05:14
*** liangbo has quit IRC05:14
*** azbiswas has quit IRC05:14
*** lajos-katona has joined #openstack-neutron05:15
openstackgerritSridhar Gaddam proposed openstack/neutron: Fix import path in neutron-sanity-check for ml2_sriov opts  https://review.openstack.org/21760905:15
*** liangbo has joined #openstack-neutron05:16
ajook, amotoki , happening if I run ./run_tests.sh neutron.tests.unit.services.qos.test_qos_plugin.TestQosPlugin.test_update_policy_rule_for_nonexistent_policy   individually on this branch05:16
ajobut not before05:16
ajolet's WIP the patch, I'll investigate05:16
amotokiagree. something seems changed.05:17
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler  https://review.openstack.org/20443605:17
*** ildikov has joined #openstack-neutron05:19
*** shivrao has joined #openstack-neutron05:20
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add functional test for availability_zone support  https://review.openstack.org/21097705:20
ajostrange thing amotoki05:20
ajoI added some debug prints to mock.py05:20
ajoand it seems to crash when trying to unpatch:05:20
ajo<class 'neutron.objects.qos.policy.QosPolicy'> get_by_id <classmethod object at 0x3910f68>05:20
amotokihmm05:21
*** minwang2 has quit IRC05:21
*** lazy_prince has joined #openstack-neutron05:21
amotokiI'd try to investigate a bit later. now i have to write monthly report.05:23
*** minwang2 has joined #openstack-neutron05:23
ajoamotoki, no worries, I keep investigating05:24
ajoamotoki, I will ping you if I fail05:24
ajothanks man05:24
*** shwetaap1 has quit IRC05:25
ajohmm, if I run run_tests -d, it won't fail05:25
ajobut it will if I let it run normally05:26
iwamotoajo: about https://review.openstack.org/#/c/153946/, do you have any good idea allocating port number per test instance?05:29
iwamotoajo: test instances are separate processes and it doesn't seem obvious to me05:30
iwamotoajo: it currently uses file based locks for mutual exclusion05:30
*** mohankumar has joined #openstack-neutron05:30
ajoiwamoto, in the functional tests for ovs-agent, we create several bridges, and separately command those05:31
ajoiwamoto, you could try to allocate a random port in a range05:31
ajowith retries...05:31
*** salv-orlando has joined #openstack-neutron05:31
ajoand then set such manager from the test into the specific bridge05:31
ajobut btw, isn't that what we already do for ovsdb native with the set-manager thing?05:32
ajoI'd ask amuller when he's online or via email05:32
*** numans has joined #openstack-neutron05:33
*** agireud has joined #openstack-neutron05:33
iwamotoajo: thanks.  I'll look at them05:33
*** nyechiel_ has quit IRC05:34
*** anilvenkata has joined #openstack-neutron05:34
*** nyechiel_ has joined #openstack-neutron05:35
*** salv-orlando has quit IRC05:36
ajoamotoki: found it, we're patching a method in the qos_plugin we get, which could be gone at test cleanup05:38
ajoamotoki: adding a fix in front of the other patch05:38
*** gongysh has joined #openstack-neutron05:38
amotokiajo: nice finding out. thanks05:39
*** mohankumar has quit IRC05:39
*** yamamoto has quit IRC05:40
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add enable_new_agents to neutron server  https://review.openstack.org/21698005:42
*** aranjan has joined #openstack-neutron05:43
*** agireud has quit IRC05:47
*** yamamoto has joined #openstack-neutron05:48
iwamotoyamamoto: hi:)05:49
yamamotohi05:49
*** neeti has joined #openstack-neutron05:49
iwamotoyamamoto: do you happen to have any idea on ajo's last comment on ovs-ofctl-to-python?05:50
iwamotoyamamoto: as the original author05:50
yamamotowhich comment?05:50
*** nlahouti1 has quit IRC05:50
*** nlahouti has joined #openstack-neutron05:50
iwamotoabout concurrently running native of_interface tests05:51
ajoiwamoto, yamamoto , in your case,05:51
*** mohankumar has joined #openstack-neutron05:52
yamamotoiwamoto: it should work05:52
ajodo you connect from ryu as a manager, or do you listen?05:52
ajoand ovs connects to you?05:52
*** yamahata has joined #openstack-neutron05:52
ajoexcuse my ignorance in that regard...05:52
amotokiovs connects to ryu.05:52
ajook, so probably it's a sort of making our use of ryu parametrizable, so we can specify a different port per test worker05:53
yamamotoiwamoto: retrying might be tricky due to the way ryu starts listening though05:53
iwamotohashing with process id would usually work ;)05:56
iwamotodoesn't sound great05:56
*** armax has quit IRC05:56
*** clev is now known as clev-away05:56
openstackgerritgongysh proposed openstack/neutron: prohibit tenant to remove provided not shared policy  https://review.openstack.org/21709205:57
gongyshajo: ping05:57
*** SridharG has joined #openstack-neutron05:57
ajohi gongysh05:58
gongyshhttps://review.openstack.org/#/c/217048/05:58
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Base on SqlTestCase to init db tables correctly  https://review.openstack.org/21704805:58
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Stops patching an object method which could be gone at cleanup  https://review.openstack.org/21810405:58
*** azbiswas has joined #openstack-neutron05:58
ajoamotoki, ^ that shall fix it ;) I pushed it in front of the SqlTestCase change05:58
gongyshok05:59
*** mohankumar has quit IRC05:59
*** bharath_ has joined #openstack-neutron05:59
ajogongysh: ack, reading :)06:00
*** Marga_ has joined #openstack-neutron06:00
*** Marga_ has quit IRC06:00
*** Marga_ has joined #openstack-neutron06:01
ajogongysh, thanks for helping with qos06:03
*** sdake_ is now known as sdake06:03
gongyshajo: why other patch work at https://review.openstack.org/#/c/218104/1/neutron/tests/unit/services/qos/test_qos_plugin.py?06:04
gongyshajo: we still have mock.patch('neutron.db.api.create_object').start() in the setup()06:05
ajogongysh, the issue is that if we mock.patch.object06:05
ajothe plugin that we just got06:05
ajoif the cleanup order is inverted06:05
ajo(first plugin unloaded, then mock cleaned...)06:05
ajomock clean fails06:05
ajothat's ok06:06
gongyshajo: got it, thanks06:06
ajothere we are patching a normal lib method06:06
ajoin the other case we were patching self.qos_plugin06:06
ajowhich was a reference to a running class instance06:06
gongyshajo: I see this weak reference error when I run the cases in eclipse, but it does not happen when I run it via tox -e py27. so I ignored it.06:08
ajogongysh, hehe, wheh those things happen it's generally related to concurrency06:08
amotokigongysh: I ran tox -r -e py27 and it failed with weakly-referenced error.06:09
ajogongysh and we should debug or find help to debug before pushing to gate, otherwise in gate it can become a non-deterministic failure06:09
ajoyep06:09
ajofor me06:09
ajo./run_tests.sh -d   did not fail06:09
ajobut ./run_tests.sh did06:09
*** mohankumar has joined #openstack-neutron06:12
*** aranjan has quit IRC06:12
amotokiand I think it depends on when python gc runs.06:13
amotokiiwamoto: yamamoto: how about adding Co-Authored-By to the commit message to ryu-ovs-work?06:16
iwamotoamotoki: otherwise my name doesn't show up on the commit log?06:17
*** salv-orlando has joined #openstack-neutron06:18
*** mickeys has joined #openstack-neutron06:18
yamamotoiwamoto: iirc stackalytics etc recongizes the tag06:18
amotokiiwamoto: yes. and I think Co-Authored-By field is counted in stackalytics or similar ones06:18
amotokiI am not sure which of you two is now a main contributor and which one is better to be a co-author. It is up to you two.06:19
*** nyechiel_ has quit IRC06:20
*** miyagishi_t has joined #openstack-neutron06:20
iwamotoadding "co-authored-by: iwamoto..." would be fine. i think06:20
*** mohankumar has quit IRC06:20
* iwamoto wonders if other project or oslo has a file based port allocator ;)06:21
*** yog_ has joined #openstack-neutron06:22
ajogongysh, I added some comments to https://review.openstack.org/#/c/217092/ , good work btw.06:24
*** minwang2 has quit IRC06:25
*** anshul has joined #openstack-neutron06:28
yamamotoiwamoto: we have too many instances of db-based resource allocators.  you should write a universal one.  https://xkcd.com/927/06:28
*** fawadkhaliq has quit IRC06:29
iwamotoyamamoto: lol06:29
*** gildub has joined #openstack-neutron06:31
*** puck has quit IRC06:32
*** clayton has quit IRC06:33
*** baojg has quit IRC06:33
*** puck has joined #openstack-neutron06:33
*** clayton has joined #openstack-neutron06:33
*** agireud has joined #openstack-neutron06:33
*** vikram has joined #openstack-neutron06:33
*** sthillma has quit IRC06:35
*** liudong has joined #openstack-neutron06:35
*** agireud has quit IRC06:38
*** ankit_ag has joined #openstack-neutron06:42
*** JoeStack has joined #openstack-neutron06:42
openstackgerritenikanorov proposed openstack/neutron: Add flows to tunnel bridge with proper cookie.  https://review.openstack.org/21811806:48
*** mickeys has quit IRC06:49
*** gberginc has quit IRC06:52
*** devvesa has joined #openstack-neutron06:55
openstackgerritgongysh proposed openstack/neutron: prohibit tenant to remove provided not shared policy  https://review.openstack.org/21709206:55
gongyshajo: https://review.openstack.org/#/c/216603/  review required too06:57
*** slaweq has joined #openstack-neutron06:58
ajoack gongysh  :)06:58
*** mohankumar has joined #openstack-neutron06:59
*** britthouser has joined #openstack-neutron07:02
*** mohankumar has quit IRC07:04
*** achanda has joined #openstack-neutron07:05
*** ronis has joined #openstack-neutron07:05
*** luqas has joined #openstack-neutron07:07
*** jpena has joined #openstack-neutron07:08
ajogongysh: some extra comments07:08
*** fzdarsky|afk has joined #openstack-neutron07:10
*** nplanel has quit IRC07:11
*** fzdarsky has joined #openstack-neutron07:11
*** spandhe has joined #openstack-neutron07:11
*** Yuji has joined #openstack-neutron07:12
gongyshajo: https://review.openstack.org/#/c/216603/3/neutron/services/qos/qos_plugin.py07:12
gongyshajo: policy object should contain no business logic, I think.07:13
*** EinstCrazy has quit IRC07:13
*** nyechiel_ has joined #openstack-neutron07:13
gongyshajo: the business logic should be in qos_plugin07:13
ajogongysh, it's it's "own business" :)07:13
ajogongysh, for get_policy, we raise the exception at lower layers07:13
ajoI believe having the raise inside makes the upper code cleaner and easier to maintain07:14
openstackgerritenikanorov proposed openstack/neutron: Add flows to tunnel bridge with proper cookie.  https://review.openstack.org/21811807:14
ajoand we stay in the same semantics like other calls in the context of qos:07:14
gongyshajo: ok07:14
ajo"trying to get, doesn't exist - > exception raised"  , while returning None is a different kind of semantics07:14
*** zhangjn has quit IRC07:15
*** mohankumar has joined #openstack-neutron07:16
*** fawadkhaliq has joined #openstack-neutron07:18
*** neeti has quit IRC07:19
*** neeti has joined #openstack-neutron07:19
*** bharathm has joined #openstack-neutron07:19
*** azbiswas has quit IRC07:19
*** jlibosva has joined #openstack-neutron07:22
*** moha_hunt has quit IRC07:22
*** Ramanjaneya has joined #openstack-neutron07:23
*** mohankumar has quit IRC07:24
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources  https://review.openstack.org/21358607:24
*** tmorin has joined #openstack-neutron07:24
*** Ramanjaneya has quit IRC07:29
*** ygbo has joined #openstack-neutron07:30
openstackgerritgongysh proposed openstack/neutron: Add policy and policy rule belongs check  https://review.openstack.org/21660307:30
*** frickler has joined #openstack-neutron07:32
*** Marga_ has quit IRC07:34
*** Marga_ has joined #openstack-neutron07:35
*** slaweq has quit IRC07:36
*** lukasa has joined #openstack-neutron07:39
openstackgerritvikram.choudhary proposed openstack/neutron: Adding networking-onos to sub_projects document  https://review.openstack.org/21813607:40
*** doude has joined #openstack-neutron07:52
openstackgerritvikram.choudhary proposed openstack/neutron: Add lieutenants contact for networking-onos  https://review.openstack.org/21814007:53
*** fawadk has joined #openstack-neutron07:53
*** fawadkhaliq has quit IRC07:53
*** nyechiel_ has quit IRC07:54
*** matrohon has joined #openstack-neutron07:55
*** alex_klimov has joined #openstack-neutron07:56
*** achanda has quit IRC07:56
*** shivrao has quit IRC07:59
*** spandhe has quit IRC07:59
*** alex_klimov has quit IRC08:01
*** alex_klimov has joined #openstack-neutron08:02
*** gberginc has joined #openstack-neutron08:02
*** nlahouti has quit IRC08:02
*** ronis has quit IRC08:02
*** nfedotov has joined #openstack-neutron08:05
*** l6unchpad has joined #openstack-neutron08:07
*** vikram has quit IRC08:09
openstackgerritenikanorov proposed openstack/neutron: Add periodic agents health check.  https://review.openstack.org/18163308:11
hanzhangwho are familiar with l2 pop codes, help to review this patch https://review.openstack.org/21546708:12
*** jistr has joined #openstack-neutron08:12
*** sdake_ has joined #openstack-neutron08:15
*** anilvenkata has quit IRC08:15
*** yamahata has quit IRC08:17
*** sdake has quit IRC08:18
*** vikram has joined #openstack-neutron08:22
*** ronis has joined #openstack-neutron08:24
*** fedexo has quit IRC08:26
*** lyanchih has quit IRC08:29
*** fawadk has quit IRC08:30
*** britthouser has quit IRC08:32
*** ihrachys has joined #openstack-neutron08:33
*** lpetrut has joined #openstack-neutron08:34
*** anilvenkata has joined #openstack-neutron08:36
*** lajos-katona1 has joined #openstack-neutron08:37
*** vikram has quit IRC08:37
*** lajos-katona has quit IRC08:38
*** lpetrut has left #openstack-neutron08:39
openstackgerritLi Yuanyuan proposed openstack/neutron: Mistake messsage for some service instance  https://review.openstack.org/21815208:39
*** dbite has quit IRC08:41
*** dguitarbite has joined #openstack-neutron08:41
*** jbonjean has joined #openstack-neutron08:43
*** lyanchih has joined #openstack-neutron08:47
*** ihrachyshka has joined #openstack-neutron08:49
*** amit213 has quit IRC08:50
*** amit213 has joined #openstack-neutron08:51
*** vikram has joined #openstack-neutron08:51
*** atuvenie has joined #openstack-neutron08:52
*** baojg has joined #openstack-neutron08:53
*** gberginc has quit IRC08:57
*** obondarev has quit IRC08:57
*** fawadkhaliq has joined #openstack-neutron08:58
*** azbiswas has joined #openstack-neutron08:58
*** akamyshnikova has quit IRC08:59
*** ilyashakhat has quit IRC08:59
*** skraynev has quit IRC09:00
*** fawadk has joined #openstack-neutron09:00
*** VikasC has joined #openstack-neutron09:01
*** oreillyd has joined #openstack-neutron09:01
*** fawadkhaliq has quit IRC09:02
*** azbiswas has quit IRC09:03
openstackgerritPraveen Kumar SM proposed openstack/neutron: Fix for bug:1489788  https://review.openstack.org/21815809:04
*** rossella_s has quit IRC09:05
*** rossella_s has joined #openstack-neutron09:06
*** y_sawai has quit IRC09:09
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports  https://review.openstack.org/21030409:10
*** lyanchih has quit IRC09:11
*** lyanchih has joined #openstack-neutron09:14
rossella_sgongysh ping...regarding https://review.openstack.org/#/c/211191/ can you have a look at my answer please?09:16
*** amotoki has quit IRC09:25
*** baojg has quit IRC09:26
*** lori|away is now known as lori09:27
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow implementation  https://review.openstack.org/15394609:31
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: Add native of_interface fullstack tests  https://review.openstack.org/21699109:31
*** claudiub has joined #openstack-neutron09:31
*** neiljerram has joined #openstack-neutron09:32
*** john-davidge has joined #openstack-neutron09:32
*** skraynev has joined #openstack-neutron09:33
*** akamyshnikova has joined #openstack-neutron09:33
*** ilyashakhat has joined #openstack-neutron09:34
*** iwamoto has quit IRC09:35
*** john-davidge has quit IRC09:37
*** john-davidge_ has joined #openstack-neutron09:37
*** lyanchih has quit IRC09:38
*** skraynev has quit IRC09:38
*** akamyshnikova has quit IRC09:39
*** ilyashakhat has quit IRC09:39
*** lajos-katona1 has quit IRC09:41
openstackgerritSridhar Gaddam proposed openstack/neutron: Add IPv6 Address Resolution protection  https://review.openstack.org/20165009:41
*** salv-orlando has quit IRC09:42
*** tflynn_ has quit IRC09:43
*** lyanchih has joined #openstack-neutron09:43
openstackgerritMerged openstack/neutron: devref: added guidelines on how to maintain sub-projects  https://review.openstack.org/21634809:44
*** tflynn has joined #openstack-neutron09:45
*** skraynev has joined #openstack-neutron09:45
*** rha has joined #openstack-neutron09:46
*** rha has joined #openstack-neutron09:46
*** akamyshnikova has joined #openstack-neutron09:48
*** alex_klimov has quit IRC09:48
*** gberginc has joined #openstack-neutron09:49
*** ushkalim has quit IRC09:49
*** Yuji has quit IRC09:49
openstackgerritshihanzhang proposed openstack/neutron-specs: Distributed DHCP Agents  https://review.openstack.org/20542909:50
*** sudipto has joined #openstack-neutron09:50
*** alex_klimov has joined #openstack-neutron09:56
*** pavel_bondar_afk has quit IRC09:56
*** dave-mccowan has joined #openstack-neutron09:58
*** lajos-katona has joined #openstack-neutron09:58
*** kiran-r has joined #openstack-neutron09:59
*** tflynn has quit IRC10:02
neiljerramMorning all.10:12
*** pc_m has joined #openstack-neutron10:13
neiljerramJust wondering - is there a systemic problem with the gate at the moment, or not?10:13
neiljerramIt appears to me that (1) there is an increasing wait time, currently up to nearly 18 hours10:13
neiljerramand (2) apparently simple changes keep getting failed by apparently unrelated dsvm-functional tests.10:14
*** gongysh has quit IRC10:14
*** annp has quit IRC10:14
*** liangbo has quit IRC10:15
neiljerramUnfortunately I don't yet have the experience to know whether this is normal or not - hence asking here.10:15
*** huazhihao has quit IRC10:15
openstackgerritZhangqi Chen proposed openstack/neutron: treat skipped updated devices as removed  https://review.openstack.org/21818810:16
openstackgerritMerged openstack/neutron: Update template for ModelMigrationSync test  https://review.openstack.org/21757310:20
*** ildikov has quit IRC10:21
*** DanyC has joined #openstack-neutron10:22
ihrachysajo: do we have strategy to get qos documented in user guides?10:22
*** DanyC has quit IRC10:22
ihrachysneiljerram: you may check with infra. and also, it is worth to refer to a failure with a link10:23
openstackgerritPraveen Kumar SM proposed openstack/neutron: Closes-Bug: #1489788  https://review.openstack.org/21815810:24
openstackbug 1489788 in neutron "typo in doc string of subnet_alloc.py" [Undecided,In progress] https://launchpad.net/bugs/1489788 - Assigned to Praveen Kumar SM (praveen-sm-kumar)10:24
*** miyagishi_t has quit IRC10:24
*** rstarr has quit IRC10:27
*** pc_m has quit IRC10:27
*** neiljerram has quit IRC10:27
*** dguitarbite has quit IRC10:27
*** jpena has quit IRC10:27
*** puck has quit IRC10:27
*** mgagne has quit IRC10:27
*** Samos123 has quit IRC10:27
*** rm_work has quit IRC10:27
*** zacksh has quit IRC10:27
*** wendar_ has quit IRC10:27
*** daneyon has quit IRC10:27
*** hakimo has quit IRC10:27
*** hanzhang has quit IRC10:27
*** adam_g has quit IRC10:27
*** aix has quit IRC10:27
*** Kennan has quit IRC10:27
*** ArchiFleKs has quit IRC10:27
*** dsneddon has quit IRC10:27
*** aveiga has quit IRC10:27
*** nihilifer has quit IRC10:27
*** jamielennox has quit IRC10:27
*** phschwartz has quit IRC10:27
*** vishwanathj has quit IRC10:27
*** stefanb has quit IRC10:27
*** sayali has quit IRC10:27
*** dr_gogeta86 has quit IRC10:27
*** marios has quit IRC10:27
*** mihero has quit IRC10:27
*** dguerri` has quit IRC10:27
*** gberginc has quit IRC10:27
*** lukasa has quit IRC10:27
*** Marga_ has quit IRC10:27
*** links has quit IRC10:27
*** comstud has quit IRC10:27
*** lbragstad has quit IRC10:27
*** sbalukoff has quit IRC10:27
*** clev-away has quit IRC10:27
*** jroll has quit IRC10:27
*** gus has quit IRC10:27
*** Nakato has quit IRC10:27
*** markvoelker has quit IRC10:27
*** afazekas has quit IRC10:27
*** adreznec has quit IRC10:27
*** raorn has quit IRC10:27
*** krotscheck has quit IRC10:27
*** anteaya has quit IRC10:27
*** harmw has quit IRC10:27
*** rstarr has joined #openstack-neutron10:29
*** gberginc has joined #openstack-neutron10:29
*** lukasa has joined #openstack-neutron10:29
*** Marga_ has joined #openstack-neutron10:29
*** links has joined #openstack-neutron10:29
*** lbragstad has joined #openstack-neutron10:29
*** comstud has joined #openstack-neutron10:29
*** sbalukoff has joined #openstack-neutron10:29
*** clev-away has joined #openstack-neutron10:29
*** jroll has joined #openstack-neutron10:29
*** gus has joined #openstack-neutron10:29
*** Nakato has joined #openstack-neutron10:29
*** markvoelker has joined #openstack-neutron10:29
*** afazekas has joined #openstack-neutron10:29
*** adreznec has joined #openstack-neutron10:29
*** raorn has joined #openstack-neutron10:29
*** krotscheck has joined #openstack-neutron10:29
*** anteaya has joined #openstack-neutron10:29
*** harmw has joined #openstack-neutron10:29
*** pc_m has joined #openstack-neutron10:30
*** neiljerram has joined #openstack-neutron10:30
*** dguitarbite has joined #openstack-neutron10:30
*** jpena has joined #openstack-neutron10:30
*** puck has joined #openstack-neutron10:30
*** mgagne has joined #openstack-neutron10:30
*** Samos123 has joined #openstack-neutron10:30
*** rm_work has joined #openstack-neutron10:30
*** zacksh has joined #openstack-neutron10:30
*** wendar_ has joined #openstack-neutron10:30
*** daneyon has joined #openstack-neutron10:30
*** hakimo has joined #openstack-neutron10:30
*** hanzhang has joined #openstack-neutron10:30
*** adam_g has joined #openstack-neutron10:30
*** aix has joined #openstack-neutron10:30
*** Kennan has joined #openstack-neutron10:30
*** ArchiFleKs has joined #openstack-neutron10:30
*** dsneddon has joined #openstack-neutron10:30
*** aveiga has joined #openstack-neutron10:30
*** nihilifer has joined #openstack-neutron10:30
*** jamielennox has joined #openstack-neutron10:30
*** vishwanathj has joined #openstack-neutron10:30
*** stefanb has joined #openstack-neutron10:30
*** sayali has joined #openstack-neutron10:30
*** dr_gogeta86 has joined #openstack-neutron10:30
*** marios has joined #openstack-neutron10:30
*** mihero has joined #openstack-neutron10:30
*** dguerri` has joined #openstack-neutron10:30
*** phschwartz has joined #openstack-neutron10:30
*** pc_m has quit IRC10:30
*** neiljerram has quit IRC10:30
*** dguitarbite has quit IRC10:30
*** jpena has quit IRC10:30
*** puck has quit IRC10:30
*** mgagne has quit IRC10:30
*** Samos123 has quit IRC10:30
*** rm_work has quit IRC10:30
*** zacksh has quit IRC10:30
*** wendar_ has quit IRC10:30
*** daneyon has quit IRC10:30
*** hakimo has quit IRC10:30
*** hanzhang has quit IRC10:30
*** adam_g has quit IRC10:30
*** aix has quit IRC10:30
*** Kennan has quit IRC10:30
*** ArchiFleKs has quit IRC10:30
*** dsneddon has quit IRC10:30
*** aveiga has quit IRC10:30
*** nihilifer has quit IRC10:30
*** jamielennox has quit IRC10:30
*** vishwanathj has quit IRC10:30
*** stefanb has quit IRC10:30
*** sayali has quit IRC10:30
*** dr_gogeta86 has quit IRC10:30
*** marios has quit IRC10:30
*** mihero has quit IRC10:30
*** dguerri` has quit IRC10:30
*** pc_m has joined #openstack-neutron10:32
*** neiljerram has joined #openstack-neutron10:32
*** dguitarbite has joined #openstack-neutron10:32
*** jpena has joined #openstack-neutron10:32
*** puck has joined #openstack-neutron10:32
*** mgagne has joined #openstack-neutron10:32
*** Samos123 has joined #openstack-neutron10:32
*** rm_work has joined #openstack-neutron10:32
*** zacksh has joined #openstack-neutron10:32
*** wendar_ has joined #openstack-neutron10:32
*** daneyon has joined #openstack-neutron10:32
*** hakimo has joined #openstack-neutron10:32
*** hanzhang has joined #openstack-neutron10:32
*** adam_g has joined #openstack-neutron10:32
*** aix has joined #openstack-neutron10:32
*** Kennan has joined #openstack-neutron10:32
*** ArchiFleKs has joined #openstack-neutron10:32
*** dsneddon has joined #openstack-neutron10:32
*** aveiga has joined #openstack-neutron10:32
*** nihilifer has joined #openstack-neutron10:32
*** jamielennox has joined #openstack-neutron10:32
*** vishwanathj has joined #openstack-neutron10:32
*** stefanb has joined #openstack-neutron10:32
*** sayali has joined #openstack-neutron10:32
*** dr_gogeta86 has joined #openstack-neutron10:32
*** marios has joined #openstack-neutron10:32
*** mihero has joined #openstack-neutron10:32
*** dguerri` has joined #openstack-neutron10:32
*** shoutm has joined #openstack-neutron10:34
*** Samos123 has quit IRC10:34
*** VikasC has quit IRC10:34
*** Samos123 has joined #openstack-neutron10:34
*** atuvenie has quit IRC10:37
openstackgerritenikanorov proposed openstack/neutron: Add flows to tunnel bridge with proper cookie.  https://review.openstack.org/21811810:40
*** zhenguo has quit IRC10:40
*** nikil89_ has joined #openstack-neutron10:41
openstackgerritPraveen Kumar SM proposed openstack/neutron: Fixed the typo in the doc string of the class SubnetPoolReader  https://review.openstack.org/21815810:41
openstackgerritRossella Sblendido proposed openstack/neutron: Don't sync all devices when some fail  https://review.openstack.org/20209810:43
*** lazy_prince has quit IRC10:47
*** Marga_ has quit IRC10:48
*** stanzgy has quit IRC10:51
openstackgerritRossella Sblendido proposed openstack/neutron: OVS agent add functional tests of OVS status  https://review.openstack.org/20544210:53
*** anilvenkata has quit IRC10:53
*** dimsum__ has joined #openstack-neutron10:53
*** apuimedo_ has joined #openstack-neutron10:55
*** numans has quit IRC10:57
*** huanxie has quit IRC10:58
ihrachyskevinbenton: https://review.openstack.org/#/c/218029/ please, it unbreaks the branch11:01
*** ildikov has joined #openstack-neutron11:06
rossella_sajo ping11:10
rossella_syamamoto ping11:12
yamamotorossella_s: pong11:12
rossella_sajo yamamoto I think we could have a problem here https://review.openstack.org/#/c/215608/ see my comment. It would break rolling upgrades I think11:13
rossella_sthe patch was already approved11:14
*** aix has quit IRC11:14
ihrachysrossella_s: should we block it then?11:14
ihrachysrossella_s: you can set Workflow-1 to do it11:15
rossella_sihrachys, thanks for the suggestion, I will do that.11:16
*** tmorin has quit IRC11:16
yamamotorossella_s: commented on review11:17
*** thorst1 has joined #openstack-neutron11:19
*** yamamoto has quit IRC11:21
*** lyanchih has quit IRC11:21
*** numans has joined #openstack-neutron11:26
*** l6unchpad has quit IRC11:26
*** SridharG has quit IRC11:27
*** devvesa has quit IRC11:37
*** apuimedo_ has quit IRC11:37
*** luqas has quit IRC11:38
*** luqas has joined #openstack-neutron11:39
*** neeti has quit IRC11:40
*** britthouser has joined #openstack-neutron11:41
*** britthou_ has joined #openstack-neutron11:42
*** britthouser has quit IRC11:46
*** l6unchpad has joined #openstack-neutron11:50
*** njohnston is now known as nate_gone11:50
*** sudipto has quit IRC11:50
*** kiran-r has quit IRC11:51
*** bharath_ has quit IRC11:51
*** apuimedo_ has joined #openstack-neutron11:51
*** julim has joined #openstack-neutron11:51
*** britthouser has joined #openstack-neutron11:52
*** devvesa has joined #openstack-neutron11:52
*** britthou_ has quit IRC11:54
*** rpothier has joined #openstack-neutron11:57
*** yamamoto has joined #openstack-neutron12:04
*** yamamoto has quit IRC12:05
*** yamamoto has joined #openstack-neutron12:05
*** vikram has quit IRC12:05
*** yamamoto_ has joined #openstack-neutron12:05
*** pradk has quit IRC12:07
*** coreycb has joined #openstack-neutron12:09
*** yamamoto has quit IRC12:10
*** yfried_ has joined #openstack-neutron12:11
*** sparkiegeek has joined #openstack-neutron12:13
coreycbhi, does the agent_type for neutron-vpn-agent make sense here?  http://paste.ubuntu.com/12213662/12:13
ajohi rossella_s, jlibosva , about https://review.openstack.org/#/c/215608/7/neutron/plugins/ml2/rpc.py let's keep it WIP for the shake of checking if we should not bump the RPC version12:13
ajoI get the point, if agent didn't subscribe to such topic, it's not strictly necessary12:14
*** ackk has joined #openstack-neutron12:14
ajosaving an RPC version not sure if buys us anything btw, but it's not *yet* an urgent patch, as long as we address it before the end of the cycle :)12:14
pc_mcoreycb: It does if you have the VPN service enabled.12:15
pc_mcoreycb: The VPN agent is derived from the L3 agent, so if you have VPN service enabled you'll see the neutron-vpn-agent (which provides both VPN and L3 capabilities).12:16
jlibosvaajo: end of the cycle or before FF?12:16
jlibosva:)12:16
ajoit's a bug, so before end of the cycle12:17
jlibosvaalright12:17
ajoand the rpc call is not triggered but for a qos policy id on the net12:17
ajoso it should be quite safe12:17
*** Kennan has quit IRC12:18
ajocrap... I twisted my foot walking down a stair today :(12:18
*** park_ has quit IRC12:18
ajohurting painfully12:18
*** lizk has quit IRC12:18
jlibosvaajo: ouch, get better soon12:18
ajostairs are dangerous ':D12:19
ajothanks jlibosva12:19
*** vikram has joined #openstack-neutron12:19
coreycbpc_m, fair enough, thanks12:19
sparkiegeekpc_m: it was surprising to see that change backported to Kilo, we had some code that was explicitly matching against binary == "neutron-l3-agent" which has broken.12:19
*** tmorin has joined #openstack-neutron12:20
sparkiegeekhttps://bugs.launchpad.net/neutron/+bug/1472494 was the bug that changed it12:20
openstackLaunchpad bug 1472494 in neutron kilo "when enable vpn agent, no vpn related infomation is shown by running 'neutron agent list' ." [Medium,Fix released] - Assigned to lee jian (leejian0612)12:20
*** fawadk has quit IRC12:21
rossigee_hi guys, i've got a Ubunto/Kilo setup here, using neutron/ml2/vlans - cirros VM starts, gets DHCP address, but struggles with metadata request - i can see it making requests on the tap interface, but nothing coming through on the qvb/qbr/qvo counterparts - pings into the maching appearing on the q* ifaces, but not arriving on the tap iface - any ideas?12:23
pc_msparkiegeek: My guess is the committer wanted it in kilo, so do it in master and then backported.12:23
*** aix has joined #openstack-neutron12:24
sparkiegeekpc_m: sure. I had (perhaps naively) considered the binary name to form part of the public API, hence my surprise at it changing in a stable release12:25
rossigee_also, in comparing it to a working setup (running havana), this setup seems to be missing 'phy-br-em2' and 'int-br-em2' when doing an 'ip l' or 'ifconfig -a' - should these still be configured on kilo, and if so where haven't i configured them :) ?12:25
*** fawadkhaliq has joined #openstack-neutron12:26
*** ihrachys has quit IRC12:27
*** ihrachys has joined #openstack-neutron12:27
*** Marga_ has joined #openstack-neutron12:29
*** fawadk has joined #openstack-neutron12:29
*** flaviof has joined #openstack-neutron12:30
*** fawadkhaliq has quit IRC12:30
*** salv-orlando has joined #openstack-neutron12:30
*** park_ has joined #openstack-neutron12:30
*** lizk has joined #openstack-neutron12:31
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994312:32
*** salv-orlando has quit IRC12:32
openstackgerritKevin Benton proposed openstack/neutron: Revert "Run py34 tests with testr"  https://review.openstack.org/21824412:33
*** Kennan has joined #openstack-neutron12:33
*** l6unchpad has quit IRC12:34
*** chlong has joined #openstack-neutron12:34
*** beagles has joined #openstack-neutron12:36
*** nyechiel_ has joined #openstack-neutron12:36
openstackgerritDaisuke Fujita proposed openstack/neutron: Output resource uuid when a resource is created  https://review.openstack.org/21824612:37
*** jpena is now known as jpena|lunch12:38
*** yamamoto_ has quit IRC12:39
*** yamamoto has joined #openstack-neutron12:39
*** yamamoto has quit IRC12:40
openstackgerritDongcan Ye proposed openstack/neutron: Fix ICMP code is not specifed validation  https://review.openstack.org/21825012:43
*** l6unchpad has joined #openstack-neutron12:43
*** nyechiel has joined #openstack-neutron12:44
*** nyechiel_ has quit IRC12:45
*** lizk has quit IRC12:45
*** park_ has quit IRC12:45
*** Kennan has quit IRC12:46
*** fawadk has quit IRC12:51
*** pavel_bondar has joined #openstack-neutron12:53
*** baoli has joined #openstack-neutron12:54
*** fawadkhaliq has joined #openstack-neutron12:55
*** y_sawai has joined #openstack-neutron12:55
*** nplanel has joined #openstack-neutron12:56
rossella_sajo for me the main point is not to spare a version, it's about getting things right, the RPC version is a complex topic, the code we merge should do the right thing and serve as example :)12:56
ajofrom the rolling upgrades point it's safe, and will work12:57
*** n00btube has joined #openstack-neutron12:57
*** lizk has joined #openstack-neutron12:57
*** park_ has joined #openstack-neutron12:57
*** y_sawai has quit IRC12:58
ajocorrect, it's correct, increasing the version won't hurt anybody.12:58
*** y_sawai has joined #openstack-neutron12:58
ajonow, the only point is... is there any other reason we didn't see which still forces us to dump the version12:58
ihrachyskevinbenton: hm. it seems we have a better fix than revert: https://review.openstack.org/#/c/217379/12:59
ajorossella_s, I guess, if it was not a fanout topic, we'd be obligated to it, but not for fanout, and selective subscription12:59
ajoand that's why it wasn't necessary for port_delete either13:00
kevinbentonihrachys: oh nice13:00
ajorossella_s, do I make sense? :)13:00
*** Kennan has joined #openstack-neutron13:00
rossella_sajo so your point is, increasing is not wrong but it's not necessary, did I get you right?13:01
*** azbiswas has joined #openstack-neutron13:01
*** baoli has quit IRC13:01
*** baoli has joined #openstack-neutron13:02
ajorossella_s, yes, "looks like it's not necessary", but I'd like to weight in kevinbenton or amotoki to see if we're missing something :)13:02
rossella_sajo agreed.13:02
kevinbentonajo: context?13:03
ajokevinbenton, sorry, link, 1 sec13:03
ajokevinbenton: see conversation by the end13:04
ajohttps://review.openstack.org/#/c/215608/7/neutron/plugins/ml2/rpc.py13:04
ajobump/nobump of version13:04
ajoand agent notification topics13:04
ajoI guess version bump is not necessary13:04
*** y_sawai has quit IRC13:04
*** amuller has joined #openstack-neutron13:04
*** anilvenkata has joined #openstack-neutron13:05
* ajo twists in pain, I'm going to bed to put my foot on a high position, I believe I sprained my ankle13:05
* ajo feels old13:05
ajoahhh, I'll better stay here, no moving13:05
*** azbiswas has quit IRC13:05
rossella_sajo get better13:06
ajothanks :']13:06
*** lyanchih has joined #openstack-neutron13:07
*** lyanchih has quit IRC13:08
*** nplanel has quit IRC13:08
kevinbentoni probably wouldn't bump the version because there is no new method to call13:09
ajokevinbenton, what do you mean13:09
ajothere is, but the old agent won't subscribe13:09
kevinbentonajo: no, i mean there isn't a method that the agent is calling on the server13:10
openstackgerritMerged openstack/neutron: devref: Add sub-project release notes  https://review.openstack.org/21772313:10
kevinbentonajo: and, like you said, the server is just sending something the agent isn't listening for13:10
kevinbentonajo: so i'm not sure the version bump really buys us anything13:10
*** aix has quit IRC13:10
ajokevinbenton , I think it doesn't , I didn't realize until rossella_s's comment13:10
ajokevinbenton, and, if we bump agent side, ... we can create an issue13:11
ajofor example13:11
ajoserver upgrades first13:11
ajoclient upgrades later...13:11
ajosorry13:11
ajoneutron-server first13:11
ajoagent later13:11
ajoif we need to modify methods we need to keep backwards compatibility to the old arguments at least for 1 cycle13:11
*** kbringard has joined #openstack-neutron13:11
rossella_sso we agree no version bump?13:12
ajoand bumping the version kills that13:12
ajorossella_s, I think we all agree, doesn't buy us anything13:12
*** SridharG has joined #openstack-neutron13:13
kevinbentonyeah13:13
*** pradk has joined #openstack-neutron13:13
*** kbyrne has quit IRC13:13
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21508613:14
*** anilvenkata has left #openstack-neutron13:16
*** annegentle has joined #openstack-neutron13:17
*** kbyrne has joined #openstack-neutron13:19
*** johnbelamaric has joined #openstack-neutron13:19
*** kbyrne has quit IRC13:20
*** kbyrne has joined #openstack-neutron13:21
ihrachysamuller: reviewed the tunneling patch for fullstack. I wonder whether all nits suggest a respin.13:22
amullerihrachys: I can if you +2 right after ;-)13:23
ihrachysamuller: I can do it, yes.13:23
*** jlibosva has quit IRC13:24
amullerihrachys: ok, coffee break then a respein13:24
ihrachysmestery: howdy!13:24
amulleror respin13:24
mesteryihrachys: howdy!13:25
mesteryamuller: Thanks for this one (https://review.openstack.org/#/c/217379/), now I see what your twitter message was about the other day :)13:25
ihrachysmestery: qos client patches are in, and we need a release to proceed with fullstack tests. should I send a release request? or how does it work now?13:25
amullermestery: I broke it, I fixed it...13:25
amullernothing to thank for13:25
mesteryamuller: Hey, however it happened, thanks for rolling yoru sleeves up and taking care of it13:25
mesteryihrachys: I was actually just today looking at that very thing, I need to understand if there is anything else we're waiting for on the client side before we do a release.13:26
mesteryI'll do that today and we'll get it sorted.13:26
*** sparkiegeek has left #openstack-neutron13:26
amullermy sleeves haven't been down for a while with how the gate is =D13:26
ihrachysmestery: nice. thanks! jlibosva should be happy.13:26
mesteryamuller: lol ... :(13:27
mesteryihrachys: ;)13:27
*** amuller is now known as amuller_afk13:27
*** breitz has joined #openstack-neutron13:28
*** links has quit IRC13:29
*** aix has joined #openstack-neutron13:29
*** lajos-katona has left #openstack-neutron13:31
HenryGihrachys: when is the global-requirements bot going to fix us?13:32
*** amotoki has joined #openstack-neutron13:32
HenryGDo you know how it works?13:33
openstackgerritNuman Siddique proposed openstack/neutron: Refactor the subnetpools API tests  https://review.openstack.org/20241913:35
*** ferest has joined #openstack-neutron13:36
*** JoeStack has quit IRC13:36
*** salv-orlando has joined #openstack-neutron13:37
*** yfried_ has quit IRC13:40
matrohonhi all13:40
openstackgerritstephen-ma proposed openstack/neutron: Resolve issue where router can't be removed from L3-agent in dvr mode  https://review.openstack.org/21751613:41
matrohonI've got a dumb question : how does launchpad know that a patch relates to a bp, and so adds the gerrit change to the whiteboard?13:41
*** park__ has joined #openstack-neutron13:42
*** dansmith is now known as superdan13:42
*** lizk_ has joined #openstack-neutron13:42
*** jpena|lunch is now known as jpena13:43
ihrachysHenryG: fix us? stable you mean?13:44
*** huanxie has joined #openstack-neutron13:44
*** Kennan2 has joined #openstack-neutron13:44
*** Kennan has quit IRC13:44
ihrachysmatrohon: you put Implements: blueprint ... in the commit message13:44
HenryGihrachys: oh, yeah, I am mixing up issues13:44
ihrachysmestery: to unbreak kilo https://review.openstack.org/#/c/218029/13:45
*** park_ has quit IRC13:45
*** dane_leblanc has joined #openstack-neutron13:45
*** lizk has quit IRC13:46
*** yamamoto has joined #openstack-neutron13:46
haleybso has anyone else noticed a lot of "DBReferenceError: (pymysql.err.IntegrityError)..." errors recently?13:47
*** nikil89_ is now known as naggappan13:47
*** gildub has quit IRC13:48
matrohonihrachys, yep, but it seems that even without this verb in the commit message, launchpad adds the 'addressed by' in the whiteboard... I don't know how, but this happened for first patches of 207371, that didn't have the "Partially-Implements" sentence...13:48
*** edmondsw has joined #openstack-neutron13:48
*** alaski is now known as lascii13:49
*** dimsum__ has quit IRC13:49
*** clev-away is now known as clev13:50
*** dimsum__ has joined #openstack-neutron13:50
ihrachysmatrohon: maybe it's enough to have it for one of the patches. maybe a topic/branch name?13:50
*** yamamoto has quit IRC13:51
*** britthouser has quit IRC13:53
matrohonihrachys, I though it was that too. I've tried to update the topic of 206536, but this didn't update the whiteboard of the launchpad related bp13:53
*** regXboi has joined #openstack-neutron13:53
regXboiamuller: gm13:54
*** carl_baldwin has joined #openstack-neutron13:54
ihrachysmatrohon: I believe it's triggered only on initial patch upload.13:54
matrohonihrachys, probably, thanks13:54
*** britthouser has joined #openstack-neutron13:56
*** fawadk has joined #openstack-neutron13:57
*** amuller_afk is now known as amuller13:57
haleybmestery: do you know if someone is chasing down any sql issues in neutron at the moment?  Put "pymysql.err.IntegrityError" in logstash and you'll see what I mean13:58
*** ankit_ag has quit IRC13:59
*** Kennan2 has quit IRC14:00
*** fawadkhaliq has quit IRC14:00
*** park__ has quit IRC14:01
*** lizk_ has quit IRC14:01
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994314:03
regXboiand while we are at it, it looks like DVR is pretty ill - I'm noticing that test_router_rescheduling has started failing in the last twelve hours14:04
*** l6unchpad has quit IRC14:04
haleybregXboi: related to the error i posted ?14:05
regXboihaleyb: don't know - pulling a job that failed and looking at the logs right now14:05
*** lyanchih has joined #openstack-neutron14:05
*** fkobzik has joined #openstack-neutron14:06
*** arajagopal has joined #openstack-neutron14:06
haleyb"Cannot add or update a child row... ON DELETE CASCADE" (not my emphasis) - that's triggering in the DVR job pretty consistently14:07
*** Piet has quit IRC14:07
regXboithat may be, but the only test that is failing is test_router_rescheduling14:07
*** fkobzik has left #openstack-neutron14:08
*** huanxie has quit IRC14:08
*** yamamoto has joined #openstack-neutron14:10
mesteryihrachys: Done! Thanks!14:11
haleybregXboi: yes, same failure, those sql errors were in my q-svc.log file14:12
mesteryhaleyb: Not that I'm aware of, sounds like you and regXboi are on this one ;)14:12
openstackgerritTerry Wilson proposed openstack/neutron: Add support for PluginWorker and Process creation notification  https://review.openstack.org/18939114:13
*** shwetaap has joined #openstack-neutron14:13
*** lizk_ has joined #openstack-neutron14:13
*** l6unchpad has joined #openstack-neutron14:13
regXboihaleyb: looking there now14:13
*** park__ has joined #openstack-neutron14:14
*** Kennan has joined #openstack-neutron14:14
*** britthouser has quit IRC14:16
*** yfried_ has joined #openstack-neutron14:17
regXboihaleyb: I'm not seing those in the failed job I'm looking at14:17
*** lori is now known as lori|away14:18
regXboihaleyb: n/m I found them14:18
haleybregXboi: ok.  I do sometimes see footprints of this one, https://bugs.launchpad.net/neutron/+bug/148957614:19
openstackLaunchpad bug 1489576 in neutron "key error in del l3_agent.pd.routers[router['id']]['subnets']" [High,In progress] - Assigned to Assaf Muller (amuller)14:19
* regXboi checking timestamps right now14:19
*** yog_ has quit IRC14:19
*** crose has joined #openstack-neutron14:21
amullerhaleyb: http://goo.gl/8xjtro14:22
amullerhaleyb: DVR failure rate is over 50%14:22
amullerI'm knocking it off the gate queue14:22
*** zhenguo has joined #openstack-neutron14:22
regXboiamuller: ack - the failure is in the test case I mentioned above (test_router_reschedule)14:22
regXboihaleyb: the integrity errors I'm seeing are not part of that test case14:23
amullerregXboi: haleyb: That failing DVR test is a new tempest test14:24
amullerhaleyb: https://review.openstack.org/#/c/209949/14:24
regXboiamuller, haleyb: it looks like it is failing on https://bugs.launchpad.net/neutron/+bug/148957614:24
openstackLaunchpad bug 1489576 in neutron "key error in del l3_agent.pd.routers[router['id']]['subnets']" [High,In progress] - Assigned to Assaf Muller (amuller)14:24
*** aix has quit IRC14:25
amullerI don't think that's the case14:25
amullerhaleyb: looks like the conditional skip is not working on that patch14:25
*** agireud has joined #openstack-neutron14:26
*** crose has quit IRC14:27
regXboiamuller: ack14:27
*** yamahata has joined #openstack-neutron14:28
haleybamuller: funny how that test is supposed to skip dvr, but it doesn't seem to be14:28
amullerhaleyb: yeah talking about it in openstack-infra14:28
amullerhaleyb: I need someone to step in quickly and fix that skip14:28
*** yamamoto has quit IRC14:28
regXboipaging armax, dougwig, mestery into -infra :)14:30
openstackgerritvikram.choudhary proposed openstack/neutron: BGP Dynamic Routing: introduce driver  https://review.openstack.org/20763514:30
*** agireud has quit IRC14:32
*** aix has joined #openstack-neutron14:38
*** ducttape_ has joined #openstack-neutron14:38
*** nfedotov has quit IRC14:38
ducttape_carl_baldwin - we are seeing this bug https://bugs.launchpad.net/neutron/+bug/1489912   If you need more info, I'm usually in the horizon room14:39
openstackLaunchpad bug 1489912 in neutron "ha dhcp agents and port mismatch kills dhcp for a tenant network" [Undecided,New]14:39
neiljerramhttps://review.openstack.org/#/c/217379/ was just failed again...14:40
*** ronis has quit IRC14:40
openstackgerritPengtao Huang proposed openstack/neutron: Replace "p" variable by "port_info"  https://review.openstack.org/21730314:41
carl_baldwinducttape_: ack14:43
*** julim has quit IRC14:43
*** ferest has quit IRC14:45
*** kbyrne has quit IRC14:45
*** vikram has quit IRC14:47
*** aix has quit IRC14:48
*** doude has quit IRC14:48
*** fitoduarte has quit IRC14:49
*** doude has joined #openstack-neutron14:49
openstackgerritAssaf Muller proposed openstack/neutron: Add amuller as an infra liason  https://review.openstack.org/21832214:50
amullermestery: ^14:50
*** lukasa has quit IRC14:52
*** lukasa has joined #openstack-neutron14:53
mesteryamuller: Looking14:53
openstackgerritMerged openstack/python-neutronclient: Update tls_container_id to tls_container_ref  https://review.openstack.org/21779214:53
*** mlavalle has joined #openstack-neutron14:53
*** mlavalle has quit IRC14:53
*** slaweq has joined #openstack-neutron14:54
*** mlavalle has joined #openstack-neutron14:54
*** jckasper has joined #openstack-neutron14:54
salv-orlandoihrachys: may I ask you to look at a 2 stable/juno patches for the vmware plugin: https://review.openstack.org/#/c/216866/ and https://review.openstack.org/#/c/216271/14:55
salv-orlandorelated master and kilo patches merged already14:55
salv-orlandoI think they meet all the backport criteria14:55
*** ihrachys_ has joined #openstack-neutron14:56
*** ihrachyshka has quit IRC14:57
*** insequent has quit IRC14:57
*** ihrachys has quit IRC14:57
*** kbyrne has joined #openstack-neutron14:57
*** mickeys has joined #openstack-neutron14:58
*** slaweq has quit IRC14:59
*** aix has joined #openstack-neutron15:00
*** vthapar has joined #openstack-neutron15:01
*** numans has quit IRC15:01
*** julim has joined #openstack-neutron15:02
*** skraynev has quit IRC15:03
*** zzzeek has joined #openstack-neutron15:03
*** jlibosva has joined #openstack-neutron15:03
*** enikanorov_ has joined #openstack-neutron15:04
*** l6unchpad has quit IRC15:05
*** skraynev has joined #openstack-neutron15:05
*** dave-mccowan has quit IRC15:06
beaglesamotoki, mestery: looking for some feedback on https://review.openstack.org/#/c/212673/ if/when you get a chance15:08
*** neophy has joined #openstack-neutron15:08
amotokibeagles: ack. look at it soon.15:09
beaglesamotoki: thx!15:09
*** ihrachyshka has joined #openstack-neutron15:09
*** jbell8 has joined #openstack-neutron15:10
openstackgerritAdelina Tuvenie proposed openstack/neutron: Fixes wrong neutron Hyper-V Agent name in constants  https://review.openstack.org/21833715:10
*** naggappan has quit IRC15:11
amotokirussellb: there is a discussion on RPC versioning for a new method in https://review.openstack.org/#/c/215608 . you have a deep experience in rpc versioning. could you look at it?15:11
russellbyes i will look15:11
amotokirussellb: thanks15:11
russellbfeel free to add me to reviews like that, it will hit my inbox15:11
*** armax has joined #openstack-neutron15:12
amotokii am afraid you have a long review queues in gerrit :-) :-(15:12
openstackgerritPengtao Huang proposed openstack/neutron: Ignore repeated values in service plugins  https://review.openstack.org/21259715:12
*** l6unchpad has joined #openstack-neutron15:13
*** boris-42 has joined #openstack-neutron15:13
*** zhenguo has quit IRC15:14
russellbamotoki: heh, fine with me15:15
*** sdake_ has quit IRC15:15
*** sdake has joined #openstack-neutron15:16
*** doude has quit IRC15:16
*** doude_ has joined #openstack-neutron15:16
*** edmondsw has quit IRC15:19
*** nate_gone is now known as njohnston15:19
*** dave-mccowan has joined #openstack-neutron15:19
*** edmondsw has joined #openstack-neutron15:20
*** TrevorV has joined #openstack-neutron15:21
*** vthapar has quit IRC15:21
*** vthapar has joined #openstack-neutron15:21
openstackgerritPengtao Huang proposed openstack/neutron: Replace "prt" variable by "port"  https://review.openstack.org/21835015:22
*** enikanorov has quit IRC15:23
*** jaypipes has joined #openstack-neutron15:23
openstackgerritAl Miller proposed openstack/neutron: Add optional file permission argument to replace_file()  https://review.openstack.org/21680715:24
*** jaypipes is now known as leakypipes15:24
*** enikanorov has joined #openstack-neutron15:25
*** bharathm has quit IRC15:25
*** apuimedo_ has quit IRC15:25
*** rossigee_ has left #openstack-neutron15:26
*** enikanorov has quit IRC15:26
*** dguitarbite has quit IRC15:26
*** yamamoto has joined #openstack-neutron15:29
*** ferest has joined #openstack-neutron15:30
*** JoeStack has joined #openstack-neutron15:31
*** jbell8 has quit IRC15:31
*** jbell8 has joined #openstack-neutron15:31
*** fawadk has quit IRC15:32
*** skraynev has quit IRC15:32
*** enikanorov has joined #openstack-neutron15:33
*** enikanorov_ has quit IRC15:33
*** akamyshnikova has quit IRC15:33
*** skraynev has joined #openstack-neutron15:33
*** yamamoto has quit IRC15:34
*** enikanorov_ has joined #openstack-neutron15:34
*** akamyshnikova has joined #openstack-neutron15:34
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Kill contention between update and delete  https://review.openstack.org/21803415:35
armaxihrachyshka, sc68cal: see what you reckon15:36
*** nyechiel has quit IRC15:36
*** jbell8_ has joined #openstack-neutron15:36
openstackgerritArmando Migliaccio proposed openstack/neutron: Testing if we can claim the bug killed once and for all  https://review.openstack.org/21803515:36
*** jbell8 has quit IRC15:36
ihrachys_armax: not_in was so clear and obvious! :( now the argument name does not imply its intent15:37
*** zzzeek has quit IRC15:37
armaxihrachys_: there’s a docstring!15:37
*** zzzeek has joined #openstack-neutron15:38
armaxihrachys_: want me to put it back?15:38
*** zzzeek has quit IRC15:38
ihrachys_armax: yeah. and before that you could understand what it's for even without reading it.15:38
*** sridhar_ram has joined #openstack-neutron15:38
ihrachys_armax: I don't know, I would prefer but don't insist15:38
armaxihrachys_: I know15:38
armaxI liked it that way to15:38
armaxbut the in=non_in15:39
armaxseemed a bit clumsy15:39
*** zzzeek has joined #openstack-neutron15:39
*** chlong has quit IRC15:39
*** akamyshnikova has quit IRC15:40
*** enikanorov_ has quit IRC15:40
ihrachys_armax: it's hard to satisfy everyone. I will step back if sc68cal thinks it's better now15:40
*** skraynev has quit IRC15:40
armaxok, let’s give him time to comment15:40
armaxihrachys_: he wasn’t complaining about the naming afaik15:40
armaxthough15:40
armaxeither way, the logic doesn’t change so we can always bike shed after the gate drill is over15:41
ihrachys_armax: yeah, but I think you changed it partially due to that. anyway, I should shut up now. :)15:41
armaxihrachys_: I am happy to respin, let’s wait for sc68cal and others to chime ijn15:42
armaxin15:42
*** ronis has joined #openstack-neutron15:47
*** akamyshnikova has joined #openstack-neutron15:48
*** SridharG has left #openstack-neutron15:48
*** enikanorov_ has joined #openstack-neutron15:49
*** skraynev has joined #openstack-neutron15:50
dougwigxgerman, armax: can we talk flavors?15:51
armaxok15:52
*** ircuser-1 has quit IRC15:52
*** enikanorov_ has quit IRC15:54
*** john-davidge_ has quit IRC15:54
*** shoutm has quit IRC15:54
*** ferest has quit IRC15:55
*** sdake_ has joined #openstack-neutron15:55
*** shoutm has joined #openstack-neutron15:55
*** nlahouti has joined #openstack-neutron15:55
*** shoutm has quit IRC15:55
*** matrohon has quit IRC15:56
*** akamyshnikova has quit IRC15:56
*** ihrachys_ is now known as ihrachys15:56
*** fitoduarte has joined #openstack-neutron15:59
*** yfried_ has quit IRC15:59
*** sdake has quit IRC15:59
*** Swami has joined #openstack-neutron16:01
*** shivrao has joined #openstack-neutron16:02
*** shivrao_ has joined #openstack-neutron16:03
*** azbiswas has joined #openstack-neutron16:03
openstackgerritSalvatore Orlando proposed openstack/neutron: Reservations: Don't count usage if resource is unlimited  https://review.openstack.org/21837516:03
xgermansure16:03
xgermandougwig, armax16:04
*** johnsom has joined #openstack-neutron16:04
*** jbell8_ has quit IRC16:04
*** jbell8 has joined #openstack-neutron16:05
*** shivrao has quit IRC16:06
*** l6unchpad has quit IRC16:06
*** shivrao_ has quit IRC16:07
*** tmorin has quit IRC16:07
*** azbiswas has quit IRC16:07
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444616:08
*** jbell8 has quit IRC16:09
*** jpena has quit IRC16:09
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445516:10
*** skraynev has quit IRC16:10
*** rotbart has quit IRC16:10
*** leakypipes has quit IRC16:11
*** vivek-ebay has joined #openstack-neutron16:11
*** bharathm has joined #openstack-neutron16:12
*** vthapar has quit IRC16:12
*** dmartls1 has joined #openstack-neutron16:13
*** kiran-r has joined #openstack-neutron16:13
*** dannywilson has joined #openstack-neutron16:17
*** alex_klimov has quit IRC16:18
*** claudiub has quit IRC16:18
*** _cjones_ has joined #openstack-neutron16:19
*** s3wong has joined #openstack-neutron16:19
Sam-I-Amamuller: ping16:20
openstackgerritMatt Riedemann proposed openstack/neutron: Remove crazy verbose debug log in _call_on_dict_driver  https://review.openstack.org/21838016:20
*** Marga_ has quit IRC16:20
*** jwarendt has joined #openstack-neutron16:21
openstackgerritMatt Riedemann proposed openstack/neutron: Remove crazy verbose debug log in _call_on_dict_driver  https://review.openstack.org/21838016:21
*** chlong has joined #openstack-neutron16:22
*** skraynev has joined #openstack-neutron16:23
*** insequent has joined #openstack-neutron16:23
*** yamamoto has joined #openstack-neutron16:24
*** kiran-r has quit IRC16:25
*** fawadkhaliq has joined #openstack-neutron16:26
*** akamyshnikova has joined #openstack-neutron16:28
amullerSam-I-Am: yo16:28
Sam-I-Amhey16:29
Sam-I-Amamuller: got a chance to review that patch?16:29
openstackgerritJames Arendt proposed openstack/neutron: Add tenant_id to flavor service profiles attributes  https://review.openstack.org/21744016:29
*** muawiakh_ has joined #openstack-neutron16:29
amullerSam-I-Am: I will now16:29
Swamicarl_baldwin: I have have addressed your review comments on that patch. #19444116:30
*** minwang2 has joined #openstack-neutron16:30
amullerSam-I-Am: only this one right? https://review.openstack.org/#/c/216462/16:30
*** aranjan has joined #openstack-neutron16:31
carl_baldwinSwami: ack16:31
Swamicarl_baldwin: thanks16:31
*** annegentle has quit IRC16:31
Sam-I-Amamuller: correct16:31
*** slaweq has joined #openstack-neutron16:31
*** lukasa has quit IRC16:32
*** britthouser has joined #openstack-neutron16:32
Sam-I-Amamuller: pretty much just applied your original comments to the linuxbridge version16:33
*** mriedem has joined #openstack-neutron16:33
Sam-I-Amand corrected some other garbage16:33
*** claudiub has joined #openstack-neutron16:33
mriedemsc68cal: mestery: carl_baldwin: can you take a look at this - seems like a no brainer https://review.openstack.org/#/c/217188/16:34
mriedemi pushed a duplicate of that this morning when looking at q-svc logs for a gate bug16:34
*** minwang2 has quit IRC16:35
openstackgerritMatt Riedemann proposed openstack/neutron: Remove ml2 resource extension success logging  https://review.openstack.org/21718816:35
*** jistr has quit IRC16:35
mriedemamuller: ^ i added the bug ref16:35
amullermriedem: thanks16:35
amullermriedem: did you get a chance to address Cedric's concerns?16:36
amullerI should really just take a look16:36
*** vivek-ebay has quit IRC16:36
mriedemno16:36
mriedemhe can fix that in a follow on patch imo16:36
*** salv-orlando has quit IRC16:36
mriedemthis impacts qa in the gate so i don't think it's worth bikeshedding on that point right now16:36
amullermriedem++16:36
mriedemmake jckasper do it16:36
jckaspermriedem: what am I doing ?16:38
mriedemfixing all the things16:38
mriedemi on the other hand am going to lunch16:38
jckasperno pressure then16:38
*** mriedem is now known as mriedem_lunch16:38
*** yamahata has quit IRC16:38
*** slaweq has quit IRC16:38
carl_baldwinmriedem_lunch: +2  I’m fine with it as is.16:38
*** annegentle has joined #openstack-neutron16:40
ihrachysmriedem_lunch: another +2. let's merge once jenkins replies16:43
*** cdl has joined #openstack-neutron16:43
*** cdl has left #openstack-neutron16:43
*** chlong has quit IRC16:45
*** jlibosva has quit IRC16:46
*** devvesa has quit IRC16:46
*** tiswanso has joined #openstack-neutron16:47
*** oreillyd has quit IRC16:49
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Kill contention between update and delete  https://review.openstack.org/21803416:50
armaxihrachyshka: ^^16:50
*** tiswanso has quit IRC16:50
armaxsc68cal: ^^16:50
*** tiswanso has joined #openstack-neutron16:51
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Kill contention between update and delete  https://review.openstack.org/21803416:51
*** pradk has quit IRC16:52
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Kill contention between update and delete  https://review.openstack.org/21803416:52
Sam-I-Amamuller: thx for the review. regarding your comment - we're trying to figure out the best way to do includes for common content16:54
amullerSam-I-Am: roger16:54
ihrachysarmax: I like it16:54
*** Guest100 has joined #openstack-neutron16:55
armaxihrachys: good, sorry it took me so long to have the revelation16:55
*** Marga_ has joined #openstack-neutron16:56
armaxihrachys: hopefully the update in place should suffice to make the threads be nice to each other16:56
*** enikanorov has quit IRC16:56
*** enikanorov has joined #openstack-neutron16:57
*** JoeStack has quit IRC17:00
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: Endpoint groups: Implements API and database  https://review.openstack.org/21269217:01
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Splitting out models from database class  https://review.openstack.org/21624817:01
*** davidjc has joined #openstack-neutron17:03
openstackgerritTerry Wilson proposed openstack/neutron: Add support for PluginWorker and Process creation notification  https://review.openstack.org/18939117:03
*** slaweq has joined #openstack-neutron17:03
*** ihrachys has quit IRC17:04
*** rossella_s has quit IRC17:05
*** julim has quit IRC17:06
*** rossella_s has joined #openstack-neutron17:06
*** abhiraut has joined #openstack-neutron17:08
*** julim has joined #openstack-neutron17:09
*** minwang2 has joined #openstack-neutron17:10
*** madhu_ak has joined #openstack-neutron17:11
*** salv-orlando has joined #openstack-neutron17:12
openstackgerritIsaku Yamahata proposed openstack/python-neutronclient: port: add --no-fixed-ip option to allow port without subnet/ip address  https://review.openstack.org/21838817:13
*** gberginc has quit IRC17:13
*** ivar-lazzaro has joined #openstack-neutron17:14
*** yamahata has joined #openstack-neutron17:14
*** shivrao has joined #openstack-neutron17:15
*** davidjc has quit IRC17:18
*** srilatha has joined #openstack-neutron17:18
*** sdake_ is now known as sdake17:19
*** abhiraut has quit IRC17:20
*** aix has quit IRC17:20
*** ihrachys has joined #openstack-neutron17:21
*** abhiraut has joined #openstack-neutron17:21
*** s3wong has quit IRC17:21
*** sridhar_ram has quit IRC17:21
*** jbell8 has joined #openstack-neutron17:22
*** njohnston is now known as nate_gone17:22
*** wendar_ is now known as wendar17:23
*** cbouch has joined #openstack-neutron17:24
cbouchIs there a core available to review this cherry-pick to kilo?  https://review.openstack.org/#/c/217682/17:25
kevinbentonyes!17:25
*** hitalia has joined #openstack-neutron17:26
*** Marga_ has quit IRC17:26
*** ivar-lazzaro has quit IRC17:27
*** ivar-lazzaro has joined #openstack-neutron17:28
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442417:28
*** johnsom has quit IRC17:28
cbouchkevinbenton:  Thank you verify much.  I was afraid of hitting a cut-off17:28
*** tflynn has joined #openstack-neutron17:29
*** azbiswas has joined #openstack-neutron17:29
*** sridhar_ram has joined #openstack-neutron17:29
*** vivek-ebay has joined #openstack-neutron17:29
*** slaweq has quit IRC17:30
*** ducttape_ has quit IRC17:30
*** sdake_ has joined #openstack-neutron17:31
*** Guest100 has quit IRC17:32
*** jbell8 has quit IRC17:32
*** changbl has quit IRC17:33
armaxamuller: I think we need a plan for unwedging the gate17:34
*** sdake has quit IRC17:34
*** nate_gone is now known as njohnston17:34
armaxamuller: we may need to stack up all changes together otherwise we’ll never get out of this mess17:34
armaxamuller: your change for py34 has failed again17:34
*** jbell8 has joined #openstack-neutron17:34
amullerarmax: the DVR fix will go in a sec17:35
armaxbut that’s tempest17:35
armax;)17:35
amullerand with the functional job disabled in the gate queue17:35
amullerI think the py34 will be able to go in17:35
armaxok17:35
amullerarmax: am I missing something?17:35
armaxwas the functional job disabled?17:36
armaxI must have missed it17:36
armaxwe still have the random failures on the api job17:36
*** ronis has quit IRC17:36
armaxbut the fix is in fwaas17:36
amulleris that fix approved?17:36
dougwigarmax: link?17:36
armaxhttps://review.openstack.org/#/c/218034/17:37
armaxamuller: only partially17:37
*** spandhe has joined #openstack-neutron17:37
*** aranjan has quit IRC17:37
*** ihrachys has quit IRC17:37
*** SourabhP has joined #openstack-neutron17:38
dougwigarmax: did that change already have a unit test that covered your revamp?17:38
armaxaye17:39
armaxI said so in the commit message17:39
armaxso you gotta believe me17:39
armax'Code being affected is covered already so no more coverage is deemed necessary.'17:39
xgermanit is now17:39
xgermandougwig17:40
dougwigarmax: i don't know what it is with your changes, but you always have answered all of my questions already and i miss it.17:40
armaxdougwig: :)17:40
*** mriedem_lunch is now known as mriedem17:40
armaxdougwig: https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/tests/unit/services/firewall/test_fwaas_plugin.py#L13917:41
armaxfor your refernece17:41
dougwigarmax: ty17:41
*** mriedem has left #openstack-neutron17:41
armaxtwo test cases that exercise my code17:41
armaxthe newly added I mean17:41
*** azbiswas has quit IRC17:41
sc68calarmax: sorry, I just got stuck on the bitwise operator because I've never seen it in the context of a sqlalchemy query17:41
amullerarmax: dougwig: going to lunch I'll sync with you after17:42
*** anshul has quit IRC17:42
armaxsc68cal: no worries, that’s how you tell SQLALchemny NOT17:42
amullerI want to start working on the functional test failures17:42
dougwigamuller: ok, enjoy, and thanks.17:42
armaxamuller: don’t eat too much17:42
kevinbentonNOT!17:42
armaxNOT!17:42
kevinbentonNOT NOT NOT NOT17:42
kevinbenton~ ~ ~ ~ ~17:42
*** amuller is now known as amuller_afk17:42
sc68calarmax: ah. I wonder why they don't use the ! operator17:42
armaxkevinbenton doing it an even numver of times is pointless17:42
armaxsc68cal: ask zzzeek ?17:43
kevinbentonsc68cal: ! is a python symbol17:43
kevinbentonsc68cal: that can't be overrided on an object IIRC17:43
kevinbentonsc68cal: one sec17:43
armaxok I could have used not_17:43
armaxsc68cal: there I said it17:43
armaxbut I didn’t want the extra import ;)17:43
*** Marga_ has joined #openstack-neutron17:44
sc68calarmax: lol ok17:44
sc68calarmax: well if I care about it so much I can push a patch _after_ we fix the real issue17:44
armaxsc68cal: that’s only fair17:44
armaxsc68cal http://docs.sqlalchemy.org/en/rel_1_0/core/sqlelement.html#sqlalchemy.sql.operators.Operators.__invert__17:45
armaxthere17:45
*** ivar-lazzaro has quit IRC17:45
kevinbentonsc68cal: when you call ~ on something17:45
kevinbentonsc68cal: it calls the magic __invert__()17:45
armaxsc68cal: please do not be mad at me17:45
sc68calarmax: nope, not mad at all17:45
kevinbentonbbl17:45
sc68calarmax: I learned something :)17:45
armaxI am lazy17:45
dougwigamuller_afk: remember, you're not done eating you're full. you're done when you hate yourself.17:45
armaxand I always get the import order wrong17:45
armaxso I was utterly scared17:46
armaxhtat I got pep8 to fail17:46
armaxand I chose the ~17:46
armaxok enough trolling…me goes and -2 some more stuff17:46
sc68callol17:46
sc68calpass the pain on17:46
*** jaypipes has joined #openstack-neutron17:46
armaxkevinbenton: any pending patch of yoyrs?17:46
armaxthat you particularly care about?17:47
*** ihrachys has joined #openstack-neutron17:48
*** tiswanso has quit IRC17:49
russellbarmax: is that question and going to -2 more stuff related?17:50
russellbit's funnier that way in my head17:51
armaxyes17:51
armaxrussellb: :)17:51
russellbteehee.17:51
*** ihrachyshka has quit IRC17:51
armaxI love pull kevinbenton’s leg17:51
*** achanda has joined #openstack-neutron17:56
*** ronis has joined #openstack-neutron17:57
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan  https://review.openstack.org/21267317:59
*** sridhar_ram1 has joined #openstack-neutron18:01
*** ivar-lazzaro has joined #openstack-neutron18:02
*** sridhar_ram has quit IRC18:02
*** Marga_ has quit IRC18:03
*** leitan has joined #openstack-neutron18:04
leitanHi guys, im wondering im somebody can lend me a hand on how public cloud providers handle their public ip space,  im wondering if they just assigng 1 to 1 a public address as a floating ip , or they use other mechansim like SRV dns records or something like that18:06
sc68calleitan: that sounds like a question for the openstack-ops mailing list18:07
openstackgerritCarl Baldwin proposed openstack/neutron: Refactor IpRouteCommand to allow using it without a device  https://review.openstack.org/21172718:08
leitansc68cal, ok ill ask there too, just wanted to get the best practice from all the networkers on the channel18:08
*** aix has joined #openstack-neutron18:14
*** ygbo has quit IRC18:15
*** muawiakh_ has quit IRC18:18
*** tiswanso has joined #openstack-neutron18:19
*** neiljerram has quit IRC18:20
openstackgerritNuman Siddique proposed openstack/neutron: Refactor the subnetpools API tests  https://review.openstack.org/20241918:21
*** dsneddon has quit IRC18:21
*** saksham has joined #openstack-neutron18:23
*** chadix has joined #openstack-neutron18:26
*** dsneddon has joined #openstack-neutron18:26
*** pradk has joined #openstack-neutron18:29
*** ihrachys has quit IRC18:29
*** amotoki has quit IRC18:31
*** Guest100 has joined #openstack-neutron18:31
*** muawiakhan has joined #openstack-neutron18:31
*** ihrachys has joined #openstack-neutron18:31
*** ihrachys has quit IRC18:32
*** lyanchih has quit IRC18:32
*** fawadkhaliq has quit IRC18:33
*** ihrachys has joined #openstack-neutron18:33
*** lyanchih has joined #openstack-neutron18:33
kevinbentonarmax: i'm back!18:34
*** insequent has quit IRC18:34
kevinbentongot pulled into an interview :)18:34
armaxkevinbenton: that’s okay18:34
kevinbentonarmax: if you want to -2 something, you could downgrade your vote on this one even further18:35
kevinbentonarmax: https://review.openstack.org/#/c/215491/18:35
armaxlet me18:35
*** Guest100 has quit IRC18:35
*** jwy has joined #openstack-neutron18:36
kevinbentonarmax: go for it. that patch is in limbo until REDACTED gets fixed18:37
armaxkevinbenton: I am still on the fence'18:37
armaxI received no answer from you yet18:37
*** muawiakhan has quit IRC18:39
*** muawiakhan has joined #openstack-neutron18:39
kevinbentonarmax: ok, answered18:40
* armax ducks18:40
*** jaypipes has quit IRC18:40
*** chadix has quit IRC18:41
*** mickeys has quit IRC18:41
*** skraynev has quit IRC18:41
*** muawiakhan has quit IRC18:41
* kevinbenton geese18:42
*** skraynev has joined #openstack-neutron18:42
*** jbell8 has quit IRC18:42
*** aranjan has joined #openstack-neutron18:43
armaxkevinbenton: there18:43
*** achanda has quit IRC18:44
*** amuller_afk is now known as amuller18:48
amullerdougwig: quoting Mr. CK are we18:48
dougwigamuller: it's so true18:48
amullerarmax: the FWaaS API tests fix affects only Neutron or other projects?18:49
armaxamuller: the former18:49
amullerok no need to ask infra to promote it then18:49
amullerarmax: are you aware of any other burning gate issues apart from the different functional job failures?18:49
armaxwell…if they promote it, why not?18:50
*** jbell8 has joined #openstack-neutron18:50
*** Guest100 has joined #openstack-neutron18:51
*** pai15 has joined #openstack-neutron18:51
*** tiswanso has quit IRC18:52
*** tiswanso has joined #openstack-neutron18:52
*** moha_hunt has joined #openstack-neutron18:53
*** Guest100 has quit IRC18:54
*** tiswanso has quit IRC18:57
*** claudiub has quit IRC18:59
openstackgerritIsaku Yamahata proposed openstack/neutron: l3_db: it updates port attribute without L2 plugin  https://review.openstack.org/20235719:01
*** jbell8 has quit IRC19:01
*** sdake_ is now known as sdake19:01
*** sridhar_ram1 has quit IRC19:02
*** saksham has quit IRC19:02
*** jbell8 has joined #openstack-neutron19:03
*** sdake_ has joined #openstack-neutron19:03
amullerarmax: I'm gonna look through the functional job failures, make sure we have elastic recheck queries on them, prioritize them, get higher visibility on them in general... raise the bug priorities when needed19:04
amullerdougwig: ^19:04
amullerarmax: dougwig: I'll also tag them with a common gate-failure tag19:04
amullerso we'll have a single launchpad queries containing all of the major offenders19:05
amullerfor any job19:05
*** ivar-laz_ has joined #openstack-neutron19:05
amullerunit, functional, api or tempest19:05
amullersounds reasonable?19:05
openstackgerritBertrand Lallau proposed openstack/neutron-fwaas: Removing unused dependency: discover  https://review.openstack.org/21842019:06
Swamicarl_baldwin: ping19:06
carl_baldwinSwami: pong19:06
Swamicarl_baldwin: quick question on the comment about the context guards.19:07
openstackgerritBertrand Lallau proposed openstack/neutron-vpnaas: Removing unused dependency: discover  https://review.openstack.org/21842219:07
*** sdake has quit IRC19:07
Swamicarl_baldwin: are you saying that I should not be using the "with" statement19:07
carl_baldwinYes19:08
*** ivar-lazzaro has quit IRC19:08
Swamicarl_baldwin: can I directly use the _make_xyz methods instead.19:08
dougwigamuller: that sounds good. Next meeting, we also need to remind folks to investigate failures, not just recheck.19:08
amullerdougwig: for sure19:08
*** dannywilson has quit IRC19:09
*** achanda has joined #openstack-neutron19:11
carl_baldwinSwami: So, “with self.subnet() as subnet1” would just become “subnet1 = self.subnet()"19:11
carl_baldwinSwami: I think that’s what you said, just restating.19:11
*** srilatha has quit IRC19:11
Swamicarl_baldwin: Directly calling the value of self.subnet resulted in error, so I will be using the _make_subnet instead will that be ok.19:12
*** sridhar_ram has joined #openstack-neutron19:12
*** srilatha has joined #openstack-neutron19:12
carl_baldwinSwami: Oh, I guess that makes sense.19:13
Swamicarl_baldwin: Ok got it.19:13
openstackgerritCarl Baldwin proposed openstack/neutron: WIP WIP Miscelaneous changes to enable address scopes in L3 agent  https://review.openstack.org/21266919:14
*** minwang2 has quit IRC19:15
*** abhiraut has quit IRC19:16
*** dave-mccowan has quit IRC19:16
*** srilatha has quit IRC19:16
*** lyanchih has quit IRC19:17
*** dannywilson has joined #openstack-neutron19:17
mesteryThanks for doing all of that work amuller, it's much appreciated :)19:23
*** slaweq has joined #openstack-neutron19:24
Sam-I-Ammestery: hows life19:24
mesterySam-I-Am: Honestly or you want me to paint it positively?19:24
Sam-I-Amhahaha19:24
Sam-I-Amjust checking in19:24
mestery:)19:24
Sam-I-Amwe work with openstack19:24
Sam-I-Amso...19:24
mesterySam-I-Am: It's going well19:24
Sam-I-Amif your life is anything like mine right now :/19:25
mestery;)19:25
Sam-I-Amthe last of the network guide scenario updates are waiting for approval, so thats good19:25
Sam-I-Amnext up - linuxbridge in the install guide19:25
Sam-I-Amand also throwing around the idea of a hybrid scenario - just drag the ext-net to all compute nodes to support both provider and project networks19:26
Sam-I-Ambut time... this thing.19:26
*** yamahata has quit IRC19:26
Sam-I-Amlooking for more contributors... tried to raise awareness at the ops meetup19:26
Sam-I-Amwe could use some well-tested configs that scale19:26
pc_mmestery: ping19:27
mesterypc_m: pong19:28
pc_mmestery: Really quick one. Can you reapprove https://review.openstack.org/#/c/216248/2 for me?19:28
mesterypc_m: I'm afraid to approve anything until the gate issues are resolved19:28
pc_mmestery: It was approved and in queue. I pushed another commit that had that as a parent commit, and it pushed a new version.19:28
*** john-davidge has joined #openstack-neutron19:29
pc_mmestery: It's totally benign - splitting model code out from database into a new module. It'll definitely pass (has passed testing several times now).19:29
pc_mmestery: But understand. Just would like to see it get through over the weekend.19:30
*** jbell8 has quit IRC19:31
amullermestery: gotta step is up a bit...19:34
mesteryamuller: +1000.519:36
*** madhu_ak has quit IRC19:37
*** changbl has joined #openstack-neutron19:37
*** devlaps has joined #openstack-neutron19:37
*** muawiakhan has joined #openstack-neutron19:38
*** vivek-ebay has quit IRC19:39
*** dmartls1 has quit IRC19:40
*** vivek-ebay has joined #openstack-neutron19:40
*** rushil has joined #openstack-neutron19:42
*** Piet has joined #openstack-neutron19:42
*** achanda has quit IRC19:43
*** nlahouti has quit IRC19:46
*** nlahouti has joined #openstack-neutron19:46
*** achanda has joined #openstack-neutron19:47
*** dave-mccowan has joined #openstack-neutron19:47
*** nlahouti has quit IRC19:48
*** fzdarsky|afk has quit IRC19:49
*** fzdarsky has quit IRC19:49
*** slaweq has quit IRC19:51
*** sdake_ is now known as sdake19:51
*** doublek has joined #openstack-neutron19:51
*** vivek-ebay has quit IRC19:51
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444119:53
*** nmagnezi has joined #openstack-neutron19:54
*** rushil has quit IRC19:54
*** rushil has joined #openstack-neutron19:54
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444119:56
*** pc_m has quit IRC20:03
*** madhu_ak has joined #openstack-neutron20:05
*** minwang2 has joined #openstack-neutron20:05
*** muawiakhan has quit IRC20:07
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444620:08
*** john-davidge has quit IRC20:08
*** john-davidge has joined #openstack-neutron20:10
*** neophy has quit IRC20:12
*** rushil has quit IRC20:13
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445520:16
*** pai15 has quit IRC20:17
Swamicarl_baldwin: I have addressed your review comments. #194441, #194446, #19445520:17
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442420:17
*** pai15 has joined #openstack-neutron20:17
*** abhiraut has joined #openstack-neutron20:19
carl_baldwinSwami: ack20:19
*** tidwellr1 has joined #openstack-neutron20:19
*** pai15 has quit IRC20:21
*** TrevorV has quit IRC20:22
*** TrevorV2 has joined #openstack-neutron20:22
*** pai15 has joined #openstack-neutron20:22
regXboikevinbenton: is there a bug for the router rescheduler failure found by the gate?20:23
*** nmagnezi has quit IRC20:25
*** arajagopal has quit IRC20:25
*** julim has quit IRC20:25
*** achanda has quit IRC20:28
Swamicarl_baldwin: thanks20:29
*** srilatha has joined #openstack-neutron20:30
regXboicarl_baldwin: I just marked https://bugs.launchpad.net/neutron/+bug/1372141 as confirmed - the logstash query shows it appearing 79 times in the last 7 days20:30
openstackLaunchpad bug 1372141 in neutron "StaleDataError while updating ml2_dvr_port_bindings" [Medium,Confirmed] - Assigned to Armando Migliaccio (armando-migliaccio)20:30
regXboiall of them from the dvr-multinode-full job20:33
*** matrohon has joined #openstack-neutron20:34
*** apuimedo_ has joined #openstack-neutron20:35
*** tidwellr1 has quit IRC20:36
*** slaweq has joined #openstack-neutron20:38
*** regXboi has quit IRC20:41
openstackgerritAssaf Muller proposed openstack/neutron: Improve logging upon failure in iptables functional tests  https://review.openstack.org/21846520:42
*** insequent has joined #openstack-neutron20:43
amullerarmax: dougwig: mestery: ^20:44
mesteryamuller: looking20:45
amullerthis is required by mriedem for an elastic recheck query20:45
* amuller is moving on to the next functional test failure classification20:45
*** nlahouti has joined #openstack-neutron20:47
*** vivek-ebay has joined #openstack-neutron20:49
*** boden has joined #openstack-neutron20:49
*** krtaylor has quit IRC20:51
*** saksham has joined #openstack-neutron20:52
*** mickeys has joined #openstack-neutron20:52
*** TrevorV2 has quit IRC20:52
*** amuller has quit IRC20:53
*** jaypipes has joined #openstack-neutron20:55
*** ivar-laz_ has quit IRC20:56
*** matrohon has quit IRC20:58
*** amuller has joined #openstack-neutron20:58
*** SourabhP has quit IRC21:01
*** SourabhP has joined #openstack-neutron21:01
openstackgerritArmando Migliaccio proposed openstack/neutron: Revert "Add support for unaddressed port"  https://review.openstack.org/21847021:03
armaxkevinbenton: ^^^21:03
armaxmestery: ^^^21:03
armaxdougwig: ^^^21:03
*** leitan has quit IRC21:03
*** krtaylor has joined #openstack-neutron21:03
mesteryarmax: I wet myself reading your revert message.21:04
armaxmestery: because you got aroused?21:04
*** _cjones_ has quit IRC21:04
armaxor because you were laughing too hard?21:04
armaxor because you got scared?21:04
* armax is confused21:04
mesteryarmax: It was a mix of all three21:04
armaxok, so all of the above?21:05
mesteryYup21:05
mestery1/3 of each21:05
*** amuller has quit IRC21:05
*** rossella_s has quit IRC21:05
*** amuller has joined #openstack-neutron21:05
*** rossella_s has joined #openstack-neutron21:06
armaxplease, no-one look at http://status.openstack.org/zuul/21:06
* mestery weeps21:07
armax217379,6 is top of the queue and we have job left21:07
*** mickeys has quit IRC21:07
armaxone job left21:07
armaxmestery: you’re watching aren’t you?21:07
*** amuller has quit IRC21:07
*** amuller_ has joined #openstack-neutron21:07
mesteryAye captain21:07
armaxyou jinks it if you do21:07
armaxdon't21:07
armax!21:07
armax*jinxs21:07
dougwigi don't think i've ever seen the gate > check.  awesome.21:07
*** amuller_ is now known as amuller21:07
mesterydougwig: Take a pic or it didn't happen21:07
amullerfrickin comcast disconnecting me every sec21:08
openstackgerritMerged openstack/neutron: Fix py34 No sql_connection parameter is established error  https://review.openstack.org/21737921:08
amullersc68cal: ^21:08
amullersc68cal: your fault!21:08
armaxmestery: now you should wet yourself21:08
mesteryamuller: lol21:08
armaxwe’re 1/3 back in business21:08
mesteryarmax: It's really wet over here indeed21:08
armaxyou’re sitting on puddle?21:09
armaxok…too much21:09
*** azbiswas has joined #openstack-neutron21:09
*** njohnston is now known as nate_gone21:09
openstackgerritCarl Baldwin proposed openstack/neutron: Move gateway processing out of init_router_port  https://review.openstack.org/21847121:10
kevinbentonarmax: holy crap that was a bit over the top21:12
armaxkevinbenton: well21:13
armaxkevinbenton: I think amuller topped us all up when he made that joke at the summit21:13
armaxso we’re good21:13
armaxbut I’ll get it down a notch21:13
armaxmestery started though21:13
mesterylol21:13
mesteryEveryone always blames me21:14
mesteryI'm used to it21:14
openstackgerritCarl Baldwin proposed openstack/neutron: Create a routing table manager  https://review.openstack.org/19203221:14
armaxeveryone blames the PTL21:14
armaxdon’t take it personal21:14
*** n00btube has quit IRC21:14
mesteryI never do21:14
armaxgood21:15
mesteryHow do you think I've survived as PTL this long?21:15
mesteryHint:21:15
armaxfair point21:15
mesteryIron underwear21:15
*** apuimedo has quit IRC21:15
*** sthillma has joined #openstack-neutron21:15
mesteryIt's flame retardant21:15
armaxon this note…I am gonna get something to eat…even though mestery made me lost the appetite21:16
mesteryrofl21:16
*** apuimedo_ has quit IRC21:16
armaxI can’t take that picture out of my head21:16
kevinbentonarmax: come to mountain view!21:16
armaxoh the shivers…21:16
armaxif I go out now I melt21:16
armaxkevinbenton: the best I can do is roll over down the stairs21:17
openstackgerritKevin Benton proposed openstack/neutron: Better message on allowed address pairs error  https://review.openstack.org/21847221:17
mesteryrofl21:17
*** shwetaap has quit IRC21:18
*** shwetaap has joined #openstack-neutron21:18
kevinbentonarmax: weak21:19
*** mickeys has joined #openstack-neutron21:22
*** neophy has joined #openstack-neutron21:24
*** SourabhP has quit IRC21:30
*** ihrachys has quit IRC21:30
*** l6unchpad has joined #openstack-neutron21:30
*** johnbelamaric has quit IRC21:32
openstackgerritCarl Baldwin proposed openstack/neutron: Refactor IpRouteCommand to allow using it without a device  https://review.openstack.org/21172721:32
openstackgerritCarl Baldwin proposed openstack/neutron: Move gateway processing out of init_router_port  https://review.openstack.org/21847121:32
openstackgerritCarl Baldwin proposed openstack/neutron: Make ip rule comparison more robust  https://review.openstack.org/21847821:32
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Enhancements to IpRouteCommand  https://review.openstack.org/21847921:32
*** sridhar_ram1 has joined #openstack-neutron21:32
*** s3wong has joined #openstack-neutron21:33
*** sridhar_ram has quit IRC21:35
openstackgerritCarl Baldwin proposed openstack/neutron: Add address scope to ports in RPC response to L3 agent  https://review.openstack.org/18994821:36
*** clev is now known as clev-away21:37
*** SourabhP has joined #openstack-neutron21:38
*** mickeys has quit IRC21:38
openstackgerritCarl Baldwin proposed openstack/neutron: Add address scope to floating IPs in RPC response to L3 agent  https://review.openstack.org/18950021:38
*** doublek has quit IRC21:38
*** cbouch has quit IRC21:38
*** mickeys has joined #openstack-neutron21:38
*** dimsum__ has quit IRC21:38
*** cbouch has joined #openstack-neutron21:39
openstackgerritCarl Baldwin proposed openstack/neutron: Create a routing table manager  https://review.openstack.org/19203221:39
*** john-davidge has quit IRC21:43
*** flaviof has quit IRC21:44
*** flaviof has joined #openstack-neutron21:45
*** annegentle has quit IRC21:47
*** jaypipes has quit IRC21:47
*** annegentle has joined #openstack-neutron21:47
*** mwagner_lap has joined #openstack-neutron21:47
*** achanda has joined #openstack-neutron21:48
*** jaypipes has joined #openstack-neutron21:49
*** tiswanso has joined #openstack-neutron21:51
*** s3wong has quit IRC21:51
*** dane_leblanc has quit IRC21:52
openstackgerritKevin Benton proposed openstack/neutron: Better message on allowed address pairs error  https://review.openstack.org/21847221:52
*** jaypipes has quit IRC21:53
*** lascii is now known as alaski21:53
*** jaypipes has joined #openstack-neutron21:53
*** SourabhP has quit IRC21:55
openstackgerritMerged openstack/neutron-fwaas: Switch to using os-testr's copy of subunit2html  https://review.openstack.org/21741121:56
*** SourabhP has joined #openstack-neutron21:56
*** boden has quit IRC21:56
*** dimsum__ has joined #openstack-neutron21:56
*** dims_ has joined #openstack-neutron22:00
*** abhiraut has quit IRC22:00
*** SourabhP has quit IRC22:01
*** abhiraut has joined #openstack-neutron22:02
openstackgerritCarl Baldwin proposed openstack/neutron: Create a routing table manager  https://review.openstack.org/19203222:02
openstackgerritCarl Baldwin proposed openstack/neutron: Add address scope to ports in RPC response to L3 agent  https://review.openstack.org/18994822:02
*** dimsum__ has quit IRC22:02
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Enhancements to IpRouteCommand  https://review.openstack.org/21847922:02
openstackgerritCarl Baldwin proposed openstack/neutron: WIP WIP Miscelaneous changes to enable address scopes in L3 agent  https://review.openstack.org/21266922:02
*** azbiswas has quit IRC22:02
*** vivek-ebay has quit IRC22:05
*** vivek-ebay has joined #openstack-neutron22:05
*** l6unchpad has quit IRC22:05
*** SourabhP has joined #openstack-neutron22:05
*** pai15 has quit IRC22:06
*** pai15 has joined #openstack-neutron22:07
*** thorst1 has quit IRC22:08
*** jaypipes has quit IRC22:08
*** SourabhP has quit IRC22:08
*** cbouch has quit IRC22:09
*** pai15 has quit IRC22:11
*** mickeys has quit IRC22:14
*** mickeys has joined #openstack-neutron22:14
*** ivar-lazzaro has joined #openstack-neutron22:17
*** edmondsw has quit IRC22:17
*** carl_baldwin has quit IRC22:18
*** ronis has quit IRC22:18
*** tiswanso has quit IRC22:20
*** tiswanso has joined #openstack-neutron22:20
*** ivar-lazzaro has quit IRC22:21
*** sridhar_ram1 has quit IRC22:27
*** zzzeek has quit IRC22:27
*** abhiraut has quit IRC22:28
*** emagana has joined #openstack-neutron22:28
*** abhiraut has joined #openstack-neutron22:31
*** abhiraut has quit IRC22:32
*** hitalia has quit IRC22:33
amullerkevinbenton: fyi http://goo.gl/iQBROu22:34
*** abhiraut has joined #openstack-neutron22:34
kevinbentonamuller: my mom told me not to click on shortened urls22:34
amullerkevinbenton: 10 failures since the 25th on the new OVS agent restart functional test22:34
armaxkevinbenton: go ahead the link is genuine22:35
amuller(going through http://status.openstack.org//elastic-recheck/data/uncategorized.html)22:35
kevinbentonamuller: might be the issue that they already have a patch for22:35
amullerlink?22:35
kevinbentonamuller: hang on, i just remember seeing they missed a case for new flows22:36
kevinbentonamuller: but it might not be related to that ovs test22:36
amullerkevinbenton: are you talking about the DVR flow?22:36
amullerkevinbenton: I remember that patch, I was looking for it for another reason and couldn't find it22:37
kevinbentonamuller: it was this one https://review.openstack.org/#/c/218118/22:37
amullerahh that's an l2pop fix22:38
amullerkevinbenton: well, off to report a new bug22:38
*** sridhar_ram has joined #openstack-neutron22:38
kevinbentonamuller: ack22:41
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378122:42
*** johnbelamaric has joined #openstack-neutron22:43
*** mickeys has quit IRC22:46
ajmillerdougwig armax Could I please beg for review on https://review.openstack.org/#/c/216807/22:51
ajmillerIt is needed by a VPNaaS patch I am working on.22:51
* armax looks22:51
*** kbringard has quit IRC22:53
amullerkevinbenton: the remaining offenders are the arp spoofing OVS tests22:53
amullerkevinbenton: I noticed you sent a patch today to improve a message there22:53
amullerkevinbenton: did you report a bug on those failures?22:53
*** minwang2 has quit IRC22:54
armaxajmiller: question22:54
armaxwhy can’t reuse the original file’s permissions?22:55
armaxit sounds the most effective fix22:55
ajmillerIt the case that I'm dealing with, it is a new file.22:56
armaxbut then why are you calling replace_file?22:56
armaxI am confused22:56
ajmillerSome background might help.22:57
*** rook has quit IRC22:57
armaxajmiller: that’s ok22:57
armaxperhaps the method name is not very apt22:57
ajmillerThe file being written here contains a VPNaaS pre-shared key.  When the VPN connection is established, it is written into this file.22:57
*** rook has joined #openstack-neutron22:58
ajmillerLater, the file may be updated whent he PSK is changed22:58
*** yamahata has joined #openstack-neutron22:58
*** mickeys has joined #openstack-neutron22:58
ajmillerFor whatever reason, VPNaaS uses replace_file for this.22:59
*** johnbelamaric has quit IRC22:59
*** minwang2 has joined #openstack-neutron22:59
ajmillerAnd replace_file hard-coded permissions as 644, but we really don't want this private key data to be widely readable.22:59
armaxajmiller: ok, I approved the change but I’ll watch the gate to go back to a much more reasonable lenght23:00
armaxajmiller: when it does, I’ll push your change in23:00
*** jamielennox is now known as jamielennox|away23:00
ajmillerarmax, OK, thanks!23:01
ajmillerThe gate does seem to be flowing better....23:01
armaxajmiller: at 139 jobs in the gate queue I beg to differ23:02
openstackgerritArmando Migliaccio proposed openstack/neutron: Fix misnomer on network attribute  https://review.openstack.org/21850823:03
*** nate_gone has quit IRC23:03
*** nate_gone has joined #openstack-neutron23:03
*** nate_gone is now known as njohnston23:03
ajmillerOh, I agree that its got a ways to go before things are back to normal....23:03
*** clev-away is now known as clev23:03
*** annegentle has quit IRC23:04
*** dannywilson has quit IRC23:05
*** shwetaap has quit IRC23:06
*** rpothier has quit IRC23:07
*** baoli has quit IRC23:10
openstackgerritMerged openstack/neutron: Add Geneve type driver support to ML2  https://review.openstack.org/18794523:12
*** johnbelamaric has joined #openstack-neutron23:13
openstackgerritMerged openstack/neutron-vpnaas: VPNaaS: Use new service name for devstack plugin  https://review.openstack.org/21325323:13
*** tiswanso has quit IRC23:15
*** tiswanso has joined #openstack-neutron23:15
spandheHi folks., my talk got accepted at the Tokyo summit.. Just wondering, is it possible to present the talk remotely?23:15
*** tiswanso has quit IRC23:19
*** clev is now known as clev-away23:21
openstackgerritKevin Benton proposed openstack/neutron: Set kernel gc_thresh values to higher config value  https://review.openstack.org/21851223:21
*** david-lyle has quit IRC23:24
*** tiswanso has joined #openstack-neutron23:25
*** vishwanathj has quit IRC23:28
*** rook has quit IRC23:38
*** amuller has quit IRC23:40
openstackgerritKevin Benton proposed openstack/neutron: [WIP] - Process user iptables rules before INVALID  https://review.openstack.org/21851723:43
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21508623:44
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/21717223:44
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/21508923:45
*** sridhar_ram has quit IRC23:45
*** salv-orlando has quit IRC23:45
*** ajmiller has quit IRC23:46
*** sridhar_ram has joined #openstack-neutron23:46
*** shivrao has quit IRC23:47
*** saksham has quit IRC23:47
*** rook has joined #openstack-neutron23:48
*** emagana has quit IRC23:51
*** emagana has joined #openstack-neutron23:51
*** dimsum__ has joined #openstack-neutron23:52
*** dims_ has quit IRC23:55
*** tflynn_ has joined #openstack-neutron23:55
*** emagana has quit IRC23:56
*** Swami has quit IRC23:57
*** vivek-ebay has quit IRC23:58
*** tflynn has quit IRC23:59
*** vivek-ebay has joined #openstack-neutron23:59

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