*** emagana has quit IRC | 00:02 | |
*** markvoelker has quit IRC | 00:03 | |
*** dims__ has joined #openstack-neutron | 00:04 | |
watanabe_isao | Good morning to ALL.:) | 00:07 |
---|---|---|
*** trad511 has quit IRC | 00:07 | |
*** apuimedo has quit IRC | 00:08 | |
*** baoli has quit IRC | 00:08 | |
*** emagana has joined #openstack-neutron | 00:08 | |
*** sambetts has quit IRC | 00:08 | |
*** dane_leblanc has quit IRC | 00:10 | |
*** shwetaap has quit IRC | 00:10 | |
*** emagana has quit IRC | 00:12 | |
*** SridharRamaswamy has joined #openstack-neutron | 00:14 | |
*** kickinz1 is now known as kickinz1|afk | 00:15 | |
*** kickinz1|afk is now known as kickinz1 | 00:16 | |
*** achanda has quit IRC | 00:16 | |
*** afazekas has quit IRC | 00:16 | |
*** dsundarraj has joined #openstack-neutron | 00:18 | |
*** aranjan_ has quit IRC | 00:21 | |
*** tflynn has quit IRC | 00:21 | |
openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 00:22 |
*** aranjan has joined #openstack-neutron | 00:24 | |
*** tsg has quit IRC | 00:24 | |
*** emagana has joined #openstack-neutron | 00:26 | |
*** banix has quit IRC | 00:27 | |
*** r-daneel has joined #openstack-neutron | 00:28 | |
*** tomoe_ has joined #openstack-neutron | 00:28 | |
*** aranjan has quit IRC | 00:29 | |
*** trad511 has joined #openstack-neutron | 00:29 | |
*** RichardR2seley has joined #openstack-neutron | 00:29 | |
RichardR2seley | I 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 IRC | 00:30 | |
*** Longgeek has quit IRC | 00:30 | |
RichardR2seley | When 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-neutron | 00:31 | |
*** tsg has joined #openstack-neutron | 00:33 | |
*** banix has joined #openstack-neutron | 00:35 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc https://review.openstack.org/155013 | 00:35 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager https://review.openstack.org/154969 | 00:35 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge https://review.openstack.org/155014 | 00:35 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from external_process https://review.openstack.org/154959 | 00:35 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess https://review.openstack.org/155101 | 00:35 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager https://review.openstack.org/154980 | 00:35 |
*** sballe__ has quit IRC | 00:37 | |
*** trad511 has quit IRC | 00:39 | |
*** avozza is now known as zz_avozza | 00:39 | |
*** suresh12 has quit IRC | 00:40 | |
*** suresh12 has joined #openstack-neutron | 00:41 | |
*** leenheer has quit IRC | 00:43 | |
*** dsneddon has joined #openstack-neutron | 00:45 | |
*** kickinz1 is now known as kickinz1|afk | 00:45 | |
*** erkules has quit IRC | 00:46 | |
*** csoukup has joined #openstack-neutron | 00:48 | |
*** leunam112 has joined #openstack-neutron | 00:50 | |
*** aranjan has joined #openstack-neutron | 00:50 | |
*** markvoelker has joined #openstack-neutron | 00:52 | |
*** leunam112 has quit IRC | 00:52 | |
*** sbfox has quit IRC | 00:53 | |
*** erkules has joined #openstack-neutron | 00:53 | |
*** tflynn has joined #openstack-neutron | 00:54 | |
*** banix has quit IRC | 00:55 | |
*** tflynn_ has joined #openstack-neutron | 00:56 | |
*** markvoelker has quit IRC | 00:57 | |
*** dims__ has quit IRC | 00:58 | |
*** markvoelker has joined #openstack-neutron | 00:59 | |
*** tflynn has quit IRC | 00:59 | |
*** tflynn_ has quit IRC | 01:01 | |
*** banix has joined #openstack-neutron | 01:01 | |
*** manishg has quit IRC | 01:01 | |
*** RichardR2seley has quit IRC | 01:02 | |
*** manishg has joined #openstack-neutron | 01:02 | |
*** RichardR1seley has quit IRC | 01:02 | |
*** ChuckC has quit IRC | 01:02 | |
*** ChuckC has joined #openstack-neutron | 01:03 | |
*** markvoelker has quit IRC | 01:04 | |
*** mlavalle has quit IRC | 01:05 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/neutron: Combining sec groups member update RPC calls https://review.openstack.org/155145 | 01:05 |
*** zhhuabj has joined #openstack-neutron | 01:06 | |
*** signed8bit is now known as signed8bit_ZZZzz | 01:06 | |
*** manishg has quit IRC | 01:06 | |
*** lnxnut has joined #openstack-neutron | 01:06 | |
*** tsg has quit IRC | 01:07 | |
*** xgerman has quit IRC | 01:08 | |
*** sbfox has joined #openstack-neutron | 01:09 | |
*** manishg has joined #openstack-neutron | 01:09 | |
*** signed8bit_ZZZzz is now known as signed8bit | 01:11 | |
*** signed8bit is now known as signed8bit_ZZZzz | 01:11 | |
*** leenheer has joined #openstack-neutron | 01:14 | |
*** nplanel_ has joined #openstack-neutron | 01:14 | |
*** zhhuabj has quit IRC | 01:14 | |
*** achanda has joined #openstack-neutron | 01:16 | |
*** manishg has quit IRC | 01:17 | |
*** achanda has quit IRC | 01:18 | |
openstackgerrit | yalei wang proposed openstack/neutron-fwaas: Implementation of Mcafee NGFW Driver https://review.openstack.org/152093 | 01:19 |
*** achanda has joined #openstack-neutron | 01:19 | |
*** dsundarraj has quit IRC | 01:20 | |
*** dsundarraj has joined #openstack-neutron | 01:20 | |
*** sbfox has quit IRC | 01:21 | |
*** dane_leblanc has joined #openstack-neutron | 01:22 | |
*** popw has joined #openstack-neutron | 01:22 | |
*** manishg has joined #openstack-neutron | 01:22 | |
armax | otherwiseguy: ping | 01:24 |
otherwiseguy | armax: pong | 01:24 |
armax | I think we need to back out your change | 01:25 |
armax | as it breaks the world :) | 01:25 |
otherwiseguy | armax: which change? | 01:25 |
armax | https://review.openstack.org/#/c/153699/ | 01:25 |
*** banix has quit IRC | 01:25 | |
otherwiseguy | uh oh. :) what part of the world did it break? | 01:26 |
*** Aish has quit IRC | 01:26 | |
armax | all the projects that use neutron as dependency | 01:26 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revert "monkey patch stdlib before importing other modules" https://review.openstack.org/155152 | 01:26 |
armax | otherwiseguy: ^^^ | 01:26 |
otherwiseguy | So things like the *aaS? | 01:27 |
armax | otherwiseguy: I think we need to work out a change that flags NO_EVENTLET the other way | 01:27 |
*** amotoki has joined #openstack-neutron | 01:27 | |
armax | yes | 01:27 |
armax | HenryG: ^^^ | 01:27 |
armax | dougwig: ^^^ | 01:27 |
openstackgerrit | Merged openstack/neutron: Move NCS mech driver to its new home https://review.openstack.org/151893 | 01:27 |
*** amotoki has quit IRC | 01:28 | |
*** zhhuabj has joined #openstack-neutron | 01:28 | |
otherwiseguy | armax: how is it breaking them? just their pep8 tests? | 01:28 |
armax | otherwiseguy: as well as UT's | 01:28 |
armax | have a look at http://status.openstack.org/zuul/ | 01:29 |
armax | and search 152093,11 for instance | 01:29 |
*** manishg has quit IRC | 01:29 | |
armax | otherwiseguy: if we added setenv = NO_EVENTLET=1 to the affected tests it would do the trick | 01:30 |
armax | but that’s too much of a change, I guess | 01:30 |
armax | is there any chance we can flip the logic? | 01:30 |
HenryG | I'm here | 01:30 |
HenryG | back it out or fix the breakages? | 01:30 |
armax | HenryG: back up until min 25 | 01:30 |
dougwig | shouldn't that fix have gone into main for the server binaries? | 01:31 |
*** banix has joined #openstack-neutron | 01:31 | |
otherwiseguy | armax: 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 |
otherwiseguy | Seems 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 IRC | 01:32 | |
armax | otherwiseguy: I’d rather not sprinkle change everywhere if possible | 01:33 |
*** zhhuabj has quit IRC | 01:33 | |
otherwiseguy | armax: is this representative of the errors? https://jenkins06.openstack.org/job/gate-neutron-fwaas-pep8/41/console | 01:33 |
*** manishg has joined #openstack-neutron | 01:34 | |
armax | what about introducing your change so that we set setenv = EVENTLET the oppositve way? | 01:34 |
armax | otherwiseguy: yes | 01:34 |
armax | that’s an example | 01:34 |
armax | like we do: | 01:34 |
armax | if os.environ.get(‘EVENTLET’) and apply on the tests? | 01:35 |
armax | on testenv | 01:35 |
dougwig | it 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 |
dougwig | not you armax, the merged change. | 01:36 |
armax | this way the change fix your issue, but preserve sanity to the other projects... | 01:36 |
armax | and we can slowly phase in the flag to other projects in due course | 01:36 |
otherwiseguy | armax: I don't understand how flipping the logic would work. All agents, etc. would need EVENTLET environment variable set. | 01:36 |
armax | otherwiseguy: now you do: | 01:37 |
armax | https://github.com/openstack/neutron/blob/master/tox.ini#L52 | 01:37 |
armax | what I am saying is do ‘positive logic' | 01:37 |
armax | and add EVENTLET=1 to testenv | 01:38 |
armax | https://github.com/openstack/neutron/blob/master/tox.ini#L6 | 01:38 |
armax | here https://github.com/openstack/neutron/blob/master/neutron/__init__.py#L19 | 01:38 |
armax | you’d do: | 01:38 |
armax | if os.environ.get(‘EVENTLET’): | 01:38 |
armax | the effect would be the same, would it? | 01:38 |
otherwiseguy | armax: 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 |
otherwiseguy | Unless I'm missing something. | 01:39 |
dougwig | why 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 |
armax | well, the binaries would need to flip the switch too | 01:39 |
*** markvoelker has joined #openstack-neutron | 01:40 | |
armax | right, dougwig’s point is the crux of the matter, to some extent | 01:40 |
otherwiseguy | The "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 |
dougwig | that seems reasonable, but this stopgap does not. :) | 01:40 |
armax | dougwig: which stopgap? | 01:41 |
otherwiseguy | My code that relies on that change hasn't been pushed, so just revert. | 01:41 |
dougwig | armax: of putting any of this into neutron/__init__.py | 01:41 |
armax | dougwig: agreed | 01:42 |
*** manishg has quit IRC | 01:42 | |
otherwiseguy | This whole using processutils thing is really more trouble than it is worth. | 01:42 |
armax | so, otherwiseguy: you think it’s sensible to revert now, and wait for the other fix you’re proposing? | 01:42 |
dougwig | might get close in the short-term by putting it in init for agent/ and server/, though. | 01:42 |
otherwiseguy | armax: 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 |
armax | would that address your concern for the tests? | 01:43 |
armax | I mean, I want revert so that we don’t cause grief to other projects | 01:43 |
otherwiseguy | armax: that works fine for me. | 01:43 |
armax | but at the same time I want to apply the fix that nails the issue you’re experiencing | 01:44 |
otherwiseguy | Sorry to break the world. :) | 01:44 |
*** tomoe_ has quit IRC | 01:44 | |
dougwig | otherwiseguy: it's a big club. :) | 01:44 |
armax | otherwiseguy: I knew when I +2 the change something was going to come up | 01:44 |
armax | :) | 01:44 |
dougwig | otherwiseguy: i think i have a permanent cot there, some days. | 01:44 |
otherwiseguy | I've got about 10+ patches removing the root_helper arg from everywhere that I'm sure will break things as well. | 01:44 |
otherwiseguy | I've been trying hard to avoid it, with the ip_lib change, etc. | 01:45 |
armax | otherwiseguy: that’s fine, the sooner you file them the sooner we fix breakages | 01:45 |
otherwiseguy | But it touches *so much code* I would be surprised if something doesn't break. | 01:45 |
*** zhhuabj has joined #openstack-neutron | 01:45 | |
* HenryG is running out of popcorn | 01:45 | |
armax | so if we’re okay on the action plan, https://review.openstack.org/#/c/155152/ can be approved? | 01:45 |
*** flaviof has quit IRC | 01:45 | |
otherwiseguy | works for me. | 01:45 |
*** stanzgy has joined #openstack-neutron | 01:46 | |
*** ChuckC_ has joined #openstack-neutron | 01:46 | |
armax | ok, as for your fix | 01:46 |
*** r-daneel has quit IRC | 01:46 | |
armax | otherwiseguy: are you going to take care of it, or shall someone else do? | 01:46 |
HenryG | otherwiseguy: I think the root_helper fixes are very low risk | 01:46 |
*** watanabe_isao has quit IRC | 01:47 | |
*** trad511 has joined #openstack-neutron | 01:47 | |
HenryG | armax: revert review imminent? | 01:48 |
dougwig | it's linked about 8 lines up | 01:48 |
otherwiseguy | $ git grep "\(self.root_helper\|root_helper=\)"|wc -l | 01:48 |
otherwiseguy | 467 | 01:48 |
armax | HenryG: https://review.openstack.org/#/c/155152/ | 01:48 |
*** _cjones_ has quit IRC | 01:48 | |
armax | dougwig rage-merged it | 01:48 |
armax | :) | 01:48 |
* HenryG cleans glasses | 01:48 | |
* armax rofl | 01:49 | |
*** ChuckC has quit IRC | 01:49 | |
otherwiseguy | So far, I have that root_helper count down to 184. | 01:50 |
otherwiseguy | And it is making me sleepy. | 01:50 |
armax | otherwiseguy: oh my | 01:50 |
dougwig | wow. | 01:50 |
armax | otherwiseguy: so as for the monkey patching | 01:50 |
*** csoukup has quit IRC | 01:50 | |
otherwiseguy | armax: I can probably work something up tomorrow to try. | 01:50 |
armax | otherwiseguy: ok, so the approach would be two-fold, just so that I got it straight,... | 01:51 |
HenryG | otherwiseguy: If you want I can try to finish up the remaining 184 | 01:51 |
otherwiseguy | armax: Either that or scuttle the processutils.execute conversion, and just add a run_as_root option to our existing utils.execute. | 01:51 |
armax | otherwiseguy: a) move the mains in one place and do the monkeypatching early | 01:51 |
armax | and b) rework your processutils patch to work around the problem? | 01:52 |
*** xuhanp has joined #openstack-neutron | 01:52 | |
*** aranjan has quit IRC | 01:52 | |
otherwiseguy | HenryG: I don't mind finishing it up. Getting pretty close. unit tests are the biggest pain. | 01:52 |
armax | otherwiseguy having said that your change did address some log traces observed in l3 agent | 01:52 |
otherwiseguy | armax: s/and/or/ | 01:52 |
armax | otherwiseguy: that’s why I said ‘and' | 01:53 |
otherwiseguy | If I do a), there won't need to be a b). | 01:54 |
armax | otherwiseguy: ic | 01:54 |
armax | your change did fix https://bugs.launchpad.net/neutron/+bug/1417386 | 01:55 |
openstack | Launchpad 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 |
armax | so I’d be definitely eager to see something else up | 01:55 |
otherwiseguy | So 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 |
otherwiseguy | I could probably send an email to Ihar, he wakes up 8 hours before me. | 01:56 |
armax | otherwiseguy: but how would that fix the issue during test execution? | 01:56 |
otherwiseguy | armax: We'd also do tests/__init__.py | 01:57 |
armax | otherwiseguy: gotcha | 01:57 |
armax | otherwiseguy: I’ll post something on his review | 01:57 |
armax | and link this conversation | 01:57 |
*** tflynn has joined #openstack-neutron | 01:57 | |
otherwiseguy | He's the one that started this monkey_patching fixing business anyway. :) | 01:57 |
otherwiseguy | And 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 |
dougwig | whoever came up with monkey patch as a term deserves a medal. | 01:59 |
*** trad511 has quit IRC | 01:59 | |
otherwiseguy | dougwig: whoever came up with it as a concept deserves a swift kick in the pants. | 01:59 |
armax | otherwiseguy: understood, I posted a bunch of comments on https://review.openstack.org/#/c/153779/ | 01:59 |
dougwig | otherwiseguy: lol. | 02:00 |
otherwiseguy | armax: thanks. | 02:00 |
armax | otherwiseguy, dougwig: I honestly believe that this eventlet monkey business is more trouble than it’s worth | 02:00 |
*** SridharRamaswamy has quit IRC | 02:00 | |
dougwig | i was talking about how cool the term 'monkey patch' sounds. it does lead to too much magic and side effects, in practice. | 02:01 |
otherwiseguy | dougwig: 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 IRC | 02:01 | |
*** tflynn has quit IRC | 02:01 | |
dougwig | i monkey patch the crappy ssl lib all the time. and ruby makes good use of it for meta-programming a lot. | 02:01 |
armax | maybe it’s just me then | 02:02 |
otherwiseguy | dougwig: Here is the evil: https://review.openstack.org/#/c/152761/6/neutron/agent/ovsdb/native/patch_idl.py | 02:02 |
otherwiseguy | armax: I would be very happy to use a much more explicit concurrency solution. | 02:02 |
HenryG | mock is monkey patching on steroids. It has its uses. | 02:02 |
armax | otherwiseguy: you just made my day | 02:03 |
*** r-daneel has joined #openstack-neutron | 02:03 | |
aleksandr_null | Hi guys =) | 02:03 |
dougwig | otherwiseguy: i think i need a cigarette after reading that comment. | 02:03 |
otherwiseguy | I'd even rather use threads natively. | 02:03 |
otherwiseguy | dougwig. :) | 02:03 |
dougwig | and i don't smoke. | 02:03 |
dougwig | :) | 02:03 |
otherwiseguy | It is...not really a nice thing to do. And it is confusing as hell. | 02:04 |
otherwiseguy | But turning O(n) into O(1) is worth a little evil. | 02:04 |
*** shashankhegde has quit IRC | 02:04 | |
otherwiseguy | imho | 02:04 |
dougwig | depends on where that loop is, but sometimes yes. | 02:04 |
armax | dougwig: I can see both sides of the argument | 02:05 |
*** sbfox has joined #openstack-neutron | 02:05 | |
*** achanda has quit IRC | 02:06 | |
*** claudiub has quit IRC | 02:06 | |
flwang1 | xuhanp: ping | 02:06 |
otherwiseguy | dougwig: 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 |
xuhanp | flwang1, pong | 02:06 |
armax | ok folks, I’ll back later, thanks a lot | 02:07 |
flwang1 | xuhanp: did you ever try route move? from one controller to another? | 02:07 |
flwang1 | i mean from on l3 agent to another | 02:07 |
otherwiseguy | armax: thanks for cleaning up my mess. ;) | 02:07 |
*** watanabe_isao has joined #openstack-neutron | 02:07 | |
armax | otherwiseguy: our mess | 02:07 |
xuhanp | flwang1, by which command or API? | 02:07 |
dougwig | armax: +1 | 02:07 |
otherwiseguy | It was always kind of a cross-your-fingers kind of change. | 02:08 |
*** zzzeek has quit IRC | 02:08 | |
flwang1 | l3-agent-router-remove l3-agent-router-add | 02:08 |
xuhanp | oh. router move, not route move | 02:08 |
xuhanp | yes, but a while ago | 02:08 |
*** signed8bit_ZZZzz is now known as signed8bit | 02:11 | |
*** achanda has joined #openstack-neutron | 02:12 | |
*** shashankhegde has joined #openstack-neutron | 02:14 | |
*** Longgeek has joined #openstack-neutron | 02:15 | |
*** salv-orlando has quit IRC | 02:15 | |
*** sbfox has quit IRC | 02:18 | |
*** banix has joined #openstack-neutron | 02:19 | |
*** MasterPiece has quit IRC | 02:19 | |
openstackgerrit | Xu Han Peng proposed openstack/neutron: Fix dist_fip_count problem during router migration https://review.openstack.org/151153 | 02:21 |
*** signed8b_ has joined #openstack-neutron | 02:22 | |
*** signed8bit has quit IRC | 02:24 | |
*** signed8b_ is now known as signed8bit | 02:25 | |
*** pal has joined #openstack-neutron | 02:26 | |
*** MasterPiece has joined #openstack-neutron | 02:28 | |
*** sbfox has joined #openstack-neutron | 02:28 | |
*** sambetts has joined #openstack-neutron | 02:29 | |
*** MasterPiece| has joined #openstack-neutron | 02:29 | |
*** armax has quit IRC | 02:29 | |
*** zzzeek has joined #openstack-neutron | 02:30 | |
*** dims__ has joined #openstack-neutron | 02:32 | |
*** MasterPiece has quit IRC | 02:33 | |
*** erkules_ has joined #openstack-neutron | 02:36 | |
*** erkules has quit IRC | 02:38 | |
*** leunam112 has joined #openstack-neutron | 02:39 | |
*** tomoe_ has joined #openstack-neutron | 02:40 | |
*** zzzeek has quit IRC | 02:40 | |
*** leunam112 has quit IRC | 02:42 | |
*** obondarev_ has joined #openstack-neutron | 02:43 | |
*** sbfox has quit IRC | 02:44 | |
*** xek_ has joined #openstack-neutron | 02:45 | |
*** dsundarraj has quit IRC | 02:46 | |
*** cgoncalv1s has joined #openstack-neutron | 02:47 | |
*** Longgeek has quit IRC | 02:48 | |
*** shakamunyi has joined #openstack-neutron | 02:49 | |
*** fouxm_ has joined #openstack-neutron | 02:50 | |
*** rodrigod` has joined #openstack-neutron | 02:50 | |
*** shashankhegde has quit IRC | 02:52 | |
*** nlahouti has quit IRC | 02:52 | |
*** fouxm has quit IRC | 02:52 | |
*** chrisf has quit IRC | 02:52 | |
*** bjornar has quit IRC | 02:52 | |
*** xek has quit IRC | 02:52 | |
*** rwsu has quit IRC | 02:52 | |
*** obondarev has quit IRC | 02:52 | |
*** cgoncalves has quit IRC | 02:52 | |
*** rodrigods has quit IRC | 02:52 | |
*** chrisf_ has joined #openstack-neutron | 02:52 | |
*** emagana has quit IRC | 02:53 | |
*** Aish has joined #openstack-neutron | 02:53 | |
*** suresh12 has quit IRC | 02:54 | |
*** emagana has joined #openstack-neutron | 02:54 | |
*** Longgeek has joined #openstack-neutron | 02:55 | |
openstackgerrit | Xu Han Peng proposed openstack/neutron: Not creating HA router when no enough l3 agents https://review.openstack.org/154374 | 02:57 |
*** cl__ has quit IRC | 02:57 | |
*** rwsu has joined #openstack-neutron | 02:57 | |
*** jckasper has joined #openstack-neutron | 02:57 | |
*** bjornar has joined #openstack-neutron | 02:58 | |
*** emagana has quit IRC | 02:58 | |
*** david-lyle is now known as david-lyle_afk | 02:59 | |
*** suresh12 has joined #openstack-neutron | 03:00 | |
*** zzzeek has joined #openstack-neutron | 03:04 | |
*** manishg has joined #openstack-neutron | 03:06 | |
*** yamahata has quit IRC | 03:06 | |
*** RichardRaseley has joined #openstack-neutron | 03:07 | |
*** RichardR1seley has joined #openstack-neutron | 03:08 | |
*** manishg_ has joined #openstack-neutron | 03:09 | |
RichardR1seley | exit | 03:10 |
*** RichardR1seley has left #openstack-neutron | 03:10 | |
*** manishg has quit IRC | 03:10 | |
*** manishg has joined #openstack-neutron | 03:11 | |
*** Marga_ has quit IRC | 03:12 | |
*** manishg_ has quit IRC | 03:13 | |
*** manishg_ has joined #openstack-neutron | 03:14 | |
*** manishg__ has joined #openstack-neutron | 03:15 | |
*** manishg has quit IRC | 03:15 | |
*** salv-orlando has joined #openstack-neutron | 03:16 | |
*** manishg has joined #openstack-neutron | 03:16 | |
*** shashankhegde has joined #openstack-neutron | 03:17 | |
*** zzzeek has quit IRC | 03:17 | |
*** manishg_ has quit IRC | 03:18 | |
*** manishg__ has quit IRC | 03:20 | |
*** salv-orlando has quit IRC | 03:21 | |
*** manishg has quit IRC | 03:21 | |
*** MasterPiece| has quit IRC | 03:21 | |
*** lazy_prince is now known as killer_prince | 03:21 | |
*** doude has quit IRC | 03:23 | |
*** doude has joined #openstack-neutron | 03:26 | |
*** EmilienM is now known as EmilienM|afk | 03:28 | |
*** aranjan has joined #openstack-neutron | 03:29 | |
*** evgenyf has joined #openstack-neutron | 03:30 | |
*** evgenyf has quit IRC | 03:34 | |
*** salv-orlando has joined #openstack-neutron | 03:36 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fix FIP agent gw port delete based on external net https://review.openstack.org/145859 | 03:37 |
*** dims__ has quit IRC | 03:37 | |
*** flwang has joined #openstack-neutron | 03:38 | |
*** suresh12 has quit IRC | 03:40 | |
*** RichardRaseley has quit IRC | 03:40 | |
openstackgerrit | Timothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 03:40 |
*** salv-orlando has quit IRC | 03:40 | |
*** salv-orlando has joined #openstack-neutron | 03:41 | |
*** shz has quit IRC | 03:43 | |
*** shz has joined #openstack-neutron | 03:43 | |
*** xgerman has joined #openstack-neutron | 03:47 | |
*** carlp has quit IRC | 03:49 | |
*** sthillma has joined #openstack-neutron | 03:50 | |
*** MasterPiece has joined #openstack-neutron | 03:51 | |
*** yamahata has joined #openstack-neutron | 03:52 | |
*** salv-orlando has quit IRC | 03:57 | |
*** shashankhegde has quit IRC | 03:57 | |
*** Mr0rris0 has quit IRC | 03:58 | |
*** flwang has quit IRC | 03:59 | |
*** achanda has quit IRC | 03:59 | |
*** salv-orlando has joined #openstack-neutron | 04:01 | |
*** achanda has joined #openstack-neutron | 04:02 | |
*** armax has joined #openstack-neutron | 04:04 | |
*** spandhe has quit IRC | 04:05 | |
*** salv-orlando has quit IRC | 04:05 | |
*** harlowja is now known as harlowja_away | 04:08 | |
*** achanda has quit IRC | 04:10 | |
*** Longgeek has quit IRC | 04:10 | |
*** Marga_ has joined #openstack-neutron | 04:16 | |
*** Marga_ has quit IRC | 04:17 | |
*** tvardeman has joined #openstack-neutron | 04:17 | |
*** Marga_ has joined #openstack-neutron | 04:17 | |
*** doude has quit IRC | 04:18 | |
*** killer_prince is now known as lazy_prince | 04:19 | |
*** doude has joined #openstack-neutron | 04:20 | |
*** Longgeek has joined #openstack-neutron | 04:25 | |
*** Longgeek has quit IRC | 04:26 | |
*** vivek-ebay has quit IRC | 04:27 | |
*** tiswanso has quit IRC | 04:27 | |
*** haramaki has joined #openstack-neutron | 04:28 | |
*** xuhanp has quit IRC | 04:30 | |
*** doude has quit IRC | 04:30 | |
*** doude has joined #openstack-neutron | 04:30 | |
*** lazy_prince has quit IRC | 04:34 | |
*** barra204 has joined #openstack-neutron | 04:35 | |
*** xgerman has quit IRC | 04:36 | |
*** salv-orlando has joined #openstack-neutron | 04:37 | |
*** ivar-lazzaro has quit IRC | 04:38 | |
*** MasterPiece| has joined #openstack-neutron | 04:38 | |
*** shakamunyi has quit IRC | 04:39 | |
*** anish_ has joined #openstack-neutron | 04:39 | |
*** MasterPiece has quit IRC | 04:40 | |
*** salv-orlando has quit IRC | 04:41 | |
*** lazy_prif has joined #openstack-neutron | 04:42 | |
*** lazy_prif is now known as lazy_prince | 04:42 | |
*** xuhanp has joined #openstack-neutron | 04:43 | |
*** doude has quit IRC | 04:50 | |
*** suresh12 has joined #openstack-neutron | 04:51 | |
*** doude has joined #openstack-neutron | 04:52 | |
*** barra204 has quit IRC | 04:53 | |
*** r-daneel has quit IRC | 04:54 | |
*** suresh12 has quit IRC | 04:55 | |
*** tflynn has joined #openstack-neutron | 04:57 | |
*** nlahouti has joined #openstack-neutron | 04:59 | |
*** salv-orlando has joined #openstack-neutron | 04:59 | |
*** nlahouti1 has joined #openstack-neutron | 05:00 | |
*** tflynn has quit IRC | 05:01 | |
*** tvardeman has quit IRC | 05:02 | |
*** salv-orlando has quit IRC | 05:02 | |
*** yatin has joined #openstack-neutron | 05:03 | |
*** nlahouti1 has quit IRC | 05:03 | |
*** nlahouti1 has joined #openstack-neutron | 05:03 | |
*** nlahouti has quit IRC | 05:04 | |
*** salv-orlando has joined #openstack-neutron | 05:05 | |
*** barra204 has joined #openstack-neutron | 05:07 | |
*** signed8b_ has joined #openstack-neutron | 05:08 | |
*** trinaths has joined #openstack-neutron | 05:08 | |
*** signed8bit has quit IRC | 05:08 | |
*** anilvenkata has joined #openstack-neutron | 05:08 | |
*** rotbeard has quit IRC | 05:08 | |
*** bobmel_ has joined #openstack-neutron | 05:11 | |
*** salv-orlando has quit IRC | 05:12 | |
*** cipcosma has joined #openstack-neutron | 05:13 | |
*** yshafrir has joined #openstack-neutron | 05:13 | |
*** bobmel has quit IRC | 05:14 | |
*** RichardRaseley has joined #openstack-neutron | 05:14 | |
*** banix has quit IRC | 05:15 | |
*** dsundarraj has joined #openstack-neutron | 05:15 | |
*** salv-orlando has joined #openstack-neutron | 05:15 | |
*** xuhanp has quit IRC | 05:16 | |
*** aranjan_ has joined #openstack-neutron | 05:17 | |
*** signed8b_ has quit IRC | 05:18 | |
*** mwagner_lap has quit IRC | 05:19 | |
*** Aish has left #openstack-neutron | 05:19 | |
*** RichardRaseley has quit IRC | 05:19 | |
*** coolsvap has joined #openstack-neutron | 05:19 | |
*** coolsvap has quit IRC | 05:21 | |
*** aranjan has quit IRC | 05:21 | |
*** dsundarraj has quit IRC | 05:23 | |
*** coolsvap has joined #openstack-neutron | 05:23 | |
*** ajo has joined #openstack-neutron | 05:24 | |
*** amotoki has joined #openstack-neutron | 05:25 | |
*** tflynn has joined #openstack-neutron | 05:26 | |
*** tflynn has joined #openstack-neutron | 05:26 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent https://review.openstack.org/155188 | 05:28 |
*** RichardRaseley has joined #openstack-neutron | 05:28 | |
*** salv-orlando has quit IRC | 05:30 | |
*** xuhanp has joined #openstack-neutron | 05:33 | |
*** dsundarraj has joined #openstack-neutron | 05:35 | |
*** chrisf_ has quit IRC | 05:36 | |
*** chrisf has joined #openstack-neutron | 05:36 | |
*** salv-orlando has joined #openstack-neutron | 05:40 | |
*** anshul has joined #openstack-neutron | 05:41 | |
*** anshul has quit IRC | 05:41 | |
*** anshul has joined #openstack-neutron | 05:42 | |
*** yfried_ has quit IRC | 05:44 | |
*** leenheer has quit IRC | 05:44 | |
*** carl_baldwin has joined #openstack-neutron | 05:46 | |
*** MasterPiece| has quit IRC | 05:51 | |
*** salv-orlando has quit IRC | 05:52 | |
*** achanda has joined #openstack-neutron | 05:52 | |
*** Murali has joined #openstack-neutron | 05:54 | |
*** RichardRaseley has quit IRC | 05:54 | |
*** achanda has quit IRC | 05:56 | |
*** dsundarraj has quit IRC | 05:58 | |
*** irenab has joined #openstack-neutron | 06:00 | |
*** carl_baldwin has quit IRC | 06:04 | |
*** Longgeek has joined #openstack-neutron | 06:07 | |
*** achanda has joined #openstack-neutron | 06:08 | |
*** salv-orlando has joined #openstack-neutron | 06:08 | |
*** crc32 has joined #openstack-neutron | 06:15 | |
*** salv-orlando has quit IRC | 06:15 | |
*** salv-orlando has joined #openstack-neutron | 06:16 | |
*** leunam112 has joined #openstack-neutron | 06:17 | |
*** numan has joined #openstack-neutron | 06:18 | |
*** salv-orlando has quit IRC | 06:18 | |
*** yfauser has joined #openstack-neutron | 06:19 | |
*** yfauser has left #openstack-neutron | 06:20 | |
*** leunam112 has quit IRC | 06:21 | |
*** salv-orlando has joined #openstack-neutron | 06:21 | |
*** cipcosma has quit IRC | 06:24 | |
*** salv-orlando has quit IRC | 06:25 | |
*** rotbeard has joined #openstack-neutron | 06:25 | |
crc32 | http://pastebin.com/cSKFNLvC | 06:26 |
crc32 | maby I'm late but is neutron off master broken? I can't install on a freshly built machine. | 06:26 |
irenab | trinaths: hi | 06:28 |
*** boris-42 has joined #openstack-neutron | 06:33 | |
*** zz_avozza is now known as avozza | 06:34 | |
crc32 | ImportError: neutron.hooks.setup_hook <-- is no one else getting this when trying to "python setup.py install" off neutron master? | 06:34 |
crc32 | I'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 IRC | 06:36 | |
*** gildub_ has quit IRC | 06:36 | |
*** trinaths has quit IRC | 06:36 | |
blogan | crc32: https://review.openstack.org/#/c/155152/ | 06:37 |
blogan | crc32: that fixes it, its still in check and needs to go through the gate | 06:37 |
*** trinaths has joined #openstack-neutron | 06:38 | |
*** dims__ has joined #openstack-neutron | 06:38 | |
crc32 | I 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-neutron | 06:42 | |
*** dims__ has quit IRC | 06:42 | |
crc32 | ok blogan that did it.. neutron installs now. | 06:45 |
*** SridharG has joined #openstack-neutron | 06:46 | |
*** anilvenkata1 has quit IRC | 06:46 | |
*** anilvenkata1 has joined #openstack-neutron | 06:47 | |
*** nplanel_ has quit IRC | 06:48 | |
*** salv-orlando has joined #openstack-neutron | 06:49 | |
*** marios has joined #openstack-neutron | 06:50 | |
*** yfried_ has joined #openstack-neutron | 06:50 | |
*** yfried__ has joined #openstack-neutron | 06:53 | |
*** yfried_ has quit IRC | 06:54 | |
*** yamamoto_ has joined #openstack-neutron | 06:56 | |
*** salv-orlando has quit IRC | 06:56 | |
*** lazy_prince has quit IRC | 06:57 | |
*** thomasem has quit IRC | 06:57 | |
*** killer_prince has joined #openstack-neutron | 06:57 | |
*** killer_prince is now known as lazy_prince | 06:57 | |
*** oomichi_ has joined #openstack-neutron | 06:58 | |
*** flwang has joined #openstack-neutron | 07:01 | |
*** thomasem has joined #openstack-neutron | 07:02 | |
*** haramaki has quit IRC | 07:03 | |
*** haramaki_ has joined #openstack-neutron | 07:03 | |
*** dsundarraj has joined #openstack-neutron | 07:05 | |
*** afazekas has joined #openstack-neutron | 07:06 | |
*** dsundarraj has quit IRC | 07:09 | |
*** shashankhegde has joined #openstack-neutron | 07:10 | |
*** scheuran has joined #openstack-neutron | 07:12 | |
*** yarker has quit IRC | 07:12 | |
*** apuimedo has joined #openstack-neutron | 07:14 | |
*** shashankhegde has quit IRC | 07:17 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Move common test helpers to neutron.test.common https://review.openstack.org/146770 | 07:18 |
*** yarker has joined #openstack-neutron | 07:18 | |
*** Marga_ has quit IRC | 07:20 | |
*** aranjan_ has quit IRC | 07:20 | |
*** irenab has quit IRC | 07:24 | |
*** tflynn has quit IRC | 07:26 | |
*** yfried__ is now known as yfried|afk | 07:26 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/153946 | 07:27 |
*** yfried|afk is now known as yfried__ | 07:29 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc https://review.openstack.org/155013 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager https://review.openstack.org/154969 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge https://review.openstack.org/155014 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from external_process https://review.openstack.org/154959 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove deprecated DEFAULT.root_helper https://review.openstack.org/153066 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Use processutils.execute for process execution https://review.openstack.org/153067 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent https://review.openstack.org/155188 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess https://review.openstack.org/155101 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg for ovs_lib https://review.openstack.org/154914 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Stop using passed root_helper in ip_lib https://review.openstack.org/145829 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager https://review.openstack.org/154980 | 07:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent https://review.openstack.org/155212 | 07:31 |
*** achanda has quit IRC | 07:31 | |
*** avozza is now known as zz_avozza | 07:33 | |
*** leunam112 has joined #openstack-neutron | 07:34 | |
*** ronis has joined #openstack-neutron | 07:34 | |
*** Miouge has joined #openstack-neutron | 07:35 | |
*** kawa2014 has joined #openstack-neutron | 07:36 | |
openstackgerrit | Matthew Thode proposed openstack/neutron: Fixes bug in interface handling of ip_lib.py https://review.openstack.org/154128 | 07:37 |
*** achanda has joined #openstack-neutron | 07:37 | |
*** RichardRaseley has joined #openstack-neutron | 07:39 | |
*** ggnel is now known as 6A4ABF1CZ | 07:41 | |
*** ggnel has joined #openstack-neutron | 07:41 | |
*** vthapar has joined #openstack-neutron | 07:42 | |
*** killer_prince has joined #openstack-neutron | 07:42 | |
*** lazy_prince has quit IRC | 07:43 | |
*** killer_prince is now known as lazy_prince | 07:43 | |
*** RichardRaseley has quit IRC | 07:44 | |
*** kickinz1|afk is now known as kickinz1 | 07:44 | |
*** xek_ has quit IRC | 07:48 | |
*** xek_ has joined #openstack-neutron | 07:48 | |
*** anish_ has quit IRC | 07:48 | |
*** anish_ has joined #openstack-neutron | 07:48 | |
*** anilvenkata has quit IRC | 07:48 | |
*** anilvenkata has joined #openstack-neutron | 07:48 | |
*** bobmel_ has quit IRC | 07:48 | |
*** bobmel_ has joined #openstack-neutron | 07:48 | |
*** yshafrir has quit IRC | 07:48 | |
*** yshafrir has joined #openstack-neutron | 07:48 | |
*** coolsvap has quit IRC | 07:48 | |
*** coolsvap has joined #openstack-neutron | 07:48 | |
*** anshul has quit IRC | 07:48 | |
*** anshul has joined #openstack-neutron | 07:48 | |
*** numan has quit IRC | 07:48 | |
*** numan has joined #openstack-neutron | 07:48 | |
*** boris-42 has quit IRC | 07:49 | |
*** boris-42 has joined #openstack-neutron | 07:49 | |
*** SridharG has quit IRC | 07:49 | |
*** SridharG has joined #openstack-neutron | 07:49 | |
*** anilvenkata1 has quit IRC | 07:49 | |
*** anilvenkata1 has joined #openstack-neutron | 07:49 | |
*** yfried__ has quit IRC | 07:49 | |
*** yfried__ has joined #openstack-neutron | 07:49 | |
*** afazekas has quit IRC | 07:49 | |
*** afazekas has joined #openstack-neutron | 07:49 | |
*** scheuran has quit IRC | 07:49 | |
*** scheuran has joined #openstack-neutron | 07:49 | |
*** Longgeek has quit IRC | 07:53 | |
*** Longgeek has joined #openstack-neutron | 07:53 | |
*** Longgeek has quit IRC | 07:53 | |
*** doude has quit IRC | 07:54 | |
*** doude has joined #openstack-neutron | 07:55 | |
*** irenab has joined #openstack-neutron | 07:58 | |
*** Mr0rris0 has joined #openstack-neutron | 08:01 | |
openstackgerrit | Jamie Lennox proposed openstack/neutron: Make nova notifier work with sessions https://review.openstack.org/142366 | 08:02 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/153946 | 08:03 |
*** Miouge_ has joined #openstack-neutron | 08:05 | |
*** pask81 has joined #openstack-neutron | 08:06 | |
*** Miouge has quit IRC | 08:06 | |
*** Miouge_ is now known as Miouge | 08:06 | |
*** nmagnezi has joined #openstack-neutron | 08:07 | |
*** itzikb has joined #openstack-neutron | 08:07 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: ofagent: Fix port_dead https://review.openstack.org/154332 | 08:09 |
*** pask81 has quit IRC | 08:10 | |
*** salv-orlando has joined #openstack-neutron | 08:11 | |
*** flwang1 has quit IRC | 08:11 | |
*** ygbo has joined #openstack-neutron | 08:12 | |
*** flwang1 has joined #openstack-neutron | 08:14 | |
*** moshele has joined #openstack-neutron | 08:16 | |
*** karimb has joined #openstack-neutron | 08:18 | |
*** pask81 has joined #openstack-neutron | 08:21 | |
*** openstackgerrit has quit IRC | 08:21 | |
*** openstackgerrit has joined #openstack-neutron | 08:22 | |
*** luqas has joined #openstack-neutron | 08:24 | |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update https://review.openstack.org/155223 | 08:24 |
*** pask81 has quit IRC | 08:25 | |
anish_ | ilyashakhat, hi around? | 08:31 |
*** achanda has quit IRC | 08:35 | |
*** pask81 has joined #openstack-neutron | 08:36 | |
*** nyechiel has joined #openstack-neutron | 08:37 | |
*** jlanoux has joined #openstack-neutron | 08:39 | |
*** RichardR1seley has joined #openstack-neutron | 08:40 | |
*** zz_avozza is now known as avozza | 08:41 | |
*** chlong has quit IRC | 08:42 | |
watanabe_isao | exit | 08:42 |
*** watanabe_isao has quit IRC | 08:43 | |
*** RichardR1seley has quit IRC | 08:44 | |
*** devvesa has joined #openstack-neutron | 08:46 | |
*** yamamoto_ has quit IRC | 08:46 | |
*** matrohon has joined #openstack-neutron | 08:46 | |
*** pal has quit IRC | 08:46 | |
*** jlibosva has joined #openstack-neutron | 08:48 | |
*** roeyc has joined #openstack-neutron | 08:53 | |
*** Longgeek has joined #openstack-neutron | 08:54 | |
*** jp_at_hp has joined #openstack-neutron | 08:54 | |
*** jistr|off is now known as jistr | 08:54 | |
*** jpena has joined #openstack-neutron | 08:54 | |
*** pal has joined #openstack-neutron | 08:55 | |
openstackgerrit | Sridhar Gaddam proposed openstack/neutron: Default route missing for IPv6 subnets in HA Router https://review.openstack.org/150787 | 08:59 |
*** flwang has quit IRC | 08:59 | |
*** evgenyf has joined #openstack-neutron | 09:00 | |
*** safchain has joined #openstack-neutron | 09:02 | |
*** tomoe__ has joined #openstack-neutron | 09:03 | |
*** hjensas has joined #openstack-neutron | 09:04 | |
*** mathias_1 has joined #openstack-neutron | 09:05 | |
hjensas | I 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 IRC | 09:05 | |
hjensas | ^^ bacround is research for bussiness usecase. | 09:06 |
*** jpich has joined #openstack-neutron | 09:06 | |
*** Longgeek has quit IRC | 09:07 | |
*** nyechiel_ has joined #openstack-neutron | 09:07 | |
openstackgerrit | watanabe isao proposed openstack/neutron: Validate string length at API level https://review.openstack.org/145660 | 09:08 |
*** jistr has quit IRC | 09:08 | |
*** mathias__ has quit IRC | 09:08 | |
*** nyechiel has quit IRC | 09:09 | |
*** popw has quit IRC | 09:11 | |
*** jistr has joined #openstack-neutron | 09:11 | |
*** enikanorov_ has quit IRC | 09:19 | |
*** enikanorov_ has joined #openstack-neutron | 09:20 | |
*** stanzgy has quit IRC | 09:21 | |
*** jckasper has quit IRC | 09:22 | |
*** yamamoto has joined #openstack-neutron | 09:22 | |
*** juice has quit IRC | 09:22 | |
*** amotoki has quit IRC | 09:23 | |
*** juice has joined #openstack-neutron | 09:24 | |
*** yamamoto has quit IRC | 09:27 | |
*** a_le has quit IRC | 09:28 | |
*** amotoki has joined #openstack-neutron | 09:28 | |
*** haramaki_ has quit IRC | 09:30 | |
*** rotbeard has quit IRC | 09:30 | |
*** zhhuabj has quit IRC | 09:30 | |
*** scheuran_ has joined #openstack-neutron | 09:31 | |
*** scheuran has quit IRC | 09:32 | |
*** yamamoto has joined #openstack-neutron | 09:33 | |
*** armax has quit IRC | 09:33 | |
*** aepifanov has joined #openstack-neutron | 09:36 | |
*** Miouge has quit IRC | 09:37 | |
*** Miouge has joined #openstack-neutron | 09:43 | |
*** nikil89_ has joined #openstack-neutron | 09:44 | |
*** nikil89_ is now known as naggappan | 09:44 | |
*** yamamoto has quit IRC | 09:46 | |
*** Miouge has quit IRC | 09:48 | |
*** flwang has joined #openstack-neutron | 09:49 | |
*** enikanorov_ has quit IRC | 09:51 | |
*** enikanorov_ has joined #openstack-neutron | 09:51 | |
*** flwang has quit IRC | 09:54 | |
*** enikanorov__ has joined #openstack-neutron | 09:56 | |
*** hichihara has quit IRC | 09:56 | |
*** enikanorov_ has quit IRC | 09:57 | |
*** aepifanov has quit IRC | 09:58 | |
*** yamamoto has joined #openstack-neutron | 10:00 | |
*** Miouge has joined #openstack-neutron | 10:00 | |
*** avozza is now known as zz_avozza | 10:02 | |
*** vthapar_ has joined #openstack-neutron | 10:02 | |
*** nyechiel__ has joined #openstack-neutron | 10:03 | |
*** nyechiel_ has quit IRC | 10:03 | |
*** nyechiel__ has quit IRC | 10:03 | |
*** nyechiel__ has joined #openstack-neutron | 10:04 | |
*** vthapar has quit IRC | 10:04 | |
*** Longgeek has joined #openstack-neutron | 10:04 | |
*** vthapar has joined #openstack-neutron | 10:06 | |
*** vthapar_ has quit IRC | 10:07 | |
*** xuhanp has quit IRC | 10:11 | |
*** trinaths has left #openstack-neutron | 10:17 | |
*** aepifanov has joined #openstack-neutron | 10:17 | |
*** zz_avozza is now known as avozza | 10:21 | |
*** yamamoto has quit IRC | 10:21 | |
ilyashakhat | anish_: i'm online | 10:22 |
anish_ | ilyashakhat, hi | 10:26 |
*** RichardR1seley has joined #openstack-neutron | 10:29 | |
*** tomoe__ has quit IRC | 10:32 | |
*** RichardR1seley has quit IRC | 10:34 | |
*** moshele has quit IRC | 10:35 | |
*** Longgeek has quit IRC | 10:38 | |
*** hanzhang has joined #openstack-neutron | 10:42 | |
*** dsundarraj has joined #openstack-neutron | 10:42 | |
*** Longgeek has joined #openstack-neutron | 10:44 | |
*** shz has quit IRC | 10:44 | |
*** yfried__ is now known as yfried|afk | 10:44 | |
*** yfried|afk is now known as yfried__ | 10:45 | |
*** marios has quit IRC | 10:46 | |
*** dsundarraj has quit IRC | 10:46 | |
*** bjornar_ has joined #openstack-neutron | 10:47 | |
*** leunam112 has quit IRC | 10:47 | |
*** ihrachyshka has joined #openstack-neutron | 10:51 | |
*** trinaths has joined #openstack-neutron | 10:52 | |
*** salv-orlando has quit IRC | 10:53 | |
*** yfried__ is now known as yfried|afk | 10:55 | |
*** marios has joined #openstack-neutron | 10:56 | |
*** pal has quit IRC | 10:56 | |
*** leunam112 has joined #openstack-neutron | 10:57 | |
*** yamahata has quit IRC | 10:59 | |
*** pal has joined #openstack-neutron | 11:03 | |
*** rotbeard has joined #openstack-neutron | 11:11 | |
*** moshele has joined #openstack-neutron | 11:12 | |
*** yfried|afk is now known as yfried__ | 11:15 | |
*** dims__ has joined #openstack-neutron | 11:15 | |
*** mathias__ has joined #openstack-neutron | 11:20 | |
*** anilvenkata1 has left #openstack-neutron | 11:22 | |
*** roeyc has quit IRC | 11:22 | |
*** mathias_1 has quit IRC | 11:23 | |
*** inara has quit IRC | 11:27 | |
*** salv-orlando has joined #openstack-neutron | 11:27 | |
*** trinaths has left #openstack-neutron | 11:29 | |
*** yuanying has quit IRC | 11:30 | |
*** inara has joined #openstack-neutron | 11:33 | |
*** markvoelker has quit IRC | 11:34 | |
*** claudiub has joined #openstack-neutron | 11:38 | |
*** pal has quit IRC | 11:41 | |
*** JasonPlank has quit IRC | 11:44 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 11:44 |
*** irenab has quit IRC | 11:44 | |
*** rotbeard has quit IRC | 11:44 | |
*** mathias_1 has joined #openstack-neutron | 11:46 | |
*** rotbeard has joined #openstack-neutron | 11:47 | |
*** rodrigod` is now known as rodrigods | 11:49 | |
*** mathias__ has quit IRC | 11:49 | |
*** mathias_1 has quit IRC | 11:50 | |
*** mathias__ has joined #openstack-neutron | 11:51 | |
*** absubram has quit IRC | 11:52 | |
*** mathias_1 has joined #openstack-neutron | 11:56 | |
*** mathias__ has quit IRC | 11:57 | |
*** amotoki has quit IRC | 11:59 | |
*** aix has quit IRC | 12:01 | |
*** Dark_Matt3r has joined #openstack-neutron | 12:03 | |
*** MasterPiece has joined #openstack-neutron | 12:05 | |
*** pc_m has joined #openstack-neutron | 12:06 | |
*** moshele has quit IRC | 12:07 | |
*** moshele has joined #openstack-neutron | 12:10 | |
*** MasterPiece has quit IRC | 12:12 | |
*** johnbelamaric has quit IRC | 12:15 | |
*** RichardR1seley has joined #openstack-neutron | 12:17 | |
*** karimb has quit IRC | 12:20 | |
*** roeyc has joined #openstack-neutron | 12:21 | |
*** MasterPiece has joined #openstack-neutron | 12:22 | |
*** RichardR1seley has quit IRC | 12:22 | |
*** pavel_bondar has joined #openstack-neutron | 12:22 | |
*** mathias__ has joined #openstack-neutron | 12:26 | |
*** salv-orlando has quit IRC | 12:29 | |
*** mathias_1 has quit IRC | 12:30 | |
*** markvoelker has joined #openstack-neutron | 12:35 | |
*** tellesnobrega_ has joined #openstack-neutron | 12:36 | |
*** pal has joined #openstack-neutron | 12:36 | |
*** dims_ has joined #openstack-neutron | 12:37 | |
*** dims__ has quit IRC | 12:38 | |
*** markvoelker has quit IRC | 12:40 | |
*** yfried__ is now known as yfried|afk | 12:41 | |
*** yfried|afk is now known as yfried__ | 12:44 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add sanity_check for keepalived https://review.openstack.org/151284 | 12:45 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: sanity_check: introduce magic to easily add new version checks https://review.openstack.org/155070 | 12:45 |
*** ollivier has joined #openstack-neutron | 12:46 | |
*** doude has quit IRC | 12:52 | |
*** doude has joined #openstack-neutron | 12:53 | |
*** dims_ has quit IRC | 12:57 | |
*** aix has joined #openstack-neutron | 12:57 | |
*** tellesnobrega_ has quit IRC | 12:58 | |
*** markvoelker has joined #openstack-neutron | 12:59 | |
*** marrusl has joined #openstack-neutron | 13:00 | |
*** irenab has joined #openstack-neutron | 13:02 | |
*** amuller has joined #openstack-neutron | 13:02 | |
*** pask81 has quit IRC | 13:02 | |
*** jckasper has joined #openstack-neutron | 13:02 | |
*** baoli has joined #openstack-neutron | 13:03 | |
*** salv-orlando has joined #openstack-neutron | 13:03 | |
*** yfried has joined #openstack-neutron | 13:04 | |
*** yatin has quit IRC | 13:04 | |
*** yfried__ has quit IRC | 13:04 | |
*** markvoelker has quit IRC | 13:05 | |
*** dims__ has joined #openstack-neutron | 13:05 | |
*** haramaki has joined #openstack-neutron | 13:06 | |
*** lazy_prince is now known as killer_prince | 13:07 | |
*** killer_prince is now known as lazy_prince | 13:08 | |
*** alexpilotti has joined #openstack-neutron | 13:09 | |
*** jckasper has quit IRC | 13:10 | |
*** thorst has joined #openstack-neutron | 13:10 | |
*** jckasper has joined #openstack-neutron | 13:10 | |
*** EmilienM|afk is now known as EmilienM | 13:11 | |
*** mwagner_lap has quit IRC | 13:11 | |
*** dims__ has quit IRC | 13:13 | |
*** yfried_ has joined #openstack-neutron | 13:14 | |
*** yfried has quit IRC | 13:14 | |
*** erkules_ is now known as erkules | 13:19 | |
*** dims__ has joined #openstack-neutron | 13:19 | |
*** oomichi_ has quit IRC | 13:23 | |
*** yshafrir has quit IRC | 13:25 | |
*** ZZelle has quit IRC | 13:26 | |
*** ZZelle has joined #openstack-neutron | 13:26 | |
*** tomoe_ has joined #openstack-neutron | 13:31 | |
*** karimb has joined #openstack-neutron | 13:31 | |
*** scheuran_ has quit IRC | 13:33 | |
*** tellesnobrega has joined #openstack-neutron | 13:37 | |
*** jgjl has joined #openstack-neutron | 13:38 | |
*** crose has joined #openstack-neutron | 13:39 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Cleanup in keepalived tests https://review.openstack.org/154925 | 13:39 |
amuller | salv-orlando: ^ | 13:41 |
*** pask81 has joined #openstack-neutron | 13:43 | |
*** jgjl has quit IRC | 13:47 | |
*** radez_g0n3 is now known as radez | 13:48 | |
*** amotoki has joined #openstack-neutron | 13:48 | |
*** jgjl has joined #openstack-neutron | 13:49 | |
*** Mr0rris0 has joined #openstack-neutron | 13:49 | |
*** Mr0rris0 has quit IRC | 13:49 | |
*** Mr0rris0 has joined #openstack-neutron | 13:50 | |
*** Mr0rris0 has joined #openstack-neutron | 13:50 | |
*** Mr0rris0 has joined #openstack-neutron | 13:51 | |
*** Mr0rris0 has quit IRC | 13:51 | |
*** flaviof has joined #openstack-neutron | 13:51 | |
*** Mr0rris0 has joined #openstack-neutron | 13:52 | |
*** Mr0rris0 has quit IRC | 13:52 | |
*** Mr0rris0 has joined #openstack-neutron | 13:52 | |
*** mathias__ has quit IRC | 13:52 | |
*** Mr0rris0 has quit IRC | 13:53 | |
*** Mr0rris0 has joined #openstack-neutron | 13:53 | |
*** Mr0rris0 has quit IRC | 13:54 | |
*** Mr0rris0 has joined #openstack-neutron | 13:54 | |
*** Mr0rris0 has quit IRC | 13:54 | |
*** haramaki has quit IRC | 13:55 | |
*** Mr0rris0 has joined #openstack-neutron | 13:55 | |
*** Mr0rris0 has quit IRC | 13:55 | |
*** Mr0rris0 has joined #openstack-neutron | 13:55 | |
*** anshul has quit IRC | 13:55 | |
*** Mr0rris0 has quit IRC | 13:56 | |
*** Mr0rris0 has joined #openstack-neutron | 13:56 | |
*** Mr0rris0 has quit IRC | 13:56 | |
*** Mr0rris0 has joined #openstack-neutron | 13:57 | |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update https://review.openstack.org/155223 | 13:58 |
openstackgerrit | ZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent https://review.openstack.org/155317 | 13:58 |
*** mwagner_lap has joined #openstack-neutron | 13:58 | |
*** crose has quit IRC | 13:59 | |
*** amotoki_ has joined #openstack-neutron | 14:00 | |
*** haramaki has joined #openstack-neutron | 14:00 | |
*** cgoncalv1s is now known as cgoncalves | 14:01 | |
*** amotoki__ has joined #openstack-neutron | 14:02 | |
*** amotoki has quit IRC | 14:02 | |
*** amotok___ has joined #openstack-neutron | 14:02 | |
*** amotoki_ has quit IRC | 14:05 | |
*** amotoki__ has quit IRC | 14:06 | |
*** RichardR1seley has joined #openstack-neutron | 14:06 | |
openstackgerrit | Lionel 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/154535 | 14:07 |
*** salv-orlando has quit IRC | 14:08 | |
*** dane_leblanc has quit IRC | 14:08 | |
*** salv-orlando has joined #openstack-neutron | 14:08 | |
*** vhoward has joined #openstack-neutron | 14:09 | |
*** RichardR1seley has quit IRC | 14:11 | |
*** kbringard has joined #openstack-neutron | 14:11 | |
*** avozza is now known as zz_avozza | 14:11 | |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: L2 agent RPC add bulk calls for device up and device down https://review.openstack.org/147435 | 14:12 |
*** salv-orlando has quit IRC | 14:13 | |
*** radez is now known as radez_g0n3 | 14:15 | |
*** cference has joined #openstack-neutron | 14:15 | |
openstackgerrit | LinChengyong proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 14:19 |
*** dsundarraj has joined #openstack-neutron | 14:20 | |
*** shwetaap has joined #openstack-neutron | 14:20 | |
*** busterswt has joined #openstack-neutron | 14:20 | |
*** irenab has quit IRC | 14:20 | |
*** dsundarraj has quit IRC | 14:24 | |
*** david-lyle_afk is now known as david-lyle | 14:25 | |
*** julien_ZTE has joined #openstack-neutron | 14:25 | |
*** anish_ has quit IRC | 14:25 | |
*** naggappan has quit IRC | 14:26 | |
ihrachyshka | otherwiseguy, hey. about to send a bunch of 'move main to neutron.cmd' patches | 14:28 |
*** alexpilotti has quit IRC | 14:28 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent https://review.openstack.org/155317 | 14:28 |
*** jgjl has quit IRC | 14:28 | |
*** alexpilotti has joined #openstack-neutron | 14:29 | |
*** humble_ has joined #openstack-neutron | 14:31 | |
*** moha_hunt has joined #openstack-neutron | 14:31 | |
*** yuanying has joined #openstack-neutron | 14:31 | |
*** boris-42 has quit IRC | 14:32 | |
*** dane_leblanc has joined #openstack-neutron | 14:32 | |
otherwiseguy | ihrachyshka: That makes you my Today's Favorite Person. :) | 14:34 |
ihrachyshka | otherwiseguy, half an hour or so, I need to kill some tests before sending it to hell | 14:35 |
ihrachyshka | sorry, gate | 14:35 |
openstackgerrit | Lionel Zerbib proposed openstack/neutron: Add rate limiting security groups rules. https://review.openstack.org/154535 | 14:35 |
*** yuanying has quit IRC | 14:35 | |
*** rotbeard has quit IRC | 14:35 | |
otherwiseguy | The gatekeeper did seem to be kind of demon-ish. | 14:35 |
*** r-daneel has joined #openstack-neutron | 14:36 | |
*** nmagnezi has quit IRC | 14:36 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Setup br-tun in secure mode to avoid broadcast storms https://review.openstack.org/155328 | 14:36 |
*** banix has joined #openstack-neutron | 14:40 | |
*** rotbeard has joined #openstack-neutron | 14:40 | |
ajo | otherwiseguy ^ ihrachyshka , if you can ;) | 14:41 |
ajo | otherwiseguy, btw, excellent work on abstracting the ovsdb calls :D | 14:41 |
*** boris-42 has joined #openstack-neutron | 14:42 | |
*** localloop127 has joined #openstack-neutron | 14:42 | |
openstackgerrit | venkata anil proposed openstack/neutron: Check for valid local_ip for OVS tunnel https://review.openstack.org/154043 | 14:42 |
otherwiseguy | ajo: looking at it. | 14:48 |
*** anilvenkata has left #openstack-neutron | 14:49 | |
otherwiseguy | ajo: so far, the only thing I wonder about is having an option for reset_bridge() that add_bridge() doesn't have. | 14:49 |
ajo | otherwiseguy, yes.. we could add it to add_bridge, but I Wonder if it was a YAGNI :) | 14:49 |
otherwiseguy | Yeah. It's a tradeoff. | 14:49 |
otherwiseguy | On libraries I tend lean more towards consistency. | 14:51 |
ajo | we have other stuff we don't use I think, like set_controller, at least the default ovs plugin | 14:51 |
ajo | otherwiseguy ^ | 14:51 |
*** tellesnobrega has quit IRC | 14:51 | |
ajo | but as you say, it's a library for everybody, consistency is a good point | 14:51 |
otherwiseguy | ajo: I seem to remember some of the other plugins using it. | 14:51 |
ajo | I'm ok with adding it | 14:51 |
*** jprovazn has joined #openstack-neutron | 14:51 | |
*** Murali has quit IRC | 14:52 | |
ajo | point it out on the review and -1 :) so there's a record :D | 14:52 |
otherwiseguy | The ibm and ofa use it. | 14:52 |
ajo | ahh | 14:52 |
ajo | good people | 14:52 |
ajo | what's OFA? | 14:52 |
otherwiseguy | open flow agent | 14:52 |
ajo | hmmm | 14:52 |
ajo | like OVS but as an openflow controller? | 14:52 |
*** manishg has joined #openstack-neutron | 14:52 | |
otherwiseguy | I think so, haven't really looked at it much. | 14:52 |
ajo | otherwiseguy, cool, is a ryu OF agent | 14:53 |
ajo | ryu is cool ;) | 14:53 |
*** tiswanso has joined #openstack-neutron | 14:53 | |
*** tvardeman has joined #openstack-neutron | 14:53 | |
ajo | my opinion is that we may eventually look on it :) | 14:54 |
otherwiseguy | ajo: 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-neutron | 14:54 | |
*** tellesnobrega_ has joined #openstack-neutron | 14:54 | |
ajo | otherwiseguy, I found them a bit weak, but... what could they do? | 14:54 |
*** manishg_ has joined #openstack-neutron | 14:54 | |
ajo | capture the calls on the other end, and just that? | 14:54 |
ajo | that sounds like an antipattern | 14:55 |
otherwiseguy | There's no verification going on other than I guess they don't throw an exception before getting to execute()... | 14:55 |
ajo | probably marun shared already with you: http://googletesting.blogspot.fr/2015/01/testing-on-toilet-change-detector-tests.html | 14:55 |
ajo | yeah | 14:55 |
*** tellesnobrega_ has quit IRC | 14:55 | |
ajo | I would have freaked out before reading that article | 14:55 |
ajo | but.. | 14:56 |
ajo | I couldn't find more to do than ... checking the calls are being done | 14:56 |
*** manishg__ has joined #openstack-neutron | 14:56 | |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 14:56 |
ajo | may be with secure mode I Can mock the secure mode ovsdb call, and make sure it's done | 14:56 |
otherwiseguy | Yeah, they're the kind of tests I wouldn't even have. They're mostly duplicated by functional tests. | 14:56 |
otherwiseguy | Not sure if I have a reset_bridge functional test. | 14:57 |
*** johnbelamaric has joined #openstack-neutron | 14:57 | |
*** manishg has quit IRC | 14:57 | |
ajo | otherwiseguy, aha, may be a functional tests is good | 14:57 |
ajo | good point | 14:57 |
*** mlavalle has joined #openstack-neutron | 14:57 | |
*** johnbelamaric has quit IRC | 14:57 | |
*** johnbelamaric has joined #openstack-neutron | 14:57 | |
*** manishg has joined #openstack-neutron | 14:57 | |
*** rotbeard has quit IRC | 14:57 | |
*** carl_baldwin has joined #openstack-neutron | 14:57 | |
ajo | we could create bridge in standard mode -> check it has the "NORMAL" rule, | 14:58 |
ajo | create in secure mode -> no rule should be | 14:58 |
*** jasondotstar has quit IRC | 14:58 | |
amuller | but then you're testing OVS and not Neutron | 14:58 |
*** manish___ has joined #openstack-neutron | 14:59 | |
*** manishg_ has quit IRC | 14:59 | |
*** haramaki has quit IRC | 14:59 | |
ajo | amuller, I'm testing that my neutron interface is doing what I want to OVS | 15:00 |
ajo | amuller, or may be I can check the ovs-vsctl output, and check that the specific bridge was set as secure | 15:00 |
ajo | (openflow rules appart) | 15:00 |
*** manishg_ has joined #openstack-neutron | 15:00 | |
*** apuimedo has quit IRC | 15:00 | |
*** manishg__ has quit IRC | 15:00 | |
amuller | meh, 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 cheap | 15:01 |
*** manishg has quit IRC | 15:01 | |
*** manishg has joined #openstack-neutron | 15:02 | |
*** SridharG has left #openstack-neutron | 15:03 | |
*** manish___ has quit IRC | 15:03 | |
*** manishg__ has joined #openstack-neutron | 15:03 | |
*** pal has quit IRC | 15:04 | |
*** manishg_ has quit IRC | 15:05 | |
*** maishsk has joined #openstack-neutron | 15:05 | |
*** manishg_ has joined #openstack-neutron | 15:05 | |
*** pal has joined #openstack-neutron | 15:05 | |
*** manishg has quit IRC | 15:06 | |
*** prad has joined #openstack-neutron | 15:06 | |
*** signed8bit has joined #openstack-neutron | 15:06 | |
*** manishg has joined #openstack-neutron | 15:06 | |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 15:06 |
ajo | amuller, and what if I added the argument, but the result is not getting the bridge in secure mode? | 15:07 |
ajo | and we believe it actually is | 15:07 |
*** manishg__ has quit IRC | 15:08 | |
ajo | I 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 abstraction | 15:08 |
ajo | otherwiseguy ^ | 15:08 |
ajo | so... we either add a functional test, or ... I test it manually | 15:08 |
*** dboik has joined #openstack-neutron | 15:08 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce ip address monitor https://review.openstack.org/149647 | 15:09 |
*** manishg_ has quit IRC | 15:09 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Change metadata driver unit tests to use monitored spawn https://review.openstack.org/155058 | 15:09 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 15:09 |
*** manishg_ has joined #openstack-neutron | 15:09 | |
*** manishg has quit IRC | 15:11 | |
*** marun has joined #openstack-neutron | 15:12 | |
*** manishg has joined #openstack-neutron | 15:12 | |
*** lazy_prince is now known as killer_prince | 15:13 | |
*** zzzeek has joined #openstack-neutron | 15:13 | |
*** manishg_ has quit IRC | 15:14 | |
otherwiseguy | ajo: 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 IRC | 15:14 | |
*** salv-orlando has joined #openstack-neutron | 15:14 | |
otherwiseguy | Functional 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 |
otherwiseguy | I could go either way. Any OVS functional test is really going to be two, one for each backend. | 15:15 |
ajo | otherwiseguy, functional seems like a very good way to do it, | 15:16 |
ajo | it saves me the manual.verification | 15:16 |
ajo | and, needing manual verification is always bad | 15:16 |
ajo | otherwiseguy, do we have the native ovsdb backend already in tree? | 15:16 |
*** manishg has quit IRC | 15:16 | |
otherwiseguy | Do the other callers of reset_bridge() also need to be u pdated to pass secure mode? | 15:17 |
*** RichardR1seley has joined #openstack-neutron | 15:17 | |
*** pal has joined #openstack-neutron | 15:17 | |
ajo | otherwiseguy, br-int calls on set-secure | 15:17 |
otherwiseguy | ajo: not quite, it's up for review and we're working out some dep issues. | 15:17 |
*** salv-orlando has quit IRC | 15:17 | |
ajo | but may be it could benefit from re-creating directly into secure | 15:17 |
*** jobewan has joined #openstack-neutron | 15:17 | |
ajo | instead of doing it on different steps (avoiding race conditions) | 15:17 |
ajo | otherwiseguy ahh.. looking forward to see it roll :D | 15:18 |
otherwiseguy | Maru has a test patch I need to depend on that does the python-openvswitch installation for the gate. | 15:18 |
ajo | otherwiseguy | 15:18 |
ajo | look at line 725 here: https://review.openstack.org/#/c/155328/1/neutron/plugins/openvswitch/agent/ovs_neutron_agent.py | 15:18 |
ajo | a very good reason to add secure_mode param to create / add bridge ... | 15:19 |
*** dsundarraj has joined #openstack-neutron | 15:19 | |
*** RichardR1seley has quit IRC | 15:22 | |
*** rkukura has quit IRC | 15:22 | |
*** nplanel_ has joined #openstack-neutron | 15:22 | |
otherwiseguy | ajo: 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" :p | 15:22 |
otherwiseguy | This is why it takes me forever to get anything done. :p | 15:22 |
otherwiseguy | I seem to try to solve problems depth-first. | 15:23 |
ajo | otherwiseguy, but this is why you get something much better in the end :D | 15:23 |
ajo | I 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_jgrimm | 15:24 | |
otherwiseguy | ajo: "I love deadlines. I love the whooshing noise the make as they go by." | 15:24 |
otherwiseguy | -- Douglas Adams | 15:24 |
*** rushil has joined #openstack-neutron | 15:25 | |
ajo | otherwiseguy, that's reality.. :) | 15:27 |
*** cipcosma has joined #openstack-neutron | 15:27 | |
ajo | deadlines go away, and we forget of the long term ;) | 15:27 |
*** dboik_ has joined #openstack-neutron | 15:28 | |
*** numan has quit IRC | 15:28 | |
*** lnxnut_ has joined #openstack-neutron | 15:28 | |
*** dboik_ has quit IRC | 15:28 | |
*** karimb has quit IRC | 15:28 | |
*** dboik_ has joined #openstack-neutron | 15:29 | |
*** moshele has quit IRC | 15:29 | |
*** jpich has quit IRC | 15:30 | |
*** dboik has quit IRC | 15:31 | |
*** lnxnut has quit IRC | 15:32 | |
*** rkukura has joined #openstack-neutron | 15:33 | |
*** irenab has joined #openstack-neutron | 15:34 | |
*** hjensas has quit IRC | 15:34 | |
*** baoli has quit IRC | 15:34 | |
*** vthapar has quit IRC | 15:35 | |
*** baoli_ has joined #openstack-neutron | 15:35 | |
*** achanda has joined #openstack-neutron | 15:36 | |
*** cipcosma has quit IRC | 15:37 | |
*** lnxnut_ has quit IRC | 15:37 | |
* russellb passes root_helper to otherwiseguy | 15:39 | |
* otherwiseguy waves at russellb | 15:40 | |
otherwiseguy | and glares a little | 15:41 |
otherwiseguy | :) | 15:41 |
russellb | umadbro? | 15:41 |
amuller | lol | 15:41 |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 15:42 |
openstackgerrit | Pavel Bondar proposed openstack/neutron: wip ipam reference driver https://review.openstack.org/150485 | 15:42 |
*** Networkn3rd has joined #openstack-neutron | 15:42 | |
openstackgerrit | LiFengkai proposed openstack/neutron-specs: implements: blueprint open-wan-as-a-service https://review.openstack.org/155364 | 15:43 |
ihrachyshka | otherwiseguy, re version checks. so it's all about balance between ease of the check and 'pureness' of the check. | 15:44 |
*** achanda has quit IRC | 15:44 | |
*** crc32 has quit IRC | 15:45 | |
otherwiseguy | ihrachyshka: 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 |
ihrachyshka | otherwiseguy, 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 IRC | 15:45 | |
amuller | ihrachyshka: otherwiseguy: thinking about it it's actually not difficult adding the necessary keepalived ipv6 default gateway sanity check | 15:45 |
amuller | dunno about the dnsmasq one | 15:45 |
otherwiseguy | ihrachyshka: That sounds like exactly the kind of feature one would backport to an older version that was in long term support. :p | 15:47 |
*** ParsectiX has joined #openstack-neutron | 15:47 | |
ihrachyshka | otherwiseguy, redhat was broken because version checks were in runtime | 15:47 |
ihrachyshka | otherwiseguy, so rebase could break | 15:48 |
otherwiseguy | ihrachyshka: 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' wat | 15:48 |
*** amotok___ has quit IRC | 15:48 | |
otherwiseguy | The sanity check failing, when a distro would actually work breaks sanity checking. | 15:49 |
ihrachyshka | there are arguments to disable specific checks if downstream assumes it's handled | 15:49 |
otherwiseguy | It makes it completely unusable. | 15:49 |
otherwiseguy | The whole point is to be able to avoid having to make downstream changes. | 15:49 |
*** kickinz1 is now known as kickinz1|afk | 15:49 | |
*** xgerman has joined #openstack-neutron | 15:49 | |
*** moshele has joined #openstack-neutron | 15:50 | |
otherwiseguy | It'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-neutron | 15:52 | |
openstackgerrit | Merged openstack/neutron: Scope state reporting rpc api using a messaging namespace https://review.openstack.org/154670 | 15:53 |
amuller | russellb: I'm getting back to this patch: https://review.openstack.org/#/c/125339/9 | 15:53 |
amuller | russellb: so I'm going to first push a patch that removes the unused RPC handler | 15:53 |
*** yfried_ is now known as yfried|afk | 15:54 | |
amuller | russellb: 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.4 | 15:54 |
*** salv-orlando has joined #openstack-neutron | 15:54 | |
amuller | russellb: 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.3 | 15:55 |
amuller | would clients connected be affected, even though they don't have any 1.4 methods? | 15:55 |
*** jlibosva has quit IRC | 15:55 | |
amuller | maybe I should just leave the version as 1.4 | 15:56 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes https://review.openstack.org/142863 | 15:56 |
*** kbringard has quit IRC | 15:57 | |
*** jlibosva has joined #openstack-neutron | 15:58 | |
ihrachyshka | otherwiseguy, ok, thanks for feedback, I will digest it later. I'll think how we can handle dnsmasq case easily | 15:58 |
pc_m | carl_baldwin: ping | 15:59 |
*** RichardR1seley has joined #openstack-neutron | 16:00 | |
*** tomoe_ has quit IRC | 16:00 | |
*** jgjl has joined #openstack-neutron | 16:00 | |
*** doude has quit IRC | 16:01 | |
carl_baldwin | pc_m: pong | 16:01 |
pc_m | Can 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 | |
ihrachyshka | otherwiseguy, 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 mess | 16:02 |
*** JPtheJP has joined #openstack-neutron | 16:02 | |
russellb | amuller: 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 all | 16:03 |
otherwiseguy | ihrachyshka: oh my. Lots of patches. | 16:03 |
*** doude has joined #openstack-neutron | 16:03 | |
otherwiseguy | ihrachyshka: maybe we could talk to dougwig/armax and see what they'd prefer? | 16:03 |
*** carlp has joined #openstack-neutron | 16:03 | |
ihrachyshka | otherwiseguy, 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 |
otherwiseguy | Since they will probably be interested in reviewing. | 16:03 |
amuller | russellb: ack | 16:03 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 16:04 |
ihrachyshka | otherwiseguy, 'interesting' | 16:04 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 16:04 |
russellb | amuller: feel free to ping me once you post so i can review | 16:04 |
ihrachyshka | otherwiseguy, sorry, I mean 'interested' | 16:04 |
otherwiseguy | ihrachyshka: :) | 16:04 |
russellb | or you can post right now :) | 16:04 |
amuller | hehe | 16:04 |
*** RichardR1seley has quit IRC | 16:04 | |
*** a_le has joined #openstack-neutron | 16:04 | |
ihrachyshka | as in 'gosh, is it diletant party?' | 16:04 |
carl_baldwin | pc_m: I will look. | 16:04 |
pc_m | carl_baldwin: Thanks. Would like to help the submitter move that commit forward. | 16:04 |
otherwiseguy | ihrachyshka: 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 | :p | 16:05 |
ihrachyshka | otherwiseguy, that's good, I appreciate the feedback | 16:05 |
*** zz_jgrimm is now known as jgrimm | 16:05 | |
ihrachyshka | otherwiseguy, 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 IRC | 16:06 | |
*** a_le has quit IRC | 16:06 | |
ihrachyshka | otherwiseguy, 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 IRC | 16:06 | |
*** claudiub has quit IRC | 16:06 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 16:06 |
*** a_le has joined #openstack-neutron | 16:07 | |
*** tvardeman has quit IRC | 16:07 | |
otherwiseguy | ihrachyshka: I appreciate it. | 16:07 |
*** lnxnut has joined #openstack-neutron | 16:07 | |
otherwiseguy | ihrachyshka: 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 IRC | 16:08 | |
*** anish_ has quit IRC | 16:08 | |
ihrachyshka | armax seems to be off | 16:08 |
ihrachyshka | dougwig, around? | 16:08 |
* ihrachyshka now feels compassion to russellb who sent tens of 'change rpc mixin' patches and was blamed for it | 16:09 | |
otherwiseguy | ihrachyshka: Also, I need to base my patches off yours, and I'm already around 12 deep. :) | 16:09 |
ihrachyshka | 16 patches... ick | 16:09 |
ihrachyshka | otherwiseguy, gosh | 16:09 |
ihrachyshka | otherwiseguy, don't do it before gate passes and we start getting +2's... | 16:10 |
ihrachyshka | I feel we'll have some rebases due to cool nits or whatever | 16:10 |
openstackgerrit | Merged openstack/neutron: Remove deprecated DEFAULT.root_helper https://review.openstack.org/153066 | 16:10 |
*** haramaki has joined #openstack-neutron | 16:10 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: Fix ml2 plugin create_port for notify l3 agent https://review.openstack.org/155317 | 16:10 |
russellb | ihrachyshka: heh yeah i got slapped | 16:11 |
ihrachyshka | ok, no one interested around, so I'm just going to bomb gate as a strawman and we'll see what people have to say | 16:11 |
*** lnxnut has quit IRC | 16:12 | |
* otherwiseguy gets some popcorn | 16:12 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Monkey patch all the code inside neutron/cmd/... https://review.openstack.org/155373 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved neutron.server.main into neutron.cmd.server https://review.openstack.org/155374 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved neutron.agent.l3_agent into neutron.cmd.agents.l3_agent https://review.openstack.org/155375 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved neutron.agent.metadata_agent into neutron.cmd.agents.metadata_agent https://review.openstack.org/155376 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for NEC agent into neutron.cmd https://review.openstack.org/155377 | 16:12 |
*** jorgem has joined #openstack-neutron | 16:12 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for Bigswitch RESTPROXY agent into neutron.cmd https://review.openstack.org/155378 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for IBM SDNVE agent into neutron.cmd https://review.openstack.org/155379 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for HyperV agent into neutron.cmd https://review.openstack.org/155380 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for oneconvergence agent into neutron.cmd https://review.openstack.org/155381 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for OVS agent into neutron.cmd https://review.openstack.org/155382 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for SR-IOV agent into neutron.cmd https://review.openstack.org/155383 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for LB agent into neutron.cmd https://review.openstack.org/155384 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for Cisco configuration agent into neutron.cmd https://review.openstack.org/155385 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for metering agent into neutron.cmd https://review.openstack.org/155386 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved neutron.agent.dhcp_agent to neutron.cmd.agents.dhcp_agent https://review.openstack.org/155387 | 16:12 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for metadata_proxy into neutron/cmd/... https://review.openstack.org/155388 | 16:12 |
* ihrachyshka hides | 16:12 | |
* otherwiseguy didn't have time to make the popcorn | 16:12 | |
ihrachyshka | https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:i_like_to_move_it,n,z | 16:12 |
ihrachyshka | starting from https://review.openstack.org/155373 | 16:13 |
* otherwiseguy wonders if there is a git-review command to download a topic of patches | 16:13 | |
amuller | I guess I'll move my new executable into cmd as well :) | 16:14 |
*** salv-orlando has joined #openstack-neutron | 16:14 | |
otherwiseguy | amuller: doooo it | 16:14 |
*** haramaki has quit IRC | 16:15 | |
otherwiseguy | ihrachyshka: I could have also just not moved to processutils.execute() :p | 16:15 |
*** manishg has joined #openstack-neutron | 16:15 | |
ihrachyshka | amuller, yeah, that's how we want to end | 16:15 |
ihrachyshka | amuller, it's nova way (c) | 16:15 |
ihrachyshka | otherwiseguy, hm, just checkout the last one | 16:16 |
*** jpena_ has joined #openstack-neutron | 16:16 | |
otherwiseguy | Neutron goes Nova. :) | 16:16 |
otherwiseguy | ihrachyshka: well sure, if you want to be smart. | 16:16 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add state_or_sec_group_updated flag in port_update https://review.openstack.org/155223 | 16:16 |
ihrachyshka | otherwiseguy, that move to processutils.execute() was in my todo, so we would hit it sooner or later | 16:16 |
otherwiseguy | :) | 16:16 |
russellb | yeah i usually just check out the last one | 16:17 |
russellb | git review -d <last#> | 16:17 |
*** maishsk has quit IRC | 16:17 | |
ihrachyshka | ok, last one is https://review.openstack.org/#/c/155388/ | 16:17 |
*** rushil has quit IRC | 16:18 | |
*** maishsk has joined #openstack-neutron | 16:18 | |
otherwiseguy | ihrachyshka: Is your patch to do tests/__init__.py still up? | 16:18 |
*** jlibosva has quit IRC | 16:20 | |
*** bjornar_ has quit IRC | 16:21 | |
*** moshele has quit IRC | 16:22 | |
*** jckasper_ has joined #openstack-neutron | 16:22 | |
*** maishsk has quit IRC | 16:22 | |
*** ihrachyshka has quit IRC | 16:22 | |
*** Parsecti_ has joined #openstack-neutron | 16:22 | |
*** sballe__ has joined #openstack-neutron | 16:23 | |
*** jckasper has quit IRC | 16:24 | |
*** ParsectiX has quit IRC | 16:24 | |
*** kbringard has joined #openstack-neutron | 16:25 | |
*** yamahata has joined #openstack-neutron | 16:26 | |
*** claudiub has joined #openstack-neutron | 16:27 | |
*** yeungp has quit IRC | 16:28 | |
*** nplanel_ has quit IRC | 16:29 | |
*** morgabra has quit IRC | 16:29 | |
*** trinaths has joined #openstack-neutron | 16:30 | |
*** jlanoux has quit IRC | 16:31 | |
*** johnbelamaric has quit IRC | 16:31 | |
*** morgabra has joined #openstack-neutron | 16:31 | |
*** johnbelamaric has joined #openstack-neutron | 16:31 | |
*** carl_baldwin has quit IRC | 16:32 | |
*** trinaths1 has joined #openstack-neutron | 16:32 | |
*** zz_avozza is now known as avozza | 16:33 | |
*** a_le has quit IRC | 16:33 | |
*** a_le has joined #openstack-neutron | 16:34 | |
*** imcsk8 has quit IRC | 16:34 | |
*** imcsk8 has joined #openstack-neutron | 16:34 | |
*** trinaths has quit IRC | 16:35 | |
*** avozza is now known as zz_avozza | 16:35 | |
*** marios has quit IRC | 16:36 | |
*** flaviof has quit IRC | 16:37 | |
*** tellesnobrega_ has joined #openstack-neutron | 16:39 | |
*** rook has joined #openstack-neutron | 16:39 | |
*** tellesnobrega_ has quit IRC | 16:39 | |
*** rook is now known as Guest39063 | 16:39 | |
*** pal_ has joined #openstack-neutron | 16:42 | |
*** trinaths1 has left #openstack-neutron | 16:42 | |
*** roeyc has quit IRC | 16:43 | |
*** jgjl has quit IRC | 16:44 | |
*** irenab has quit IRC | 16:44 | |
*** yeungp has joined #openstack-neutron | 16:44 | |
*** pal has quit IRC | 16:45 | |
*** yamahata has quit IRC | 16:45 | |
*** afazekas has quit IRC | 16:46 | |
*** yamahata has joined #openstack-neutron | 16:46 | |
openstackgerrit | Lionel Zerbib proposed openstack/neutron: Add rate limiting security groups rules. https://review.openstack.org/154535 | 16:47 |
*** anshul has joined #openstack-neutron | 16:48 | |
*** vivek-ebay has joined #openstack-neutron | 16:48 | |
*** ChuckC_ has quit IRC | 16:51 | |
*** ChuckC_ has joined #openstack-neutron | 16:51 | |
*** ChuckC_ is now known as ChuckC | 16:51 | |
*** tellesnobrega_ has joined #openstack-neutron | 16:52 | |
*** ihrachyshka has joined #openstack-neutron | 16:53 | |
ihrachyshka | marun, https://review.openstack.org/133934 | 16:53 |
dougwig | ihrachyshka: lol, good show | 16:53 |
ihrachyshka | dougwig, ... :) | 16:54 |
*** tvardeman has joined #openstack-neutron | 16:54 | |
*** lnxnut has joined #openstack-neutron | 16:54 | |
ihrachyshka | otherwiseguy, the very first patch in the series does it | 16:55 |
*** evgenyf has quit IRC | 16:55 | |
ihrachyshka | dougwig, 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 IRC | 16:56 | |
dougwig | it doesn't bother me. nor do long ones for things that are straightforward. | 16:56 |
ihrachyshka | ok, I'll leave those as-is unless someone complains | 16:56 |
*** dims__ has joined #openstack-neutron | 16:57 | |
*** leunam112 has quit IRC | 16:59 | |
*** baoli_ has quit IRC | 17:00 | |
*** anish_ has joined #openstack-neutron | 17:00 | |
*** tflynn has joined #openstack-neutron | 17:00 | |
*** tflynn has quit IRC | 17:00 | |
*** tflynn has joined #openstack-neutron | 17:01 | |
*** apuimedo has joined #openstack-neutron | 17:03 | |
ihrachyshka | dougwig, ok, so I need to respin anyway, so I'll squash some of those | 17:03 |
*** _cjones_ has joined #openstack-neutron | 17:04 | |
ZZelle | ihrachyshka, hi | 17:04 |
ihrachyshka | ZZelle, hi | 17:04 |
ZZelle | ihrachyshka, you should also associate these changes with a bug | 17:05 |
ihrachyshka | ZZelle, do you have number in mind? | 17:05 |
*** RichardR1seley has joined #openstack-neutron | 17:05 | |
ihrachyshka | otherwiseguy, ^^ ? | 17:06 |
openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 17:06 |
otherwiseguy | ihrachyshka: I think I added a neutron impact to the one you opened for oslo. | 17:08 |
ZZelle | ihrachyshka, => https://bugs.launchpad.net/oslo.concurrency/+bug/1418541 | 17:08 |
openstack | Launchpad bug 1418541 in neutron "processutils checks whether stdlib is monkey patched during import" [Undecided,Fix committed] - Assigned to Terry Wilson (otherwiseguy) | 17:08 |
ZZelle | but it does not really feet | 17:09 |
ihrachyshka | ZZelle, right. ok, let's track it there | 17:09 |
dougwig | ihrachyshka: 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 |
otherwiseguy | ZZelle: ihrachyshka : yep, that's the one. It didn't fit exactly, but it was close enough. :) | 17:09 |
*** aranjan has joined #openstack-neutron | 17:09 | |
ihrachyshka | otherwiseguy, good for Related- | 17:09 |
ihrachyshka | dougwig, ack, will do in 1-2 hours | 17:10 |
dougwig | ihrachyshka: ty | 17:10 |
*** SridharRamaswamy has joined #openstack-neutron | 17:10 | |
*** jlibosva has joined #openstack-neutron | 17:10 | |
*** Parsecti_ is now known as ParsectiX | 17:11 | |
*** coolsvap is now known as coolsvap_ | 17:12 | |
*** bobmel has joined #openstack-neutron | 17:13 | |
*** SridharRamaswam1 has joined #openstack-neutron | 17:13 | |
*** thorst has quit IRC | 17:13 | |
*** vthapar has joined #openstack-neutron | 17:13 | |
ihrachyshka | markmcclain, 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 IRC | 17:14 | |
*** bobmel_ has quit IRC | 17:15 | |
*** tflynn has quit IRC | 17:15 | |
*** carl_baldwin has joined #openstack-neutron | 17:15 | |
*** tflynn has joined #openstack-neutron | 17:15 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for metadata_proxy into neutron/cmd/... https://review.openstack.org/155388 | 17:17 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Monkey patch all the code inside neutron/cmd/... https://review.openstack.org/155373 | 17:17 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved several services into neutron.cmd https://review.openstack.org/155374 | 17:17 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for metering agent into neutron.cmd https://review.openstack.org/155386 | 17:17 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for 3party plugins into neutron.cmd https://review.openstack.org/155412 | 17:17 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Moved main() for open plugins into neutron.cmd https://review.openstack.org/155413 | 17:17 |
*** jorgem has quit IRC | 17:17 | |
*** hjensas has joined #openstack-neutron | 17:18 | |
openstackgerrit | Manish Godara proposed openstack/neutron: Add filter for provider network attributes in ML2 https://review.openstack.org/124917 | 17:21 |
ajo | salv-orlando, if you have a moment to take a look: https://review.openstack.org/#/c/155328/ broadcast storm fun on br-tun | 17:22 |
ajo | obondarev_ ^ | 17:22 |
salv-orlando | k | 17:23 |
*** trad511 has joined #openstack-neutron | 17:23 | |
*** banix has quit IRC | 17:25 | |
*** EmilienM is now known as EmilienM|afk | 17:25 | |
*** dsundarr_ has joined #openstack-neutron | 17:27 | |
*** jistr has quit IRC | 17:27 | |
*** salv-orlando has quit IRC | 17:29 | |
*** armax has joined #openstack-neutron | 17:29 | |
*** salv-orlando has joined #openstack-neutron | 17:29 | |
*** apuimedo has quit IRC | 17:29 | |
*** devvesa has quit IRC | 17:30 | |
openstackgerrit | Merged openstack/neutron: Revert "monkey patch stdlib before importing other modules" https://review.openstack.org/155152 | 17:30 |
*** dsundarraj has quit IRC | 17:30 | |
*** pask81 has quit IRC | 17:31 | |
*** aepifanov has quit IRC | 17:34 | |
*** rushil has joined #openstack-neutron | 17:37 | |
*** matrohon has quit IRC | 17:38 | |
ajo | salv-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 IRC | 17:39 | |
*** Marga_ has joined #openstack-neutron | 17:41 | |
*** leenheer has joined #openstack-neutron | 17:41 | |
*** tflynn has joined #openstack-neutron | 17:41 | |
salv-orlando | ajo: yeah I just do not remember very well the behaviour of secure mode and I'm doing some tests now | 17:41 |
salv-orlando | but if you can help me | 17:41 |
ajo | salv-orlando, sure, let me look for the reference | 17:41 |
*** ChuckC has quit IRC | 17:42 | |
salv-orlando | I think with secure mode br-tun never sets up flows unless you i)have a connected controller or ii) do it explictly | 17:42 |
ajo | http://openvswitch.org/support/dist-docs/tutorial/Tutorial.md.txt | 17:42 |
*** emagana has joined #openstack-neutron | 17:42 | |
ajo | look for secure | 17:42 |
*** ChuckC has joined #openstack-neutron | 17:42 | |
*** Marga_ has quit IRC | 17:42 | |
ajo | "For our purpose, this just means that the OpenFlow flow table starts out empty." | 17:42 |
salv-orlando | ajo: what are you quoting? | 17:43 |
ajo | the 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-neutron | 17:43 | |
ajo | ahh | 17:43 |
ajo | correct, | 17:43 |
ajo | what you said is correct | 17:43 |
ajo | no rules unless controller conected or explicitly set | 17:43 |
salv-orlando | ajo: that's fine I got the normal problem. I just want to make sure that we then don't wedge br-tun | 17:43 |
*** spandhe has joined #openstack-neutron | 17:43 | |
ajo | salv-orlando, sure, | 17:43 |
ajo | i tested at least for vxlan, not gre | 17:43 |
ajo | but it looks like it works, actually, we delete all the rules after reseting the bridge | 17:44 |
ajo | which is useless now, let me look for the line where we do that.. | 17:44 |
ajo | but | 17:44 |
ajo | yes, let's make sure there are no more implications | 17:44 |
ajo | the rule stuff is clear | 17:44 |
*** jlibosva has quit IRC | 17:45 | |
*** tflynn has quit IRC | 17:45 | |
ajo | salv-orlando: man 5 ovs-vswitchd.conf.db | 17:46 |
*** tflynn has joined #openstack-neutron | 17:46 | |
ajo | http://www.fpaste.org/184823/42376320/ | 17:46 |
*** johnbelamaric has quit IRC | 17:47 | |
ajo | ok, it seems that we just prevent the normal rule (switch acting as a normal MAC-learning switch) | 17:47 |
salv-orlando | ajo: yes it sounds like this | 17:48 |
ajo | ah, nice it seems like I didn't run relevant tests locally... | 17:49 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes https://review.openstack.org/142863 | 17:49 |
ajo | neutron.tests.unit.openvswitch.test_ovs_tunnel failing because of not expecting the new parameter set | 17:49 |
salv-orlando | probably 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 connected | 17:49 |
*** safchain has quit IRC | 17:50 | |
*** vivek-ebay has quit IRC | 17:50 | |
*** vivek-ebay has joined #openstack-neutron | 17:50 | |
ajo | missleading, yes | 17:51 |
ajo | the actual behavior is that we can really add them via ovs-ofctl | 17:51 |
ajo | manually tested.. | 17:51 |
ajo | but 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 correctly | 17:52 |
*** SumitNaiksatam has joined #openstack-neutron | 17:52 | |
*** anish_ has quit IRC | 17:52 | |
*** marun has quit IRC | 17:52 | |
ajo | btw, I guess that works, since we're using it for br-int | 17:53 |
*** baoli has joined #openstack-neutron | 17:53 | |
*** humble_ has quit IRC | 17:53 | |
*** tellesnobrega_ has quit IRC | 17:53 | |
*** banix has joined #openstack-neutron | 17:54 | |
*** vivek-ebay has quit IRC | 17:54 | |
salv-orlando | ajo: 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 ok | 17:54 |
ajo | that's checked :) | 17:54 |
salv-orlando | I 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-neutron | 17:54 | |
ajo | I checked it to make sure the problem didn't reproduce | 17:55 |
ajo | with cross-node communication | 17:55 |
ajo | etc | 17:55 |
ajo | dhcp, floating ips, separate compute nodes, network nodes... | 17:55 |
*** coolsvap_ is now known as coolsvap | 17:55 | |
*** dsundarr_ has quit IRC | 17:56 | |
ajo | of course, feel free to double-check, as you said, not tested on the gate is always a high risk | 17:56 |
*** jkyle has quit IRC | 17:56 | |
ajo | did we ever had multi node gate tests? | 17:56 |
*** jkyle1 has joined #openstack-neutron | 17:57 | |
*** baoli has quit IRC | 17:57 | |
ajo | to 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 too | 17:57 |
ajo | secure + rules not broken: that is tested | 17:57 |
*** tellesnobrega_ has joined #openstack-neutron | 17:58 | |
*** _cjones_ has quit IRC | 17:59 | |
salv-orlando | ajo: yes that would be enough for me. We have never done multi node testing in the gate unfortunately | 17:59 |
*** anshul has quit IRC | 18:00 | |
*** Youcef has joined #openstack-neutron | 18:00 | |
*** leunam112 has joined #openstack-neutron | 18:00 | |
*** aix has quit IRC | 18:00 | |
*** tiswanso has quit IRC | 18:03 | |
*** leunam112 has quit IRC | 18:04 | |
*** tiswanso_ has joined #openstack-neutron | 18:04 | |
*** claudiub has quit IRC | 18:04 | |
*** jp_at_hp has quit IRC | 18:04 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:06 | |
*** rushil has quit IRC | 18:06 | |
*** ivar-lazzaro has quit IRC | 18:07 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:08 | |
*** rushil has joined #openstack-neutron | 18:08 | |
*** nlahouti1 has left #openstack-neutron | 18:09 | |
*** nlahouti has joined #openstack-neutron | 18:09 | |
*** salv-orlando has quit IRC | 18:10 | |
*** salv-orlando has joined #openstack-neutron | 18:10 | |
*** salv-orl_ has joined #openstack-neutron | 18:12 | |
*** salv-orl_ has quit IRC | 18:12 | |
*** tflynn has quit IRC | 18:12 | |
*** salv-orl_ has joined #openstack-neutron | 18:13 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method https://review.openstack.org/154103 | 18:13 |
*** dboik_ has quit IRC | 18:13 | |
openstackgerrit | Merged openstack/neutron: Check conntrack rule cleanup on floating IP disassociate https://review.openstack.org/133934 | 18:13 |
pc_m | Wondering 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_hook | 18:14 |
pc_m | I 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 IRC | 18:16 | |
*** s3wong has joined #openstack-neutron | 18:16 | |
pc_m | It 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-neutron | 18:17 | |
pc_m | Anyone have any ideas? | 18:17 |
*** _cjones_ has joined #openstack-neutron | 18:17 | |
*** aranjan has quit IRC | 18:18 | |
pc_m | Tox output... http://paste.openstack.org/show/172257 | 18:18 |
*** baoli has joined #openstack-neutron | 18:18 | |
dougwig | that should've been fixed with 155152 (neutron) | 18:19 |
dougwig | pc_m: ^^ | 18:19 |
openstackgerrit | Merged openstack/neutron: keepalived: use sh instead of bash for notifier scripts https://review.openstack.org/154931 | 18:19 |
*** salv-orlando has quit IRC | 18:20 | |
*** dboik has joined #openstack-neutron | 18:20 | |
*** dboik has quit IRC | 18:20 | |
*** dboik has joined #openstack-neutron | 18:20 | |
pc_m | dougwig: Thanks! Looks like it just merged. I'll try refreshing my neutron repo. | 18:20 |
*** ZZelle_ has joined #openstack-neutron | 18:20 | |
*** harlowja_away is now known as harlowja | 18:21 | |
*** baoli has quit IRC | 18:22 | |
*** EmilienM|afk is now known as EmilienM | 18:22 | |
*** MasterPiece has quit IRC | 18:23 | |
*** tflynn has joined #openstack-neutron | 18:23 | |
*** dane_leblanc has quit IRC | 18:23 | |
*** SridharRamaswam1 has quit IRC | 18:24 | |
*** shashankhegde has joined #openstack-neutron | 18:24 | |
*** localloop127 has quit IRC | 18:25 | |
*** achanda has joined #openstack-neutron | 18:25 | |
*** localloop127 has joined #openstack-neutron | 18:27 | |
openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 18:28 |
*** emagana has quit IRC | 18:29 | |
*** johnbelamaric has joined #openstack-neutron | 18:29 | |
*** bjornar_ has joined #openstack-neutron | 18:29 | |
*** emagana has joined #openstack-neutron | 18:29 | |
*** yeungp has quit IRC | 18:30 | |
*** zz_avozza is now known as avozza | 18:30 | |
*** emagana has quit IRC | 18:30 | |
*** emagana has joined #openstack-neutron | 18:30 | |
*** tellesnobrega_ has quit IRC | 18:31 | |
*** annegent_ has joined #openstack-neutron | 18:32 | |
*** boris-42 has quit IRC | 18:32 | |
*** dboik has quit IRC | 18:32 | |
*** yfried_ has quit IRC | 18:34 | |
*** dsundarraj has joined #openstack-neutron | 18:35 | |
*** tellesnobrega_ has joined #openstack-neutron | 18:35 | |
*** jaypipes has quit IRC | 18:36 | |
*** raildo has joined #openstack-neutron | 18:37 | |
*** dboik has joined #openstack-neutron | 18:38 | |
*** dsundarraj has quit IRC | 18:38 | |
*** baoli has joined #openstack-neutron | 18:40 | |
*** nyechiel__ has joined #openstack-neutron | 18:40 | |
*** yeungp has joined #openstack-neutron | 18:40 | |
*** dsundarraj has joined #openstack-neutron | 18:40 | |
*** tiswanso_ has quit IRC | 18:41 | |
*** rushil_ has joined #openstack-neutron | 18:42 | |
*** nyechiel__ has quit IRC | 18:42 | |
*** nyechiel__ has joined #openstack-neutron | 18:43 | |
*** avozza is now known as zz_avozza | 18:44 | |
*** zz_avozza is now known as avozza | 18:44 | |
*** rushil has quit IRC | 18:44 | |
*** nyechiel__ has quit IRC | 18:46 | |
*** nyechiel__ has joined #openstack-neutron | 18:46 | |
*** tflynn has quit IRC | 18:48 | |
*** tflynn has joined #openstack-neutron | 18:48 | |
*** SridharRamaswamy has joined #openstack-neutron | 18:49 | |
*** dane_leblanc has joined #openstack-neutron | 18:49 | |
*** MasterPiece has joined #openstack-neutron | 18:49 | |
*** ihrachyshka has quit IRC | 18:50 | |
*** devvesa has joined #openstack-neutron | 18:50 | |
openstackgerrit | German Eichberger proposed openstack/neutron-vpnaas: Reorder Neutron import statements in file https://review.openstack.org/142507 | 18:51 |
*** Longgeek has quit IRC | 18:51 | |
*** nmagnezi has joined #openstack-neutron | 18:51 | |
*** Longgeek has joined #openstack-neutron | 18:52 | |
*** kbringard has quit IRC | 18:52 | |
*** Marga_ has quit IRC | 18:53 | |
*** ParsectiX has quit IRC | 18:53 | |
*** Marga_ has joined #openstack-neutron | 18:53 | |
*** ParsectiX has joined #openstack-neutron | 18:53 | |
*** ihrachyshka has joined #openstack-neutron | 18:54 | |
*** dboik has quit IRC | 18:54 | |
*** apuimedo has joined #openstack-neutron | 18:55 | |
*** dboik has joined #openstack-neutron | 18:55 | |
*** Longgeek has quit IRC | 18:56 | |
anteaya | dougwig: why are import statements being reordered? | 18:56 |
anteaya | dougwig: I had thought that the order of import statements was important to keep to avoid package collision | 18:57 |
anteaya | dougwig: am I missing something in my understanding? | 18:57 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Raise timeout for test_conntrack_disassociate_fip https://review.openstack.org/155445 | 18:57 |
*** coolsvap is now known as coolsvap_ | 18:57 | |
*** trad5111 has joined #openstack-neutron | 18:57 | |
*** doude has quit IRC | 18:58 | |
*** tiswanso has joined #openstack-neutron | 18:58 | |
*** Marga_ has quit IRC | 18:59 | |
*** avozza is now known as zz_avozza | 18:59 | |
*** Marga_ has joined #openstack-neutron | 18:59 | |
*** _cjones_ has quit IRC | 19:00 | |
*** johnbelamaric has quit IRC | 19:00 | |
*** trad511 has quit IRC | 19:00 | |
*** johnbelamaric has joined #openstack-neutron | 19:00 | |
*** tomoe_ has joined #openstack-neutron | 19:01 | |
*** pal_ has quit IRC | 19:01 | |
*** tflynn has quit IRC | 19:01 | |
*** tflynn has joined #openstack-neutron | 19:02 | |
*** _cjones_ has joined #openstack-neutron | 19:03 | |
*** tomoe_ has quit IRC | 19:05 | |
*** dboik has quit IRC | 19:05 | |
*** dboik has joined #openstack-neutron | 19:06 | |
*** doude has joined #openstack-neutron | 19:06 | |
*** thedodd has joined #openstack-neutron | 19:06 | |
*** Marga_ has quit IRC | 19:08 | |
*** suresh12 has joined #openstack-neutron | 19:08 | |
*** Marga_ has joined #openstack-neutron | 19:08 | |
*** Marga_ has quit IRC | 19:09 | |
*** Marga_ has joined #openstack-neutron | 19:09 | |
HenryG | anteaya: Where are import statements being reordered? | 19:09 |
*** tflynn has quit IRC | 19:09 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources https://review.openstack.org/153422 | 19:09 |
ZZelle_ | salv-orl_, hi | 19:10 |
*** achanda has quit IRC | 19:11 | |
*** alexpilotti has quit IRC | 19:11 | |
HenryG | anteaya: I guess you are referring to https://review.openstack.org/142507 | 19:12 |
*** banix has quit IRC | 19:12 | |
*** moshele has joined #openstack-neutron | 19:12 | |
HenryG | anteaya: Import order does not matter in Python (except for some special cases like monkey patching) | 19:13 |
*** johnbelamaric has quit IRC | 19:13 | |
HenryG | anteaya: We have some hacking rules to group and alphabetically order import statements | 19:13 |
*** moshele has quit IRC | 19:14 | |
*** banix has joined #openstack-neutron | 19:14 | |
*** baoli has quit IRC | 19:15 | |
prometheanfire | anyone around for a +2 https://review.openstack.org/#/c/154128/ | 19:15 |
*** kawa2014 has quit IRC | 19:15 | |
anteaya | HenryG: sorry was having lunch, yes that was the patch to which I was referring | 19:16 |
anteaya | HenryG: okay thanks | 19:16 |
*** mwagner_lap has joined #openstack-neutron | 19:18 | |
kevinbenton | ihrachyshka: hey, don't you think 100 seconds is a little high? | 19:18 |
openstackgerrit | Sridhar Ramaswamy proposed openstack/neutron-vpnaas: Implementation of Brocade Vyatta VPNaaS Plugin https://review.openstack.org/154720 | 19:19 |
kevinbenton | ihrachyshka: this test is waiting for a netcat timeout, right? | 19:19 |
*** aranjan has joined #openstack-neutron | 19:20 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Refactor management of namespaces in the L3 Agent https://review.openstack.org/147744 | 19:21 |
*** apuimedo has quit IRC | 19:21 | |
kevinbenton | is the next OpenStack release officially called Liberty? | 19:21 |
*** vthapar has quit IRC | 19:22 | |
HenryG | kevinbenton: yup :( | 19:22 |
*** devvesa has quit IRC | 19:22 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from SecurityGroupAgentRpc https://review.openstack.org/155013 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IptablesManager https://review.openstack.org/154969 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from linuxbridge https://review.openstack.org/155014 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent https://review.openstack.org/155212 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from external_process https://review.openstack.org/154959 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Add run_as_root option to utils.execute https://review.openstack.org/153067 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent https://review.openstack.org/155188 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess https://review.openstack.org/155101 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg for ovs_lib https://review.openstack.org/154914 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Stop using passed root_helper in ip_lib https://review.openstack.org/145829 | 19:22 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from IpsetManager https://review.openstack.org/154980 | 19:22 |
markmcclain | ihrachyshka: yeah I think the discussion part is needed | 19:22 |
*** busterswt has quit IRC | 19:22 | |
otherwiseguy | ihrachyshka: ok, I have untangled my root_helper stuff from the processutils.execute/monkey_patch mess. :) | 19:23 |
kevinbenton | HenryG: sounds very patriotic | 19:24 |
kevinbenton | otherwiseguy, markmcclain: we are bombarding ihrachyshka with IRC messages :) | 19:25 |
kevinbenton | once he gets back to his compute he will think there is a crisis | 19:25 |
markmcclain | haha | 19:25 |
ihrachyshka | kevinbenton, well, I stole 100 from another func test that uses the assert | 19:25 |
otherwiseguy | kevinbenton: and he should be drinking a beer right now, getting kind of late there. | 19:25 |
*** nmagnezi has quit IRC | 19:26 | |
carl_baldwin | mlavalle: ping | 19:26 |
ihrachyshka | markmcclain, ack, just sent an email: http://lists.openstack.org/pipermail/openstack-dev/2015-February/056790.html | 19:26 |
* otherwiseguy owes ihrachyshka several beers by now | 19:26 | |
*** emagana has quit IRC | 19:26 | |
kevinbenton | ihrachyshka: can we decrease the netcat timeout time just in the interest of fast functional tests when possible? | 19:26 |
ihrachyshka | kevinbenton, it takes ~12 secs on my machine | 19:26 |
kevinbenton | ihrachyshka: or where is the bottleneck? | 19:26 |
ihrachyshka | kevinbenton, it's always crisis, it's neutron | 19:26 |
*** emagana has joined #openstack-neutron | 19:26 | |
markmcclain | ihrachyshka: awesome thanks | 19:27 |
kevinbenton | ihrachyshka: i like to call it realtime development | 19:27 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add callbacks-based system to Neutron https://review.openstack.org/153774 | 19:27 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 19:27 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 base and VPN service plugins during router operations https://review.openstack.org/153776 | 19:27 |
ihrachyshka | kevinbenton, I can look into it tomorrow, though we need band-aid now | 19:27 |
kevinbenton | ihrachyshka: ack | 19:27 |
*** dsundarraj has quit IRC | 19:27 | |
*** dsundarraj has joined #openstack-neutron | 19:27 | |
ihrachyshka | otherwiseguy, that series ^^ makes me feel better (I'm not the only one bombing gate with dilletante patches) | 19:28 |
otherwiseguy | ihrachyshka: :) If anyone wants to review it as a single patch, I'll happily squash it for them...and watch them go insane. | 19:28 |
ihrachyshka | otherwiseguy, ok, I will not forget that several beers thing if I get to CA | 19:29 |
ihrachyshka | otherwiseguy, no no no, please don't | 19:29 |
otherwiseguy | I'm surprised I'm still marginally functioning. They are not particularly interesting patches to write. :) | 19:30 |
*** LeanReox has joined #openstack-neutron | 19:30 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add callbacks-based system to Neutron https://review.openstack.org/153774 | 19:30 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 19:30 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 base and VPN service plugins during router operations https://review.openstack.org/153776 | 19:30 |
otherwiseguy | But it will be quite satisfying when they're all finished. | 19:30 |
ihrachyshka | ok, now going for beer. cheers to everyone, and please baby-sit that band-aid patch if possible | 19:31 |
LeanReox | hi 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 it | 19:31 |
*** emagana has quit IRC | 19:31 | |
LeanReox | im running juno release | 19:31 |
*** johnbelamaric has joined #openstack-neutron | 19:31 | |
jogo | your breaking global-requirements on master | 19:33 |
*** achanda has joined #openstack-neutron | 19:33 | |
jogo | http://logs.openstack.org/82/155182/1/check/check-requirements-integration-dsvm/ea9a7d2/console.html#_2015-02-12_13_25_28_078 | 19:33 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Automate host configuration for functional testing https://review.openstack.org/146113 | 19:34 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 19:34 |
jogo | kevinbenton: ^ | 19:35 |
openstackgerrit | Erik Colnick proposed openstack/neutron: Improve DVR scale performance https://review.openstack.org/150110 | 19:35 |
openstackgerrit | Erik Colnick proposed openstack/neutron: Refactor to facilitate DVR scale performance https://review.openstack.org/150576 | 19:35 |
*** leunam112 has joined #openstack-neutron | 19:35 | |
kevinbenton | jogo: what causes that? | 19:36 |
otherwiseguy | ugh. 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-neutron | 19:37 | |
*** leunam112 has quit IRC | 19:37 | |
kevinbenton | jogo: that's just a warning right? the failure is coming from that neutron.hooks.setup_hook below, right? | 19:39 |
jogo | kevinbenton: correct | 19:40 |
jogo | setuphook is faiing | 19:40 |
dougwig | anteaya: 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 fourth | 19:40 |
dougwig | block 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 |
jogo | talking about in -qa | 19:40 |
anteaya | dougwig: wow | 19:41 |
anteaya | and yeah, perhaps I mean requirements | 19:41 |
*** aepifanov has joined #openstack-neutron | 19:42 | |
kevinbenton | dougwig: i have a meeting starting now. can you look into this? | 19:42 |
anteaya | dougwig: how does that work for horizon? | 19:42 |
dougwig | kevinbenton: sure, what am i looking at? | 19:42 |
HenryG | jogo: Let me try to help | 19:43 |
mlavalle | carl_baldwin: pong | 19:43 |
kevinbenton | jogo: oh, based on the discussion in openstack-qa it sounds like a fix was just merged? | 19:43 |
carl_baldwin | mlavalle: I updated https://review.openstack.org/#/c/147744/ and I’m nearly finished with rebasing it. | 19:43 |
*** emagana has joined #openstack-neutron | 19:44 | |
*** dsundarr_ has joined #openstack-neutron | 19:44 | |
dougwig | anteaya: i meant "should not", not "should" up there. | 19:44 |
openstackgerrit | Sridhar Ramaswamy proposed openstack/neutron-vpnaas: Implementation of Brocade Vyatta VPNaaS Plugin https://review.openstack.org/154720 | 19:44 |
jogo | yeah sounds like it, will know soon | 19:44 |
carl_baldwin | mlavalle: I addresses a lot of my feedback on PS16 but you might want to read through it. | 19:45 |
anteaya | dougwig: yeah, I heard should not in the context | 19:45 |
dougwig | anteaya: our flake8 seems to be lax about import order and grouping. what context of "work for horizon" are you referring to? | 19:45 |
mlavalle | carl_baldwin: how do you see us moving ahead with this? Are you going to review it and I continue from there? | 19:45 |
HenryG | jogo, kevinbenton, dougwig: yes that looks like the monkey patch issue that was reverted, https://review.openstack.org/155152 | 19:45 |
dougwig | anteaya: and by "our" in that last, i mean "openstack's flake8", not neutron. | 19:45 |
anteaya | dougwig: you mentioned that you are copying what horizon does, does what horizon does work for horizon? | 19:45 |
dougwig | i am the kind of imprecise today. | 19:45 |
dougwig | /kind/king. | 19:45 |
dougwig | sigh. | 19:45 |
anteaya | dougwig: I feel ya | 19:45 |
jogo | HenryG: correct | 19:45 |
mlavalle | carl_baldwin: or you want to conitnue working on it? I want to help but I don't want to get in the way | 19:46 |
dougwig | anteaya: 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 |
mlavalle | carl_baldwin: either way or a combinatio of both is fine with me | 19:46 |
carl_baldwin | mlavalle: 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 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Refactor management of namespaces in the L3 Agent https://review.openstack.org/147744 | 19:47 |
*** dsundarraj has quit IRC | 19:47 | |
carl_baldwin | mlavalle: I’ll hand the reins back to you. | 19:47 |
carl_baldwin | mlavalle: ^^ | 19:47 |
anteaya | dougwig: ah okay cool I mis-understood | 19:47 |
*** lnxnut has quit IRC | 19:47 | |
anteaya | dougwig: thanks for the education | 19:47 |
*** dims__ has quit IRC | 19:48 | |
mlavalle | carl_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 IRC | 19:48 | |
*** vinsh has joined #openstack-neutron | 19:48 | |
*** johnbelamaric has joined #openstack-neutron | 19:48 | |
carl_baldwin | mlavalle: okay | 19:49 |
*** emagana has quit IRC | 19:49 | |
mlavalle | carl_baldwin: thanks for the help | 19:49 |
carl_baldwin | mlavalle: You too. | 19:49 |
*** emagana has joined #openstack-neutron | 19:49 | |
*** dims__ has joined #openstack-neutron | 19:50 | |
*** nyechiel has joined #openstack-neutron | 19:50 | |
mlavalle | carl_baldwin: I like this.... I get to learn from you and contribute at the same time. thanks | 19:50 |
*** _cjones_ has quit IRC | 19:52 | |
*** nyechiel__ has quit IRC | 19:52 | |
carl_baldwin | mlavalle: I’m hope I’m not leading you astray. ;) | 19:52 |
carl_baldwin | johnbelamaric: Are you around? | 19:53 |
*** Longgeek has joined #openstack-neutron | 19:53 | |
carl_baldwin | mlavalle: 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 IRC | 19:54 | |
mlavalle | mlavalle: ok, i'll wathc it | 19:54 |
*** zz_avozza is now known as avozza | 19:54 | |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 19:55 |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 19:56 |
roaet | <- pro git review spammer | 19:56 |
roaet | :P | 19:56 |
*** nyechiel has quit IRC | 19:57 | |
carl_baldwin | johnbelamaric: ping | 19:57 |
*** nyechiel_ has joined #openstack-neutron | 19:57 | |
roaet | please forgive me :( | 19:57 |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 19:57 |
*** achanda has quit IRC | 19:58 | |
*** achanda has joined #openstack-neutron | 19:58 | |
*** tflynn has joined #openstack-neutron | 19:59 | |
*** manishg_ has joined #openstack-neutron | 20:00 | |
*** manishg has quit IRC | 20:00 | |
*** ygbo has quit IRC | 20:01 | |
*** manishg_ has quit IRC | 20:01 | |
*** manishg has joined #openstack-neutron | 20:02 | |
*** Longgeek has quit IRC | 20:02 | |
*** Miouge has quit IRC | 20:03 | |
*** yfried_ has joined #openstack-neutron | 20:04 | |
*** LeanReox has left #openstack-neutron | 20:04 | |
*** shwetaap has quit IRC | 20:05 | |
*** yfried_ has quit IRC | 20:05 | |
*** shwetaap has joined #openstack-neutron | 20:05 | |
*** flaviof has joined #openstack-neutron | 20:06 | |
*** briancurtin has quit IRC | 20:07 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 20:07 |
*** s0mik_ has quit IRC | 20:08 | |
*** zhiyan has quit IRC | 20:08 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 20:08 |
*** ctracey has quit IRC | 20:08 | |
*** emagana has joined #openstack-neutron | 20:09 | |
*** doude has quit IRC | 20:10 | |
*** manishg has quit IRC | 20:11 | |
*** manishg has joined #openstack-neutron | 20:11 | |
*** doude has joined #openstack-neutron | 20:12 | |
*** s3wong has quit IRC | 20:12 | |
*** Youcef has quit IRC | 20:13 | |
dougwig | roaet: scroll up to see ihrachyshka post about 20 in a row. you have some catching up to do. :) | 20:14 |
*** s3wong has joined #openstack-neutron | 20:14 | |
roaet | dougwig: ha.. awesome | 20:14 |
roaet | I don't feel so bad | 20:14 |
dougwig | you're just consuming your own companies VM nodes, so... :) | 20:14 |
roaet | mm. never considered it that way | 20:15 |
*** manishg has quit IRC | 20:16 | |
dougwig | company's. i think i need to go back to bed today. i can't grammar. | 20:16 |
*** s0mik_ has joined #openstack-neutron | 20:16 | |
*** ctracey has joined #openstack-neutron | 20:16 | |
*** _cjones_ has joined #openstack-neutron | 20:16 | |
*** briancurtin has joined #openstack-neutron | 20:17 | |
*** zhiyan has joined #openstack-neutron | 20:18 | |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 20:19 |
*** kfox1111 has joined #openstack-neutron | 20:19 | |
*** ronis_ has joined #openstack-neutron | 20:20 | |
*** ronis has quit IRC | 20:21 | |
kfox1111 | ok. running into a problem. just upgraded from icehouse to juno. almost everything seems to be working. | 20:21 |
kfox1111 | Ive got a lbaas pool that isn't working tough... its tap is: | 20:21 |
kfox1111 | Port "tap1ca011a2-c8" tag: 4095 | 20:22 |
kfox1111 | badly tagged. | 20:22 |
*** stefanb has quit IRC | 20:22 | |
kfox1111 | ideas? | 20:22 |
*** Marga_ has quit IRC | 20:22 | |
*** Marga_ has joined #openstack-neutron | 20:23 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during create/update operations https://review.openstack.org/153775 | 20:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation https://review.openstack.org/155470 | 20:25 |
*** shashankhegde has quit IRC | 20:27 | |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 20:30 |
*** annegent_ has quit IRC | 20:32 | |
*** nyechiel__ has joined #openstack-neutron | 20:32 | |
*** nyechiel_ has quit IRC | 20:33 | |
*** tvardeman has quit IRC | 20:33 | |
*** maishsk has joined #openstack-neutron | 20:35 | |
kfox1111 | why would lbaas ports go 4095 when the rest of the services are working? | 20:36 |
*** manishg has joined #openstack-neutron | 20:38 | |
*** salv-orlando has joined #openstack-neutron | 20:38 | |
*** prad has quit IRC | 20:38 | |
*** dsundarr_ has quit IRC | 20:38 | |
*** hjensas has quit IRC | 20:40 | |
openstackgerrit | Shiv Haris proposed openstack/neutron: Do not review. https://review.openstack.org/152383 | 20:40 |
*** tellesnobrega_ has quit IRC | 20:41 | |
*** kbringard has joined #openstack-neutron | 20:41 | |
*** salv-orl_ has quit IRC | 20:41 | |
*** manishg_ has joined #openstack-neutron | 20:41 | |
*** markvoelker has joined #openstack-neutron | 20:41 | |
*** manishg has quit IRC | 20:43 | |
*** Marga_ has quit IRC | 20:44 | |
*** yfauser has joined #openstack-neutron | 20:44 | |
*** dsundarraj has joined #openstack-neutron | 20:44 | |
*** spandhe has quit IRC | 20:44 | |
*** spandhe has joined #openstack-neutron | 20:45 | |
*** Marga_ has joined #openstack-neutron | 20:46 | |
*** yamahata has quit IRC | 20:46 | |
*** yfauser has left #openstack-neutron | 20:48 | |
*** s3wong has quit IRC | 20:49 | |
johnbelamaric | carl_baldwin: pong | 20:50 |
*** Marga_ has quit IRC | 20:50 | |
carl_baldwin | johnbelamaric: Do you have a moment to talk about ipam? | 20:50 |
johnbelamaric | sure | 20:50 |
*** pc_m has quit IRC | 20:50 | |
*** busterswt has joined #openstack-neutron | 20:51 | |
carl_baldwin | johnbelamaric: So, I’ve tried to address most of the feedback on https://review.openstack.org/#/c/134339/ | 20:51 |
carl_baldwin | The PS18 L221 discussion remains. | 20:52 |
*** manishg_ has quit IRC | 20:52 | |
carl_baldwin | johnbelamaric: 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_baldwin | johnbelamaric: ^ That might be a problem in and of itself. | 20:53 |
johnbelamaric | carl_baldwin: ah. shows what i know | 20:53 |
*** s3wong has joined #openstack-neutron | 20:53 | |
johnbelamaric | carl_baldwin: seems odd | 20:53 |
*** leenheer has quit IRC | 20:53 | |
*** Zemeio has quit IRC | 20:53 | |
carl_baldwin | johnbelamaric: Agreed that it does seem a bit odd. | 20:54 |
carl_baldwin | johnbelamaric: I haven’t decided how to make it better yet. | 20:54 |
kfox1111 | hmm... the config all looks just like my other cloud that's working. :/ | 20:54 |
*** RichardR1seley has quit IRC | 20:55 | |
johnbelamaric | carl_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 odd | 20:55 |
*** leenheer has joined #openstack-neutron | 20:55 | |
carl_baldwin | johnbelamaric: 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 |
johnbelamaric | carl_baldwin: yes especially across various patch sets | 20:56 |
*** nlahouti has quit IRC | 20:56 | |
carl_baldwin | johnbelamaric: Which classes passing around dicts? That is true in the API. | 20:56 |
johnbelamaric | carl_baldwin: let me look at the code for a minute | 20:56 |
*** nlahouti has joined #openstack-neutron | 20:56 | |
*** kickinz1|afk is now known as kickinz1 | 20:57 | |
johnbelamaric | carl_baldwin: look in db_base_plugin_v2 it's all over. this will be the primary consumer of the interface | 20:58 |
carl_baldwin | johnbelamaric: Right, there are lots of dicts in there. | 20:58 |
*** annegent_ has joined #openstack-neutron | 20:58 | |
anteaya | for any neutron folks who care, nova meeting is about to start in -meeting | 21:00 |
johnbelamaric | carl_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 Kilo | 21:01 |
amuller | johnbelamaric++ | 21:01 |
amuller | be part of the solution ;-) | 21:01 |
carl_baldwin | johnbelamaric: 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 |
johnbelamaric | amuller: :) | 21:02 |
johnbelamaric | carl_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 way | 21:02 |
*** jckasper has joined #openstack-neutron | 21:02 | |
carl_baldwin | johnbelamaric: I certainly did not want to proliferate the usage of those dicts by having them passed to the ipam api. | 21:02 |
kfox1111 | hmm... I restarted some services, and now I have less ports tagged with 4095, but still a few... | 21:02 |
kfox1111 | why would that be flaky? | 21:02 |
johnbelamaric | carl_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 ID | 21:03 |
*** dmellado has quit IRC | 21:03 | |
johnbelamaric | carl_baldwin: we could pass just ID to update, but then the driver would have to go to the DB to read the object | 21:03 |
carl_baldwin | kfox1111: 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_baldwin | johnbelamaric: delete (actually deallocate) takes only the cidr in the current form. | 21:04 |
*** dmellado has joined #openstack-neutron | 21:04 | |
carl_baldwin | johnbelamaric: No, I don’t want the driver to read from the db. | 21:04 |
johnbelamaric | carl_baldwin: that should be enough in the context of the pool | 21:04 |
johnbelamaric | carl_baldwin: for delete i mean | 21:04 |
*** jckasper_ has quit IRC | 21:05 | |
kfox1111 | carl_baldwin: I'm coming in cold with a problem. Not sure who to address comments to... | 21:05 |
johnbelamaric | carl_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 update | 21:05 |
johnbelamaric | carl_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 at | 21:06 |
*** baoli has joined #openstack-neutron | 21:06 | |
*** jckasper has quit IRC | 21:07 | |
*** jckasper has joined #openstack-neutron | 21:07 | |
*** claudiub has joined #openstack-neutron | 21:07 | |
carl_baldwin | johnbelamaric: 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 |
johnbelamaric | carl_baldwin: sounds good, i need to head out shortly anyway | 21:09 |
carl_baldwin | johnbelamaric: okay, ttyl. | 21:09 |
*** jckasper_ has joined #openstack-neutron | 21:09 | |
johnbelamaric | carl_baldwin: ttyl | 21:09 |
*** boris-42 has joined #openstack-neutron | 21:09 | |
*** baoli has quit IRC | 21:11 | |
*** baoli has joined #openstack-neutron | 21:11 | |
*** nyechiel__ has quit IRC | 21:12 | |
*** jckasper has quit IRC | 21:12 | |
*** shashankhegde has joined #openstack-neutron | 21:13 | |
*** Marga_ has joined #openstack-neutron | 21:15 | |
*** banix has quit IRC | 21:17 | |
ZZelle_ | dougwig, hi | 21:19 |
*** Marga_ has quit IRC | 21:19 | |
*** leenheer has quit IRC | 21:20 | |
*** dsundarraj has quit IRC | 21:20 | |
*** leenheer has joined #openstack-neutron | 21:20 | |
*** MasterPiece has quit IRC | 21:20 | |
*** claudiub has quit IRC | 21:22 | |
*** banix has joined #openstack-neutron | 21:23 | |
*** leunam112 has joined #openstack-neutron | 21:24 | |
*** dmellado has quit IRC | 21:25 | |
*** TrevorV_ has joined #openstack-neutron | 21:25 | |
*** dmellado has joined #openstack-neutron | 21:26 | |
kevinbenton | rkukura: ping | 21:27 |
openstackgerrit | Erik Colnick proposed openstack/neutron: Improve DVR scale performance https://review.openstack.org/150110 | 21:27 |
*** leunam112 has quit IRC | 21:27 | |
*** emagana has quit IRC | 21:30 | |
rkukura | kevinbenton: pong | 21:31 |
*** emagana has joined #openstack-neutron | 21:31 | |
kevinbenton | rkukura: 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 IRC | 21:31 | |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: right, but i'm pointing out that this is usually the opposite with our VLAN based deployments | 21:34 |
*** emagana has quit IRC | 21:34 | |
kevinbenton | rkukura: we don't use a separate bridge for external networks | 21:34 |
*** emagana has joined #openstack-neutron | 21:35 | |
rkukura | kevinbenton: 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 |
rkukura | kevinbenton: I agree that many do not use the external bridge. If not using this, the provider attributes need to be supplied, right? | 21:35 |
kevinbenton | rkukura: no, if no provider attributes are supplied, it will go through the auto allocation | 21:36 |
*** manishg has joined #openstack-neutron | 21:36 | |
rkukura | kevinbenton: Right, and this patch is changing that to use ‘local’ instead of the tenent_network_types config variable. | 21:37 |
*** baoli has quit IRC | 21:37 | |
*** aepifanov has quit IRC | 21:37 | |
*** sinh has joined #openstack-neutron | 21:37 | |
rkukura | Only when provider attributes are not supplied and external=True | 21:37 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 21:37 |
*** Sukhdev has quit IRC | 21:37 | |
*** baoli has joined #openstack-neutron | 21:37 | |
kevinbenton | rkukura: yeah, i don't like that :) | 21:37 |
kevinbenton | rkukura: i do see where it is nice for overlays | 21:38 |
kevinbenton | rkukura: since the external network is frequently not a tunnel network | 21:38 |
rkukura | kevinbenton: Even with VLANs, the one randomly allocated is not likely the one that connects to the outside world. | 21:39 |
*** jaypipes has joined #openstack-neutron | 21:39 | |
rkukura | With tunnels, wasting a segmentation_id value isn’t a big deal, but with VLANs it might be. | 21:39 |
kevinbenton | rkukura: it's not a waste. it's what the external network is on | 21:39 |
kevinbenton | rkukura: in our workflow we let openstack pick a vlan for itself | 21:40 |
rkukura | kevinbenton: 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 IRC | 21:40 | |
*** manishg has quit IRC | 21:41 | |
*** dmellado has quit IRC | 21:41 | |
kevinbenton | rkukura: yes, in our fabric the network UUID is the identifier for a broadcast domain | 21:41 |
kevinbenton | rkukura: VLANs are just used to define membership associations with specific ports | 21:41 |
*** jpena has quit IRC | 21:42 | |
kevinbenton | rkukura: 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 VLAN | 21:42 |
*** jpena_ has quit IRC | 21:42 | |
rkukura | kevinbenton: Interesting. | 21:42 |
*** dmellado has joined #openstack-neutron | 21:42 | |
kevinbenton | rkukura: then the admin can add whatever rules to that segment to connect in the router to the network segment | 21:43 |
rkukura | kevinbenton: Sounds like maybe you should be using HPB and dynamic VLAN segments for the switch<->node link. | 21:43 |
rkukura | kevinbenton: Anyway, I see why this patch could be problematic for you right now. | 21:44 |
*** aepifanov has joined #openstack-neutron | 21:44 | |
rkukura | kevinbenton: How about the idea of a default_external_network_type config variable? | 21:45 |
*** trad511 has joined #openstack-neutron | 21:45 | |
kevinbenton | rkukura: we happen to work with flexible VLANs to the edge, but that might not always be the case | 21:45 |
kevinbenton | rkukura: either way, i don't like this automagic re-mapping based on it being an external network | 21:46 |
kevinbenton | rkukura: if we are going to go down this route, i would rather it just force the user to specify | 21:47 |
*** trad5111 has quit IRC | 21:47 | |
*** trad511 has quit IRC | 21:47 | |
*** Longgeek has joined #openstack-neutron | 21:47 | |
*** cference has quit IRC | 21:47 | |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: but what is proposed there will break anything that sets this to '' https://github.com/openstack/neutron/blob/master/etc/l3_agent.ini#L46 | 21:49 |
rkukura | kevinbenton: 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-neutron | 21:49 | |
amuller | armax: https://bugs.launchpad.net/neutron/+bug/1418097 | 21:50 |
openstack | Launchpad bug 1418097 in neutron "Kernel panic or ovs-vswitch crash when running DVR functional test" [Undecided,Confirmed] | 21:50 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 21:50 |
amuller | please see comments 9 and 10 | 21:50 |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: i know. i'm talking about when it is empty | 21:52 |
kevinbenton | that's what i meant by '' | 21:52 |
*** TrevorV_ has quit IRC | 21:52 | |
rkukura | kevinbenton: Looked like one double-quote rather than two single-quotes. | 21:53 |
kevinbenton | rkukura: ah | 21:53 |
kevinbenton | rkukura: and we prefer that deployment because then external networks follow the same semantics as all tenant networks | 21:53 |
rkukura | kevinbenton: OK, so I see how this breaks your case needing a real VLAN tag assigned. | 21:53 |
kevinbenton | rkukura: there aren't different debugging paths, etc | 21:53 |
kevinbenton | rkukura: and we can support multiple external networks if people want them | 21:54 |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: what happens if local is chosen when there isn't a different external bridge? | 21:54 |
kevinbenton | rkukura: does that become untagged? | 21:55 |
*** ihrachyshka has quit IRC | 21:55 | |
rkukura | kevinbenton: In that case, the external network isn’t very external ;) | 21:55 |
rkukura | kevinbenton: With type ‘local’ traffic never leaves br-int. | 21:56 |
*** Marga_ has joined #openstack-neutron | 21:56 | |
kevinbenton | rkukura: ok | 21:56 |
*** banix has quit IRC | 21:56 | |
rkukura | kevinbenton: So you’ve convinced my that there is a valid use case for external networks being allocated just like normal tenant networks. | 21:57 |
kevinbenton | rkukura: we need a setting on the server side to know whether or not external bridges will be used | 21:57 |
kevinbenton | rkukura: because then we can put warning logging in when there is a misconfigured l3 agent | 21:57 |
kevinbenton | rkukura: and we can also branch on that for this automatic mapping to local | 21:58 |
*** apuimedo has joined #openstack-neutron | 21:58 | |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: well it will control this mapping. | 21:59 |
kevinbenton | rkukura: and we have no way to detect now if there is a mix of configurations between l3 agents | 21:59 |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: so the warning would be bonus debugging | 21:59 |
*** dmellado has quit IRC | 22:00 | |
maishsk | EE390F79 | 22:00 |
*** maishsk has quit IRC | 22:00 | |
rkukura | kevinbenton: I’d prefer to just start deprecating external_network_bridge. | 22:00 |
*** emagana has quit IRC | 22:01 | |
kevinbenton | rkukura: that might be hard | 22:01 |
*** dmellado has joined #openstack-neutron | 22:01 | |
*** aranjan has quit IRC | 22:01 | |
kevinbenton | rkukura: unfortunately it can be pretty useful because people can setup that bridge to do all sorts of stuff | 22:01 |
kevinbenton | rkukura: other physical interfaces could be on it | 22:02 |
kevinbenton | rkukura: or tunnels to narnia or whatever | 22:02 |
*** jprovazn has quit IRC | 22:02 | |
*** s3wong has quit IRC | 22:02 | |
*** aranjan has joined #openstack-neutron | 22:02 | |
rkukura | kevinbenton: Same could be done with a physical_network name mapping to a dedicated bridge. | 22:02 |
kevinbenton | rkukura: i'm fine with those config semantics you suggested for default_external_network_type | 22:03 |
kevinbenton | rkukura: oh right, i forgot about that | 22:03 |
kevinbenton | rkukura: always just use one | 22:03 |
*** manishg has joined #openstack-neutron | 22:04 | |
kevinbenton | rkukura: ok, so let's do that then | 22:04 |
rkukura | kevinbenton: 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 |
kevinbenton | rkukura: add default_external_network_type | 22:04 |
kevinbenton | rkukura: we can fight about empty vs. local | 22:04 |
*** yfauser1 has joined #openstack-neutron | 22:05 | |
*** yfauser1 has left #openstack-neutron | 22:05 | |
kevinbenton | rkukura: i think i'm fine with defaulting to local | 22:06 |
kevinbenton | rkukura: because tunnels are more common | 22:06 |
kevinbenton | rkukura: we can just clear that in our deployment tools | 22:06 |
*** dmellado has quit IRC | 22:07 | |
*** s3wong has joined #openstack-neutron | 22:08 | |
*** dmellado has joined #openstack-neutron | 22:08 | |
*** spandhe has quit IRC | 22:09 | |
kevinbenton | rkukura: i added a comment to the review | 22:09 |
rkukura | kevinbenton: so did I | 22:09 |
kevinbenton | rkukura: 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 |
rkukura | kevinbenton: Just did ;) | 22:11 |
*** spandhe has joined #openstack-neutron | 22:12 | |
*** emagana has joined #openstack-neutron | 22:12 | |
*** annegent_ has quit IRC | 22:13 | |
kevinbenton | rkukura: great! | 22:13 |
*** annegent_ has joined #openstack-neutron | 22:15 | |
*** aranjan has quit IRC | 22:15 | |
*** _cjones_ has quit IRC | 22:16 | |
*** tiswanso has quit IRC | 22:17 | |
*** regXboi has joined #openstack-neutron | 22:17 | |
*** harlowja_ has joined #openstack-neutron | 22:18 | |
*** salv-orlando has quit IRC | 22:19 | |
*** s3wong has quit IRC | 22:19 | |
*** s3wong has joined #openstack-neutron | 22:19 | |
*** andreykurilin has quit IRC | 22:19 | |
*** salv-orlando has joined #openstack-neutron | 22:19 | |
*** Longgeek has quit IRC | 22:19 | |
*** _cjones_ has joined #openstack-neutron | 22:19 | |
*** andreykurilin has joined #openstack-neutron | 22:19 | |
*** Apsu has quit IRC | 22:20 | |
*** Apsu has joined #openstack-neutron | 22:21 | |
*** harlowja has quit IRC | 22:21 | |
*** amuller has quit IRC | 22:21 | |
*** aranjan has joined #openstack-neutron | 22:22 | |
*** dmellado has quit IRC | 22:22 | |
*** dmellado has joined #openstack-neutron | 22:24 | |
*** thedodd has quit IRC | 22:24 | |
*** yamahata has joined #openstack-neutron | 22:25 | |
*** aranjan has quit IRC | 22:26 | |
*** ronis_ has quit IRC | 22:32 | |
*** banix has joined #openstack-neutron | 22:32 | |
*** vivek-ebay has quit IRC | 22:33 | |
*** ParsectiX has quit IRC | 22:34 | |
*** ParsectiX has joined #openstack-neutron | 22:35 | |
*** thedodd has joined #openstack-neutron | 22:36 | |
*** thedodd has quit IRC | 22:37 | |
*** tellesnobrega_ has joined #openstack-neutron | 22:39 | |
*** carl_baldwin has quit IRC | 22:40 | |
*** localloop127 has quit IRC | 22:44 | |
*** Networkn3rd has quit IRC | 22:47 | |
*** zzzeek_ has joined #openstack-neutron | 22:49 | |
*** annegent_ has quit IRC | 22:49 | |
*** regXboi has quit IRC | 22:50 | |
*** zzzeek has quit IRC | 22:51 | |
*** zzzeek_ is now known as zzzeek | 22:51 | |
*** yuanying has joined #openstack-neutron | 22:55 | |
*** trad511 has joined #openstack-neutron | 23:03 | |
*** busterswt has quit IRC | 23:05 | |
*** vivek-ebay has joined #openstack-neutron | 23:05 | |
*** dmellado has quit IRC | 23:05 | |
*** jgrimm is now known as zz_jgrimm | 23:06 | |
*** dmellado has joined #openstack-neutron | 23:06 | |
*** ParsectiX has quit IRC | 23:07 | |
*** dmellado has quit IRC | 23:07 | |
*** kbringard has quit IRC | 23:07 | |
*** ParsectiX has joined #openstack-neutron | 23:08 | |
*** dmellado has joined #openstack-neutron | 23:08 | |
*** a_le has quit IRC | 23:09 | |
*** a_le has joined #openstack-neutron | 23:09 | |
*** bjornar_ has quit IRC | 23:09 | |
*** dboik_ has joined #openstack-neutron | 23:09 | |
*** julien_ZTE has quit IRC | 23:11 | |
*** a_le has quit IRC | 23:12 | |
*** dboik has quit IRC | 23:13 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent https://review.openstack.org/155212 | 23:13 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent https://review.openstack.org/155188 | 23:13 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess https://review.openstack.org/155101 | 23:13 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from linux interface https://review.openstack.org/155522 | 23:13 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove remaining root_helper args from plugins https://review.openstack.org/155523 | 23:13 |
*** avozza is now known as zz_avozza | 23:13 | |
*** dboik_ has quit IRC | 23:14 | |
*** a_le has joined #openstack-neutron | 23:14 | |
*** leunam112 has joined #openstack-neutron | 23:14 | |
*** vivek-ebay has quit IRC | 23:16 | |
*** aranjan has joined #openstack-neutron | 23:16 | |
*** cference has joined #openstack-neutron | 23:16 | |
*** chlong has joined #openstack-neutron | 23:17 | |
*** leunam112 has quit IRC | 23:18 | |
*** gildub_ has joined #openstack-neutron | 23:20 | |
*** aranjan has quit IRC | 23:21 | |
*** zzzeek_ has joined #openstack-neutron | 23:22 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Fix AttributeError exception for API's test_network_lifecycle https://review.openstack.org/155528 | 23:23 |
*** zzzeek has quit IRC | 23:24 | |
*** zzzeek_ is now known as zzzeek | 23:24 | |
*** hichihara has joined #openstack-neutron | 23:24 | |
*** watanabe_isao has joined #openstack-neutron | 23:27 | |
*** gildub has joined #openstack-neutron | 23:27 | |
*** watanabe_isao has quit IRC | 23:28 | |
jogo | so this is still a top gate bug: http://status.openstack.org/elastic-recheck/index.html#1323658 | 23:32 |
jogo | kevinbenton: ^ | 23:32 |
jogo | neutron only | 23:32 |
*** watanabe_isao has joined #openstack-neutron | 23:32 | |
watanabe_isao | Good morning to ALL...:) | 23:33 |
*** dsneddon_ has joined #openstack-neutron | 23:33 | |
*** apuimedo has quit IRC | 23:34 | |
*** Mr0rris0 has quit IRC | 23:35 | |
*** EmilienM is now known as EmilienM|afk | 23:35 | |
*** olc has quit IRC | 23:36 | |
*** dsneddon has quit IRC | 23:36 | |
*** aranjan has joined #openstack-neutron | 23:38 | |
*** yamahata has quit IRC | 23:39 | |
jogo | anteaya: so one of the top gate bugs is a neutron+nova bug its been around for way too long | 23:39 |
*** tellesnobrega_ has quit IRC | 23:39 | |
*** yamahata has joined #openstack-neutron | 23:40 | |
jogo | http://status.openstack.org/elastic-recheck/index.html#1323658 | 23:40 |
*** openstack has joined #openstack-neutron | 23:41 | |
*** baoli has quit IRC | 23:42 | |
*** baoli has joined #openstack-neutron | 23:42 | |
*** shwetaap has quit IRC | 23:48 | |
*** oomichi has joined #openstack-neutron | 23:50 | |
armax | anteaya: ping | 23:52 |
*** carl_baldwin has joined #openstack-neutron | 23:54 | |
*** jobewan has quit IRC | 23:54 | |
*** salv-orlando has quit IRC | 23:54 | |
*** aepifanov has quit IRC | 23:56 | |
*** krtaylor has quit IRC | 23:57 | |
*** annegent_ has joined #openstack-neutron | 23:59 | |
*** _cjones_ has quit IRC | 23:59 | |
*** trad511 has quit IRC | 23:59 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-vpnaas: DO NOT MERGE, test units https://review.openstack.org/155535 | 23:59 |
*** trad511 has joined #openstack-neutron | 23:59 | |
*** _cjones_ has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!