Thursday, 2015-02-12

*** emagana has quit IRC00:02
*** markvoelker has quit IRC00:03
*** dims__ has joined #openstack-neutron00:04
watanabe_isaoGood morning to ALL.:)00:07
*** trad511 has quit IRC00:07
*** apuimedo has quit IRC00:08
*** baoli has quit IRC00:08
*** emagana has joined #openstack-neutron00:08
*** sambetts has quit IRC00:08
*** dane_leblanc has quit IRC00:10
*** shwetaap has quit IRC00:10
*** emagana has quit IRC00:12
*** SridharRamaswamy has joined #openstack-neutron00:14
*** kickinz1 is now known as kickinz1|afk00:15
*** kickinz1|afk is now known as kickinz100:16
*** achanda has quit IRC00:16
*** afazekas has quit IRC00:16
*** dsundarraj has joined #openstack-neutron00:18
*** aranjan_ has quit IRC00:21
*** tflynn has quit IRC00:21
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113000:22
*** aranjan has joined #openstack-neutron00:24
*** tsg has quit IRC00:24
*** emagana has joined #openstack-neutron00:26
*** banix has quit IRC00:27
*** r-daneel has joined #openstack-neutron00:28
*** tomoe_ has joined #openstack-neutron00:28
*** aranjan has quit IRC00:29
*** trad511 has joined #openstack-neutron00:29
*** RichardR2seley has joined #openstack-neutron00:29
RichardR2seleyI am experiencing a weird issue whereby when I create a new network and launch instances manually, those instances can communicate outbound (to the rest of my internal network and the world at large) without any issues (as is expected). When I do the same with Heat my instances cannot communicate outbound whatsoever unless they have a floating IP assigned to them.00:29
*** ijw has quit IRC00:30
*** Longgeek has quit IRC00:30
RichardR2seleyWhen I compare the properties of the routers, networks, subnets, etc. they look identical. Is there a specific parameter of a network which allows this kind of NATing by default?00:30
*** claudiub has joined #openstack-neutron00:31
*** tsg has joined #openstack-neutron00:33
*** banix has joined #openstack-neutron00:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc  https://review.openstack.org/15501300:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager  https://review.openstack.org/15496900:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge  https://review.openstack.org/15501400:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from external_process  https://review.openstack.org/15495900:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess  https://review.openstack.org/15510100:35
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager  https://review.openstack.org/15498000:35
*** sballe__ has quit IRC00:37
*** trad511 has quit IRC00:39
*** avozza is now known as zz_avozza00:39
*** suresh12 has quit IRC00:40
*** suresh12 has joined #openstack-neutron00:41
*** leenheer has quit IRC00:43
*** dsneddon has joined #openstack-neutron00:45
*** kickinz1 is now known as kickinz1|afk00:45
*** erkules has quit IRC00:46
*** csoukup has joined #openstack-neutron00:48
*** leunam112 has joined #openstack-neutron00:50
*** aranjan has joined #openstack-neutron00:50
*** markvoelker has joined #openstack-neutron00:52
*** leunam112 has quit IRC00:52
*** sbfox has quit IRC00:53
*** erkules has joined #openstack-neutron00:53
*** tflynn has joined #openstack-neutron00:54
*** banix has quit IRC00:55
*** tflynn_ has joined #openstack-neutron00:56
*** markvoelker has quit IRC00:57
*** dims__ has quit IRC00:58
*** markvoelker has joined #openstack-neutron00:59
*** tflynn has quit IRC00:59
*** tflynn_ has quit IRC01:01
*** banix has joined #openstack-neutron01:01
*** manishg has quit IRC01:01
*** RichardR2seley has quit IRC01:02
*** manishg has joined #openstack-neutron01:02
*** RichardR1seley has quit IRC01:02
*** ChuckC has quit IRC01:02
*** ChuckC has joined #openstack-neutron01:03
*** markvoelker has quit IRC01:04
*** mlavalle has quit IRC01:05
openstackgerritMohammad Banikazemi proposed openstack/neutron: Combining sec groups member update RPC calls  https://review.openstack.org/15514501:05
*** zhhuabj has joined #openstack-neutron01:06
*** signed8bit is now known as signed8bit_ZZZzz01:06
*** manishg has quit IRC01:06
*** lnxnut has joined #openstack-neutron01:06
*** tsg has quit IRC01:07
*** xgerman has quit IRC01:08
*** sbfox has joined #openstack-neutron01:09
*** manishg has joined #openstack-neutron01:09
*** signed8bit_ZZZzz is now known as signed8bit01:11
*** signed8bit is now known as signed8bit_ZZZzz01:11
*** leenheer has joined #openstack-neutron01:14
*** nplanel_ has joined #openstack-neutron01:14
*** zhhuabj has quit IRC01:14
*** achanda has joined #openstack-neutron01:16
*** manishg has quit IRC01:17
*** achanda has quit IRC01:18
openstackgerrityalei wang proposed openstack/neutron-fwaas: Implementation of Mcafee NGFW Driver  https://review.openstack.org/15209301:19
*** achanda has joined #openstack-neutron01:19
*** dsundarraj has quit IRC01:20
*** dsundarraj has joined #openstack-neutron01:20
*** sbfox has quit IRC01:21
*** dane_leblanc has joined #openstack-neutron01:22
*** popw has joined #openstack-neutron01:22
*** manishg has joined #openstack-neutron01:22
armaxotherwiseguy: ping01:24
otherwiseguyarmax: pong01:24
armaxI think we need to back out your change01:25
armaxas it breaks the world :)01:25
otherwiseguyarmax: which change?01:25
armaxhttps://review.openstack.org/#/c/153699/01:25
*** banix has quit IRC01:25
otherwiseguyuh oh. :) what part of the world did it break?01:26
*** Aish has quit IRC01:26
armaxall the projects that use neutron as dependency01:26
openstackgerritArmando Migliaccio proposed openstack/neutron: Revert "monkey patch stdlib before importing other modules"  https://review.openstack.org/15515201:26
armaxotherwiseguy: ^^^01:26
otherwiseguySo things like the *aaS?01:27
armaxotherwiseguy: I think we need to work out a change that flags NO_EVENTLET the other way01:27
*** amotoki has joined #openstack-neutron01:27
armaxyes01:27
armaxHenryG:  ^^^01:27
armaxdougwig: ^^^01:27
openstackgerritMerged openstack/neutron: Move NCS mech driver to its new home  https://review.openstack.org/15189301:27
*** amotoki has quit IRC01:28
*** zhhuabj has joined #openstack-neutron01:28
otherwiseguyarmax: how is it breaking them? just their pep8 tests?01:28
armaxotherwiseguy: as well as UT's01:28
armaxhave a look at http://status.openstack.org/zuul/01:29
armaxand search 152093,11 for instance01:29
*** manishg has quit IRC01:29
armaxotherwiseguy: if we added setenv = NO_EVENTLET=1 to the affected tests it would do the trick01:30
armaxbut that’s too much of a change, I guess01:30
armaxis there any chance we can flip the logic?01:30
HenryGI'm here01:30
HenryGback it out or fix the breakages?01:30
armaxHenryG: back up until min 2501:30
dougwigshouldn't that fix have gone into main for the server binaries?01:31
*** banix has joined #openstack-neutron01:31
otherwiseguyarmax: If we plan on removing all of the individual monkey_patches like Ihar's patch, it seems like inverting the logic would be even more problematic.01:31
otherwiseguySeems like it should be possible to fix the external things. Trying to find log file that makes sense to me.01:32
*** banix has quit IRC01:32
armaxotherwiseguy: I’d rather not sprinkle change everywhere if possible01:33
*** zhhuabj has quit IRC01:33
otherwiseguyarmax: is this representative of the errors? https://jenkins06.openstack.org/job/gate-neutron-fwaas-pep8/41/console01:33
*** manishg has joined #openstack-neutron01:34
armaxwhat about introducing your change so that  we set setenv = EVENTLET the oppositve way?01:34
armaxotherwiseguy: yes01:34
armaxthat’s an example01:34
armaxlike we do:01:34
armaxif os.environ.get(‘EVENTLET’) and apply on the tests?01:35
armaxon testenv01:35
dougwigit seems like we do more with neutron *without* eventlet than with, so that does seem backwards.  double-pain because it's in __init__ and happens as magic.01:35
dougwignot you armax, the merged change.01:36
armaxthis way the change fix your issue, but preserve sanity to the other projects...01:36
armaxand we can slowly phase in the flag to other projects in due course01:36
otherwiseguyarmax: I don't understand how flipping the logic would work. All agents, etc. would need EVENTLET environment variable set.01:36
armaxotherwiseguy: now you do:01:37
armaxhttps://github.com/openstack/neutron/blob/master/tox.ini#L5201:37
armaxwhat I am saying is do ‘positive logic'01:37
armaxand add EVENTLET=1 to testenv01:38
armaxhttps://github.com/openstack/neutron/blob/master/tox.ini#L601:38
armaxhere https://github.com/openstack/neutron/blob/master/neutron/__init__.py#L1901:38
armaxyou’d do:01:38
armaxif os.environ.get(‘EVENTLET’):01:38
armaxthe effect would be the same, would it?01:38
otherwiseguyarmax: right, but __init__.py checks for the variable before doing the monkey_patch, and the plan was to remove all of the monkey_patch() calls from the rest of the tree. Can't do that with positive logic or it would break absolutely everything.01:39
otherwiseguyUnless I'm missing something.01:39
dougwigwhy is that the plan?  doesn't that mean that everything using neutron as a lib is doing more than it should?  and we do use neutron as a lib without eventlet a lot.01:39
armaxwell, the binaries would need to flip the switch too01:39
*** markvoelker has joined #openstack-neutron01:40
armaxright, dougwig’s point is the crux of the matter, to some extent01:40
otherwiseguyThe "real fix" is to move all binary mains() to run from the same place, like cmd/ and do cmd/__init__.py monkey patching like in Nova.01:40
dougwigthat seems reasonable, but this stopgap does not.  :)01:40
armaxdougwig: which stopgap?01:41
otherwiseguyMy code that relies on that change hasn't been pushed, so just revert.01:41
dougwigarmax: of putting any of this into neutron/__init__.py01:41
armaxdougwig: agreed01:42
*** manishg has quit IRC01:42
otherwiseguyThis whole using processutils thing is really more trouble than it is worth.01:42
armaxso, otherwiseguy: you think it’s sensible to revert now, and wait for the other fix you’re proposing?01:42
dougwigmight get close in the short-term by putting it in init for agent/ and server/, though.01:42
otherwiseguyarmax: yeah, that's fine. The processutils patch is the only thing that really exposes the issue this tries to fix, and I can probably rework things to not even use processutils.01:43
armaxwould that address your concern for the tests?01:43
armaxI mean, I want revert so that we don’t cause grief to other projects01:43
otherwiseguyarmax: that works fine for me.01:43
armaxbut at the same time I want to apply the fix that nails the issue you’re experiencing01:44
otherwiseguySorry to break the world. :)01:44
*** tomoe_ has quit IRC01:44
dougwigotherwiseguy: it's a big club.  :)01:44
armaxotherwiseguy: I knew when I +2 the change something was going to come up01:44
armax:)01:44
dougwigotherwiseguy: i think i have a permanent cot there, some days.01:44
otherwiseguyI've got about 10+ patches removing the root_helper arg from everywhere that I'm sure will break things as well.01:44
otherwiseguyI've been trying hard to avoid it, with the ip_lib change, etc.01:45
armaxotherwiseguy: that’s fine, the sooner you file them the sooner we fix breakages01:45
otherwiseguyBut it touches *so much code* I would be surprised if something doesn't break.01:45
*** zhhuabj has joined #openstack-neutron01:45
* HenryG is running out of popcorn01:45
armaxso if we’re okay on the action plan, https://review.openstack.org/#/c/155152/ can be approved?01:45
*** flaviof has quit IRC01:45
otherwiseguyworks for me.01:45
*** stanzgy has joined #openstack-neutron01:46
*** ChuckC_ has joined #openstack-neutron01:46
armaxok, as for your fix01:46
*** r-daneel has quit IRC01:46
armaxotherwiseguy: are you going to take care of it, or shall someone else do?01:46
HenryGotherwiseguy: I think the root_helper fixes are very low risk01:46
*** watanabe_isao has quit IRC01:47
*** trad511 has joined #openstack-neutron01:47
HenryGarmax: revert review imminent?01:48
dougwigit's linked about 8 lines up01:48
otherwiseguy$ git grep "\(self.root_helper\|root_helper=\)"|wc -l01:48
otherwiseguy     46701:48
armaxHenryG: https://review.openstack.org/#/c/155152/01:48
*** _cjones_ has quit IRC01:48
armaxdougwig rage-merged it01:48
armax:)01:48
* HenryG cleans glasses01:48
* armax rofl01:49
*** ChuckC has quit IRC01:49
otherwiseguySo far, I have that root_helper count down to 184.01:50
otherwiseguyAnd it is making me sleepy.01:50
armaxotherwiseguy: oh my01:50
dougwigwow.01:50
armaxotherwiseguy: so as for the monkey patching01:50
*** csoukup has quit IRC01:50
otherwiseguyarmax: I can probably work something up tomorrow to try.01:50
armaxotherwiseguy: ok, so the approach would be two-fold, just so that I got it straight,...01:51
HenryGotherwiseguy: If you want I can try to finish up the remaining 18401:51
otherwiseguyarmax: Either that or scuttle the processutils.execute conversion, and just add a run_as_root option to our existing utils.execute.01:51
armaxotherwiseguy: a) move the mains in one place and do the monkeypatching early01:51
armaxand b) rework your processutils patch to work around the problem?01:52
*** xuhanp has joined #openstack-neutron01:52
*** aranjan has quit IRC01:52
otherwiseguyHenryG: I don't mind finishing it up. Getting pretty close. unit tests are the biggest pain.01:52
armaxotherwiseguy having said that your change did address some log traces observed in l3 agent01:52
otherwiseguyarmax: s/and/or/01:52
armaxotherwiseguy: that’s why I said ‘and'01:53
otherwiseguyIf I do a), there won't need to be a b).01:54
armaxotherwiseguy: ic01:54
armaxyour change did fix https://bugs.launchpad.net/neutron/+bug/141738601:55
openstackLaunchpad bug 1417386 in neutron "AttributeError: _oslo_messaging_localcontext errors found in neutron l3-agent logs" [High,Confirmed] - Assigned to venkata anil (anil-venkata)01:55
armaxso I’d be definitely eager to see something else up01:55
otherwiseguySo we should re-fix it soon, then, I guess. I wouldn't be broken-hearted if someone else figured out the move-everything-to-cmd thing while I was asleep... :)01:56
otherwiseguyI could probably send an email to Ihar, he wakes up 8 hours before me.01:56
armaxotherwiseguy: but how would that fix the issue during test execution?01:56
otherwiseguyarmax: We'd also do tests/__init__.py01:57
armaxotherwiseguy: gotcha01:57
armaxotherwiseguy: I’ll post something on his review01:57
armaxand link this conversation01:57
*** tflynn has joined #openstack-neutron01:57
otherwiseguyHe's the one that started this monkey_patching fixing business anyway. :)01:57
otherwiseguyAnd his patch wouldn't have broken the world (it was originally just tests/__init__.py, but I was also getting some tempest failures with processutils.execute without other early monkey_patching)01:58
dougwigwhoever came up with monkey patch as a term deserves a medal.01:59
*** trad511 has quit IRC01:59
otherwiseguydougwig: whoever came up with it as a concept deserves a swift kick in the pants.01:59
armaxotherwiseguy: understood, I posted a bunch of comments on https://review.openstack.org/#/c/153779/01:59
dougwigotherwiseguy: lol.02:00
otherwiseguyarmax: thanks.02:00
armaxotherwiseguy, dougwig: I honestly believe that this eventlet monkey business is more trouble than it’s worth02:00
*** SridharRamaswamy has quit IRC02:00
dougwigi was talking about how cool the term 'monkey patch' sounds.  it does lead to too much magic and side effects, in practice.02:01
otherwiseguydougwig: I say that, but my native OVSDB patch pretty much monkey_patches the OVS python library for what seems like a good reason...02:01
*** flwang has quit IRC02:01
*** tflynn has quit IRC02:01
dougwigi monkey patch the crappy ssl lib all the time.  and ruby makes good use of it for meta-programming a lot.02:01
armaxmaybe it’s just me then02:02
otherwiseguydougwig: Here is the evil: https://review.openstack.org/#/c/152761/6/neutron/agent/ovsdb/native/patch_idl.py02:02
otherwiseguyarmax: I would be very happy to use a much more explicit concurrency solution.02:02
HenryGmock is monkey patching on steroids. It has its uses.02:02
armaxotherwiseguy: you just made my day02:03
*** r-daneel has joined #openstack-neutron02:03
aleksandr_nullHi guys =)02:03
dougwigotherwiseguy: i think i need a cigarette after reading that comment.02:03
otherwiseguyI'd even rather use threads natively.02:03
otherwiseguydougwig. :)02:03
dougwigand i don't smoke.02:03
dougwig:)02:03
otherwiseguyIt is...not really a nice thing to do. And it is confusing as hell.02:04
otherwiseguyBut turning O(n) into O(1) is worth a little evil.02:04
*** shashankhegde has quit IRC02:04
otherwiseguyimho02:04
dougwigdepends on where that loop is, but sometimes yes.02:04
armaxdougwig: I can see both sides of the argument02:05
*** sbfox has joined #openstack-neutron02:05
*** achanda has quit IRC02:06
*** claudiub has quit IRC02:06
flwang1xuhanp: ping02:06
otherwiseguydougwig: true. and with this it's useful on really heavily loaded systems with many, many ports, etc. But yeah. I plan on actually proposing changes to the OVS library itself at some point when I can find the time. (along with getting the library put in PyPI).02:06
xuhanpflwang1, pong02:06
armaxok folks, I’ll back later, thanks a lot02:07
flwang1xuhanp: did you ever try route move? from one controller to another?02:07
flwang1i mean from on l3 agent to another02:07
otherwiseguyarmax: thanks for cleaning up my mess. ;)02:07
*** watanabe_isao has joined #openstack-neutron02:07
armaxotherwiseguy: our mess02:07
xuhanpflwang1, by which command or API?02:07
dougwigarmax: +102:07
otherwiseguyIt was always kind of a cross-your-fingers kind of change.02:08
*** zzzeek has quit IRC02:08
flwang1l3-agent-router-remove l3-agent-router-add02:08
xuhanpoh. router move, not route move02:08
xuhanpyes, but a while ago02:08
*** signed8bit_ZZZzz is now known as signed8bit02:11
*** achanda has joined #openstack-neutron02:12
*** shashankhegde has joined #openstack-neutron02:14
*** Longgeek has joined #openstack-neutron02:15
*** salv-orlando has quit IRC02:15
*** sbfox has quit IRC02:18
*** banix has joined #openstack-neutron02:19
*** MasterPiece has quit IRC02:19
openstackgerritXu Han Peng proposed openstack/neutron: Fix dist_fip_count problem during router migration  https://review.openstack.org/15115302:21
*** signed8b_ has joined #openstack-neutron02:22
*** signed8bit has quit IRC02:24
*** signed8b_ is now known as signed8bit02:25
*** pal has joined #openstack-neutron02:26
*** MasterPiece has joined #openstack-neutron02:28
*** sbfox has joined #openstack-neutron02:28
*** sambetts has joined #openstack-neutron02:29
*** MasterPiece| has joined #openstack-neutron02:29
*** armax has quit IRC02:29
*** zzzeek has joined #openstack-neutron02:30
*** dims__ has joined #openstack-neutron02:32
*** MasterPiece has quit IRC02:33
*** erkules_ has joined #openstack-neutron02:36
*** erkules has quit IRC02:38
*** leunam112 has joined #openstack-neutron02:39
*** tomoe_ has joined #openstack-neutron02:40
*** zzzeek has quit IRC02:40
*** leunam112 has quit IRC02:42
*** obondarev_ has joined #openstack-neutron02:43
*** sbfox has quit IRC02:44
*** xek_ has joined #openstack-neutron02:45
*** dsundarraj has quit IRC02:46
*** cgoncalv1s has joined #openstack-neutron02:47
*** Longgeek has quit IRC02:48
*** shakamunyi has joined #openstack-neutron02:49
*** fouxm_ has joined #openstack-neutron02:50
*** rodrigod` has joined #openstack-neutron02:50
*** shashankhegde has quit IRC02:52
*** nlahouti has quit IRC02:52
*** fouxm has quit IRC02:52
*** chrisf has quit IRC02:52
*** bjornar has quit IRC02:52
*** xek has quit IRC02:52
*** rwsu has quit IRC02:52
*** obondarev has quit IRC02:52
*** cgoncalves has quit IRC02:52
*** rodrigods has quit IRC02:52
*** chrisf_ has joined #openstack-neutron02:52
*** emagana has quit IRC02:53
*** Aish has joined #openstack-neutron02:53
*** suresh12 has quit IRC02:54
*** emagana has joined #openstack-neutron02:54
*** Longgeek has joined #openstack-neutron02:55
openstackgerritXu Han Peng proposed openstack/neutron: Not creating HA router when no enough l3 agents  https://review.openstack.org/15437402:57
*** cl__ has quit IRC02:57
*** rwsu has joined #openstack-neutron02:57
*** jckasper has joined #openstack-neutron02:57
*** bjornar has joined #openstack-neutron02:58
*** emagana has quit IRC02:58
*** david-lyle is now known as david-lyle_afk02:59
*** suresh12 has joined #openstack-neutron03:00
*** zzzeek has joined #openstack-neutron03:04
*** manishg has joined #openstack-neutron03:06
*** yamahata has quit IRC03:06
*** RichardRaseley has joined #openstack-neutron03:07
*** RichardR1seley has joined #openstack-neutron03:08
*** manishg_ has joined #openstack-neutron03:09
RichardR1seleyexit03:10
*** RichardR1seley has left #openstack-neutron03:10
*** manishg has quit IRC03:10
*** manishg has joined #openstack-neutron03:11
*** Marga_ has quit IRC03:12
*** manishg_ has quit IRC03:13
*** manishg_ has joined #openstack-neutron03:14
*** manishg__ has joined #openstack-neutron03:15
*** manishg has quit IRC03:15
*** salv-orlando has joined #openstack-neutron03:16
*** manishg has joined #openstack-neutron03:16
*** shashankhegde has joined #openstack-neutron03:17
*** zzzeek has quit IRC03:17
*** manishg_ has quit IRC03:18
*** manishg__ has quit IRC03:20
*** salv-orlando has quit IRC03:21
*** manishg has quit IRC03:21
*** MasterPiece| has quit IRC03:21
*** lazy_prince is now known as killer_prince03:21
*** doude has quit IRC03:23
*** doude has joined #openstack-neutron03:26
*** EmilienM is now known as EmilienM|afk03:28
*** aranjan has joined #openstack-neutron03:29
*** evgenyf has joined #openstack-neutron03:30
*** evgenyf has quit IRC03:34
*** salv-orlando has joined #openstack-neutron03:36
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fix FIP agent gw port delete based on external net  https://review.openstack.org/14585903:37
*** dims__ has quit IRC03:37
*** flwang has joined #openstack-neutron03:38
*** suresh12 has quit IRC03:40
*** RichardRaseley has quit IRC03:40
openstackgerritTimothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config  https://review.openstack.org/15373303:40
*** salv-orlando has quit IRC03:40
*** salv-orlando has joined #openstack-neutron03:41
*** shz has quit IRC03:43
*** shz has joined #openstack-neutron03:43
*** xgerman has joined #openstack-neutron03:47
*** carlp has quit IRC03:49
*** sthillma has joined #openstack-neutron03:50
*** MasterPiece has joined #openstack-neutron03:51
*** yamahata has joined #openstack-neutron03:52
*** salv-orlando has quit IRC03:57
*** shashankhegde has quit IRC03:57
*** Mr0rris0 has quit IRC03:58
*** flwang has quit IRC03:59
*** achanda has quit IRC03:59
*** salv-orlando has joined #openstack-neutron04:01
*** achanda has joined #openstack-neutron04:02
*** armax has joined #openstack-neutron04:04
*** spandhe has quit IRC04:05
*** salv-orlando has quit IRC04:05
*** harlowja is now known as harlowja_away04:08
*** achanda has quit IRC04:10
*** Longgeek has quit IRC04:10
*** Marga_ has joined #openstack-neutron04:16
*** Marga_ has quit IRC04:17
*** tvardeman has joined #openstack-neutron04:17
*** Marga_ has joined #openstack-neutron04:17
*** doude has quit IRC04:18
*** killer_prince is now known as lazy_prince04:19
*** doude has joined #openstack-neutron04:20
*** Longgeek has joined #openstack-neutron04:25
*** Longgeek has quit IRC04:26
*** vivek-ebay has quit IRC04:27
*** tiswanso has quit IRC04:27
*** haramaki has joined #openstack-neutron04:28
*** xuhanp has quit IRC04:30
*** doude has quit IRC04:30
*** doude has joined #openstack-neutron04:30
*** lazy_prince has quit IRC04:34
*** barra204 has joined #openstack-neutron04:35
*** xgerman has quit IRC04:36
*** salv-orlando has joined #openstack-neutron04:37
*** ivar-lazzaro has quit IRC04:38
*** MasterPiece| has joined #openstack-neutron04:38
*** shakamunyi has quit IRC04:39
*** anish_ has joined #openstack-neutron04:39
*** MasterPiece has quit IRC04:40
*** salv-orlando has quit IRC04:41
*** lazy_prif has joined #openstack-neutron04:42
*** lazy_prif is now known as lazy_prince04:42
*** xuhanp has joined #openstack-neutron04:43
*** doude has quit IRC04:50
*** suresh12 has joined #openstack-neutron04:51
*** doude has joined #openstack-neutron04:52
*** barra204 has quit IRC04:53
*** r-daneel has quit IRC04:54
*** suresh12 has quit IRC04:55
*** tflynn has joined #openstack-neutron04:57
*** nlahouti has joined #openstack-neutron04:59
*** salv-orlando has joined #openstack-neutron04:59
*** nlahouti1 has joined #openstack-neutron05:00
*** tflynn has quit IRC05:01
*** tvardeman has quit IRC05:02
*** salv-orlando has quit IRC05:02
*** yatin has joined #openstack-neutron05:03
*** nlahouti1 has quit IRC05:03
*** nlahouti1 has joined #openstack-neutron05:03
*** nlahouti has quit IRC05:04
*** salv-orlando has joined #openstack-neutron05:05
*** barra204 has joined #openstack-neutron05:07
*** signed8b_ has joined #openstack-neutron05:08
*** trinaths has joined #openstack-neutron05:08
*** signed8bit has quit IRC05:08
*** anilvenkata has joined #openstack-neutron05:08
*** rotbeard has quit IRC05:08
*** bobmel_ has joined #openstack-neutron05:11
*** salv-orlando has quit IRC05:12
*** cipcosma has joined #openstack-neutron05:13
*** yshafrir has joined #openstack-neutron05:13
*** bobmel has quit IRC05:14
*** RichardRaseley has joined #openstack-neutron05:14
*** banix has quit IRC05:15
*** dsundarraj has joined #openstack-neutron05:15
*** salv-orlando has joined #openstack-neutron05:15
*** xuhanp has quit IRC05:16
*** aranjan_ has joined #openstack-neutron05:17
*** signed8b_ has quit IRC05:18
*** mwagner_lap has quit IRC05:19
*** Aish has left #openstack-neutron05:19
*** RichardRaseley has quit IRC05:19
*** coolsvap has joined #openstack-neutron05:19
*** coolsvap has quit IRC05:21
*** aranjan has quit IRC05:21
*** dsundarraj has quit IRC05:23
*** coolsvap has joined #openstack-neutron05:23
*** ajo has joined #openstack-neutron05:24
*** amotoki has joined #openstack-neutron05:25
*** tflynn has joined #openstack-neutron05:26
*** tflynn has joined #openstack-neutron05:26
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent  https://review.openstack.org/15518805:28
*** RichardRaseley has joined #openstack-neutron05:28
*** salv-orlando has quit IRC05:30
*** xuhanp has joined #openstack-neutron05:33
*** dsundarraj has joined #openstack-neutron05:35
*** chrisf_ has quit IRC05:36
*** chrisf has joined #openstack-neutron05:36
*** salv-orlando has joined #openstack-neutron05:40
*** anshul has joined #openstack-neutron05:41
*** anshul has quit IRC05:41
*** anshul has joined #openstack-neutron05:42
*** yfried_ has quit IRC05:44
*** leenheer has quit IRC05:44
*** carl_baldwin has joined #openstack-neutron05:46
*** MasterPiece| has quit IRC05:51
*** salv-orlando has quit IRC05:52
*** achanda has joined #openstack-neutron05:52
*** Murali has joined #openstack-neutron05:54
*** RichardRaseley has quit IRC05:54
*** achanda has quit IRC05:56
*** dsundarraj has quit IRC05:58
*** irenab has joined #openstack-neutron06:00
*** carl_baldwin has quit IRC06:04
*** Longgeek has joined #openstack-neutron06:07
*** achanda has joined #openstack-neutron06:08
*** salv-orlando has joined #openstack-neutron06:08
*** crc32 has joined #openstack-neutron06:15
*** salv-orlando has quit IRC06:15
*** salv-orlando has joined #openstack-neutron06:16
*** leunam112 has joined #openstack-neutron06:17
*** numan has joined #openstack-neutron06:18
*** salv-orlando has quit IRC06:18
*** yfauser has joined #openstack-neutron06:19
*** yfauser has left #openstack-neutron06:20
*** leunam112 has quit IRC06:21
*** salv-orlando has joined #openstack-neutron06:21
*** cipcosma has quit IRC06:24
*** salv-orlando has quit IRC06:25
*** rotbeard has joined #openstack-neutron06:25
crc32http://pastebin.com/cSKFNLvC06:26
crc32maby I'm late but  is neutron off master broken? I can't install on a freshly built machine.06:26
irenabtrinaths: hi06:28
*** boris-42 has joined #openstack-neutron06:33
*** zz_avozza is now known as avozza06:34
crc32ImportError: neutron.hooks.setup_hook <-- is no one else getting this when trying to "python setup.py install" off neutron master?06:34
crc32I'm new to the whole pbr bork fix bork thing. Any one have any ideas how to move past this?06:35
*** barra204 has quit IRC06:36
*** gildub_ has quit IRC06:36
*** trinaths has quit IRC06:36
blogancrc32: https://review.openstack.org/#/c/155152/06:37
blogancrc32: that fixes it, its still in check and needs to go through the gate06:37
*** trinaths has joined #openstack-neutron06:38
*** dims__ has joined #openstack-neutron06:38
crc32I don't think its just other projects that don't play nicely. neutron flat out won't install period.06:39
*** mwagner_lap has joined #openstack-neutron06:42
*** dims__ has quit IRC06:42
crc32ok blogan that did it.. neutron installs now.06:45
*** SridharG has joined #openstack-neutron06:46
*** anilvenkata1 has quit IRC06:46
*** anilvenkata1 has joined #openstack-neutron06:47
*** nplanel_ has quit IRC06:48
*** salv-orlando has joined #openstack-neutron06:49
*** marios has joined #openstack-neutron06:50
*** yfried_ has joined #openstack-neutron06:50
*** yfried__ has joined #openstack-neutron06:53
*** yfried_ has quit IRC06:54
*** yamamoto_ has joined #openstack-neutron06:56
*** salv-orlando has quit IRC06:56
*** lazy_prince has quit IRC06:57
*** thomasem has quit IRC06:57
*** killer_prince has joined #openstack-neutron06:57
*** killer_prince is now known as lazy_prince06:57
*** oomichi_ has joined #openstack-neutron06:58
*** flwang has joined #openstack-neutron07:01
*** thomasem has joined #openstack-neutron07:02
*** haramaki has quit IRC07:03
*** haramaki_ has joined #openstack-neutron07:03
*** dsundarraj has joined #openstack-neutron07:05
*** afazekas has joined #openstack-neutron07:06
*** dsundarraj has quit IRC07:09
*** shashankhegde has joined #openstack-neutron07:10
*** scheuran has joined #openstack-neutron07:12
*** yarker has quit IRC07:12
*** apuimedo has joined #openstack-neutron07:14
*** shashankhegde has quit IRC07:17
openstackgerritJohn Schwarz proposed openstack/neutron: Move common test helpers to neutron.test.common  https://review.openstack.org/14677007:18
*** yarker has joined #openstack-neutron07:18
*** Marga_ has quit IRC07:20
*** aranjan_ has quit IRC07:20
*** irenab has quit IRC07:24
*** tflynn has quit IRC07:26
*** yfried__ is now known as yfried|afk07:26
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command  https://review.openstack.org/15394607:27
*** yfried|afk is now known as yfried__07:29
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc  https://review.openstack.org/15501307:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager  https://review.openstack.org/15496907:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge  https://review.openstack.org/15501407:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from external_process  https://review.openstack.org/15495907:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove deprecated DEFAULT.root_helper  https://review.openstack.org/15306607:31
openstackgerritTerry Wilson proposed openstack/neutron: Use processutils.execute for process execution  https://review.openstack.org/15306707:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent  https://review.openstack.org/15518807:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess  https://review.openstack.org/15510107:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg for ovs_lib  https://review.openstack.org/15491407:31
openstackgerritTerry Wilson proposed openstack/neutron: Stop using passed root_helper in ip_lib  https://review.openstack.org/14582907:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager  https://review.openstack.org/15498007:31
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent  https://review.openstack.org/15521207:31
*** achanda has quit IRC07:31
*** avozza is now known as zz_avozza07:33
*** leunam112 has joined #openstack-neutron07:34
*** ronis has joined #openstack-neutron07:34
*** Miouge has joined #openstack-neutron07:35
*** kawa2014 has joined #openstack-neutron07:36
openstackgerritMatthew Thode proposed openstack/neutron: Fixes bug in interface handling of ip_lib.py  https://review.openstack.org/15412807:37
*** achanda has joined #openstack-neutron07:37
*** RichardRaseley has joined #openstack-neutron07:39
*** ggnel is now known as 6A4ABF1CZ07:41
*** ggnel has joined #openstack-neutron07:41
*** vthapar has joined #openstack-neutron07:42
*** killer_prince has joined #openstack-neutron07:42
*** lazy_prince has quit IRC07:43
*** killer_prince is now known as lazy_prince07:43
*** RichardRaseley has quit IRC07:44
*** kickinz1|afk is now known as kickinz107:44
*** xek_ has quit IRC07:48
*** xek_ has joined #openstack-neutron07:48
*** anish_ has quit IRC07:48
*** anish_ has joined #openstack-neutron07:48
*** anilvenkata has quit IRC07:48
*** anilvenkata has joined #openstack-neutron07:48
*** bobmel_ has quit IRC07:48
*** bobmel_ has joined #openstack-neutron07:48
*** yshafrir has quit IRC07:48
*** yshafrir has joined #openstack-neutron07:48
*** coolsvap has quit IRC07:48
*** coolsvap has joined #openstack-neutron07:48
*** anshul has quit IRC07:48
*** anshul has joined #openstack-neutron07:48
*** numan has quit IRC07:48
*** numan has joined #openstack-neutron07:48
*** boris-42 has quit IRC07:49
*** boris-42 has joined #openstack-neutron07:49
*** SridharG has quit IRC07:49
*** SridharG has joined #openstack-neutron07:49
*** anilvenkata1 has quit IRC07:49
*** anilvenkata1 has joined #openstack-neutron07:49
*** yfried__ has quit IRC07:49
*** yfried__ has joined #openstack-neutron07:49
*** afazekas has quit IRC07:49
*** afazekas has joined #openstack-neutron07:49
*** scheuran has quit IRC07:49
*** scheuran has joined #openstack-neutron07:49
*** Longgeek has quit IRC07:53
*** Longgeek has joined #openstack-neutron07:53
*** Longgeek has quit IRC07:53
*** doude has quit IRC07:54
*** doude has joined #openstack-neutron07:55
*** irenab has joined #openstack-neutron07:58
*** Mr0rris0 has joined #openstack-neutron08:01
openstackgerritJamie Lennox proposed openstack/neutron: Make nova notifier work with sessions  https://review.openstack.org/14236608:02
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command  https://review.openstack.org/15394608:03
*** Miouge_ has joined #openstack-neutron08:05
*** pask81 has joined #openstack-neutron08:06
*** Miouge has quit IRC08:06
*** Miouge_ is now known as Miouge08:06
*** nmagnezi has joined #openstack-neutron08:07
*** itzikb has joined #openstack-neutron08:07
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ofagent: Fix port_dead  https://review.openstack.org/15433208:09
*** pask81 has quit IRC08:10
*** salv-orlando has joined #openstack-neutron08:11
*** flwang1 has quit IRC08:11
*** ygbo has joined #openstack-neutron08:12
*** flwang1 has joined #openstack-neutron08:14
*** moshele has joined #openstack-neutron08:16
*** karimb has joined #openstack-neutron08:18
*** pask81 has joined #openstack-neutron08:21
*** openstackgerrit has quit IRC08:21
*** openstackgerrit has joined #openstack-neutron08:22
*** luqas has joined #openstack-neutron08:24
openstackgerritRossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update  https://review.openstack.org/15522308:24
*** pask81 has quit IRC08:25
anish_ilyashakhat, hi around?08:31
*** achanda has quit IRC08:35
*** pask81 has joined #openstack-neutron08:36
*** nyechiel has joined #openstack-neutron08:37
*** jlanoux has joined #openstack-neutron08:39
*** RichardR1seley has joined #openstack-neutron08:40
*** zz_avozza is now known as avozza08:41
*** chlong has quit IRC08:42
watanabe_isaoexit08:42
*** watanabe_isao has quit IRC08:43
*** RichardR1seley has quit IRC08:44
*** devvesa has joined #openstack-neutron08:46
*** yamamoto_ has quit IRC08:46
*** matrohon has joined #openstack-neutron08:46
*** pal has quit IRC08:46
*** jlibosva has joined #openstack-neutron08:48
*** roeyc has joined #openstack-neutron08:53
*** Longgeek has joined #openstack-neutron08:54
*** jp_at_hp has joined #openstack-neutron08:54
*** jistr|off is now known as jistr08:54
*** jpena has joined #openstack-neutron08:54
*** pal has joined #openstack-neutron08:55
openstackgerritSridhar Gaddam proposed openstack/neutron: Default route missing for IPv6 subnets in HA Router  https://review.openstack.org/15078708:59
*** flwang has quit IRC08:59
*** evgenyf has joined #openstack-neutron09:00
*** safchain has joined #openstack-neutron09:02
*** tomoe__ has joined #openstack-neutron09:03
*** hjensas has joined #openstack-neutron09:04
*** mathias_1 has joined #openstack-neutron09:05
hjensasI am not being successful finding good docs for VPNaaS. I am looking for details regarding how it is scheduled. Can it be deployed in a resilient way?09:05
*** tomoe_ has quit IRC09:05
hjensas^^ bacround is research for bussiness usecase.09:06
*** jpich has joined #openstack-neutron09:06
*** Longgeek has quit IRC09:07
*** nyechiel_ has joined #openstack-neutron09:07
openstackgerritwatanabe isao proposed openstack/neutron: Validate string length at API level  https://review.openstack.org/14566009:08
*** jistr has quit IRC09:08
*** mathias__ has quit IRC09:08
*** nyechiel has quit IRC09:09
*** popw has quit IRC09:11
*** jistr has joined #openstack-neutron09:11
*** enikanorov_ has quit IRC09:19
*** enikanorov_ has joined #openstack-neutron09:20
*** stanzgy has quit IRC09:21
*** jckasper has quit IRC09:22
*** yamamoto has joined #openstack-neutron09:22
*** juice has quit IRC09:22
*** amotoki has quit IRC09:23
*** juice has joined #openstack-neutron09:24
*** yamamoto has quit IRC09:27
*** a_le has quit IRC09:28
*** amotoki has joined #openstack-neutron09:28
*** haramaki_ has quit IRC09:30
*** rotbeard has quit IRC09:30
*** zhhuabj has quit IRC09:30
*** scheuran_ has joined #openstack-neutron09:31
*** scheuran has quit IRC09:32
*** yamamoto has joined #openstack-neutron09:33
*** armax has quit IRC09:33
*** aepifanov has joined #openstack-neutron09:36
*** Miouge has quit IRC09:37
*** Miouge has joined #openstack-neutron09:43
*** nikil89_ has joined #openstack-neutron09:44
*** nikil89_ is now known as naggappan09:44
*** yamamoto has quit IRC09:46
*** Miouge has quit IRC09:48
*** flwang has joined #openstack-neutron09:49
*** enikanorov_ has quit IRC09:51
*** enikanorov_ has joined #openstack-neutron09:51
*** flwang has quit IRC09:54
*** enikanorov__ has joined #openstack-neutron09:56
*** hichihara has quit IRC09:56
*** enikanorov_ has quit IRC09:57
*** aepifanov has quit IRC09:58
*** yamamoto has joined #openstack-neutron10:00
*** Miouge has joined #openstack-neutron10:00
*** avozza is now known as zz_avozza10:02
*** vthapar_ has joined #openstack-neutron10:02
*** nyechiel__ has joined #openstack-neutron10:03
*** nyechiel_ has quit IRC10:03
*** nyechiel__ has quit IRC10:03
*** nyechiel__ has joined #openstack-neutron10:04
*** vthapar has quit IRC10:04
*** Longgeek has joined #openstack-neutron10:04
*** vthapar has joined #openstack-neutron10:06
*** vthapar_ has quit IRC10:07
*** xuhanp has quit IRC10:11
*** trinaths has left #openstack-neutron10:17
*** aepifanov has joined #openstack-neutron10:17
*** zz_avozza is now known as avozza10:21
*** yamamoto has quit IRC10:21
ilyashakhatanish_: i'm online10:22
anish_ilyashakhat, hi10:26
*** RichardR1seley has joined #openstack-neutron10:29
*** tomoe__ has quit IRC10:32
*** RichardR1seley has quit IRC10:34
*** moshele has quit IRC10:35
*** Longgeek has quit IRC10:38
*** hanzhang has joined #openstack-neutron10:42
*** dsundarraj has joined #openstack-neutron10:42
*** Longgeek has joined #openstack-neutron10:44
*** shz has quit IRC10:44
*** yfried__ is now known as yfried|afk10:44
*** yfried|afk is now known as yfried__10:45
*** marios has quit IRC10:46
*** dsundarraj has quit IRC10:46
*** bjornar_ has joined #openstack-neutron10:47
*** leunam112 has quit IRC10:47
*** ihrachyshka has joined #openstack-neutron10:51
*** trinaths has joined #openstack-neutron10:52
*** salv-orlando has quit IRC10:53
*** yfried__ is now known as yfried|afk10:55
*** marios has joined #openstack-neutron10:56
*** pal has quit IRC10:56
*** leunam112 has joined #openstack-neutron10:57
*** yamahata has quit IRC10:59
*** pal has joined #openstack-neutron11:03
*** rotbeard has joined #openstack-neutron11:11
*** moshele has joined #openstack-neutron11:12
*** yfried|afk is now known as yfried__11:15
*** dims__ has joined #openstack-neutron11:15
*** mathias__ has joined #openstack-neutron11:20
*** anilvenkata1 has left #openstack-neutron11:22
*** roeyc has quit IRC11:22
*** mathias_1 has quit IRC11:23
*** inara has quit IRC11:27
*** salv-orlando has joined #openstack-neutron11:27
*** trinaths has left #openstack-neutron11:29
*** yuanying has quit IRC11:30
*** inara has joined #openstack-neutron11:33
*** markvoelker has quit IRC11:34
*** claudiub has joined #openstack-neutron11:38
*** pal has quit IRC11:41
*** JasonPlank has quit IRC11:44
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439111:44
*** irenab has quit IRC11:44
*** rotbeard has quit IRC11:44
*** mathias_1 has joined #openstack-neutron11:46
*** rotbeard has joined #openstack-neutron11:47
*** rodrigod` is now known as rodrigods11:49
*** mathias__ has quit IRC11:49
*** mathias_1 has quit IRC11:50
*** mathias__ has joined #openstack-neutron11:51
*** absubram has quit IRC11:52
*** mathias_1 has joined #openstack-neutron11:56
*** mathias__ has quit IRC11:57
*** amotoki has quit IRC11:59
*** aix has quit IRC12:01
*** Dark_Matt3r has joined #openstack-neutron12:03
*** MasterPiece has joined #openstack-neutron12:05
*** pc_m has joined #openstack-neutron12:06
*** moshele has quit IRC12:07
*** moshele has joined #openstack-neutron12:10
*** MasterPiece has quit IRC12:12
*** johnbelamaric has quit IRC12:15
*** RichardR1seley has joined #openstack-neutron12:17
*** karimb has quit IRC12:20
*** roeyc has joined #openstack-neutron12:21
*** MasterPiece has joined #openstack-neutron12:22
*** RichardR1seley has quit IRC12:22
*** pavel_bondar has joined #openstack-neutron12:22
*** mathias__ has joined #openstack-neutron12:26
*** salv-orlando has quit IRC12:29
*** mathias_1 has quit IRC12:30
*** markvoelker has joined #openstack-neutron12:35
*** tellesnobrega_ has joined #openstack-neutron12:36
*** pal has joined #openstack-neutron12:36
*** dims_ has joined #openstack-neutron12:37
*** dims__ has quit IRC12:38
*** markvoelker has quit IRC12:40
*** yfried__ is now known as yfried|afk12:41
*** yfried|afk is now known as yfried__12:44
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add sanity_check for keepalived  https://review.openstack.org/15128412:45
openstackgerritIhar Hrachyshka proposed openstack/neutron: sanity_check: introduce magic to easily add new version checks  https://review.openstack.org/15507012:45
*** ollivier has joined #openstack-neutron12:46
*** doude has quit IRC12:52
*** doude has joined #openstack-neutron12:53
*** dims_ has quit IRC12:57
*** aix has joined #openstack-neutron12:57
*** tellesnobrega_ has quit IRC12:58
*** markvoelker has joined #openstack-neutron12:59
*** marrusl has joined #openstack-neutron13:00
*** irenab has joined #openstack-neutron13:02
*** amuller has joined #openstack-neutron13:02
*** pask81 has quit IRC13:02
*** jckasper has joined #openstack-neutron13:02
*** baoli has joined #openstack-neutron13:03
*** salv-orlando has joined #openstack-neutron13:03
*** yfried has joined #openstack-neutron13:04
*** yatin has quit IRC13:04
*** yfried__ has quit IRC13:04
*** markvoelker has quit IRC13:05
*** dims__ has joined #openstack-neutron13:05
*** haramaki has joined #openstack-neutron13:06
*** lazy_prince is now known as killer_prince13:07
*** killer_prince is now known as lazy_prince13:08
*** alexpilotti has joined #openstack-neutron13:09
*** jckasper has quit IRC13:10
*** thorst has joined #openstack-neutron13:10
*** jckasper has joined #openstack-neutron13:10
*** EmilienM|afk is now known as EmilienM13:11
*** mwagner_lap has quit IRC13:11
*** dims__ has quit IRC13:13
*** yfried_ has joined #openstack-neutron13:14
*** yfried has quit IRC13:14
*** erkules_ is now known as erkules13:19
*** dims__ has joined #openstack-neutron13:19
*** oomichi_ has quit IRC13:23
*** yshafrir has quit IRC13:25
*** ZZelle has quit IRC13:26
*** ZZelle has joined #openstack-neutron13:26
*** tomoe_ has joined #openstack-neutron13:31
*** karimb has joined #openstack-neutron13:31
*** scheuran_ has quit IRC13:33
*** tellesnobrega has joined #openstack-neutron13:37
*** jgjl has joined #openstack-neutron13:38
*** crose has joined #openstack-neutron13:39
openstackgerritAssaf Muller proposed openstack/neutron: Cleanup in keepalived tests  https://review.openstack.org/15492513:39
amullersalv-orlando: ^13:41
*** pask81 has joined #openstack-neutron13:43
*** jgjl has quit IRC13:47
*** radez_g0n3 is now known as radez13:48
*** amotoki has joined #openstack-neutron13:48
*** jgjl has joined #openstack-neutron13:49
*** Mr0rris0 has joined #openstack-neutron13:49
*** Mr0rris0 has quit IRC13:49
*** Mr0rris0 has joined #openstack-neutron13:50
*** Mr0rris0 has joined #openstack-neutron13:50
*** Mr0rris0 has joined #openstack-neutron13:51
*** Mr0rris0 has quit IRC13:51
*** flaviof has joined #openstack-neutron13:51
*** Mr0rris0 has joined #openstack-neutron13:52
*** Mr0rris0 has quit IRC13:52
*** Mr0rris0 has joined #openstack-neutron13:52
*** mathias__ has quit IRC13:52
*** Mr0rris0 has quit IRC13:53
*** Mr0rris0 has joined #openstack-neutron13:53
*** Mr0rris0 has quit IRC13:54
*** Mr0rris0 has joined #openstack-neutron13:54
*** Mr0rris0 has quit IRC13:54
*** haramaki has quit IRC13:55
*** Mr0rris0 has joined #openstack-neutron13:55
*** Mr0rris0 has quit IRC13:55
*** Mr0rris0 has joined #openstack-neutron13:55
*** anshul has quit IRC13:55
*** Mr0rris0 has quit IRC13:56
*** Mr0rris0 has joined #openstack-neutron13:56
*** Mr0rris0 has quit IRC13:56
*** Mr0rris0 has joined #openstack-neutron13:57
openstackgerritRossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update  https://review.openstack.org/15522313:58
openstackgerritZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent  https://review.openstack.org/15531713:58
*** mwagner_lap has joined #openstack-neutron13:58
*** crose has quit IRC13:59
*** amotoki_ has joined #openstack-neutron14:00
*** haramaki has joined #openstack-neutron14:00
*** cgoncalv1s is now known as cgoncalves14:01
*** amotoki__ has joined #openstack-neutron14:02
*** amotoki has quit IRC14:02
*** amotok___ has joined #openstack-neutron14:02
*** amotoki_ has quit IRC14:05
*** amotoki__ has quit IRC14:06
*** RichardR1seley has joined #openstack-neutron14:06
openstackgerritLionel Zerbib proposed openstack/neutron: Add rate limiting security groups rules. This rules allow to drop packets or to block the origin IP beyond a rate threshold.  https://review.openstack.org/15453514:07
*** salv-orlando has quit IRC14:08
*** dane_leblanc has quit IRC14:08
*** salv-orlando has joined #openstack-neutron14:08
*** vhoward has joined #openstack-neutron14:09
*** RichardR1seley has quit IRC14:11
*** kbringard has joined #openstack-neutron14:11
*** avozza is now known as zz_avozza14:11
openstackgerritRossella Sblendido proposed openstack/neutron: L2 agent RPC add bulk calls for device up and device down  https://review.openstack.org/14743514:12
*** salv-orlando has quit IRC14:13
*** radez is now known as radez_g0n314:15
*** cference has joined #openstack-neutron14:15
openstackgerritLinChengyong proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor  https://review.openstack.org/14238614:19
*** dsundarraj has joined #openstack-neutron14:20
*** shwetaap has joined #openstack-neutron14:20
*** busterswt has joined #openstack-neutron14:20
*** irenab has quit IRC14:20
*** dsundarraj has quit IRC14:24
*** david-lyle_afk is now known as david-lyle14:25
*** julien_ZTE has joined #openstack-neutron14:25
*** anish_ has quit IRC14:25
*** naggappan has quit IRC14:26
ihrachyshkaotherwiseguy, hey. about to send a bunch of 'move main to neutron.cmd' patches14:28
*** alexpilotti has quit IRC14:28
openstackgerritZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent  https://review.openstack.org/15531714:28
*** jgjl has quit IRC14:28
*** alexpilotti has joined #openstack-neutron14:29
*** humble_ has joined #openstack-neutron14:31
*** moha_hunt has joined #openstack-neutron14:31
*** yuanying has joined #openstack-neutron14:31
*** boris-42 has quit IRC14:32
*** dane_leblanc has joined #openstack-neutron14:32
otherwiseguyihrachyshka: That makes you my Today's Favorite Person. :)14:34
ihrachyshkaotherwiseguy, half an hour or so, I need to kill some tests before sending it to hell14:35
ihrachyshkasorry, gate14:35
openstackgerritLionel Zerbib proposed openstack/neutron: Add rate limiting security groups rules.  https://review.openstack.org/15453514:35
*** yuanying has quit IRC14:35
*** rotbeard has quit IRC14:35
otherwiseguyThe gatekeeper did seem to be kind of demon-ish.14:35
*** r-daneel has joined #openstack-neutron14:36
*** nmagnezi has quit IRC14:36
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Setup br-tun in secure mode to avoid broadcast storms  https://review.openstack.org/15532814:36
*** banix has joined #openstack-neutron14:40
*** rotbeard has joined #openstack-neutron14:40
ajootherwiseguy ^  ihrachyshka , if you can ;)14:41
ajootherwiseguy, btw, excellent work on abstracting the ovsdb calls :D14:41
*** boris-42 has joined #openstack-neutron14:42
*** localloop127 has joined #openstack-neutron14:42
openstackgerritvenkata anil proposed openstack/neutron: Check for valid local_ip for OVS tunnel  https://review.openstack.org/15404314:42
otherwiseguyajo: looking at it.14:48
*** anilvenkata has left #openstack-neutron14:49
otherwiseguyajo: so far, the only thing I wonder about is having an option for reset_bridge() that add_bridge() doesn't have.14:49
ajootherwiseguy, yes.. we could add it to add_bridge, but I Wonder if it was a YAGNI :)14:49
otherwiseguyYeah. It's a tradeoff.14:49
otherwiseguyOn libraries I tend lean more towards consistency.14:51
ajowe have other stuff we don't use I think, like set_controller, at least the default ovs plugin14:51
ajootherwiseguy ^14:51
*** tellesnobrega has quit IRC14:51
ajobut as you say, it's  a library for everybody, consistency is a good point14:51
otherwiseguyajo: I seem to remember some of the other plugins using it.14:51
ajoI'm ok with adding it14:51
*** jprovazn has joined #openstack-neutron14:51
*** Murali has quit IRC14:52
ajopoint it out on the review and -1 :) so there's a record :D14:52
otherwiseguyThe ibm and ofa use it.14:52
ajoahh14:52
ajogood people14:52
ajowhat's OFA?14:52
otherwiseguyopen flow agent14:52
ajohmmm14:52
ajolike OVS but as an openflow controller?14:52
*** manishg has joined #openstack-neutron14:52
otherwiseguyI think so, haven't really looked at it much.14:52
ajootherwiseguy, cool, is a ryu OF agent14:53
ajoryu is cool ;)14:53
*** tiswanso has joined #openstack-neutron14:53
*** tvardeman has joined #openstack-neutron14:53
ajomy opinion is that we may eventually look on it :)14:54
otherwiseguyajo: do you have any idea what those test_ovs_lib unit tests are actually testing by just calling the functions?14:54
*** yeungp has joined #openstack-neutron14:54
*** tellesnobrega_ has joined #openstack-neutron14:54
ajootherwiseguy, I found them a bit weak, but... what could they do?14:54
*** manishg_ has joined #openstack-neutron14:54
ajocapture the calls on the other end, and just that?14:54
ajothat sounds like an antipattern14:55
otherwiseguyThere's no verification going on other than I guess they don't throw an exception before getting to execute()...14:55
ajoprobably marun shared already with you: http://googletesting.blogspot.fr/2015/01/testing-on-toilet-change-detector-tests.html14:55
ajoyeah14:55
*** tellesnobrega_ has quit IRC14:55
ajoI would have freaked out before reading that article14:55
ajobut..14:56
ajoI couldn't find more to do than ... checking the calls are being done14:56
*** manishg__ has joined #openstack-neutron14:56
openstackgerritPradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes  https://review.openstack.org/15492114:56
ajomay be with secure mode I Can mock the secure mode ovsdb call, and make sure it's done14:56
otherwiseguyYeah, they're the kind of tests I wouldn't even have. They're mostly duplicated by functional tests.14:56
otherwiseguyNot sure if I have a reset_bridge functional test.14:57
*** johnbelamaric has joined #openstack-neutron14:57
*** manishg has quit IRC14:57
ajootherwiseguy, aha, may be a functional tests is good14:57
ajogood point14:57
*** mlavalle has joined #openstack-neutron14:57
*** johnbelamaric has quit IRC14:57
*** johnbelamaric has joined #openstack-neutron14:57
*** manishg has joined #openstack-neutron14:57
*** rotbeard has quit IRC14:57
*** carl_baldwin has joined #openstack-neutron14:57
ajowe could create bridge in standard mode -> check it has the "NORMAL" rule,14:58
ajocreate in secure mode -> no rule should be14:58
*** jasondotstar has quit IRC14:58
amullerbut then you're testing OVS and not Neutron14:58
*** manish___ has joined #openstack-neutron14:59
*** manishg_ has quit IRC14:59
*** haramaki has quit IRC14:59
ajoamuller, I'm testing that my neutron interface is doing what I want to OVS15:00
ajoamuller, or may be I can check the ovs-vsctl output, and check that the specific bridge was set as secure15:00
ajo(openflow rules appart)15:00
*** manishg_ has joined #openstack-neutron15:00
*** apuimedo has quit IRC15:00
*** manishg__ has quit IRC15:00
amullermeh, I don't have a strong opinion here other then that you're testing the addition of an argument so that the test should be simple and cheap15:01
*** manishg has quit IRC15:01
*** manishg has joined #openstack-neutron15:02
*** SridharG has left #openstack-neutron15:03
*** manish___ has quit IRC15:03
*** manishg__ has joined #openstack-neutron15:03
*** pal has quit IRC15:04
*** manishg_ has quit IRC15:05
*** maishsk has joined #openstack-neutron15:05
*** manishg_ has joined #openstack-neutron15:05
*** pal has joined #openstack-neutron15:05
*** manishg has quit IRC15:06
*** prad has joined #openstack-neutron15:06
*** signed8bit has joined #openstack-neutron15:06
*** manishg has joined #openstack-neutron15:06
openstackgerritPradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes  https://review.openstack.org/15492115:06
ajoamuller, and what if I added the argument, but the result is not getting the bridge in secure mode?15:07
ajoand we believe it actually is15:07
*** manishg__ has quit IRC15:08
ajoI have a patch for juno, that works... and I manually tested there, but... I must admit I didn't do with transactions and the ovsdb abstraction15:08
ajootherwiseguy ^15:08
ajoso... we either add a functional test, or ... I test it manually15:08
*** dboik has joined #openstack-neutron15:08
openstackgerritAssaf Muller proposed openstack/neutron: Introduce ip address monitor  https://review.openstack.org/14964715:09
*** manishg_ has quit IRC15:09
openstackgerritAssaf Muller proposed openstack/neutron: Change metadata driver unit tests to use monitored spawn  https://review.openstack.org/15505815:09
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538415:09
*** manishg_ has joined #openstack-neutron15:09
*** manishg has quit IRC15:11
*** marun has joined #openstack-neutron15:12
*** manishg has joined #openstack-neutron15:12
*** lazy_prince is now known as killer_prince15:13
*** zzzeek has joined #openstack-neutron15:13
*** manishg_ has quit IRC15:14
otherwiseguyajo: If it was a unit test, it would probably need to mock/verify at the ovsdb api level, as doing any execute mocking wouldn't work for both backends. This also means that the whole test_ovs_lib unit test will only be "useful" for the vsctl backend, and even then only as long as it is the default. Otherwise, the test breaks.15:14
*** pal has quit IRC15:14
*** salv-orlando has joined #openstack-neutron15:14
otherwiseguyFunctional test is a little bit heavy, and there are functional tests that at least show that doing a multi-op transaction works already.15:14
otherwiseguyI could go either way. Any OVS functional test is really going to be two, one for each backend.15:15
ajootherwiseguy, functional seems like a very good way to do it,15:16
ajoit saves me the manual.verification15:16
ajoand, needing manual verification is always bad15:16
ajootherwiseguy, do we have the native ovsdb backend already in tree?15:16
*** manishg has quit IRC15:16
otherwiseguyDo the other callers of reset_bridge() also need to be u pdated to pass secure mode?15:17
*** RichardR1seley has joined #openstack-neutron15:17
*** pal has joined #openstack-neutron15:17
ajootherwiseguy, br-int calls on set-secure15:17
otherwiseguyajo: not quite, it's up for review and we're working out some dep issues.15:17
*** salv-orlando has quit IRC15:17
ajobut may be it could benefit from re-creating directly into secure15:17
*** jobewan has joined #openstack-neutron15:17
ajoinstead of doing it on different steps (avoiding race conditions)15:17
ajootherwiseguy ahh.. looking forward to see it roll :D15:18
otherwiseguyMaru has a test patch I need to depend on that does the python-openvswitch installation for the gate.15:18
ajootherwiseguy15:18
ajolook at line 725 here: https://review.openstack.org/#/c/155328/1/neutron/plugins/openvswitch/agent/ovs_neutron_agent.py15:18
ajoa very good reason to add secure_mode param to create / add bridge ...15:19
*** dsundarraj has joined #openstack-neutron15:19
*** RichardR1seley has quit IRC15:22
*** rkukura has quit IRC15:22
*** nplanel_ has joined #openstack-neutron15:22
otherwiseguyajo: makes sense. The API writer in me immediately jumps to thinking, "what if we need to create bridges with more options set, like we do with add_port" :p15:22
otherwiseguyThis is why it takes me forever to get anything done. :p15:22
otherwiseguyI seem to try to solve problems depth-first.15:23
ajootherwiseguy, but this is why you get something much better in the end :D15:23
ajoI was more used on solving all I could by myself when I was on a tiny company, so, I tend to limit depth to keep running.15:23
*** jgrimm is now known as zz_jgrimm15:24
otherwiseguyajo: "I love deadlines. I love the whooshing noise the make as they go by."15:24
otherwiseguy-- Douglas Adams15:24
*** rushil has joined #openstack-neutron15:25
ajootherwiseguy, that's reality.. :)15:27
*** cipcosma has joined #openstack-neutron15:27
ajodeadlines go away, and we forget of the long term ;)15:27
*** dboik_ has joined #openstack-neutron15:28
*** numan has quit IRC15:28
*** lnxnut_ has joined #openstack-neutron15:28
*** dboik_ has quit IRC15:28
*** karimb has quit IRC15:28
*** dboik_ has joined #openstack-neutron15:29
*** moshele has quit IRC15:29
*** jpich has quit IRC15:30
*** dboik has quit IRC15:31
*** lnxnut has quit IRC15:32
*** rkukura has joined #openstack-neutron15:33
*** irenab has joined #openstack-neutron15:34
*** hjensas has quit IRC15:34
*** baoli has quit IRC15:34
*** vthapar has quit IRC15:35
*** baoli_ has joined #openstack-neutron15:35
*** achanda has joined #openstack-neutron15:36
*** cipcosma has quit IRC15:37
*** lnxnut_ has quit IRC15:37
* russellb passes root_helper to otherwiseguy 15:39
* otherwiseguy waves at russellb 15:40
otherwiseguyand glares a little15:41
otherwiseguy:)15:41
russellbumadbro?15:41
amullerlol15:41
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323615:42
openstackgerritPavel Bondar proposed openstack/neutron: wip ipam reference driver  https://review.openstack.org/15048515:42
*** Networkn3rd has joined #openstack-neutron15:42
openstackgerritLiFengkai proposed openstack/neutron-specs: implements: blueprint open-wan-as-a-service  https://review.openstack.org/15536415:43
ihrachyshkaotherwiseguy, re version checks. so it's all about balance between ease of the check and 'pureness' of the check.15:44
*** achanda has quit IRC15:44
*** crc32 has quit IRC15:45
otherwiseguyihrachyshka: there is no balance. checking the version number will almost always eventually bite you. I wrote sanity checks specifically because version checks in the code were breaking RH stuff due to backported features.15:45
ihrachyshkaotherwiseguy, f.e. being able to check that dnsmasq supports hwaddr matching for ipv6 would mean starting dnsmasq AND dhcpv6 client, and diagnosing whether client get the address (sets it on iface etc.)15:45
*** itzikb has quit IRC15:45
amullerihrachyshka: otherwiseguy: thinking about it it's actually not difficult adding the necessary keepalived ipv6 default gateway sanity check15:45
amullerdunno about the dnsmasq one15:45
otherwiseguyihrachyshka: That sounds like exactly the kind of feature one would backport to an older version that was in long term support. :p15:47
*** ParsectiX has joined #openstack-neutron15:47
ihrachyshkaotherwiseguy, redhat was broken because version checks were in runtime15:47
ihrachyshkaotherwiseguy, so rebase could break15:48
otherwiseguyihrachyshka: and breaking the sanity check system so that it can't be used does not fix that problem.15:48
ihrachyshka'can't be used' wat15:48
*** amotok___ has quit IRC15:48
otherwiseguyThe sanity check failing, when a distro would actually work breaks sanity checking.15:49
ihrachyshkathere are arguments to disable specific checks if downstream assumes it's handled15:49
otherwiseguyIt makes it completely unusable.15:49
otherwiseguyThe whole point is to be able to avoid having to make downstream changes.15:49
*** kickinz1 is now known as kickinz1|afk15:49
*** xgerman has joined #openstack-neutron15:49
*** moshele has joined #openstack-neutron15:50
otherwiseguyIt's as bad as web pages specifically checking browser versions for features. It is just a very bad way to test for features.15:50
*** anish_ has joined #openstack-neutron15:52
openstackgerritMerged openstack/neutron: Scope state reporting rpc api using a messaging namespace  https://review.openstack.org/15467015:53
amullerrussellb: I'm getting back to this patch: https://review.openstack.org/#/c/125339/915:53
amullerrussellb: so I'm going to first push a patch that removes the unused RPC handler15:53
*** yfried_ is now known as yfried|afk15:54
amullerrussellb: I think it's correct to actually reduce the minor version of the server-side RPC, in this case? From 1.4 to 1.3? The L3 agent RPC client doesn't, won't, have any methods that use 1.415:54
*** salv-orlando has joined #openstack-neutron15:54
amullerrussellb: but juno neutron server uses 1.4, what would happen if this patch was merged but the follow up to re-add the function won't? During an upgrade a Juno server will restart, reducing its RPC server version from 1.4 to 1.315:55
amullerwould clients connected be affected, even though they don't have any 1.4 methods?15:55
*** jlibosva has quit IRC15:55
amullermaybe I should just leave the version as 1.415:56
openstackgerritCarl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes  https://review.openstack.org/14286315:56
*** kbringard has quit IRC15:57
*** jlibosva has joined #openstack-neutron15:58
ihrachyshkaotherwiseguy, ok, thanks for feedback, I will digest it later. I'll think how we can handle dnsmasq case easily15:58
pc_mcarl_baldwin: ping15:59
*** RichardR1seley has joined #openstack-neutron16:00
*** tomoe_ has quit IRC16:00
*** jgjl has joined #openstack-neutron16:00
*** doude has quit IRC16:01
carl_baldwinpc_m: pong16:01
pc_mCan you do a follow-up review on https://review.openstack.org/#/c/145005? I was wondering if an exception should be defined for the lookup failure. Thoughts?16:01
*** yfried|afk is now known as yfried_16:01
ihrachyshkaotherwiseguy, on irrelevant note - those patches moving main()s, I have plenty of them (16), one per service. do you think I can send them like that? I'm afraid if I squash those, review will become mess16:02
*** JPtheJP has joined #openstack-neutron16:02
russellbamuller: i wouldn't reduce the version.  if the method was never used, i don't think it's important to mess with the version at all16:03
otherwiseguyihrachyshka: oh my. Lots of patches.16:03
*** doude has joined #openstack-neutron16:03
otherwiseguyihrachyshka: maybe we could talk to dougwig/armax and see what they'd prefer?16:03
*** carlp has joined #openstack-neutron16:03
ihrachyshkaotherwiseguy, I'm not sure those are all, I see some agents in the tree that are not patched at all (I need to investigate why)16:03
otherwiseguySince they will probably be interested in reviewing.16:03
amullerrussellb: ack16:03
openstackgerritAssaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor  https://review.openstack.org/12538416:04
ihrachyshkaotherwiseguy, 'interesting'16:04
openstackgerritAssaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method  https://review.openstack.org/15537016:04
russellbamuller: feel free to ping me once you post so i can review16:04
ihrachyshkaotherwiseguy, sorry, I mean 'interested'16:04
otherwiseguyihrachyshka: :)16:04
russellbor you can post right now :)16:04
amullerhehe16:04
*** RichardR1seley has quit IRC16:04
*** a_le has joined #openstack-neutron16:04
ihrachyshkaas in 'gosh, is it diletant party?'16:04
carl_baldwinpc_m: I will look.16:04
pc_mcarl_baldwin: Thanks. Would like to help the submitter move that commit forward.16:04
otherwiseguyihrachyshka: I'll review them whenever you post them (in fact, checking reviews where you were the owner led me to see the version checking stuff).16:05
otherwiseguy:p16:05
ihrachyshkaotherwiseguy, that's good, I appreciate the feedback16:05
*** zz_jgrimm is now known as jgrimm16:05
ihrachyshkaotherwiseguy, when I moved runtime check there, I solved immediate problem (moving problem from agent), but maybe I should go further and actually fix the thing.16:05
*** nyechiel__ has quit IRC16:06
*** a_le has quit IRC16:06
ihrachyshkaotherwiseguy, for version checks, it was solving complexity for Sridhar's patch, but maybe it's also wrong way to go, so. I'll look into it, believe me.16:06
*** JPtheJP has quit IRC16:06
*** claudiub has quit IRC16:06
openstackgerritAssaf Muller proposed openstack/neutron: Reuse nova batch notifier  https://review.openstack.org/12801316:06
*** a_le has joined #openstack-neutron16:07
*** tvardeman has quit IRC16:07
otherwiseguyihrachyshka: I appreciate it.16:07
*** lnxnut has joined #openstack-neutron16:07
otherwiseguyihrachyshka: And I also appreciate you fixing the monkey_patching stuff, since, you know, my try at it broke the world. ;)16:08
*** salv-orlando has quit IRC16:08
*** anish_ has quit IRC16:08
ihrachyshkaarmax seems to be off16:08
ihrachyshkadougwig, around?16:08
* ihrachyshka now feels compassion to russellb who sent tens of 'change rpc mixin' patches and was blamed for it16:09
otherwiseguyihrachyshka: Also, I need to base my patches off yours, and I'm already around 12 deep. :)16:09
ihrachyshka16 patches... ick16:09
ihrachyshkaotherwiseguy, gosh16:09
ihrachyshkaotherwiseguy, don't do it before gate passes and we start getting +2's...16:10
ihrachyshkaI feel we'll have some rebases due to cool nits or whatever16:10
openstackgerritMerged openstack/neutron: Remove deprecated DEFAULT.root_helper  https://review.openstack.org/15306616:10
*** haramaki has joined #openstack-neutron16:10
openstackgerritZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent  https://review.openstack.org/15531716:10
russellbihrachyshka: heh yeah i got slapped16:11
ihrachyshkaok, no one interested around, so I'm just going to bomb gate as a strawman and we'll see what people have to say16:11
*** lnxnut has quit IRC16:12
* otherwiseguy gets some popcorn16:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Monkey patch all the code inside neutron/cmd/...  https://review.openstack.org/15537316:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved neutron.server.main into neutron.cmd.server  https://review.openstack.org/15537416:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved neutron.agent.l3_agent into neutron.cmd.agents.l3_agent  https://review.openstack.org/15537516:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved neutron.agent.metadata_agent into neutron.cmd.agents.metadata_agent  https://review.openstack.org/15537616:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for NEC agent into neutron.cmd  https://review.openstack.org/15537716:12
*** jorgem has joined #openstack-neutron16:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for Bigswitch RESTPROXY agent into neutron.cmd  https://review.openstack.org/15537816:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for IBM SDNVE agent into neutron.cmd  https://review.openstack.org/15537916:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for HyperV agent into neutron.cmd  https://review.openstack.org/15538016:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for oneconvergence agent into neutron.cmd  https://review.openstack.org/15538116:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for OVS agent into neutron.cmd  https://review.openstack.org/15538216:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for SR-IOV agent into neutron.cmd  https://review.openstack.org/15538316:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for LB agent into neutron.cmd  https://review.openstack.org/15538416:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for Cisco configuration agent into neutron.cmd  https://review.openstack.org/15538516:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for metering agent into neutron.cmd  https://review.openstack.org/15538616:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved neutron.agent.dhcp_agent to neutron.cmd.agents.dhcp_agent  https://review.openstack.org/15538716:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for metadata_proxy into neutron/cmd/...  https://review.openstack.org/15538816:12
* ihrachyshka hides16:12
* otherwiseguy didn't have time to make the popcorn16:12
ihrachyshkahttps://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:i_like_to_move_it,n,z16:12
ihrachyshkastarting from https://review.openstack.org/15537316:13
* otherwiseguy wonders if there is a git-review command to download a topic of patches16:13
amullerI guess I'll move my new executable into cmd as well :)16:14
*** salv-orlando has joined #openstack-neutron16:14
otherwiseguyamuller: doooo it16:14
*** haramaki has quit IRC16:15
otherwiseguyihrachyshka: I could have also just not moved to processutils.execute() :p16:15
*** manishg has joined #openstack-neutron16:15
ihrachyshkaamuller, yeah, that's how we want to end16:15
ihrachyshkaamuller, it's nova way (c)16:15
ihrachyshkaotherwiseguy, hm, just checkout the last one16:16
*** jpena_ has joined #openstack-neutron16:16
otherwiseguyNeutron goes Nova. :)16:16
otherwiseguyihrachyshka: well sure, if you want to be smart.16:16
openstackgerritRossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update  https://review.openstack.org/15522316:16
ihrachyshkaotherwiseguy, that move to processutils.execute() was in my todo, so we would hit it sooner or later16:16
otherwiseguy:)16:16
russellbyeah i usually just check out the last one16:17
russellbgit review -d <last#>16:17
*** maishsk has quit IRC16:17
ihrachyshkaok, last one is https://review.openstack.org/#/c/155388/16:17
*** rushil has quit IRC16:18
*** maishsk has joined #openstack-neutron16:18
otherwiseguyihrachyshka: Is your patch to do tests/__init__.py still up?16:18
*** jlibosva has quit IRC16:20
*** bjornar_ has quit IRC16:21
*** moshele has quit IRC16:22
*** jckasper_ has joined #openstack-neutron16:22
*** maishsk has quit IRC16:22
*** ihrachyshka has quit IRC16:22
*** Parsecti_ has joined #openstack-neutron16:22
*** sballe__ has joined #openstack-neutron16:23
*** jckasper has quit IRC16:24
*** ParsectiX has quit IRC16:24
*** kbringard has joined #openstack-neutron16:25
*** yamahata has joined #openstack-neutron16:26
*** claudiub has joined #openstack-neutron16:27
*** yeungp has quit IRC16:28
*** nplanel_ has quit IRC16:29
*** morgabra has quit IRC16:29
*** trinaths has joined #openstack-neutron16:30
*** jlanoux has quit IRC16:31
*** johnbelamaric has quit IRC16:31
*** morgabra has joined #openstack-neutron16:31
*** johnbelamaric has joined #openstack-neutron16:31
*** carl_baldwin has quit IRC16:32
*** trinaths1 has joined #openstack-neutron16:32
*** zz_avozza is now known as avozza16:33
*** a_le has quit IRC16:33
*** a_le has joined #openstack-neutron16:34
*** imcsk8 has quit IRC16:34
*** imcsk8 has joined #openstack-neutron16:34
*** trinaths has quit IRC16:35
*** avozza is now known as zz_avozza16:35
*** marios has quit IRC16:36
*** flaviof has quit IRC16:37
*** tellesnobrega_ has joined #openstack-neutron16:39
*** rook has joined #openstack-neutron16:39
*** tellesnobrega_ has quit IRC16:39
*** rook is now known as Guest3906316:39
*** pal_ has joined #openstack-neutron16:42
*** trinaths1 has left #openstack-neutron16:42
*** roeyc has quit IRC16:43
*** jgjl has quit IRC16:44
*** irenab has quit IRC16:44
*** yeungp has joined #openstack-neutron16:44
*** pal has quit IRC16:45
*** yamahata has quit IRC16:45
*** afazekas has quit IRC16:46
*** yamahata has joined #openstack-neutron16:46
openstackgerritLionel Zerbib proposed openstack/neutron: Add rate limiting security groups rules.  https://review.openstack.org/15453516:47
*** anshul has joined #openstack-neutron16:48
*** vivek-ebay has joined #openstack-neutron16:48
*** ChuckC_ has quit IRC16:51
*** ChuckC_ has joined #openstack-neutron16:51
*** ChuckC_ is now known as ChuckC16:51
*** tellesnobrega_ has joined #openstack-neutron16:52
*** ihrachyshka has joined #openstack-neutron16:53
ihrachyshkamarun, https://review.openstack.org/13393416:53
dougwigihrachyshka: lol, good show16:53
ihrachyshkadougwig, ... :)16:54
*** tvardeman has joined #openstack-neutron16:54
*** lnxnut has joined #openstack-neutron16:54
ihrachyshkaotherwiseguy, the very first patch in the series does it16:55
*** evgenyf has quit IRC16:55
ihrachyshkadougwig, I can try to reduce the number of patches by squashing them (squashing four of those will be straightforward for reviewers I guess, other - not sure)16:55
*** dims__ has quit IRC16:56
dougwigit doesn't bother me.  nor do long ones for things that are straightforward.16:56
ihrachyshkaok, I'll leave those as-is unless someone complains16:56
*** dims__ has joined #openstack-neutron16:57
*** leunam112 has quit IRC16:59
*** baoli_ has quit IRC17:00
*** anish_ has joined #openstack-neutron17:00
*** tflynn has joined #openstack-neutron17:00
*** tflynn has quit IRC17:00
*** tflynn has joined #openstack-neutron17:01
*** apuimedo has joined #openstack-neutron17:03
ihrachyshkadougwig, ok, so I need to respin anyway, so I'll squash some of those17:03
*** _cjones_ has joined #openstack-neutron17:04
ZZelleihrachyshka, hi17:04
ihrachyshkaZZelle, hi17:04
ZZelleihrachyshka, you should also associate these changes with a bug17:05
ihrachyshkaZZelle, do you have number in mind?17:05
*** RichardR1seley has joined #openstack-neutron17:05
ihrachyshkaotherwiseguy, ^^ ?17:06
openstackgerritClaudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition  https://review.openstack.org/15325917:06
otherwiseguyihrachyshka: I think I added a neutron impact to the one you opened for oslo.17:08
ZZelleihrachyshka,  => https://bugs.launchpad.net/oslo.concurrency/+bug/141854117:08
openstackLaunchpad bug 1418541 in neutron "processutils checks whether stdlib is monkey patched during import" [Undecided,Fix committed] - Assigned to Terry Wilson (otherwiseguy)17:08
ZZellebut it does not really feet17:09
ihrachyshkaZZelle, right. ok, let's track it there17:09
dougwigihrachyshka: if you have time, can you put your desired method of handling oslo with neutron-lib into that review, and then i'll respin the doc and post to the ML.17:09
dougwig?17:09
otherwiseguyZZelle: ihrachyshka : yep, that's the one. It didn't fit exactly, but it was close enough. :)17:09
*** aranjan has joined #openstack-neutron17:09
ihrachyshkaotherwiseguy, good for Related-17:09
ihrachyshkadougwig, ack, will do in 1-2 hours17:10
dougwigihrachyshka: ty17:10
*** SridharRamaswamy has joined #openstack-neutron17:10
*** jlibosva has joined #openstack-neutron17:10
*** Parsecti_ is now known as ParsectiX17:11
*** coolsvap is now known as coolsvap_17:12
*** bobmel has joined #openstack-neutron17:13
*** SridharRamaswam1 has joined #openstack-neutron17:13
*** thorst has quit IRC17:13
*** vthapar has joined #openstack-neutron17:13
ihrachyshkamarkmcclain, thanks for the comment on monkey patch series. I wonder whether it's misplaced (it refers to options moved while this particular patch does not move anything). Or do you refer to other patches? would it be better to move the whole files instead of just main()s for all of them? I also agree it needs discussion, I'll write an email.17:14
*** SridharRamaswamy has quit IRC17:14
*** bobmel_ has quit IRC17:15
*** tflynn has quit IRC17:15
*** carl_baldwin has joined #openstack-neutron17:15
*** tflynn has joined #openstack-neutron17:15
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for metadata_proxy into neutron/cmd/...  https://review.openstack.org/15538817:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Monkey patch all the code inside neutron/cmd/...  https://review.openstack.org/15537317:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved several services into neutron.cmd  https://review.openstack.org/15537417:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for metering agent into neutron.cmd  https://review.openstack.org/15538617:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for 3party plugins into neutron.cmd  https://review.openstack.org/15541217:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Moved main() for open plugins into neutron.cmd  https://review.openstack.org/15541317:17
*** jorgem has quit IRC17:17
*** hjensas has joined #openstack-neutron17:18
openstackgerritManish Godara proposed openstack/neutron: Add filter for provider network attributes in ML2  https://review.openstack.org/12491717:21
ajosalv-orlando, if you have a moment to take a look: https://review.openstack.org/#/c/155328/   broadcast storm fun on br-tun17:22
ajoobondarev_ ^17:22
salv-orlandok17:23
*** trad511 has joined #openstack-neutron17:23
*** banix has quit IRC17:25
*** EmilienM is now known as EmilienM|afk17:25
*** dsundarr_ has joined #openstack-neutron17:27
*** jistr has quit IRC17:27
*** salv-orlando has quit IRC17:29
*** armax has joined #openstack-neutron17:29
*** salv-orlando has joined #openstack-neutron17:29
*** apuimedo has quit IRC17:29
*** devvesa has quit IRC17:30
openstackgerritMerged openstack/neutron: Revert "monkey patch stdlib before importing other modules"  https://review.openstack.org/15515217:30
*** dsundarraj has quit IRC17:30
*** pask81 has quit IRC17:31
*** aepifanov has quit IRC17:34
*** rushil has joined #openstack-neutron17:37
*** matrohon has quit IRC17:38
ajosalv-orlando, thanks I thought it's a bit critical, because we got to kill a lab several times because of the bug ':)17:39
*** tflynn has quit IRC17:39
*** Marga_ has joined #openstack-neutron17:41
*** leenheer has joined #openstack-neutron17:41
*** tflynn has joined #openstack-neutron17:41
salv-orlandoajo: yeah I just do not remember very well the behaviour of secure mode and I'm doing some tests now17:41
salv-orlandobut if you can help me17:41
ajosalv-orlando, sure, let me look for the reference17:41
*** ChuckC has quit IRC17:42
salv-orlandoI think with secure mode br-tun never sets up flows unless you i)have a connected controller or ii) do it explictly17:42
ajohttp://openvswitch.org/support/dist-docs/tutorial/Tutorial.md.txt17:42
*** emagana has joined #openstack-neutron17:42
ajolook for secure17:42
*** ChuckC has joined #openstack-neutron17:42
*** Marga_ has quit IRC17:42
ajo"For our purpose, this just means that the OpenFlow flow table starts out empty."17:42
salv-orlandoajo: what are you quoting?17:43
ajothe problem with non-secure is that it starts with a NORMAL rule, which forward to the relevant ports (or ALL but "in" in case of broadcast/multicast)17:43
*** Marga_ has joined #openstack-neutron17:43
ajoahh17:43
ajocorrect,17:43
ajowhat you said is correct17:43
ajono rules unless controller conected or explicitly set17:43
salv-orlandoajo: that's fine I got the normal problem. I just want to make sure that we then don't wedge br-tun17:43
*** spandhe has joined #openstack-neutron17:43
ajosalv-orlando, sure,17:43
ajoi tested at least for vxlan, not gre17:43
ajobut it looks like it works, actually, we delete all the rules after reseting the bridge17:44
ajowhich is useless now, let me look for the line where we do that..17:44
ajobut17:44
ajoyes, let's make sure there are no more implications17:44
ajothe rule stuff is clear17:44
*** jlibosva has quit IRC17:45
*** tflynn has quit IRC17:45
ajosalv-orlando: man 5 ovs-vswitchd.conf.db17:46
*** tflynn has joined #openstack-neutron17:46
ajohttp://www.fpaste.org/184823/42376320/17:46
*** johnbelamaric has quit IRC17:47
ajook, it seems that we just prevent the normal rule (switch acting as a normal MAC-learning switch)17:47
salv-orlandoajo: yes it sounds like this17:48
ajoah, nice it seems like I didn't run relevant tests locally...17:49
openstackgerritCarl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes  https://review.openstack.org/14286317:49
ajoneutron.tests.unit.openvswitch.test_ovs_tunnel failing because of not expecting the new parameter set17:49
salv-orlandoprobably just misleading docs.. if you read man ovs-vsctl, line 341 it makes it sound like in secure mode the switch does not set up any flow in secure mode unless connected17:49
*** safchain has quit IRC17:50
*** vivek-ebay has quit IRC17:50
*** vivek-ebay has joined #openstack-neutron17:50
ajomissleading, yes17:51
ajothe actual behavior is that we can really add them via ovs-ofctl17:51
ajomanually tested..17:51
ajobut I guess we don't need to do tests to test ovs... yet I want to add a functional one to make sure we set the bridge to secure correctly17:52
*** SumitNaiksatam has joined #openstack-neutron17:52
*** anish_ has quit IRC17:52
*** marun has quit IRC17:52
ajobtw, I guess that works, since we're using it for br-int17:53
*** baoli has joined #openstack-neutron17:53
*** humble_ has quit IRC17:53
*** tellesnobrega_ has quit IRC17:53
*** banix has joined #openstack-neutron17:54
*** vivek-ebay has quit IRC17:54
salv-orlandoajo: yup. If we can be sure that it won't block the flows which are normally automatically set up at the data path level that's ok17:54
ajothat's checked :)17:54
salv-orlandoI want to be careful because that's one of those things we do not test in the gate ;)17:54
*** vivek-ebay has joined #openstack-neutron17:54
ajoI checked it to make sure the problem didn't reproduce17:55
ajowith cross-node communication17:55
ajoetc17:55
ajodhcp, floating ips, separate compute nodes, network nodes...17:55
*** coolsvap_ is now known as coolsvap17:55
*** dsundarr_ has quit IRC17:56
ajoof course, feel free to double-check, as you said, not tested on the gate is always a high risk17:56
*** jkyle has quit IRC17:56
ajodid we ever had multi node gate tests?17:56
*** jkyle1 has joined #openstack-neutron17:57
*** baoli has quit IRC17:57
ajoto be clear, I didn't check this exact implementation, but the backported one to Juno (APIs differ sightly because of the abstraction layer for ovsdb) but it's in my plans to run test br-tun on master too17:57
ajosecure + rules not broken: that is tested17:57
*** tellesnobrega_ has joined #openstack-neutron17:58
*** _cjones_ has quit IRC17:59
salv-orlandoajo: yes that would be enough for me. We have never done multi node testing in the gate unfortunately17:59
*** anshul has quit IRC18:00
*** Youcef has joined #openstack-neutron18:00
*** leunam112 has joined #openstack-neutron18:00
*** aix has quit IRC18:00
*** tiswanso has quit IRC18:03
*** leunam112 has quit IRC18:04
*** tiswanso_ has joined #openstack-neutron18:04
*** claudiub has quit IRC18:04
*** jp_at_hp has quit IRC18:04
*** ivar-lazzaro has joined #openstack-neutron18:06
*** rushil has quit IRC18:06
*** ivar-lazzaro has quit IRC18:07
*** ivar-lazzaro has joined #openstack-neutron18:08
*** rushil has joined #openstack-neutron18:08
*** nlahouti1 has left #openstack-neutron18:09
*** nlahouti has joined #openstack-neutron18:09
*** salv-orlando has quit IRC18:10
*** salv-orlando has joined #openstack-neutron18:10
*** salv-orl_ has joined #openstack-neutron18:12
*** salv-orl_ has quit IRC18:12
*** tflynn has quit IRC18:12
*** salv-orl_ has joined #openstack-neutron18:13
openstackgerritZengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method  https://review.openstack.org/15410318:13
*** dboik_ has quit IRC18:13
openstackgerritMerged openstack/neutron: Check conntrack rule cleanup on floating IP disassociate  https://review.openstack.org/13393418:13
pc_mWondering if anyone has seen this... I updated a patch for a review in my neutron-vpnaas repo, and tried to run tox. It is failing with an ImportError: neutron.hooks.setup_hook18:14
pc_mI tried removing the .tox area, updating the neutron repo to latest, running setup.py install in neutron repo. Can't seem to clear this error.18:15
*** mwagner_lap has quit IRC18:16
*** s3wong has joined #openstack-neutron18:16
pc_mIt seems to be right at the end of installing the pbr egg (says installed, and then next line has E"RROR:root:Error parsing" and a T/B with the import error.18:16
*** afazekas has joined #openstack-neutron18:17
pc_mAnyone have any ideas?18:17
*** _cjones_ has joined #openstack-neutron18:17
*** aranjan has quit IRC18:18
pc_mTox output... http://paste.openstack.org/show/17225718:18
*** baoli has joined #openstack-neutron18:18
dougwigthat should've been fixed with 155152 (neutron)18:19
dougwigpc_m: ^^18:19
openstackgerritMerged openstack/neutron: keepalived: use sh instead of bash for notifier scripts  https://review.openstack.org/15493118:19
*** salv-orlando has quit IRC18:20
*** dboik has joined #openstack-neutron18:20
*** dboik has quit IRC18:20
*** dboik has joined #openstack-neutron18:20
pc_mdougwig: Thanks! Looks like it just merged. I'll try refreshing my neutron repo.18:20
*** ZZelle_ has joined #openstack-neutron18:20
*** harlowja_away is now known as harlowja18:21
*** baoli has quit IRC18:22
*** EmilienM|afk is now known as EmilienM18:22
*** MasterPiece has quit IRC18:23
*** tflynn has joined #openstack-neutron18:23
*** dane_leblanc has quit IRC18:23
*** SridharRamaswam1 has quit IRC18:24
*** shashankhegde has joined #openstack-neutron18:24
*** localloop127 has quit IRC18:25
*** achanda has joined #openstack-neutron18:25
*** localloop127 has joined #openstack-neutron18:27
openstackgerritSandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver  https://review.openstack.org/15543618:28
*** emagana has quit IRC18:29
*** johnbelamaric has joined #openstack-neutron18:29
*** bjornar_ has joined #openstack-neutron18:29
*** emagana has joined #openstack-neutron18:29
*** yeungp has quit IRC18:30
*** zz_avozza is now known as avozza18:30
*** emagana has quit IRC18:30
*** emagana has joined #openstack-neutron18:30
*** tellesnobrega_ has quit IRC18:31
*** annegent_ has joined #openstack-neutron18:32
*** boris-42 has quit IRC18:32
*** dboik has quit IRC18:32
*** yfried_ has quit IRC18:34
*** dsundarraj has joined #openstack-neutron18:35
*** tellesnobrega_ has joined #openstack-neutron18:35
*** jaypipes has quit IRC18:36
*** raildo has joined #openstack-neutron18:37
*** dboik has joined #openstack-neutron18:38
*** dsundarraj has quit IRC18:38
*** baoli has joined #openstack-neutron18:40
*** nyechiel__ has joined #openstack-neutron18:40
*** yeungp has joined #openstack-neutron18:40
*** dsundarraj has joined #openstack-neutron18:40
*** tiswanso_ has quit IRC18:41
*** rushil_ has joined #openstack-neutron18:42
*** nyechiel__ has quit IRC18:42
*** nyechiel__ has joined #openstack-neutron18:43
*** avozza is now known as zz_avozza18:44
*** zz_avozza is now known as avozza18:44
*** rushil has quit IRC18:44
*** nyechiel__ has quit IRC18:46
*** nyechiel__ has joined #openstack-neutron18:46
*** tflynn has quit IRC18:48
*** tflynn has joined #openstack-neutron18:48
*** SridharRamaswamy has joined #openstack-neutron18:49
*** dane_leblanc has joined #openstack-neutron18:49
*** MasterPiece has joined #openstack-neutron18:49
*** ihrachyshka has quit IRC18:50
*** devvesa has joined #openstack-neutron18:50
openstackgerritGerman Eichberger proposed openstack/neutron-vpnaas: Reorder Neutron import statements in file  https://review.openstack.org/14250718:51
*** Longgeek has quit IRC18:51
*** nmagnezi has joined #openstack-neutron18:51
*** Longgeek has joined #openstack-neutron18:52
*** kbringard has quit IRC18:52
*** Marga_ has quit IRC18:53
*** ParsectiX has quit IRC18:53
*** Marga_ has joined #openstack-neutron18:53
*** ParsectiX has joined #openstack-neutron18:53
*** ihrachyshka has joined #openstack-neutron18:54
*** dboik has quit IRC18:54
*** apuimedo has joined #openstack-neutron18:55
*** dboik has joined #openstack-neutron18:55
*** Longgeek has quit IRC18:56
anteayadougwig: why are import statements being reordered?18:56
anteayadougwig: I had thought that the order of import statements was important to keep to avoid package collision18:57
anteayadougwig: am I missing something in my understanding?18:57
openstackgerritIhar Hrachyshka proposed openstack/neutron: Raise timeout for test_conntrack_disassociate_fip  https://review.openstack.org/15544518:57
*** coolsvap is now known as coolsvap_18:57
*** trad5111 has joined #openstack-neutron18:57
*** doude has quit IRC18:58
*** tiswanso has joined #openstack-neutron18:58
*** Marga_ has quit IRC18:59
*** avozza is now known as zz_avozza18:59
*** Marga_ has joined #openstack-neutron18:59
*** _cjones_ has quit IRC19:00
*** johnbelamaric has quit IRC19:00
*** trad511 has quit IRC19:00
*** johnbelamaric has joined #openstack-neutron19:00
*** tomoe_ has joined #openstack-neutron19:01
*** pal_ has quit IRC19:01
*** tflynn has quit IRC19:01
*** tflynn has joined #openstack-neutron19:02
*** _cjones_ has joined #openstack-neutron19:03
*** tomoe_ has quit IRC19:05
*** dboik has quit IRC19:05
*** dboik has joined #openstack-neutron19:06
*** doude has joined #openstack-neutron19:06
*** thedodd has joined #openstack-neutron19:06
*** Marga_ has quit IRC19:08
*** suresh12 has joined #openstack-neutron19:08
*** Marga_ has joined #openstack-neutron19:08
*** Marga_ has quit IRC19:09
*** Marga_ has joined #openstack-neutron19:09
HenryGanteaya: Where are import statements being reordered?19:09
*** tflynn has quit IRC19:09
openstackgerritRajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources  https://review.openstack.org/15342219:09
ZZelle_salv-orl_, hi19:10
*** achanda has quit IRC19:11
*** alexpilotti has quit IRC19:11
HenryGanteaya: I guess you are referring to https://review.openstack.org/14250719:12
*** banix has quit IRC19:12
*** moshele has joined #openstack-neutron19:12
HenryGanteaya: Import order does not matter in Python (except for some special cases like monkey patching)19:13
*** johnbelamaric has quit IRC19:13
HenryGanteaya: We have some hacking rules to group and alphabetically order import statements19:13
*** moshele has quit IRC19:14
*** banix has joined #openstack-neutron19:14
*** baoli has quit IRC19:15
prometheanfireanyone around for a +2 https://review.openstack.org/#/c/154128/19:15
*** kawa2014 has quit IRC19:15
anteayaHenryG: sorry was having lunch, yes that was the patch to which I was referring19:16
anteayaHenryG: okay thanks19:16
*** mwagner_lap has joined #openstack-neutron19:18
kevinbentonihrachyshka: hey, don't you think 100 seconds is a little high?19:18
openstackgerritSridhar Ramaswamy proposed openstack/neutron-vpnaas: Implementation of Brocade Vyatta VPNaaS Plugin  https://review.openstack.org/15472019:19
kevinbentonihrachyshka: this test is waiting for a netcat timeout, right?19:19
*** aranjan has joined #openstack-neutron19:20
openstackgerritCarl Baldwin proposed openstack/neutron: Refactor management of namespaces in the L3 Agent  https://review.openstack.org/14774419:21
*** apuimedo has quit IRC19:21
kevinbentonis the next OpenStack release officially called Liberty?19:21
*** vthapar has quit IRC19:22
HenryGkevinbenton: yup :(19:22
*** devvesa has quit IRC19:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc  https://review.openstack.org/15501319:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager  https://review.openstack.org/15496919:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge  https://review.openstack.org/15501419:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent  https://review.openstack.org/15521219:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from external_process  https://review.openstack.org/15495919:22
openstackgerritTerry Wilson proposed openstack/neutron: Add run_as_root option to utils.execute  https://review.openstack.org/15306719:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent  https://review.openstack.org/15518819:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess  https://review.openstack.org/15510119:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg for ovs_lib  https://review.openstack.org/15491419:22
openstackgerritTerry Wilson proposed openstack/neutron: Stop using passed root_helper in ip_lib  https://review.openstack.org/14582919:22
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager  https://review.openstack.org/15498019:22
markmcclainihrachyshka: yeah I think the discussion part is needed19:22
*** busterswt has quit IRC19:22
otherwiseguyihrachyshka: ok, I have untangled my root_helper stuff from the processutils.execute/monkey_patch mess. :)19:23
kevinbentonHenryG: sounds very patriotic19:24
kevinbentonotherwiseguy, markmcclain: we are bombarding ihrachyshka with IRC messages :)19:25
kevinbentononce he gets back to his compute he will think there is a crisis19:25
markmcclainhaha19:25
ihrachyshkakevinbenton, well, I stole 100 from another func test that uses the assert19:25
otherwiseguykevinbenton: and he should be drinking a beer right now, getting kind of late there.19:25
*** nmagnezi has quit IRC19:26
carl_baldwinmlavalle: ping19:26
ihrachyshkamarkmcclain, ack, just sent an email: http://lists.openstack.org/pipermail/openstack-dev/2015-February/056790.html19:26
* otherwiseguy owes ihrachyshka several beers by now19:26
*** emagana has quit IRC19:26
kevinbentonihrachyshka: can we decrease the netcat timeout time just in the interest of fast functional tests when possible?19:26
ihrachyshkakevinbenton, it takes ~12 secs on my machine19:26
kevinbentonihrachyshka: or where is the bottleneck?19:26
ihrachyshkakevinbenton, it's always crisis, it's neutron19:26
*** emagana has joined #openstack-neutron19:26
markmcclainihrachyshka: awesome thanks19:27
kevinbentonihrachyshka: i like to call it realtime development19:27
openstackgerritArmando Migliaccio proposed openstack/neutron: Add callbacks-based system to Neutron  https://review.openstack.org/15377419:27
openstackgerritArmando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration  https://review.openstack.org/15474919:27
openstackgerritArmando Migliaccio proposed openstack/neutron: Decouple L3 base and VPN service plugins during router operations  https://review.openstack.org/15377619:27
ihrachyshkakevinbenton, I can look into it tomorrow, though we need band-aid now19:27
kevinbentonihrachyshka: ack19:27
*** dsundarraj has quit IRC19:27
*** dsundarraj has joined #openstack-neutron19:27
ihrachyshkaotherwiseguy, that series ^^ makes me feel better (I'm not the only one bombing gate with dilletante patches)19:28
otherwiseguyihrachyshka: :) If anyone wants to review it as a single patch, I'll happily squash it for them...and watch them go insane.19:28
ihrachyshkaotherwiseguy, ok, I will not forget that several beers thing if I get to CA19:29
ihrachyshkaotherwiseguy, no no no, please don't19:29
otherwiseguyI'm surprised I'm still marginally functioning. They are not particularly interesting patches to write. :)19:30
*** LeanReox has joined #openstack-neutron19:30
openstackgerritArmando Migliaccio proposed openstack/neutron: Add callbacks-based system to Neutron  https://review.openstack.org/15377419:30
openstackgerritArmando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration  https://review.openstack.org/15474919:30
openstackgerritArmando Migliaccio proposed openstack/neutron: Decouple L3 base and VPN service plugins during router operations  https://review.openstack.org/15377619:30
otherwiseguyBut it will be quite satisfying when they're all finished.19:30
ihrachyshkaok, now going for beer. cheers to everyone, and please baby-sit that band-aid patch if possible19:31
LeanReoxhi guys, is there any known issue with centos5 VMs trying to access metadata ? cause centos 6 and 7 and ubuntu works just fine, but centos5 can ping metadata ip but cant curl to it19:31
*** emagana has quit IRC19:31
LeanReoxim running juno release19:31
*** johnbelamaric has joined #openstack-neutron19:31
jogoyour breaking global-requirements on master19:33
*** achanda has joined #openstack-neutron19:33
jogohttp://logs.openstack.org/82/155182/1/check/check-requirements-integration-dsvm/ea9a7d2/console.html#_2015-02-12_13_25_28_07819:33
openstackgerritTerry Wilson proposed openstack/neutron: Automate host configuration for functional testing  https://review.openstack.org/14611319:34
openstackgerritTerry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API  https://review.openstack.org/15276119:34
jogokevinbenton: ^19:35
openstackgerritErik Colnick proposed openstack/neutron: Improve DVR scale performance  https://review.openstack.org/15011019:35
openstackgerritErik Colnick proposed openstack/neutron: Refactor to facilitate DVR scale performance  https://review.openstack.org/15057619:35
*** leunam112 has joined #openstack-neutron19:35
kevinbentonjogo: what causes that?19:36
otherwiseguyugh. maru, I think I just effectively rebased your automate host configuration for functional testing patch because I am not a smart man.19:36
*** Sukhdev has joined #openstack-neutron19:37
*** leunam112 has quit IRC19:37
kevinbentonjogo: that's just a warning right? the failure is coming from that neutron.hooks.setup_hook below, right?19:39
jogokevinbenton: correct19:40
jogosetuphook is faiing19:40
dougwiganteaya: order of imports should affect packaging at all (you might mean dependencies and requirements for that?)   but from the context of the new repos, neutron is now a third-party library, so pep8 says that imports of neutron should be in the second block of imports, not the third.  there was another proposal to do what horizon does and make a new fourth19:40
dougwigblock for neutron imports (so, "standard, third-party, neutron, self" ordering.)  lbaas re-ordered with 3 to fit pep8, there's a review out to do the same for vpn, and i don't think fwaas has changed anything.19:40
jogotalking about in -qa19:40
anteayadougwig: wow19:41
anteayaand yeah, perhaps I mean requirements19:41
*** aepifanov has joined #openstack-neutron19:42
kevinbentondougwig: i have a meeting starting now. can you look into this?19:42
anteayadougwig: how does that work for horizon?19:42
dougwigkevinbenton: sure, what am i looking at?19:42
HenryGjogo: Let me try to help19:43
mlavallecarl_baldwin: pong19:43
kevinbentonjogo: oh, based on the discussion in openstack-qa it sounds like a fix was just merged?19:43
carl_baldwinmlavalle: I updated https://review.openstack.org/#/c/147744/ and I’m nearly finished with rebasing it.19:43
*** emagana has joined #openstack-neutron19:44
*** dsundarr_ has joined #openstack-neutron19:44
dougwiganteaya: i meant "should not", not "should" up there.19:44
openstackgerritSridhar Ramaswamy proposed openstack/neutron-vpnaas: Implementation of Brocade Vyatta VPNaaS Plugin  https://review.openstack.org/15472019:44
jogoyeah sounds like it, will know soon19:44
carl_baldwinmlavalle: I addresses a lot of my feedback on PS16 but you might want to read through it.19:45
anteayadougwig: yeah, I heard should not in the context19:45
dougwiganteaya: our flake8 seems to be lax about import order and grouping.  what context of "work for horizon" are you referring to?19:45
mlavallecarl_baldwin: how do you see us moving ahead with this? Are you going to review it and I continue from there?19:45
HenryGjogo, kevinbenton, dougwig: yes that looks like the monkey patch issue that was reverted, https://review.openstack.org/15515219:45
dougwiganteaya: and by "our" in that last, i mean "openstack's flake8", not neutron.19:45
anteayadougwig: you mentioned that you are copying what horizon does, does what horizon does work for horizon?19:45
dougwigi am the kind of imprecise today.19:45
dougwig /kind/king.19:45
dougwigsigh.19:45
anteayadougwig: I feel ya19:45
jogoHenryG: correct19:45
mlavallecarl_baldwin: or you want to conitnue working on it? I want to help but I don't want to get in the way19:46
dougwiganteaya: we're actually currently following strict pep8, and doing "standard, third-party, self", where neutron is a third-party module.  we had one reviewer give feedback suggesting the horizon method, during the vpn code review to reorder stuff.  i don't think any discussion or decision has happened on that.19:46
mlavallecarl_baldwin: either way or a combinatio of both is fine with me19:46
carl_baldwinmlavalle: I thought I’d take care of much of yesterday’s feedback since you were otherwise occupied.  Today, I thought I’d take care of the rebase because I merged Henry’s radvd refactor.19:47
openstackgerritCarl Baldwin proposed openstack/neutron: Refactor management of namespaces in the L3 Agent  https://review.openstack.org/14774419:47
*** dsundarraj has quit IRC19:47
carl_baldwinmlavalle: I’ll hand the reins back to you.19:47
carl_baldwinmlavalle: ^^19:47
anteayadougwig: ah okay cool I mis-understood19:47
*** lnxnut has quit IRC19:47
anteayadougwig: thanks for the education19:47
*** dims__ has quit IRC19:48
mlavallecarl_baldwin: great. I'll use the next couple of hours to do some things for my day job adn then i'll go back to this patchset :-)19:48
*** johnbelamaric has quit IRC19:48
*** vinsh has joined #openstack-neutron19:48
*** johnbelamaric has joined #openstack-neutron19:48
carl_baldwinmlavalle: okay19:49
*** emagana has quit IRC19:49
mlavallecarl_baldwin: thanks for the help19:49
carl_baldwinmlavalle: You too.19:49
*** emagana has joined #openstack-neutron19:49
*** dims__ has joined #openstack-neutron19:50
*** nyechiel has joined #openstack-neutron19:50
mlavallecarl_baldwin: I like this.... I get to learn from you and contribute at the same time. thanks19:50
*** _cjones_ has quit IRC19:52
*** nyechiel__ has quit IRC19:52
carl_baldwinmlavalle: I’m hope I’m not leading you astray.  ;)19:52
carl_baldwinjohnbelamaric: Are you around?19:53
*** Longgeek has joined #openstack-neutron19:53
carl_baldwinmlavalle: I hope I got all of the tests passing with the patch.  I got to the point where I thought they were all good.  We’ll see if Jenkins agrees.19:53
*** emagana has quit IRC19:54
mlavallemlavalle: ok, i'll wathc it19:54
*** zz_avozza is now known as avozza19:54
openstackgerritJustin Hammond proposed openstack/python-neutronclient: Client command extension support  https://review.openstack.org/14831819:55
openstackgerritJustin Hammond proposed openstack/python-neutronclient: Client command extension support  https://review.openstack.org/14831819:56
roaet<- pro git review spammer19:56
roaet:P19:56
*** nyechiel has quit IRC19:57
carl_baldwinjohnbelamaric: ping19:57
*** nyechiel_ has joined #openstack-neutron19:57
roaetplease forgive me :(19:57
openstackgerritJustin Hammond proposed openstack/python-neutronclient: Client command extension support  https://review.openstack.org/14831819:57
*** achanda has quit IRC19:58
*** achanda has joined #openstack-neutron19:58
*** tflynn has joined #openstack-neutron19:59
*** manishg_ has joined #openstack-neutron20:00
*** manishg has quit IRC20:00
*** ygbo has quit IRC20:01
*** manishg_ has quit IRC20:01
*** manishg has joined #openstack-neutron20:02
*** Longgeek has quit IRC20:02
*** Miouge has quit IRC20:03
*** yfried_ has joined #openstack-neutron20:04
*** LeanReox has left #openstack-neutron20:04
*** shwetaap has quit IRC20:05
*** yfried_ has quit IRC20:05
*** shwetaap has joined #openstack-neutron20:05
*** flaviof has joined #openstack-neutron20:06
*** briancurtin has quit IRC20:07
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433920:07
*** s0mik_ has quit IRC20:08
*** zhiyan has quit IRC20:08
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433920:08
*** ctracey has quit IRC20:08
*** emagana has joined #openstack-neutron20:09
*** doude has quit IRC20:10
*** manishg has quit IRC20:11
*** manishg has joined #openstack-neutron20:11
*** doude has joined #openstack-neutron20:12
*** s3wong has quit IRC20:12
*** Youcef has quit IRC20:13
dougwigroaet: scroll up to see ihrachyshka post about 20 in a row.  you have some catching up to do.  :)20:14
*** s3wong has joined #openstack-neutron20:14
roaetdougwig: ha.. awesome20:14
roaetI don't feel so bad20:14
dougwigyou're just consuming your own companies VM nodes, so...  :)20:14
roaetmm. never considered it that way20:15
*** manishg has quit IRC20:16
dougwigcompany's.  i think i need to go back to bed today.  i can't grammar.20:16
*** s0mik_ has joined #openstack-neutron20:16
*** ctracey has joined #openstack-neutron20:16
*** _cjones_ has joined #openstack-neutron20:16
*** briancurtin has joined #openstack-neutron20:17
*** zhiyan has joined #openstack-neutron20:18
openstackgerritJustin Hammond proposed openstack/python-neutronclient: Client command extension support  https://review.openstack.org/14831820:19
*** kfox1111 has joined #openstack-neutron20:19
*** ronis_ has joined #openstack-neutron20:20
*** ronis has quit IRC20:21
kfox1111ok. running into a problem. just upgraded from icehouse to juno. almost everything seems to be working.20:21
kfox1111Ive got a lbaas pool that isn't working tough... its tap is:20:21
kfox1111Port "tap1ca011a2-c8" tag: 409520:22
kfox1111badly tagged.20:22
*** stefanb has quit IRC20:22
kfox1111ideas?20:22
*** Marga_ has quit IRC20:22
*** Marga_ has joined #openstack-neutron20:23
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during create/update operations  https://review.openstack.org/15377520:25
openstackgerritArmando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation  https://review.openstack.org/15547020:25
*** shashankhegde has quit IRC20:27
openstackgerritJustin Hammond proposed openstack/python-neutronclient: Client command extension support  https://review.openstack.org/14831820:30
*** annegent_ has quit IRC20:32
*** nyechiel__ has joined #openstack-neutron20:32
*** nyechiel_ has quit IRC20:33
*** tvardeman has quit IRC20:33
*** maishsk has joined #openstack-neutron20:35
kfox1111why would lbaas ports go 4095 when the rest of the services are working?20:36
*** manishg has joined #openstack-neutron20:38
*** salv-orlando has joined #openstack-neutron20:38
*** prad has quit IRC20:38
*** dsundarr_ has quit IRC20:38
*** hjensas has quit IRC20:40
openstackgerritShiv Haris proposed openstack/neutron: Do not review.  https://review.openstack.org/15238320:40
*** tellesnobrega_ has quit IRC20:41
*** kbringard has joined #openstack-neutron20:41
*** salv-orl_ has quit IRC20:41
*** manishg_ has joined #openstack-neutron20:41
*** markvoelker has joined #openstack-neutron20:41
*** manishg has quit IRC20:43
*** Marga_ has quit IRC20:44
*** yfauser has joined #openstack-neutron20:44
*** dsundarraj has joined #openstack-neutron20:44
*** spandhe has quit IRC20:44
*** spandhe has joined #openstack-neutron20:45
*** Marga_ has joined #openstack-neutron20:46
*** yamahata has quit IRC20:46
*** yfauser has left #openstack-neutron20:48
*** s3wong has quit IRC20:49
johnbelamariccarl_baldwin: pong20:50
*** Marga_ has quit IRC20:50
carl_baldwinjohnbelamaric: Do you have a moment to talk about ipam?20:50
johnbelamaricsure20:50
*** pc_m has quit IRC20:50
*** busterswt has joined #openstack-neutron20:51
carl_baldwinjohnbelamaric: So, I’ve tried to address most of the feedback on https://review.openstack.org/#/c/134339/20:51
carl_baldwinThe PS18 L221 discussion remains.20:52
*** manishg_ has quit IRC20:52
carl_baldwinjohnbelamaric: I think your last suggestion was to pass in the Subnet object.  The issue I see with that is that the Subnet object doesn’t actually have any of the subnet details on it.20:52
carl_baldwinjohnbelamaric: ^ That might be a problem in and of itself.20:53
johnbelamariccarl_baldwin: ah. shows what i know20:53
*** s3wong has joined #openstack-neutron20:53
johnbelamariccarl_baldwin: seems odd20:53
*** leenheer has quit IRC20:53
*** Zemeio has quit IRC20:53
carl_baldwinjohnbelamaric: Agreed that it does seem a bit odd.20:54
carl_baldwinjohnbelamaric: I haven’t decided how to make it better yet.20:54
kfox1111hmm... the config all looks just like my other cloud that's working. :/20:54
*** RichardR1seley has quit IRC20:55
johnbelamariccarl_baldwin: I recall a discussion that these classes are passing around dicts instead of real objects. this is very strange to me but it may be my Ruby on Rails background that makes it look so odd20:55
*** leenheer has joined #openstack-neutron20:55
carl_baldwinjohnbelamaric: I just noticed more comments that I need to catch up on from you and Akihiro from PS18.  So hard to stay on top of the comments sometimes.20:55
johnbelamariccarl_baldwin: yes especially across various patch sets20:56
*** nlahouti has quit IRC20:56
carl_baldwinjohnbelamaric: Which classes passing around dicts?  That is true in the API.20:56
johnbelamariccarl_baldwin: let me look at the code for a minute20:56
*** nlahouti has joined #openstack-neutron20:56
*** kickinz1|afk is now known as kickinz120:57
johnbelamariccarl_baldwin: look in db_base_plugin_v2 it's all over. this will be the primary consumer of the interface20:58
carl_baldwinjohnbelamaric: Right, there are lots of dicts in there.20:58
*** annegent_ has joined #openstack-neutron20:58
anteayafor any neutron folks who care, nova meeting is about to start in -meeting21:00
johnbelamariccarl_baldwin: so, if you look at update_subnet in there, it's getting a subnet dict i believe. so, there was discussion at some point of stopping passing around dicts and use real objects - i figured if we are building an interface we should do that, even if the actual db_base_plugin changes to do that are not going to happen in Kilo21:01
amullerjohnbelamaric++21:01
amullerbe part of the solution ;-)21:01
carl_baldwinjohnbelamaric: This ipam interface won’t accept any of those dicts.  The consumer will have to massage the data from the dicts in to the ipam api.21:01
johnbelamaricamuller: :)21:02
johnbelamariccarl_baldwin: right, but for update_subnet, what should it take? assuming we want to inform IPAM of the changes, which would generally be to name, allocation pool, maybe someone could try to change gateway_ip that way21:02
*** jckasper has joined #openstack-neutron21:02
carl_baldwinjohnbelamaric: I certainly did not want to proliferate the usage of those dicts by having them passed to the ipam api.21:02
kfox1111hmm... I restarted some services, and now I have less ports tagged with 4095, but still a few...21:02
kfox1111why would that be flaky?21:02
johnbelamariccarl_baldwin: generally we don't need to pass them in - we have request objects - but for update/delete we need to pass something. delete maybe just ID21:03
*** dmellado has quit IRC21:03
johnbelamariccarl_baldwin: we could pass just ID to update, but then the driver would have to go to the DB to read the object21:03
carl_baldwinkfox1111: It will help if you tag the specific user to whom you’re addressing your comments.  Given the current context, they seem to be coming out of nowhere and are likely lost in the noise.21:03
carl_baldwinjohnbelamaric: delete (actually deallocate) takes only the cidr in the current form.21:04
*** dmellado has joined #openstack-neutron21:04
carl_baldwinjohnbelamaric: No, I don’t want the driver to read from the db.21:04
johnbelamariccarl_baldwin: that should be enough in the context of the pool21:04
johnbelamariccarl_baldwin: for delete i mean21:04
*** jckasper_ has quit IRC21:05
kfox1111carl_baldwin: I'm coming in cold with a problem. Not sure who to address comments to...21:05
johnbelamariccarl_baldwin: ok. so, that's sort of what the discussion is getting at - how does the external IPAM know about the meta-data, in both the create and the update21:05
johnbelamariccarl_baldwin: if you look at my comment from 2/5 9:01am on line 221 of PS 18, that's what I am trying to get at21:06
*** baoli has joined #openstack-neutron21:06
*** jckasper has quit IRC21:07
*** jckasper has joined #openstack-neutron21:07
*** claudiub has joined #openstack-neutron21:07
carl_baldwinjohnbelamaric: I think I have my head wrapped around the problem.  Unfortunately, I have a meeting very soon that I need to get to.  I will make another update to the interface to address it and then I’ll ping you for confirmation.21:08
johnbelamariccarl_baldwin: sounds good, i need to head out shortly anyway21:09
carl_baldwinjohnbelamaric: okay, ttyl.21:09
*** jckasper_ has joined #openstack-neutron21:09
johnbelamariccarl_baldwin: ttyl21:09
*** boris-42 has joined #openstack-neutron21:09
*** baoli has quit IRC21:11
*** baoli has joined #openstack-neutron21:11
*** nyechiel__ has quit IRC21:12
*** jckasper has quit IRC21:12
*** shashankhegde has joined #openstack-neutron21:13
*** Marga_ has joined #openstack-neutron21:15
*** banix has quit IRC21:17
ZZelle_dougwig, hi21:19
*** Marga_ has quit IRC21:19
*** leenheer has quit IRC21:20
*** dsundarraj has quit IRC21:20
*** leenheer has joined #openstack-neutron21:20
*** MasterPiece has quit IRC21:20
*** claudiub has quit IRC21:22
*** banix has joined #openstack-neutron21:23
*** leunam112 has joined #openstack-neutron21:24
*** dmellado has quit IRC21:25
*** TrevorV_ has joined #openstack-neutron21:25
*** dmellado has joined #openstack-neutron21:26
kevinbentonrkukura: ping21:27
openstackgerritErik Colnick proposed openstack/neutron: Improve DVR scale performance  https://review.openstack.org/15011021:27
*** leunam112 has quit IRC21:27
*** emagana has quit IRC21:30
rkukurakevinbenton: pong21:31
*** emagana has joined #openstack-neutron21:31
kevinbentonrkukura: does what i said on this review make sense, or am i misinterpreting the change? https://review.openstack.org/#/c/149546/21:31
*** rushil_ has quit IRC21:31
rkukurakevinbenton: The issue this is addressing is that, at least most of the time, people create their external network as a normal tenant network (VLAN, GRE, VXLAN, whatever), and this allocates a segmentation_id that will never be used because they are using the external bridge, which ignores the network’s definition.21:34
kevinbentonrkukura: right, but i'm pointing out that this is usually the opposite with our VLAN based deployments21:34
*** emagana has quit IRC21:34
kevinbentonrkukura: we don't use a separate bridge for external networks21:34
*** emagana has joined #openstack-neutron21:35
rkukurakevinbenton: But I do kind of agree we shouldn’t be hard-wiring in a specific type. One option would be to add a new default_external_network_type config variable, and maybe have it default to ‘local’.21:35
rkukurakevinbenton: I agree that many do not use the external bridge. If not using this, the provider attributes need to be supplied, right?21:35
kevinbentonrkukura: no, if no provider attributes are supplied, it will go through the auto allocation21:36
*** manishg has joined #openstack-neutron21:36
rkukurakevinbenton: Right, and this patch is changing that to use ‘local’ instead of the tenent_network_types config variable.21:37
*** baoli has quit IRC21:37
*** aepifanov has quit IRC21:37
*** sinh has joined #openstack-neutron21:37
rkukuraOnly when provider attributes are not supplied and external=True21:37
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433921:37
*** Sukhdev has quit IRC21:37
*** baoli has joined #openstack-neutron21:37
kevinbentonrkukura: yeah, i don't like that :)21:37
kevinbentonrkukura: i do see where it is nice for overlays21:38
kevinbentonrkukura: since the external network is frequently not a tunnel network21:38
rkukurakevinbenton: Even with VLANs, the one randomly allocated is not likely the one that connects to the outside world.21:39
*** jaypipes has joined #openstack-neutron21:39
rkukuraWith tunnels, wasting a segmentation_id value isn’t a big deal, but with VLANs it might be.21:39
kevinbentonrkukura: it's not a waste. it's what the external network is on21:39
kevinbentonrkukura: in our workflow we let openstack pick a vlan for itself21:40
rkukurakevinbenton: Are you saying there is a common use case where the external network gets created with a random VLAN tag, and then someone goes into the switch and configures that VLAN to be routed externally?21:40
*** jaypipes has quit IRC21:40
*** manishg has quit IRC21:41
*** dmellado has quit IRC21:41
kevinbentonrkukura: yes, in our fabric the network UUID is the identifier for a broadcast domain21:41
kevinbentonrkukura: VLANs are just used to define membership associations with specific ports21:41
*** jpena has quit IRC21:42
kevinbentonrkukura: so the VLAN openstack picks can be random, that's just used to associate the ports with compute nodes to the external segment with that VLAN21:42
*** jpena_ has quit IRC21:42
rkukurakevinbenton: Interesting.21:42
*** dmellado has joined #openstack-neutron21:42
kevinbentonrkukura: then the admin can add whatever rules to that segment to connect in the router to the network segment21:43
rkukurakevinbenton: Sounds like maybe you should be using HPB and dynamic VLAN segments for the switch<->node link.21:43
rkukurakevinbenton: Anyway, I see why this patch could be problematic for you right now.21:44
*** aepifanov has joined #openstack-neutron21:44
rkukurakevinbenton: How about the idea of a default_external_network_type config variable?21:45
*** trad511 has joined #openstack-neutron21:45
kevinbentonrkukura: we happen to work with flexible VLANs to the edge, but that might not always be the case21:45
kevinbentonrkukura: either way, i don't like this automagic re-mapping based on it being an external network21:46
kevinbentonrkukura: if we are going to go down this route, i would rather it just force the user to specify21:47
*** trad5111 has quit IRC21:47
*** trad511 has quit IRC21:47
*** Longgeek has joined #openstack-neutron21:47
*** cference has quit IRC21:47
rkukurakevinbenton: I think that would break devstack and lots of scripts and so forth that expect to be using an external bridge and ignoring whatever gets allocated for the external network.21:48
kevinbentonrkukura: but what is proposed there will break anything that sets this to '' https://github.com/openstack/neutron/blob/master/etc/l3_agent.ini#L4621:49
rkukurakevinbenton: I certainly don’t feel the automagic re-mapping is absolutely necessary. It just seemed like a minor tweek to avoding wasting VLAN tags in a common case.21:49
*** jaypipes has joined #openstack-neutron21:49
amullerarmax: https://bugs.launchpad.net/neutron/+bug/141809721:50
openstackLaunchpad bug 1418097 in neutron "Kernel panic or ovs-vswitch crash when running DVR functional test" [Undecided,Confirmed]21:50
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433921:50
amullerplease see comments 9 and 1021:50
rkukurakevinbenton: When external_network_bridge is not empty (including the default of ‘br-ex’), network_type is ignored, and ‘local’ works fine. In fact, that’s what I’ve always recommended people specify when creating an external network and using an external bridge.21:51
kevinbentonrkukura: i know. i'm talking about when it is empty21:52
kevinbentonthat's what i meant by ''21:52
*** TrevorV_ has quit IRC21:52
rkukurakevinbenton: Looked like one double-quote rather than two single-quotes.21:53
kevinbentonrkukura: ah21:53
kevinbentonrkukura: and we prefer that deployment because then external networks follow the same semantics as all tenant networks21:53
rkukurakevinbenton: OK, so I see how this breaks your case needing a real VLAN tag assigned.21:53
kevinbentonrkukura: there aren't different debugging paths, etc21:53
kevinbentonrkukura: and we can support multiple external networks if people want them21:54
rkukurakevinbenton: Completely agree with you an avoiding the special external bridge. I really wish we could remove that, or at least change the default to “”.21:54
kevinbentonrkukura: what happens if local is chosen when there isn't a different external bridge?21:54
kevinbentonrkukura: does that become untagged?21:55
*** ihrachyshka has quit IRC21:55
rkukurakevinbenton: In that case, the external network isn’t very external ;)21:55
rkukurakevinbenton: With type ‘local’ traffic never leaves br-int.21:56
*** Marga_ has joined #openstack-neutron21:56
kevinbentonrkukura: ok21:56
*** banix has quit IRC21:56
rkukurakevinbenton: So you’ve convinced my that there is a valid use case for external networks being allocated just like normal tenant networks.21:57
kevinbentonrkukura: we need a setting on the server side to know whether or not external bridges will be used21:57
kevinbentonrkukura: because then we can put warning logging in when there is a misconfigured l3 agent21:57
kevinbentonrkukura: and we can also branch on that for this automatic mapping to local21:58
*** apuimedo has joined #openstack-neutron21:58
rkukurakevinbenton: I’d hate to add a setting that does nothing but control whether a warning gets generated. And I agree we can’t count on agent config being available in the server.21:58
kevinbentonrkukura: well it will control this mapping.21:59
kevinbentonrkukura: and we have no way to detect now if there is a mix of configurations between l3 agents21:59
rkukurakevinbenton: Why not just add a default_external_network_type config variable. If this is empty, then tenant_network_types is used, just like today. If its set, then that type is used when external is set and no provider attributes are specified.21:59
kevinbentonrkukura: so the warning would be bonus debugging21:59
*** dmellado has quit IRC22:00
maishskEE390F7922:00
*** maishsk has quit IRC22:00
rkukurakevinbenton: I’d prefer to just start deprecating external_network_bridge.22:00
*** emagana has quit IRC22:01
kevinbentonrkukura: that might be hard22:01
*** dmellado has joined #openstack-neutron22:01
*** aranjan has quit IRC22:01
kevinbentonrkukura: unfortunately it can be pretty useful because people can setup that bridge to do all sorts of stuff22:01
kevinbentonrkukura: other physical interfaces could be on it22:02
kevinbentonrkukura: or tunnels to narnia or whatever22:02
*** jprovazn has quit IRC22:02
*** s3wong has quit IRC22:02
*** aranjan has joined #openstack-neutron22:02
rkukurakevinbenton: Same could be done with a physical_network name mapping to a dedicated bridge.22:02
kevinbentonrkukura: i'm fine with those config semantics you suggested for default_external_network_type22:03
kevinbentonrkukura: oh right, i forgot about that22:03
kevinbentonrkukura: always just use one22:03
*** manishg has joined #openstack-neutron22:04
kevinbentonrkukura: ok, so let's do that then22:04
rkukurakevinbenton: I’ll add a quick comment to the review suggesting to consider adding the default_external_network_type config variable. The tricky part would be what its default should be - empty vs. ‘local’22:04
kevinbentonrkukura: add default_external_network_type22:04
kevinbentonrkukura: we can fight about empty vs. local22:04
*** yfauser1 has joined #openstack-neutron22:05
*** yfauser1 has left #openstack-neutron22:05
kevinbentonrkukura: i think i'm fine with defaulting to local22:06
kevinbentonrkukura: because tunnels are more common22:06
kevinbentonrkukura: we can just clear that in our deployment tools22:06
*** dmellado has quit IRC22:07
*** s3wong has joined #openstack-neutron22:08
*** dmellado has joined #openstack-neutron22:08
*** spandhe has quit IRC22:09
kevinbentonrkukura: i added a comment to the review22:09
rkukurakevinbenton: so did I22:09
kevinbentonrkukura: ok, can you check the outline of the code i put in my comment to make sure we are on the same page?22:10
rkukurakevinbenton: Just did ;)22:11
*** spandhe has joined #openstack-neutron22:12
*** emagana has joined #openstack-neutron22:12
*** annegent_ has quit IRC22:13
kevinbentonrkukura: great!22:13
*** annegent_ has joined #openstack-neutron22:15
*** aranjan has quit IRC22:15
*** _cjones_ has quit IRC22:16
*** tiswanso has quit IRC22:17
*** regXboi has joined #openstack-neutron22:17
*** harlowja_ has joined #openstack-neutron22:18
*** salv-orlando has quit IRC22:19
*** s3wong has quit IRC22:19
*** s3wong has joined #openstack-neutron22:19
*** andreykurilin has quit IRC22:19
*** salv-orlando has joined #openstack-neutron22:19
*** Longgeek has quit IRC22:19
*** _cjones_ has joined #openstack-neutron22:19
*** andreykurilin has joined #openstack-neutron22:19
*** Apsu has quit IRC22:20
*** Apsu has joined #openstack-neutron22:21
*** harlowja has quit IRC22:21
*** amuller has quit IRC22:21
*** aranjan has joined #openstack-neutron22:22
*** dmellado has quit IRC22:22
*** dmellado has joined #openstack-neutron22:24
*** thedodd has quit IRC22:24
*** yamahata has joined #openstack-neutron22:25
*** aranjan has quit IRC22:26
*** ronis_ has quit IRC22:32
*** banix has joined #openstack-neutron22:32
*** vivek-ebay has quit IRC22:33
*** ParsectiX has quit IRC22:34
*** ParsectiX has joined #openstack-neutron22:35
*** thedodd has joined #openstack-neutron22:36
*** thedodd has quit IRC22:37
*** tellesnobrega_ has joined #openstack-neutron22:39
*** carl_baldwin has quit IRC22:40
*** localloop127 has quit IRC22:44
*** Networkn3rd has quit IRC22:47
*** zzzeek_ has joined #openstack-neutron22:49
*** annegent_ has quit IRC22:49
*** regXboi has quit IRC22:50
*** zzzeek has quit IRC22:51
*** zzzeek_ is now known as zzzeek22:51
*** yuanying has joined #openstack-neutron22:55
*** trad511 has joined #openstack-neutron23:03
*** busterswt has quit IRC23:05
*** vivek-ebay has joined #openstack-neutron23:05
*** dmellado has quit IRC23:05
*** jgrimm is now known as zz_jgrimm23:06
*** dmellado has joined #openstack-neutron23:06
*** ParsectiX has quit IRC23:07
*** dmellado has quit IRC23:07
*** kbringard has quit IRC23:07
*** ParsectiX has joined #openstack-neutron23:08
*** dmellado has joined #openstack-neutron23:08
*** a_le has quit IRC23:09
*** a_le has joined #openstack-neutron23:09
*** bjornar_ has quit IRC23:09
*** dboik_ has joined #openstack-neutron23:09
*** julien_ZTE has quit IRC23:11
*** a_le has quit IRC23:12
*** dboik has quit IRC23:13
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent  https://review.openstack.org/15521223:13
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent  https://review.openstack.org/15518823:13
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess  https://review.openstack.org/15510123:13
openstackgerritTerry Wilson proposed openstack/neutron: Remove root_helper arg from linux interface  https://review.openstack.org/15552223:13
openstackgerritTerry Wilson proposed openstack/neutron: Remove remaining root_helper args from plugins  https://review.openstack.org/15552323:13
*** avozza is now known as zz_avozza23:13
*** dboik_ has quit IRC23:14
*** a_le has joined #openstack-neutron23:14
*** leunam112 has joined #openstack-neutron23:14
*** vivek-ebay has quit IRC23:16
*** aranjan has joined #openstack-neutron23:16
*** cference has joined #openstack-neutron23:16
*** chlong has joined #openstack-neutron23:17
*** leunam112 has quit IRC23:18
*** gildub_ has joined #openstack-neutron23:20
*** aranjan has quit IRC23:21
*** zzzeek_ has joined #openstack-neutron23:22
openstackgerritArmando Migliaccio proposed openstack/neutron: Fix AttributeError exception for API's test_network_lifecycle  https://review.openstack.org/15552823:23
*** zzzeek has quit IRC23:24
*** zzzeek_ is now known as zzzeek23:24
*** hichihara has joined #openstack-neutron23:24
*** watanabe_isao has joined #openstack-neutron23:27
*** gildub has joined #openstack-neutron23:27
*** watanabe_isao has quit IRC23:28
jogoso this is still a top gate bug: http://status.openstack.org/elastic-recheck/index.html#132365823:32
jogokevinbenton: ^23:32
jogoneutron only23:32
*** watanabe_isao has joined #openstack-neutron23:32
watanabe_isaoGood morning to ALL...:)23:33
*** dsneddon_ has joined #openstack-neutron23:33
*** apuimedo has quit IRC23:34
*** Mr0rris0 has quit IRC23:35
*** EmilienM is now known as EmilienM|afk23:35
*** olc has quit IRC23:36
*** dsneddon has quit IRC23:36
*** aranjan has joined #openstack-neutron23:38
*** yamahata has quit IRC23:39
jogoanteaya: so one of the top gate bugs is a neutron+nova bug its been around for way too long23:39
*** tellesnobrega_ has quit IRC23:39
*** yamahata has joined #openstack-neutron23:40
jogohttp://status.openstack.org/elastic-recheck/index.html#132365823:40
*** openstack has joined #openstack-neutron23:41
*** baoli has quit IRC23:42
*** baoli has joined #openstack-neutron23:42
*** shwetaap has quit IRC23:48
*** oomichi has joined #openstack-neutron23:50
armaxanteaya: ping23:52
*** carl_baldwin has joined #openstack-neutron23:54
*** jobewan has quit IRC23:54
*** salv-orlando has quit IRC23:54
*** aepifanov has quit IRC23:56
*** krtaylor has quit IRC23:57
*** annegent_ has joined #openstack-neutron23:59
*** _cjones_ has quit IRC23:59
*** trad511 has quit IRC23:59
openstackgerritDoug Wiegley proposed openstack/neutron-vpnaas: DO NOT MERGE, test units  https://review.openstack.org/15553523:59
*** trad511 has joined #openstack-neutron23:59
*** _cjones_ has joined #openstack-neutron23:59

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