*** xuhanp has joined #openstack-neutron | 00:00 | |
*** yamahata has quit IRC | 00:04 | |
*** WackoRobie has quit IRC | 00:07 | |
*** WackoRobie has joined #openstack-neutron | 00:07 | |
*** ihrachyshka has quit IRC | 00:08 | |
*** xuhanp has quit IRC | 00:25 | |
*** markmcclain has joined #openstack-neutron | 00:38 | |
*** sputnik13net has joined #openstack-neutron | 00:41 | |
*** markmcclain has quit IRC | 00:43 | |
*** yamahata has joined #openstack-neutron | 00:54 | |
*** yongli has joined #openstack-neutron | 00:57 | |
*** sp has joined #openstack-neutron | 00:59 | |
*** mwagner_lap has joined #openstack-neutron | 01:00 | |
*** alagalah has quit IRC | 01:01 | |
*** idella4 has joined #openstack-neutron | 01:13 | |
*** dims has joined #openstack-neutron | 01:21 | |
*** dkehn__ has quit IRC | 01:24 | |
*** dkehn__ has joined #openstack-neutron | 01:25 | |
*** idella4 has left #openstack-neutron | 01:27 | |
*** iwamoto has joined #openstack-neutron | 01:28 | |
*** yamahata has quit IRC | 01:33 | |
*** ijw has quit IRC | 01:33 | |
*** WackoRobie has quit IRC | 01:37 | |
*** yamahata has joined #openstack-neutron | 01:48 | |
*** WackoRobie has joined #openstack-neutron | 01:50 | |
*** RajeshMohan has quit IRC | 01:51 | |
*** RajeshMohan has joined #openstack-neutron | 01:53 | |
*** yamahata has quit IRC | 01:53 | |
*** morganfainberg_Z is now known as morganfainberg | 01:56 | |
*** WackoRob_ has joined #openstack-neutron | 01:57 | |
*** WackoRobie has quit IRC | 01:57 | |
*** networkstatic has joined #openstack-neutron | 02:08 | |
*** xianghui has joined #openstack-neutron | 02:12 | |
*** yongli has quit IRC | 02:31 | |
*** yamahata has joined #openstack-neutron | 02:36 | |
*** dkehn__ is now known as dkehn_ | 02:45 | |
*** ijw has joined #openstack-neutron | 03:00 | |
*** morganfainberg is now known as morganfainberg_Z | 03:19 | |
*** morganfainberg_Z is now known as morganfainberg | 03:20 | |
*** sputnik13net has quit IRC | 03:27 | |
*** harlowja is now known as harlowja_away | 03:30 | |
*** suresh12 has joined #openstack-neutron | 03:31 | |
*** markmcclain has joined #openstack-neutron | 03:31 | |
openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers https://review.openstack.org/72252 | 03:35 |
---|---|---|
*** banix has quit IRC | 03:40 | |
*** alagalah has joined #openstack-neutron | 03:41 | |
*** irenab has joined #openstack-neutron | 03:46 | |
irenab | Looking for core team member for question regarding patch approval | 03:54 |
*** sp has quit IRC | 03:58 | |
*** WackoRob_ has quit IRC | 04:02 | |
openstackgerrit | Chengli Xu proposed a change to openstack/python-neutronclient: str2dict support commas within quotes https://review.openstack.org/65097 | 04:05 |
*** dguitarbite has joined #openstack-neutron | 04:06 | |
*** baoli has quit IRC | 04:06 | |
*** MM_at_HP has quit IRC | 04:07 | |
*** MM_at_HP has joined #openstack-neutron | 04:07 | |
markmcclain | irenab: I'm around.. which patch? | 04:10 |
irenab | markmcclain: https://review.openstack.org/#/c/53609/ | 04:15 |
irenab | Thanks! | 04:15 |
* markmcclain looking | 04:16 | |
markmcclain | irenab: I'm going to run a re-check since the last one was almost two week ago.. it is passes, I'll approve | 04:17 |
markmcclain | s/it is/if it/ | 04:18 |
irenab | markmcclain: Thanks a lot | 04:18 |
*** oda-g has quit IRC | 04:22 | |
*** oda-g has joined #openstack-neutron | 04:24 | |
*** gongysh has joined #openstack-neutron | 04:25 | |
openstackgerrit | A change was merged to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/71761 | 04:26 |
openstackgerrit | mark mcclain proposed a change to openstack/neutron: Delete DHCP port without DHCP server on a network node https://review.openstack.org/56740 | 04:29 |
openstackgerrit | mark mcclain proposed a change to openstack/neutron: Delete DHCP port without DHCP server on a net node https://review.openstack.org/56740 | 04:31 |
*** WackoRobie has joined #openstack-neutron | 04:35 | |
*** irenab has quit IRC | 04:38 | |
openstackgerrit | A change was merged to openstack/neutron: Fix typo in rootwrap files: neuton -> neutron https://review.openstack.org/72134 | 04:40 |
*** WackoRobie has quit IRC | 04:41 | |
*** yfried has joined #openstack-neutron | 04:56 | |
*** alagalah has quit IRC | 05:13 | |
*** chandankumar_ has joined #openstack-neutron | 05:17 | |
*** irenab has joined #openstack-neutron | 05:20 | |
*** aymenfrikha has joined #openstack-neutron | 05:34 | |
*** gdubreui has quit IRC | 05:35 | |
*** gdubreui has joined #openstack-neutron | 05:35 | |
*** WackoRobie has joined #openstack-neutron | 05:38 | |
*** dguitarbite has quit IRC | 05:42 | |
*** markwash has joined #openstack-neutron | 05:44 | |
*** rotbeard has joined #openstack-neutron | 05:44 | |
*** WackoRobie has quit IRC | 05:46 | |
*** markwash has quit IRC | 05:48 | |
*** markwash has joined #openstack-neutron | 05:49 | |
*** dguitarbite has joined #openstack-neutron | 05:49 | |
openstackgerrit | A change was merged to openstack/neutron: Return request-id in API response https://review.openstack.org/58270 | 05:53 |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Change tenant network type usage for IB Fabric https://review.openstack.org/67189 | 05:55 |
*** vmtrooper has quit IRC | 05:59 | |
*** vmtrooper has joined #openstack-neutron | 06:00 | |
*** amotoki has joined #openstack-neutron | 06:05 | |
*** aymenfrikha has quit IRC | 06:06 | |
*** WackoRobie has joined #openstack-neutron | 06:13 | |
*** markmcclain has quit IRC | 06:13 | |
*** markmcclain has joined #openstack-neutron | 06:16 | |
*** WackoRobie has quit IRC | 06:18 | |
*** markmcclain1 has joined #openstack-neutron | 06:18 | |
*** markmcclain has quit IRC | 06:21 | |
openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/72271 | 06:24 |
*** pbeskow has quit IRC | 06:36 | |
*** gdubreui has quit IRC | 06:37 | |
*** pbeskow has joined #openstack-neutron | 06:37 | |
openstackgerrit | Hyunsun Moon proposed a change to openstack/neutron: Fix binding:host_id is set to None when port update https://review.openstack.org/62829 | 06:41 |
*** markmcclain1 has quit IRC | 06:42 | |
*** WackoRobie has joined #openstack-neutron | 06:43 | |
*** ihrachyshka has joined #openstack-neutron | 06:47 | |
*** WackoRobie has quit IRC | 06:48 | |
openstackgerrit | Isaku Yamahata proposed a change to openstack/neutron: mailmap: update .mailmap https://review.openstack.org/72274 | 06:49 |
*** saju_m has joined #openstack-neutron | 06:53 | |
*** saju_m has quit IRC | 07:00 | |
openstackgerrit | Isaku Yamahata proposed a change to openstack/neutron: tests/service: consolidate setUp/tearDown logic https://review.openstack.org/70668 | 07:03 |
*** ajo has joined #openstack-neutron | 07:04 | |
*** chandankumar_ has quit IRC | 07:07 | |
*** chandan_kumar has joined #openstack-neutron | 07:07 | |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add update from agent to plugin on device up https://review.openstack.org/53609 | 07:08 |
*** uaberme has joined #openstack-neutron | 07:08 | |
*** akamyshnikova has joined #openstack-neutron | 07:12 | |
*** WackoRobie has joined #openstack-neutron | 07:14 | |
*** saju_m has joined #openstack-neutron | 07:16 | |
*** igor has joined #openstack-neutron | 07:16 | |
*** dims has quit IRC | 07:18 | |
*** dims has joined #openstack-neutron | 07:19 | |
*** WackoRobie has quit IRC | 07:20 | |
*** igor has quit IRC | 07:20 | |
*** igor has joined #openstack-neutron | 07:21 | |
doude | gongysh: hi | 07:21 |
gongysh | doude: hi | 07:21 |
*** ijw has quit IRC | 07:21 | |
doude | gongysh: Do you have time to have a look of my draft https://review.openstack.org/#/c/68694/ ? | 07:21 |
doude | gongysh: this is not urgent, it's just to know | 07:23 |
gongysh | doude: why do u need to do this? | 07:24 |
gongysh | doude: all the dhcp notification is in https://review.openstack.org/#/c/68694/2/neutron/api/v2/base.py | 07:25 |
doude | gongysh: I propose that in order thaht DHCP agent is aware of l3 port delete, update or create | 07:25 |
gongysh | doude: why do u delete them there? | 07:25 |
doude | gongysh: why the DHCP notification are in neutron/api/v2/base.py ? | 07:25 |
gongysh | doude: because this is a common feature for all plugins. | 07:26 |
doude | when a service like l3 , lbaas, fwaas... make a change on a ressource, the agent DHCP doesn't have any notifications | 07:26 |
doude | gongysh: all plugin implements inevitably DHCP agent? | 07:27 |
gongysh | doude: almost. | 07:27 |
amotoki | doude: gongysh: i don't think DHCP notification is a common feature, although most plugins/drivers use dhcp-agent now. | 07:27 |
doude | gongysh: ok | 07:28 |
gongysh | amotoki: this is decided when we implemented the dhcp agent notification. | 07:28 |
doude | gongysh: but I propose that in order the DHCP agent know l3 port modification and activate or desactivate metadata proxy service | 07:28 |
doude | actually it starts inevitably a metadata proxy | 07:29 |
doude | event if a gw port is configured on the network | 07:29 |
amotoki | gongysh: i am okay that it is placed in api.v2.base at now, but I believe it is not a requirement. | 07:30 |
doude | gongysh, amotoki : and we lost all ressource modifications notifications when they are created by the plugin (an not thourgh the API) | 07:31 |
gongysh | doude: previoiusly, we thought only the compute's port should be dhcp supported. | 07:32 |
doude | gongysh: ha ok, I understand | 07:33 |
gongysh | doude: why do u want the l3 ports be dhcp supported? | 07:33 |
doude | gongysh: it's not to be DHCP supported but DHCP agent need to know if l3 port status on the network to know if it needs to activate or not (update) metadata proxy service | 07:34 |
doude | gongysh: I like to propose this improveme | 07:35 |
gongysh | doude: then you can send an ad-hoc dhcp notification. | 07:37 |
doude | gongysh: I like to propose this improvement because in our production we saw that metadata proxy consume lot of memory and actually a metadata proxy was started by the l3 agent and DHCP agent at the same time for a same network | 07:38 |
gongysh | doude: why not disable the dhcp agent started metadata proxy? | 07:39 |
doude | and lot of memory are consume uselessly on the network and decrease the Neutron network node scalability | 07:39 |
doude | gongysh: in the case of the network is isolated, the DHCP agent should provide the metadata proxy | 07:40 |
openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Raise an error from ovs_lib list operations https://review.openstack.org/72282 | 07:41 |
openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers https://review.openstack.org/72252 | 07:41 |
gongysh | doude: yes | 07:43 |
doude | gongysh: it's a proposal and just want to know your opinion | 07:44 |
openstackgerrit | Ofer Barkai proposed a change to openstack/neutron: Enhance floating IP router lookup https://review.openstack.org/55987 | 07:45 |
*** WackoRobie has joined #openstack-neutron | 07:46 | |
*** ihrachyshka has quit IRC | 07:46 | |
*** mflobo has joined #openstack-neutron | 07:46 | |
*** HenryG has quit IRC | 07:47 | |
*** ihrachyshka has joined #openstack-neutron | 07:47 | |
gongysh | doude: what is your plan? you want to shutdown the dhcp started proxy when the network is changed from isolated into router connected? | 07:47 |
*** HenryG has joined #openstack-neutron | 07:48 | |
doude | gongysh: yes and remove DHCP static route from the lease | 07:48 |
gongysh | doude: sound good. but don't change the overall dhcp notification framework, which will be a big change. | 07:50 |
*** WackoRobie has quit IRC | 07:51 | |
*** ihrachyshka has quit IRC | 07:51 | |
doude | gongysh: but I require to change DHCP notification framework to notify l3 port update | 07:52 |
gongysh | doude: no need, you just need to send an ad-hoc notification to dhcp agent if the port is l3 port. | 07:53 |
doude | gongysh: ok, but do you think we could have a new service (XaaS) that needs to be dhcp supported? | 07:55 |
*** sweston_ has joined #openstack-neutron | 07:57 | |
*** evgenyf has joined #openstack-neutron | 07:58 | |
*** sweston has quit IRC | 07:58 | |
*** salv-orlando has joined #openstack-neutron | 07:59 | |
*** markwash has quit IRC | 07:59 | |
*** ijw has joined #openstack-neutron | 08:00 | |
*** ekarlso has quit IRC | 08:04 | |
*** ekarlso has joined #openstack-neutron | 08:05 | |
*** ekarlso has quit IRC | 08:12 | |
openstackgerrit | Isaku Yamahata proposed a change to openstack/neutron: tests/service: consolidate setUp/tearDown logic https://review.openstack.org/70668 | 08:12 |
*** ekarlso has joined #openstack-neutron | 08:13 | |
*** pasquier-s has joined #openstack-neutron | 08:14 | |
*** WackoRobie has joined #openstack-neutron | 08:17 | |
*** joshh has quit IRC | 08:21 | |
*** morganfainberg has quit IRC | 08:22 | |
*** joshh has joined #openstack-neutron | 08:23 | |
*** WackoRobie has quit IRC | 08:23 | |
*** morganfainberg has joined #openstack-neutron | 08:23 | |
*** ihrachyshka has joined #openstack-neutron | 08:25 | |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: move agent based driver files into a separate dir https://review.openstack.org/61339 | 08:26 |
*** jlibosva has joined #openstack-neutron | 08:32 | |
openstackgerrit | IWAMOTO Toshihiro proposed a change to openstack/neutron: LBaaS: implements LVS driver (WORKINPROGRESS) https://review.openstack.org/67405 | 08:37 |
*** iwamoto has quit IRC | 08:38 | |
*** suresh12 has quit IRC | 08:41 | |
ihrachys | amotoki: could you please check the following simple fix for neutronclient: https://review.openstack.org/#/c/71551/ | 08:42 |
ihrachys | amotoki: sorry to reach directly, but I need this to be checked by core devs asap to proceed with another performance fix for neutron | 08:43 |
amotoki | ihrachys: sure. looking now. | 08:45 |
ajo | hmmm | 08:49 |
*** jistr has joined #openstack-neutron | 08:49 | |
ajo | I'm seeing failures after only changing the commit message here: https://review.openstack.org/#/c/68601/ , it's a backport to havana | 08:49 |
*** ygbo has joined #openstack-neutron | 08:50 | |
*** feleouet has joined #openstack-neutron | 08:50 | |
ajo | SSHTimeout: Connection to the 172.24.4.228 via SSH timed out. | 08:50 |
*** WackoRobie has joined #openstack-neutron | 08:50 | |
*** ijw has quit IRC | 08:51 | |
*** WackoRobie has quit IRC | 08:54 | |
*** matrohon has joined #openstack-neutron | 08:58 | |
openstackgerrit | Edgar Magana proposed a change to openstack/neutron: Implements provider network support in PLUMgrid plugin https://review.openstack.org/60383 | 08:59 |
*** safchain has joined #openstack-neutron | 09:00 | |
openstackgerrit | liusheng proposed a change to openstack/python-neutronclient: Remove vi modelines https://review.openstack.org/72297 | 09:02 |
*** jpich has joined #openstack-neutron | 09:04 | |
openstackgerrit | A change was merged to openstack/neutron: Fix misspellings in neutron https://review.openstack.org/71781 | 09:06 |
openstackgerrit | A change was merged to openstack/neutron: Disallow non-admin users update net's shared attribute https://review.openstack.org/69095 | 09:08 |
*** jlibosva has quit IRC | 09:14 | |
*** jlibosva has joined #openstack-neutron | 09:17 | |
ihrachys | amotoki: thanks for +2! any other core dev for neutronclient willing to add another +2 to obvious bug fix with great impact on neutron scalability? :) https://review.openstack.org/#/c/71551/ | 09:20 |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: Introduce Loadbalancer instance https://review.openstack.org/60207 | 09:20 |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: apply agent scheduling for loadbalancer instance https://review.openstack.org/71472 | 09:20 |
*** WackoRobie has joined #openstack-neutron | 09:21 | |
*** rotbeard has quit IRC | 09:25 | |
*** WackoRobie has quit IRC | 09:25 | |
*** rotbeard has joined #openstack-neutron | 09:28 | |
*** morganfainberg is now known as morganfainberg_Z | 09:28 | |
*** mjbot has joined #openstack-neutron | 09:28 | |
*** afazekas has joined #openstack-neutron | 09:29 | |
*** rotbeard has quit IRC | 09:32 | |
openstackgerrit | Jun Jie Nan proposed a change to openstack/neutron: Check vxlan enablement via modinfo https://review.openstack.org/65902 | 09:35 |
*** rotbeard has joined #openstack-neutron | 09:37 | |
*** jlibosva has quit IRC | 09:39 | |
*** bvandenh has joined #openstack-neutron | 09:39 | |
openstackgerrit | A change was merged to openstack/neutron: add quota support for ryu plugin https://review.openstack.org/65245 | 09:40 |
openstackgerrit | A change was merged to openstack/neutron: ML2 plugin cannot raise NoResultFound exception https://review.openstack.org/72096 | 09:44 |
*** pradipta` has joined #openstack-neutron | 09:50 | |
*** WackoRobie has joined #openstack-neutron | 09:52 | |
*** pradipta has quit IRC | 09:54 | |
*** dguitarbite has quit IRC | 09:55 | |
*** WackoRobie has quit IRC | 09:56 | |
*** jlibosva has joined #openstack-neutron | 09:56 | |
*** networkstatic has quit IRC | 09:57 | |
*** dguitarbite has joined #openstack-neutron | 10:00 | |
*** jp_at_hp has joined #openstack-neutron | 10:01 | |
*** ihrachyshka has quit IRC | 10:08 | |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: apply agent scheduling for loadbalancer instance https://review.openstack.org/71472 | 10:10 |
openstackgerrit | Xiang Hui proposed a change to openstack/neutron: Remove legacy quantum config path https://review.openstack.org/68846 | 10:13 |
*** gongysh has quit IRC | 10:20 | |
*** networkstatic has joined #openstack-neutron | 10:20 | |
*** WackoRobie has joined #openstack-neutron | 10:22 | |
*** WackoRobie has quit IRC | 10:27 | |
EmilienM | enikanorov: hi Eugene, I've abandoned https://review.openstack.org/#/c/63240/ - you can see why in the last comment | 10:29 |
*** xianghui has quit IRC | 10:31 | |
*** RajeshMohan has quit IRC | 10:33 | |
*** RajeshMohan has joined #openstack-neutron | 10:34 | |
*** amotoki has quit IRC | 10:38 | |
*** yamahata has quit IRC | 10:41 | |
*** djoreilly has joined #openstack-neutron | 10:51 | |
*** irenab has quit IRC | 11:08 | |
*** safchain has quit IRC | 11:11 | |
*** safchain has joined #openstack-neutron | 11:12 | |
enikanorov | EmilienM: ok, but i see Nach's patch is abandoned as well | 11:17 |
enikanorov | *Nachi's | 11:17 |
openstackgerrit | Christoph Thiel proposed a change to openstack/python-neutronclient: Allow allocating specific floating ip for admins https://review.openstack.org/23562 | 11:22 |
*** WackoRobie has joined #openstack-neutron | 11:24 | |
*** pcm_ has joined #openstack-neutron | 11:26 | |
*** pcm_ has quit IRC | 11:29 | |
*** pcm_ has joined #openstack-neutron | 11:30 | |
*** dguitarbite has quit IRC | 11:30 | |
*** sweston_ has quit IRC | 11:31 | |
*** WackoRobie has quit IRC | 11:32 | |
akamyshnikova | anteaya, Hello! I'm writing to you because you did review of my change https://review.openstack.org/55411 some weeks ago. At that time there was problems with passing tests. Now everything works correct. Please, visit my change https://review.openstack.org/55411 one more time and remove -1 if nothing else confuses you. | 11:38 |
*** RajeshMohan has quit IRC | 11:40 | |
*** RajeshMohan has joined #openstack-neutron | 11:41 | |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: DB Mappings for NSX security groups https://review.openstack.org/71692 | 11:44 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NSX plugin: Introduce operational status for security groups https://review.openstack.org/71693 | 11:44 |
openstackgerrit | Christoph Thiel proposed a change to openstack/python-neutronclient: Allow allocating specific floating ip for admins https://review.openstack.org/23562 | 11:50 |
openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Fix import NeutronManager https://review.openstack.org/68887 | 11:51 |
*** pradipta` is now known as pradipta | 11:55 | |
openstackgerrit | A change was merged to openstack/neutron: Fix binding:host_id is set to None when port update https://review.openstack.org/62829 | 11:56 |
*** HenryG has quit IRC | 11:58 | |
*** WackoRobie has joined #openstack-neutron | 11:59 | |
*** jroovers has joined #openstack-neutron | 12:01 | |
*** joshh has quit IRC | 12:01 | |
jroovers | hi yfried | 12:03 |
*** WackoRobie has quit IRC | 12:03 | |
*** cburgess has quit IRC | 12:04 | |
*** cburgess has joined #openstack-neutron | 12:11 | |
*** joshh has joined #openstack-neutron | 12:13 | |
*** irenab has joined #openstack-neutron | 12:14 | |
*** HenryG has joined #openstack-neutron | 12:14 | |
HenryG | enikanorov: ping | 12:15 |
enikanorov | HenryG: pong | 12:19 |
HenryG | enikanorov: Hi, you gave a -1 here and rcurran answered your question, can you check? https://review.openstack.org/#/c/56478/4 | 12:21 |
enikanorov | HenryG: sure, let me look | 12:22 |
HenryG | enikanorov: thanks! | 12:22 |
*** yamahata has joined #openstack-neutron | 12:22 | |
yfried | jroovers: looking for me? | 12:22 |
jroovers | yep, how are you, it's been a while since I've been here? | 12:23 |
yfried | I'm ok. how can I help you? | 12:23 |
jroovers | yfried, https://review.openstack.org/#/c/66854/ | 12:23 |
jroovers | this patch set is currently abandoned, but that is not my question | 12:24 |
jroovers | can you have a look at Masayuki's comment | 12:24 |
yfried | jroovers: what about it? | 12:25 |
jroovers | yfried, well, I was wondering whether you have any insight into this. | 12:25 |
jroovers | is there a guideline around scenario/functional tests and neutron moving forward? | 12:26 |
enikanorov | HenryG: ok, so I'm still abit confused about this part of code. it calls ..get_bininG() and expects bindingS | 12:27 |
enikanorov | and checks for amount of bindings | 12:27 |
enikanorov | HenryG: do you know what is expected at line 107? | 12:28 |
HenryG | enikanorov: checking ... | 12:29 |
yfried | jroovers: I understand his point - the strict definition of scenario tests is at least 2 components working together | 12:30 |
jroovers | yfried, I agree | 12:30 |
jroovers | yfried, I do think that a tests like the one this patchset provides can be very relevant. | 12:30 |
jroovers | yfried, so wondering where it would belong | 12:30 |
yfried | jroovers: I'm not seeing any actual tests in the patch - just moving network related scenario to a new folder | 12:31 |
jroovers | sorry (working on too much stuff at the same time), I based a different one on it: https://review.openstack.org/#/c/65515 | 12:32 |
yfried | jroovers: However, in practice, I agree with your approach - there can be a lot of network targeted tests that conform to that definition but actually test only neutron functionality | 12:32 |
*** IanGovett has joined #openstack-neutron | 12:33 | |
yfried | jroovers: so to what comment are you referring? | 12:34 |
jroovers | yfried, so I based this VPN scenario test on the directory restructuring | 12:35 |
jroovers | so 65515 is based on 66854 | 12:35 |
jroovers | as I wanted to bring some more structure | 12:35 |
HenryG | enikanorov: I agree get_..._binding() should be plural (and the comment in the method fixed) | 12:36 |
jroovers | I asked sdague about this, and he agreed | 12:36 |
jroovers | yfried, but I also agree Masayuki's point | 12:36 |
jroovers | by create a separate network directory, it does seems as if the tests are only focussed on the network. While this is not really so. In the case of the VPN scenario tests, we do spin up vm's in the networks (=nova). | 12:37 |
yfried | jroovers: tell you what - push the test independently, as it's a good idea, then, once you have 3 network tests, file a bug and get support for the new dir | 12:38 |
*** akamyshnikova has quit IRC | 12:38 | |
jroovers | yfried, sounds good, thanks for the guidance :-) | 12:38 |
HenryG | enikanorov: however, changing that would ripple out to 6 places where it is called. I think rcurran may want to avoid that for his bug. | 12:39 |
yfried | a directory for 2 tests is not really that strong. | 12:39 |
yfried | jroovers: however, you should add LBaaS and FWaaS to such directory | 12:39 |
enikanorov | sure, no prob, the question is about L107, is that 'else' expects more than 1 binding? | 12:39 |
enikanorov | or no bindings? | 12:39 |
yfried | jroovers: because, disregarding the strict definition for a moment, once we get too many scenario modules we will need some order in that directory | 12:40 |
jroovers | yfried, yes, I agree :-) | 12:41 |
HenryG | enikanorov: ah, ok. Are you prepared to learn about the quirks of configuring nexus TOR switches? | 12:41 |
yfried | jroovers: maybe you'll get other ideas on how to classify scenario tests, aside from service based | 12:41 |
enikanorov | HenryG: nope :) I'm just trying to check correspondence between client code (the patch) and get_bindings | 12:42 |
yfried | jroovers: also - I don't really have time for review (I'm on leave for exams) but if you are using methods from other modules (such as connectivity tests), try to get the code to manager.py | 12:43 |
yfried | jroovers: IMO - copy-paste makes for bad coding, and difficult maintenance | 12:43 |
jroovers | yfried, I agree. Sorry, this code is one of my first contributions, I wanted to keep the patch rather simple. | 12:45 |
jroovers | yfried, but I will follow up on that, thanks! | 12:45 |
*** banix has joined #openstack-neutron | 12:45 | |
yfried | jroovers: I understand the reluctance, but you will be thanked for it later. | 12:45 |
openstackgerrit | Sascha Peilicke proposed a change to openstack/neutron: Support passing 'neutron_insecure' to neutronclient https://review.openstack.org/65696 | 12:47 |
yfried | jroovers: good job so far - keep it on | 12:48 |
jroovers | yfried, thanks :-) | 12:48 |
*** banix has quit IRC | 12:50 | |
*** bada has joined #openstack-neutron | 12:51 | |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations https://review.openstack.org/55411 | 12:54 |
*** baoli has joined #openstack-neutron | 12:55 | |
HenryG | enikanorov: The method _configure_switch_entry() is called at post_commit, so bindings cannot be 0. | 12:58 |
HenryG | enikanorov: does that answer your question? | 12:58 |
enikanorov | HenryG: yes. thanks! | 12:59 |
enikanorov | HenryG: i've removed -1 | 13:00 |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add support to request vnic type on port https://review.openstack.org/72334 | 13:01 |
openstackgerrit | Christoph Thiel proposed a change to openstack/python-neutronclient: Allow allocating specific floating ip for admins https://review.openstack.org/23562 | 13:02 |
ajo | http://logs.openstack.org/01/68601/2/check/gate-neutron-python26/69a7a57/console.html#_2014-02-10_10_31_41_311 | 13:03 |
ajo | wow | 13:03 |
ajo | has anybody seen a failure like this before? | 13:03 |
ajo | it failed with a timeout on testools/subunit itself for py26 | 13:03 |
EmilienM | obondarev: ping | 13:03 |
EmilienM | enikanorov: does LBaaS need a public IP address ? or does it use L3 agent to manage public VIPs? | 13:05 |
HenryG | enikanorov: great! thanks! | 13:05 |
enikanorov | EmilienM: haproxy vips are on tenant networks so they require floating ip to be associated to be reachable | 13:06 |
EmilienM | enikanorov: but if i install the lbaas agent on another node than l3 agent, do I need public network access? | 13:07 |
enikanorov | that doesn't matter where lbaas & l3 agents are | 13:08 |
enikanorov | l3 agent will NAT from external network to a vip on tenant network | 13:08 |
EmilienM | enikanorov: i just wonder if lbaas server can be in internal network | 13:08 |
EmilienM | so yes, it seems to be, thanks :-) | 13:08 |
enikanorov | it is | 13:08 |
EmilienM | enikanorov: thx :() | 13:08 |
*** saju_m has quit IRC | 13:12 | |
*** ilyashakhat has quit IRC | 13:14 | |
*** dave_tucker is now known as dave_tucker_zzz | 13:17 | |
anteaya | akamyshnikova: my -1 disappeared when you submitted patchset 13, I hope it passes check tests for you | 13:17 |
openstackgerrit | A change was merged to openstack/neutron: Support building wheels (PEP-427) https://review.openstack.org/57124 | 13:20 |
ajo | https://bugs.launchpad.net/neutron/+bug/1278430 I submited this bug, it seems that unit test fails when sending a possitive test result... | 13:20 |
ajo | may be the testing systems are a little bit too stressed | 13:21 |
ajo | in fact, it seems that most gate-nova-python26 at test jobs | 13:23 |
ajo | remain in "queued" in the gate | 13:23 |
openstackgerrit | enikanorov proposed a change to openstack/neutron: Introduce Loadbalancer instance https://review.openstack.org/60207 | 13:25 |
ajo | anteaya ^^ | 13:25 |
ajo | btw, what does mean that a test is marked as "LOST" ? :? :) | 13:26 |
*** WackoRobie has joined #openstack-neutron | 13:30 | |
*** alagalah has joined #openstack-neutron | 13:31 | |
*** ilyashakhat has joined #openstack-neutron | 13:32 | |
anteaya | ajo: so you are asking me what it means if the logs from a test are marked as logged, is that correct? | 13:33 |
ajo | no | 13:33 |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations https://review.openstack.org/55411 | 13:33 |
*** jgallard has joined #openstack-neutron | 13:33 | |
ajo | Im asking about the bug I submitted | 13:33 |
ajo | It seems that we're running into timeouts for py26 tests, | 13:33 |
ajo | normal passing tests | 13:34 |
anteaya | do you have the url for a patch that shows this? | 13:34 |
ajo | sure, wait, it's a cherry pick backport | 13:34 |
ajo | https://review.openstack.org/#/c/68601/ | 13:35 |
ajo | If I didn't understand the traceback wrong, it seems that the unit test passed, but when unittest tried to report the success of this test (via it's internal mechanism... it ended in a timeout) | 13:37 |
*** tongli has joined #openstack-neutron | 13:38 | |
*** baoli_ has joined #openstack-neutron | 13:38 | |
*** baoli has quit IRC | 13:40 | |
*** dave_tucker_zzz is now known as dave_tucker | 13:46 | |
*** dims has quit IRC | 13:47 | |
openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Fix Metering doesn't respect the l3 agent binding https://review.openstack.org/60016 | 13:48 |
anteaya | ajo infra is having random failure issues this morning, it appears, I am trying to understand why | 13:48 |
ajo | oh, ok, thanks anteaya , If they're are aware, then it will be ok | 13:49 |
anteaya | we are trying to understand and compare notes regarding what we are aware of | 13:50 |
anteaya | right now we seem to be aware of different things | 13:50 |
anteaya | trying to share and sort out what is relevant | 13:50 |
*** baoli_ is now known as baoli | 13:50 | |
*** dims has joined #openstack-neutron | 13:50 | |
*** yamahata has quit IRC | 13:56 | |
*** safchain has quit IRC | 13:58 | |
*** yamahata has joined #openstack-neutron | 13:59 | |
*** safchain has joined #openstack-neutron | 14:00 | |
openstackgerrit | Avishay Balderman proposed a change to openstack/neutron: LBaaS L7 model (WIP) https://review.openstack.org/61721 | 14:00 |
*** xuhanp has joined #openstack-neutron | 14:01 | |
*** xuhanp has quit IRC | 14:25 | |
*** rossella_s has joined #openstack-neutron | 14:25 | |
*** jecarey has joined #openstack-neutron | 14:28 | |
*** jecarey_ has joined #openstack-neutron | 14:29 | |
*** jistr has quit IRC | 14:29 | |
openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Add support to request vnic type on port https://review.openstack.org/72334 | 14:34 |
*** ramishra has joined #openstack-neutron | 14:35 | |
*** jistr has joined #openstack-neutron | 14:39 | |
*** obondarev_ has joined #openstack-neutron | 14:41 | |
*** peristeri has joined #openstack-neutron | 14:42 | |
*** matsuhashi has joined #openstack-neutron | 14:49 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Metadata agent caches networks for routers https://review.openstack.org/70689 | 14:54 |
*** ramishra has quit IRC | 14:54 | |
*** ramishra has joined #openstack-neutron | 14:55 | |
*** yamahata has quit IRC | 14:59 | |
*** ramishra_ has joined #openstack-neutron | 15:00 | |
ihrachys | let's assume we get a fix in neutronclient that unlocks review for another fix to neutron. so to be able to pass jenkins checks, we probably need a new neutronclient version to depend on for neutron (I guess we can't depend on nightly releases). Does it mean that the sequence of steps to get neutron fix that depends on neutronclient fix is as follows: 1. release new neutronclient with the fix; 2. | 15:00 |
*** banix has joined #openstack-neutron | 15:00 | |
ihrachys | update global requirements for neutronclient to enforce new release version; 3. update requirements in neutron based on new global requirements; 4. finally, send the fix for neutron and get it merged. | 15:00 |
ihrachys | does anyone follow me? :) | 15:01 |
*** ramishra has quit IRC | 15:01 | |
*** AramakiHiroki has joined #openstack-neutron | 15:01 | |
*** irenab has quit IRC | 15:01 | |
*** saju_m has joined #openstack-neutron | 15:01 | |
*** yamahata has joined #openstack-neutron | 15:03 | |
*** saju_m has quit IRC | 15:03 | |
*** saju_m has joined #openstack-neutron | 15:04 | |
*** clev has joined #openstack-neutron | 15:07 | |
ajo | ihrachys, I follow you :) | 15:07 |
*** markwash has joined #openstack-neutron | 15:07 | |
*** otherwiseguy has joined #openstack-neutron | 15:08 | |
*** sdague has quit IRC | 15:12 | |
*** jgrimm has joined #openstack-neutron | 15:16 | |
*** rwsu has joined #openstack-neutron | 15:17 | |
*** chandan_kumar has quit IRC | 15:20 | |
*** sdague has joined #openstack-neutron | 15:29 | |
*** AramakiHiroki has quit IRC | 15:31 | |
*** AramakiHiroki has joined #openstack-neutron | 15:32 | |
*** markwash has quit IRC | 15:32 | |
*** sweston has joined #openstack-neutron | 15:33 | |
*** armax has joined #openstack-neutron | 15:34 | |
*** AramakiHiroki has quit IRC | 15:36 | |
*** matsuhashi has quit IRC | 15:37 | |
*** yamahata has quit IRC | 15:37 | |
*** matsuhashi has joined #openstack-neutron | 15:37 | |
*** saju_m has quit IRC | 15:37 | |
*** sweston has quit IRC | 15:38 | |
*** saju_m has joined #openstack-neutron | 15:39 | |
*** dims has quit IRC | 15:41 | |
*** markwash has joined #openstack-neutron | 15:42 | |
*** dims has joined #openstack-neutron | 15:45 | |
*** markwash has quit IRC | 15:46 | |
*** afazekas has quit IRC | 15:47 | |
*** rotbeard has quit IRC | 15:58 | |
*** AlexF has joined #openstack-neutron | 15:59 | |
*** carl_baldwin has joined #openstack-neutron | 15:59 | |
*** yfried has quit IRC | 15:59 | |
*** MM_at_HP has quit IRC | 16:01 | |
*** AlexF has quit IRC | 16:01 | |
otherwiseguy | anteaya: do you know the status of the gate for the havana branches? Didn't know if I should try to recheck the polling minimization patches yet or not. | 16:02 |
*** dave_tucker is now known as dave_tucker_zzz | 16:05 | |
*** aymenfrikha has joined #openstack-neutron | 16:06 | |
*** matsuhashi has quit IRC | 16:07 | |
*** AlexF has joined #openstack-neutron | 16:08 | |
*** sweston has joined #openstack-neutron | 16:09 | |
anteaya | otherwiseguy: the status of the gate right now for everything is poor | 16:09 |
anteaya | nodepool was eating up swap this morning and we don't know why | 16:09 |
*** sweston is now known as sweston_ | 16:09 | |
otherwiseguy | anteaya: so a day like any other! :) | 16:09 |
anteaya | so the host rebooted the vm nodepool was on | 16:09 |
anteaya | which puts us in a bad spot | 16:09 |
otherwiseguy | indeed. | 16:09 |
anteaya | the weekend was good | 16:10 |
*** sbalukoff has quit IRC | 16:12 | |
*** terrylhowe has joined #openstack-neutron | 16:12 | |
*** networkstatic has joined #openstack-neutron | 16:17 | |
*** dave_tucker_zzz is now known as dave_tucker | 16:17 | |
*** thedodd has joined #openstack-neutron | 16:19 | |
*** bvandenh has quit IRC | 16:21 | |
openstackgerrit | Edgar Magana proposed a change to openstack/neutron: Implements provider network support in PLUMgrid plugin https://review.openstack.org/60383 | 16:21 |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Metadata agent caches networks for routers https://review.openstack.org/70689 | 16:22 |
*** ijw has joined #openstack-neutron | 16:23 | |
anteaya | otherwiseguy: jobs are running again though we do have a back log | 16:24 |
anteaya | as for status of the gate for just havana branches, I'm not sure what to look for to evaluate that | 16:24 |
anteaya | let me give it some thought | 16:24 |
*** ramishra_ has quit IRC | 16:25 | |
*** _gk_ has joined #openstack-neutron | 16:27 | |
openstackgerrit | Edgar Magana proposed a change to openstack/neutron: Implements provider network support in PLUMgrid plugin https://review.openstack.org/60383 | 16:28 |
*** ramishra has joined #openstack-neutron | 16:29 | |
*** arosen has joined #openstack-neutron | 16:32 | |
*** jistr has quit IRC | 16:33 | |
*** jistr has joined #openstack-neutron | 16:34 | |
anteaya | otherwiseguy: just looked at a bunch of links | 16:37 |
anteaya | started here: https://review.openstack.org/#/q/status:merged+project:openstack/neutron+branch:stable/havana,n,z | 16:37 |
anteaya | a patch merged to stable/havana yesterday | 16:37 |
*** mlavalle has joined #openstack-neutron | 16:37 | |
anteaya | then looking at job builds: https://jenkins03.openstack.org/job/gate-tempest-dsvm-neutron/ seems to be building fine | 16:38 |
anteaya | https://jenkins01.openstack.org/job/gate-tempest-dsvm-neutron-pg/ seems good | 16:38 |
*** thuc has joined #openstack-neutron | 16:38 | |
anteaya | ditto for: https://jenkins04.openstack.org/job/gate-tempest-dsvm-neutron-isolated/ | 16:39 |
*** thuc has quit IRC | 16:39 | |
anteaya | last successful build was 4 days ago for https://jenkins05.openstack.org/job/gate-tempest-dsvm-neutron-pg-isolated/ | 16:39 |
otherwiseguy | anteaya: ok. thanks. I know that mark was looking at something related to gate stability with havana last week. maybe things got resolved? I just know that the polling minimization patches seemed to take a couple of tries to get through a while ago. | 16:39 |
anteaya | working on a build now | 16:39 |
*** thuc has joined #openstack-neutron | 16:39 | |
anteaya | I haven't talked to mark in a few days | 16:40 |
anteaya | best get an update from him | 16:40 |
otherwiseguy | anteaya: ok. thanks. | 16:40 |
anteaya | np | 16:40 |
*** terrylhowe_ has joined #openstack-neutron | 16:40 | |
otherwiseguy | just wanted to see if we could get the patches in in time for build. :/ | 16:41 |
*** terrylhowe_ has left #openstack-neutron | 16:41 | |
*** jroovers has quit IRC | 16:42 | |
anteaya | yes a worthy goal | 16:42 |
*** markmcclain has joined #openstack-neutron | 16:43 | |
anteaya | speak of the devil | 16:43 |
*** jgallard has quit IRC | 16:46 | |
*** markmcclain1 has joined #openstack-neutron | 16:46 | |
* markmcclain1 reads scrollback | 16:47 | |
*** igor has quit IRC | 16:47 | |
*** AlexF has quit IRC | 16:47 | |
*** markmcclain has quit IRC | 16:47 | |
*** igor has joined #openstack-neutron | 16:47 | |
*** evgenyf has quit IRC | 16:50 | |
openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add OpenDaylight ML2 MechanismDriver https://review.openstack.org/69775 | 16:50 |
*** igor has quit IRC | 16:52 | |
*** _gk_ has quit IRC | 16:55 | |
markmcclain1 | so stable branch still is seeing several failures | 16:58 |
*** saju_m has quit IRC | 16:59 | |
anteaya | markmcclain1: are any of those failures trackable for those wanting to follow along at home? | 17:00 |
markmcclain1 | we still have to dig through the latest batch of failures | 17:00 |
markmcclain1 | and figure out if they're neutron items or caused by something else | 17:00 |
*** _gk_ has joined #openstack-neutron | 17:01 | |
*** dave_tucker is now known as dave_tucker_zzz | 17:01 | |
markmcclain1 | we're still not consistent +1s from the checks | 17:03 |
markmcclain1 | https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:stable/havana,n,z | 17:03 |
*** _gk_ has quit IRC | 17:03 | |
*** baoli has quit IRC | 17:04 | |
*** saju_m has joined #openstack-neutron | 17:04 | |
*** dguitarbite has joined #openstack-neutron | 17:04 | |
*** yamahata has joined #openstack-neutron | 17:04 | |
*** dguitarbite has left #openstack-neutron | 17:04 | |
*** baoli has joined #openstack-neutron | 17:04 | |
*** markwash has joined #openstack-neutron | 17:05 | |
*** yfried has joined #openstack-neutron | 17:06 | |
openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Metadata agent caches networks for routers https://review.openstack.org/70689 | 17:06 |
anteaya | markmcclain1: marun was telling me last week about some of the havana check jobs being noop | 17:07 |
anteaya | I'm not sure if I fully understood his point though | 17:07 |
anteaya | there is a large chance I didn't | 17:07 |
markmcclain1 | yeah they're def not a no-op if we're seeing -1s | 17:07 |
anteaya | that was what didn't make sense to me | 17:08 |
anteaya | my brain is still mid-foggy though so I haven't gone back to try to understand it | 17:08 |
*** yamahata has quit IRC | 17:08 | |
anteaya | okay, I'll leave it for now | 17:08 |
*** dave_tucker_zzz is now known as dave_tucker | 17:09 | |
*** yamahata has joined #openstack-neutron | 17:09 | |
*** markmcclain1 has quit IRC | 17:09 | |
*** networkstatic has quit IRC | 17:09 | |
*** markmcclain has joined #openstack-neutron | 17:10 | |
*** jlibosva has quit IRC | 17:11 | |
openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Add multiple RPC workers https://review.openstack.org/72425 | 17:12 |
*** rossella_s has quit IRC | 17:13 | |
*** dave_tucker is now known as dave_tucker_zzz | 17:16 | |
*** spandhe has joined #openstack-neutron | 17:16 | |
spandhe | Hi salv-orlando yt? | 17:17 |
*** itzikb has quit IRC | 17:17 | |
*** jroovers has joined #openstack-neutron | 17:17 | |
salv-orlando | ih spandhe but I need to reboot my laptop in 10 minutes as it's acting crazy | 17:17 |
openstackgerrit | Paul Ward proposed a change to openstack/neutron: Expose ML2 mechanism driver exceptions to the user https://review.openstack.org/69748 | 17:18 |
spandhe | salv-orlando: ok.. I have a question regarding http://lists.openstack.org/pipermail/openstack-dev/2013-June/010805.html.. we can talk once your laptop problem is resolved.. | 17:19 |
*** _gk_ has joined #openstack-neutron | 17:19 | |
*** saju_m has quit IRC | 17:19 | |
*** yfried has quit IRC | 17:21 | |
*** dave_tucker_zzz is now known as dave_tucker | 17:21 | |
openstackgerrit | Edgar Magana proposed a change to openstack/neutron: Implements provider network support in PLUMgrid plugin https://review.openstack.org/60383 | 17:23 |
*** bada_ has joined #openstack-neutron | 17:24 | |
*** bada has quit IRC | 17:25 | |
*** sweston_ has quit IRC | 17:25 | |
openstackgerrit | Paul Ward proposed a change to openstack/neutron: Expose ML2 mechanism driver exceptions to the user https://review.openstack.org/69748 | 17:25 |
*** dave_tucker is now known as dave_tucker_zzz | 17:26 | |
*** jpich has quit IRC | 17:26 | |
*** jroovers has quit IRC | 17:27 | |
*** salv-orlando has quit IRC | 17:27 | |
openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Developer documentation https://review.openstack.org/72428 | 17:29 |
*** salv-orlando has joined #openstack-neutron | 17:30 | |
*** sn6i23a has joined #openstack-neutron | 17:31 | |
*** sn6i23a has quit IRC | 17:31 | |
*** yfried has joined #openstack-neutron | 17:31 | |
*** sn6i23a has joined #openstack-neutron | 17:32 | |
*** saju_m has joined #openstack-neutron | 17:32 | |
salv-orlando | spandhe: I resurrected | 17:32 |
spandhe | salv-orlando: great! :) | 17:33 |
salv-orlando | so… about your question? | 17:33 |
spandhe | In my setup. I faced the same issue as mentioned in this thread: http://lists.openstack.org/pipermail/openstack-dev/2013-June/010799.html | 17:33 |
spandhe | I used the workaround as suggested, added sql_dbpool_enable = True in the ini file. | 17:34 |
*** jistr has quit IRC | 17:34 | |
spandhe | salv-orlando: What does that setting exactly achieve? | 17:34 |
spandhe | and after enabling that setting, that issue got resolved, but some other problems started to appear.. | 17:35 |
*** jistr has joined #openstack-neutron | 17:35 | |
salv-orlando | spandhe: That setting enables db connection pooling in the sqlalchemy backend. Depending on your plugin a few things might be sorted, but other might screw up | 17:35 |
salv-orlando | what plugin are you using? If using ml2, what drivers are you using? | 17:36 |
spandhe | i am using nvp plugin | 17:36 |
spandhe | the other issue that we observed after adding the setting was: if the system was kept idle for about 5 minutes, quantum lost connectivity to the database. OperationalError: (2006, 'MySQL server has gone away') | 17:38 |
spandhe | salv-orlando: ^ | 17:38 |
salv-orlando | We've never seen the latter issue; and no, don't use db pooling with the nvp plugin. | 17:39 |
salv-orlando | Because it will trigger out of order operations on the NVP backend | 17:39 |
*** AlexF has joined #openstack-neutron | 17:40 | |
spandhe | salv-orlando: ok.. then whats the solution for allowing concurrent create_ports? | 17:40 |
salv-orlando | The NVP (now NSX) plugin is being progressively improved to support concurrent operations as well asynchronous backend | 17:40 |
salv-orlando | one common way however of deploying it so far is to replicate the server process and put it behind a load balancer | 17:41 |
*** dave_tucker_zzz is now known as dave_tucker | 17:42 | |
*** markmcclain has quit IRC | 17:42 | |
spandhe | salv-orlando: thanks a lot! Is that the only solution? I will look into it. | 17:42 |
salv-orlando | up to havana yes | 17:42 |
salv-orlando | We've been trying db pooling internally, but created a lot of problems with the dhcp agent | 17:43 |
carl_baldwin | otherwiseguy: ping | 17:43 |
otherwiseguy | carl_baldwin: pong | 17:43 |
salv-orlando | spandhe: I'm not sure if that the same problem you're seeing, but we'll be happy to improve concurrency also for pre-icehouse releases | 17:44 |
carl_baldwin | I just noticed your review to add multiple process rpc. I just started hacking one out myself last week. | 17:44 |
carl_baldwin | I did not post anything for review yet, though. | 17:45 |
otherwiseguy | carl_baldwin: Mine is really simple, I've tested it and it appears to work, but there may be something missing that I didn't think of. | 17:45 |
spandhe | salv-orlando: we are not using dhcp, as far as I know.. and the "SQL gone" error happens on any simple command such as "nova list".. but thats if the system is idle for about 5 minutes. | 17:46 |
otherwiseguy | It's 10 lines of code, so ... | 17:46 |
otherwiseguy | I was trying to just get up to speed on how it all fit together for most of the time. | 17:46 |
*** ygbo has quit IRC | 17:46 | |
otherwiseguy | Is your implementation appreciably different? | 17:47 |
carl_baldwin | otherwiseguy: I'll put some comments in to the review. Mainly, I was struggling with the fact that each plugin starts RPC as part of its own init. | 17:47 |
carl_baldwin | otherwiseguy: I have not finished looking over yours. I just didn't want to wait to get in touch with you on it. | 17:47 |
otherwiseguy | carl_baldwin: Yeah. I basically just swoop in and set up the RPC workers first. But because of how the plugin is instantiated, unlike the api workers patch, the parent process will end up with an rpc worker as well with my patch. | 17:48 |
*** _gk_ has quit IRC | 17:48 | |
otherwiseguy | carl_baldwin: blueprint spec here: https://docs.google.com/document/d/1LYgoARbgFrdMNte1ki1aC-pXRk_r0ZsqnEop9BedM10/edit# | 17:49 |
carl_baldwin | Mine followed the pattern of the multiple api workers patch. There is a little more complexity in it. I was taking care of a few more of those loose ends. | 17:49 |
*** jistr has quit IRC | 17:50 | |
openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Developer documentation https://review.openstack.org/72428 | 17:50 |
*** safchain has quit IRC | 17:50 | |
otherwiseguy | I figured it didn't really matter if the RPC worker ran in the main process or not. Thought about just passing in rpc_workers - 1 to the ProcessLauncher stuff so the number came out right. Figured it would be good to just post something up for discussion. | 17:51 |
carl_baldwin | otherwiseguy: I'll have a more thorough look at yours. I think I can help you fix some of those loose ends. | 17:51 |
*** dave_tucker is now known as dave_tucker_zzz | 17:51 | |
otherwiseguy | carl_baldwin: Cool, thanks. I was pretty sure there had to be several lurking in there to tie up. :) | 17:52 |
*** alagalah has quit IRC | 17:52 | |
salv-orlando | spandhe: does the sql gone away error occur only with sql_db_pool enabled? | 17:53 |
salv-orlando | on the other hand, another big difference is: are you running grizzly or havana | 17:53 |
carl_baldwin | otherwiseguy: I can put what I have up as a draft for you to look at for comparison. I'll find some time to clean it up and put it up today. | 17:53 |
spandhe | salv-orlando: yes. | 17:55 |
salv-orlando | spandhe: thanks. Grizzly or havana? | 17:55 |
*** bjornar has joined #openstack-neutron | 17:55 | |
otherwiseguy | carl_baldwin: thanks! | 17:56 |
carl_baldwin | otherwiseguy: thanks, I look forward to working with you. | 17:56 |
spandhe | salv-orlando: Grizzly.. | 17:56 |
otherwiseguy | carl_baldwin: same here. I'm sure I will reap the most benefit out of that. ;) | 17:57 |
*** igor_ has joined #openstack-neutron | 17:58 | |
*** harlowja_away is now known as harlowja | 17:58 | |
salv-orlando | spandhe: cool. So what might be causing your mysql gone away problem is a pool exhaustion with no connection recycling. | 17:59 |
salv-orlando | This is actually caused by a nasty behaviour on the neutron authZ checks which trigger a DB operation for each item in a list operation | 17:59 |
salv-orlando | when you get to some scale (say 1,000 ports) | 17:59 |
salv-orlando | and you run commands such as nova-list, you start having problems. | 18:00 |
spandhe | salv-orlando: my scale is actually way low.. 5 ports or so.. | 18:00 |
salv-orlando | This issue was fixed in havana, but the fix was too large for being considered for backport | 18:00 |
salv-orlando | spandhe: well, that's interesting. You should hardly exhaust the pool with so little connections. | 18:01 |
*** jroovers has joined #openstack-neutron | 18:01 | |
*** carl_baldwin has quit IRC | 18:01 | |
spandhe | salv-orlando: I have default pool size,, I think the default is 5.. | 18:01 |
*** igor_ has quit IRC | 18:02 | |
salv-orlando | spandhe: yup but then again, sqlalchemy also adds overflow connections in the pool. Also, as you witness this only when the neutron server has been idle, this makes the problem even stranger. | 18:02 |
*** carl_baldwin has joined #openstack-neutron | 18:04 | |
salv-orlando | I think there was some history around the db pool mechanisms used in Grizzly. Basically we ended up moving to something else for Havana, as stated in the discussion thread you were reading. | 18:04 |
*** markmcclain has joined #openstack-neutron | 18:04 | |
*** clev has quit IRC | 18:04 | |
spandhe | salv-orlando: yes.. should I try increasing sqlalchemy_pool_size? | 18:04 |
salv-orlando | spandhe: that helps but does not solve the issue. | 18:04 |
spandhe | Or you think the issue probably wont be seen in Havana? | 18:04 |
salv-orlando | spandhe: in my opinion this was a problem with Grizzly. But I am unable to confirm it now. | 18:05 |
*** ijw has quit IRC | 18:05 | |
spandhe | salv-orlando: ok.. thanks! | 18:07 |
salv-orlando | spandhe: np, I'm here if you need anything else. | 18:08 |
*** ijw has joined #openstack-neutron | 18:10 | |
*** saju_m has quit IRC | 18:12 | |
spandhe | salv-orlando: but in general, you wouldnt recommend using dbpool for NSX plugin, right? | 18:13 |
marun | anteaya: I'm not sure I claimed the havana jobs were noop. | 18:14 |
salv-orlando | spandhe: In general not. If you are hitting performance problems without it using grizzly | 18:14 |
salv-orlando | spandhe: ping me and we'll see how that can be sorted together | 18:14 |
spandhe | salv-orlando: sure.. thanks! | 18:15 |
*** jp_at_hp has quit IRC | 18:16 | |
*** suresh12 has joined #openstack-neutron | 18:26 | |
markmcclain | beagles, carl_baldwin, sc68cal, mlavalle, mestery, rkukura: don't forget to update the agenda | 18:28 |
rkukura | ok | 18:28 |
*** saju_m has joined #openstack-neutron | 18:29 | |
carl_baldwin | ok | 18:29 |
*** zzelle has joined #openstack-neutron | 18:30 | |
beagles | ack | 18:30 |
*** alexpilotti has joined #openstack-neutron | 18:31 | |
*** sweston has joined #openstack-neutron | 18:35 | |
*** rwsu has quit IRC | 18:36 | |
openstackgerrit | Terry Wilson proposed a change to openstack/neutron: Remove psutil dependency https://review.openstack.org/68993 | 18:40 |
mlavalle | markmcclain: I'll do it over the next 90 minutes or so | 18:41 |
mlavalle | :-) | 18:41 |
markmcclain | rkukura, carl_baldwin, beagles, mlavalle: thanks | 18:41 |
*** ijw_ has joined #openstack-neutron | 18:41 | |
*** Sukhdev has joined #openstack-neutron | 18:43 | |
*** ijw has quit IRC | 18:44 | |
*** zigo has quit IRC | 18:45 | |
mestery | markmcclain: Ack on that | 18:46 |
*** haleyb has quit IRC | 18:46 | |
*** zigo has joined #openstack-neutron | 18:49 | |
*** rwsu has joined #openstack-neutron | 18:51 | |
*** ijw_ has quit IRC | 18:51 | |
*** saju_m has quit IRC | 18:51 | |
*** saju_m has joined #openstack-neutron | 18:52 | |
*** morganfainberg_Z is now known as morganfainberg | 18:53 | |
*** sweston has quit IRC | 18:56 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BigSwitch: Fix tenant_id for shared net requests https://review.openstack.org/72450 | 18:57 |
*** sbalukoff has joined #openstack-neutron | 18:57 | |
*** saju_m has quit IRC | 18:58 | |
*** otherwiseguy has quit IRC | 18:59 | |
*** markmcclain has quit IRC | 19:01 | |
*** markmcclain has joined #openstack-neutron | 19:02 | |
*** haleyb has joined #openstack-neutron | 19:02 | |
*** clev has joined #openstack-neutron | 19:03 | |
*** ijw has joined #openstack-neutron | 19:03 | |
*** AlexF has quit IRC | 19:04 | |
*** carl_baldwin has quit IRC | 19:11 | |
*** ramishra has quit IRC | 19:14 | |
openstackgerrit | Robert Kukura proposed a change to openstack/neutron: Replace binding:capabilities with binding:vif_details https://review.openstack.org/72452 | 19:14 |
*** ramishra has joined #openstack-neutron | 19:14 | |
*** beagles is now known as beagles_brb | 19:26 | |
openstackgerrit | Luis A. Garcia proposed a change to openstack/neutron: Re-enable lazy translation https://review.openstack.org/66977 | 19:28 |
openstackgerrit | Luis A. Garcia proposed a change to openstack/neutron: Do not append to messages with + https://review.openstack.org/66980 | 19:28 |
*** djoreilly has quit IRC | 19:30 | |
*** thedodd has quit IRC | 19:30 | |
*** markmcclain has quit IRC | 19:38 | |
*** saju_m has joined #openstack-neutron | 19:38 | |
*** sweston has joined #openstack-neutron | 19:40 | |
*** ijw has quit IRC | 19:41 | |
*** godara has joined #openstack-neutron | 19:41 | |
*** ijw has joined #openstack-neutron | 19:41 | |
*** otherwiseguy has joined #openstack-neutron | 19:42 | |
*** _gk_ has joined #openstack-neutron | 19:45 | |
*** ijw has quit IRC | 19:46 | |
*** thuc has quit IRC | 19:46 | |
*** ramishra has quit IRC | 19:46 | |
*** thuc has joined #openstack-neutron | 19:46 | |
anteaya | marun: it is entirely possible I mis-understood | 19:47 |
*** zigo has quit IRC | 19:48 | |
*** _gk_ has quit IRC | 19:50 | |
*** thuc has quit IRC | 19:50 | |
openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Add migration support from agent to NSX dhcp/metadata services https://review.openstack.org/51325 | 19:54 |
*** markmcclain has joined #openstack-neutron | 19:58 | |
*** saju_m has quit IRC | 20:01 | |
*** openstack has joined #openstack-neutron | 21:33 | |
-dickson.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 21:33 | |
openstackgerrit | A change was merged to openstack/neutron: Improve handling of security group updates https://review.openstack.org/63100 | 21:38 |
*** alagalah has quit IRC | 21:38 | |
*** ramishra has joined #openstack-neutron | 21:48 | |
*** baoli has quit IRC | 21:51 | |
*** ramishra has quit IRC | 21:52 | |
openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: apply agent scheduling for loadbalancer instance https://review.openstack.org/71472 | 21:53 |
*** aymenfrikha1 has joined #openstack-neutron | 21:55 | |
*** aymenfrikha has quit IRC | 21:55 | |
*** RajeshMohan has quit IRC | 22:02 | |
*** carl_baldwin has quit IRC | 22:02 | |
enikanorov | markmcclain: hey, got some time to discuss lb instance? | 22:02 |
markmcclain | right now.. I've got to return to meetings | 22:03 |
markmcclain | I figured we can discuss on Thursday | 22:03 |
enikanorov | markmcclain: ok, i remember you've sent me an email on that | 22:03 |
marun | arosen: ping | 22:03 |
enikanorov | markmcclain: and i've replied with quite detailed explanation. if you could look at it, we could discuss based on that | 22:04 |
beagles | sweston: ping | 22:04 |
*** RajeshMohan has joined #openstack-neutron | 22:04 | |
markmcclain | ok | 22:04 |
sweston | beagles: pong | 22:04 |
enikanorov | thanks | 22:04 |
*** carl_baldwin has joined #openstack-neutron | 22:04 | |
sweston | beagles: I have to prepare for another meeting in an hour, what is your availability later today, or tomorrow? | 22:05 |
beagles | sweston: so the things I was referring to in the meeting are referred to here https://wiki.openstack.org/wiki/NovaNeutronGapHighlights | 22:05 |
yamahata | enikanorov, ping | 22:05 |
enikanorov | yamahata: pong | 22:05 |
yamahata | enikanorov, can we chat a bit on servicevm? | 22:05 |
arosen | marun: pong | 22:05 |
enikanorov | yamahata: yeah, but i'll be halp asleep :) | 22:05 |
enikanorov | *half | 22:05 |
marun | arosen: I have a question regarding nova/neutron interaction... | 22:06 |
yamahata | enikanorov, sorry. I'll be quick. | 22:06 |
*** jecarey_ has quit IRC | 22:06 | |
marun | arosen: I noticed recently that nova network does synchronous network provisioning. | 22:06 |
*** jecarey_ has joined #openstack-neutron | 22:06 | |
yamahata | enikanorov, per your review, you're claiming to change both REST API and data base model? | 22:06 |
sweston | beagles: ok, I will take a look at it, but don't have time now :-(. May we chat later? | 22:07 |
beagles | sweston, I'm pretty much done for today here (TZ is -330 GMT). Tuesdays are pretty rough sometimes.. I've back to back meetings from 9:00 till 12:30 and then another at 14:30 (my TZ) I'll ping you iin there somewhere | 22:07 |
yamahata | enikanorov, hosting_device -> device, logical_service_instance -> service_instance | 22:07 |
enikanorov | yamahata: are you talking about 'servicetypes'? | 22:07 |
marun | arosen: I found myself wondering if nova shouldn't be checking in with neutron to ensure active port status before booting a vm | 22:07 |
enikanorov | yamahata: i think some of the names may be shortened, expecially the attribute names | 22:07 |
*** ywu has joined #openstack-neutron | 22:07 | |
enikanorov | *especially | 22:08 |
marun | arosen: to maximize the chances of cloud-init/dhcp having connectivity to succeed | 22:08 |
yamahata | enikanorov, Ok. I'll rename it. And servicetypes. | 22:08 |
*** ramishra has joined #openstack-neutron | 22:08 | |
enikanorov | so do you think storing service types as a list of strings makes sense? | 22:08 |
yamahata | enikanorov, yes. | 22:09 |
marun | arosen: this could avoid the problem of instance boot racing with network configuration when neutron is used | 22:09 |
enikanorov | ok, good that we agree on that | 22:09 |
sweston | beagles: ok, I will be available. that will give me some time to read over what you want in the document and think about it before tomorrow. | 22:09 |
marun | arosen: what do you think? | 22:09 |
yamahata | enikanorov, thanks. | 22:09 |
*** WackoRobie has quit IRC | 22:09 | |
*** salv-orlando_ has joined #openstack-neutron | 22:10 | |
arosen | marun: it sounds logical though i think it would probably make things more complex. The cloud-init stuff does a ton of retry so I'm not sure if it would really help | 22:11 |
openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Add L3 VRRP HA base classes https://review.openstack.org/64553 | 22:11 |
marun | arosen: we have persistent problems with dhcp, though | 22:11 |
marun | arosen: and consider the case of containers | 22:11 |
marun | arosen: they will always win a race with neutron | 22:11 |
marun | arosen: I would suggest that the lack of nova coordinating directly with neutron is why we see so many non-deterministic failures. | 22:12 |
arosen | marun: i like the idea. It would be interesting to see if it would help. What do we do if the port never becomes active? | 22:12 |
marun | arosen: if nova were checking with neutron, we would at least have a clear indication of failure. | 22:12 |
*** _gk_ has quit IRC | 22:13 | |
*** ramishra has quit IRC | 22:13 | |
marun | arosen: there would have to be a timeout | 22:13 |
arosen | then i guess put the vm in error state | 22:13 |
marun | arosen: and then vm boot could be failed | 22:13 |
arosen | yea, i think it makes sense. | 22:13 |
marun | arosen: yes, exactly | 22:14 |
*** salv-orlando has quit IRC | 22:14 | |
*** salv-orlando_ is now known as salv-orlando | 22:14 | |
*** _gk_ has joined #openstack-neutron | 22:14 | |
marun | arosen: the port being active won't guarantee success, of course, since it provides no guarantees of dhcp being configured | 22:15 |
marun | arosen: but I think it's a good starting point. | 22:15 |
*** _gk_ has quit IRC | 22:15 | |
marun | arosen: Do you think it would be feasible to submit something by next week? I'm happy to do the work if you don't have time, though my ignorance of nova will mean I will take longer. | 22:16 |
marun | arosen: I don't think it should be the default, but I think it would be valuable to get it out there for people to test with this cycle. | 22:17 |
lifeless | marun: dhcp problems ? | 22:19 |
marun | lifeless: :( | 22:19 |
arosen | marun: give me a few sec i want to check where the call is that puts the external_id in ovsdb | 22:19 |
marun | lifeless: regarding eventual consistency I presume? | 22:20 |
arosen | depending on where that is this sounds pretty straightforward to add. | 22:20 |
lifeless | marun: you said above we have lots of dhcp problems | 22:20 |
lifeless | marun: I'm wondering how many are really just https://bugs.launchpad.net/neutron/+bug/1271344 | 22:20 |
marun | lifeless: ah, sorry. | 22:20 |
marun | lifeless: i think that's a different problem | 22:20 |
lifeless | marun: I've seen two DHCP problems recently. That one, and the br-int ofctl flows being bong for some unknown reason | 22:21 |
lifeless | which is fixed by neutron-openvswitch-agent restarts. | 22:21 |
marun | lifeless: better nova/neutron coordination is meant to solve the problem of l2 and dhcp agent racing nova boot and losing when a system is overloaded | 22:21 |
lifeless | marun: oh right, sure, I know the one. | 22:22 |
*** jlibosva has quit IRC | 22:23 | |
*** peristeri has quit IRC | 22:25 | |
enikanorov | rkukura: hi, a quick question, is it possible to use gre/vxlan networks and vlan networks simultaneously with ml2 plugin? | 22:29 |
openstackgerrit | A change was merged to openstack/neutron: Fix race condition in network scheduling to dhcp agent https://review.openstack.org/71393 | 22:29 |
*** tongli has quit IRC | 22:34 | |
rkukura | enikanorov: yes | 22:34 |
*** arosen has quit IRC | 22:35 | |
rkukura | enikanorov: Combining tunnel-based tenant networks with vlan and flat provider networks is a common approach | 22:35 |
*** arosen has joined #openstack-neutron | 22:35 | |
rkukura | With ml2, you can also list multiple tenant_network_types, so you could have a set of available VLAN tenant networks, and if these are exhausted, allocate GRE or VXLAN tenant networks instead. | 22:36 |
rkukura | Also, if you change tenant_network_types (i.e. to switch from GRE to VXLAN), existing tunnel networks of the old type should continue to work. | 22:37 |
*** bjornar has quit IRC | 22:39 | |
*** salv-orlando_ has joined #openstack-neutron | 22:39 | |
arosen | lifeless: i think i know why you are hitting that dhcp issue | 22:40 |
arosen | I commented on the launchpad issue. | 22:40 |
*** gdubreui has joined #openstack-neutron | 22:42 | |
lifeless | arosen: dhcp_release is meant to be called ? | 22:42 |
arosen | lifeless: it's getting called. | 22:42 |
arosen | I think the packet is getting blocked. | 22:43 |
lifeless | arosen: the packet from the neutron node to the dnsmasq on the neutron node? | 22:43 |
arosen | The gate was running for like a month without security groups working which is why it was working. | 22:43 |
arosen | lifeless: i believe so. Should be sure in a sec. | 22:43 |
*** salv-orlando has quit IRC | 22:43 | |
*** salv-orlando_ is now known as salv-orlando | 22:43 | |
lifeless | arosen: I'm clearly confused. I didn't realise the netns for dhcp would have security groups applied to it. | 22:44 |
lifeless | arosen: since its not a VM | 22:44 |
arosen | lifeless: there are iptable rules there though saying a vm can only receive traffic from other vms and dhcp_offers | 22:44 |
arosen | I think the release might be getting blocked there. | 22:45 |
lifeless | arosen: but those rules are on an entirely different machine | 22:45 |
lifeless | http://paste.ubuntu.com/6911812/ | 22:45 |
arosen | the l2-agent runs on the same machine as the dhcp-agent though which is what applies those iptable rules. | 22:45 |
lifeless | iptables-save from the undercloud machine where clint just saw this | 22:45 |
lifeless | arosen: Except it has no vifs to apply them too | 22:47 |
*** armax has quit IRC | 22:47 | |
lifeless | arosen: see the pastebins[6~ | 22:47 |
lifeless | oh, ip netns exec qdhcp-e85b44c7-1136-4217-954e-cdf0acdddfe1 iptables-save | pastebinit | 22:47 |
lifeless | http://paste.ubuntu.com/6911818/ | 22:47 |
lifeless | is the second one | 22:47 |
lifeless | for the netns that the dhcp_release call will be made from | 22:47 |
*** clev has quit IRC | 22:47 | |
arosen | yup | 22:48 |
*** jecarey_ has quit IRC | 22:50 | |
zzelle | hi, a small question, why ip veth are used to connect br-int to physnets for vlans and ovs patchs for tunnels ? | 22:51 |
openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Add migration support from agent to NSX dhcp/metadata services https://review.openstack.org/51325 | 22:51 |
lifeless | arosen: I just ran it by hand getting | 22:51 |
lifeless | Feb 10 22:51:10 undercloud-notcompute-jws3awlsb2kh dnsmasq-dhcp[26029]: DHCPRELEASE(tap56efc4c6-55) 10.10.16.141 78:e7:d1:23:97:cd | 22:51 |
lifeless | in syslog | 22:51 |
arosen | lifeless: I ran it by hand and it didn't seem to fix the issue for me. | 22:52 |
arosen | Did it fix it for you? | 22:52 |
lifeless | arosen: we have restarted dnsmasq already so I can't confirm or deny | 22:53 |
lifeless | arosen: but it logged that it saw it | 22:53 |
lifeless | arosen: aee though that the earlier releases say 'lease unknown' which is ... interesting | 22:53 |
*** Sukhdev has quit IRC | 22:54 | |
*** armax has joined #openstack-neutron | 22:54 | |
lifeless | arosen: btw that server is running 7918ff71079239ffab9c16a6c65be3a876cbbdb0 | 22:55 |
arosen | lifeless: hrm I was able to reproduce this a few min ago but now i'm having trouble getting this to occur again | 22:55 |
lifeless | arosen: because I had to monkey that on | 22:55 |
lifeless | arosen: first mainline commit is 44649bd7c9bbf5ce27ffc4162980afb02ae5bdf5 which is feb 6th, so quite recent | 22:55 |
*** zzelle has quit IRC | 22:57 | |
openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BigSwitch: Fix tenant_id for shared net requests https://review.openstack.org/72450 | 22:57 |
*** nati_uen_ has quit IRC | 23:01 | |
*** sweston has quit IRC | 23:02 | |
*** packet has quit IRC | 23:02 | |
*** banix has quit IRC | 23:04 | |
*** alagalah has joined #openstack-neutron | 23:05 | |
*** ramishra has joined #openstack-neutron | 23:09 | |
*** dave_tucker_zzz is now known as dave_tucker | 23:10 | |
*** jecarey has joined #openstack-neutron | 23:11 | |
openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Rename L2 Switch/Gateway related methods for VMware NSX plugin https://review.openstack.org/67353 | 23:11 |
openstackgerrit | Amir Sadoughi proposed a change to openstack/neutron: Assign local VLANs before port filters https://review.openstack.org/65557 | 23:12 |
*** thedodd has quit IRC | 23:13 | |
*** ramishra has quit IRC | 23:13 | |
*** alexpilotti has quit IRC | 23:13 | |
*** nati_ueno has joined #openstack-neutron | 23:17 | |
*** jgrimm has quit IRC | 23:19 | |
*** AramakiHiroki has joined #openstack-neutron | 23:19 | |
*** nati_ueno has quit IRC | 23:23 | |
arosen | marun: One change that will need to be made for what you proposed is there is a call to: self.driver.plug_vifs(instance, net_info) in compute/manager.py that puts the neutron port-id in ovsdb that the l2-agent looks for needs to be moved into neutronv2/api.py i think | 23:28 |
marun | arosen: I'm not sure I follow | 23:28 |
*** armax has quit IRC | 23:29 | |
arosen | so the l2-agent looks in ovsdb for the neutron port-id | 23:29 |
marun | arosen: right | 23:29 |
arosen | that doesn't get inserted into ovsdb until after we've exited out of the neutronv2/api.py code. | 23:30 |
*** ramishra has joined #openstack-neutron | 23:30 | |
arosen | I was thinking that the sleep/retry logic on port status would be done in the neutronv2/api.py code. What were you thinking? | 23:30 |
*** rms_13 has joined #openstack-neutron | 23:30 | |
marun | arosen: It sounds like a chicken/egg issue though. | 23:31 |
arosen | I think the best place to do the sleep/retry logic would be in that file. I was just saying we'd need to move that line so that we insert the id in ovsdb before we start polling. | 23:32 |
marun | arosen: How can we insert that id until vif plug? | 23:32 |
marun | arosen: I mean, how do we even know which host to target until vif plug? | 23:32 |
marun | arosen: My thought was the sync point has to be post-vif plug. | 23:32 |
arosen | The vif.plug() is run on the compute host. | 23:32 |
arosen | marun: k, that works too. Though we'd need to add a call to neutronv2/api.py to do the polling somewhere in the compute manager. | 23:33 |
marun | arosen: yeah, that's where I'm a bit fuzzy. | 23:33 |
arosen | I think that's kind of odd to do because nova-network doesn't need this call that's why we should just sleep retry in allocate_for_instance in neutronv2/api.py | 23:33 |
marun | arosen: well, nova network does thing synchronously though | 23:34 |
*** mtreinish has quit IRC | 23:34 | |
marun | arosen: with neutron, nova calls to provision port (trigger async provisioning0, then schedules and boots vm | 23:35 |
arosen | marun: what do you mean? | 23:35 |
*** jroovers has quit IRC | 23:35 | |
arosen | why do you say that's synchronous? | 23:35 |
marun | arosen: I mean, nova network performs network provisioning via a call(), and does not proceed on error | 23:35 |
marun | arosen: I should dig that up | 23:35 |
*** mtreinish has joined #openstack-neutron | 23:39 | |
marun | arosen: While I'm looking, I'm still confused as to where else you could poll for port status other than after vif plug | 23:39 |
marun | arosen: what am I missing? | 23:40 |
arosen | marun: right I think we'd need to either add a call to the compute manager to neutronv2/api.py to do the polling | 23:41 |
arosen | or i think move the vif.plug() call | 23:41 |
*** mlavalle has quit IRC | 23:46 | |
marun | arosen: where would you move the vif plug call to? | 23:47 |
marun | arosen: as to nova network being synchronous... | 23:47 |
marun | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1046 | 23:48 |
marun | The underlying call is 'async' only in that it doesn't block until access to the result is attempted | 23:48 |
marun | But this access will be performed at some point during the booting of an instance | 23:48 |
marun | so there is essentially a sync point that happens before instance boot that ensures that, say, dhcp will be available for the instance's ip | 23:49 |
*** ijw_ has joined #openstack-neutron | 23:50 | |
*** markwash has quit IRC | 23:51 | |
marun | arosen: As to the options you suggest, I understand the need to add a call to the neutron api to poll for port status | 23:53 |
*** markmcclain has quit IRC | 23:53 | |
*** ijw has quit IRC | 23:53 | |
marun | arosen: I'm still unclear on how to move the vif plug call. | 23:53 |
marun | arosen: I apologize if I'm missing something obvious. | 23:54 |
*** AramakiHiroki has quit IRC | 23:54 | |
*** arosen has quit IRC | 23:54 | |
*** sweston has joined #openstack-neutron | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!