*** seizadi has joined #openstack-neutron | 00:00 | |
*** aranjan has quit IRC | 00:01 | |
*** afazekas has quit IRC | 00:01 | |
*** moha_hunt has quit IRC | 00:02 | |
*** suresh12 has joined #openstack-neutron | 00:03 | |
*** markvoelker has joined #openstack-neutron | 00:04 | |
*** luqas__ has joined #openstack-neutron | 00:07 | |
*** manishg has quit IRC | 00:08 | |
*** manishg_ has joined #openstack-neutron | 00:08 | |
*** markvoelker has quit IRC | 00:09 | |
*** luqas has quit IRC | 00:09 | |
*** leenheer has quit IRC | 00:12 | |
HenryG | otherwiseguy: ping | 00:13 |
---|---|---|
*** sbalukoff has quit IRC | 00:13 | |
*** mlavalle has quit IRC | 00:15 | |
*** seizadi has quit IRC | 00:16 | |
*** baoli has joined #openstack-neutron | 00:19 | |
HenryG | otherwiseguy: Left you a question here: https://review.openstack.org/155906 | 00:21 |
*** nlahouti has quit IRC | 00:23 | |
*** nlahouti has joined #openstack-neutron | 00:25 | |
*** nlahouti has quit IRC | 00:26 | |
*** Longgeek has joined #openstack-neutron | 00:27 | |
*** dims_ has quit IRC | 00:27 | |
openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 00:28 |
*** sbalukoff has joined #openstack-neutron | 00:29 | |
*** dims__ has joined #openstack-neutron | 00:30 | |
*** nlahouti has joined #openstack-neutron | 00:30 | |
*** manishg_ has quit IRC | 00:35 | |
openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron: Enable to apply policies to resources with special plural https://review.openstack.org/145998 | 00:40 |
*** ramishra_ has joined #openstack-neutron | 00:44 | |
*** leenheer has joined #openstack-neutron | 00:44 | |
*** leenheer has quit IRC | 00:45 | |
*** vinsh_ has quit IRC | 00:45 | |
*** tomoe_ has joined #openstack-neutron | 00:45 | |
*** tomoe__ has joined #openstack-neutron | 00:46 | |
*** zhhuabj has quit IRC | 00:47 | |
*** zhhuabj has joined #openstack-neutron | 00:48 | |
*** tomoe_ has quit IRC | 00:50 | |
*** manishg has joined #openstack-neutron | 00:51 | |
*** pavel_bondar has quit IRC | 00:52 | |
*** SridharRamaswamy has joined #openstack-neutron | 00:52 | |
*** SridharRamaswamy has quit IRC | 00:52 | |
*** pavel_bondar has joined #openstack-neutron | 00:52 | |
*** SridharRamaswamy has joined #openstack-neutron | 00:52 | |
*** aranjan has joined #openstack-neutron | 00:54 | |
*** SridharRamaswamy has quit IRC | 00:57 | |
*** yamamoto has quit IRC | 00:57 | |
*** leenheer has joined #openstack-neutron | 01:00 | |
*** Longgeek has quit IRC | 01:00 | |
*** ramishra_ has quit IRC | 01:02 | |
*** seizadi has joined #openstack-neutron | 01:02 | |
*** ramishra_ has joined #openstack-neutron | 01:05 | |
*** ramishra_ has quit IRC | 01:05 | |
*** markvoelker has joined #openstack-neutron | 01:05 | |
*** ramishra_ has joined #openstack-neutron | 01:06 | |
*** ramishra_ has quit IRC | 01:06 | |
otherwiseguy | HenryG: pong | 01:07 |
* otherwiseguy reads the question | 01:07 | |
*** ramishra_ has joined #openstack-neutron | 01:07 | |
* HenryG waits for response | 01:08 | |
*** ramishra_ has quit IRC | 01:08 | |
*** markvoelker has quit IRC | 01:11 | |
otherwiseguy | HenryG: response left on patch. | 01:11 |
*** seizadi has quit IRC | 01:11 | |
*** vinsh has joined #openstack-neutron | 01:11 | |
*** ramishra_ has joined #openstack-neutron | 01:12 | |
HenryG | otherwiseguy: Got it, thanks. So the commit msg isn't *quite* true. ;) | 01:13 |
otherwiseguy | HenryG: Would really like to get the patches that have passed all tests merged as soon as possible. 15 changes w/ 111 files +724/-1124 gets kind of hard to maintain. | 01:14 |
HenryG | otherwiseguy: I'll help them along | 01:14 |
otherwiseguy | HenryG: That makes you a wonderful person in my book. | 01:15 |
HenryG | otherwiseguy: I just want to make sure markmcclain and amotoki are OK with the first one, https://review.openstack.org/153067 | 01:16 |
*** jamielennox is now known as jamielennox|away | 01:16 | |
HenryG | otherwiseguy: Once that is OK'd we can open the sluice gates | 01:16 |
otherwiseguy | HenryG: If they aren't, I may have to give up. :) Changing it at this point means editing 111 files. :) | 01:16 |
otherwiseguy | And by "give up" I mean, cry, drink heavily for a bit, and begin again. | 01:17 |
*** banix has quit IRC | 01:19 | |
*** zzzeek has quit IRC | 01:21 | |
*** signed8bit is now known as signed8bit_ZZZzz | 01:22 | |
*** zzzeek has joined #openstack-neutron | 01:23 | |
*** seizadi has joined #openstack-neutron | 01:23 | |
*** banix has joined #openstack-neutron | 01:23 | |
*** addnull has left #openstack-neutron | 01:25 | |
*** manishg has quit IRC | 01:26 | |
*** baoli has quit IRC | 01:27 | |
*** nlahouti has quit IRC | 01:27 | |
*** a_le has quit IRC | 01:28 | |
*** Longgeek has joined #openstack-neutron | 01:28 | |
*** Longgeek has quit IRC | 01:28 | |
*** nlahouti has joined #openstack-neutron | 01:28 | |
*** nlahouti has quit IRC | 01:29 | |
*** Longgeek has joined #openstack-neutron | 01:30 | |
*** jamielennox|away is now known as jamielennox | 01:31 | |
*** tiswanso has quit IRC | 01:34 | |
*** xuhanp has joined #openstack-neutron | 01:35 | |
*** a_le has joined #openstack-neutron | 01:36 | |
*** nlahouti has joined #openstack-neutron | 01:36 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Fix the api job https://review.openstack.org/156436 | 01:36 |
*** zzzeek has quit IRC | 01:39 | |
*** yamamoto_ has joined #openstack-neutron | 01:40 | |
*** shwetaap has joined #openstack-neutron | 01:40 | |
*** jamielennox is now known as jamielennox|away | 01:41 | |
*** avozza is now known as zz_avozza | 01:56 | |
*** apuimedo has quit IRC | 02:00 | |
*** zhenq has joined #openstack-neutron | 02:02 | |
*** kickinz1 is now known as kickinz1|afk | 02:04 | |
*** trad511 has quit IRC | 02:06 | |
*** nlahouti has quit IRC | 02:07 | |
*** markvoelker has joined #openstack-neutron | 02:08 | |
HenryG | otherwiseguy: another question on https://review.openstack.org/155101 | 02:08 |
*** _cjones_ has quit IRC | 02:09 | |
*** jamielennox|away is now known as jamielennox | 02:09 | |
*** shwetaap1 has joined #openstack-neutron | 02:09 | |
*** shwetaap has quit IRC | 02:12 | |
*** markvoelker has quit IRC | 02:13 | |
*** manishg has joined #openstack-neutron | 02:13 | |
*** tiswanso has joined #openstack-neutron | 02:13 | |
*** salv-orlando has quit IRC | 02:13 | |
*** pal has joined #openstack-neutron | 02:13 | |
*** vinsh has quit IRC | 02:17 | |
*** manishg has quit IRC | 02:18 | |
*** manishg has joined #openstack-neutron | 02:18 | |
*** manishg has quit IRC | 02:23 | |
*** erkules_ has joined #openstack-neutron | 02:29 | |
*** ramishra_ has quit IRC | 02:30 | |
*** a_le has quit IRC | 02:30 | |
*** erkules has quit IRC | 02:31 | |
*** ramishra_ has joined #openstack-neutron | 02:31 | |
otherwiseguy | HenryG: Yeah, that was mistakenly left in after I was getting a weird timeout related to new test that was added since I initially wrote the patch. Removing now. | 02:31 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from AsyncProcess https://review.openstack.org/155101 | 02:33 |
otherwiseguy | HenryG: ^ | 02:33 |
HenryG | otherwiseguy: ack, waiting for jenkins results | 02:34 |
otherwiseguy | HenryG: Thanks again for all of the review. I know it is not a very entertaining patch set to review. :) | 02:35 |
otherwiseguy | Tomorrow morning I'll start working on the L3 Agent patch test failures--hopefully a pretty easy fix. | 02:36 |
*** pal has quit IRC | 02:36 | |
*** shwetaap1 has quit IRC | 02:36 | |
*** shwetaap has joined #openstack-neutron | 02:36 | |
*** dims__ has quit IRC | 02:40 | |
*** a_le has joined #openstack-neutron | 02:40 | |
*** ramishra_ has quit IRC | 02:43 | |
*** tomoe__ has quit IRC | 02:46 | |
*** manishg has joined #openstack-neutron | 02:46 | |
*** ramishra_ has joined #openstack-neutron | 02:53 | |
*** hichihara has quit IRC | 02:54 | |
*** yalie has left #openstack-neutron | 02:57 | |
*** yalie has joined #openstack-neutron | 02:57 | |
*** amuller has quit IRC | 02:57 | |
*** zhenq has quit IRC | 03:01 | |
*** armax has joined #openstack-neutron | 03:06 | |
*** armax has quit IRC | 03:06 | |
*** marun has quit IRC | 03:07 | |
*** vinsh has joined #openstack-neutron | 03:09 | |
*** markvoelker has joined #openstack-neutron | 03:09 | |
*** yalie has left #openstack-neutron | 03:11 | |
*** salv-orlando has joined #openstack-neutron | 03:14 | |
*** markvoelker has quit IRC | 03:14 | |
*** manishg has quit IRC | 03:21 | |
*** manishg has joined #openstack-neutron | 03:22 | |
*** manishg has quit IRC | 03:26 | |
openstackgerrit | Shweta P proposed openstack/neutron: Passes the plugin context variable in the ML2 Extension Driver API https://review.openstack.org/152759 | 03:28 |
*** anish_ has joined #openstack-neutron | 03:32 | |
*** aepifanov has joined #openstack-neutron | 03:34 | |
*** bradjones has quit IRC | 03:34 | |
*** pal has joined #openstack-neutron | 03:38 | |
*** dims__ has joined #openstack-neutron | 03:40 | |
openstackgerrit | Merged openstack/python-neutronclient: Add ip_version to extra dhcp opts https://review.openstack.org/142015 | 03:43 |
*** amotoki_ has joined #openstack-neutron | 03:43 | |
*** vinsh has quit IRC | 03:45 | |
*** Ajay__ has joined #openstack-neutron | 03:45 | |
*** amotoki_ has quit IRC | 03:45 | |
*** dims__ has quit IRC | 03:45 | |
*** aepifanov has quit IRC | 03:46 | |
*** tomoe_ has joined #openstack-neutron | 03:46 | |
openstackgerrit | Merged openstack/python-neutronclient: Split base function of v2_0.Client into a separate class https://review.openstack.org/150512 | 03:46 |
openstackgerrit | Dane LeBlanc proposed openstack/neutron: Include IPv6 SLAAC addresses implicitly for port create/update https://review.openstack.org/113339 | 03:48 |
openstackgerrit | Merged openstack/python-neutronclient: Fix for incorrect parameter in user-id error message in shell.py https://review.openstack.org/153829 | 03:49 |
*** tomoe_ has quit IRC | 03:50 | |
*** emagana has joined #openstack-neutron | 03:51 | |
openstackgerrit | Dane LeBlanc proposed openstack/neutron: Include IPv6 SLAAC addresses implicitly for port create/update https://review.openstack.org/113339 | 03:52 |
*** seizadi has quit IRC | 03:53 | |
*** tomoe_ has joined #openstack-neutron | 03:55 | |
*** reed has quit IRC | 03:56 | |
*** claudiub has quit IRC | 03:56 | |
*** achanda has joined #openstack-neutron | 03:56 | |
openstackgerrit | Merged openstack/neutron-vpnaas: Add index on tenant_id https://review.openstack.org/154506 | 04:03 |
openstackgerrit | Merged openstack/neutron: OVS DVR: Use a right mac address value to compose a flow https://review.openstack.org/154330 | 04:03 |
*** Longgeek has quit IRC | 04:04 | |
*** ramishra_ has quit IRC | 04:08 | |
*** suresh12 has quit IRC | 04:09 | |
*** moshele has joined #openstack-neutron | 04:09 | |
*** markvoelker has joined #openstack-neutron | 04:10 | |
*** Marga_ has quit IRC | 04:12 | |
*** moshele has quit IRC | 04:16 | |
*** markvoelker has quit IRC | 04:16 | |
*** nlahouti has joined #openstack-neutron | 04:19 | |
*** ramishra_ has joined #openstack-neutron | 04:20 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/153946 | 04:22 |
*** leenheer has joined #openstack-neutron | 04:22 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Make l2pop UT independent from OVS https://review.openstack.org/118620 | 04:22 |
*** ajmiller_ is now known as ajmiller | 04:22 | |
*** nlahouti has quit IRC | 04:23 | |
*** nlahouti has joined #openstack-neutron | 04:23 | |
*** coolsvap_ is now known as coolsvap | 04:31 | |
*** yamamoto_ has quit IRC | 04:33 | |
*** SridharRamaswamy has joined #openstack-neutron | 04:35 | |
*** yamamoto_ has joined #openstack-neutron | 04:35 | |
*** Marga_ has joined #openstack-neutron | 04:36 | |
*** emagana_ has joined #openstack-neutron | 04:37 | |
*** Ajay__ has quit IRC | 04:37 | |
*** tiswanso has quit IRC | 04:38 | |
*** SridharRamaswamy has quit IRC | 04:39 | |
*** SridharRamaswamy has joined #openstack-neutron | 04:39 | |
*** emagana has quit IRC | 04:40 | |
*** jckasper has joined #openstack-neutron | 04:52 | |
*** jckasper_ has quit IRC | 04:53 | |
*** anilvenkata has joined #openstack-neutron | 04:54 | |
*** Marga_ has quit IRC | 04:55 | |
*** Marga_ has joined #openstack-neutron | 04:55 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhance to use DeferredOVSBridge https://review.openstack.org/156289 | 04:55 |
*** trinaths has joined #openstack-neutron | 04:57 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhance to use DeferredOVSBridge https://review.openstack.org/156289 | 04:59 |
*** radez is now known as radez_g0n3 | 05:01 | |
*** leenheer has left #openstack-neutron | 05:02 | |
*** coolsvap is now known as coolsvap_ | 05:02 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 05:05 |
*** vthapar has joined #openstack-neutron | 05:06 | |
*** trinaths1 has joined #openstack-neutron | 05:08 | |
*** banix has quit IRC | 05:08 | |
*** coolsvap_ is now known as coolsvap | 05:09 | |
*** _cjones_ has joined #openstack-neutron | 05:10 | |
*** trinaths has quit IRC | 05:10 | |
*** nikil89_ has joined #openstack-neutron | 05:11 | |
*** nikil89_ is now known as naggappan | 05:11 | |
*** markvoelker has joined #openstack-neutron | 05:12 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS UT: Fix some confusions between local vlan id and segmentation id https://review.openstack.org/156464 | 05:12 |
*** aepifanov has joined #openstack-neutron | 05:14 | |
*** _cjones_ has quit IRC | 05:14 | |
*** yamahata has joined #openstack-neutron | 05:14 | |
*** gildub has quit IRC | 05:14 | |
*** devlaps has quit IRC | 05:16 | |
*** markvoelker has quit IRC | 05:17 | |
*** pradipta has joined #openstack-neutron | 05:22 | |
*** dane_leblanc_ has joined #openstack-neutron | 05:22 | |
*** manishg has joined #openstack-neutron | 05:23 | |
*** dane_leblanc has quit IRC | 05:23 | |
*** murugan has joined #openstack-neutron | 05:24 | |
*** _cjones_ has joined #openstack-neutron | 05:26 | |
*** ajmiller has quit IRC | 05:27 | |
*** ramishra_ has quit IRC | 05:27 | |
*** yfried has quit IRC | 05:31 | |
*** manishg has quit IRC | 05:37 | |
*** yshafrir has joined #openstack-neutron | 05:37 | |
*** manishg has joined #openstack-neutron | 05:37 | |
*** ramishra_ has joined #openstack-neutron | 05:41 | |
*** manishg has quit IRC | 05:42 | |
*** achanda has quit IRC | 05:43 | |
openstackgerrit | Sachi King proposed openstack/neutron: Return exception when attempting to add duplicate VIP https://review.openstack.org/142630 | 05:45 |
*** achanda has joined #openstack-neutron | 05:46 | |
*** pcaruana has quit IRC | 05:47 | |
*** killer_prince is now known as lazy_prince | 05:48 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/153946 | 05:52 |
*** SridharG has joined #openstack-neutron | 05:52 | |
*** _cjones_ has quit IRC | 05:53 | |
*** coolsvap is now known as coolsvap_ | 05:53 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/153946 | 05:56 |
*** ramishra_ has quit IRC | 05:57 | |
*** saju_m has joined #openstack-neutron | 05:57 | |
*** seizadi has joined #openstack-neutron | 06:00 | |
*** coolsvap_ is now known as coolsvap | 06:00 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 06:05 |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 06:06 |
*** anish__ has joined #openstack-neutron | 06:06 | |
*** ramishra_ has joined #openstack-neutron | 06:06 | |
*** anish_ has quit IRC | 06:07 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 06:08 |
*** anshul has joined #openstack-neutron | 06:08 | |
*** Marga_ has quit IRC | 06:09 | |
*** markvoelker has joined #openstack-neutron | 06:13 | |
*** SridharRamaswam1 has joined #openstack-neutron | 06:15 | |
*** SridharRamaswamy has quit IRC | 06:16 | |
*** cipcosma has joined #openstack-neutron | 06:17 | |
*** tflynn_ has joined #openstack-neutron | 06:18 | |
*** markvoelker has quit IRC | 06:19 | |
*** tflynn has quit IRC | 06:21 | |
*** seizadi has quit IRC | 06:22 | |
*** tflynn_ has quit IRC | 06:22 | |
*** seizadi has joined #openstack-neutron | 06:22 | |
*** SridharRamaswam1 has quit IRC | 06:25 | |
*** seizadi1 has joined #openstack-neutron | 06:26 | |
*** seizadi has quit IRC | 06:27 | |
*** seizadi1 has quit IRC | 06:30 | |
*** kickinz1|afk is now known as kickinz1 | 06:30 | |
*** seizadi has joined #openstack-neutron | 06:30 | |
*** zz_avozza is now known as avozza | 06:31 | |
*** anish__ has quit IRC | 06:34 | |
*** ramishra_ has quit IRC | 06:34 | |
*** anish_ has joined #openstack-neutron | 06:35 | |
*** seizadi has quit IRC | 06:40 | |
*** seizadi has joined #openstack-neutron | 06:41 | |
*** vinsh has joined #openstack-neutron | 06:45 | |
*** Murali has joined #openstack-neutron | 06:47 | |
*** saju_m has quit IRC | 06:48 | |
*** vinsh has quit IRC | 06:50 | |
*** irenab has joined #openstack-neutron | 06:52 | |
*** yamamoto_ has quit IRC | 06:57 | |
openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 06:57 |
*** aranjan has quit IRC | 06:57 | |
*** yamahata has quit IRC | 07:00 | |
*** aranjan has joined #openstack-neutron | 07:00 | |
openstackgerrit | Amandeep proposed openstack/python-neutronclient: Ambiguous cli output between "None" and NoneType https://review.openstack.org/156489 | 07:01 |
*** neeti has joined #openstack-neutron | 07:03 | |
*** seizadi1 has joined #openstack-neutron | 07:05 | |
*** seizadi has quit IRC | 07:05 | |
*** evgenyf has joined #openstack-neutron | 07:07 | |
*** salv-orlando has quit IRC | 07:07 | |
*** leenheer has joined #openstack-neutron | 07:09 | |
*** seizadi1 has quit IRC | 07:09 | |
*** saju_m has joined #openstack-neutron | 07:09 | |
*** yfried has joined #openstack-neutron | 07:09 | |
*** seizadi has joined #openstack-neutron | 07:09 | |
*** yamamoto_ has joined #openstack-neutron | 07:10 | |
*** tflynn has joined #openstack-neutron | 07:12 | |
*** yamamoto_ has quit IRC | 07:12 | |
*** yfried_ has joined #openstack-neutron | 07:13 | |
*** tflynn has joined #openstack-neutron | 07:13 | |
*** seizadi1 has joined #openstack-neutron | 07:13 | |
*** yfried has quit IRC | 07:14 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 07:14 |
*** seizadi has quit IRC | 07:14 | |
*** markvoelker has joined #openstack-neutron | 07:15 | |
*** seizadi has joined #openstack-neutron | 07:17 | |
*** seizadi1 has quit IRC | 07:18 | |
*** dims__ has joined #openstack-neutron | 07:18 | |
*** yfried_ is now known as yfried|afk | 07:19 | |
*** scheuran has joined #openstack-neutron | 07:19 | |
*** markvoelker has quit IRC | 07:21 | |
*** mzbik has joined #openstack-neutron | 07:21 | |
*** dims__ has quit IRC | 07:23 | |
*** thomasem has quit IRC | 07:23 | |
*** thomasem has joined #openstack-neutron | 07:24 | |
*** yfried|afk is now known as yfried_ | 07:25 | |
*** aranjan has quit IRC | 07:25 | |
*** gridinv has joined #openstack-neutron | 07:29 | |
*** ZZelle has quit IRC | 07:32 | |
*** ZZelle has joined #openstack-neutron | 07:32 | |
*** ramishra_ has joined #openstack-neutron | 07:35 | |
*** yfried_ is now known as yfried|afk | 07:35 | |
*** nyechiel_ has joined #openstack-neutron | 07:36 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: setup_tunnel_br enhanced to use DeferredOVSBridge https://review.openstack.org/156289 | 07:38 |
*** seizadi has quit IRC | 07:39 | |
*** seizadi has joined #openstack-neutron | 07:40 | |
*** ramishra_ has quit IRC | 07:40 | |
*** devvesa has joined #openstack-neutron | 07:41 | |
*** shwetaap has quit IRC | 07:41 | |
*** lihit has joined #openstack-neutron | 07:41 | |
*** nlahouti1 has joined #openstack-neutron | 07:42 | |
*** gridinv has quit IRC | 07:43 | |
*** nlahouti has quit IRC | 07:44 | |
*** seizadi1 has joined #openstack-neutron | 07:44 | |
*** seizadi has quit IRC | 07:44 | |
*** nlahouti1 has quit IRC | 07:46 | |
*** moshele has joined #openstack-neutron | 07:47 | |
*** achanda has quit IRC | 07:47 | |
*** moshele has quit IRC | 07:48 | |
*** ParsectiX has joined #openstack-neutron | 07:48 | |
*** seizadi1 has quit IRC | 07:49 | |
*** seizadi has joined #openstack-neutron | 07:49 | |
*** nyechiel_ has quit IRC | 07:51 | |
*** nyechiel_ has joined #openstack-neutron | 07:51 | |
*** seizadi1 has joined #openstack-neutron | 07:52 | |
*** yfried|afk is now known as yfried_ | 07:53 | |
*** mkerrin has joined #openstack-neutron | 07:54 | |
*** achanda has joined #openstack-neutron | 07:54 | |
*** seizadi has quit IRC | 07:54 | |
*** seizadi1 has quit IRC | 07:56 | |
*** rm_work is now known as rm_work|away | 07:57 | |
*** nmagnezi has joined #openstack-neutron | 07:59 | |
*** pal has quit IRC | 08:01 | |
*** leenheer has quit IRC | 08:02 | |
*** kawa2014 has joined #openstack-neutron | 08:08 | |
*** lihit has joined #openstack-neutron | 08:09 | |
*** lihit has quit IRC | 08:09 | |
*** fifieldt has joined #openstack-neutron | 08:09 | |
*** belmoreira has joined #openstack-neutron | 08:09 | |
*** chlong has quit IRC | 08:11 | |
*** belmoreira has quit IRC | 08:14 | |
*** pask81 has joined #openstack-neutron | 08:14 | |
*** pal has joined #openstack-neutron | 08:17 | |
*** markvoelker has joined #openstack-neutron | 08:17 | |
*** seizadi has joined #openstack-neutron | 08:18 | |
*** gridinv has joined #openstack-neutron | 08:20 | |
*** achanda has quit IRC | 08:20 | |
*** markvoelker has quit IRC | 08:22 | |
*** achanda has joined #openstack-neutron | 08:24 | |
*** sbelous has quit IRC | 08:24 | |
*** aranjan has joined #openstack-neutron | 08:26 | |
*** ronis has joined #openstack-neutron | 08:27 | |
*** seizadi has quit IRC | 08:27 | |
*** aepifanov has quit IRC | 08:28 | |
*** aepifanov has joined #openstack-neutron | 08:28 | |
*** achanda has quit IRC | 08:30 | |
*** aranjan has quit IRC | 08:30 | |
*** tflynn has quit IRC | 08:34 | |
*** tflynn has joined #openstack-neutron | 08:35 | |
*** matrohon has joined #openstack-neutron | 08:36 | |
*** ramishra_ has joined #openstack-neutron | 08:36 | |
*** achanda has joined #openstack-neutron | 08:37 | |
*** salv-orlando has joined #openstack-neutron | 08:38 | |
*** ramishra_ has quit IRC | 08:41 | |
openstackgerrit | Lionel Zerbib proposed openstack/python-neutronclient: Add rate limiting security groups rules CLI extension https://review.openstack.org/156205 | 08:42 |
*** tflynn has quit IRC | 08:42 | |
*** jpena has joined #openstack-neutron | 08:45 | |
*** achanda has quit IRC | 08:47 | |
*** ivar-lazzaro has quit IRC | 08:47 | |
*** anilvenkata has left #openstack-neutron | 08:47 | |
*** jistr has joined #openstack-neutron | 08:51 | |
*** roeyc has joined #openstack-neutron | 08:52 | |
*** jlibosva has joined #openstack-neutron | 08:53 | |
*** mkerrin has quit IRC | 08:53 | |
*** _cjones_ has joined #openstack-neutron | 08:54 | |
*** fifieldt has quit IRC | 08:56 | |
*** _cjones_ has quit IRC | 08:58 | |
*** jp_at_hp has joined #openstack-neutron | 08:59 | |
*** karimb has joined #openstack-neutron | 09:00 | |
*** obondarev_ is now known as obondarev | 09:03 | |
*** a_le has quit IRC | 09:07 | |
*** a_le has joined #openstack-neutron | 09:08 | |
*** salv-orlando has quit IRC | 09:15 | |
*** markvoelker has joined #openstack-neutron | 09:18 | |
*** aix has quit IRC | 09:21 | |
*** a_le_ has joined #openstack-neutron | 09:21 | |
*** ygbo has joined #openstack-neutron | 09:22 | |
*** markvoelker has quit IRC | 09:23 | |
*** a_le has quit IRC | 09:24 | |
*** jlanoux has joined #openstack-neutron | 09:25 | |
*** avozza is now known as zz_avozza | 09:26 | |
*** ihrachyshka has joined #openstack-neutron | 09:26 | |
*** aranjan has joined #openstack-neutron | 09:27 | |
*** dsneddon has quit IRC | 09:30 | |
*** yfried_ is now known as yfried|afk | 09:31 | |
*** aranjan has quit IRC | 09:31 | |
*** nivtuckio has joined #openstack-neutron | 09:35 | |
*** yfried|afk is now known as yfried_ | 09:38 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Refactor the ProcessMonitor API https://review.openstack.org/154464 | 09:39 |
*** apuimedo has joined #openstack-neutron | 09:44 | |
*** karimb has quit IRC | 09:45 | |
*** aix has joined #openstack-neutron | 09:48 | |
*** tomoe_ has quit IRC | 09:51 | |
*** zz_avozza is now known as avozza | 09:55 | |
*** arif-ali has joined #openstack-neutron | 10:00 | |
*** moshele has joined #openstack-neutron | 10:02 | |
openstackgerrit | Fawad Khaliq proposed openstack/neutron: pg-ci test commit (do not review) https://review.openstack.org/156529 | 10:02 |
*** salv-orlando has joined #openstack-neutron | 10:03 | |
*** aix has quit IRC | 10:05 | |
*** avozza is now known as zz_avozza | 10:06 | |
*** saju_m has quit IRC | 10:09 | |
*** a_le_ has quit IRC | 10:09 | |
*** aix has joined #openstack-neutron | 10:17 | |
openstackgerrit | Bradley Jones proposed openstack/neutron: Advertise mtu over dhcp https://review.openstack.org/155874 | 10:18 |
*** markvoelker has joined #openstack-neutron | 10:19 | |
*** afazekas has joined #openstack-neutron | 10:23 | |
*** markvoelker has quit IRC | 10:24 | |
*** yfried_ is now known as yfried|afk | 10:24 | |
*** ajo has joined #openstack-neutron | 10:25 | |
*** zz_avozza is now known as avozza | 10:26 | |
*** jlanoux_ has joined #openstack-neutron | 10:27 | |
*** aranjan has joined #openstack-neutron | 10:28 | |
*** jlanoux has quit IRC | 10:30 | |
*** aranjan has quit IRC | 10:32 | |
*** ilyashakhat_ has quit IRC | 10:32 | |
*** ilyashakhat has joined #openstack-neutron | 10:32 | |
*** shilpa_ has joined #openstack-neutron | 10:34 | |
*** ramishra_ has joined #openstack-neutron | 10:38 | |
*** saju_m has joined #openstack-neutron | 10:39 | |
*** sbelous has joined #openstack-neutron | 10:40 | |
*** ramishra_ has quit IRC | 10:43 | |
*** jlanoux has joined #openstack-neutron | 10:43 | |
*** yfried|afk is now known as yfried_ | 10:44 | |
*** jlanoux_ has quit IRC | 10:45 | |
*** txomon|fon has quit IRC | 10:51 | |
*** tomoe_ has joined #openstack-neutron | 10:51 | |
*** txomon|fon has joined #openstack-neutron | 10:53 | |
*** yfried_ is now known as yfried|afk | 10:54 | |
*** yfried|afk is now known as yfried_ | 10:56 | |
*** tomoe_ has quit IRC | 10:56 | |
openstackgerrit | Aman Kumar proposed openstack/neutron: External Networks in ML2 have invalid providernet types https://review.openstack.org/149546 | 10:58 |
*** nivtuckio has quit IRC | 10:58 | |
*** pc_m has joined #openstack-neutron | 10:59 | |
openstackgerrit | Fawad Khaliq proposed openstack/neutron: pg-ci test commit (do not review) https://review.openstack.org/156529 | 11:01 |
*** salv-orlando has quit IRC | 11:02 | |
*** anish_ has quit IRC | 11:09 | |
openstackgerrit | P. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help router-update" help info updated https://review.openstack.org/141006 | 11:20 |
*** markvoelker has joined #openstack-neutron | 11:20 | |
*** markvoelker has quit IRC | 11:25 | |
*** jlanoux_ has joined #openstack-neutron | 11:27 | |
*** aranjan has joined #openstack-neutron | 11:28 | |
*** jlanoux has quit IRC | 11:30 | |
*** jordandh has quit IRC | 11:30 | |
*** jordandh has joined #openstack-neutron | 11:31 | |
*** bobmel has quit IRC | 11:31 | |
*** bobmel has joined #openstack-neutron | 11:31 | |
*** dims__ has joined #openstack-neutron | 11:32 | |
*** aranjan has quit IRC | 11:33 | |
*** yamamoto has joined #openstack-neutron | 11:37 | |
*** ramishra_ has joined #openstack-neutron | 11:39 | |
*** jlanoux has joined #openstack-neutron | 11:43 | |
*** ramishra_ has quit IRC | 11:43 | |
*** jlanoux_ has quit IRC | 11:46 | |
*** shilpa_ has quit IRC | 11:48 | |
*** SridharG has left #openstack-neutron | 11:54 | |
*** sbelous has quit IRC | 12:02 | |
*** salv-orlando has joined #openstack-neutron | 12:02 | |
*** yamamoto has quit IRC | 12:02 | |
*** johnbelamaric1 has quit IRC | 12:07 | |
*** johnbelamaric has joined #openstack-neutron | 12:12 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Refactor the ProcessMonitor API https://review.openstack.org/154464 | 12:14 |
*** avozza is now known as zz_avozza | 12:14 | |
*** banix has joined #openstack-neutron | 12:19 | |
*** markvoelker has joined #openstack-neutron | 12:21 | |
openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script. https://review.openstack.org/156563 | 12:23 |
openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script. https://review.openstack.org/156563 | 12:25 |
*** markvoelker has quit IRC | 12:26 | |
*** amuller has joined #openstack-neutron | 12:26 | |
openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script. https://review.openstack.org/156563 | 12:27 |
openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script. https://review.openstack.org/156563 | 12:28 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: WIP: Testing for bug/1421772 https://review.openstack.org/156566 | 12:29 |
*** aranjan has joined #openstack-neutron | 12:29 | |
openstackgerrit | Vivekanandan Narasimhan proposed openstack/neutron: Hyper-V: Fixes security groups issue https://review.openstack.org/150826 | 12:29 |
*** mzbik_ has joined #openstack-neutron | 12:31 | |
*** pal has quit IRC | 12:31 | |
*** mzbik has quit IRC | 12:33 | |
*** aranjan has quit IRC | 12:33 | |
*** zhenq has joined #openstack-neutron | 12:34 | |
*** tomoe_ has joined #openstack-neutron | 12:38 | |
*** ramishra_ has joined #openstack-neutron | 12:40 | |
*** yfried_ is now known as yfried|afk | 12:41 | |
*** yfried|afk is now known as yfried_ | 12:43 | |
*** ramishra_ has quit IRC | 12:44 | |
*** baoli has joined #openstack-neutron | 12:44 | |
*** pradipta has quit IRC | 12:49 | |
*** baoli has quit IRC | 12:49 | |
*** baoli has joined #openstack-neutron | 12:50 | |
*** salv-orlando has quit IRC | 12:50 | |
*** salv-orlando has joined #openstack-neutron | 12:50 | |
*** markvoelker has joined #openstack-neutron | 12:52 | |
*** banix has quit IRC | 12:57 | |
*** yamamoto has joined #openstack-neutron | 12:57 | |
*** zz_avozza is now known as avozza | 13:05 | |
*** avozza is now known as zz_avozza | 13:07 | |
*** trinaths1 has quit IRC | 13:08 | |
*** irenab has quit IRC | 13:08 | |
*** yamamoto has quit IRC | 13:12 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Refactor the ProcessMonitor API https://review.openstack.org/154464 | 13:13 |
*** jckasper has quit IRC | 13:13 | |
*** irenab has joined #openstack-neutron | 13:14 | |
*** irenab has quit IRC | 13:15 | |
*** zz_avozza is now known as avozza | 13:15 | |
*** mwagner_lap has quit IRC | 13:15 | |
*** irenab has joined #openstack-neutron | 13:16 | |
*** tiswanso has joined #openstack-neutron | 13:18 | |
*** ParsectiX has quit IRC | 13:20 | |
*** Marga_ has joined #openstack-neutron | 13:22 | |
*** saju_m has quit IRC | 13:27 | |
*** irenab has quit IRC | 13:28 | |
*** aranjan has joined #openstack-neutron | 13:30 | |
*** aranjan has quit IRC | 13:35 | |
*** bjornar has quit IRC | 13:36 | |
*** yfried_ is now known as yfried|afk | 13:37 | |
*** yfried|afk is now known as yfried_ | 13:38 | |
*** claudiub has joined #openstack-neutron | 13:39 | |
*** pal has joined #openstack-neutron | 13:40 | |
*** afazekas has quit IRC | 13:40 | |
*** ramishra_ has joined #openstack-neutron | 13:40 | |
*** Marga_ has quit IRC | 13:44 | |
*** ramishra_ has quit IRC | 13:45 | |
*** neeti has quit IRC | 13:47 | |
*** jckasper has joined #openstack-neutron | 13:47 | |
*** amotoki has quit IRC | 13:49 | |
*** HenryG has quit IRC | 13:49 | |
*** vthapar has quit IRC | 13:51 | |
*** dane_leblanc_ has quit IRC | 13:52 | |
*** HenryG has joined #openstack-neutron | 13:52 | |
*** amotoki has joined #openstack-neutron | 13:54 | |
*** amotoki has quit IRC | 13:54 | |
*** belmoreira has joined #openstack-neutron | 13:54 | |
*** FJB has joined #openstack-neutron | 13:55 | |
openstackgerrit | garyk proposed openstack/neutron: Fix parameters in exception handling https://review.openstack.org/156590 | 13:55 |
*** radez_g0n3 is now known as radez | 13:56 | |
*** yshafrir has quit IRC | 13:58 | |
*** rushil has joined #openstack-neutron | 13:59 | |
*** tiswanso has quit IRC | 14:01 | |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 14:01 |
*** trown|outttypeww is now known as trown | 14:02 | |
*** sgordon is now known as steveg | 14:02 | |
*** irenab has joined #openstack-neutron | 14:03 | |
*** avozza is now known as zz_avozza | 14:04 | |
*** csoukup has joined #openstack-neutron | 14:06 | |
*** SridharG has joined #openstack-neutron | 14:09 | |
*** banix has joined #openstack-neutron | 14:10 | |
*** tidwellr has joined #openstack-neutron | 14:10 | |
*** signed8bit_ZZZzz is now known as signed8bit | 14:10 | |
*** signed8bit is now known as signed8bit_ZZZzz | 14:10 | |
*** signed8bit_ZZZzz is now known as signed8bit | 14:10 | |
*** changbl has quit IRC | 14:10 | |
*** tomoe__ has joined #openstack-neutron | 14:11 | |
*** dane_leblanc has joined #openstack-neutron | 14:11 | |
*** tomoe_ has quit IRC | 14:13 | |
*** Marga_ has joined #openstack-neutron | 14:15 | |
*** kbringard has joined #openstack-neutron | 14:16 | |
*** anshul has quit IRC | 14:17 | |
*** FJB has quit IRC | 14:19 | |
*** Marga_ has quit IRC | 14:20 | |
*** rpothier has joined #openstack-neutron | 14:20 | |
*** watanabe_isao has quit IRC | 14:20 | |
*** tiswanso has joined #openstack-neutron | 14:21 | |
*** tiswanso has quit IRC | 14:22 | |
*** tiswanso has joined #openstack-neutron | 14:22 | |
*** maishsk has joined #openstack-neutron | 14:24 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: Fallback to new l3 agent main() location https://review.openstack.org/156602 | 14:25 |
*** tiswanso has quit IRC | 14:26 | |
*** johnbelamaric has quit IRC | 14:26 | |
*** tiswanso has joined #openstack-neutron | 14:27 | |
*** shwetaap has joined #openstack-neutron | 14:28 | |
*** xuhanp has quit IRC | 14:28 | |
*** johnbelamaric has joined #openstack-neutron | 14:28 | |
otherwiseguy | markmcclain: ping | 14:29 |
*** regXboi has joined #openstack-neutron | 14:29 | |
*** shwetaap1 has joined #openstack-neutron | 14:29 | |
*** murugan has quit IRC | 14:30 | |
*** aranjan has joined #openstack-neutron | 14:31 | |
otherwiseguy | markmcclain: Does the explanation/TODO I added to https://review.openstack.org/#/c/153067/ address your concerns? Two cores have +2'd the patch, and one has +1'd, but they wanted to give you a chance to comment. | 14:31 |
*** anish_ has joined #openstack-neutron | 14:31 | |
*** pradk has quit IRC | 14:32 | |
*** shwetaap has quit IRC | 14:32 | |
*** aranjan has quit IRC | 14:35 | |
*** ramishra_ has joined #openstack-neutron | 14:39 | |
*** pradk has joined #openstack-neutron | 14:39 | |
*** zz_jgrimm is now known as jgrimm | 14:40 | |
*** mzbik_ has quit IRC | 14:40 | |
*** tomoe__ has quit IRC | 14:42 | |
*** ramishra_ has quit IRC | 14:43 | |
ihrachyshka | otherwiseguy, ajo, do you know what's the big reason to have netns_cleanup tool monkey-patched? | 14:45 |
*** yamamoto has joined #openstack-neutron | 14:46 | |
otherwiseguy | ihrachyshka: I can't immediately think of any reason for it to be. | 14:47 |
* otherwiseguy looks | 14:47 | |
*** tomoe_ has joined #openstack-neutron | 14:47 | |
ihrachyshka | otherwiseguy, I wonder whether we should move cli tools into e.g. neutron/cmd/tools/... and all patched entry points for agents and services into neutron/cmd/{agents,services}, and monkey patch only the latter | 14:48 |
*** localloop127 has joined #openstack-neutron | 14:48 | |
otherwiseguy | ihrachyshka: looks like its been that way since markmcclain wrote it, maybe he knows? | 14:49 |
ihrachyshka | otherwiseguy, though that would mean that we run tools unpatched while we test them in patched unit test env... | 14:49 |
otherwiseguy | ihrachyshka: if it hasn't hurt anything yet being monkeypatched when it didn't need to be, maybe just leave everything under cmd with patched __init__.py and get more complicated only if we have to? | 14:52 |
ihrachyshka | otherwiseguy, yeah, but at the same time there is stuff in the tree that runs unpatched, and I hate it :) | 14:53 |
ihrachyshka | otherwiseguy, so it's a change for one side or another | 14:53 |
ihrachyshka | probably patching everything is the right thing.. | 14:53 |
ajo | ihrachyshka, not sure, I guess because we're using eventlet without calling the native methods? | 14:54 |
ajo | where native=eventlet specific methods | 14:54 |
*** dboik has joined #openstack-neutron | 14:55 | |
ajo | ihrachyshka, why would you like to un-monkeypatch? | 14:55 |
otherwiseguy | ihrachyshka: Have you considered just migrating everything off of eventlet as the fix? :) | 14:55 |
otherwiseguy | depth-first fixing FTW :) | 14:55 |
ajo | :D | 14:56 |
ihrachyshka | otherwiseguy, hehe, oslo team would celebrate the fact for several months once it occurs | 14:56 |
*** zz_avozza is now known as avozza | 14:56 | |
ajo | the change in neutron would be massive | 14:56 |
otherwiseguy | ihrachyshka: Be the hero we need. :D | 14:56 |
*** lazy_prince is now known as killer_prince | 14:56 | |
ihrachyshka | otherwiseguy, though... I don't believe neutron community is capable to start writing thread safe code in python immediately | 14:56 |
ihrachyshka | otherwiseguy, I know multiple places in the code that rely on specific sections to not yield | 14:57 |
otherwiseguy | ihrachyshka: Some of us have C backgrounds and have done all kinds of work with threads. :) | 14:57 |
*** tiswanso has quit IRC | 14:57 | |
ihrachyshka | otherwiseguy, yeah, I know. I wrote in C before neutron. something for bind9 with all kinds of weird paranoid locks | 14:58 |
ihrachyshka | otherwiseguy, but it's not a general thing I guess | 14:58 |
ihrachyshka | otherwiseguy, it's so easy to rely on eventlet not to yield | 14:58 |
ihrachyshka | 'rely' as in 'hope' | 14:58 |
*** banix has quit IRC | 15:00 | |
*** mwagner_lap has joined #openstack-neutron | 15:01 | |
*** mestery_ is now known as mestery | 15:01 | |
*** irenab has quit IRC | 15:03 | |
*** dims__ has quit IRC | 15:03 | |
*** dims__ has joined #openstack-neutron | 15:04 | |
*** dims__ has quit IRC | 15:04 | |
*** trad511 has joined #openstack-neutron | 15:04 | |
*** tflynn has joined #openstack-neutron | 15:04 | |
*** dims__ has joined #openstack-neutron | 15:05 | |
*** banix has joined #openstack-neutron | 15:05 | |
*** tflynn has quit IRC | 15:05 | |
*** tflynn has joined #openstack-neutron | 15:06 | |
*** seizadi has joined #openstack-neutron | 15:06 | |
*** julien_ZTE has joined #openstack-neutron | 15:07 | |
*** amotoki has joined #openstack-neutron | 15:07 | |
*** absubram has joined #openstack-neutron | 15:09 | |
*** salv-orlando has quit IRC | 15:11 | |
*** salv-orlando has joined #openstack-neutron | 15:13 | |
*** maishsk has quit IRC | 15:14 | |
*** carl_baldwin has joined #openstack-neutron | 15:16 | |
*** shaohe_feng has joined #openstack-neutron | 15:16 | |
*** Marga_ has joined #openstack-neutron | 15:16 | |
*** a_le has joined #openstack-neutron | 15:16 | |
*** tarun has quit IRC | 15:16 | |
*** marrusl has quit IRC | 15:18 | |
*** rm_work|away is now known as rm_work | 15:18 | |
*** Murali has quit IRC | 15:18 | |
*** txomon|fon has quit IRC | 15:19 | |
*** tflynn has quit IRC | 15:19 | |
*** maishsk has joined #openstack-neutron | 15:19 | |
*** tflynn has joined #openstack-neutron | 15:19 | |
*** marrusl has joined #openstack-neutron | 15:20 | |
*** Marga_ has quit IRC | 15:21 | |
*** Networkn3rd has joined #openstack-neutron | 15:21 | |
ihrachyshka | amotoki, hey! | 15:22 |
amotoki | ihrachyshka: hi | 15:23 |
ihrachyshka | amotoki, re https://review.openstack.org/#/c/155412/1/neutron/cmd/agents/plugins/bigswitch/restproxy_agent.py | 15:23 |
ihrachyshka | amotoki, do you think we should make the same for all services? | 15:23 |
ihrachyshka | amotoki, or just 3party? | 15:23 |
*** nmagnezi has quit IRC | 15:23 | |
*** jaypipes has quit IRC | 15:24 | |
amotoki | ihrachyshka: I think it is specific to thirdparty. In general, it is good to keep these kind of things in cmd/* | 15:25 |
ihrachyshka | amotoki, but it's not clear where to divide agent code into cmd part and library part | 15:25 |
amotoki | ihrachyshka: When looking at nova/cmd for example, they have similar contents in cmd repo | 15:25 |
*** seizadi1 has joined #openstack-neutron | 15:25 | |
ihrachyshka | amotoki, I think that was one of concerns from markmcclain that I move parts of agent code deliberately | 15:26 |
*** tiswanso has joined #openstack-neutron | 15:26 | |
*** seizadi has quit IRC | 15:26 | |
*** jobewan has joined #openstack-neutron | 15:27 | |
ihrachyshka | amotoki, obviously dumb main() would mean another level of traceback (entry_point calls to dumb main(), dumb main() calls to real main()) | 15:27 |
*** vinsh has joined #openstack-neutron | 15:27 | |
amotoki | ihrachyshka: yeah, it is difficult... dumb main() vs small stuff. | 15:28 |
*** naggappan has quit IRC | 15:28 | |
markmcclain | so reading the scrollback is that we're scattering stuff everywhere in teh code base | 15:28 |
*** yfried_ has quit IRC | 15:28 | |
ihrachyshka | amotoki, so I would better not leave two different approaches in the tree and make everything uniform (either have all real main()s in neutron/cmd, or dumb only for all services) | 15:28 |
*** seizadi has joined #openstack-neutron | 15:29 | |
ihrachyshka | markmcclain, would dumb main()s that only call to real main() located in the old location work for you? | 15:29 |
markmcclain | what's the utility of the indirection? | 15:29 |
*** nlahouti has joined #openstack-neutron | 15:30 | |
ihrachyshka | markmcclain, having eventlet monkey_patch consolidated in one place and applied to all entry points | 15:30 |
*** seizadi1 has quit IRC | 15:30 | |
markmcclain | ihrachyshka: what if we're doing it wrong? | 15:30 |
markmcclain | ie what if eventlet and monkeypatching is the real problem | 15:30 |
ihrachyshka | markmcclain, elaborate | 15:30 |
ihrachyshka | markmcclain, ah, well... yeah, that's a problem | 15:30 |
ihrachyshka | but it's not something we can solve now | 15:31 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Skip DBDuplicateEntry exception in security group creation https://review.openstack.org/155789 | 15:31 |
ihrachyshka | even oslo libs are not ready to drop it | 15:31 |
ihrachyshka | markmcclain, in long term, sure we should switch to threads | 15:31 |
*** aranjan has joined #openstack-neutron | 15:31 | |
markmcclain | ihrachyshka: I thought many of the oslo threads will just work with threads | 15:32 |
*** seizadi1 has joined #openstack-neutron | 15:32 | |
markmcclain | it was using other forms of concurrency that might not work (asyncio, gevent directly, etc) | 15:32 |
*** achanda has joined #openstack-neutron | 15:32 | |
markmcclain | s/oslo threads/oslo libs/ | 15:32 |
ihrachyshka | markmcclain, I agree we should look into this direction | 15:33 |
otherwiseguy | markmcclain: if you have a couple of seconds to respond to my response to your concerns on run_as_root patch (https://review.openstack.org/#/c/153067), I'd be very appreciative. :) | 15:33 |
*** seizadi has quit IRC | 15:33 | |
markmcclain | otherwiseguy: it is on my stack of things to catch up on after the long weekend | 15:33 |
ihrachyshka | markmcclain, but I have plenty of examples where neutron code is just not ready to switch contexts deliberately | 15:33 |
*** jobewan has quit IRC | 15:34 | |
*** tiswanso has quit IRC | 15:34 | |
*** nlahouti has quit IRC | 15:34 | |
*** tiswanso has joined #openstack-neutron | 15:34 | |
*** tflynn has quit IRC | 15:34 | |
markmcclain | ihrachyshka: example? | 15:35 |
ihrachyshka | markmcclain, neutron/agent/l3/router_processing_queue.py | 15:35 |
ihrachyshka | markmcclain, the way we fill in the queue assumes we don't yield randomly | 15:36 |
*** seizadi has joined #openstack-neutron | 15:36 | |
*** aranjan has quit IRC | 15:36 | |
markmcclain | eventlet naturally yeilds randomly | 15:36 |
*** shilpa_ has joined #openstack-neutron | 15:36 | |
ihrachyshka | markmcclain, not at all | 15:36 |
*** seizadi1 has quit IRC | 15:37 | |
*** Aish has joined #openstack-neutron | 15:37 | |
openstackgerrit | Merged openstack/neutron: Remove RPC dependency to create FIP agent gw port https://review.openstack.org/153729 | 15:37 |
ihrachyshka | markmcclain, it yields in special magic cases everyone should keep in his mind | 15:37 |
markmcclain | it's the reason we find so many random deadlocks | 15:37 |
markmcclain | most devs don't expect eventlet to yield in teh places it does | 15:37 |
*** tomoe_ has quit IRC | 15:37 | |
*** tflynn has joined #openstack-neutron | 15:37 | |
ihrachyshka | yeah, but they are not actually random, it's just that keeping all places where it may yield in mind is hard | 15:37 |
markmcclain | not certain I agree with taht… we'll often find devs adding log messages for other purposes that then cascade into other layers | 15:38 |
ihrachyshka | sure, logs can yield (sometimes) | 15:39 |
*** Marga_ has joined #openstack-neutron | 15:39 | |
*** manishg has joined #openstack-neutron | 15:40 | |
ihrachyshka | markmcclain, btw I feel dejavu, we had similar conversation ~ a year ago :) | 15:40 |
amotoki | agree. we are using many libs and small change (with output) leads to eventlet yield changes.,, it seems random from most devs. | 15:40 |
*** seizadi1 has joined #openstack-neutron | 15:40 | |
*** seizadi has quit IRC | 15:40 | |
*** Marga_ has quit IRC | 15:40 | |
ihrachyshka | amotoki, if it would yield truly randomly, we would get deadlocks every minute :) | 15:41 |
*** pal has quit IRC | 15:41 | |
*** Marga_ has joined #openstack-neutron | 15:41 | |
markmcclain | ihrachyshka: it is kind of is | 15:41 |
*** achanda has quit IRC | 15:41 | |
*** maishsk has quit IRC | 15:41 | |
markmcclain | ihrachyshka: all you need is a slow worker or slower connection and the timings can change significantly | 15:42 |
markmcclain | it's the reason why nova tests can variable between runs with nova-net and neutron | 15:42 |
ihrachyshka | markmcclain, zzzeek from oslo.db actually pushs for using threads. we'll see how it goes. | 15:42 |
*** maishsk has joined #openstack-neutron | 15:42 | |
markmcclain | ihrachyshka: yeah… his commentary was a good explanation of the issue | 15:43 |
markmcclain | ihrachyshka: several components of openstack-infra use native threads and have for some time :) | 15:43 |
ihrachyshka | markmcclain, that's nova legacy, playing with async was huge that time I guess :D | 15:44 |
*** seizadi1 has quit IRC | 15:44 | |
*** devvesa_ has joined #openstack-neutron | 15:44 | |
openstackgerrit | Merged openstack/neutron: Setup br-tun in secure fail mode to avoid broadcast storms https://review.openstack.org/155328 | 15:44 |
*** maishsk has quit IRC | 15:44 | |
*** tiswanso has quit IRC | 15:44 | |
markmcclain | yeah and also some parts started with twisted in early iterations | 15:45 |
ihrachyshka | markmcclain, do you want us to consider threads for the next cycle? not complete switch, but adding a threading mode | 15:45 |
markmcclain | ihrachyshka: I do think looking at threads in some of the agents is something we should consider | 15:45 |
*** seizadi has joined #openstack-neutron | 15:45 | |
ihrachyshka | markmcclain, right, we may try to switch one by one. cool, I'll think about it more. | 15:45 |
*** maishsk has joined #openstack-neutron | 15:45 | |
*** seizadi has quit IRC | 15:46 | |
markmcclain | ihrachyshka: right they're isolated in a way that we have that option | 15:46 |
markmcclain | it's also the reason why moving everything to a central cmd module makes me uneasy | 15:47 |
*** devvesa has quit IRC | 15:47 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Skip DBDuplicateEntry exception in security group creation https://review.openstack.org/155789 | 15:48 |
*** a_le has quit IRC | 15:48 | |
ihrachyshka | markmcclain, it won't be hard to split neutron/cmd/ into two parts, one patched and one unpatched, later if we want to do it | 15:49 |
markmcclain | ihrachyshka: it's more that we end of moving option definition out of the location where they are used | 15:49 |
*** xgerman has joined #openstack-neutron | 15:49 | |
*** tflynn has quit IRC | 15:49 | |
markmcclain | not certain I want to repeat the anti-pattern we have in common/constants | 15:49 |
*** tflynn has joined #openstack-neutron | 15:50 | |
ihrachyshka | markmcclain, that's why I ask whether putting dumb main()s under neutron/cmd/... instead would be a better options | 15:50 |
markmcclain | except what do you dumb mains() avoid? three lines of explict monkeypatching? | 15:51 |
*** zzzeek has joined #openstack-neutron | 15:51 | |
*** rdo has quit IRC | 15:51 | |
markmcclain | s/you/the/ | 15:51 |
* markmcclain needs more coffee this morning | 15:51 | |
*** maishsk has quit IRC | 15:51 | |
*** a_le has joined #openstack-neutron | 15:52 | |
*** pal has joined #openstack-neutron | 15:52 | |
ihrachyshka | markmcclain, it avoids loosing monkey patch randomly during refactoring. it also makes sure we monkey patch before any other imports. | 15:52 |
ihrachyshka | markmcclain, we got monkey_patch() lost before during l3 agent refactoring | 15:52 |
ihrachyshka | markmcclain, https://bugs.launchpad.net/neutron/+bug/1417386 | 15:53 |
openstack | Launchpad bug 1417386 in neutron "AttributeError: _oslo_messaging_localcontext errors found in neutron l3-agent logs" [High,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 15:53 |
*** rdo has joined #openstack-neutron | 15:53 | |
*** Marga_ has quit IRC | 15:53 | |
*** r-daneel has joined #openstack-neutron | 15:54 | |
*** manishg has quit IRC | 15:54 | |
ihrachyshka | markmcclain, also see https://review.openstack.org/#/c/154642/1/specs/eventlet-best-practices.rst for some context on how to use eventlet in sane way. | 15:55 |
*** ajmiller has joined #openstack-neutron | 15:56 | |
markmcclain | right.. that's in my review queue | 15:57 |
*** marun has joined #openstack-neutron | 15:57 | |
ihrachyshka | markmcclain, ok, I think I will now stop bothering you too much and allow you to do some actual work. :) thanks for the chat. | 15:58 |
roaet | how can I search review.openstack for the change Merge "Split base function of v2_0.Client into a separate class"? I am getting 'invalid operators' in the search area | 15:59 |
amotoki | ihrachyshka: let me think a bit more tomorrow. I started to think thim main() looks better, but I had beer tonight and am going to bed. | 16:00 |
ihrachyshka | amotoki, enjoy! no rush :) | 16:00 |
roaet | nevermind was searching wrong project D: | 16:00 |
*** sdague has quit IRC | 16:00 | |
ihrachyshka | amotoki, you can also reply to corresponding thread in openstack-dev@ | 16:00 |
*** tiswanso has joined #openstack-neutron | 16:01 | |
amotoki | ihrachyshka: sure. I just remember you sent a mail to the dev list. | 16:01 |
*** baoli has quit IRC | 16:01 | |
*** marun has quit IRC | 16:01 | |
roaet | amotoki: Ah you leaving? I was wondering if the changes to https://review.openstack.org/#/c/148318/ would suffice, but I understand you're probably sleepy heh. | 16:01 |
*** sdague has joined #openstack-neutron | 16:02 | |
roaet | it's failing jenkins... somehow, but I'll figure it out. silly rebases | 16:02 |
*** achanda has joined #openstack-neutron | 16:05 | |
amotoki | roaet: yeah, leaving now. Let me check tomorrow or the other day. | 16:06 |
roaet | amotoki: it's cool. I'll fix the UT, I appreciate your time. | 16:06 |
amotoki | roaet: re: another question from you "how can I search..." what is the problem? | 16:06 |
*** achanda has quit IRC | 16:06 | |
roaet | amotoki: I figured it out, thank you. I was searching the wrong project. I wanted to search by 'commit title' but merged+project was fine. | 16:07 |
amotoki | roaet: I think the easiest way is to input CHnage-ID. | 16:07 |
*** tflynn has quit IRC | 16:07 | |
roaet | amotoki: i know the commit is yours, but I'm getting failures on SSL+client. Trying to solve the UT fails. | 16:08 |
*** jorgem has joined #openstack-neutron | 16:08 | |
amotoki | roaet: thanks. sorry if there is some actual failure. | 16:09 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS Enable coverage testing for functional tests https://review.openstack.org/155889 | 16:09 |
roaet | amotoki: I will debug. I am sure it's just randomness. | 16:09 |
*** MarkB1 has joined #openstack-neutron | 16:10 | |
*** armax has joined #openstack-neutron | 16:11 | |
*** reed has joined #openstack-neutron | 16:11 | |
*** amotoki has quit IRC | 16:13 | |
*** roeyc has quit IRC | 16:14 | |
*** Sam______ has joined #openstack-neutron | 16:14 | |
Sam______ | Hi neutron people ! | 16:14 |
Sam______ | I'm looking for a way to get all not assigned Ip of a neutron network | 16:14 |
Sam______ | is that possible plz ? | 16:15 |
Sam______ | I want to select an Ip according to certain fact, and make a nova boot with a fixed IP | 16:15 |
Sam______ | but I'm unable to find the way to retrieve all free ip ... | 16:15 |
openstackgerrit | Justin Hammond proposed openstack/python-neutronclient: Client command extension support https://review.openstack.org/148318 | 16:17 |
*** aranjan has joined #openstack-neutron | 16:17 | |
enikanorov__ | Sam______: you may inspect target subnet and already allocated ports | 16:21 |
enikanorov__ | that should give you a range of available IPs, you need to exclude allocated | 16:21 |
*** tflynn has joined #openstack-neutron | 16:22 | |
*** aranjan has quit IRC | 16:22 | |
*** changbl has joined #openstack-neutron | 16:26 | |
Sam______ | enikanorov__: thanks, I was wondering if there is a wrapper in nutron for that like neutron subnet <my_subnet> get-free-ip | 16:28 |
*** ramishra_ has joined #openstack-neutron | 16:31 | |
prometheanfire | so.... about parsing intface names and us sucking at it... | 16:33 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: Fallback to new l3 agent main() location https://review.openstack.org/156602 | 16:33 |
prometheanfire | I'm working on getting interface names properly, not via iproute2 since it's next to imposible with all the interface types and all | 16:33 |
prometheanfire | here's the changeset currently https://review.openstack.org/#/c/154128/4 | 16:33 |
prometheanfire | I want to just poll sysfs for it instead | 16:34 |
prometheanfire | I asked the iproute2 devs how we should pull interface names from them and they said we shouldn't (something I also heavilly agree with) | 16:34 |
prometheanfire | http://marc.info/?l=linux-netdev&m=142418664816389 | 16:34 |
prometheanfire | so, what should be the method we pull names with? | 16:34 |
prometheanfire | the current method in that changeset doesn't work with macvlan, but that could be fixed easilly | 16:36 |
prometheanfire | but it's always going to be another thing | 16:36 |
haleyb | prometheanfire: i think there was a change recently to lok in /sys/class/net for tap devices, https://review.openstack.org/#/c/154021/ so perhaps that is best? | 16:37 |
*** SridharRamaswamy has joined #openstack-neutron | 16:37 | |
*** seizadi has joined #openstack-neutron | 16:38 | |
haleyb | i think that was something shemminger mentioned | 16:38 |
*** nlahouti has joined #openstack-neutron | 16:38 | |
*** gridinv has quit IRC | 16:39 | |
prometheanfire | ya, I generally prefer to look in sys for this type of thing | 16:40 |
prometheanfire | I wanted to do it for another change as well but went another way (we were using modinfo, which didn't work for built ins) | 16:41 |
prometheanfire | ya, that change is good | 16:41 |
haleyb | i just wonder what in ip_lib would need to change | 16:41 |
*** manishg has joined #openstack-neutron | 16:41 | |
prometheanfire | neutron/agent/linux/ip_lib.py | 16:41 |
*** ronis has quit IRC | 16:41 | |
prometheanfire | 101 class IPWrapper(SubProcessBase): | 16:42 |
haleyb | yes, the evil parser of /sbin/ip :) | 16:42 |
prometheanfire | 110 def get_devices(self, exclude_loopback=False): | 16:42 |
openstackgerrit | enikanorov proposed openstack/neutron: Unify logic that determines liveliness of DHCP agent https://review.openstack.org/152891 | 16:42 |
prometheanfire | since we are just getting info in that method it's easier | 16:42 |
*** tflynn has quit IRC | 16:42 | |
prometheanfire | and it fixes a single bug | 16:42 |
*** shaohe_feng has quit IRC | 16:45 | |
*** pal has quit IRC | 16:47 | |
*** ihrachyshka has quit IRC | 16:49 | |
*** Marga_ has joined #openstack-neutron | 16:49 | |
*** coolsvap is now known as coolsvap_ | 16:51 | |
*** devvesa_ has quit IRC | 16:52 | |
*** rwsu has joined #openstack-neutron | 16:52 | |
anteaya | obondarev: have you considered venturing into the nova channel? | 16:54 |
anteaya | obondarev: and talking to the nova devs? | 16:54 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Get rid of rpc to fetch fip agent port on agent. https://review.openstack.org/153735 | 16:56 |
*** nivtuckio has joined #openstack-neutron | 16:57 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fix FIP agent gw port delete based on external net https://review.openstack.org/145859 | 16:57 |
amuller | armax: Can you take a look at https://review.openstack.org/#/c/155813/ again? | 16:58 |
amuller | armax: Only change since Gary +2'd and Rajeev and Swami +1'd is that I simplified the functional tests and added some more info the commit message, no changes to the actual agent fix | 16:58 |
amuller | armax: I'd like to merge that today if possible so that my VMs could stop crashing every time I forget not to run that one test =D | 16:59 |
*** carlp has joined #openstack-neutron | 16:59 | |
*** mlavalle has joined #openstack-neutron | 17:00 | |
*** scheuran has quit IRC | 17:00 | |
*** simon-AS559 has joined #openstack-neutron | 17:00 | |
*** Aish has quit IRC | 17:01 | |
*** Guest31726 is now known as dank_ | 17:01 | |
*** matrohon has quit IRC | 17:01 | |
*** claudiub_ has joined #openstack-neutron | 17:04 | |
*** marun has joined #openstack-neutron | 17:04 | |
prometheanfire | is it odd to -1 your own changeset | 17:04 |
prometheanfire | ? | 17:04 |
amuller | prometheanfire: not at all | 17:05 |
*** baoli has joined #openstack-neutron | 17:05 | |
amuller | prometheanfire: you could also mark as WIP and explain what's left to do in the commit message | 17:05 |
haleyb | marking WIP will also stop it from merging accidentally | 17:06 |
*** ivar-lazzaro has joined #openstack-neutron | 17:06 | |
*** claudiub has quit IRC | 17:07 | |
*** ivar-lazzaro has quit IRC | 17:09 | |
*** ivar-lazzaro has joined #openstack-neutron | 17:10 | |
*** evgenyf has quit IRC | 17:10 | |
*** ramishra_ has quit IRC | 17:10 | |
*** emagana_ has quit IRC | 17:10 | |
*** _cjones_ has joined #openstack-neutron | 17:10 | |
*** ygbo has quit IRC | 17:11 | |
*** julien_ZTE has quit IRC | 17:11 | |
*** julien_ZTE has joined #openstack-neutron | 17:12 | |
*** julien_ZTE has quit IRC | 17:12 | |
*** jlanoux has quit IRC | 17:12 | |
*** julien_ZTE has joined #openstack-neutron | 17:13 | |
*** changbl has quit IRC | 17:13 | |
*** pask81 has quit IRC | 17:15 | |
*** simon-AS5591 has joined #openstack-neutron | 17:15 | |
*** julien_ZTE has quit IRC | 17:17 | |
*** simon-AS559 has quit IRC | 17:18 | |
openstackgerrit | garyk proposed openstack/neutron: Fix parameters in exception handling https://review.openstack.org/156590 | 17:18 |
*** moshele has quit IRC | 17:20 | |
*** shilpa_ has quit IRC | 17:20 | |
*** aepifanov has quit IRC | 17:20 | |
*** belmoreira has quit IRC | 17:21 | |
*** MarkB1 has quit IRC | 17:21 | |
*** simon-AS5591 has left #openstack-neutron | 17:21 | |
openstackgerrit | Brent Eagles proposed openstack/neutron: Make shallow copy of attribute map a little deeper https://review.openstack.org/156670 | 17:21 |
*** doude has quit IRC | 17:21 | |
*** a_le has quit IRC | 17:22 | |
*** jorgem has quit IRC | 17:23 | |
*** carl_baldwin has quit IRC | 17:23 | |
*** tflynn has joined #openstack-neutron | 17:23 | |
*** jistr has quit IRC | 17:24 | |
*** jaypipes has joined #openstack-neutron | 17:26 | |
*** Networkn3rd has quit IRC | 17:26 | |
*** dboik_ has joined #openstack-neutron | 17:28 | |
*** jobewan has joined #openstack-neutron | 17:29 | |
*** annegent_ has joined #openstack-neutron | 17:29 | |
*** dboik_ has quit IRC | 17:29 | |
*** dboik_ has joined #openstack-neutron | 17:30 | |
*** Aish has joined #openstack-neutron | 17:31 | |
*** tflynn has quit IRC | 17:32 | |
*** SridharG1 has joined #openstack-neutron | 17:32 | |
*** dboik has quit IRC | 17:32 | |
*** SridharG has quit IRC | 17:32 | |
*** a_le has joined #openstack-neutron | 17:35 | |
*** SridharG has joined #openstack-neutron | 17:36 | |
*** SridharG1 has quit IRC | 17:36 | |
*** suresh12 has joined #openstack-neutron | 17:37 | |
*** nyechiel_ has quit IRC | 17:37 | |
*** SridharG has left #openstack-neutron | 17:40 | |
*** jprovazn has joined #openstack-neutron | 17:42 | |
*** pal has joined #openstack-neutron | 17:45 | |
*** cipcosma has quit IRC | 17:45 | |
*** thedodd has joined #openstack-neutron | 17:46 | |
*** dboik_ has quit IRC | 17:47 | |
*** jorgem has joined #openstack-neutron | 17:47 | |
*** dboik has joined #openstack-neutron | 17:47 | |
*** SridharRamaswamy has quit IRC | 17:50 | |
*** cipcosma has joined #openstack-neutron | 17:50 | |
*** rotbeard has joined #openstack-neutron | 17:50 | |
*** carl_baldwin has joined #openstack-neutron | 17:52 | |
*** marrusl has quit IRC | 17:54 | |
pc_m | dougwig: ping | 17:54 |
*** carl_baldwin_ has joined #openstack-neutron | 17:55 | |
*** carl_baldwin has quit IRC | 17:56 | |
*** carl_baldwin_ is now known as carl_baldwin | 17:56 | |
*** tflynn has joined #openstack-neutron | 17:56 | |
*** Aish has left #openstack-neutron | 17:58 | |
pc_m | Hi folks! I'm seeing some odd things with cross project dependencies and am wondering if I'm missing some steps, can anyone advise? | 17:58 |
anteaya | pc_m: what are you seeing? | 17:58 |
anteaya | and since they are new it might be a bug | 17:58 |
*** yamamoto has quit IRC | 17:59 | |
*** anish_ has quit IRC | 17:59 | |
pc_m | Here's the scenario... I have the latest neutron code on my machine. In the neutron-vpnaas repo, I patch in (git review -d) code that is out for review. | 17:59 |
anteaya | which patch? | 17:59 |
*** jschwarz has joined #openstack-neutron | 18:00 | |
*** ronis has joined #openstack-neutron | 18:00 | |
pc_m | When I run UTs, they fail because the patch set is not up to date with the latest. | 18:00 |
pc_m | 144391 | 18:00 |
*** wverdugo500 has joined #openstack-neutron | 18:00 | |
pc_m | However, for the developer, the patch set is working. I think he has an older neutron commit. | 18:00 |
anteaya | so your question is about your unit test failure? | 18:01 |
pc_m | So, that had me wondering how one can make sure they have the two in sync. | 18:01 |
pc_m | anteaya: I was able to solve the unit test failure, by rebasing. What is disturbing, is that the python27 passes in the review | 18:02 |
anteaya | let's start again | 18:02 |
*** aepifanov has joined #openstack-neutron | 18:02 | |
*** dsneddon has joined #openstack-neutron | 18:03 | |
anteaya | so far this has nothing to do with cross repo dependencies | 18:03 |
*** tflynn has quit IRC | 18:03 | |
pc_m | If it had the latest neutron, it should have failed | 18:03 |
anteaya | so for starters what is the exact command you ran to get the patch to your local environment? | 18:03 |
pc_m | OK. Let me try to replay this scenario. I'm not being clear... | 18:03 |
anteaya | I'm trying to reproduce | 18:04 |
pc_m | Code revuew 144391 passes python27 in Jenkins. | 18:04 |
anteaya | what is the exact git review -d command you used? | 18:04 |
pc_m | On my system, I did a update of my neutron repo to match latest upstream. | 18:04 |
pc_m | In the VPN repo, I did 'git review -d 144391' | 18:04 |
pc_m | When I run 'tox -e py27', the tests fail. | 18:05 |
pc_m | The failure is because of a change that was made in neutron, and was missing a change in neutron-vpnaas. Commit 1fed46b2 fixes that. | 18:06 |
pc_m | If I rebased this neutron-vpnaas view with latest upstream, the code gets the commit for 1fed46b2 and tests pass. | 18:06 |
*** yamahata has joined #openstack-neutron | 18:06 | |
pc_m | Questions are... why is the Jenkins test passing (and it is working in the committer repo AFIK)? | 18:07 |
pc_m | If there is a change in our local neutron repo, how do we make sure that our local neutron-vpnaas is indeed picking up the change? | 18:08 |
pc_m | anteaya: That's what I'm wondering. Does that make sense (the questions at least)? | 18:08 |
anteaya | I'm trying to follow you | 18:08 |
anteaya | I find when we talk you are usually way ahead of me using different words that I would use for things | 18:09 |
anteaya | so I am trying to catch up to you | 18:09 |
jamielennox | hi mestery, any word on that neutronclient release? | 18:09 |
pc_m | My guess on the second question is that I need to run setup.py install in the neutron directory so that the neutron-vpnaas gets the updated egg | 18:09 |
anteaya | let's deal with one thing at a time | 18:09 |
*** jpena has quit IRC | 18:09 | |
anteaya | on gerrit | 18:09 |
*** ZZelle_ has joined #openstack-neutron | 18:09 | |
pc_m | anteaya: sorry, I'm probably using the wrong terms. | 18:10 |
mestery | jamielennox: I'm awaiting one more patch which I hope will merge today/tomorrow, then I'll cut it. Sound ok? | 18:10 |
openstackgerrit | Terry Wilson proposed openstack/neutron-vpnaas: Stop storing and passing root_helper https://review.openstack.org/156690 | 18:10 |
openstackgerrit | Terry Wilson proposed openstack/neutron-vpnaas: Remove remaining root_helper references https://review.openstack.org/156691 | 18:10 |
pc_m | anteaya: sure...go ahead... | 18:10 |
anteaya | pc_m: so when neutron master changes, does neutron-vpnaas pick that up? | 18:10 |
jamielennox | mestery: yep - that's fine, just checking you hadn't forgotten - thanks | 18:10 |
mestery | jamielennox: No worries, and thanks for following up! | 18:10 |
anteaya | pc_m: for the neutron-vpnaas gerrit patches? | 18:10 |
*** harlowja_away is now known as harlowja_ | 18:10 | |
*** carl_baldwin has quit IRC | 18:11 | |
openstackgerrit | Terry Wilson proposed openstack/neutron-fwaas: Stop storing and passing root_helper https://review.openstack.org/156692 | 18:11 |
*** emagana has joined #openstack-neutron | 18:12 | |
pc_m | anteaya: Yeah, I guess that is the question. Neutron had a change, which broke VPN repo (not exactly sure when that happened, but the fix on neutron-vpnaas went in on 2/12). | 18:13 |
anteaya | as of right now | 18:13 |
anteaya | do neutron-vpnaas patches incorporate lastest neutron master for testing? | 18:13 |
pc_m | anteaya: With patch set 35 of 144391, it went up on 2/15 (appears to not have picked up that change). | 18:13 |
*** ihrachyshka has joined #openstack-neutron | 18:13 | |
roaet | anyone run into 'str' does not support the buffer interface when installing sphinx into py34 tox env? | 18:13 |
*** annegent_ has quit IRC | 18:14 | |
pc_m | anteaya: I'm not sure exactly how that works. It uses an egg file | 18:14 |
*** carl_baldwin has joined #openstack-neutron | 18:14 | |
anteaya | so if neutron-vpnaas patches are not picking up changes to neutron master then that is something that needs to be addressed | 18:14 |
anteaya | I am not sure either, who wrote the tests | 18:14 |
pc_m | anteaya: The unit tests? they've been around for a long time. | 18:15 |
*** emagana has quit IRC | 18:15 | |
anteaya | as for your local neutron-vpnaas knowing about a local neutron change, that isn't something that gerrit can control | 18:15 |
anteaya | the set up in the tests for neutron-vpnaas to depend on neutron | 18:15 |
pc_m | anteaya: sure. Just wondering if setup.py install is the way to sync those locally. | 18:16 |
*** jbonjean has quit IRC | 18:16 | |
anteaya | once the service was split out, neutron-vpnaas needed to know about neutron to consume it, yes? | 18:16 |
anteaya | pc_m: let's work on one issue at a time | 18:16 |
otherwiseguy | pc_m: anteaya: btw, my root_helper patches could potentially break some of the *aaS stuff. I've gone through and tried to get the "fix" patches put up for review with Depends-On headers. I think we should be fine as long as we follow the depends. | 18:16 |
*** MentalRay has joined #openstack-neutron | 18:16 | |
*** jbonjean has joined #openstack-neutron | 18:17 | |
anteaya | because if the way your zuul tests are set up doesn't incorporate latest neutron master, that needs to be addressed | 18:17 |
otherwiseguy | In any case, once those start merging, I'll definitely be around for doing in quick fixups that need to be done. | 18:17 |
anteaya | otherwiseguy: thanks for jumping in | 18:17 |
pc_m | otherwiseguy: That's what happened in one case. root_helper was changed in neutron, but not in neutron_vpnaas. However, a fix went into neutron-vpnaas for that on 1/12. | 18:18 |
*** moshele has joined #openstack-neutron | 18:18 | |
*** achanda has joined #openstack-neutron | 18:18 | |
*** thedodd has quit IRC | 18:18 | |
pc_m | otherwiseguy: What's confusing me, is that there is a commit to neutron-vpnaas with a patch on 1/15 that passes, but does not have the VPN side fix. | 18:18 |
otherwiseguy | pc_m: at least after I kill root_helper entirely that particular change shouldn't cause breakage in the future. :) | 18:18 |
otherwiseguy | (and by kill, I mean kill the passing of it everywhere) | 18:19 |
*** kawa2014 has quit IRC | 18:19 | |
pc_m | otherwiseguy: So we have review 144391 with patch set 35 on 2/15, which should have the fix from 2/12, but does not. However, it passes Jenkins UT. | 18:20 |
*** leenheer has joined #openstack-neutron | 18:21 | |
*** jbonjean has quit IRC | 18:21 | |
* pc_m I meant 2/12 on the date that the fix went in above (not 1/12) | 18:21 | |
*** SridharRamaswamy has joined #openstack-neutron | 18:22 | |
*** jp_at_hp has quit IRC | 18:22 | |
*** julien_ZTE has joined #openstack-neutron | 18:22 | |
pc_m | otherwiseguy: anteaya: I could image that the developer maybe forgot a rebase on neutron-vpnaas repo, but am wondering if that should have been detected, and why the jenkins passed for UT (gate-neutron-vpnaas-python27) | 18:22 |
otherwiseguy | pc_m: I'm currently on a conference call, so my attention is a bit divided right now. :) | 18:23 |
pc_m | otherwiseguy: understood. | 18:23 |
pc_m | otherwiseguy: me too :) | 18:23 |
*** nlahouti has quit IRC | 18:24 | |
*** changbl has joined #openstack-neutron | 18:24 | |
anteaya | okay | 18:25 |
anteaya | so it is possible this is a legitimate bug in dependencies | 18:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add callbacks-based system to Neutron https://review.openstack.org/153774 | 18:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation https://review.openstack.org/155470 | 18:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during create/update operations https://review.openstack.org/153775 | 18:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 18:25 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 base and VPN service plugins during router operations https://review.openstack.org/153776 | 18:25 |
*** nlahouti has joined #openstack-neutron | 18:25 | |
anteaya | it is also possible that neutron-vpnaas is not testing off of latest neutron master | 18:25 |
anteaya | so let's confirm that neutron-vpnaas tests use latest neutron master when testing | 18:26 |
pc_m | anteaya: That's what I'm worried about (not using latest neutron for some resone) | 18:26 |
anteaya | can we agree on this is a first place to being | 18:26 |
anteaya | okay | 18:26 |
pc_m | anteaya: That would be great | 18:26 |
anteaya | so can you find the code that sets up the tests? | 18:26 |
anteaya | it should be in openstack-infra/project-config | 18:26 |
pc_m | OK. I'll look | 18:27 |
*** jlibosva has quit IRC | 18:28 | |
*** dane_leblanc_ has joined #openstack-neutron | 18:30 | |
*** saju_m has joined #openstack-neutron | 18:30 | |
anteaya | thanks | 18:30 |
anteaya | let me know how you do | 18:30 |
*** jbonjean has joined #openstack-neutron | 18:31 | |
anteaya | I'd suggest zuul/layout.yaml and jenkins/project.yaml for starters | 18:31 |
*** irenab has joined #openstack-neutron | 18:31 | |
*** julien_ZTE has quit IRC | 18:31 | |
dougwig | pc_m, anteaya - yes, there is a definite wart that currently neutron/neutron-*aas are testing master of each other, and having either out of date puts you in undefined land. | 18:32 |
*** carl_baldwin has quit IRC | 18:32 | |
dougwig | pc_m, anteaya - i'm floating this: https://review.openstack.org/#/c/154736/ as the ultimate solution, which is to get rid of this cross-repo intermingling. | 18:32 |
*** dane_leblanc has quit IRC | 18:33 | |
dougwig | we don't so much have cross-repo dependencies, but rather dangerously co-mingled internal interfaces. | 18:33 |
anteaya | that sounds like the phrase I was looking for | 18:33 |
pc_m | dougwig: So in this case, it looks like a neutron-vpnaas commit, which is not up to date, is using a not up-to-date neutron. | 18:34 |
*** moshele has quit IRC | 18:34 | |
*** dane_leblanc_ has quit IRC | 18:34 | |
*** pal has quit IRC | 18:35 | |
*** aranjan has joined #openstack-neutron | 18:35 | |
pc_m | dougwig: If it was using latest master on neutron, I would have thought it would have failed. Confused to say the least. | 18:37 |
*** jschwarz has quit IRC | 18:37 | |
*** tomoe_ has joined #openstack-neutron | 18:38 | |
dougwig | so, we need to be testing master, but re-used tox environments let neutron get stale. which will over get worse over time. hmm. | 18:38 |
dougwig | is there a way to make tox do the moral equivalent of 'git pull' on the neutron egg regularly? | 18:39 |
*** spandhe has joined #openstack-neutron | 18:40 | |
*** devlaps has joined #openstack-neutron | 18:40 | |
*** martingc_ is now known as martingc | 18:40 | |
pc_m | dougwig: So are you saying that Jenkins is reusing the tox environment? | 18:40 |
dougwig | pc_m: no, jenkins does not. it starts fresh every time. | 18:40 |
dougwig | but un-rebased patches may not have lockstep-necessary changes. | 18:41 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fix FIP agent gw port delete based on external net https://review.openstack.org/145859 | 18:41 |
*** ramishra_ has joined #openstack-neutron | 18:41 | |
pc_m | dougwig: So I'm confused on why PS 35 passed jenkins with out of date vpn code | 18:41 |
*** tomoe_ has quit IRC | 18:42 | |
openstackgerrit | Brent Eagles proposed openstack/neutron: Make shallow copy of attribute map a little deeper https://review.openstack.org/156670 | 18:43 |
pc_m | dougwig: I would have thought neutron would have the latest code (with the change to root_helper), and this patch set does not have your 2/12 fix, so it should have failed, but it passed. | 18:44 |
anteaya | dougwig: where is the code that sets up the tests now for neutron-vpnaas? | 18:44 |
anteaya | the part that says use this neutron egg | 18:45 |
pc_m | dougwig: I did a "git review -d 144391" and it does not have the changes you made. | 18:45 |
*** jasondotstar has joined #openstack-neutron | 18:45 | |
pc_m | dougwig: and once I rebased in my area, everything was fine (it had the changes and passed the tests). | 18:45 |
*** ramishra_ has quit IRC | 18:46 | |
dougwig | anteaya: in tox.ini, when it's setting up dependencies. the *aas repos have a special line for including neutron. | 18:47 |
*** moha_hunt has joined #openstack-neutron | 18:47 | |
dougwig | ok, step back, i'd have to look at the code in question to see why that instance worked. but, i'm not sure it's worth the time, since there is a greater problem identified, it will continue to plague us, and the only way out of it is to break this co-mingled dance. IMO. | 18:48 |
pc_m | dougwig: So that is taking latest from git.openstack.org for neutron? | 18:48 |
anteaya | http://git.openstack.org/cgit/openstack/neutron-vpnaas/tree/tox.ini#n13 | 18:48 |
anteaya | so an egg is an archive yes? | 18:48 |
*** s3wong has joined #openstack-neutron | 18:48 | |
anteaya | some kind of package | 18:48 |
anteaya | anyone know how often it gets created? | 18:49 |
pc_m | anteaya: yeah as far as I know. | 18:49 |
pc_m | anteaya: Wondering if it is not getting updated.... | 18:49 |
anteaya | doesn't seem that it is | 18:49 |
pc_m | anteaya: Right. | 18:49 |
*** devlaps has quit IRC | 18:49 | |
pc_m | anteaya: Should the whole gating process, when a commit is approved, update the egg? | 18:50 |
*** jkyle1 has quit IRC | 18:50 | |
anteaya | sounds like that is what is happening | 18:51 |
anteaya | so somehow the dependency doesn't contain the latest code though | 18:51 |
anteaya | do we have a clear example of that? | 18:51 |
*** irenab has quit IRC | 18:51 | |
dougwig | anteaya: the egg is what setup.py/setup.cfg builds. | 18:51 |
anteaya | awesome | 18:52 |
dougwig | pc_m: gate gets fresh eggs, since it starts with bare vms. | 18:52 |
dougwig | and now i'm getting hungry. | 18:52 |
dougwig | anyone want breakfast? | 18:52 |
anteaya | so what happened on the out of date code? | 18:52 |
* anteaya likes that idea | 18:52 | |
anteaya | 'cept I have already had some | 18:52 |
anteaya | lunch is what I need next | 18:52 |
*** dane_leblanc has joined #openstack-neutron | 18:54 | |
*** carl_baldwin has joined #openstack-neutron | 18:54 | |
pc_m | dougwig: hmm. I'm at a loss as to why this job is passing for the patch set. | 18:55 |
*** stefanb has quit IRC | 18:55 | |
*** MentalRay has quit IRC | 18:55 | |
roaet | Meh. I can't figure out why my unit test is failing. Anyone noticing any issues with Security Groups' test_list_*_extend_* tests on py34? | 18:57 |
armax | roaet: looking | 18:57 |
roaet | armax: http://logs.openstack.org/18/148318/12/check/gate-python-neutronclient-python34/a133ec4/ | 18:57 |
roaet | if that helps | 18:57 |
roaet | it's neutronclient either way. | 18:58 |
*** dboik_ has joined #openstack-neutron | 18:58 | |
*** dboik_ has quit IRC | 19:01 | |
banix | roaet: the recheck passed | 19:01 |
*** dane_leblanc has quit IRC | 19:01 | |
roaet | Ah.. sweet. banix thanks. I think there is a problem then on the SG tests. | 19:01 |
*** tiswanso has quit IRC | 19:01 | |
roaet | I think it failed due to the querystring being out of order. | 19:01 |
*** dboik_ has joined #openstack-neutron | 19:01 | |
*** dboik has quit IRC | 19:01 | |
roaet | fields=name&id=myid1&id=myid3&id=myid2 vs fields=name&id=myid1&id=myid2&id=myid3 | 19:02 |
*** tiswanso_ has joined #openstack-neutron | 19:02 | |
roaet | I guess I can report that | 19:02 |
*** SridharRamaswam1 has joined #openstack-neutron | 19:04 | |
banix | roaet: sounds like it | 19:04 |
*** ihrachyshka has quit IRC | 19:04 | |
*** SridharRamaswamy has quit IRC | 19:05 | |
*** dane_leblanc has joined #openstack-neutron | 19:07 | |
*** stefanb has joined #openstack-neutron | 19:10 | |
*** ajo has quit IRC | 19:13 | |
*** annegent_ has joined #openstack-neutron | 19:15 | |
*** jprovazn has quit IRC | 19:16 | |
*** irenab has joined #openstack-neutron | 19:17 | |
*** openstackgerrit has quit IRC | 19:20 | |
*** tflynn has joined #openstack-neutron | 19:20 | |
*** annegent_ has quit IRC | 19:20 | |
*** openstackgerrit has joined #openstack-neutron | 19:20 | |
*** marun_ has joined #openstack-neutron | 19:21 | |
*** ajo has joined #openstack-neutron | 19:22 | |
*** marun has quit IRC | 19:23 | |
*** markmcclain has quit IRC | 19:24 | |
*** markmcclain has joined #openstack-neutron | 19:24 | |
*** julien_ZTE has joined #openstack-neutron | 19:28 | |
*** marun_ is now known as marun | 19:39 | |
*** annegent_ has joined #openstack-neutron | 19:40 | |
*** julien_ZTE has quit IRC | 19:42 | |
openstackgerrit | Merged openstack/neutron: Delete qg device during DVR-SNAT router deletion https://review.openstack.org/155813 | 19:42 |
*** ramishra_ has joined #openstack-neutron | 19:42 | |
*** ramishra_ has quit IRC | 19:47 | |
prometheanfire | so, what do you think of this? https://gist.github.com/prometheanfire/2fb592e3eaa699f55f89 | 19:47 |
prometheanfire | haven't changed the tests yet, but it'll fix errors methinks :P | 19:48 |
roaet | banix: is there a way to get the recheck on my change? | 19:48 |
roaet | (as it is still showing -1) | 19:48 |
*** a_le has quit IRC | 19:49 | |
banix | roaet: it hasn’t finished; i think it will finish shortly | 19:50 |
banix | roaet: http://status.openstack.org/zuul/ | 19:50 |
*** nivtuckio has quit IRC | 19:50 | |
roaet | banix: oh! i forgot all about that. Thanks again. | 19:51 |
*** a_le has joined #openstack-neutron | 19:51 | |
banix | roaet: python34 which had failed, passed a while back and that’s why i said the rechek has passed | 19:52 |
roaet | banix: yeah. I thought it'd only check failed ones. but yea | 19:52 |
roaet | Very helpful reminder. thank you :) | 19:53 |
banix | roaet: np | 19:53 |
openstackgerrit | Manish Godara proposed openstack/neutron: Add filter for provider network attributes in ML2 https://review.openstack.org/124917 | 19:54 |
*** irenab has quit IRC | 19:57 | |
*** a_le has quit IRC | 20:02 | |
*** a_le has joined #openstack-neutron | 20:02 | |
*** SridharRamaswam1 has quit IRC | 20:03 | |
*** Marga_ has quit IRC | 20:03 | |
*** MentalRay has joined #openstack-neutron | 20:04 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from DHCP agent https://review.openstack.org/155188 | 20:08 |
rdo | hi all, anyone know of a way to manage the default security group? i.e. setting the template? I want to ensure that every tenant I create gets, say, port 22 open by default | 20:08 |
*** SridharRamaswamy has joined #openstack-neutron | 20:09 | |
pc_m | dougwig: ping | 20:11 |
*** tiswanso_ has quit IRC | 20:13 | |
*** rushil_ has joined #openstack-neutron | 20:14 | |
*** aranjan has quit IRC | 20:15 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from L3 Agent https://review.openstack.org/155212 | 20:15 |
*** rushil has quit IRC | 20:15 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from linux interface https://review.openstack.org/155522 | 20:15 |
*** aranjan has joined #openstack-neutron | 20:16 | |
*** _cjones_ has quit IRC | 20:16 | |
*** _cjones_ has joined #openstack-neutron | 20:16 | |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove remaining root_helper args from plugins https://review.openstack.org/155523 | 20:17 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove root_helper arg from sanity checks https://review.openstack.org/155905 | 20:17 |
haleyb | marun: you made a change the other day for functional testing, right? I now see this error running tox, "ERROR: InvocationError: could not find executable 'dsvm-functional:'" and reverting your change fixes it | 20:17 |
haleyb | and i'm just running a unit test | 20:17 |
openstackgerrit | Terry Wilson proposed openstack/neutron: Remove remaining uses of passing root_helper https://review.openstack.org/155906 | 20:17 |
*** leenheer has quit IRC | 20:18 | |
*** annegent_ has quit IRC | 20:18 | |
*** jordandh has quit IRC | 20:19 | |
*** rpothier has quit IRC | 20:20 | |
*** jordandh has joined #openstack-neutron | 20:20 | |
*** jorgem has quit IRC | 20:21 | |
*** jorgem has joined #openstack-neutron | 20:22 | |
*** ronis has quit IRC | 20:25 | |
openstackgerrit | Fei Long Wang proposed openstack/neutron: Fix metering agent failure when chain missing https://review.openstack.org/156198 | 20:26 |
*** dboik_ has quit IRC | 20:28 | |
*** dboik has joined #openstack-neutron | 20:30 | |
*** leenheer has joined #openstack-neutron | 20:31 | |
*** aranjan has quit IRC | 20:32 | |
*** achanda has quit IRC | 20:36 | |
*** annegent_ has joined #openstack-neutron | 20:41 | |
*** tiswanso has joined #openstack-neutron | 20:42 | |
*** MentalRay has quit IRC | 20:42 | |
*** _cjones_ has quit IRC | 20:42 | |
*** csoukup has quit IRC | 20:42 | |
*** ramishra_ has joined #openstack-neutron | 20:43 | |
*** dboik has quit IRC | 20:43 | |
*** dboik has joined #openstack-neutron | 20:44 | |
*** dboik has quit IRC | 20:44 | |
amuller | haleyb: marun: Seeing the same symptom | 20:44 |
*** saju_m has quit IRC | 20:44 | |
amuller | marun: I think we're copying the rootwrap stuff for enviornments other than the functional one | 20:45 |
haleyb | amuller: seems marun is offline? i'm looking at it now | 20:45 |
*** dboik has joined #openstack-neutron | 20:45 | |
amuller | haleyb: it's done in tox.ini | 20:45 |
amuller | the new copy directive | 20:45 |
haleyb | yes, i'm assuming it should move lower in the file | 20:45 |
amuller | haleyb: the fix would be to make sure it happens only for the dsvm-functional env | 20:45 |
amuller | and possibly the functional env, I'm not sure about that | 20:45 |
*** smcquaid has joined #openstack-neutron | 20:47 | |
*** aranjan has joined #openstack-neutron | 20:47 | |
*** ramishra_ has quit IRC | 20:48 | |
*** Marga_ has joined #openstack-neutron | 20:48 | |
*** vinsh has quit IRC | 20:48 | |
*** csoukup has joined #openstack-neutron | 20:49 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes https://review.openstack.org/142863 | 20:50 |
*** smcquaid has quit IRC | 20:50 | |
*** jgrimm is now known as zz_jgrimm | 20:51 | |
*** annegent_ has quit IRC | 20:53 | |
*** cipcosma has quit IRC | 20:54 | |
*** suresh12 has quit IRC | 20:56 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Un-break tox for unit tests https://review.openstack.org/156767 | 20:57 |
haleyb | amuller: try that ^^ | 20:58 |
*** zz_jgrimm is now known as jgrimm | 20:58 | |
HenryG | haleyb: amuller: I think you folks might just need to get a newer tox? | 20:59 |
*** leenheer has quit IRC | 20:59 | |
haleyb | HenryG: well, when i revert that things are fine, and i'm on the one shipping with 14.04 | 20:59 |
HenryG | haleyb: what version is that? | 21:00 |
HenryG | tox --version | 21:00 |
haleyb | 1.6.0 imported from /usr/lib/python2.7/dist-packages/tox/__init__.pyc | 21:00 |
*** leenheer has joined #openstack-neutron | 21:00 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: WIP fix for _get_external_device_interface_name trace https://review.openstack.org/156357 | 21:00 |
*** localloop127 has quit IRC | 21:00 | |
haleyb | HenryG: this was a brand-new cloned repo too | 21:00 |
*** moshele has joined #openstack-neutron | 21:01 | |
HenryG | haleyb: understood, just noting that the feature in tox.ini works with tox 1.8.1 | 21:01 |
haleyb | which is probaly why it passed the gate | 21:02 |
clarkb | 1.6.0 had the broken parameter substituion iirc | 21:02 |
clarkb | let me double check that | 21:02 |
*** SridharRamaswamy has quit IRC | 21:03 | |
haleyb | i'll update the commit string to mention the version of tox i'm running, but i think my change would work for both | 21:03 |
clarkb | ah no that was 1.6.2 | 21:03 |
*** SridharRamaswamy has joined #openstack-neutron | 21:04 | |
haleyb | i suppose i could pip update too, but i shouldn't have to do that | 21:04 |
*** marun has quit IRC | 21:04 | |
HenryG | haleyb: Both marun and I tried {[testenv]commands} and it didn't work | 21:04 |
*** moshele has quit IRC | 21:05 | |
haleyb | HenryG: well it gets the unit tests working again :) i can duplicate the other line, but what is the correct fix - force a tox update on everyone? | 21:06 |
*** jorgem has quit IRC | 21:07 | |
*** suresh12 has joined #openstack-neutron | 21:08 | |
*** localloop127 has joined #openstack-neutron | 21:08 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Un-break tox for unit tests https://review.openstack.org/156767 | 21:09 |
*** _cjones_ has joined #openstack-neutron | 21:11 | |
*** jorgem has joined #openstack-neutron | 21:11 | |
HenryG | haleyb: The way it was before my oblivious comment caused Maru to mess it up for you. :) --> https://review.openstack.org/#/c/146113/31/tox.ini | 21:11 |
*** achanda has joined #openstack-neutron | 21:11 | |
openstackgerrit | Rajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources https://review.openstack.org/153422 | 21:11 |
*** achanda has quit IRC | 21:11 | |
HenryG | haleyb: You might want to replicate his inline comments from there. | 21:12 |
*** achanda has joined #openstack-neutron | 21:12 | |
*** matrohon has joined #openstack-neutron | 21:12 | |
haleyb | HenryG: thanks, i'll put the comments in | 21:12 |
*** watanabe_isao has joined #openstack-neutron | 21:14 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Un-break tox for unit tests https://review.openstack.org/156767 | 21:15 |
*** drjones has joined #openstack-neutron | 21:16 | |
*** _cjones_ has quit IRC | 21:16 | |
*** rushil_ has quit IRC | 21:17 | |
*** annegent_ has joined #openstack-neutron | 21:17 | |
*** aix has quit IRC | 21:20 | |
*** moshele has joined #openstack-neutron | 21:21 | |
*** suresh12 has quit IRC | 21:21 | |
*** Marga_ has quit IRC | 21:24 | |
*** Marga_ has joined #openstack-neutron | 21:25 | |
*** Marga_ has quit IRC | 21:25 | |
*** Marga_ has joined #openstack-neutron | 21:25 | |
haleyb | HenryG: guess the other option is to change that minversion line to 1.8, which is where that option was added. My "brand new" 14.04 VM does have 1.8.1, so it's just my old workstation (that i don't always to a pip update on) that's broken | 21:29 |
*** annegent_ has quit IRC | 21:29 | |
*** Sam______ has quit IRC | 21:29 | |
*** a_le has quit IRC | 21:30 | |
*** drjones has quit IRC | 21:31 | |
*** a_le has joined #openstack-neutron | 21:31 | |
*** _cjones_ has joined #openstack-neutron | 21:31 | |
HenryG | haleyb: That makes sense to me, but I don't want to cause pain | 21:32 |
HenryG | haleyb: I always run bleeding edge on all the things ;) | 21:32 |
haleyb | yes, bumping the version will cause someone else to scream when they don't have 1.8, then i'd be on the hot seat | 21:33 |
*** jorgem1 has joined #openstack-neutron | 21:33 | |
*** changbl has quit IRC | 21:33 | |
*** rotbeard has quit IRC | 21:34 | |
HenryG | haleyb: If things get hot just open a window :) | 21:34 |
*** mwagner_lap has quit IRC | 21:35 | |
*** jorgem has quit IRC | 21:35 | |
openstackgerrit | Matthew Thode proposed openstack/neutron: Fixes bug in interface handling of ip_lib.py https://review.openstack.org/154128 | 21:35 |
haleyb | HenryG: or go roll around in the 100 tons of snow in my yard :) | 21:36 |
HenryG | haleyb: tell me about it! It's a real winter this year. | 21:37 |
*** changbl has joined #openstack-neutron | 21:38 | |
haleyb | complete with great skiing :) | 21:39 |
prometheanfire | I'd like feedback on the switch to getting interface names from sysfs instead of trying to parse iproute2 if any of you have time https://review.openstack.org/#/c/154128/ | 21:41 |
Apsu | prometheanfire: +1 | 21:41 |
prometheanfire | always welcome :P | 21:42 |
*** rm_work is now known as rm_work|away | 21:44 | |
*** ramishra_ has joined #openstack-neutron | 21:44 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS Enable coverage testing for functional tests https://review.openstack.org/155889 | 21:46 |
*** ramishra_ has quit IRC | 21:49 | |
*** julien_ZTE has joined #openstack-neutron | 21:49 | |
*** aranjan has quit IRC | 21:49 | |
*** changbl has quit IRC | 21:51 | |
*** zzzeek has quit IRC | 21:51 | |
*** changbl has joined #openstack-neutron | 21:52 | |
*** aranjan has joined #openstack-neutron | 21:54 | |
*** aepifanov has quit IRC | 21:55 | |
*** vinsh has joined #openstack-neutron | 21:56 | |
*** banix has quit IRC | 21:57 | |
*** jordandh has quit IRC | 21:59 | |
*** jorgem1 has quit IRC | 22:00 | |
*** jordandh has joined #openstack-neutron | 22:01 | |
openstackgerrit | Merged openstack/neutron: Fix FIP agent gw port delete based on external net https://review.openstack.org/145859 | 22:01 |
*** leenheer has quit IRC | 22:02 | |
*** marun has joined #openstack-neutron | 22:02 | |
*** aranjan has quit IRC | 22:03 | |
marun | HenryG, haleyb, amuller: stop using distro-packaged python anything | 22:03 |
marun | it's instantly outdated | 22:03 |
marun | fine for long-running services, stupid for developers | 22:03 |
*** obondarev_ has joined #openstack-neutron | 22:04 | |
marun | devstack is working on stopping the insanity: https://review.openstack.org/#/c/156594/ | 22:04 |
*** SridharRamaswamy has quit IRC | 22:05 | |
haleyb | marun: easily enough said, but the minversion in tox.ini says 1.6 is fine and it's not any more | 22:06 |
*** obondarev has quit IRC | 22:06 | |
marun | haleyb: I'm not sure I understand why updating that version wouldn't have been your first choice, then. | 22:06 |
marun | haleyb: it's not like we're talking about stuff that has to be packaged for production | 22:07 |
haleyb | because i looked at your change and saw what got broken? | 22:07 |
prometheanfire | marun: if it's within the requirements.txt limits >=1.2.3,<2.3.4 then what does it matter? | 22:07 |
marun | prometheanfire: you're assuming tox is in our requirements.txt | 22:07 |
*** thomasem has quit IRC | 22:08 | |
marun | prometheanfire: it's kind of a hole | 22:08 |
*** SridharRamaswamy has joined #openstack-neutron | 22:08 | |
prometheanfire | it is | 22:08 |
marun | prometheanfire: where? | 22:08 |
prometheanfire | for tox specificly | 22:08 |
prometheanfire | I was agreeing that it is a hole :D | 22:08 |
marun | ah | 22:08 |
marun | haleyb: I'm sorry for not fixing this sooner. | 22:09 |
marun | haleyb: armax and I were discussing the issue earlier - he ran into it too - but I didn't have a chance to get to it yet. | 22:09 |
prometheanfire | I wonder if it should be in test-requiremetts | 22:09 |
haleyb | marun: i can change the minversion, until HenryG pointed out the page he cited i had no idea why it was working that way | 22:09 |
haleyb | prometheanfire: tox.ini does specify a version | 22:10 |
prometheanfire | that helps | 22:10 |
marun | prometheanfire: I'm assuming the reason for not having it in requirements is that invocation is generally outside of the venv | 22:10 |
prometheanfire | ya | 22:10 |
*** achanda has quit IRC | 22:10 | |
*** salv-orlando has quit IRC | 22:10 | |
prometheanfire | I see that reason | 22:10 |
prometheanfire | we don't even run tests through tox | 22:11 |
marun | haleyb: Maybe it's best that we go back to how I originally had it. | 22:11 |
prometheanfire | we use distro installed python packages :P | 22:11 |
prometheanfire | so it's harder | 22:11 |
HenryG | haleyb: If you set minversion in tox.ini, what happens if you use tox < minversion ? | 22:11 |
marun | prometheanfire: just what we need, more friction | 22:11 |
haleyb | ERROR: tox version is 1.6, required is at least 1.8 | 22:11 |
haleyb | HenryG: ^^ | 22:11 |
prometheanfire | marun: this is all on our side (distro packaging) | 22:11 |
prometheanfire | we allow for testinging before install | 22:12 |
marun | as a python developer, I pretty much hate distro packaging of python deps | 22:12 |
prometheanfire | all other distros don't (that I know of) | 22:12 |
marun | but i'm a developer. | 22:12 |
prometheanfire | I think we do it very well in gentoo :P | 22:12 |
*** crc32 has joined #openstack-neutron | 22:12 | |
marun | distro packages are not for developers | 22:12 |
prometheanfire | how is what we install diferent? | 22:12 |
*** changbl has quit IRC | 22:12 | |
prometheanfire | specific to gentoo? we just install the pypi version 99% of the time | 22:13 |
*** salv-orlando has joined #openstack-neutron | 22:13 | |
marun | prometheanfire: I have zero interest in the extra work of packaging something that is already consumable. | 22:13 |
prometheanfire | the other 1% is when upstream hasn't merged a fix | 22:13 |
prometheanfire | we need stable, testable systems :P | 22:13 |
marun | prometheanfire: but I work for rh, so... :/ | 22:13 |
prometheanfire | LOL | 22:13 |
prometheanfire | ya | 22:13 |
prometheanfire | I'm sorry | 22:13 |
marun | it's good for folks that need something as stable as possible. | 22:14 |
marun | haleyb, HenryG: so, stick to old version of tox to make life easier for everyone that is using distro-supplied tox? | 22:14 |
prometheanfire | I know, just the dichotomy | 22:15 |
prometheanfire | we have tox 1.8.1 stable in tree if it matters :D | 22:15 |
haleyb | if the correct version of tox was installed in .tox (like the rest of the world that it installs)... but that doesn't work since it's the one running things | 22:15 |
*** jobewan has quit IRC | 22:15 | |
HenryG | marun: My vote is minver 1.8 in tox.ini - sorry haleyb | 22:15 |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Add unit tests to ha.py for L3 agent https://review.openstack.org/156801 | 22:15 |
haleyb | marun: i'm fine changing new version, just happy to resolve issue | 22:16 |
*** julien_ZTE has quit IRC | 22:16 | |
haleyb | let me update the review | 22:17 |
*** crc32 has quit IRC | 22:18 | |
marun | haleyb: thank you for doing that | 22:20 |
HenryG | haleyb: marun: I'm off to hear about type hints in Python! I'll check in later tonight to see where things are with tox. | 22:20 |
*** leenheer has joined #openstack-neutron | 22:21 | |
*** shwetaap1 has quit IRC | 22:23 | |
marun | haleyb: I'll get that fix merged asap to minimize the trouble this is causing folks. | 22:23 |
*** leenheer has quit IRC | 22:23 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Un-break tox for unit tests https://review.openstack.org/156767 | 22:23 |
haleyb | marun: thanks, and it might have only bit two luddites on this channel :) | 22:24 |
marun | haleyb: definitely not :) | 22:24 |
marun | haleyb: both armando and I saw it this morning too | 22:24 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce ip address monitor https://review.openstack.org/149647 | 22:24 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Change metadata driver unit tests to use monitored spawn https://review.openstack.org/155058 | 22:24 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 22:24 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 22:24 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 22:24 |
marun | amuller: Oh, I do love the smell of patch series in the morning | 22:25 |
haleyb | that one-liner worked on my "new" VM with 1.8.1 | 22:25 |
amuller | marun: just the first one is updated, https://review.openstack.org/149647 | 22:25 |
amuller | carl_baldwin: I addressed reviewr comments from armax and amotoki | 22:25 |
marun | amuller: oh, I love that even more :D | 22:25 |
*** aranjan has joined #openstack-neutron | 22:26 | |
marun | amuller: I can taste the gate churn from here.... | 22:26 |
* marun hides | 22:26 | |
amuller | carl_baldwin: marun: armax: Feel free to take a look again | 22:26 |
*** suresh12 has joined #openstack-neutron | 22:26 | |
amuller | marun: I tried just pushing the first one but git review went insane and I couldn't be bothered to re-remember the full paths via git push | 22:26 |
amuller | marun: so, now you get the entire series :) | 22:26 |
amuller | marun: rebasing the rest of the series and the entire feature should be working now | 22:27 |
*** dboik has quit IRC | 22:28 | |
marun | amuller: cool | 22:28 |
carl_baldwin | amuller: ack | 22:28 |
*** achanda has joined #openstack-neutron | 22:29 | |
*** crc32 has joined #openstack-neutron | 22:31 | |
*** suresh12 has quit IRC | 22:31 | |
*** Marga_ has quit IRC | 22:32 | |
*** moshele has quit IRC | 22:32 | |
*** Marga_ has joined #openstack-neutron | 22:32 | |
*** manishg has quit IRC | 22:34 | |
*** manishg has joined #openstack-neutron | 22:35 | |
marun | carl_baldwin: hi | 22:36 |
marun | carl_baldwin: re: https://review.openstack.org/#/c/156801 | 22:36 |
marun | carl_baldwin: do you have any relationship with the patch author? | 22:36 |
carl_baldwin | marun: hi | 22:36 |
* carl_baldwin looking... | 22:36 | |
carl_baldwin | marun: Yes, I work with him. | 22:36 |
marun | carl_baldwin: I'm concerned that the proposed tests is mock-heavy and would act as little more than change detectors | 22:37 |
amuller | carl_baldwin: Can you get him on IRC? | 22:37 |
*** zzzeek has joined #openstack-neutron | 22:37 | |
marun | carl_baldwin: if he's looking for something to do, maybe we can direct him in a more fruitful direction. | 22:37 |
carl_baldwin | marun: amuller: I’ll see if he’s around. | 22:37 |
*** rotbart has joined #openstack-neutron | 22:37 | |
*** tiswanso has quit IRC | 22:37 | |
marun | carl_baldwin: thank you | 22:38 |
marun | ! | 22:38 |
*** Zengfa has joined #openstack-neutron | 22:40 | |
carl_baldwin | marun: amuller: meet Zengfa | 22:41 |
marun | Zengfa: hi! | 22:41 |
Zengfa | Hi | 22:42 |
amuller | Zengfa: Heya | 22:42 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Get rid of rpc to fetch fip agent port on agent. https://review.openstack.org/153735 | 22:42 |
marun | Zengfa: re: https://review.openstack.org/#/c/156801 | 22:42 |
marun | Zengfa: I'm assuming you were following the existing strategy of mocking things in an attempt to be consistent. | 22:43 |
marun | Zengfa: I don't think that strategy is worth replicating, though. | 22:43 |
marun | Zengfa: this style of testing often does more to increase maintenance cost than improve quality | 22:44 |
marun | Zengfa: see: http://googletesting.blogspot.com/2015/01/testing-on-toilet-change-detector-tests.html | 22:44 |
marun | Zengfa: I would much rather see you refactor the target code so that it can be tested with a minimum of mocking. | 22:44 |
amuller | Just a small unrelated note to what Maru is saying: ensure_keepalived_alive will be going away very soon via https://review.openstack.org/#/c/153162/ | 22:44 |
*** Networkn3rd has joined #openstack-neutron | 22:44 | |
*** Marga_ has quit IRC | 22:44 | |
marun | Zengfa: does that make sense? | 22:44 |
amuller | Zengfa: So I guess other units would be more worthwhile your effort, I'd just remove the last test | 22:44 |
*** Marga_ has joined #openstack-neutron | 22:44 | |
*** ramishra_ has joined #openstack-neutron | 22:45 | |
*** regXboi has quit IRC | 22:45 | |
*** pasha117 has joined #openstack-neutron | 22:45 | |
*** leenheer has joined #openstack-neutron | 22:45 | |
Zengfa | marun: I will look at the changes, then modify it later. | 22:46 |
*** pc_m has quit IRC | 22:46 | |
marun | Zengfa: I'm assuming that was for amuller? | 22:46 |
*** suresh12 has joined #openstack-neutron | 22:47 | |
*** rotbart has quit IRC | 22:47 | |
*** seizadi has quit IRC | 22:48 | |
*** seizadi1 has joined #openstack-neutron | 22:48 | |
Zengfa | amuller: I just noticed that there is no unit testing for this class, I think we should have the tests. I am new to the testings, will look at this to see what I should change. | 22:48 |
amuller | Zengfa: We do have functional testing for the L3 agent and HA routers | 22:49 |
amuller | It's higher level testing, and it covers a lot of the code | 22:49 |
*** zhenq has quit IRC | 22:49 | |
*** pavel_bondar has quit IRC | 22:49 | |
*** ramishra_ has quit IRC | 22:49 | |
amuller | It doesn't replace the lack of unit tests. If there's units that you find are not covered, it'd be great if you could test them, we just have to find an effective way to do so. | 22:49 |
amuller | There's no reason to mock stuff that doesn't interact with the system, for example | 22:50 |
*** ivar-laz_ has joined #openstack-neutron | 22:52 | |
amuller | Zengfa: It'd keep mocking to a minimum. For example the add_notifiers function doesn't interact with the system directly, it's a few layers down, you could mock that | 22:52 |
*** manishg has quit IRC | 22:52 | |
*** manishg has joined #openstack-neutron | 22:53 | |
Zengfa | amuller: Thanks for your suggestion. I will look at my changes, try to improve it. | 22:54 |
openstackgerrit | Ronak Shah proposed openstack/neutron: Nuage core plugin decomposition https://review.openstack.org/156823 | 22:54 |
amuller | Zengfa: Sounds good :) I'll try to review your next patchsets quickly | 22:55 |
*** ivar-lazzaro has quit IRC | 22:55 | |
*** MentalRay has joined #openstack-neutron | 22:56 | |
*** jkyle1 has joined #openstack-neutron | 22:56 | |
*** manishg has quit IRC | 22:57 | |
*** jorgem has joined #openstack-neutron | 22:58 | |
amuller | marun: the ip monitor functional tests are still failing at the gate: https://jenkins01.openstack.org/job/check-neutron-dsvm-functional/3371/console | 22:59 |
amuller | Although I solved all of that crap locally | 22:59 |
*** jorgem has quit IRC | 22:59 | |
amuller | now I really don't know why it says no filter found | 22:59 |
*** jorgem has joined #openstack-neutron | 22:59 | |
*** localloop127 has quit IRC | 23:00 | |
*** manishg has joined #openstack-neutron | 23:01 | |
*** jckasper has quit IRC | 23:03 | |
*** zhenq has joined #openstack-neutron | 23:04 | |
*** matrohon has quit IRC | 23:04 | |
*** Marga_ has quit IRC | 23:05 | |
*** amuller has quit IRC | 23:05 | |
*** Marga_ has joined #openstack-neutron | 23:06 | |
*** thedodd has joined #openstack-neutron | 23:07 | |
marun | ZZelle_: I'm really sorry that I've been so slow in reviewing this past week :/ | 23:07 |
openstackgerrit | Brian Haley proposed openstack/neutron: Check if routing rule exists before adding https://review.openstack.org/152330 | 23:08 |
*** aranjan has quit IRC | 23:08 | |
*** chlong has joined #openstack-neutron | 23:08 | |
*** leenheer has quit IRC | 23:08 | |
*** kbringard has quit IRC | 23:09 | |
*** hichihara has joined #openstack-neutron | 23:10 | |
*** csoukup has quit IRC | 23:11 | |
*** julien_ZTE has joined #openstack-neutron | 23:12 | |
*** seizadi1 has quit IRC | 23:13 | |
*** gduan has joined #openstack-neutron | 23:16 | |
*** garyduan has quit IRC | 23:18 | |
*** julien_ZTE has quit IRC | 23:19 | |
ZZelle_ | marun, hi, no pb | 23:21 |
*** yamamoto has joined #openstack-neutron | 23:21 | |
marun | ZZelle_: I'm juggling too much, and have been climbing a curve to provide some help on the infra side. | 23:22 |
marun | ZZelle_: I do want to see us get to the point of being able to do api and scenario testing this cycle, and your and John's work is critical to both of thos. | 23:22 |
marun | ZZelle_: in-tree testing, I mean | 23:23 |
*** yuanying has joined #openstack-neutron | 23:23 | |
*** leenheer has joined #openstack-neutron | 23:24 | |
*** leenheer has quit IRC | 23:24 | |
*** leenheer has joined #openstack-neutron | 23:25 | |
*** signed8bit has quit IRC | 23:25 | |
*** Networkn3rd has quit IRC | 23:26 | |
*** Networkn3rd has joined #openstack-neutron | 23:28 | |
*** aranjan has joined #openstack-neutron | 23:28 | |
*** pradk has quit IRC | 23:29 | |
*** yamamoto has quit IRC | 23:29 | |
*** bobmel has quit IRC | 23:31 | |
*** bobmel has joined #openstack-neutron | 23:31 | |
*** gildub has joined #openstack-neutron | 23:32 | |
*** nlahouti has quit IRC | 23:32 | |
*** Networkn3rd has quit IRC | 23:32 | |
*** jobewan has joined #openstack-neutron | 23:34 | |
*** leenheer has quit IRC | 23:34 | |
*** leenheer has joined #openstack-neutron | 23:35 | |
*** MentalRay has quit IRC | 23:35 | |
*** shwetaap has joined #openstack-neutron | 23:36 | |
*** zhenq has quit IRC | 23:36 | |
*** aranjan has quit IRC | 23:39 | |
*** ramishra_ has joined #openstack-neutron | 23:45 | |
*** ivar-laz_ has quit IRC | 23:48 | |
*** ivar-lazzaro has joined #openstack-neutron | 23:49 | |
*** _cjones_ has quit IRC | 23:50 | |
*** shaohe_feng has joined #openstack-neutron | 23:50 | |
*** ramishra_ has quit IRC | 23:50 | |
*** signed8bit has joined #openstack-neutron | 23:52 | |
*** yamamoto has joined #openstack-neutron | 23:52 | |
*** shwetaap has quit IRC | 23:52 | |
*** _cjones_ has joined #openstack-neutron | 23:53 | |
*** rotbart has joined #openstack-neutron | 23:53 | |
*** ZZelle_ has quit IRC | 23:53 | |
*** zhenq has joined #openstack-neutron | 23:53 | |
*** jobewan has quit IRC | 23:54 | |
*** nlahouti has joined #openstack-neutron | 23:55 | |
*** nlahouti1 has joined #openstack-neutron | 23:55 | |
*** aranjan has joined #openstack-neutron | 23:56 | |
*** manishg has quit IRC | 23:58 | |
*** manishg has joined #openstack-neutron | 23:59 | |
*** manishg has quit IRC | 23:59 | |
*** ajmiller has quit IRC | 23:59 | |
openstackgerrit | Merged openstack/neutron: Add run_as_root option to utils.execute https://review.openstack.org/153067 | 23:59 |
*** manishg has joined #openstack-neutron | 23:59 | |
*** nlahouti has quit IRC | 23:59 | |
*** julien_ZTE has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!