| *** baoli_ has quit IRC | 00:00 | |
| kevinbenton | amuller: this seems to be what hits the db from your service | 00:00 |
|---|---|---|
| kevinbenton | amuller: commenting out L104 of neutron/wsgi.py | 00:00 |
| kevinbenton | amuller: fixes | 00:00 |
| *** baoli has joined #openstack-neutron | 00:01 | |
| amuller | huh | 00:01 |
| *** Longgeek has joined #openstack-neutron | 00:01 | |
| *** dannywilson has quit IRC | 00:01 | |
| *** yamamoto has quit IRC | 00:01 | |
| *** crc32 has quit IRC | 00:01 | |
| kevinbenton | amuller: calling pool.dispose() on an in-memory sqlite database i think wipes it out | 00:01 |
| amuller | so it's using the same DB configuration, it just erases its content | 00:02 |
| *** shwetaap has joined #openstack-neutron | 00:02 | |
| kevinbenton | amuller: still doing some digging here | 00:02 |
| *** dboik has quit IRC | 00:02 | |
| kevinbenton | amuller: not sure why dispose should wipe that out | 00:02 |
| *** dboik_ has quit IRC | 00:02 | |
| *** shwetaap1 has joined #openstack-neutron | 00:03 | |
| kevinbenton | amuller: ok | 00:04 |
| kevinbenton | amuller: i think i see now | 00:05 |
| *** prad has quit IRC | 00:05 | |
| *** baoli has quit IRC | 00:05 | |
| kevinbenton | amuller: you launch a greenthread which doesn't call pool.dispose() until the next test has already gone in an setup the tables | 00:05 |
| kevinbenton | if you add the following to the end of _test_spawn_metadata_proxy, it will pass | 00:06 |
| kevinbenton | import eventlet | 00:06 |
| kevinbenton | eventlet.sleep(5) | 00:06 |
| *** hichihara has joined #openstack-neutron | 00:07 | |
| *** shwetaap has quit IRC | 00:07 | |
| kevinbenton | amuller: you're throwing a tiny grenade into the future that calls pool.dispose() after the table init is done in neutron/tests/unit/testlib_api.py | 00:08 |
| kevinbenton | amuller: and since the flag is set in neutron/tests/unit/testlib_api.py that the table init is done, it never tries again | 00:08 |
| amuller | New ways to wreak havoc :) | 00:09 |
| *** aranjan has quit IRC | 00:09 | |
| *** baoli has joined #openstack-neutron | 00:09 | |
| *** aranjan has joined #openstack-neutron | 00:11 | |
| amuller | kevinbenton: Interesting that agent code does any DB interaction at all | 00:11 |
| kevinbenton | amuller: it's just a side effect of the WorkerService class | 00:12 |
| *** banix has quit IRC | 00:12 | |
| amuller | kevinbenton: hmm, what if you have a L3 agent running on the same where the Neutron server is running | 00:12 |
| *** crc32 has joined #openstack-neutron | 00:12 | |
| amuller | kevinbenton: the L3 agent is passed a neutron.conf with a valid DB config | 00:12 |
| kevinbenton | amuller: still separate processes so that should be safe | 00:12 |
| *** marun has joined #openstack-neutron | 00:12 | |
| amuller | so the L3 agent initializes an actual connection pool | 00:13 |
| amuller | does no harm, but still | 00:13 |
| *** shaohe_feng1 has quit IRC | 00:13 | |
| kevinbenton | amuller: so dispose dumps any connections that process has | 00:14 |
| kevinbenton | amuller: but i'm not sure if get_engine() automatically creates a connection | 00:14 |
| kevinbenton | amuller: if it does, the l3 agent could be wasting a connection to the db | 00:14 |
| *** baoli has quit IRC | 00:14 | |
| *** markvoelker has quit IRC | 00:15 | |
| *** aranjan has quit IRC | 00:15 | |
| *** achanda has quit IRC | 00:15 | |
| amuller | kevinbenton: this isn't unique to the new service though, all of our agents inherit from that class iirc | 00:16 |
| *** sripriya has quit IRC | 00:16 | |
| amuller | kevinbenton: so any agents running together with the server exhibit the same issue | 00:16 |
| *** achanda has joined #openstack-neutron | 00:16 | |
| kevinbenton | amuller: right. i'm looking at if an enginefacade actually creates a connection on creation | 00:17 |
| kevinbenton | amuller: if it does, we should stop this | 00:17 |
| kevinbenton | amuller: if it doesn't, dispose() will be a noop so it won't matter | 00:17 |
| amuller | I think I saw something in the docs which suggest it doesn't, http://docs.sqlalchemy.org/en/latest/core/connections.html#sqlalchemy.engine.Engine.dispose | 00:17 |
| *** oomichi has quit IRC | 00:17 | |
| amuller | kevinbenton: going out for dinner let's talk later | 00:17 |
| *** baoli has joined #openstack-neutron | 00:18 | |
| kevinbenton | amuller: you are not allowed to have dinner until this is resolved! | 00:18 |
| *** achanda has quit IRC | 00:18 | |
| *** achanda has joined #openstack-neutron | 00:18 | |
| *** Longgeek has quit IRC | 00:18 | |
| *** pavel_bondar has quit IRC | 00:19 | |
| kevinbenton | amuller: sounds like we are safe | 00:19 |
| kevinbenton | amuller: http://docs.sqlalchemy.org/en/rel_0_9/core/engines.html | 00:19 |
| *** pavel_bondar has joined #openstack-neutron | 00:19 | |
| *** vivek-ebay has quit IRC | 00:19 | |
| kevinbenton | amuller: sqlalchemy doesn't actually connect until something attempts a transaction | 00:19 |
| kevinbenton | amuller: i'm still going to submit a patch to make this dispose operation optional | 00:20 |
| kevinbenton | amuller: just to avoid wasting the time setting up all of that | 00:20 |
| yamahata | ZZelle: ping? | 00:20 |
| amuller | kevinbenton: This is one of those issues I'm fully capable of detaching from :) | 00:21 |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 00:21 |
| *** aranjan has joined #openstack-neutron | 00:22 | |
| kevinbenton | amuller: you're free to eat dinner now :) | 00:22 |
| *** rook-tmp has joined #openstack-neutron | 00:22 | |
| *** nati_ueno has quit IRC | 00:25 | |
| *** natarajk has left #openstack-neutron | 00:30 | |
| *** achanda has quit IRC | 00:30 | |
| amuller | kevinbenton: Aye aye! | 00:31 |
| *** markvoelker has joined #openstack-neutron | 00:32 | |
| *** yamamoto has joined #openstack-neutron | 00:33 | |
| *** watanabe_isao has joined #openstack-neutron | 00:33 | |
| watanabe_isao | Good morning to all:) | 00:33 |
| *** shwetaap1 has quit IRC | 00:34 | |
| *** dboik has joined #openstack-neutron | 00:35 | |
| *** vmtrooper has joined #openstack-neutron | 00:35 | |
| *** xgerman has quit IRC | 00:36 | |
| *** dboik_ has joined #openstack-neutron | 00:36 | |
| *** mlavalle has quit IRC | 00:38 | |
| *** amuller has quit IRC | 00:39 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 00:39 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 00:39 |
| *** dboik has quit IRC | 00:40 | |
| *** vmtrooper has quit IRC | 00:40 | |
| *** catohornet has joined #openstack-neutron | 00:41 | |
| *** diegows has quit IRC | 00:41 | |
| *** gongysh has quit IRC | 00:42 | |
| *** diegows has joined #openstack-neutron | 00:42 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 00:42 |
| *** _cjones_ has quit IRC | 00:43 | |
| *** banix has joined #openstack-neutron | 00:43 | |
| *** pritesh has joined #openstack-neutron | 00:45 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 00:45 |
| *** yalie has joined #openstack-neutron | 00:48 | |
| *** pritesh_ has quit IRC | 00:48 | |
| *** topshare has joined #openstack-neutron | 00:48 | |
| *** melwitt has quit IRC | 00:51 | |
| *** abehl has quit IRC | 00:52 | |
| *** ChuckC has quit IRC | 00:53 | |
| *** baoli has quit IRC | 00:55 | |
| openstackgerrit | Merged openstack/neutron-fwaas: Provide hooks for FWaaS repo functional gate https://review.openstack.org/161479 | 00:55 |
| *** rook-tmp has quit IRC | 00:56 | |
| openstackgerrit | Merged openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 00:56 |
| *** banix has quit IRC | 00:57 | |
| *** Longgeek has joined #openstack-neutron | 00:59 | |
| openstackgerrit | pritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient https://review.openstack.org/162040 | 01:00 |
| *** nlahouti has quit IRC | 01:02 | |
| *** aranjan has quit IRC | 01:03 | |
| *** salv-orl_ has quit IRC | 01:04 | |
| *** bandarji has joined #openstack-neutron | 01:12 | |
| *** dane_leblanc has quit IRC | 01:14 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 01:15 |
| *** fredlhsu has joined #openstack-neutron | 01:18 | |
| *** yamamoto has quit IRC | 01:19 | |
| *** tidwellr has left #openstack-neutron | 01:19 | |
| *** padkrish_ has quit IRC | 01:21 | |
| *** padkrish has joined #openstack-neutron | 01:22 | |
| openstackgerrit | pritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient https://review.openstack.org/162040 | 01:23 |
| *** vivek-ebay has joined #openstack-neutron | 01:23 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Fix minor decomp progress chart issues https://review.openstack.org/165267 | 01:24 |
| yalie | hi mestery? | 01:25 |
| *** apuimedo has quit IRC | 01:25 | |
| *** padkrish has quit IRC | 01:27 | |
| yalie | marun leaves a comments on https://review.openstack.org/#/c/126552/ | 01:27 |
| yalie | and marun? | 01:28 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Use iptables zone to seperate different conntrack https://review.openstack.org/118274 | 01:29 |
| yalie | could we make a decision here which direction I should go? | 01:29 |
| *** vivek-eb_ has joined #openstack-neutron | 01:30 | |
| *** tflynn_ has joined #openstack-neutron | 01:30 | |
| *** padkrish has joined #openstack-neutron | 01:31 | |
| *** amuller has joined #openstack-neutron | 01:31 | |
| *** stanzgy_ has joined #openstack-neutron | 01:31 | |
| *** stanzgy has joined #openstack-neutron | 01:31 | |
| amuller | marun: still up? | 01:32 |
| *** vivek-ebay has quit IRC | 01:32 | |
| *** popw has joined #openstack-neutron | 01:32 | |
| yalie | not on line? | 01:33 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 01:33 |
| *** tflynn has quit IRC | 01:33 | |
| ijw | woot | 01:33 |
| *** tflynn_ has quit IRC | 01:35 | |
| *** shaohe_feng1 has joined #openstack-neutron | 01:38 | |
| *** pal has joined #openstack-neutron | 01:39 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 01:40 |
| mestery | yalie: yo | 01:40 |
| mestery | yalie: Keep them together please, don't split them. | 01:40 |
| *** s3wong_ has quit IRC | 01:40 | |
| mestery | yalie: This means we'll need to merge yamahata's functional change first, hoping marun can get his eyes on this one tonight yet (https://review.openstack.org/#/c/160051/) | 01:41 |
| *** rdo has quit IRC | 01:41 | |
| amuller | marun: well, I replied on the patch, I don't know what addtional test you had in mind, I don't see the need for any | 01:42 |
| *** yamamoto has joined #openstack-neutron | 01:42 | |
| *** tsg has joined #openstack-neutron | 01:42 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete https://review.openstack.org/161073 | 01:43 |
| *** padkrish has quit IRC | 01:43 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete https://review.openstack.org/161073 | 01:43 |
| *** banix has joined #openstack-neutron | 01:43 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 01:47 |
| *** Jianyong has joined #openstack-neutron | 01:47 | |
| *** pritesh has quit IRC | 01:47 | |
| *** bandarji has quit IRC | 01:49 | |
| *** leenheer has quit IRC | 01:52 | |
| *** rook-tmp has joined #openstack-neutron | 01:53 | |
| *** ivar-laz_ has joined #openstack-neutron | 01:55 | |
| *** amuller has quit IRC | 01:55 | |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 01:55 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Auto-update gateway port after subnet-create https://review.openstack.org/161085 | 01:56 |
| *** ivar-lazzaro has quit IRC | 01:59 | |
| *** igordcard_ has quit IRC | 01:59 | |
| *** dims has quit IRC | 01:59 | |
| *** ivar-laz_ has quit IRC | 01:59 | |
| *** britthouser has quit IRC | 02:03 | |
| *** tiswanso has joined #openstack-neutron | 02:03 | |
| *** britthouser has joined #openstack-neutron | 02:03 | |
| *** britthouser has quit IRC | 02:04 | |
| *** britthouser has joined #openstack-neutron | 02:05 | |
| *** tiswanso has quit IRC | 02:06 | |
| *** harlowja_ is now known as harlowja_away | 02:06 | |
| *** tiswanso has joined #openstack-neutron | 02:06 | |
| *** catohornet has quit IRC | 02:09 | |
| *** yamahata has quit IRC | 02:09 | |
| *** fredlhsu has quit IRC | 02:10 | |
| HenryG | mestery: yalie: I reviewed and left my thoughts on https://review.openstack.org/160051 - no +A yet in case you still want marun to take a look. | 02:10 |
| *** crc32 has quit IRC | 02:11 | |
| mestery | HenryG: Please merge that one if you think it's ready, I appreciate your review! | 02:11 |
| mestery | HenryG: It also unblocks the subsequent port security stuff, which we can hopefully land tomorrow | 02:11 |
| ijw | port security ftw | 02:12 |
| mestery | ijw: You're like a troll who pops in when the words "port" and "security" come together on IRC :) | 02:12 |
| ijw | Nah | 02:12 |
| ijw | It's nothing to do with irc | 02:12 |
| * ijw crawls undr his bridge | 02:12 | |
| openstackgerrit | Merged openstack/neutron: Don't delete HA router primary VIP on agent restarts https://review.openstack.org/164841 | 02:13 |
| openstackgerrit | Merged openstack/neutron: Don't delete HA router primary VIP on agent restarts https://review.openstack.org/164841 | 02:13 |
| HenryG | mestery: I am sure marun is keeping a black list of all the "I promise I will refactor the tests as a follow-up" comments. :) | 02:15 |
| *** gongysh has joined #openstack-neutron | 02:15 | |
| mestery | HenryG: lol | 02:15 |
| * HenryG looks for hidden etherpads created by marun | 02:16 | |
| *** topshare has quit IRC | 02:17 | |
| *** nati_ueno has joined #openstack-neutron | 02:17 | |
| *** salv-orlando has joined #openstack-neutron | 02:18 | |
| mestery | ijw: lol | 02:18 |
| *** pal has quit IRC | 02:18 | |
| *** topshare has joined #openstack-neutron | 02:19 | |
| *** coolsvap|afk is now known as coolsvap | 02:19 | |
| *** crc32 has joined #openstack-neutron | 02:20 | |
| HenryG | mestery: I don't know how you can still be going but I am running out of steam for today. I'll be up early tomorrow. | 02:20 |
| ijw | wuss | 02:21 |
| *** erkules_ has joined #openstack-neutron | 02:21 | |
| HenryG | raspberry | 02:21 |
| dougwig | HenryG: i added to the co-gate today, so i think i shot to the top of said list. | 02:23 |
| *** erkules has quit IRC | 02:23 | |
| openstackgerrit | Nachi Ueno proposed openstack/neutron: OpenContrail plugin code split https://review.openstack.org/163218 | 02:23 |
| openstackgerrit | badveli_vishnuus proposed openstack/neutron-fwaas: Adding mandatory parameters for Router Info in vArmour fwaas agent https://review.openstack.org/161424 | 02:24 |
| *** vmtrooper has joined #openstack-neutron | 02:24 | |
| HenryG | armax: I think this one is OK, https://review.openstack.org/154709 | 02:25 |
| *** marun has quit IRC | 02:25 | |
| *** topshare has quit IRC | 02:25 | |
| *** topshare has joined #openstack-neutron | 02:26 | |
| dougwig | HenryG, armax - i'll get it. | 02:26 |
| *** rook-tmp has quit IRC | 02:26 | |
| armax | dougwig: thanks, I have been waiting to see how the CI behaves | 02:27 |
| armax | but nothing is coming up so far | 02:27 |
| dougwig | armax: ok | 02:28 |
| *** vmtrooper has quit IRC | 02:29 | |
| *** diegows has quit IRC | 02:30 | |
| *** britthou_ has joined #openstack-neutron | 02:32 | |
| *** britthouser has quit IRC | 02:32 | |
| *** coolsvap is now known as coolsvap|afk | 02:33 | |
| *** zzzeek has joined #openstack-neutron | 02:35 | |
| *** topshare has quit IRC | 02:36 | |
| *** topshare has joined #openstack-neutron | 02:36 | |
| *** baoli has joined #openstack-neutron | 02:38 | |
| openstackgerrit | Merged openstack/python-neutronclient: Updating lbaas cli for TLS https://review.openstack.org/161404 | 02:39 |
| openstackgerrit | Merged openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv https://review.openstack.org/161019 | 02:41 |
| *** topshare has quit IRC | 02:42 | |
| *** chandankumar has joined #openstack-neutron | 02:45 | |
| *** coolsvap|afk is now known as coolsvap | 02:45 | |
| *** yuanying has quit IRC | 02:46 | |
| *** topshare has joined #openstack-neutron | 02:47 | |
| *** melwitt has joined #openstack-neutron | 02:52 | |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 02:52 |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Add some unit tests for strongswan driver https://review.openstack.org/165159 | 02:52 |
| *** Murali has joined #openstack-neutron | 02:52 | |
| *** padkrish has joined #openstack-neutron | 02:54 | |
| *** lazy_prince has quit IRC | 02:57 | |
| *** _cjones_ has joined #openstack-neutron | 02:57 | |
| *** pal has joined #openstack-neutron | 02:58 | |
| *** padkrish has quit IRC | 02:59 | |
| *** baoli has quit IRC | 03:01 | |
| *** zzzeek has quit IRC | 03:01 | |
| *** baoli has joined #openstack-neutron | 03:01 | |
| *** killer_prince has joined #openstack-neutron | 03:04 | |
| *** killer_prince is now known as lazy_prince | 03:04 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 03:04 |
| *** chandankumar has quit IRC | 03:05 | |
| *** marun has joined #openstack-neutron | 03:06 | |
| *** sripriya has joined #openstack-neutron | 03:07 | |
| watanabe_isao | hello, mestery. May I ask you a question about SF? | 03:07 |
| *** coolsvap is now known as coolsvap|afk | 03:09 | |
| *** Marga_ has quit IRC | 03:09 | |
| watanabe_isao | What will be freeze by SF, please: 1) Adding a new message? 2) Or even add a logic to export existing message? | 03:10 |
| *** shwetaap has joined #openstack-neutron | 03:10 | |
| *** dboik_ has quit IRC | 03:13 | |
| *** tiswanso has quit IRC | 03:13 | |
| *** sripriya has quit IRC | 03:14 | |
| *** coolsvap|afk is now known as coolsvap | 03:16 | |
| *** sripriya has joined #openstack-neutron | 03:19 | |
| *** changbl has joined #openstack-neutron | 03:20 | |
| *** rook-tmp has joined #openstack-neutron | 03:23 | |
| *** britthou_ has quit IRC | 03:23 | |
| *** banix has quit IRC | 03:25 | |
| *** crc32 has quit IRC | 03:27 | |
| openstackgerrit | Merged openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 03:28 |
| openstackgerrit | Merged openstack/neutron: Imported Translations from Transifex https://review.openstack.org/163708 | 03:28 |
| *** crc32 has joined #openstack-neutron | 03:29 | |
| *** salv-orlando has quit IRC | 03:29 | |
| *** Longgeek has quit IRC | 03:29 | |
| *** sudipto has joined #openstack-neutron | 03:30 | |
| *** gildub has quit IRC | 03:31 | |
| *** csoukup has joined #openstack-neutron | 03:33 | |
| *** yuanying has joined #openstack-neutron | 03:37 | |
| *** yuanying has quit IRC | 03:38 | |
| *** yuanying has joined #openstack-neutron | 03:38 | |
| *** sbfox has joined #openstack-neutron | 03:39 | |
| *** chandankumar has joined #openstack-neutron | 03:39 | |
| *** vishwanathj has joined #openstack-neutron | 03:41 | |
| *** melwitt has quit IRC | 03:47 | |
| *** gildub has joined #openstack-neutron | 03:47 | |
| *** csoukup has quit IRC | 03:48 | |
| *** _cjones_ has quit IRC | 03:48 | |
| *** jckasper has joined #openstack-neutron | 03:56 | |
| *** rook-tmp has quit IRC | 03:57 | |
| *** baoli has quit IRC | 03:58 | |
| *** baoli has joined #openstack-neutron | 03:59 | |
| *** sthillma has joined #openstack-neutron | 03:59 | |
| *** ijw has quit IRC | 04:02 | |
| *** sthillma has quit IRC | 04:02 | |
| *** Murali has quit IRC | 04:03 | |
| *** SumitNaiksatam has joined #openstack-neutron | 04:04 | |
| *** csoukup has joined #openstack-neutron | 04:04 | |
| *** csoukup has quit IRC | 04:09 | |
| *** vthapar has joined #openstack-neutron | 04:09 | |
| *** ajmiller has joined #openstack-neutron | 04:09 | |
| *** Marga_ has joined #openstack-neutron | 04:11 | |
| *** SumitNaiksatam has quit IRC | 04:11 | |
| *** Marga__ has joined #openstack-neutron | 04:12 | |
| *** vmtrooper has joined #openstack-neutron | 04:13 | |
| *** sudipto has quit IRC | 04:13 | |
| *** Marga_ has quit IRC | 04:16 | |
| *** SumitNaiksatam has joined #openstack-neutron | 04:17 | |
| *** gongysh has quit IRC | 04:18 | |
| *** vmtrooper has quit IRC | 04:18 | |
| *** britthouser has joined #openstack-neutron | 04:23 | |
| *** tflynn has joined #openstack-neutron | 04:28 | |
| *** yatin has joined #openstack-neutron | 04:28 | |
| *** salv-orlando has joined #openstack-neutron | 04:28 | |
| *** yfried|afk is now known as yfried_ | 04:30 | |
| *** Longgeek has joined #openstack-neutron | 04:30 | |
| *** tsg has quit IRC | 04:30 | |
| *** Longgeek has quit IRC | 04:37 | |
| *** Longgeek has joined #openstack-neutron | 04:38 | |
| *** aranjan has joined #openstack-neutron | 04:40 | |
| *** baoli has quit IRC | 04:42 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix DBDuplicateError handling in _ensure_default_security_group https://review.openstack.org/165290 | 04:43 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Fix a regression of the recent ip_lib change https://review.openstack.org/165291 | 04:44 |
| *** jckasper has quit IRC | 04:45 | |
| *** tflynn has quit IRC | 04:45 | |
| *** nlahouti has joined #openstack-neutron | 04:46 | |
| yamamoto | enikanorov: ping | 04:46 |
| *** ajmiller has quit IRC | 04:46 | |
| *** ajmiller has joined #openstack-neutron | 04:46 | |
| *** boris-42 has joined #openstack-neutron | 04:47 | |
| *** tflynn has joined #openstack-neutron | 04:47 | |
| *** marun has quit IRC | 04:48 | |
| *** nati_uen_ has joined #openstack-neutron | 04:50 | |
| *** dims has joined #openstack-neutron | 04:52 | |
| *** nati_uen_ has quit IRC | 04:53 | |
| *** rook-tmp has joined #openstack-neutron | 04:54 | |
| *** nati_ueno has quit IRC | 04:54 | |
| *** nlahouti has quit IRC | 04:56 | |
| *** nlahouti has joined #openstack-neutron | 04:56 | |
| *** dims has quit IRC | 04:57 | |
| *** yfauser has joined #openstack-neutron | 04:57 | |
| *** britthouser has quit IRC | 04:58 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: linuxbridge UT: Fix a regression of the recent ip_lib change https://review.openstack.org/165291 | 04:59 |
| prometheanfire | yamamoto: how did that get pased testing? | 05:03 |
| prometheanfire | https://review.openstack.org/165291 | 05:03 |
| yamamoto | prometheanfire: because everyone runs UTs on a system with sysfs? | 05:04 |
| *** achanda has joined #openstack-neutron | 05:06 | |
| prometheanfire | should we expect that? | 05:06 |
| prometheanfire | afaik, openstack is meant to be run on linux only | 05:06 |
| yamamoto | prometheanfire: UTs should not depends on system state as far as possible | 05:07 |
| prometheanfire | right, guess I didn't see that it was still calling out to that | 05:07 |
| yamamoto | prometheanfire: while sysfs might exist everywhere, who knows its state is consistent to the UT cases' expectation? | 05:08 |
| openstackgerrit | pritesh proposed openstack/python-neutronclient: Adding VLAN Transparency support to neutronclient https://review.openstack.org/162040 | 05:08 |
| prometheanfire | yamamoto: ya, I get that | 05:09 |
| prometheanfire | the problem wasn't accessing sysfs, it was accessing system state | 05:09 |
| *** britthouser has joined #openstack-neutron | 05:09 | |
| *** Networkn3rd has quit IRC | 05:09 | |
| yamamoto | prometheanfire: and, fwiw, i often actually run neutron UTs on non-linux systems. | 05:09 |
| *** yapeng has quit IRC | 05:11 | |
| prometheanfire | makes sense | 05:11 |
| *** xek has quit IRC | 05:18 | |
| *** anilvenkata has joined #openstack-neutron | 05:18 | |
| *** sbfox has quit IRC | 05:23 | |
| *** Networkn3rd has joined #openstack-neutron | 05:26 | |
| *** rook-tmp has quit IRC | 05:27 | |
| *** pradip_vedams has joined #openstack-neutron | 05:29 | |
| *** tflynn has quit IRC | 05:30 | |
| *** salv-orlando has quit IRC | 05:31 | |
| *** markvoelker has quit IRC | 05:33 | |
| *** yfauser has left #openstack-neutron | 05:35 | |
| *** shwetaap has quit IRC | 05:38 | |
| *** vthapar has quit IRC | 05:41 | |
| *** gongysh has joined #openstack-neutron | 05:41 | |
| *** tflynn has joined #openstack-neutron | 05:43 | |
| *** armax has quit IRC | 05:44 | |
| *** sripriya has quit IRC | 05:45 | |
| *** garyk has quit IRC | 05:46 | |
| openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 05:46 |
| *** Sukhdev has joined #openstack-neutron | 05:47 | |
| *** sbfox has joined #openstack-neutron | 05:49 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix DBDuplicateError handling in _ensure_default_security_group https://review.openstack.org/165290 | 05:51 |
| *** __afazekas is now known as afazekas | 05:52 | |
| *** afazekas is now known as __afazekas | 05:53 | |
| *** vthapar has joined #openstack-neutron | 05:53 | |
| *** vmtrooper has joined #openstack-neutron | 06:02 | |
| *** jschwarz has joined #openstack-neutron | 06:05 | |
| *** anilvenkata has left #openstack-neutron | 06:06 | |
| *** anilvenkata has joined #openstack-neutron | 06:07 | |
| *** vmtrooper has quit IRC | 06:07 | |
| *** britthouser has quit IRC | 06:09 | |
| *** lpetrut has joined #openstack-neutron | 06:10 | |
| *** vivek-eb_ has quit IRC | 06:13 | |
| *** aepifanov has joined #openstack-neutron | 06:14 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 06:18 |
| *** tflynn has quit IRC | 06:20 | |
| *** pradip_rawat has joined #openstack-neutron | 06:21 | |
| *** pradip_vedams has quit IRC | 06:23 | |
| *** pradip_rawat is now known as pradip_vedams | 06:23 | |
| *** rook-tmp has joined #openstack-neutron | 06:24 | |
| *** yog__ has joined #openstack-neutron | 06:25 | |
| *** achanda has quit IRC | 06:25 | |
| *** achanda has joined #openstack-neutron | 06:27 | |
| *** ferest has joined #openstack-neutron | 06:27 | |
| *** rook-tmp has quit IRC | 06:29 | |
| *** liusheng has quit IRC | 06:31 | |
| *** liusheng has joined #openstack-neutron | 06:31 | |
| *** ferest has quit IRC | 06:32 | |
| *** salv-orlando has joined #openstack-neutron | 06:33 | |
| *** markvoelker has joined #openstack-neutron | 06:33 | |
| *** shaohe_feng1 has quit IRC | 06:33 | |
| *** ishant has joined #openstack-neutron | 06:35 | |
| *** angela-s has left #openstack-neutron | 06:38 | |
| *** markvoelker has quit IRC | 06:38 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 06:40 |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 06:43 |
| *** afazekas has joined #openstack-neutron | 06:47 | |
| *** _cjones_ has joined #openstack-neutron | 06:49 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: WIP Prototype of fullstack framework https://review.openstack.org/128259 | 06:51 |
| openstackgerrit | Aman Kumar proposed openstack/neutron: Added config variable for External Network type in ML2 https://review.openstack.org/149546 | 06:51 |
| *** gildub has quit IRC | 06:52 | |
| *** ronis has joined #openstack-neutron | 06:52 | |
| *** SridharG has joined #openstack-neutron | 06:54 | |
| *** _cjones_ has quit IRC | 06:54 | |
| *** mestery has quit IRC | 06:58 | |
| *** numan has joined #openstack-neutron | 07:00 | |
| *** Sukhdev has quit IRC | 07:03 | |
| *** Trozz has joined #openstack-neutron | 07:05 | |
| *** nmagnezi has joined #openstack-neutron | 07:05 | |
| *** nlahouti has quit IRC | 07:08 | |
| *** kickinz1|afk is now known as kickinz1 | 07:10 | |
| *** sripriya has joined #openstack-neutron | 07:12 | |
| *** Bhavya has joined #openstack-neutron | 07:12 | |
| *** itzikb has joined #openstack-neutron | 07:14 | |
| *** aswadr has joined #openstack-neutron | 07:14 | |
| *** garyk has joined #openstack-neutron | 07:15 | |
| *** adelinatuvenie has joined #openstack-neutron | 07:16 | |
| *** sripriya has quit IRC | 07:17 | |
| *** markvoelker has joined #openstack-neutron | 07:17 | |
| *** Murali has joined #openstack-neutron | 07:19 | |
| *** aix has quit IRC | 07:20 | |
| *** markvoelker has quit IRC | 07:21 | |
| *** anshul has joined #openstack-neutron | 07:24 | |
| *** scheuran has joined #openstack-neutron | 07:24 | |
| *** rook-tmp has joined #openstack-neutron | 07:25 | |
| *** neeti has joined #openstack-neutron | 07:27 | |
| *** Networkn_ has joined #openstack-neutron | 07:27 | |
| *** pradipta has joined #openstack-neutron | 07:28 | |
| *** rook-tmp has quit IRC | 07:30 | |
| *** Networkn3rd has quit IRC | 07:30 | |
| *** lionelz has joined #openstack-neutron | 07:30 | |
| *** yatin has quit IRC | 07:31 | |
| *** ronis has quit IRC | 07:32 | |
| *** aranjan has quit IRC | 07:32 | |
| *** rotbeard has joined #openstack-neutron | 07:32 | |
| *** salv-orlando has quit IRC | 07:33 | |
| *** jprovazn has joined #openstack-neutron | 07:34 | |
| *** achanda has quit IRC | 07:35 | |
| *** lpetrut has quit IRC | 07:37 | |
| *** achanda has joined #openstack-neutron | 07:39 | |
| *** nmagnezi has quit IRC | 07:42 | |
| *** salv-orlando has joined #openstack-neutron | 07:42 | |
| itzikb | heyongli: ping | 07:44 |
| *** vthapar has quit IRC | 07:44 | |
| *** vthapar has joined #openstack-neutron | 07:44 | |
| *** adelinatuvenie has quit IRC | 07:49 | |
| openstackgerrit | vishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 07:50 |
| *** ishant has quit IRC | 07:50 | |
| *** vmtrooper has joined #openstack-neutron | 07:51 | |
| *** ishant has joined #openstack-neutron | 07:51 | |
| *** yatin has joined #openstack-neutron | 07:51 | |
| *** gsagie has joined #openstack-neutron | 07:52 | |
| *** vmtrooper has quit IRC | 07:56 | |
| *** doude has joined #openstack-neutron | 07:58 | |
| *** Bsony has joined #openstack-neutron | 07:58 | |
| *** bobmel has joined #openstack-neutron | 08:00 | |
| *** jlanoux has joined #openstack-neutron | 08:03 | |
| *** nyechiel__ has joined #openstack-neutron | 08:06 | |
| *** belmoreira has joined #openstack-neutron | 08:07 | |
| *** moha_hunt has joined #openstack-neutron | 08:07 | |
| *** kawa2014 has joined #openstack-neutron | 08:08 | |
| *** yrn has quit IRC | 08:08 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 08:08 |
| *** nmagnezi has joined #openstack-neutron | 08:09 | |
| *** topshare has quit IRC | 08:10 | |
| *** achanda has quit IRC | 08:10 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 08:11 |
| *** topshare has joined #openstack-neutron | 08:13 | |
| *** Marga__ has quit IRC | 08:13 | |
| *** pradipta has quit IRC | 08:14 | |
| *** topshare has quit IRC | 08:16 | |
| *** markvoelker has joined #openstack-neutron | 08:18 | |
| *** criptoverso has joined #openstack-neutron | 08:18 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 08:20 |
| *** mleroy_ has joined #openstack-neutron | 08:22 | |
| *** jistr has joined #openstack-neutron | 08:22 | |
| *** markvoelker has quit IRC | 08:23 | |
| *** matrohon has joined #openstack-neutron | 08:23 | |
| *** rook-tmp has joined #openstack-neutron | 08:26 | |
| *** adelinatuvenie has joined #openstack-neutron | 08:27 | |
| *** yuanying has quit IRC | 08:27 | |
| *** Trozz is now known as zzTrozz | 08:28 | |
| *** zzTrozz is now known as Trozz | 08:28 | |
| *** pradipta has joined #openstack-neutron | 08:30 | |
| *** rook-tmp has quit IRC | 08:30 | |
| *** sbfox has quit IRC | 08:30 | |
| *** moshele has joined #openstack-neutron | 08:31 | |
| *** yfried_ has quit IRC | 08:32 | |
| *** uschreiber_ has joined #openstack-neutron | 08:32 | |
| *** uschreiber_ has quit IRC | 08:33 | |
| *** ronis has joined #openstack-neutron | 08:34 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 08:34 |
| *** apuimedo has joined #openstack-neutron | 08:35 | |
| *** topshare has joined #openstack-neutron | 08:35 | |
| *** jlanoux has quit IRC | 08:35 | |
| *** safchain has joined #openstack-neutron | 08:36 | |
| *** uschreiber_ has joined #openstack-neutron | 08:38 | |
| *** mleroy_ has quit IRC | 08:38 | |
| *** uschreiber_ has quit IRC | 08:39 | |
| *** nyechiel has joined #openstack-neutron | 08:39 | |
| *** yfried has joined #openstack-neutron | 08:41 | |
| *** sudipto_ has joined #openstack-neutron | 08:41 | |
| *** uschreiber_ has joined #openstack-neutron | 08:41 | |
| *** uschreiber_ has quit IRC | 08:42 | |
| *** jpena has joined #openstack-neutron | 08:42 | |
| *** uschreiber_ has joined #openstack-neutron | 08:42 | |
| *** uschreiber_ has quit IRC | 08:42 | |
| sudipto_ | Hi, is there a good document/guideline to follow - if we introduce a new exception in neutron? I am referring to the messaging changes needed... | 08:42 |
| *** nyechiel__ has quit IRC | 08:42 | |
| *** topshare_ has joined #openstack-neutron | 08:47 | |
| *** topshare has quit IRC | 08:50 | |
| *** nyechiel_ has joined #openstack-neutron | 08:51 | |
| *** ajo has joined #openstack-neutron | 08:51 | |
| *** nyechiel has quit IRC | 08:54 | |
| *** topshare_ has quit IRC | 08:54 | |
| *** numan has quit IRC | 08:56 | |
| *** rotbeard has quit IRC | 08:57 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 08:58 |
| *** topshare has joined #openstack-neutron | 09:04 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 09:05 |
| *** erkules_ is now known as erkules | 09:07 | |
| *** erkules has joined #openstack-neutron | 09:07 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 09:08 |
| *** Networkn_ has quit IRC | 09:10 | |
| *** vivek-ebay has joined #openstack-neutron | 09:10 | |
| *** numan has joined #openstack-neutron | 09:10 | |
| *** Networkn3rd has joined #openstack-neutron | 09:11 | |
| *** gongysh has quit IRC | 09:11 | |
| *** topshare has quit IRC | 09:12 | |
| *** boris-42 has quit IRC | 09:12 | |
| *** topshare has joined #openstack-neutron | 09:12 | |
| *** ygbo has joined #openstack-neutron | 09:14 | |
| *** vivek-ebay has quit IRC | 09:14 | |
| *** watanabe_isao has quit IRC | 09:18 | |
| *** markvoelker has joined #openstack-neutron | 09:19 | |
| *** hichihara has quit IRC | 09:23 | |
| *** markvoelker has quit IRC | 09:23 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack tests framework https://review.openstack.org/128259 | 09:24 |
| *** blues-man has joined #openstack-neutron | 09:25 | |
| *** topshare_ has joined #openstack-neutron | 09:25 | |
| *** Networkn3rd has quit IRC | 09:26 | |
| *** rook-tmp has joined #openstack-neutron | 09:26 | |
| *** Networkn3rd has joined #openstack-neutron | 09:27 | |
| *** mleroy_ has joined #openstack-neutron | 09:28 | |
| *** topshare has quit IRC | 09:28 | |
| *** afazekas_ has joined #openstack-neutron | 09:31 | |
| *** rook-tmp has quit IRC | 09:33 | |
| *** numan has quit IRC | 09:35 | |
| *** mleroy_ has quit IRC | 09:35 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 09:36 |
| jschwarz | ZZelle, hello :) | 09:36 |
| ZZelle | jschwarz, hallo | 09:36 |
| jschwarz | ZZelle, sorry I've been away for a while | 09:36 |
| jschwarz | ZZelle, any patches you need me to review? :) | 09:37 |
| ZZelle | jschwarz, https://review.openstack.org/159852 and sons ... but i need to resolve merge conflicts (again :() | 09:38 |
| jschwarz | ZZelle, roger. once you do I'll have a look and do my best to get it | 09:38 |
| ZZelle | jschwarz, thx | 09:38 |
| *** dmellado has quit IRC | 09:38 | |
| jschwarz | ZZelle, though it's tests code so rumor has it that it only needs to go in by RC and not K-3 | 09:38 |
| *** nuritv has joined #openstack-neutron | 09:39 | |
| gsagie | Hello all, i am trying to debug the l3-agent with pycharm, but my break points are not stopping, anything special i need to do ? | 09:39 |
| ZZelle | jschwarz, k3 is tomorrow? | 09:39 |
| *** topshare_ has quit IRC | 09:40 | |
| *** gongysh has joined #openstack-neutron | 09:40 | |
| jschwarz | ZZelle, I believe so | 09:40 |
| *** vmtrooper has joined #openstack-neutron | 09:40 | |
| jschwarz | so RC1 is next week? or the week after? | 09:40 |
| ZZelle | jschwarz, we should also look at https://review.openstack.org/160051 and see how we can transform kernel bridges into BridgeFixture | 09:40 |
| ZZelle | jschwarz, no idea | 09:40 |
| jschwarz | https://wiki.openstack.org/wiki/Kilo_Release_Schedule | 09:40 |
| jschwarz | ohhhh RC1 is on the week of April 9th! | 09:40 |
| jschwarz | I didn't know we have so long! | 09:41 |
| ZZelle | jschwarz, i hope https://review.openstack.org/165115 will be merged before kilo | 09:41 |
| jschwarz | ZZelle, I didn't know that was still there | 09:42 |
| jschwarz | oh it's new | 09:42 |
| jschwarz | I was sure we discussed this patch a few months ago | 09:42 |
| *** gongysh has quit IRC | 09:43 | |
| ZZelle | jschwarz, honestly it's a little bit difficult to document the change and its parent correctly :( | 09:43 |
| *** ZZelle has quit IRC | 09:43 | |
| *** dims_ has joined #openstack-neutron | 09:43 | |
| *** ZZelle has joined #openstack-neutron | 09:43 | |
| jschwarz | ZZelle, :(\ | 09:44 |
| ZZelle | jschwarz, yes, but i was waiting for https://review.openstack.org/147437 merge | 09:44 |
| *** jlanoux has joined #openstack-neutron | 09:44 | |
| *** kerpan has joined #openstack-neutron | 09:45 | |
| *** afazekas_ has quit IRC | 09:45 | |
| *** vmtrooper has quit IRC | 09:45 | |
| *** alexpilotti has joined #openstack-neutron | 09:47 | |
| *** numan has joined #openstack-neutron | 09:48 | |
| *** yog__ has quit IRC | 09:49 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add sanity check for wrong MTU when using tunnels https://review.openstack.org/129852 | 09:52 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 09:53 |
| *** yog__ has joined #openstack-neutron | 09:55 | |
| *** john-davidge has joined #openstack-neutron | 09:59 | |
| *** yog__ has quit IRC | 09:59 | |
| *** topshare has joined #openstack-neutron | 09:59 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 10:00 |
| *** crc32 has quit IRC | 10:01 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add sanity check for wrong MTU when using tunnels https://review.openstack.org/129852 | 10:05 |
| *** topshare has quit IRC | 10:05 | |
| *** gildub has joined #openstack-neutron | 10:09 | |
| *** halfss has joined #openstack-neutron | 10:11 | |
| *** xek has joined #openstack-neutron | 10:12 | |
| halfss | Hi, i have an openstack juno with neutron DVR, an then i launch an instance on host "compute", but neutron try to band port on host "control", is there some one can help me? | 10:13 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 10:17 |
| *** markvoelker has joined #openstack-neutron | 10:19 | |
| *** yalie has quit IRC | 10:21 | |
| *** topshare has joined #openstack-neutron | 10:22 | |
| *** markvoelker has quit IRC | 10:24 | |
| *** ihrachyshka has joined #openstack-neutron | 10:24 | |
| *** mestery has joined #openstack-neutron | 10:26 | |
| *** halfss has quit IRC | 10:27 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 10:28 |
| *** rook-tmp has joined #openstack-neutron | 10:29 | |
| *** saggi has quit IRC | 10:32 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default. https://review.openstack.org/165117 | 10:32 |
| ajo | jschwarz ^ | 10:33 |
| ajo | I was having an issue, when enabling it by default, where the "monitoring thread" stayed alive across unit tests... when somebody spawned an agent that spawned a process monitor... | 10:34 |
| ajo | and then it tried to restart processes in the background :) | 10:34 |
| *** rook-tmp has quit IRC | 10:35 | |
| openstackgerrit | Darragh O'Reilly proposed openstack/neutron: Make get_binding_levels work with short port_id https://review.openstack.org/165351 | 10:37 |
| *** yfried is now known as yfried|afk | 10:37 | |
| openstackgerrit | Boris Pavlovic proposed openstack/neutron: Run more Rally benchmark on every patch https://review.openstack.org/164428 | 10:38 |
| *** yfried|afk is now known as yfried | 10:40 | |
| *** crc32 has joined #openstack-neutron | 10:40 | |
| *** rotbeard has joined #openstack-neutron | 10:43 | |
| sudipto_ | jschwarz: Hi, if i have introduce a new exception...I believe the exception message needs to have the translation changes in place...could you please tell me what i need to do to w.r.t introducing a new exception? | 10:49 |
| *** boris-42 has joined #openstack-neutron | 10:51 | |
| *** jyuso1 has quit IRC | 10:51 | |
| *** stanzgy_ has quit IRC | 10:53 | |
| *** stanzgy has quit IRC | 10:53 | |
| *** Bsony_ has joined #openstack-neutron | 10:54 | |
| *** geguileo has joined #openstack-neutron | 10:54 | |
| *** yamamoto has quit IRC | 10:55 | |
| ajo | ping enikanorov , you were working on the DB retry mechanisms to avoid SELECT FOR UPDATE queries, right? | 10:55 |
| ajo | enikanorov, what's the final status / final solution? | 10:55 |
| ajo | enikanorov, I know geguileo is tackling the same stuff for other openstack components, and I thought it was a good idea asking | 10:56 |
| *** Bsony has quit IRC | 10:57 | |
| *** gsagie has quit IRC | 10:59 | |
| *** baoli has joined #openstack-neutron | 11:02 | |
| *** pc_m has joined #openstack-neutron | 11:02 | |
| jschwarz | sudipto_, I believe you can simply follow the conventions that are already in that file | 11:05 |
| jschwarz | sudipto_, where are you adding the new exception? | 11:05 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 11:06 |
| *** doude has quit IRC | 11:07 | |
| sudipto_ | jschwarz, planning to add the exception inside agent.py | 11:07 |
| sudipto_ | jschwarz, neutron/extensions/agent.py | 11:08 |
| jschwarz | sudipto_, I believe just surrounding the message with a _(...) is enough | 11:08 |
| jschwarz | sudipto_, if it isn't it will come up during the review process :) | 11:08 |
| sudipto_ | jschwarz, yeah just did that much and thought i will wait for the review :) | 11:09 |
| sudipto_ | jschwarz, btw - your thoughts on https://bugs.launchpad.net/neutron/+bug/1432582 I was actually writing a fix for this. | 11:10 |
| openstack | Launchpad bug 1432582 in neutron "OVS agent shows connected to AMQP but neutron server rejects the request." [Undecided,In progress] - Assigned to Sudipta Biswas (sbiswas7) | 11:10 |
| *** amotoki has joined #openstack-neutron | 11:10 | |
| jschwarz | sudipto_, I think that fixing that bug sounds awesome | 11:11 |
| sudipto_ | jschwarz: Ok :) | 11:11 |
| jschwarz | sudipto_, once it's up it will hopefully get some reviewers to look at it :) | 11:12 |
| jschwarz | sudipto_, thanks for working on this | 11:12 |
| *** mwagner_lap has quit IRC | 11:12 | |
| alexpilotti | salv-orlando: hello | 11:13 |
| sudipto_ | jschwarz: Yeah, i spotted that in my own environment. BTW, i have been trying to get my hands dirty to help contribute to neutron. I have read most of the docs on the net about the process et all, but would you have suggestions on how i could help more effectively? | 11:13 |
| jschwarz | sudipto_, it's really like writing any other code, except the review process takes longer, so patience is key :) | 11:14 |
| jschwarz | sudipto_, if you hang about long enough you'll get the hang of things | 11:14 |
| kevinbenton | enikanorov: poing | 11:14 |
| sudipto_ | jschwarz: I shall. :) | 11:14 |
| kevinbenton | enikanorov: whoops, ping* | 11:14 |
| jschwarz | sudipto_, excellent :) | 11:14 |
| jschwarz | sudipto_, make sure you add me as a reviewer when your patch is up and I'll try to have a look at it: jschwarz@redhat.com | 11:15 |
| sudipto_ | jchwarz: Sure i will. Thanks!! :) | 11:15 |
| sudipto_ | jschwarz: Also if there's anything, that you may want me to take a look at, do let me know. Not sure if it really works this way, but I am just trying to find a headstart of sorts :) | 11:16 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Only call get_engine().pool.dispose if _FACADE https://review.openstack.org/165364 | 11:18 |
| jschwarz | sudipto_, you might want to have a look at the low-hanging fruits -- http://goo.gl/1y0yZ8 | 11:18 |
| sudipto_ | jschwarz: Yeah scanned that. All seems to be taken there :) | 11:19 |
| jschwarz | sudipto_, that link I gave you specifically gives you unassigned bugs | 11:19 |
| *** markvoelker has joined #openstack-neutron | 11:20 | |
| sudipto_ | jschwarz: Thanks, just saw :) | 11:20 |
| *** baoli has quit IRC | 11:21 | |
| jschwarz | sudipto_, sure think XD | 11:21 |
| jschwarz | s/think/thing/g | 11:21 |
| *** baoli has joined #openstack-neutron | 11:21 | |
| *** yamamoto has joined #openstack-neutron | 11:21 | |
| *** topshare has quit IRC | 11:23 | |
| *** yalie has joined #openstack-neutron | 11:24 | |
| *** jp_at_hp has joined #openstack-neutron | 11:24 | |
| *** markvoelker has quit IRC | 11:25 | |
| *** bashok has joined #openstack-neutron | 11:27 | |
| *** vmtrooper has joined #openstack-neutron | 11:29 | |
| *** rook-tmp has joined #openstack-neutron | 11:31 | |
| *** numan has quit IRC | 11:33 | |
| *** vmtrooper has quit IRC | 11:34 | |
| openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 11:34 |
| *** rook-tmp has quit IRC | 11:36 | |
| *** jprovazn has quit IRC | 11:37 | |
| *** jprovazn has joined #openstack-neutron | 11:40 | |
| *** marrusl has joined #openstack-neutron | 11:40 | |
| *** setuid has quit IRC | 11:44 | |
| *** achanda has joined #openstack-neutron | 11:45 | |
| *** setuid has joined #openstack-neutron | 11:45 | |
| *** ChrisNBlum has joined #openstack-neutron | 11:45 | |
| *** yrn has joined #openstack-neutron | 11:46 | |
| yrn | Hi, did anyone here used linuxbridge with flat network? | 11:46 |
| *** achanda has quit IRC | 11:49 | |
| *** saggi has joined #openstack-neutron | 11:49 | |
| beagles | rkukura, are you about? | 11:56 |
| *** ronis has quit IRC | 11:58 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 12:00 |
| *** david-lyle_afk is now known as david-lyle | 12:00 | |
| *** Bhavya has quit IRC | 12:02 | |
| *** markvoelker has joined #openstack-neutron | 12:02 | |
| *** Bhavya has joined #openstack-neutron | 12:02 | |
| *** Miouge has joined #openstack-neutron | 12:02 | |
| *** mwagner_lap has joined #openstack-neutron | 12:03 | |
| *** ajo has quit IRC | 12:03 | |
| *** ajo has joined #openstack-neutron | 12:03 | |
| *** pradipta has quit IRC | 12:04 | |
| *** jschwarz has quit IRC | 12:10 | |
| *** gildub has quit IRC | 12:11 | |
| *** marun has joined #openstack-neutron | 12:12 | |
| *** shwetaap has joined #openstack-neutron | 12:13 | |
| *** sudipto_ has quit IRC | 12:16 | |
| *** wverdugo500 has left #openstack-neutron | 12:17 | |
| *** marun has quit IRC | 12:17 | |
| *** bradjones has joined #openstack-neutron | 12:17 | |
| *** bradjones has quit IRC | 12:17 | |
| *** bradjones has joined #openstack-neutron | 12:17 | |
| openstackgerrit | Paul Michali proposed openstack/neutron: Refactoring of L3 agent notifications for router https://review.openstack.org/164466 | 12:19 |
| *** yfauser has joined #openstack-neutron | 12:20 | |
| *** dims_ has quit IRC | 12:22 | |
| *** dims_ has joined #openstack-neutron | 12:22 | |
| *** gsagie has joined #openstack-neutron | 12:23 | |
| *** yfauser has left #openstack-neutron | 12:23 | |
| *** popw has quit IRC | 12:23 | |
| *** amuller has joined #openstack-neutron | 12:25 | |
| *** dmellado has joined #openstack-neutron | 12:26 | |
| *** kickinz1 is now known as kickinz1|afk | 12:26 | |
| *** kickinz1|afk is now known as kickinz1 | 12:26 | |
| *** ajmiller has quit IRC | 12:26 | |
| *** raildo_away is now known as raildo | 12:28 | |
| *** blues-man has quit IRC | 12:29 | |
| *** rook-tmp has joined #openstack-neutron | 12:32 | |
| *** saggi has quit IRC | 12:34 | |
| *** wverdugo500 has joined #openstack-neutron | 12:35 | |
| *** wverdugo500 has left #openstack-neutron | 12:35 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 12:36 |
| *** rook-tmp has quit IRC | 12:37 | |
| *** yalie has left #openstack-neutron | 12:38 | |
| *** jistr is now known as jistr|biab | 12:39 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 12:40 |
| *** jpena is now known as jpena|lunch | 12:41 | |
| *** saggi has joined #openstack-neutron | 12:42 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 12:42 |
| *** pal_ has joined #openstack-neutron | 12:44 | |
| *** pal__ has joined #openstack-neutron | 12:46 | |
| *** pal has quit IRC | 12:46 | |
| *** pal has joined #openstack-neutron | 12:48 | |
| *** pal_ has quit IRC | 12:49 | |
| *** geguileo has quit IRC | 12:50 | |
| *** geguileo has joined #openstack-neutron | 12:51 | |
| *** aepifanov has quit IRC | 12:51 | |
| *** pal__ has quit IRC | 12:52 | |
| *** pal has quit IRC | 12:53 | |
| openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 12:54 |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 12:55 |
| *** geguileo has quit IRC | 12:56 | |
| *** salv-orlando has quit IRC | 12:56 | |
| *** salv-orlando has joined #openstack-neutron | 12:57 | |
| openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 13:00 |
| *** Murali has quit IRC | 13:00 | |
| *** changbl has quit IRC | 13:00 | |
| openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 13:01 |
| *** boris-42 has quit IRC | 13:02 | |
| openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 13:02 |
| *** amotoki has quit IRC | 13:03 | |
| *** amotoki has joined #openstack-neutron | 13:05 | |
| *** banix has joined #openstack-neutron | 13:06 | |
| *** banix has quit IRC | 13:07 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 13:11 |
| openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 13:11 |
| *** kbringard has joined #openstack-neutron | 13:13 | |
| *** julim has joined #openstack-neutron | 13:14 | |
| *** diegows has joined #openstack-neutron | 13:14 | |
| *** vmtrooper has joined #openstack-neutron | 13:17 | |
| *** absubram has joined #openstack-neutron | 13:19 | |
| absubram | amotoki: Hi.. do you a minute? I wanted to mention to you something about the NEC tempest tests | 13:20 |
| amotoki | absubram: hi | 13:20 |
| *** britthouser has joined #openstack-neutron | 13:20 | |
| absubram | hey Akihiro :).. Just wanted to bring to your attention - https://review.openstack.org/#/c/156283/ | 13:21 |
| absubram | this will break one of the NEC tests - | 13:21 |
| *** Bhavya has quit IRC | 13:21 | |
| absubram | networking_nec.tests.unit.openflow.test_router.NecRouterOpenFlowTestCase | 13:21 |
| amotoki | absubram: thanks for bringing up. | 13:21 |
| absubram | I have a fix.. if you need it :) | 13:21 |
| absubram | it is because of a referenced subnets[0].. | 13:22 |
| amotoki | absubram: ah, great! | 13:22 |
| *** vmtrooper has quit IRC | 13:22 | |
| absubram | with this change, gateways will not necessarily always have a subnet associated with them.. that’s what was breaking your test | 13:22 |
| *** setuid has quit IRC | 13:22 | |
| amotoki | absubram: IPv6 router? | 13:23 |
| absubram | so that’s what the fix in the NEC code needs to do.. | 13:23 |
| absubram | yes | 13:23 |
| absubram | if you look at patchset 8 - that had a fix for the NEC code before it got moved to stackforge | 13:24 |
| amotoki | absubram: let me check the unit test chnage in your review. I can disable it for a while. | 13:24 |
| *** setuid has joined #openstack-neutron | 13:24 | |
| absubram | sure.. it doesn’t give me a -1.. so it’s ok.. but the non-voting one fails.. becuase of this one test | 13:25 |
| *** diegows has quit IRC | 13:25 | |
| amotoki | absubram: yeah, it is the reason of non-voting and it is my intention. you are the first person who letting me know the break :-) | 13:26 |
| absubram | haha ok :) | 13:26 |
| *** prad has joined #openstack-neutron | 13:27 | |
| *** jckasper has joined #openstack-neutron | 13:27 | |
| amotoki | absubram: sometimes, intentionally or not, changes in the unit test in neutorn breaks unit tests of various vendor pluigns/drivers :-( it is one of the downside of decomposition. | 13:27 |
| absubram | so the code has changed since patchset version 8.. but you need to essentially check in line 155 for subnet_id before referenceing it | 13:27 |
| absubram | in nec_router.py | 13:27 |
| amotoki | I think your change is intentional and sounds good to me. | 13:28 |
| *** ChuckC has joined #openstack-neutron | 13:28 | |
| amotoki | absubram: looking | 13:28 |
| absubram | yes.. this is true.. but the point of this BP was to change the API behavior.. thanks for agreeing to the change :) | 13:29 |
| *** yatin has quit IRC | 13:29 | |
| *** Miouge has quit IRC | 13:30 | |
| absubram | sorry you renamed the file from nec_rouer.py to router.py it looks like | 13:31 |
| amotoki | absubram: most of our current code focus on IPv4 and i know some changes require to support IPv6. | 13:32 |
| absubram | the change I mentioned in patchset 8 will not strictly translate now because of some of the code from my dependency patch.. but it will be a refernce for you.. I don’t konw how to test a fix in the stackforge repo :( | 13:32 |
| absubram | amotoki: that is correct.. it is why this test failed for the nec case | 13:32 |
| amotoki | absubram: I will post a patch to skip that test once and then investigate the detail :-) nec plugin has two backend: l3-agent and openflow router, so I need more investigation. | 13:33 |
| *** rook-tmp has joined #openstack-neutron | 13:33 | |
| *** chem has quit IRC | 13:33 | |
| absubram | amotoki: sure thing.. ty | 13:33 |
| *** salv-orlando has quit IRC | 13:34 | |
| *** zzzeek has joined #openstack-neutron | 13:34 | |
| *** alexpilotti has quit IRC | 13:35 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 13:35 |
| *** rook-rook has joined #openstack-neutron | 13:36 | |
| *** shwetaap has quit IRC | 13:36 | |
| *** rook has quit IRC | 13:36 | |
| *** rook-rook is now known as rook | 13:36 | |
| *** rook is now known as Guest90235 | 13:37 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 13:38 |
| *** crc32 has quit IRC | 13:38 | |
| *** dane_leblanc has joined #openstack-neutron | 13:38 | |
| *** dboik has joined #openstack-neutron | 13:39 | |
| *** jistr|biab is now known as jistr | 13:39 | |
| gsagie | Hello all, when debugging the L3 agent code with Pycharm the debugger doesnt stop on my breakpoints (i have Gevent compatible debugging enabled and debug neutron server correctly) anyone else experienced the same problem? | 13:39 |
| *** boris-42 has joined #openstack-neutron | 13:40 | |
| *** topshare has joined #openstack-neutron | 13:41 | |
| *** shwetaap has joined #openstack-neutron | 13:47 | |
| *** enikanorov_ has joined #openstack-neutron | 13:47 | |
| *** csoukup has joined #openstack-neutron | 13:48 | |
| *** ishant has quit IRC | 13:48 | |
| *** enikanorov has quit IRC | 13:49 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: don't rely on configuration files outside tests directory https://review.openstack.org/165237 | 13:50 |
| *** numan has joined #openstack-neutron | 13:50 | |
| *** fifieldt has quit IRC | 13:51 | |
| *** shwetaap has quit IRC | 13:51 | |
| *** zzzeek has quit IRC | 13:52 | |
| *** blues-man has joined #openstack-neutron | 13:54 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-fwaas: tests: stop overwriting neutron BaseTestCase configuration files https://review.openstack.org/165419 | 13:54 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 13:55 |
| *** nyechiel_ has quit IRC | 13:55 | |
| *** thomasem has joined #openstack-neutron | 13:56 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: tests: stop overwriting neutron BaseTestCase configuration files https://review.openstack.org/165420 | 13:56 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: tests: stop overwriting neutron BaseTestCase configuration files https://review.openstack.org/165421 | 13:57 |
| *** zz_jgrimm is now known as jgrimm | 13:57 | |
| *** kbringard1 has joined #openstack-neutron | 13:58 | |
| ihrachyshka | pc_m, zigo, https://review.openstack.org/#/q/If1f5ebd981cf06558d5102524211799676068889,n,z I expect that series to help with the config file problem when running unit tests for *aas repos | 13:59 |
| *** kbringard has quit IRC | 14:00 | |
| *** blues-man has quit IRC | 14:00 | |
| *** blues-man has joined #openstack-neutron | 14:00 | |
| *** rushil has quit IRC | 14:01 | |
| *** rushil has joined #openstack-neutron | 14:01 | |
| *** kbringard1 is now known as kbringard | 14:01 | |
| *** thomasem has quit IRC | 14:01 | |
| *** vthapar has quit IRC | 14:02 | |
| *** lykinsbd has quit IRC | 14:02 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 14:02 |
| *** fredlhsu has joined #openstack-neutron | 14:02 | |
| *** alexpilotti has joined #openstack-neutron | 14:03 | |
| *** jschwarz has joined #openstack-neutron | 14:04 | |
| *** thomasem has joined #openstack-neutron | 14:04 | |
| *** adelinatuvenie has quit IRC | 14:05 | |
| pc_m | ihrachyshka: Can you elaborate on what is going on with the change? It looks like the base test will no longer look in /opt/stack/neutron/etc/ for neutron.conf. Is that correct? Relying on the one in /opt/stack/neutron/tests/etc/? | 14:06 |
| *** changbl has joined #openstack-neutron | 14:06 | |
| *** rook-tmp has quit IRC | 14:07 | |
| *** sudipto_ has joined #openstack-neutron | 14:07 | |
| openstackgerrit | Merged openstack/python-neutronclient: Added client calls for the lbaas v2 agent scheduler https://review.openstack.org/159218 | 14:08 |
| *** chem has joined #openstack-neutron | 14:08 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 14:08 |
| *** lykinsbd has joined #openstack-neutron | 14:08 | |
| ihrachyshka | pc_m, right. it always read neutron/tests/etc/neutron.conf.test, but my recent change also made it to read etc/neutron.conf (just to make oslo.config also pick up etc/policy.json). the problem with that approach is that etc/policy.json is not available under the same relative directory starting from neutron/tests/base.py as in two cases - one for running tests from the tree, another one - running tests from site-packages i | 14:09 |
| ihrachyshka | nstallation (where ../../../etc/neutron.conf won't locate the file since it's in different place determined by pip installation rules) | 14:09 |
| *** thomasem has quit IRC | 14:09 | |
| *** thomasem has joined #openstack-neutron | 14:09 | |
| ihrachyshka | pc_m, the fix is to isolate all conf files used by neutron BaseTestCase inside tests tree to avoid difference in config file paths | 14:10 |
| ihrachyshka | pc_m, as for *aas patches, there is some hack around that plays bad with the changes in neutron (since it overrides the fix), and it should be safe to kill | 14:10 |
| *** topshare has quit IRC | 14:13 | |
| pc_m | ihrachyshka: So there are no cases, where UT code relies on the original /opt/stack/neutron/etc/neutron.conf file, versus the test file? | 14:13 |
| ihrachyshka | pc_m, I don't know of any, and I would also say it would be a bad thing to do | 14:14 |
| ihrachyshka | pc_m, also, neutron.conf is pretty emty | 14:14 |
| *** alexpilotti has quit IRC | 14:14 | |
| ihrachyshka | *empty | 14:14 |
| pc_m | ihrachyshka: Yeah, I guess it has been moved to *aas conf files for a bunch. | 14:15 |
| pc_m | ihrachyshka: So the policy.json file, the *aaS code is overriding to look in /opt/stack/neutron/etc/policy.json. Is there one in the test area? | 14:15 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports https://review.openstack.org/156360 | 14:15 |
| *** alexpilotti has joined #openstack-neutron | 14:15 | |
| *** tflynn has joined #openstack-neutron | 14:15 | |
| pc_m | ihrachyshka: I don't see one. | 14:16 |
| *** localloop127 has joined #openstack-neutron | 14:16 | |
| pc_m | ihrachyshka: It looks like the override was to point to /opt/stack/neutron/etc/policy.json | 14:17 |
| *** ronis has joined #openstack-neutron | 14:17 | |
| pc_m | ihrachyshka: nevermind. I see you have added one. | 14:17 |
| pc_m | ihrachyshka: So the only concern is that people have to keep two copies of that file up to date. | 14:18 |
| pc_m | ihrachyshka: I see you have a test though. | 14:19 |
| pc_m | ihrachyshka: So this file can't be referenced in /opt/stack/neutron/etc, because of running tests from site packages? | 14:19 |
| ihrachyshka | pc_m, right. there is no immediate need to make them identical, but it's better to enforce it (so that people don't get confused when they don't have access to policy rules they add in unit testing) | 14:19 |
| ihrachyshka | pc_m, there is no /opt/stack/ outside devstack at all | 14:20 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete https://review.openstack.org/161073 | 14:20 |
| ihrachyshka | pc_m, and obviously packagers don't use devstack :) | 14:20 |
| openstackgerrit | Andrew Boik proposed openstack/neutron: Auto-update gateway port after subnet-create https://review.openstack.org/161085 | 14:20 |
| pc_m | ihrachyshka: thanks for the clarification! | 14:21 |
| pc_m | ihrachyshka: and the work to clean this up! | 14:21 |
| *** shwetaap has joined #openstack-neutron | 14:22 | |
| ihrachyshka | pc_m, that was my bad that I introduced the issue in the first place | 14:22 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 14:22 |
| *** tflynn has quit IRC | 14:22 | |
| pc_m | ihrachyshka: hey s2t happens :) | 14:23 |
| *** lazy_prince is now known as killer_prince | 14:23 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 14:25 |
| *** tflynn has joined #openstack-neutron | 14:26 | |
| *** mestery has quit IRC | 14:27 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 14:27 |
| ZZelle | kevinbenton, hi | 14:27 |
| *** moha_hunt has quit IRC | 14:27 | |
| openstackgerrit | Nurit Vilosny proposed openstack/neutron: Add notification flag (True / False) to extension_manager process_update_port https://review.openstack.org/162648 | 14:28 |
| gsagie | Romil Gupta? | 14:28 |
| openstackgerrit | Merged openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 14:29 |
| *** tsg_ has joined #openstack-neutron | 14:30 | |
| *** tflynn has quit IRC | 14:32 | |
| *** tsg_ has quit IRC | 14:32 | |
| openstackgerrit | Merged openstack/python-neutronclient: Updates pool session persistence options https://review.openstack.org/163682 | 14:33 |
| *** lykinsbd has quit IRC | 14:33 | |
| *** lykinsbd has joined #openstack-neutron | 14:34 | |
| *** ajmiller has joined #openstack-neutron | 14:34 | |
| *** changbl has quit IRC | 14:38 | |
| *** jprovazn has quit IRC | 14:38 | |
| *** rotbeard has quit IRC | 14:38 | |
| *** rotbeard has joined #openstack-neutron | 14:40 | |
| *** adelinatuvenie has joined #openstack-neutron | 14:41 | |
| *** jprovazn has joined #openstack-neutron | 14:41 | |
| ZZelle | ihrachyshka, hi | 14:41 |
| *** killer_prince is now known as lazy_prince | 14:42 | |
| *** nyechiel_ has joined #openstack-neutron | 14:42 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Include IPv6 SLAAC addresses implicitly for port update https://review.openstack.org/161033 | 14:43 |
| *** anilvenkata has quit IRC | 14:45 | |
| *** achanda has joined #openstack-neutron | 14:47 | |
| *** Guest90235 is now known as rook | 14:49 | |
| *** neeti has quit IRC | 14:49 | |
| *** aranjan has joined #openstack-neutron | 14:49 | |
| *** SridharG has left #openstack-neutron | 14:50 | |
| *** trad511 has joined #openstack-neutron | 14:50 | |
| *** shaohe_feng1 has joined #openstack-neutron | 14:50 | |
| *** tsufiev_ has left #openstack-neutron | 14:51 | |
| *** achanda has quit IRC | 14:52 | |
| *** fredlhsu has quit IRC | 14:52 | |
| *** itzikb has quit IRC | 14:53 | |
| *** tflynn has joined #openstack-neutron | 14:54 | |
| *** aranjan has quit IRC | 14:55 | |
| *** Bsony has joined #openstack-neutron | 14:55 | |
| *** yfried is now known as yfried|afk | 14:56 | |
| *** trad511 has quit IRC | 14:56 | |
| *** trad511 has joined #openstack-neutron | 14:56 | |
| *** Longgeek has quit IRC | 14:56 | |
| *** dave-mccowan has joined #openstack-neutron | 14:57 | |
| *** Bsony_ has quit IRC | 14:57 | |
| *** yamamoto has quit IRC | 14:58 | |
| *** tflynn has quit IRC | 14:58 | |
| *** yamamoto has joined #openstack-neutron | 14:59 | |
| *** yfried|afk is now known as yfried | 14:59 | |
| *** reed has joined #openstack-neutron | 14:59 | |
| *** chandankumar has quit IRC | 14:59 | |
| *** aranjan has joined #openstack-neutron | 14:59 | |
| *** chandankumar has joined #openstack-neutron | 14:59 | |
| *** aranjan_ has joined #openstack-neutron | 15:00 | |
| *** aranjan has quit IRC | 15:00 | |
| *** pc_m has quit IRC | 15:00 | |
| *** changbl has joined #openstack-neutron | 15:01 | |
| *** nyechiel_ is now known as nyechiel | 15:02 | |
| *** rook-tmp has joined #openstack-neutron | 15:03 | |
| *** sridhar_ram has joined #openstack-neutron | 15:04 | |
| *** shaohe_feng1 has quit IRC | 15:04 | |
| *** TrevorV_ has joined #openstack-neutron | 15:05 | |
| *** aranjan_ has quit IRC | 15:05 | |
| *** amotoki has quit IRC | 15:06 | |
| *** vmtrooper has joined #openstack-neutron | 15:06 | |
| *** garyk has quit IRC | 15:07 | |
| *** bobmel has quit IRC | 15:07 | |
| *** bobmel has joined #openstack-neutron | 15:07 | |
| *** Marga_ has joined #openstack-neutron | 15:07 | |
| *** salv-orlando has joined #openstack-neutron | 15:09 | |
| *** Trozz has quit IRC | 15:10 | |
| *** Trozz has joined #openstack-neutron | 15:11 | |
| *** claudiub has joined #openstack-neutron | 15:11 | |
| *** vmtrooper has quit IRC | 15:11 | |
| *** jaypipes has quit IRC | 15:12 | |
| *** nlahouti has joined #openstack-neutron | 15:13 | |
| *** watanabe_isao has joined #openstack-neutron | 15:15 | |
| *** shaohe_feng1 has joined #openstack-neutron | 15:16 | |
| *** amotoki has joined #openstack-neutron | 15:16 | |
| *** carl_baldwin has joined #openstack-neutron | 15:16 | |
| *** salv-orlando has quit IRC | 15:17 | |
| *** dave-mccowan has quit IRC | 15:17 | |
| *** regXboi has joined #openstack-neutron | 15:17 | |
| *** regXboi has left #openstack-neutron | 15:18 | |
| *** pcaruana has joined #openstack-neutron | 15:19 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack tests framework https://review.openstack.org/128259 | 15:19 |
| *** dave-mccowan has joined #openstack-neutron | 15:20 | |
| openstackgerrit | Jiaqiang Liu proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor https://review.openstack.org/142386 | 15:21 |
| *** watanabe_isao has quit IRC | 15:21 | |
| *** vishwanathj has quit IRC | 15:22 | |
| *** crc32 has joined #openstack-neutron | 15:26 | |
| *** ajmiller has quit IRC | 15:26 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 15:27 |
| *** lizk_ has joined #openstack-neutron | 15:27 | |
| *** lazy_prince is now known as killer_prince | 15:29 | |
| *** bobmel has quit IRC | 15:30 | |
| *** pc_m has joined #openstack-neutron | 15:30 | |
| *** shivharis-1 has joined #openstack-neutron | 15:30 | |
| *** ajmiller has joined #openstack-neutron | 15:30 | |
| *** bobmel has joined #openstack-neutron | 15:30 | |
| *** crc32 has quit IRC | 15:30 | |
| *** yfried is now known as yfried|afk | 15:32 | |
| *** sudipto_ has quit IRC | 15:33 | |
| *** jp_at_hp has quit IRC | 15:33 | |
| *** bobmel has quit IRC | 15:34 | |
| *** yfried|afk is now known as yfried | 15:35 | |
| *** _cjones_ has joined #openstack-neutron | 15:36 | |
| *** armax has joined #openstack-neutron | 15:37 | |
| *** rook-tmp has quit IRC | 15:37 | |
| *** banix has joined #openstack-neutron | 15:37 | |
| *** SridharGaddam has quit IRC | 15:37 | |
| *** amotoki has quit IRC | 15:38 | |
| *** julim has quit IRC | 15:38 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 15:38 |
| *** absubram has quit IRC | 15:40 | |
| *** bobmel has joined #openstack-neutron | 15:40 | |
| *** topshare has joined #openstack-neutron | 15:40 | |
| *** Zengfa has joined #openstack-neutron | 15:40 | |
| *** julim has joined #openstack-neutron | 15:41 | |
| *** packet has joined #openstack-neutron | 15:41 | |
| *** changbl has quit IRC | 15:44 | |
| *** changbl has joined #openstack-neutron | 15:45 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Test the shiny new WeightScheduler https://review.openstack.org/162231 | 15:45 |
| *** jpena|lunch is now known as jpena | 15:46 | |
| *** yatin has joined #openstack-neutron | 15:46 | |
| *** matrohon has quit IRC | 15:47 | |
| *** mestery has joined #openstack-neutron | 15:52 | |
| *** Sukhdev has joined #openstack-neutron | 15:53 | |
| *** garyk has joined #openstack-neutron | 15:53 | |
| *** tidwellr has joined #openstack-neutron | 15:54 | |
| *** bobmel_ has joined #openstack-neutron | 15:55 | |
| *** blues-man has quit IRC | 15:56 | |
| *** SumitNaiksatam has quit IRC | 15:57 | |
| *** salv-orl_ has joined #openstack-neutron | 15:58 | |
| *** bobmel has quit IRC | 15:58 | |
| *** vishwanathj has joined #openstack-neutron | 15:59 | |
| *** thedodd has joined #openstack-neutron | 15:59 | |
| *** tflynn_ has joined #openstack-neutron | 15:59 | |
| openstackgerrit | Saggi Mizrahi proposed openstack/neutron: Fix minor nits in _notify_l3_agent_new_port() https://review.openstack.org/165472 | 15:59 |
| openstackgerrit | Saggi Mizrahi proposed openstack/neutron: Use notification on port deletion https://review.openstack.org/165473 | 16:00 |
| *** ijw has joined #openstack-neutron | 16:00 | |
| *** bobmel has joined #openstack-neutron | 16:00 | |
| *** nmagnezi has quit IRC | 16:01 | |
| *** xgerman has joined #openstack-neutron | 16:01 | |
| *** yamahata has joined #openstack-neutron | 16:02 | |
| *** rook-tmp has joined #openstack-neutron | 16:02 | |
| *** ijw has quit IRC | 16:02 | |
| *** tflynn_ has quit IRC | 16:02 | |
| *** ijw has joined #openstack-neutron | 16:02 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 16:02 |
| *** chandankumar has quit IRC | 16:03 | |
| *** tflynn has joined #openstack-neutron | 16:03 | |
| *** topshare has quit IRC | 16:03 | |
| *** apuimedo has quit IRC | 16:04 | |
| *** bobmel_ has quit IRC | 16:04 | |
| openstackgerrit | Merged openstack/neutron: Run more Rally benchmark on every patch https://review.openstack.org/164428 | 16:04 |
| *** ijw has quit IRC | 16:04 | |
| mestery | carl_baldwin armax salv-orl_: I'm moving IPAM and subnet pool BPs to essential and RC1, they all get FFEs. Please continue merging patches for kilo-3 as appropriate, but they have some additional time now. | 16:05 |
| *** ijw has joined #openstack-neutron | 16:05 | |
| armax | mestery: ok | 16:06 |
| carl_baldwin | mestery: ack | 16:06 |
| mestery | armax: They wouldn't complete by today anyways, so just moved them to RC1. | 16:06 |
| mestery | Keep the momemntum going though! :) | 16:06 |
| *** changbl has quit IRC | 16:06 | |
| *** belmoreira has quit IRC | 16:07 | |
| sc68cal | mestery: based on some of my concerns re prefix delegation we may want to ping the bp owners about moving to liberty | 16:07 |
| * carl_baldwin going to take a nap. ;) | 16:07 | |
| *** jgrimm_ has joined #openstack-neutron | 16:07 | |
| *** banix_ has joined #openstack-neutron | 16:07 | |
| sc68cal | since I'm advocating PD depend on subnet pool work | 16:07 |
| mestery | sc68cal: Yes, I saw that, and I think it's effectively out given those concerns and a few other things | 16:07 |
| mestery | carl_baldwin: lucky guy :) | 16:07 |
| carl_baldwin | mestery: Just kidding. Going to keep the momentum going. :) | 16:09 |
| ZZelle | marios, hi | 16:09 |
| mestery | lol | 16:09 |
| *** jgrimm- has joined #openstack-neutron | 16:09 | |
| *** banix has quit IRC | 16:10 | |
| *** banix__ has joined #openstack-neutron | 16:10 | |
| * sc68cal crawls back into his devstack rabbit hole | 16:10 | |
| *** jgrimm has quit IRC | 16:10 | |
| mestery | sc68cal: This is the one, right? https://blueprints.launchpad.net/neutron/+spec/multiple-ipv6-prefixes | 16:11 |
| *** sripriya has joined #openstack-neutron | 16:12 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 16:12 |
| mestery | sc68cal: Thoughts on some of these, like htis one: https://review.openstack.org/#/c/113339/27 | 16:12 |
| *** banix_ has quit IRC | 16:12 | |
| sc68cal | mestery: will review | 16:13 |
| *** Bsony has quit IRC | 16:13 | |
| *** vivek-ebay has joined #openstack-neutron | 16:13 | |
| sc68cal | been uh, a couple patch sets since I last reviewed it :) | 16:13 |
| *** jgrimm_ has quit IRC | 16:13 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 16:13 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 16:16 |
| *** Swami has joined #openstack-neutron | 16:16 | |
| *** vishwanathj has quit IRC | 16:17 | |
| sc68cal | mestery: lgtm | 16:18 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 16:19 |
| *** rook-tmp has quit IRC | 16:19 | |
| *** vishwanathj has joined #openstack-neutron | 16:19 | |
| *** rook-tmp has joined #openstack-neutron | 16:19 | |
| *** killer_prince has quit IRC | 16:19 | |
| openstackgerrit | Mohammad Banikazemi proposed openstack/neutron: Adding a missing section header in devref https://review.openstack.org/165487 | 16:20 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS agent shouldnt start when dvr is enabled but l2pop is not https://review.openstack.org/165311 | 16:20 |
| *** killer_prince has joined #openstack-neutron | 16:20 | |
| *** killer_prince is now known as lazy_prince | 16:20 | |
| *** ronis has quit IRC | 16:22 | |
| *** gsagie has quit IRC | 16:22 | |
| *** anshul has quit IRC | 16:26 | |
| *** vhosakot has joined #openstack-neutron | 16:26 | |
| *** rotbeard has quit IRC | 16:27 | |
| *** lazy_prince has quit IRC | 16:29 | |
| *** jschwarz has quit IRC | 16:29 | |
| *** tflynn_ has joined #openstack-neutron | 16:29 | |
| *** trad511 has quit IRC | 16:29 | |
| *** leenheer has joined #openstack-neutron | 16:30 | |
| *** killer_prince has joined #openstack-neutron | 16:30 | |
| *** killer_prince is now known as lazy_prince | 16:30 | |
| *** tflynn_ has quit IRC | 16:31 | |
| *** scheuran has quit IRC | 16:31 | |
| *** rotbeard has joined #openstack-neutron | 16:31 | |
| *** tflynn_ has joined #openstack-neutron | 16:31 | |
| *** marun has joined #openstack-neutron | 16:31 | |
| *** tflynn has quit IRC | 16:32 | |
| *** vivek-ebay has quit IRC | 16:33 | |
| vhosakot | What is the right way to install python-neutronclient from source? I cloned the python-neutronclient repo from git.openstack.org (git clone git://git.openstack.org/openstack/python-neutronclient.git). Then, I ran these three commands - cd python-neutronclient, python setup.py build, python setup.py install. The installation passed, but I see this error when I run the “neutron” CLI command, I see the error “-bash: /us | 16:34 |
| vhosakot | I see that the installation was done in /usr/local/bin/neutron. And, running “which neutron” shows /usr/local/bin/neutron. Can anyone help me figure out what is wrong.. | 16:34 |
| *** aranjan has joined #openstack-neutron | 16:34 | |
| *** vishwana_ has joined #openstack-neutron | 16:36 | |
| *** lazy_prince has quit IRC | 16:37 | |
| *** zzzeek has joined #openstack-neutron | 16:38 | |
| *** killer_prince has joined #openstack-neutron | 16:38 | |
| *** killer_prince is now known as lazy_prince | 16:38 | |
| *** vishwanathj has quit IRC | 16:40 | |
| ZZelle | marun, hi | 16:40 |
| *** irenab has quit IRC | 16:41 | |
| *** aranjan has quit IRC | 16:41 | |
| *** vishwana_ has quit IRC | 16:41 | |
| *** aranjan has joined #openstack-neutron | 16:41 | |
| *** Bsony has joined #openstack-neutron | 16:41 | |
| marun | ZZelle: hi | 16:42 |
| *** Marga_ has quit IRC | 16:42 | |
| *** shivharis-1 has quit IRC | 16:42 | |
| *** Marga_ has joined #openstack-neutron | 16:43 | |
| *** ajmiller has quit IRC | 16:44 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 16:44 |
| ZZelle | marun, i updated https://review.openstack.org/159852 and added linuxbridge support as it's required by some iptables functional tests | 16:44 |
| *** akanksha has joined #openstack-neutron | 16:44 | |
| *** scheuran has joined #openstack-neutron | 16:44 | |
| marun | ZZelle: whoah | 16:44 |
| marun | ZZelle: I'd suggest adding the linuxbridge stuff in a separate patch, btw. | 16:45 |
| amuller | marun: Can we talk later about the functional test you had in mind for https://review.openstack.org/#/c/125384/43/neutron/agent/l3/keepalived_state_change.py? | 16:45 |
| *** lazy_prince has quit IRC | 16:46 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 16:46 |
| amuller | marun: I started working on something and it's growing to be pretty complicated | 16:46 |
| marun | amuller: I wasn't going to mandate more testing tbh | 16:46 |
| ZZelle | marun, adding lb support in a separate implies to rollback some refactoring so i did all in one change | 16:46 |
| *** thomasem has quit IRC | 16:46 | |
| amuller | marun: that's what you communicated... | 16:46 |
| marun | amuller: THe only actionable comment was my question re: error handling | 16:47 |
| *** jschwarz has joined #openstack-neutron | 16:47 | |
| marun | amuller: I'm not sure there's any easy fix for my frustration with the lingering complexity involved :/] | 16:47 |
| marun | ZZelle: :/ | 16:48 |
| marun | ZZelle: So was there lb support previously? | 16:48 |
| openstackgerrit | Merged openstack/neutron: Add portsecurity extension support https://review.openstack.org/126552 | 16:48 |
| marun | ZZelle: or is this new? | 16:48 |
| ZZelle | marun, it's new | 16:49 |
| *** killer_prince has joined #openstack-neutron | 16:49 | |
| *** killer_prince is now known as lazy_prince | 16:49 | |
| *** nati_ueno has joined #openstack-neutron | 16:50 | |
| *** imcsk8 has quit IRC | 16:51 | |
| *** imcsk8 has joined #openstack-neutron | 16:51 | |
| marun | ZZelle: conflating a refactor with new code always complicates reviewing | 16:52 |
| *** Sukhdev has quit IRC | 16:52 | |
| marun | ZZelle: Also, are only helpers involved? | 16:52 |
| *** sripriya has quit IRC | 16:53 | |
| *** padkrish has joined #openstack-neutron | 16:53 | |
| marun | ZZelle: i.e. is the lb support actually used by this patch? | 16:53 |
| *** sripriya has joined #openstack-neutron | 16:53 | |
| marun | atch -> change | 16:53 |
| marun | ZZelle: I'll review in any case | 16:54 |
| *** vivek-ebay has joined #openstack-neutron | 16:54 | |
| marun | ZZelle: also, something to think about: we may need to start testing our testing infrastructure | 16:54 |
| marun | ZZelle: I can imagine *aas starting to rely on it such that we're going to have to maintain some kind of stability guarantee | 16:55 |
| *** absubram has joined #openstack-neutron | 16:55 | |
| *** divya has joined #openstack-neutron | 16:55 | |
| marun | ZZelle: This is more long-term, of course. The burden of reuse. | 16:55 |
| *** vmtrooper has joined #openstack-neutron | 16:55 | |
| mestery | ijw: portsecurity has landed sir | 16:56 |
| *** melwitt has joined #openstack-neutron | 16:56 | |
| ijw | Congratulations | 16:56 |
| *** ihrachyshka has quit IRC | 16:57 | |
| jschwarz | ijw, heard also that the MTU patches landed! :) :) | 16:57 |
| ijw | orly? | 16:57 |
| ijw | ooooh | 16:57 |
| ijw | Just VLANs to go, I think | 16:57 |
| jschwarz | that's what HenryG said | 16:57 |
| ijw | And that's comparatively straightforward | 16:57 |
| ijw | You're right, all the ones that are up have gone in | 16:58 |
| jschwarz | ) | 16:58 |
| jschwarz | :) | 16:58 |
| *** openstackgerrit has quit IRC | 16:59 | |
| divya | banix__: ping | 16:59 |
| *** openstackgerrit has joined #openstack-neutron | 16:59 | |
| banix__ | divya: hi | 16:59 |
| divya | banix__: was looking at this patch https://review.openstack.org/#/c/165487/1/doc/source/devref/contribute.rst | 16:59 |
| divya | banix__: and noticed that nuage section is empty, we have completed decomposition | 17:00 |
| divya | banix__: how do we update the document with the external repo | 17:00 |
| *** vmtrooper has quit IRC | 17:00 | |
| banix__ | divya: is your decomposition patchset already merged? | 17:01 |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 17:01 |
| *** gsagie has joined #openstack-neutron | 17:01 | |
| divya | banix__: yes it is already merged ... | 17:01 |
| *** alexpilotti has quit IRC | 17:02 | |
| divya | banix__: repo at https://github.com/nuage-networks/nuage-openstack-neutron | 17:02 |
| *** alexpilotti_ has joined #openstack-neutron | 17:02 | |
| divya | banix__: we don't have pypi package yet | 17:02 |
| banix__ | divya: i think the easiest would be simply submitting another patch similar to the one you referred to and update the devref dile | 17:02 |
| banix__ | file | 17:02 |
| *** safchain has quit IRC | 17:02 | |
| *** rook-tmp has quit IRC | 17:02 | |
| *** rotbeard has quit IRC | 17:02 | |
| banix__ | divya: does that make sense? | 17:04 |
| divya | banix__: sorry did not quite get it ... do you mean another devref file? | 17:04 |
| *** achanda has joined #openstack-neutron | 17:04 | |
| *** ajmiller has joined #openstack-neutron | 17:05 | |
| *** _cjones_ has quit IRC | 17:05 | |
| *** jlanoux has quit IRC | 17:05 | |
| banix__ | divya: have a patch in which you update the file doc/source/devref/contribute.rst and add the required test for nuage similar to what others have in that file | 17:05 |
| divya | banix__: for your reference the nuage decomposition which got merged https://review.openstack.org/#/c/156823/ | 17:05 |
| divya | banix__: ok got it .. thanks .. | 17:06 |
| *** emagana has joined #openstack-neutron | 17:06 | |
| banix__ | divya: I meant “required text” nor “required test" | 17:06 |
| banix__ | divya: np | 17:07 |
| divya | banix__: sure .. thanks so much | 17:07 |
| banix__ | divya: np | 17:07 |
| *** jistr has quit IRC | 17:08 | |
| *** yfried is now known as yfried|afk | 17:08 | |
| *** scheuran has quit IRC | 17:08 | |
| *** _cjones_ has joined #openstack-neutron | 17:10 | |
| *** mlavalle has joined #openstack-neutron | 17:10 | |
| *** sbalukoff has quit IRC | 17:12 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 17:12 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 17:13 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 17:13 |
| *** bradjones has quit IRC | 17:14 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 17:16 |
| *** sridhar_ram has quit IRC | 17:18 | |
| *** rotbeard has joined #openstack-neutron | 17:18 | |
| *** salv-orl_ has quit IRC | 17:19 | |
| *** jschwarz has quit IRC | 17:21 | |
| *** s3wong has joined #openstack-neutron | 17:22 | |
| *** salv-orlando has joined #openstack-neutron | 17:25 | |
| *** harlowja_away is now known as harlowja_ | 17:27 | |
| *** trad511 has joined #openstack-neutron | 17:27 | |
| *** Marga_ has quit IRC | 17:28 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 17:28 |
| *** yfried|afk is now known as yfried | 17:29 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 17:29 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Test the shiny new WeightScheduler https://review.openstack.org/162231 | 17:31 |
| *** pritesh has joined #openstack-neutron | 17:32 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 17:33 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 17:33 |
| *** vthapar has joined #openstack-neutron | 17:35 | |
| *** kerpan has quit IRC | 17:35 | |
| *** vishwanathj has joined #openstack-neutron | 17:37 | |
| *** rdo has joined #openstack-neutron | 17:38 | |
| *** sbfox has joined #openstack-neutron | 17:40 | |
| *** ygbo has quit IRC | 17:40 | |
| mestery | HenryG: If we start merging these IPV6 patches in this series (https://review.openstack.org/#/c/113339/27), do we need them all? I'm concerned about getting them all landed for kilo | 17:40 |
| mestery | sc68cal: ^^^^ | 17:40 |
| mestery | armax: ^^^ | 17:40 |
| *** coolsvap is now known as coolsvap|afk | 17:41 | |
| *** vishwanathj has quit IRC | 17:42 | |
| *** arosen1 has quit IRC | 17:42 | |
| HenryG | mestery: The "slaac hardening" ones are really sort of bug fixes | 17:43 |
| mestery | HenryG: That was my thinking as well to be honest | 17:43 |
| *** lykinsbd has quit IRC | 17:43 | |
| *** lykinsbd has joined #openstack-neutron | 17:44 | |
| *** aveiga has quit IRC | 17:44 | |
| *** julim has quit IRC | 17:46 | |
| *** gsagie has quit IRC | 17:48 | |
| *** lykinsbd has quit IRC | 17:48 | |
| *** ajo has quit IRC | 17:48 | |
| *** ParsectiX has joined #openstack-neutron | 17:49 | |
| mestery | otherwiseguy: Just FYI, I think we're good to land the ovs_lib patch oince you address marun's concerns. Please file a bug for the testing efforts, which would be ideal to land in the RC timeframe. | 17:49 |
| mestery | otherwiseguy: Thanks for all your efforts here! | 17:49 |
| otherwiseguy | mestery: Addressing them right now. :) Thanks! | 17:49 |
| *** kawa2014 has quit IRC | 17:49 | |
| otherwiseguy | Most of the way through them. | 17:49 |
| mestery | otherwiseguy: Excellent! | 17:50 |
| *** sudipto_ has joined #openstack-neutron | 17:50 | |
| *** sridhar_ram has joined #openstack-neutron | 17:50 | |
| *** aepifanov has joined #openstack-neutron | 17:50 | |
| *** fredlhsu has joined #openstack-neutron | 17:51 | |
| *** aranjan has quit IRC | 17:51 | |
| *** bobmel has quit IRC | 17:51 | |
| *** aranjan has joined #openstack-neutron | 17:51 | |
| *** bobmel has joined #openstack-neutron | 17:51 | |
| *** ChuckC has quit IRC | 17:51 | |
| *** sripriya has quit IRC | 17:51 | |
| *** ChuckC_ has joined #openstack-neutron | 17:51 | |
| *** ChuckC_ is now known as ChuckC | 17:52 | |
| openstackgerrit | Sayaji Patil proposed openstack/python-neutronclient: Creating separate folder for nuage extensions https://review.openstack.org/165535 | 17:53 |
| *** arosen has joined #openstack-neutron | 17:53 | |
| *** bobmel has quit IRC | 17:56 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding Tempest API tests for VLAN Transparency support https://review.openstack.org/164887 | 17:56 |
| *** ronis has joined #openstack-neutron | 17:56 | |
| *** xgerman has quit IRC | 17:56 | |
| *** gsagie has joined #openstack-neutron | 17:56 | |
| *** thomasem has joined #openstack-neutron | 18:00 | |
| *** thomasem has quit IRC | 18:00 | |
| *** john-davidge has quit IRC | 18:00 | |
| *** crc32 has joined #openstack-neutron | 18:00 | |
| *** jorgem has joined #openstack-neutron | 18:01 | |
| *** pritesh_ has joined #openstack-neutron | 18:02 | |
| openstackgerrit | Divya ChanneGowda proposed openstack/neutron: Update contribute.rst with Nuage plugin decomposition https://review.openstack.org/165536 | 18:02 |
| *** nati_ueno has quit IRC | 18:02 | |
| *** sridhar_ram1 has joined #openstack-neutron | 18:03 | |
| *** thomasem has joined #openstack-neutron | 18:03 | |
| *** thomasem has quit IRC | 18:04 | |
| *** sridhar_ram has quit IRC | 18:04 | |
| openstackgerrit | Divya ChanneGowda proposed openstack/neutron: Update contribute.rst with Nuage plugin decomposition https://review.openstack.org/165536 | 18:04 |
| *** thomasem has joined #openstack-neutron | 18:04 | |
| *** nati_ueno has joined #openstack-neutron | 18:05 | |
| *** pritesh has quit IRC | 18:05 | |
| *** criptoverso has quit IRC | 18:06 | |
| sc68cal | mestery: https://review.openstack.org/#/c/113339/27 is a good patch that I think can be merged by itself | 18:07 |
| banix__ | armax: in your https://review.openstack.org/#/c/165267/ there is one more thing to correct. I have the correction in https://review.openstack.org/#/c/165487/ . I can update your patch and abandon mine if that’s ok with your. | 18:07 |
| sc68cal | mestery: since it fixes some behaviors | 18:07 |
| *** ParsectiX has quit IRC | 18:07 | |
| mestery | sc68cal: ack, was thinking the same | 18:08 |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 18:08 |
| *** ParsectiX has joined #openstack-neutron | 18:08 | |
| *** yamamoto has quit IRC | 18:08 | |
| sc68cal | mestery: I'll review the patches that have set it as a dependency https://review.openstack.org/#/c/161033/ and https://review.openstack.org/#/c/164943/ | 18:08 |
| mestery | sc68cal: Doing the same | 18:08 |
| *** jyuso1 has joined #openstack-neutron | 18:09 | |
| HenryG | mestery: These are the IPv6 multiple prefix ones needed, https://review.openstack.org/149068 https://review.openstack.org/161073 https://review.openstack.org/156360 | 18:10 |
| *** jpena is now known as jpena|away | 18:10 | |
| HenryG | carl_baldwin: Would you have time to check them out? ^^ | 18:10 |
| *** xgerman has joined #openstack-neutron | 18:10 | |
| carl_baldwin | HenryG: Yes, I will get to them soon. | 18:11 |
| *** Sukhdev has joined #openstack-neutron | 18:11 | |
| *** Marga_ has joined #openstack-neutron | 18:12 | |
| sc68cal | HenryG: I just posted a comment on https://review.openstack.org/#/c/149068/ | 18:13 |
| *** bobmel has joined #openstack-neutron | 18:13 | |
| sc68cal | I think for dual stack changes it's a mix of additions to Neutron, Tempest, and DevStack | 18:13 |
| sc68cal | *dual stack testing | 18:13 |
| dboik | mestery, HenryG: even https://review.openstack.org/161073 could be a bug fix I think | 18:13 |
| *** sridhar_ram1 has left #openstack-neutron | 18:14 | |
| sc68cal | HenryG: there are still bugs lurking around so I'm using https://review.openstack.org/#/c/160856/ to uncover | 18:14 |
| mestery | dboik: Except it's in the middle of a chain of dependent patches | 18:14 |
| HenryG | sc68cal: nice! thanks! | 18:14 |
| *** numan has quit IRC | 18:14 | |
| *** ijw has quit IRC | 18:15 | |
| dboik | mestery: that one is just to pave the way for its dependent patch. It's not really needed for anything else, so both patches could be moved out | 18:16 |
| *** igordcard has quit IRC | 18:16 | |
| *** nyechiel has quit IRC | 18:16 | |
| *** nati_ueno has quit IRC | 18:16 | |
| *** ijw has joined #openstack-neutron | 18:16 | |
| *** nati_ueno has joined #openstack-neutron | 18:16 | |
| *** lykinsbd has joined #openstack-neutron | 18:17 | |
| openstackgerrit | Mohammad Banikazemi proposed openstack/neutron: Fix minor decomp progress chart issues https://review.openstack.org/165267 | 18:17 |
| dboik | mestery, HenryG: The main ones that add new functionality are https://review.openstack.org/149068 and https://review.openstack.org/156360 | 18:17 |
| dboik | the patch by dane_leblanc is especially important because it enables dual-stack. Mine is less important. | 18:18 |
| *** SumitNaiksatam has joined #openstack-neutron | 18:18 | |
| mestery | dboik: ack | 18:18 |
| *** padkrish has quit IRC | 18:18 | |
| *** jorgem has quit IRC | 18:19 | |
| *** gildub has joined #openstack-neutron | 18:19 | |
| *** gsagie has quit IRC | 18:19 | |
| *** padkrish has joined #openstack-neutron | 18:19 | |
| HenryG | dboik: thanks, almost done reviewing https://review.openstack.org/156360 and hopefully carl_baldwin will also be on it | 18:19 |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 18:19 |
| *** bashok has quit IRC | 18:20 | |
| dboik | HenryG: awesome, thx | 18:20 |
| carl_baldwin | HenryG: Now reviewing https://review.openstack.org/#/c/149068/ | 18:20 |
| *** padkrish_ has joined #openstack-neutron | 18:20 | |
| HenryG | carl_baldwin: thanks (I found that one quite pleasant to review, I hope you agree) | 18:21 |
| *** SumitNaiksatam has left #openstack-neutron | 18:21 | |
| *** SumitNaiksatam has joined #openstack-neutron | 18:22 | |
| sc68cal | that patch is where gertty shines compared to the web interface | 18:22 |
| sc68cal | although gitweb also has it as a single patch that you can scroll. | 18:22 |
| sc68cal | I hate gerrit's "single tab for each file" on patches like these | 18:23 |
| *** padkrish has quit IRC | 18:23 | |
| *** sbalukoff has joined #openstack-neutron | 18:23 | |
| *** vishwanathj has joined #openstack-neutron | 18:26 | |
| openstackgerrit | Michael Smith proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/139686 | 18:27 |
| *** afazekas has quit IRC | 18:31 | |
| *** bobmel has quit IRC | 18:31 | |
| *** bobmel has joined #openstack-neutron | 18:31 | |
| *** _cjones_ has quit IRC | 18:32 | |
| *** _cjones_ has joined #openstack-neutron | 18:32 | |
| *** dannywilson has joined #openstack-neutron | 18:35 | |
| *** jorgem has joined #openstack-neutron | 18:35 | |
| *** tflynn_ has quit IRC | 18:36 | |
| *** bobmel has quit IRC | 18:36 | |
| *** tflynn has joined #openstack-neutron | 18:37 | |
| *** rushil has quit IRC | 18:37 | |
| *** jyuso1 has quit IRC | 18:37 | |
| *** aranjan has quit IRC | 18:38 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 18:39 |
| openstackgerrit | Roey Chen proposed openstack/neutron: Allow plugin to specify security-group rules ids upon creation https://review.openstack.org/165553 | 18:40 |
| *** tiswanso has joined #openstack-neutron | 18:41 | |
| *** itzikb has joined #openstack-neutron | 18:42 | |
| *** vmtrooper has joined #openstack-neutron | 18:44 | |
| *** tiswanso has quit IRC | 18:45 | |
| *** tiswanso has joined #openstack-neutron | 18:45 | |
| *** yfried has quit IRC | 18:47 | |
| *** moshele has quit IRC | 18:47 | |
| *** moshele has joined #openstack-neutron | 18:47 | |
| sc68cal | pritesh_: For the nfv-vlan-trunks - are there mech drivers that are utilizing the API changes? or is that work forthcoming? | 18:48 |
| *** kickinz1 is now known as kickinz1|afk | 18:48 | |
| *** achanda has quit IRC | 18:48 | |
| pritesh_ | sc68cal: still forthcoming. | 18:48 |
| *** pritesh_ is now known as pritesh | 18:48 | |
| *** moshele has quit IRC | 18:49 | |
| carl_baldwin | HenryG: dane_leblanc: I’m looking at https://review.openstack.org/#/c/149068/27/neutron/agent/l3/ha_router.py | 18:49 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 18:49 |
| *** vmtrooper has quit IRC | 18:49 | |
| carl_baldwin | Have any of the HA developers looked at this? Has it been tested with HA routers? | 18:49 |
| mestery | pritesh: You're on fire with hte patchsets | 18:49 |
| pritesh | mestery: :) | 18:50 |
| dane_leblanc | carl_baldwin: I have not tried with HA routers | 18:50 |
| *** julim has joined #openstack-neutron | 18:50 | |
| HenryG | mestery: that update from pritesh is my fault | 18:50 |
| *** yatin has quit IRC | 18:51 | |
| dane_leblanc | carl_baldwin: So relying on gating tests for HA :( | 18:51 |
| * mestery makes a note to hand HenryG a gerrit screen without the "rebase" button :P | 18:51 | |
| *** Marga_ has quit IRC | 18:51 | |
| carl_baldwin | dane_leblanc: I’m just wondering about the loop around creating the keepalived instance. | 18:51 |
| carl_baldwin | I’m trying to wrap my head around what this means. | 18:52 |
| carl_baldwin | amuller: ping | 18:52 |
| carl_baldwin | amuller: ^ | 18:52 |
| *** Marga_ has joined #openstack-neutron | 18:53 | |
| dane_leblanc | carl_baldwin: Can we have multiple keepalived instances, or can one keepalived instance have multiple CIDRs | 18:53 |
| amuller | carl_baldwin: dane_leblanc: reading backlog, give me a min | 18:53 |
| *** sudipto_ has quit IRC | 18:54 | |
| carl_baldwin | amuller: Thanks. | 18:54 |
| HenryG | mestery: I just felt the user-facing help texts should be clear | 18:55 |
| mestery | HenryG: Ack to that! Thanks for the cleanup actually, I was just joking ;) | 18:55 |
| *** sbfox has quit IRC | 18:55 | |
| * amuller looking at the patch... | 18:57 | |
| *** jprovazn has quit IRC | 18:57 | |
| *** sbfox has joined #openstack-neutron | 18:58 | |
| amuller | dane_leblanc: Forget HA routers for a sec, I don't understand why you don't test a dual gateway router in the L3 agent functional tests? | 19:00 |
| amuller | a dual gateway legacy router | 19:00 |
| *** achanda has joined #openstack-neutron | 19:01 | |
| amuller | that seems like step 0 for such a change | 19:01 |
| *** padkrish_ has quit IRC | 19:02 | |
| *** sridhar_ram has joined #openstack-neutron | 19:02 | |
| dane_leblanc | amuller: Would Neutron API tests be a better suited for testing dual stack gateway? | 19:02 |
| *** padkrish has joined #openstack-neutron | 19:02 | |
| amuller | dane_leblanc: You can test it a lower level (Functional test) | 19:02 |
| amuller | seems like a really easy addition too | 19:02 |
| *** lykinsbd has quit IRC | 19:02 | |
| armax | banix__: ping | 19:03 |
| amuller | configure a legacy router with ipv4 and ipv6 gateway, and in the lifecycle test for legacy routers make sure that both gateways are configured | 19:03 |
| banix__ | armax: hi | 19:03 |
| armax | banix__: ok, checking | 19:03 |
| *** padkrish_ has joined #openstack-neutron | 19:03 | |
| armax | banix__: thanks for the heads-up | 19:03 |
| banix__ | armax: np. thx | 19:03 |
| *** alexpilotti_ has quit IRC | 19:04 | |
| dane_leblanc | amuller: Yes, will look into adding that | 19:04 |
| amuller | dane_leblanc: Now, about the modification to HA routers, I'm assuming that was done with no manual testing even... | 19:04 |
| *** Marga_ has quit IRC | 19:04 | |
| dane_leblanc | amuller: Yes | 19:04 |
| *** Marga_ has joined #openstack-neutron | 19:05 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Fix minor decomp progress chart issues https://review.openstack.org/165267 | 19:05 |
| *** nati_ueno has quit IRC | 19:05 | |
| *** liusheng has quit IRC | 19:05 | |
| amuller | dane_leblanc: Do you have a VM or something you run the functional tests on? | 19:05 |
| *** nati_ueno has joined #openstack-neutron | 19:05 | |
| dane_leblanc | amuller: Yes, VM or sometimes directly on Linux host | 19:06 |
| *** liusheng has joined #openstack-neutron | 19:06 | |
| *** emagana has quit IRC | 19:06 | |
| *** padkrish has quit IRC | 19:06 | |
| *** marun has quit IRC | 19:07 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 19:07 |
| amuller | dane_leblanc: Check out https://review.openstack.org/#/c/149068/27/neutron/tests/functional/agent/test_l3_agent.py, line 440 | 19:07 |
| *** ivar-lazzaro has joined #openstack-neutron | 19:07 | |
| *** vivek-eb_ has joined #openstack-neutron | 19:08 | |
| *** yamamoto has joined #openstack-neutron | 19:09 | |
| *** emagana has joined #openstack-neutron | 19:09 | |
| *** diegows has joined #openstack-neutron | 19:09 | |
| *** cburgess has quit IRC | 19:10 | |
| dane_leblanc | amuller: So I assume the Ubuntu still doesn't support the keepalived | 19:10 |
| amuller | dane_leblanc: recent versions of keepalived support ipv6 gateway | 19:10 |
| amuller | but not the version the gate runs, last time I checked | 19:10 |
| *** vivek-ebay has quit IRC | 19:10 | |
| amuller | so you won't be able to add a dual-gateway test for HA routers, for now | 19:10 |
| amuller | but, can you tell me what keepalived version you have running? | 19:10 |
| amuller | I'd like to ask you to perform a local test | 19:10 |
| amuller | HenryG: ping about https://review.openstack.org/#/c/149068/27 | 19:10 |
| HenryG | amuller: pong | 19:11 |
| amuller | dane_leblanc: it supports keepalived, just an old version | 19:11 |
| *** cburgess has joined #openstack-neutron | 19:11 | |
| amuller | dane_leblanc: if you have a machine capable of running the L3 agent functional tests you have keepalived installed | 19:11 |
| amuller | can you echo keepalived --version? | 19:11 |
| dane_leblanc | 1.2.7 | 19:11 |
| amuller | HenryG: When is that patch supposed to go in? | 19:11 |
| HenryG | amuller: yesterday :) | 19:11 |
| amuller | =/ | 19:11 |
| amuller | I was hoping not to hear that :) | 19:11 |
| *** nati_ueno has quit IRC | 19:12 | |
| *** nati_ueno has joined #openstack-neutron | 19:12 | |
| amuller | HenryG: I'll be honest I'm baffled as to why this patch is not tested via the functional tests | 19:12 |
| *** dsneddon is now known as dsneddon_lunch | 19:12 | |
| *** vishwana_ has joined #openstack-neutron | 19:13 | |
| *** rushil has joined #openstack-neutron | 19:13 | |
| *** yamamoto has quit IRC | 19:13 | |
| amuller | dane_leblanc: Do you think you could realistically modify the lifecycle test for legacy routers today? | 19:14 |
| amuller | dane_leblanc: Also, is there a newer version available for keepalived on your system? | 19:14 |
| *** vishwanathj has quit IRC | 19:15 | |
| *** yfried has joined #openstack-neutron | 19:15 | |
| *** moha_hunt has joined #openstack-neutron | 19:15 | |
| *** _cjones_ has quit IRC | 19:16 | |
| dane_leblanc | amuller: I can try an upgrade, what is the min version that I need? | 19:16 |
| amuller | 1.2.10 | 19:16 |
| *** _cjones_ has joined #openstack-neutron | 19:16 | |
| pc_m | armax: I responded to your comments on . If you get a chance, please look if I addressed your questions/concerns. https://review.openstack.org/164466 | 19:18 |
| *** rotbeard has quit IRC | 19:21 | |
| *** vivek-eb_ has quit IRC | 19:21 | |
| *** vivek-ebay has joined #openstack-neutron | 19:21 | |
| *** salv-orlando has quit IRC | 19:24 | |
| *** adelinatuvenie has quit IRC | 19:25 | |
| *** plaurin_ has left #openstack-neutron | 19:26 | |
| *** dsneddon_lunch is now known as dsneddon | 19:29 | |
| *** irenab has joined #openstack-neutron | 19:29 | |
| *** salv-orlando has joined #openstack-neutron | 19:30 | |
| *** lykinsbd has joined #openstack-neutron | 19:31 | |
| *** nati_ueno has quit IRC | 19:32 | |
| *** nati_ueno has joined #openstack-neutron | 19:33 | |
| *** aswadr has quit IRC | 19:34 | |
| *** padkrish has joined #openstack-neutron | 19:37 | |
| *** boris-42 has quit IRC | 19:37 | |
| *** sballe has quit IRC | 19:37 | |
| *** edwarnicke has quit IRC | 19:37 | |
| *** padkrish_ has quit IRC | 19:39 | |
| *** rstarr has quit IRC | 19:39 | |
| *** jorgem has quit IRC | 19:39 | |
| *** krtaylor has quit IRC | 19:39 | |
| *** carlp has quit IRC | 19:39 | |
| *** comstud has quit IRC | 19:39 | |
| *** zacksh has quit IRC | 19:39 | |
| *** lbragstad has quit IRC | 19:39 | |
| *** zacksh has joined #openstack-neutron | 19:40 | |
| *** rstarr has joined #openstack-neutron | 19:40 | |
| *** vishwana_ has quit IRC | 19:40 | |
| *** lbragstad has joined #openstack-neutron | 19:40 | |
| *** sballe has joined #openstack-neutron | 19:41 | |
| *** carlp has joined #openstack-neutron | 19:41 | |
| *** comstud has joined #openstack-neutron | 19:41 | |
| *** krtaylor has joined #openstack-neutron | 19:41 | |
| *** edwarnicke has joined #openstack-neutron | 19:41 | |
| *** boris-42 has joined #openstack-neutron | 19:41 | |
| *** Marga_ has quit IRC | 19:43 | |
| *** nlahouti has quit IRC | 19:43 | |
| *** Marga_ has joined #openstack-neutron | 19:44 | |
| *** diegows has quit IRC | 19:44 | |
| *** aranjan has joined #openstack-neutron | 19:45 | |
| *** rushil has quit IRC | 19:45 | |
| *** diegows has joined #openstack-neutron | 19:45 | |
| *** nati_ueno has quit IRC | 19:47 | |
| *** rushil has joined #openstack-neutron | 19:48 | |
| carl_baldwin | HenryG: dane_leblanc: Just posted a review to https://review.openstack.org/#/c/149068/ | 19:48 |
| *** aranjan has quit IRC | 19:49 | |
| dane_leblanc | carl_baldwin: Thanks! | 19:49 |
| carl_baldwin | I’ll be around for about 2 more hours if you want to discuss anything. | 19:49 |
| *** diegows_ has joined #openstack-neutron | 19:50 | |
| *** TrevorV_ has quit IRC | 19:50 | |
| *** crc32 has quit IRC | 19:53 | |
| *** sripriya has joined #openstack-neutron | 19:53 | |
| *** diegows has quit IRC | 19:53 | |
| *** ijw has quit IRC | 19:53 | |
| *** Marga_ has quit IRC | 19:54 | |
| *** matrohon has joined #openstack-neutron | 19:54 | |
| *** devlaps has joined #openstack-neutron | 19:55 | |
| *** zigo__ has joined #openstack-neutron | 19:56 | |
| *** zigo has quit IRC | 19:56 | |
| claudiub | armax, mestery: hello | 19:57 |
| mestery | claudiub: Hello! | 19:57 |
| *** roaet_ has quit IRC | 19:57 | |
| claudiub | mestery: tomorrow is the general FF, right? I was wondering, is that going to affect the hyper-v decomposition? | 19:58 |
| *** sridhar_ram has quit IRC | 19:58 | |
| *** roaet_ has joined #openstack-neutron | 19:58 | |
| mestery | claudiub: The decomposition is a wishlist bug, so once the CI works, we can merge that post-FF. Does that make sense? | 19:58 |
| *** puck has quit IRC | 19:58 | |
| claudiub | mestery: or could it get an exception, since it's pretty much ready to merge, the Hyper-V CI is currently running on it | 19:58 |
| mestery | claudiub: decomp is fine. What about the OVS agent support for Windows? How is the CI run coming on that? | 19:59 |
| claudiub | mestery: I see, cool. :D thanks for the info! :) | 19:59 |
| *** puck has joined #openstack-neutron | 19:59 | |
| *** vivek-ebay has quit IRC | 19:59 | |
| *** flwang has quit IRC | 19:59 | |
| mestery | claudiub: armax and I think that one is ready too once the CI gets running on it. | 19:59 |
| *** vivek-ebay has joined #openstack-neutron | 19:59 | |
| *** achanda has quit IRC | 20:00 | |
| *** thomasem has quit IRC | 20:00 | |
| *** flwang has joined #openstack-neutron | 20:01 | |
| *** emagana has quit IRC | 20:01 | |
| *** britthouser has quit IRC | 20:01 | |
| *** rook-tmp has joined #openstack-neutron | 20:01 | |
| _cjones_ | Hey guys. Think I found a bug w/ LBaaS. Anyone here that I can quickly discuss with? | 20:01 |
| *** jorgem has joined #openstack-neutron | 20:02 | |
| claudiub | mestery: I do not know about the OVS agent support on Windows, my coleague Adelina is working on it, so I don't know all the details. Plus, I see you have already commented on that review. | 20:02 |
| _cjones_ | ...or know who would be the best point of contact for LBaaS? | 20:02 |
| mestery | claudiub: Yup, hope it goes in. | 20:02 |
| mestery | _cjones_: Have you met dougwig or blogan? They are the lbaas folks :) | 20:02 |
| blogan | _cjones_: hi | 20:03 |
| mestery | thanks blogan :) | 20:03 |
| claudiub | mestery: as for the CI, we're making a few changes on it, to take into account the decomposition. :) | 20:03 |
| mestery | claudiub: Sure! I know you're working hard on it, thanks for that! We'll help as we can. :) | 20:04 |
| claudiub | mestery: thanks! :D | 20:04 |
| *** fredlhsu has quit IRC | 20:05 | |
| *** angela-s has joined #openstack-neutron | 20:06 | |
| * mestery is having a spectacular day today. | 20:07 | |
| mestery | I think it's because it's almost spring. | 20:07 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 20:07 |
| claudiub | i'd say it's because it's not monday. :P | 20:07 |
| sc68cal | happy PTL happy openstack project | 20:08 |
| mestery | lol | 20:08 |
| _cjones_ | Hi blogan | 20:08 |
| dougwig | _cjones_: hiya, and i'd invite you over to #openstack-lbaas as well. | 20:08 |
| _cjones_ | Thanks metery. | 20:08 |
| _cjones_ | Thanks mestery. | 20:08 |
| _cjones_ | Hi dougwig. | 20:08 |
| carl_baldwin | dane_leblanc: ping | 20:08 |
| _cjones_ | Just doing a bit of experimentation here with LBaaS. | 20:08 |
| *** thomasem has joined #openstack-neutron | 20:09 | |
| _cjones_ | Noticed that I could create a LB Pool with a project ID that doesn't exist in keystone. | 20:09 |
| _cjones_ | Wondering if this is a bug or by design. | 20:09 |
| _cjones_ | s/project_id/tenant_id (sorry...bad habbit) | 20:09 |
| dane_leblanc | carl_baldwin: pong | 20:09 |
| carl_baldwin | dane_leblanc: Should the author have changed on that patch? | 20:10 |
| openstackgerrit | Merged openstack/neutron: Api tests to cover network mtu attribute https://review.openstack.org/165201 | 20:10 |
| carl_baldwin | dane_leblanc: I just refreshed, looks like it should not have. | 20:10 |
| *** Sukhdev has quit IRC | 20:11 | |
| dane_leblanc | carl_baldwin: I see the author as myself | 20:11 |
| _cjones_ | Hi dougwig and blogan! Just a repost from #openstack-neutron | 20:11 |
| carl_baldwin | dane_leblanc: In PS27 the author is Abishek. | 20:12 |
| carl_baldwin | dane_leblanc: https://review.openstack.org/#/c/149068/27//COMMIT_MSG | 20:12 |
| dane_leblanc | carl_baldwin: Ah, that should not have changed | 20:12 |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 20:12 |
| *** tiswanso has quit IRC | 20:13 | |
| *** openstackgerrit has quit IRC | 20:14 | |
| *** openstackgerrit has joined #openstack-neutron | 20:15 | |
| dane_leblanc | carl_baldwin: Will switch that back on next upload | 20:15 |
| carl_baldwin | dane_leblanc: Most of the other stuff in that patch can be accepted as long as you post a follow-up to address feedback. | 20:16 |
| carl_baldwin | I think I’ll fix this one myself in a new patch: https://review.openstack.org/#/c/149068/27/neutron/agent/linux/ra.py | 20:16 |
| carl_baldwin | dane_leblanc: ^ | 20:17 |
| *** changbl has joined #openstack-neutron | 20:17 | |
| carl_baldwin | (a small reward for having to pick through the logic and come up with a better way) | 20:17 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 20:18 |
| flwang | kevinbenton: ping | 20:19 |
| *** diegows_ has quit IRC | 20:19 | |
| kevinbenton | flwang: pong | 20:20 |
| flwang | kevinbenton: could you pls revisit https://review.openstack.org/#/c/156198/ ? | 20:20 |
| dane_leblanc | carl_baldwin: So leave the crazy for/else/break/else for now, and you'll fix this in a follow-up? | 20:20 |
| flwang | I'm trying to address your comments | 20:20 |
| carl_baldwin | dane_leblanc: Either way. | 20:21 |
| *** jorgem has quit IRC | 20:21 | |
| *** Marga_ has joined #openstack-neutron | 20:21 | |
| carl_baldwin | dane_leblanc: If it is easier to leave it then I’ve already got a follow-up ready. | 20:22 |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Replace complicated use of for/break/else with a simpler equivalent https://review.openstack.org/165586 | 20:23 |
| amuller | armax: Pushed a new revision of https://review.openstack.org/#/c/125384/ | 20:23 |
| *** prad has quit IRC | 20:23 | |
| dane_leblanc | carl_baldwin: Cool. Thanks! | 20:24 |
| carl_baldwin | dane_leblanc: I’m willing to accept the patch (with the author reset back to you) if you post a follow-up for the rest of the feedback. | 20:24 |
| carl_baldwin | dane_leblanc: Too bad about that accidental author change. | 20:24 |
| dane_leblanc | carl_baldwin: No worries. I'll post a followup. Thanks | 20:25 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 20:26 |
| *** fredlhsu has joined #openstack-neutron | 20:26 | |
| *** ihrachyshka has joined #openstack-neutron | 20:26 | |
| *** moshele has joined #openstack-neutron | 20:30 | |
| matrohon | armax, mestery : the extension framework that you validated for the python-neutronclient has a little bug. | 20:31 |
| *** rotbeard has joined #openstack-neutron | 20:31 | |
| matrohon | armax, mestery : I proposed a bug fix here : https://review.openstack.org/#/c/163478/ | 20:31 |
| mestery | matrohon: Looking | 20:31 |
| flwang | kevinbenton: still around? | 20:32 |
| *** fredlhsu has quit IRC | 20:32 | |
| mestery | matrohon: Nice work, thanks! | 20:32 |
| matrohon | mestery : you're welcome, this framework is very usefu! | 20:33 |
| *** rotbeard has quit IRC | 20:33 | |
| *** vmtrooper has joined #openstack-neutron | 20:33 | |
| mestery | matrohon: Sweet! :) | 20:33 |
| mestery | matrohon: Thank roaet for the awesome work there. :) | 20:33 |
| matrohon | mestery : I've used it here https://review.openstack.org/#/c/165096/ | 20:34 |
| kevinbenton | flwang: hey, sorry I'm on my phone right now | 20:34 |
| *** Bsony has quit IRC | 20:34 | |
| matrohon | roaet : ^^ | 20:34 |
| kevinbenton | flwang: I prefer not having the else clause because it's not necessary | 20:34 |
| flwang | kevinbenton: ok, pls ping me when you're available | 20:34 |
| mestery | matrohon: Wow, nice! | 20:34 |
| flwang | kevinbenton: but both ways should work, and I think there is no difference. so i'm wondering if it deserves a -1 :) | 20:35 |
| kevinbenton | flwang: well code readability for reviewers is important. Using 'continue' instead of else would reduce the patch size because nothing below would be changed | 20:37 |
| kevinbenton | flwang: by adding else it forced an indent | 20:37 |
| *** vmtrooper has quit IRC | 20:38 | |
| *** Bsony has joined #openstack-neutron | 20:38 | |
| kevinbenton | flwang: using else there also tends to imply that there will be more code afterwards that will happen whether or not there was an exception | 20:39 |
| *** lizk has quit IRC | 20:39 | |
| kevinbenton | flwang: but that's not the case here, it's all in the else clause | 20:40 |
| flwang | kevinbenton: ok, make more sense for me | 20:40 |
| flwang | kevinbenton: I will submit a new ps soon | 20:40 |
| flwang | thanks for the comments | 20:40 |
| amuller | kevinbenton: https://review.openstack.org/#/c/125384/ is up for review if you're interested | 20:41 |
| *** lizk has joined #openstack-neutron | 20:41 | |
| *** rushil has quit IRC | 20:42 | |
| kevinbenton | flwang: sounds good | 20:42 |
| kevinbenton | amuller: will check it out tonight. On my phone right now in a presentation | 20:43 |
| amuller | kevinbenton: well then stop checking your phone and listen to whoever is in front of you :) | 20:43 |
| amuller | ... and thank you | 20:43 |
| kevinbenton | amuller: ok :) | 20:44 |
| *** rotbeard has joined #openstack-neutron | 20:45 | |
| *** carlp has quit IRC | 20:46 | |
| *** achanda has joined #openstack-neutron | 20:48 | |
| roaet | matrohon: *blush* glad you found it useful. We have an auth extension that I don't think will make it in but it might help you too. +mestery | 20:48 |
| openstackgerrit | Roey Chen proposed openstack/neutron: Allow plugin to specify security-group rules ids upon creation https://review.openstack.org/165553 | 20:48 |
| *** itzikb has quit IRC | 20:49 | |
| mestery | kevinbenton: If you can't multi-task, I suggest not paying attention to your presentation. :) | 20:49 |
| *** nlahouti has joined #openstack-neutron | 20:49 | |
| kevinbenton | mestery: that could cause a deadlock :) | 20:50 |
| mestery | kevinbenton: lol | 20:50 |
| *** ronis has quit IRC | 20:51 | |
| *** ParsectiX has quit IRC | 20:52 | |
| *** ParsectiX has joined #openstack-neutron | 20:53 | |
| sc68cal | dboik: I just reviewed https://review.openstack.org/#/c/161085/ | 20:56 |
| sc68cal | the code looked good but I started going cross-eye on the variables for the unit test | 20:56 |
| *** alexpilotti has joined #openstack-neutron | 20:56 | |
| *** padkrish has quit IRC | 20:57 | |
| *** padkrish has joined #openstack-neutron | 20:57 | |
| *** jorgem has joined #openstack-neutron | 20:58 | |
| *** jpena|away is now known as jpena | 21:00 | |
| *** thomasem has quit IRC | 21:01 | |
| *** padkrish has quit IRC | 21:02 | |
| *** amuller is now known as amuller_walk | 21:03 | |
| *** sbfox has quit IRC | 21:05 | |
| *** aranjan has joined #openstack-neutron | 21:05 | |
| *** Bsony has quit IRC | 21:06 | |
| dboik | sc68cal: haha. Thanks. I'll try to make the names less confusing. | 21:08 |
| *** carlp has joined #openstack-neutron | 21:08 | |
| *** jorgem has quit IRC | 21:08 | |
| *** jorgem has joined #openstack-neutron | 21:08 | |
| sc68cal | dboik: sorry - I know it's a cluster!#$& pulling stuff out of dicts | 21:08 |
| sc68cal | and giving them reasonable names | 21:09 |
| *** nati_ueno has joined #openstack-neutron | 21:10 | |
| *** yamamoto has joined #openstack-neutron | 21:11 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 21:11 |
| openstackgerrit | Darragh O'Reilly proposed openstack/neutron: Fix port status not being updated properly https://review.openstack.org/165351 | 21:11 |
| *** nati_ueno has quit IRC | 21:12 | |
| *** nati_ueno has joined #openstack-neutron | 21:12 | |
| *** banix__ has quit IRC | 21:13 | |
| openstackgerrit | Darragh O'Reilly proposed openstack/neutron: Fix port status not being updated properly https://review.openstack.org/165351 | 21:15 |
| *** yamamoto has quit IRC | 21:15 | |
| *** ihrachyshka has quit IRC | 21:16 | |
| *** padkrish has joined #openstack-neutron | 21:17 | |
| *** Swami has quit IRC | 21:17 | |
| *** ChuckC has quit IRC | 21:19 | |
| *** jorgem has quit IRC | 21:19 | |
| *** ParsectiX has quit IRC | 21:21 | |
| *** ParsectiX has joined #openstack-neutron | 21:21 | |
| *** vthapar has quit IRC | 21:25 | |
| *** __TheDodd__ has joined #openstack-neutron | 21:27 | |
| *** Swami has joined #openstack-neutron | 21:27 | |
| *** jorgem has joined #openstack-neutron | 21:27 | |
| *** thedodd has quit IRC | 21:28 | |
| *** jamielennox is now known as jamielennox|away | 21:31 | |
| openstackgerrit | Merged openstack/python-neutronclient: Add commands from extensions to available commands https://review.openstack.org/163478 | 21:32 |
| *** pc_m has quit IRC | 21:33 | |
| *** ihrachyshka has joined #openstack-neutron | 21:34 | |
| *** aepifanov has quit IRC | 21:34 | |
| *** changbl has quit IRC | 21:37 | |
| *** ParsectiX has quit IRC | 21:38 | |
| *** ParsectiX has joined #openstack-neutron | 21:38 | |
| *** circ-user-tNCMC has joined #openstack-neutron | 21:38 | |
| *** leenheer has quit IRC | 21:38 | |
| *** Marga_ has quit IRC | 21:39 | |
| *** circ-user-tNCMC is now known as dedery | 21:39 | |
| *** dedery has quit IRC | 21:39 | |
| *** dedery has joined #openstack-neutron | 21:40 | |
| *** dedery has quit IRC | 21:40 | |
| *** Marga_ has joined #openstack-neutron | 21:40 | |
| *** Marga_ has quit IRC | 21:40 | |
| *** Marga_ has joined #openstack-neutron | 21:40 | |
| *** yfried is now known as yfried|afk | 21:41 | |
| *** devlaps has quit IRC | 21:43 | |
| *** jobewan has joined #openstack-neutron | 21:44 | |
| *** vishwanathj has joined #openstack-neutron | 21:45 | |
| *** jckasper has quit IRC | 21:45 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 21:47 |
| *** ParsectiX has quit IRC | 21:49 | |
| *** mwagner_lap has quit IRC | 21:51 | |
| *** Jianyong has quit IRC | 21:51 | |
| claudiub | mestery: hello | 21:51 |
| mestery | yo! | 21:51 |
| mestery | claudiub: ^^^ | 21:51 |
| claudiub | mestery: good news, hyper-v ci is happy. :D https://review.openstack.org/#/c/153259/ | 21:52 |
| *** yamahata has quit IRC | 21:52 | |
| mestery | claudiub: Yay! +A! :) | 21:52 |
| *** yamahata has joined #openstack-neutron | 21:52 | |
| claudiub | mestery: thanks! :D | 21:53 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 21:53 |
| alexpilotti | mestery claudiub: yeiii :-) | 21:53 |
| mestery | alexpilotti claudiub: :D | 21:53 |
| *** matrohon has quit IRC | 21:54 | |
| *** csoukup has quit IRC | 21:59 | |
| *** padkrish has quit IRC | 21:59 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 21:59 |
| *** padkrish has joined #openstack-neutron | 21:59 | |
| *** yfried|afk is now known as yfried | 22:01 | |
| *** boris-42 has quit IRC | 22:02 | |
| *** padkrish_ has joined #openstack-neutron | 22:03 | |
| *** dave-mccowan has quit IRC | 22:04 | |
| *** padkrish has quit IRC | 22:04 | |
| *** jpena has quit IRC | 22:05 | |
| openstackgerrit | Fei Long Wang proposed openstack/neutron: Fix metering agent failure when chain missing https://review.openstack.org/156198 | 22:06 |
| *** shwetaap has quit IRC | 22:08 | |
| *** moha_hunt has quit IRC | 22:08 | |
| *** ZZelle_ has joined #openstack-neutron | 22:09 | |
| ZZelle_ | kevinbenton, hi | 22:10 |
| *** marun has joined #openstack-neutron | 22:10 | |
| *** carl_baldwin has quit IRC | 22:11 | |
| *** yfried is now known as yfried|afk | 22:11 | |
| *** carlp has quit IRC | 22:11 | |
| *** jamielennox|away is now known as jamielennox | 22:12 | |
| *** Bsony has joined #openstack-neutron | 22:13 | |
| *** crose has quit IRC | 22:14 | |
| *** jorgem has quit IRC | 22:14 | |
| *** garyduan has quit IRC | 22:14 | |
| *** Bsony has quit IRC | 22:18 | |
| *** dboik has quit IRC | 22:18 | |
| *** packet has quit IRC | 22:19 | |
| *** localloop127 has quit IRC | 22:19 | |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 22:20 |
| *** Jianyong has joined #openstack-neutron | 22:20 | |
| *** vhosakot has quit IRC | 22:21 | |
| *** lykinsbd has quit IRC | 22:21 | |
| *** lykinsbd has joined #openstack-neutron | 22:21 | |
| *** vmtrooper has joined #openstack-neutron | 22:22 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 22:24 |
| *** kbringard has quit IRC | 22:24 | |
| *** vishwanathj has quit IRC | 22:25 | |
| openstackgerrit | Rajeev Grover proposed openstack/neutron: HA for DVR - Neutron Server side code changes https://review.openstack.org/143169 | 22:26 |
| *** lykinsbd has quit IRC | 22:26 | |
| *** vmtrooper has quit IRC | 22:27 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 22:29 |
| marun | amuller_walk: Please fix import nit for https://review.openstack.org/#/c/125384/ | 22:29 |
| armax | otherwiseguy: ping | 22:32 |
| otherwiseguy | armax: pong | 22:32 |
| tidwellr | ihrachyshka: https://review.openstack.org/#/c/148698/ PS51 concerns addressed | 22:32 |
| armax | regarding change 152761 | 22:32 |
| *** diegows_ has joined #openstack-neutron | 22:32 | |
| *** carlp has joined #openstack-neutron | 22:33 | |
| armax | mestery mentioned something about an irc conversation | 22:33 |
| armax | can you point me to the timestamp or fill me in quickly? | 22:33 |
| *** melwitt has quit IRC | 22:34 | |
| *** Sukhdev has joined #openstack-neutron | 22:35 | |
| otherwiseguy | armax: I wasn't actually around for the IRC conversation he referred to, unfortunately. :/ | 22:35 |
| armax | oh..ic | 22:35 |
| *** ajmiller_ has joined #openstack-neutron | 22:36 | |
| armax | otherwiseguy: you mentioned you were working of a few tests | 22:36 |
| armax | ? | 22:36 |
| otherwiseguy | I went ahead and added the test scenarios marun mentioned (except the TransactionQueue ones). | 22:36 |
| *** Marga_ has quit IRC | 22:36 | |
| otherwiseguy | armax: The added assertRaises tests in test_ovs_lib functional tests. | 22:37 |
| armax | otherwiseguy: ok, looking | 22:37 |
| mestery | otherwiseguy armax: Just the IRC conversation around the fact armax marun and I were good with the patch going in | 22:37 |
| mestery | and the tests could be added in RC timeframe | 22:37 |
| mestery | armax: Make sense? | 22:37 |
| *** absubram has quit IRC | 22:37 | |
| armax | mestery: yes it does | 22:37 |
| mestery | armax: cool | 22:38 |
| *** s3wong has quit IRC | 22:38 | |
| *** nati_ueno has quit IRC | 22:39 | |
| *** ajmiller has quit IRC | 22:39 | |
| otherwiseguy | The only tests specifically requested that don't exist right now are the TransactionQueue tests. It is basically just Queue.Queue from the standard library, just with an added signal through reading/writing to a pipe that can be used to wake up a select(). | 22:40 |
| *** nati_ueno has joined #openstack-neutron | 22:40 | |
| *** stefanb has joined #openstack-neutron | 22:40 | |
| otherwiseguy | To really test it well would involve setting up threads, etc. | 22:40 |
| *** dboik has joined #openstack-neutron | 22:41 | |
| *** s3wong has joined #openstack-neutron | 22:42 | |
| *** dboik_ has joined #openstack-neutron | 22:43 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 22:44 |
| *** absubram has joined #openstack-neutron | 22:45 | |
| *** chem has quit IRC | 22:46 | |
| *** dboik has quit IRC | 22:46 | |
| *** Marga_ has joined #openstack-neutron | 22:47 | |
| *** leenheer has joined #openstack-neutron | 22:47 | |
| ZZelle_ | mestery, hi | 22:48 |
| *** jobewan has quit IRC | 22:48 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 22:48 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Test shiny new ovslib backend https://review.openstack.org/165642 | 22:52 |
| *** amuller_walk is now known as amuller | 22:55 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 22:56 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 22:56 |
| *** zzzeek has quit IRC | 22:59 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 23:00 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 23:00 |
| amuller | marun: ^ | 23:00 |
| amuller | marun: You always review my stuff when I'm not around somehow | 23:01 |
| *** sridhar_ram has joined #openstack-neutron | 23:02 | |
| *** ihrachyshka has quit IRC | 23:03 | |
| amuller | armax: Ping about https://review.openstack.org/#/c/164466/4/neutron/tests/functional/agent/test_l3_agent.py | 23:04 |
| *** carlp has quit IRC | 23:07 | |
| armax | amuller: pong | 23:08 |
| amuller | armax: I'm positive I don't understand what you're suggesting there | 23:09 |
| amuller | Can I ask you to explain? About mocking the registry in the functional tests specifically | 23:09 |
| *** fredlhsu has joined #openstack-neutron | 23:09 | |
| armax | amuller: ok, I can explain but I don’t understand why we’re wasting precious cycles on this the day before FF | 23:09 |
| amuller | armax: I'm not the author | 23:09 |
| armax | amuller: that’s not the point, as for the patch, I think I have overlooked the fact that the test affected was indeed functional | 23:11 |
| amuller | alright | 23:11 |
| armax | amuller: that said, I am not sure I understand yours of comments :) | 23:12 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow to request metadata proxy only from internal interfaces https://review.openstack.org/133506 | 23:12 |
| *** fredlhsu has quit IRC | 23:12 | |
| amuller | armax: which comment? | 23:12 |
| *** yamamoto has joined #openstack-neutron | 23:12 | |
| armax | the one you made in response to my comment | 23:12 |
| *** hichihara has joined #openstack-neutron | 23:13 | |
| *** absubram has quit IRC | 23:14 | |
| amuller | armax: I meant that if you mock the registry in the functional tests, the metadata service won't get its notifications. In the router_lifecycle test for example, it's asserting that the metaproxy was started after the router was configured, and destroyed after the router is deleted | 23:14 |
| amuller | armax: So the that test would fail | 23:14 |
| amuller | Along with the specific metadata ones | 23:14 |
| armax | amuller: ok then, but if the proxy is not started/stopped then perhaps we don’t need to clear the registry at all, at least in this patch? | 23:15 |
| armax | I need to spend more time on this patch, I simply don’t have it right now | 23:15 |
| *** ajmiller_ is now known as ajmiller | 23:16 | |
| amuller | armax: I had issues with the previous mechanism which was also a singleton in functional testing | 23:16 |
| amuller | let me look it up | 23:16 |
| armax | amuller: I recall | 23:16 |
| amuller | I thought we'd be preemptive this time | 23:16 |
| amuller | avoid any issues | 23:17 |
| amuller | provide isolation | 23:17 |
| *** nlahouti has quit IRC | 23:17 | |
| *** yamamoto has quit IRC | 23:17 | |
| armax | amuller: I think it’s doable to provide isolation | 23:17 |
| armax | amuller: in the sense that we could give an instance of a registry to each test by mocking the get_registry_manager | 23:18 |
| amuller | ahh, there ya go | 23:18 |
| amuller | that's something else | 23:18 |
| armax | amuller: sorry I mean _get_callback_manager | 23:18 |
| amuller | something like that is also definitely viable for the functional tests | 23:18 |
| armax | the singleton and the callback manager are not bolted onto each other | 23:18 |
| amuller | yep | 23:18 |
| armax | amuller: so that’s what I really meant :P | 23:19 |
| *** yamamoto has joined #openstack-neutron | 23:19 | |
| * armax cheats | 23:19 | |
| amuller | that was my first thought, well let's just not use the singleton, but then I suggested to just clear the registry because it's a one liner, very easy | 23:19 |
| armax | amuller: I know, and that makes sense | 23:19 |
| armax | amuller: if not it might bite back us in the future | 23:20 |
| armax | amuller: we might as well get it right in the first place | 23:20 |
| amuller | okay, well feel free to suggest that in the patch | 23:20 |
| armax | amuller: since this is not critical for Kilo, I don’t understand the rush of +2 and a rain of +1 | 23:20 |
| armax | I sort of did, didn’t I? | 23:20 |
| *** carlp has joined #openstack-neutron | 23:20 | |
| amuller | yeah but now you have something more concrete to suggest that is more actionable by Pau;l | 23:21 |
| amuller | Paul | 23:21 |
| *** ChrisNBlum has quit IRC | 23:21 | |
| amuller | armax: I don't exactly know what pcm had planned on the vpnaas side tbh | 23:21 |
| amuller | I think he wanted to refactor some stuff there for Kilo | 23:21 |
| *** claudiub has quit IRC | 23:21 | |
| otherwiseguy | armax: I'm a little fuzzy on how all of the gate hook stuff works exactly, so i'm not sure if the tempest tests will have the same ovs-vsctl set-manager command run to enable ovsdb-server on TCP:6640 or not. | 23:21 |
| amuller | dunno if there's any significance for this landing in Kilo or L | 23:21 |
| armax | amuller: I think there’s quite a bit of actionable feedback to go on about | 23:21 |
| otherwiseguy | I know it is set up for functional tests, didn't know if it was the same for tempest or not. | 23:22 |
| armax | amuller: but I can further clarify when I don’t have 50 tabs open on my browser, thank you very much | 23:22 |
| armax | :) | 23:22 |
| *** Swami has quit IRC | 23:22 | |
| amuller | armax: Speaking of which: https://review.openstack.org/#/c/125384/ | 23:22 |
| amuller | gogo :) | 23:22 |
| amuller | I want to shove that series along | 23:23 |
| armax | amuller: I have that in one of my tabs :) | 23:23 |
| amuller | move it to the left a bit eh? ;-) | 23:23 |
| *** yuanying has joined #openstack-neutron | 23:23 | |
| armax | amuller: …or right... | 23:23 |
| armax | otherwiseguy: not sure I understand fully | 23:24 |
| amuller | whereever you read first from | 23:24 |
| armax | amuller: randomly perhaps? | 23:25 |
| amuller | naw I trust you prioritize your tabs in some way | 23:26 |
| otherwiseguy | armax: I know for functional tests, tools/configure_for_func_testing.sh sets it up so ovs-vsctl set-manager ptcp:6640:127.0.0.1 is run, which ovsdb_interface=native requires. | 23:26 |
| armax | otherwiseguy: not sure if tempest goes this deep in dealing with the environment | 23:26 |
| otherwiseguy | Wasn't sure if tempest tests end up having that run as well. | 23:26 |
| armax | right | 23:26 |
| *** mlavalle has quit IRC | 23:26 | |
| amuller | otherwiseguy: I don't think that's the case | 23:26 |
| armax | otherwiseguy: I can see that in the script | 23:27 |
| armax | otherwiseguy: looking | 23:27 |
| armax | otherwiseguy: on a related note | 23:28 |
| armax | otherwiseguy: see here: | 23:28 |
| armax | http://133.242.19.163:8000/neutron-ci-logs/check-nec-plugin/19859/logs/devstack.txt.gz | 23:28 |
| armax | I tried this patch | 23:28 |
| *** pritesh has quit IRC | 23:29 | |
| armax | otherwiseguy:https://review.openstack.org/#/c/165642/, I don’t recall correctly, is the IDL library an explicit requirement? | 23:29 |
| amuller | marun: ping about https://review.openstack.org/#/c/128013/15/neutron/notifiers/batch_notifier.py | 23:29 |
| armax | otherwiseguy: I guess python-openvswitch is not pulled automatically | 23:29 |
| *** dims__ has joined #openstack-neutron | 23:29 | |
| otherwiseguy | armax: correct. python-openvswitch is required, and packaged by distros, but not available in PyPI. | 23:30 |
| *** boris-42 has joined #openstack-neutron | 23:30 | |
| otherwiseguy | I could probably create a sanity check for that. | 23:30 |
| *** watanabe_isao has joined #openstack-neutron | 23:32 | |
| armax | otherwiseguy: so going back to your point | 23:32 |
| armax | otherwiseguy: what’s your doubt? | 23:33 |
| *** dims_ has quit IRC | 23:33 | |
| armax | the set-manager is certainly executed during the functional job setup | 23:33 |
| otherwiseguy | armax: Was just wondering if it was going to cause 165642 tempest tests to fail. | 23:35 |
| armax | well | 23:35 |
| armax | otherwiseguy: it looks like we need python-openvswitch installed too for sure | 23:36 |
| otherwiseguy | Which it is. :) | 23:36 |
| armax | the same reason of the failure I showed you before | 23:36 |
| *** diegows_ has quit IRC | 23:36 | |
| otherwiseguy | armax: I'd really like to get the python-openvswitch stuff in PyPI. Kept planning to try to get that done on the OVS side, but just haven't had much time. | 23:37 |
| otherwiseguy | Also seems like it would be handy to have the OVS IDL stuff outside of the OVS tree since it doesn't rely on any of the C code. No reason that releases need to be synced up, etc. | 23:37 |
| armax | otherwiseguy: makes sense | 23:38 |
| armax | so where were you looking at the 165642 tempest tests failures | 23:38 |
| armax | ? | 23:38 |
| armax | otherwiseguy: or was that just hypotetical | 23:38 |
| armax | ? | 23:38 |
| otherwiseguy | Just hypothetical, then started looking at zuul. | 23:39 |
| HenryG | armax: ping, while the stars are aligned, https://review.openstack.org/157243 | 23:45 |
| armax | HenryG: this most likely will need to line up with other patches with migrations | 23:46 |
| HenryG | armax: OK. Are some in flight? | 23:47 |
| armax | yees | 23:47 |
| armax | there’s one in the gate queue right now | 23:47 |
| *** padkrish_ has quit IRC | 23:48 | |
| armax | and it looks like we’re experiencing some instability to the PG job | 23:48 |
| HenryG | blegh | 23:49 |
| *** padkrish has joined #openstack-neutron | 23:49 | |
| *** sripriya has quit IRC | 23:49 | |
| armax | HenryG: chasing down... | 23:49 |
| amuller | marun: armax: Next patch in line should be ready: https://review.openstack.org/#/c/155370/ | 23:50 |
| *** apuimedo has joined #openstack-neutron | 23:50 | |
| *** nlahouti has joined #openstack-neutron | 23:53 | |
| *** padkrish has quit IRC | 23:54 | |
| *** xgerman has quit IRC | 23:54 | |
| *** ajmiller has quit IRC | 23:55 | |
| *** dave-mccowan has joined #openstack-neutron | 23:56 | |
| *** sthillma has joined #openstack-neutron | 23:59 | |
| *** yamamoto has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!