| *** nus has quit IRC | 00:00 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 00:02 |
|---|---|---|
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 00:02 |
| *** claired has joined #openstack-neutron | 00:03 | |
| *** ZZelle has quit IRC | 00:03 | |
| *** Guest48568 has quit IRC | 00:03 | |
| *** baoli has quit IRC | 00:04 | |
| Groink | imcsk8: Yeah, I have it set to 1:4094, only using id 3 | 00:05 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 00:05 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 00:05 |
| *** mlavalle has quit IRC | 00:07 | |
| *** markvoelker has quit IRC | 00:07 | |
| *** ZZelle has joined #openstack-neutron | 00:11 | |
| *** prad has quit IRC | 00:15 | |
| *** r-daneel has quit IRC | 00:15 | |
| *** s3wong_ has joined #openstack-neutron | 00:15 | |
| *** s3wong has quit IRC | 00:15 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 00:16 |
| *** thedodd has quit IRC | 00:16 | |
| *** carl_baldwin_ has joined #openstack-neutron | 00:17 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy to log with nobody user/group https://review.openstack.org/161494 | 00:18 |
| *** s3wong__ has joined #openstack-neutron | 00:18 | |
| *** s3wong_ has quit IRC | 00:18 | |
| *** carl_baldwin has quit IRC | 00:19 | |
| *** carl_baldwin_ is now known as carl_baldwin | 00:19 | |
| *** s3wong__ has quit IRC | 00:20 | |
| *** s3wong__ has joined #openstack-neutron | 00:20 | |
| *** annegentle has joined #openstack-neutron | 00:24 | |
| *** s3wong__ has quit IRC | 00:24 | |
| *** s3wong has joined #openstack-neutron | 00:24 | |
| *** Marga_ has quit IRC | 00:24 | |
| *** Marga_ has joined #openstack-neutron | 00:25 | |
| *** xuhanp has joined #openstack-neutron | 00:25 | |
| *** tsg_ has quit IRC | 00:25 | |
| *** watanabe_isao has joined #openstack-neutron | 00:27 | |
| watanabe_isao | Good morning to all.:-) | 00:27 |
| *** amuller has quit IRC | 00:27 | |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 00:29 |
| *** bquan_ has quit IRC | 00:30 | |
| *** klindgren_ has joined #openstack-neutron | 00:30 | |
| *** sbfox1 has joined #openstack-neutron | 00:31 | |
| *** klindgren_ has left #openstack-neutron | 00:33 | |
| *** SridharRamaswamy has quit IRC | 00:34 | |
| *** carl_baldwin has quit IRC | 00:34 | |
| *** sbfox has quit IRC | 00:34 | |
| *** annegentle has quit IRC | 00:36 | |
| *** trad511 has quit IRC | 00:37 | |
| *** sbfox1 has quit IRC | 00:38 | |
| *** aranjan_ has joined #openstack-neutron | 00:38 | |
| *** aranjan has quit IRC | 00:38 | |
| *** aranjan_ has quit IRC | 00:39 | |
| *** shwetaap has quit IRC | 00:40 | |
| *** tomoe_ has joined #openstack-neutron | 00:40 | |
| *** devlaps has quit IRC | 00:41 | |
| *** crc32 has quit IRC | 00:41 | |
| *** manishg has joined #openstack-neutron | 00:41 | |
| *** manishg has quit IRC | 00:42 | |
| *** manishg has joined #openstack-neutron | 00:43 | |
| *** aranjan has joined #openstack-neutron | 00:44 | |
| *** SridharRamaswamy has joined #openstack-neutron | 00:47 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS DVR UT: Remove an inappropriate str() conversion https://review.openstack.org/161524 | 00:47 |
| *** badveli_ has joined #openstack-neutron | 00:48 | |
| badveli_ | assaf muller | 00:49 |
| *** aranjan has quit IRC | 00:49 | |
| *** ajmiller has quit IRC | 00:49 | |
| badveli_ | amullar | 00:49 |
| *** shaohe_feng has joined #openstack-neutron | 00:49 | |
| *** claired has quit IRC | 00:55 | |
| *** chuckC_ has quit IRC | 00:58 | |
| *** aranjan has joined #openstack-neutron | 00:58 | |
| *** ijw has quit IRC | 01:01 | |
| *** leenheer has joined #openstack-neutron | 01:02 | |
| *** Madkiss_ has joined #openstack-neutron | 01:02 | |
| *** Madkiss has quit IRC | 01:02 | |
| *** Madkiss_ is now known as Madkiss | 01:02 | |
| *** yamamoto has quit IRC | 01:04 | |
| *** bquan has joined #openstack-neutron | 01:05 | |
| *** bquan has quit IRC | 01:07 | |
| *** carl_baldwin has joined #openstack-neutron | 01:07 | |
| *** markvoelker has joined #openstack-neutron | 01:08 | |
| *** aranjan has quit IRC | 01:08 | |
| *** tomoe_ has quit IRC | 01:11 | |
| *** leenheer has quit IRC | 01:11 | |
| *** SumitNaiksatam has quit IRC | 01:11 | |
| *** ZZelle_ has quit IRC | 01:12 | |
| openstackgerrit | Merged openstack/neutron: Remove HyperVNeutronPlugin https://review.openstack.org/153394 | 01:12 |
| *** markvoelker has quit IRC | 01:13 | |
| openstackgerrit | Merged openstack/neutron: Introduce ip address monitor https://review.openstack.org/149647 | 01:14 |
| *** annegentle has joined #openstack-neutron | 01:15 | |
| *** topshare has joined #openstack-neutron | 01:15 | |
| *** markvoelker has joined #openstack-neutron | 01:15 | |
| *** salv-orlando has quit IRC | 01:16 | |
| *** annegentle has quit IRC | 01:16 | |
| *** s3wong has quit IRC | 01:16 | |
| *** annegentle has joined #openstack-neutron | 01:16 | |
| *** zzzeek has quit IRC | 01:17 | |
| *** tyjawe has quit IRC | 01:18 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Big Switch Networks code split https://review.openstack.org/160360 | 01:20 |
| *** markvoelker has quit IRC | 01:20 | |
| *** 20WAARP23 has quit IRC | 01:24 | |
| *** 32NABGXUF has quit IRC | 01:24 | |
| *** tomoe_ has joined #openstack-neutron | 01:26 | |
| *** rook has joined #openstack-neutron | 01:27 | |
| *** aranjan has joined #openstack-neutron | 01:27 | |
| *** carl_baldwin has quit IRC | 01:27 | |
| *** rook is now known as Guest15040 | 01:27 | |
| openstackgerrit | Merged openstack/neutron: Use oslo_config choices support https://review.openstack.org/159288 | 01:27 |
| *** Guest15040 has quit IRC | 01:27 | |
| *** annegentle has quit IRC | 01:27 | |
| *** bobmel has quit IRC | 01:27 | |
| *** bobmel has joined #openstack-neutron | 01:28 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add ability to run pylint check on modified files only https://review.openstack.org/159341 | 01:32 |
| *** xgerman has quit IRC | 01:35 | |
| *** amotoki has joined #openstack-neutron | 01:35 | |
| *** stanzgy has joined #openstack-neutron | 01:35 | |
| *** stanzgy_ has joined #openstack-neutron | 01:35 | |
| *** zhenq has quit IRC | 01:35 | |
| *** ijw has joined #openstack-neutron | 01:36 | |
| *** Marga_ has quit IRC | 01:37 | |
| *** ijw has quit IRC | 01:38 | |
| *** liusheng has quit IRC | 01:38 | |
| *** zhenq has joined #openstack-neutron | 01:40 | |
| *** dims has quit IRC | 01:43 | |
| *** dboik has joined #openstack-neutron | 01:44 | |
| *** tsg has joined #openstack-neutron | 01:47 | |
| *** dboik has quit IRC | 01:48 | |
| *** jamielennox is now known as jamielennox|lunc | 01:51 | |
| *** tiswanso has joined #openstack-neutron | 01:51 | |
| *** tiswanso has quit IRC | 01:53 | |
| *** tiswanso has joined #openstack-neutron | 01:53 | |
| *** claudiub has quit IRC | 01:54 | |
| *** aranjan has quit IRC | 01:56 | |
| *** _cjones_ has quit IRC | 01:58 | |
| *** popw has joined #openstack-neutron | 02:03 | |
| *** vivek-eb_ has joined #openstack-neutron | 02:06 | |
| *** vivek-ebay has quit IRC | 02:06 | |
| *** Networkn3rd has joined #openstack-neutron | 02:06 | |
| openstackgerrit | Merged openstack/neutron: ml2 extension driver: more tests, fix data argument inconsistency https://review.openstack.org/129178 | 02:07 |
| *** vivek-eb_ has quit IRC | 02:07 | |
| *** vivek-ebay has joined #openstack-neutron | 02:08 | |
| *** Marga_ has joined #openstack-neutron | 02:09 | |
| *** Marga_ has quit IRC | 02:09 | |
| *** erkules_ has joined #openstack-neutron | 02:09 | |
| *** Marga_ has joined #openstack-neutron | 02:10 | |
| *** popw has quit IRC | 02:10 | |
| *** erkules has quit IRC | 02:12 | |
| *** popw has joined #openstack-neutron | 02:12 | |
| *** vishwana_ has joined #openstack-neutron | 02:16 | |
| *** enikanorov_ has joined #openstack-neutron | 02:16 | |
| *** Longgeek has joined #openstack-neutron | 02:16 | |
| *** obondarev_ has joined #openstack-neutron | 02:16 | |
| *** markvoelker has joined #openstack-neutron | 02:16 | |
| *** vivek-ebay has quit IRC | 02:17 | |
| *** vivek-ebay has joined #openstack-neutron | 02:17 | |
| *** tsg_ has joined #openstack-neutron | 02:18 | |
| *** jgrimmn has joined #openstack-neutron | 02:20 | |
| *** promethe1nfire has joined #openstack-neutron | 02:20 | |
| *** pcarver_ has joined #openstack-neutron | 02:20 | |
| *** rwsu_ has joined #openstack-neutron | 02:20 | |
| *** zz_jgrimm has quit IRC | 02:21 | |
| *** vishwanathj has quit IRC | 02:21 | |
| *** prometheanfire has quit IRC | 02:21 | |
| *** nwonknu has quit IRC | 02:21 | |
| *** tsg has quit IRC | 02:21 | |
| *** reed has quit IRC | 02:21 | |
| *** pcarver has quit IRC | 02:21 | |
| *** rwsu has quit IRC | 02:21 | |
| *** obondarev has quit IRC | 02:21 | |
| *** JoeJulian has quit IRC | 02:21 | |
| *** enikanorov__ has quit IRC | 02:21 | |
| *** tsg_ has quit IRC | 02:21 | |
| *** tsg_ has joined #openstack-neutron | 02:21 | |
| *** jgrimmn has quit IRC | 02:21 | |
| *** jgrimmn has joined #openstack-neutron | 02:21 | |
| *** markvoelker has quit IRC | 02:21 | |
| *** nwonknu has joined #openstack-neutron | 02:22 | |
| *** JoeJulian has joined #openstack-neutron | 02:22 | |
| *** coolsvap_ is now known as coolsvap | 02:22 | |
| *** Longgeek has quit IRC | 02:22 | |
| *** stefanb has quit IRC | 02:24 | |
| *** Longgeek has joined #openstack-neutron | 02:24 | |
| *** Longgeek has quit IRC | 02:25 | |
| *** Longgeek has joined #openstack-neutron | 02:25 | |
| *** leenheer has joined #openstack-neutron | 02:25 | |
| *** SridharRamaswamy has quit IRC | 02:25 | |
| *** ChuckC has joined #openstack-neutron | 02:26 | |
| *** reed has joined #openstack-neutron | 02:32 | |
| *** rwsu_ is now known as rwsu-afk | 02:35 | |
| *** reed has quit IRC | 02:36 | |
| *** reed has joined #openstack-neutron | 02:37 | |
| *** dims has joined #openstack-neutron | 02:39 | |
| *** zhhuabj has joined #openstack-neutron | 02:39 | |
| *** coolsvap is now known as coolsvap_ | 02:41 | |
| *** Murali has joined #openstack-neutron | 02:43 | |
| *** tsg_ has quit IRC | 02:46 | |
| *** tsg has joined #openstack-neutron | 02:47 | |
| *** jamielennox|lunc is now known as jamielennox | 02:47 | |
| *** shwetaap has joined #openstack-neutron | 02:49 | |
| *** leenheer has quit IRC | 02:49 | |
| *** sthillma has quit IRC | 02:50 | |
| watanabe_isao | I cannot find destination or nexthop of routers described in API reference. And 400 BadRequest returned when updating them using REST API. Does any body know that is changing destination or nexthop of routers forbidden with neutron or not? | 02:52 |
| *** leenheer has joined #openstack-neutron | 02:54 | |
| watanabe_isao | BTW, some tests in neutron/tests/unit/test_extension_extraroute.py mentioned _update destination and nexthop. But I cannot figure out the root code. | 02:55 |
| *** tflynn_ has joined #openstack-neutron | 02:55 | |
| *** achanda has quit IRC | 02:55 | |
| *** yog_ has joined #openstack-neutron | 02:56 | |
| watanabe_isao | ^^^hope any core member or knowledgeable person | 02:58 |
| watanabe_isao | awake... :-( | 02:58 |
| *** tflynn has quit IRC | 02:59 | |
| watanabe_isao | Ops, google translate gave me a return line. | 02:59 |
| *** tflynn_ has quit IRC | 03:00 | |
| *** yamahata has quit IRC | 03:00 | |
| *** ChuckC has quit IRC | 03:01 | |
| *** rm_work is now known as rm_work|away | 03:02 | |
| *** ChuckC has joined #openstack-neutron | 03:06 | |
| *** flwang has quit IRC | 03:08 | |
| *** flwang has joined #openstack-neutron | 03:10 | |
| openstackgerrit | Merged openstack/python-neutronclient: Honor allow_names in *-update command https://review.openstack.org/158152 | 03:10 |
| *** markvoelker has joined #openstack-neutron | 03:17 | |
| *** jyuso310 has joined #openstack-neutron | 03:18 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during delete operation https://review.openstack.org/155470 | 03:20 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Break coupling between ML2 and L3 during create/update operations https://review.openstack.org/153775 | 03:20 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and service plugins during DVR router migration https://review.openstack.org/154749 | 03:20 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Decouple L3 and VPN service plugins during router operations https://review.openstack.org/153776 | 03:20 |
| *** Murali has quit IRC | 03:20 | |
| *** tomoe_ has quit IRC | 03:21 | |
| *** vivek-ebay has quit IRC | 03:22 | |
| *** markvoelker has quit IRC | 03:22 | |
| *** badveli_ has quit IRC | 03:27 | |
| *** yamamoto has joined #openstack-neutron | 03:28 | |
| *** achanda has joined #openstack-neutron | 03:29 | |
| *** spandhe has quit IRC | 03:30 | |
| *** SumitNaiksatam has joined #openstack-neutron | 03:32 | |
| *** zhenq has quit IRC | 03:34 | |
| *** watanabe_isao has quit IRC | 03:35 | |
| *** SumitNaiksatam has quit IRC | 03:38 | |
| HenryG | armax: around? | 03:40 |
| armax | HenryG: yes | 03:41 |
| HenryG | armax: Do you understand the functional job gate_hook stuff? | 03:41 |
| *** yamamoto has quit IRC | 03:41 | |
| armax | HenryG: it depends :) | 03:41 |
| HenryG | armax: :) | 03:41 |
| armax | what is that you’re looking for? | 03:41 |
| HenryG | armax: see the infra channel if you have scrollback there, else I can recap | 03:42 |
| armax | since what min? | 03:42 |
| *** tsg has quit IRC | 03:42 | |
| *** Networkn_ has joined #openstack-neutron | 03:42 | |
| HenryG | 22:33 EST :) | 03:43 |
| armax | HenryG: um… | 03:43 |
| HenryG | about 10 minutes ago | 03:43 |
| *** Networkn3rd has quit IRC | 03:44 | |
| armax | HenryG: do you know what is the latest successful run for the job? | 03:44 |
| armax | Maru has been filing a few changes to d-g and the functional job, HenryG: perhaps something went bad with https://review.openstack.org/#/c/155957/ | 03:45 |
| armax | but I made the point about stable branches and he seemed to have taken care of it | 03:45 |
| *** jamielennox is now known as jamielennox|away | 03:45 | |
| armax | we’d need to trace when the job broke | 03:45 |
| *** tsg has joined #openstack-neutron | 03:45 | |
| *** manishg_ has joined #openstack-neutron | 03:47 | |
| HenryG | hmmm, I am seeing some fn jobs passing between failures | 03:47 |
| *** manishg has quit IRC | 03:48 | |
| armax | on the stable branch? | 03:48 |
| HenryG | yes, juno | 03:48 |
| armax | pointer? | 03:48 |
| *** pcarver_ is now known as pcarver | 03:49 | |
| armax | HenryG: if I see here: https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:stable/juno,n,z | 03:49 |
| HenryG | This one passed https://review.openstack.org/161270 | 03:49 |
| armax | um | 03:49 |
| *** vivek-ebay has joined #openstack-neutron | 03:50 | |
| armax | I wonder if that’s a red-herring | 03:50 |
| armax | the patch right below failed 10 mins after | 03:50 |
| HenryG | right | 03:51 |
| armax | and all the others that ran today broke | 03:51 |
| HenryG | yup | 03:51 |
| *** flwang has quit IRC | 03:51 | |
| HenryG | ANd the error is the same for all: /opt/stack/new/devstack-gate/devstack-vm-gate.sh: No such file or directory | 03:52 |
| *** watanabe_isao has joined #openstack-neutron | 03:52 | |
| armax | the only change I could fault is this one: https://review.openstack.org/#/c/155958/ | 03:53 |
| armax | but I am not sure I have enough information yet to make such a claim | 03:53 |
| *** manishg_ has quit IRC | 03:53 | |
| *** yamamoto has joined #openstack-neutron | 03:54 | |
| *** vivek-eb_ has joined #openstack-neutron | 03:54 | |
| *** yog_ has quit IRC | 03:55 | |
| *** vivek-ebay has quit IRC | 03:55 | |
| armax | HenryG: besides, the gate is so slow that we really can’t tell for sure if we’re broken in master to | 03:56 |
| armax | o | 03:56 |
| armax | HenryG: but perhaps not | 03:57 |
| HenryG | armax: Yes, the gate is very backlogged | 03:58 |
| armax | HenryG: I am a bit baffled as to why this would break | 03:59 |
| armax | the file is there regardless of the branch | 03:59 |
| armax | HenryG: for instance we call it here: | 04:00 |
| armax | https://github.com/openstack/neutron/blob/master/neutron/tests/contrib/gate_hook.sh#L28 | 04:00 |
| *** vthapar has joined #openstack-neutron | 04:00 | |
| armax | on master | 04:00 |
| armax | for the api job | 04:00 |
| armax | instead we fail here: | 04:00 |
| HenryG | armax: that is why I asked in infra | 04:00 |
| armax | https://github.com/openstack/neutron/blob/stable/juno/neutron/tests/functional/contrib/gate_hook.sh#L5 | 04:00 |
| armax | so there must be something done in the pre hook that prevents that file to be dropped | 04:00 |
| armax | on the node | 04:00 |
| armax | hence the failure | 04:01 |
| *** yog_ has joined #openstack-neutron | 04:01 | |
| openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Always run ip cmd as root https://review.openstack.org/161567 | 04:02 |
| *** nlahouti has quit IRC | 04:02 | |
| *** tomoe_ has joined #openstack-neutron | 04:04 | |
| *** shwetaap has quit IRC | 04:04 | |
| *** shwetaap has joined #openstack-neutron | 04:05 | |
| armax | HenryG: | 04:06 |
| armax | ping | 04:06 |
| armax | looks like the problem is here: http://logs.openstack.org/73/161273/1/check/check-neutron-dsvm-functional/56af914/console.html#_2015-03-05_00_26_21_129 | 04:06 |
| *** rkukura has quit IRC | 04:07 | |
| armax | HenryG: the test fails | 04:07 |
| armax | because BRANCH_OVERRIDE is default | 04:07 |
| armax | instead of stable/juno | 04:07 |
| armax | so this does break | 04:08 |
| *** SumitNaiksatam has joined #openstack-neutron | 04:08 | |
| armax | HenryG: you with me? | 04:08 |
| HenryG | armax: Lost at devstack-vm-gate.sh vs. devstack-vm-gate-wrap.sh ? | 04:08 |
| armax | look here: http://logs.openstack.org/73/161273/1/check/check-neutron-dsvm-functional/56af914/console.html#_2015-03-05_00_26_21_129 | 04:09 |
| armax | this test | 04:09 |
| armax | [[ ! default =~ ^stable/(icehouse|juno)$ ]] | 04:09 |
| HenryG | armax: aha | 04:09 |
| armax | succeeds and then DEVSTACK_GATE_PROJECTS_OVERRIDE gets set | 04:09 |
| armax | but then look here: | 04:09 |
| armax | https://review.openstack.org/#/c/155958/3/jenkins/jobs/neutron.yaml | 04:09 |
| armax | read the comment ;) | 04:09 |
| armax | it looks like marun expected BRANCH_OVERRIDE to be something like stable/juno | 04:10 |
| armax | instead it’s ‘default’ for whatever reason | 04:10 |
| armax | and bam | 04:10 |
| armax | kaboom | 04:10 |
| armax | everything breaks :) | 04:11 |
| *** jamiec has quit IRC | 04:11 | |
| *** anish_ has joined #openstack-neutron | 04:11 | |
| *** leenheer has quit IRC | 04:11 | |
| HenryG | armax: So if $BRANCH_OVERRIDE does not contain the branch-override, what does? | 04:15 |
| *** jamiec has joined #openstack-neutron | 04:15 | |
| *** reed has quit IRC | 04:15 | |
| armax | HenryG: good question | 04:15 |
| armax | HenryG: I am trying to find out | 04:15 |
| armax | but for that folks from infra should be able to redirect us | 04:16 |
| *** sbfox has joined #openstack-neutron | 04:16 | |
| HenryG | In the yamls I see a lot of export BRANCH_OVERRIDE={branch-override} | 04:16 |
| HenryG | So something is overriding the override? | 04:16 |
| *** salv-orlando has joined #openstack-neutron | 04:16 | |
| *** yamamoto has quit IRC | 04:17 | |
| armax | OVERRIDE_ZUUL_BRANCH/ | 04:17 |
| armax | ? | 04:17 |
| HenryG | Try it :) | 04:17 |
| armax | or perhaps is branch-designator | 04:17 |
| armax | HenryG: what do you mean? | 04:17 |
| *** zhhuabj has quit IRC | 04:18 | |
| HenryG | if [[ ! "$OVERRIDE_ZUUL_BRANCH" ... | 04:18 |
| armax | or perhaps it’s just BRANCH | 04:19 |
| HenryG | heh | 04:19 |
| armax | but we can’t tell until we merge | 04:19 |
| *** markvoelker has joined #openstack-neutron | 04:19 | |
| armax | so we really need some folks from infra who know the trick | 04:19 |
| HenryG | I think they had a hard day and have drunk themselves to sleep ? | 04:20 |
| armax | HenryG: possibly | 04:20 |
| armax | let me try to file a patch | 04:20 |
| armax | just to signal that things are broken | 04:20 |
| armax | and get the discussion in the right place | 04:20 |
| HenryG | armax: sounds good | 04:21 |
| HenryG | thanks | 04:21 |
| *** salv-orlando has quit IRC | 04:21 | |
| HenryG | Add me as reviewer | 04:21 |
| armax | will do | 04:21 |
| *** yamahata has joined #openstack-neutron | 04:22 | |
| *** achanda has quit IRC | 04:22 | |
| *** dims has quit IRC | 04:22 | |
| HenryG | armax: oh, and https://review.openstack.org/160175 ;) | 04:22 |
| *** markvoelker has quit IRC | 04:23 | |
| armax | HenryG: done | 04:23 |
| *** ivar-lazzaro has joined #openstack-neutron | 04:24 | |
| HenryG | armax: responded | 04:24 |
| *** manishg has joined #openstack-neutron | 04:24 | |
| armax | k | 04:25 |
| armax | HenryG: looking | 04:25 |
| HenryG | we have some code duplication for now | 04:25 |
| armax | eventually it’ll be cleaned | 04:26 |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: Add availability_zone support https://review.openstack.org/149940 | 04:26 |
| HenryG | yes | 04:26 |
| *** ivar-laz_ has quit IRC | 04:27 | |
| HenryG | armax: I am off to bed now. Will check for your infra patch in the morning. | 04:28 |
| armax | HenryG: I posted it | 04:28 |
| armax | HenryG: nite | 04:28 |
| *** ivar-lazzaro has quit IRC | 04:29 | |
| *** manishg has quit IRC | 04:29 | |
| *** yamamoto has joined #openstack-neutron | 04:30 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Change metadata driver unit tests to use monitored spawn https://review.openstack.org/155058 | 04:34 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 04:34 |
| *** leenheer has joined #openstack-neutron | 04:37 | |
| *** leenheer has quit IRC | 04:37 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 04:43 |
| *** yamamoto_ has joined #openstack-neutron | 04:43 | |
| *** yamamoto has quit IRC | 04:43 | |
| *** tiswanso has quit IRC | 04:45 | |
| *** vishy has quit IRC | 04:47 | |
| *** yamamoto_ has quit IRC | 04:48 | |
| openstackgerrit | Merged openstack/neutron: Metaplugin decomposition https://review.openstack.org/151464 | 04:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/161577 | 04:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/160569 | 04:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/160571 | 04:52 |
| *** ajmiller has joined #openstack-neutron | 04:54 | |
| *** shwetaap has quit IRC | 04:54 | |
| *** harlowja is now known as harlowja_away | 04:57 | |
| *** bquan has joined #openstack-neutron | 04:58 | |
| *** tflynn has joined #openstack-neutron | 04:58 | |
| *** melwitt has quit IRC | 04:59 | |
| *** shwetaap has joined #openstack-neutron | 04:59 | |
| *** manishg has joined #openstack-neutron | 04:59 | |
| *** bquan_ has joined #openstack-neutron | 05:00 | |
| *** gongysh_ has joined #openstack-neutron | 05:00 | |
| *** clarkb has quit IRC | 05:00 | |
| *** bquan_ has quit IRC | 05:01 | |
| *** bquan has quit IRC | 05:01 | |
| *** anilvenkata has joined #openstack-neutron | 05:01 | |
| *** bquan has joined #openstack-neutron | 05:01 | |
| *** tflynn has quit IRC | 05:03 | |
| *** manishg has quit IRC | 05:03 | |
| *** bquan_ has joined #openstack-neutron | 05:05 | |
| openstackgerrit | Yatin Kumbhare proposed openstack/neutron: opencontrail extension support https://review.openstack.org/145290 | 05:07 |
| *** spandhe has joined #openstack-neutron | 05:07 | |
| *** tomoe_ has quit IRC | 05:07 | |
| *** Zemeio has quit IRC | 05:08 | |
| *** bquan has quit IRC | 05:08 | |
| *** carl_baldwin has joined #openstack-neutron | 05:08 | |
| *** carl_baldwin has quit IRC | 05:08 | |
| *** shwetaap has quit IRC | 05:10 | |
| *** bquan__ has joined #openstack-neutron | 05:14 | |
| *** bquan_ has quit IRC | 05:14 | |
| *** spandhe_ has joined #openstack-neutron | 05:17 | |
| *** yamamoto has joined #openstack-neutron | 05:17 | |
| *** vivek-eb_ has quit IRC | 05:17 | |
| *** spandhe has quit IRC | 05:18 | |
| *** spandhe_ is now known as spandhe | 05:18 | |
| *** bquan_ has joined #openstack-neutron | 05:18 | |
| *** yalie1 has quit IRC | 05:19 | |
| *** markvoelker has joined #openstack-neutron | 05:20 | |
| *** markmcclain has quit IRC | 05:20 | |
| *** yshafrir has joined #openstack-neutron | 05:20 | |
| *** bquan__ has quit IRC | 05:20 | |
| *** pradipta has joined #openstack-neutron | 05:21 | |
| *** markmcclain has joined #openstack-neutron | 05:22 | |
| *** achanda has joined #openstack-neutron | 05:23 | |
| *** dims has joined #openstack-neutron | 05:23 | |
| *** tomoe_ has joined #openstack-neutron | 05:23 | |
| *** achanda has quit IRC | 05:23 | |
| *** achanda has joined #openstack-neutron | 05:23 | |
| *** markvoelker has quit IRC | 05:25 | |
| *** tsg_ has joined #openstack-neutron | 05:26 | |
| *** ParsectiX has quit IRC | 05:26 | |
| *** ParsectiX has joined #openstack-neutron | 05:27 | |
| *** dims has quit IRC | 05:28 | |
| *** tsg has quit IRC | 05:29 | |
| *** irenab has joined #openstack-neutron | 05:31 | |
| *** Murali has joined #openstack-neutron | 05:32 | |
| *** sbfox has quit IRC | 05:33 | |
| *** vishwana_ has quit IRC | 05:35 | |
| openstackgerrit | gongysh proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 05:36 |
| *** rkukura has joined #openstack-neutron | 05:37 | |
| *** tflynn has joined #openstack-neutron | 05:38 | |
| *** tflynn_ has joined #openstack-neutron | 05:38 | |
| *** trinaths has joined #openstack-neutron | 05:41 | |
| *** tflynn has quit IRC | 05:42 | |
| *** Murali_ has joined #openstack-neutron | 05:44 | |
| *** Murali has quit IRC | 05:45 | |
| *** Murali_ is now known as Murali | 05:45 | |
| *** bquan_ has quit IRC | 05:46 | |
| *** flwang has joined #openstack-neutron | 05:46 | |
| *** neeti has joined #openstack-neutron | 05:49 | |
| *** tomoe_ has quit IRC | 05:53 | |
| *** tsg_ has quit IRC | 05:54 | |
| *** tomoe_ has joined #openstack-neutron | 05:56 | |
| *** sluo_wfh has joined #openstack-neutron | 05:56 | |
| *** garyk has quit IRC | 05:59 | |
| *** irenab has quit IRC | 05:59 | |
| *** manishg has joined #openstack-neutron | 06:00 | |
| *** tomoe_ has quit IRC | 06:00 | |
| *** yalie has joined #openstack-neutron | 06:01 | |
| *** yatin has joined #openstack-neutron | 06:01 | |
| *** spandhe has quit IRC | 06:03 | |
| *** sluo_wfh has quit IRC | 06:04 | |
| *** manishg has quit IRC | 06:05 | |
| *** lazy_prince has quit IRC | 06:08 | |
| *** ParsectiX has quit IRC | 06:08 | |
| *** killer_prince has joined #openstack-neutron | 06:08 | |
| *** killer_prince is now known as lazy_prince | 06:08 | |
| *** yongshenggong_ has joined #openstack-neutron | 06:10 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info https://review.openstack.org/152512 | 06:10 |
| *** sluo_wfh has joined #openstack-neutron | 06:14 | |
| *** jschwarz has joined #openstack-neutron | 06:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/161614 | 06:15 |
| *** _cerberus_ has quit IRC | 06:16 | |
| *** _cerberus_ has joined #openstack-neutron | 06:17 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add sanity check for wrong MTU when using tunnels https://review.openstack.org/129852 | 06:17 |
| *** spandhe has joined #openstack-neutron | 06:22 | |
| *** markvoelker has joined #openstack-neutron | 06:22 | |
| *** leenheer has joined #openstack-neutron | 06:25 | |
| *** markvoelker has quit IRC | 06:27 | |
| *** chandankumar has joined #openstack-neutron | 06:28 | |
| *** leenheer has quit IRC | 06:29 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Vxlan UDP port value should be updated in neutron DB https://review.openstack.org/153891 | 06:29 |
| *** leenheer has joined #openstack-neutron | 06:31 | |
| *** tomoe_ has joined #openstack-neutron | 06:33 | |
| trinaths | Hello Neutroners, Very Good Morning | 06:40 |
| *** SridharG has joined #openstack-neutron | 06:41 | |
| *** sbfox has joined #openstack-neutron | 06:42 | |
| trinaths | Can some one review this change, https://review.openstack.org/#/c/160702/ please | 06:43 |
| openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 06:44 |
| *** gridinv has joined #openstack-neutron | 06:46 | |
| *** flwang has quit IRC | 06:47 | |
| *** Krast has joined #openstack-neutron | 06:51 | |
| *** gildub has quit IRC | 06:54 | |
| *** gridinv has quit IRC | 06:55 | |
| *** rkukura has quit IRC | 06:59 | |
| *** jyuso310 has quit IRC | 06:59 | |
| *** flwang has joined #openstack-neutron | 07:00 | |
| *** manishg has joined #openstack-neutron | 07:01 | |
| openstackgerrit | yalei wang proposed openstack/neutron-fwaas: Add the missing parameter agent_mode to the FWaaSDriverBase class methods https://review.openstack.org/153930 | 07:02 |
| *** claudiub has joined #openstack-neutron | 07:03 | |
| *** garyk has joined #openstack-neutron | 07:05 | |
| *** manishg has quit IRC | 07:05 | |
| *** flwang has quit IRC | 07:07 | |
| *** tflynn_ has quit IRC | 07:13 | |
| *** tflynn has joined #openstack-neutron | 07:14 | |
| *** salv-orlando has joined #openstack-neutron | 07:14 | |
| *** moshele has joined #openstack-neutron | 07:14 | |
| *** salv-orlando has quit IRC | 07:19 | |
| *** flwang has joined #openstack-neutron | 07:20 | |
| *** Marga_ has quit IRC | 07:20 | |
| *** markvoelker has joined #openstack-neutron | 07:22 | |
| *** irenab has joined #openstack-neutron | 07:23 | |
| *** evgenyf has joined #openstack-neutron | 07:24 | |
| *** nlahouti has joined #openstack-neutron | 07:24 | |
| *** anshul has joined #openstack-neutron | 07:26 | |
| *** salv-orlando has joined #openstack-neutron | 07:27 | |
| *** lhcheng has joined #openstack-neutron | 07:28 | |
| *** markvoelker has quit IRC | 07:28 | |
| *** nlahouti has quit IRC | 07:30 | |
| *** nlahouti has joined #openstack-neutron | 07:31 | |
| *** liusheng has joined #openstack-neutron | 07:32 | |
| *** spandhe has quit IRC | 07:32 | |
| *** openstackgerrit has quit IRC | 07:36 | |
| *** openstackgerrit has joined #openstack-neutron | 07:36 | |
| *** killer_prince has joined #openstack-neutron | 07:37 | |
| *** lazy_prince has quit IRC | 07:38 | |
| *** killer_prince is now known as lazy_prince | 07:38 | |
| *** nyechiel__ has joined #openstack-neutron | 07:39 | |
| *** roeyc has joined #openstack-neutron | 07:40 | |
| *** leenheer has quit IRC | 07:40 | |
| *** _cerberus_ has quit IRC | 07:42 | |
| *** yfauser has joined #openstack-neutron | 07:45 | |
| *** yfauser has left #openstack-neutron | 07:45 | |
| *** nmagnezi has joined #openstack-neutron | 07:46 | |
| *** sthillma has joined #openstack-neutron | 07:46 | |
| *** numan has joined #openstack-neutron | 07:46 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: Add availability_zone support https://review.openstack.org/149940 | 07:50 |
| *** ajo has quit IRC | 07:52 | |
| *** ajo has joined #openstack-neutron | 07:53 | |
| *** yog_ has quit IRC | 07:55 | |
| *** apuimedo has joined #openstack-neutron | 07:55 | |
| *** scheuran has joined #openstack-neutron | 07:56 | |
| *** yongshenggong_ has quit IRC | 07:56 | |
| *** garyk has quit IRC | 07:56 | |
| *** tflynn has quit IRC | 08:01 | |
| *** aepifanov has joined #openstack-neutron | 08:01 | |
| *** manishg has joined #openstack-neutron | 08:02 | |
| *** garyk has joined #openstack-neutron | 08:02 | |
| *** sthillma has quit IRC | 08:05 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: Add availability_zone support https://review.openstack.org/149940 | 08:06 |
| *** Bsony has joined #openstack-neutron | 08:06 | |
| *** manishg has quit IRC | 08:06 | |
| *** aepifanov has quit IRC | 08:08 | |
| *** matrohon has joined #openstack-neutron | 08:08 | |
| *** nlahouti has quit IRC | 08:09 | |
| *** pask81 has joined #openstack-neutron | 08:10 | |
| *** Groink has quit IRC | 08:12 | |
| *** ajo has quit IRC | 08:14 | |
| *** vivek-ebay has joined #openstack-neutron | 08:14 | |
| *** achanda has quit IRC | 08:16 | |
| *** kickinz1|afk is now known as kickinz1 | 08:17 | |
| *** atmark has joined #openstack-neutron | 08:17 | |
| *** ronis has joined #openstack-neutron | 08:18 | |
| atmark | Hi all, /close | 08:18 |
| *** atmark has left #openstack-neutron | 08:18 | |
| *** vivek-ebay has quit IRC | 08:19 | |
| *** kawa2014 has joined #openstack-neutron | 08:22 | |
| *** markvoelker has joined #openstack-neutron | 08:25 | |
| *** nmagnezi has quit IRC | 08:28 | |
| *** nmagnezi has joined #openstack-neutron | 08:28 | |
| *** markvoelker has quit IRC | 08:29 | |
| *** pradipta has quit IRC | 08:29 | |
| *** kawa2014 has quit IRC | 08:31 | |
| *** Krast has quit IRC | 08:33 | |
| *** achanda has joined #openstack-neutron | 08:33 | |
| *** insequent has quit IRC | 08:33 | |
| *** salv-orlando has quit IRC | 08:33 | |
| *** afazekas has joined #openstack-neutron | 08:33 | |
| *** luqas__ has joined #openstack-neutron | 08:36 | |
| *** pradipta has joined #openstack-neutron | 08:42 | |
| *** jlanoux has joined #openstack-neutron | 08:42 | |
| *** ggnel has quit IRC | 08:43 | |
| *** kawa2014 has joined #openstack-neutron | 08:43 | |
| *** sbalukoff has quit IRC | 08:43 | |
| *** brad[] has quit IRC | 08:44 | |
| *** devvesa has joined #openstack-neutron | 08:44 | |
| *** devvesa has quit IRC | 08:47 | |
| *** devvesa has joined #openstack-neutron | 08:48 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 08:48 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 08:48 |
| *** lhcheng has left #openstack-neutron | 08:48 | |
| *** jistr has joined #openstack-neutron | 08:49 | |
| *** sbalukoff has joined #openstack-neutron | 08:55 | |
| *** kawa2014 has quit IRC | 08:55 | |
| *** hichihara has quit IRC | 08:56 | |
| *** jpena has joined #openstack-neutron | 08:57 | |
| *** karimb has joined #openstack-neutron | 09:00 | |
| *** dims has joined #openstack-neutron | 09:01 | |
| *** ajo has joined #openstack-neutron | 09:01 | |
| *** aepifanov has joined #openstack-neutron | 09:02 | |
| *** manishg has joined #openstack-neutron | 09:02 | |
| *** moha_hunt has joined #openstack-neutron | 09:03 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add sanity check for wrong MTU when using tunnels https://review.openstack.org/129852 | 09:04 |
| *** salv-orlando has joined #openstack-neutron | 09:05 | |
| *** dims has quit IRC | 09:06 | |
| *** manishg has quit IRC | 09:07 | |
| *** yamahata has quit IRC | 09:07 | |
| *** kawa2014 has joined #openstack-neutron | 09:10 | |
| *** roeyc has quit IRC | 09:12 | |
| *** achanda has quit IRC | 09:12 | |
| *** watanabe_isao has quit IRC | 09:15 | |
| *** ygbo has joined #openstack-neutron | 09:18 | |
| *** SridharG has left #openstack-neutron | 09:20 | |
| *** ihrachyshka has joined #openstack-neutron | 09:22 | |
| *** markvoelker has joined #openstack-neutron | 09:26 | |
| *** markvoelker has quit IRC | 09:31 | |
| *** vthapar has quit IRC | 09:31 | |
| *** popw has quit IRC | 09:36 | |
| ZZelle | jschwarz, hi | 09:36 |
| jschwarz | ZZelle, hello :) | 09:37 |
| ZZelle | jschwarz, i responded to your remarks | 09:37 |
| jschwarz | ZZelle, I saw. I'll probably have time to look at it in an hour or so | 09:38 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 09:38 |
| ZZelle | jschwarz, ^^ | 09:39 |
| *** erkules_ is now known as erkules | 09:39 | |
| *** stanzgy_ has quit IRC | 09:40 | |
| *** stanzgy has quit IRC | 09:40 | |
| *** gongysh_ has quit IRC | 09:40 | |
| openstackgerrit | Moshe Levi proposed openstack/neutron: Add mellanox decomposition progress to chart https://review.openstack.org/161651 | 09:41 |
| openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script https://review.openstack.org/156563 | 09:42 |
| *** doude has joined #openstack-neutron | 09:47 | |
| *** kawa2014 has quit IRC | 09:47 | |
| openstackgerrit | gongysh proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 09:49 |
| *** igordcard has joined #openstack-neutron | 09:49 | |
| *** hanzhang has joined #openstack-neutron | 09:52 | |
| *** gongysh_ has joined #openstack-neutron | 09:53 | |
| *** moshele has quit IRC | 09:54 | |
| *** salv-orlando has quit IRC | 09:59 | |
| *** safchain has joined #openstack-neutron | 10:00 | |
| *** liusheng has quit IRC | 10:00 | |
| *** kawa2014 has joined #openstack-neutron | 10:01 | |
| *** liusheng has joined #openstack-neutron | 10:01 | |
| *** aepifanov has quit IRC | 10:01 | |
| *** manishg has joined #openstack-neutron | 10:03 | |
| *** aepifanov has joined #openstack-neutron | 10:04 | |
| *** yuanying has quit IRC | 10:05 | |
| *** anilvenkata has left #openstack-neutron | 10:06 | |
| *** wendar has quit IRC | 10:06 | |
| *** manishg has quit IRC | 10:08 | |
| *** numan has quit IRC | 10:08 | |
| *** txomon|fon has quit IRC | 10:08 | |
| *** moshele has joined #openstack-neutron | 10:09 | |
| *** topshare has quit IRC | 10:11 | |
| *** yushiro has joined #openstack-neutron | 10:12 | |
| *** kawa2014 has quit IRC | 10:14 | |
| matrohon | safchain : hi | 10:15 |
| *** wendar has joined #openstack-neutron | 10:16 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas: Insert validation in creating/updating firewall https://review.openstack.org/147396 | 10:17 |
| *** tomoe_ has quit IRC | 10:17 | |
| *** nyechiel__ has quit IRC | 10:22 | |
| *** markmcclain has quit IRC | 10:22 | |
| *** sluo_wfh has quit IRC | 10:22 | |
| safchain | matrohon, hi | 10:23 |
| matrohon | safchain : when you'll have spare time, please give us your feedback on : https://review.openstack.org/#/c/141114/ | 10:24 |
| safchain | matrohon, sure I'll try | 10:24 |
| matrohon | safchain : thanks | 10:25 |
| *** markmcclain has joined #openstack-neutron | 10:26 | |
| *** markvoelker has joined #openstack-neutron | 10:27 | |
| *** kawa2014 has joined #openstack-neutron | 10:28 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron: Vxlan UDP port value should be updated in neutron DB https://review.openstack.org/153891 | 10:28 |
| *** markvoelker has quit IRC | 10:31 | |
| *** saju_m has joined #openstack-neutron | 10:33 | |
| *** alexpilotti has joined #openstack-neutron | 10:34 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 10:36 |
| *** shaohe_feng has quit IRC | 10:38 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 10:38 |
| openstackgerrit | Sergey Belous proposed openstack/neutron: Added check for emptyness where in_ is being used https://review.openstack.org/150034 | 10:40 |
| openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 10:48 |
| *** yushiro has quit IRC | 10:49 | |
| *** jp_at_hp has joined #openstack-neutron | 10:51 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy to log with nobody user/group https://review.openstack.org/161494 | 10:54 |
| *** aix has joined #openstack-neutron | 10:55 | |
| *** dims has joined #openstack-neutron | 10:55 | |
| *** nyechiel__ has joined #openstack-neutron | 10:56 | |
| *** lazy_prince is now known as killer_prince | 10:57 | |
| *** salv-orlando has joined #openstack-neutron | 11:00 | |
| *** xuhanp has quit IRC | 11:04 | |
| *** manishg has joined #openstack-neutron | 11:04 | |
| *** manishg has quit IRC | 11:08 | |
| *** trinaths has left #openstack-neutron | 11:08 | |
| *** rotbeard has joined #openstack-neutron | 11:09 | |
| *** nyechiel__ has quit IRC | 11:10 | |
| *** pc_m has joined #openstack-neutron | 11:12 | |
| *** kawa2014 has quit IRC | 11:14 | |
| *** kawa2014 has joined #openstack-neutron | 11:14 | |
| openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script https://review.openstack.org/156563 | 11:14 |
| *** pradipta has quit IRC | 11:16 | |
| *** kickinz1 is now known as kickinz1|afk | 11:19 | |
| openstackgerrit | Moshe Levi proposed openstack/neutron: Add Mellanox decomposition progress to chart https://review.openstack.org/161651 | 11:19 |
| *** kickinz1|afk is now known as kickinz1 | 11:21 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Fix validation of physical network name for flat nets https://review.openstack.org/160842 | 11:23 |
| *** doude_ has joined #openstack-neutron | 11:24 | |
| *** bjornar has joined #openstack-neutron | 11:25 | |
| *** sbfox has quit IRC | 11:27 | |
| *** anish_ has quit IRC | 11:27 | |
| *** doude has quit IRC | 11:28 | |
| *** markvoelker has joined #openstack-neutron | 11:28 | |
| *** markvoelker has quit IRC | 11:32 | |
| *** nus has joined #openstack-neutron | 11:33 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 11:34 |
| openstackgerrit | Sergey Belous proposed openstack/neutron: Added check for emptyness where in_ is being used https://review.openstack.org/150034 | 11:38 |
| *** salv-orlando has quit IRC | 11:42 | |
| *** yog_ has joined #openstack-neutron | 11:44 | |
| *** saju_m has quit IRC | 11:53 | |
| openstackgerrit | yalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 11:53 |
| *** devvesa has quit IRC | 11:54 | |
| *** irenab has quit IRC | 11:54 | |
| *** saju_m has joined #openstack-neutron | 11:55 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS DVR UT: Reduce indentation depth https://review.openstack.org/161101 | 11:55 |
| openstackgerrit | yalei wang proposed openstack/neutron: Add portsecurity extension support https://review.openstack.org/126552 | 11:58 |
| *** irenab has joined #openstack-neutron | 11:59 | |
| *** EmilienM|afk is now known as EmilienM | 12:04 | |
| *** manishg has joined #openstack-neutron | 12:04 | |
| *** saju_m has quit IRC | 12:06 | |
| *** afazekas has quit IRC | 12:08 | |
| *** manishg has quit IRC | 12:09 | |
| *** blues-man has joined #openstack-neutron | 12:09 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Big Switch Networks code split https://review.openstack.org/160360 | 12:13 |
| *** claudiub has quit IRC | 12:16 | |
| *** evgenyf has quit IRC | 12:21 | |
| *** kirshil has joined #openstack-neutron | 12:22 | |
| *** afazekas has joined #openstack-neutron | 12:22 | |
| openstackgerrit | Sam Betts proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 12:26 |
| *** mwagner_lap has quit IRC | 12:26 | |
| *** evgenyf 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 |
| *** markvoelker has joined #openstack-neutron | 12:29 | |
| *** afazekas has quit IRC | 12:31 | |
| *** evgenyf has quit IRC | 12:31 | |
| *** moha_hunt has quit IRC | 12:32 | |
| *** Marga_ has joined #openstack-neutron | 12:33 | |
| *** markvoelker has quit IRC | 12:34 | |
| *** irenab has quit IRC | 12:36 | |
| *** evgenyf has joined #openstack-neutron | 12:37 | |
| *** devvesa has joined #openstack-neutron | 12:37 | |
| *** markvoelker has joined #openstack-neutron | 12:38 | |
| *** coolsvap_ has quit IRC | 12:38 | |
| openstackgerrit | Sam Betts proposed openstack/neutron: Add an internal DHCPv6 client for Prefix Delegation https://review.openstack.org/161381 | 12:39 |
| *** Marga_ has quit IRC | 12:40 | |
| *** afazekas has joined #openstack-neutron | 12:43 | |
| *** blues-man has quit IRC | 12:43 | |
| *** coolsvap_ has joined #openstack-neutron | 12:46 | |
| *** neeti has quit IRC | 12:46 | |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: Prevent calling waitall() inside a GreenPool's greenthread https://review.openstack.org/157320 | 12:47 |
| openstackgerrit | Livnat Peer proposed openstack/neutron: Use an existing function in process monitor tests https://review.openstack.org/161698 | 12:48 |
| *** raildo_away is now known as raildo | 12:51 | |
| *** dmellado has quit IRC | 12:52 | |
| *** salv-orlando has joined #openstack-neutron | 12:56 | |
| *** yatin has quit IRC | 12:56 | |
| *** baoli has joined #openstack-neutron | 12:57 | |
| *** banix has joined #openstack-neutron | 12:57 | |
| *** baoli has quit IRC | 12:57 | |
| *** tsg has joined #openstack-neutron | 12:58 | |
| *** mwagner_lap has joined #openstack-neutron | 13:01 | |
| *** dmellado has joined #openstack-neutron | 13:02 | |
| *** manishg has joined #openstack-neutron | 13:05 | |
| *** baoli has joined #openstack-neutron | 13:06 | |
| *** jckasper has quit IRC | 13:08 | |
| *** manishg has quit IRC | 13:10 | |
| *** zhenq has joined #openstack-neutron | 13:11 | |
| *** yamamoto has quit IRC | 13:13 | |
| openstackgerrit | Yoni Shafrir proposed openstack/neutron: Adding support for Keepalived health check script https://review.openstack.org/156563 | 13:15 |
| *** ihrachyshka has quit IRC | 13:18 | |
| *** yfauser1 has joined #openstack-neutron | 13:19 | |
| *** devvesa has quit IRC | 13:20 | |
| *** devvesa has joined #openstack-neutron | 13:20 | |
| *** amotoki has quit IRC | 13:21 | |
| *** ajmiller_ has joined #openstack-neutron | 13:21 | |
| *** yfauser1 has left #openstack-neutron | 13:22 | |
| *** ihrachyshka has joined #openstack-neutron | 13:22 | |
| *** [1]evgenyf has joined #openstack-neutron | 13:22 | |
| *** ajmiller has quit IRC | 13:24 | |
| *** Bsony_ has joined #openstack-neutron | 13:24 | |
| *** evgenyf has quit IRC | 13:25 | |
| *** [1]evgenyf is now known as evgenyf | 13:25 | |
| *** Bsony has quit IRC | 13:26 | |
| openstackgerrit | gongysh proposed openstack/neutron: Add extra subnet route to ha router https://review.openstack.org/159772 | 13:27 |
| *** bobmel has quit IRC | 13:27 | |
| *** Marga_ has joined #openstack-neutron | 13:28 | |
| *** bobmel has joined #openstack-neutron | 13:28 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 13:30 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: Decompose the VMware plugin https://review.openstack.org/160463 | 13:30 |
| *** banix has quit IRC | 13:31 | |
| jschwarz | ZZelle, ping | 13:34 |
| jschwarz | ZZelle, responded to your comments. ready to ping pong with you :> | 13:34 |
| *** aix has quit IRC | 13:34 | |
| *** yamamoto has joined #openstack-neutron | 13:37 | |
| *** yog__ has joined #openstack-neutron | 13:38 | |
| *** dmellado_ has joined #openstack-neutron | 13:40 | |
| *** cipcosma has joined #openstack-neutron | 13:40 | |
| *** dmellado has quit IRC | 13:40 | |
| *** dmellado_ is now known as dmellado | 13:40 | |
| *** yog_ has quit IRC | 13:42 | |
| *** amuller has joined #openstack-neutron | 13:44 | |
| *** yshafrir has quit IRC | 13:44 | |
| *** Marga_ has quit IRC | 13:47 | |
| openstackgerrit | Bradley Jones proposed openstack/neutron: Add an internal DHCPv6 client for Prefix Delegation https://review.openstack.org/161381 | 13:48 |
| *** anish__ has joined #openstack-neutron | 13:48 | |
| *** chandankumar has quit IRC | 13:50 | |
| *** dims has quit IRC | 13:51 | |
| mestery | Morning folks! | 13:51 |
| mestery | Reminder: Today is Feature Proposal Freeze (FPF): https://wiki.openstack.org/wiki/FeatureProposalFreeze | 13:52 |
| *** dims has joined #openstack-neutron | 13:52 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook https://review.openstack.org/161714 | 13:52 |
| openstackgerrit | Mike Kolesnik proposed openstack/neutron: Add proccess monitor to keepalived https://review.openstack.org/153162 | 13:52 |
| openstackgerrit | Mike Kolesnik proposed openstack/neutron: Refactor the ProcessMonitor API https://review.openstack.org/154464 | 13:52 |
| amuller | armax: Good morning | 13:52 |
| *** karimb has quit IRC | 13:53 | |
| *** cbouch has joined #openstack-neutron | 13:54 | |
| jschwarz | today is the FPF? :O | 13:55 |
| *** moha_hunt has joined #openstack-neutron | 13:55 | |
| amuller | jschwarz: that means proposal of new feature patches | 13:56 |
| amuller | jschwarz: not the merge of said patches | 13:56 |
| *** Marga_ has joined #openstack-neutron | 13:57 | |
| jschwarz | amuller, but I had a sudden urge to propose hunderds of patches today :< | 13:57 |
| amuller | Today you still can, go ahead :) | 13:57 |
| amuller | mestery won't mind ;-) | 13:57 |
| jschwarz | I meant tomorrow!! | 13:57 |
| jschwarz | not today! | 13:57 |
| jschwarz | >.> | 13:57 |
| jschwarz | <.< | 13:57 |
| jschwarz | >.< | 13:57 |
| mestery | lol | 13:57 |
| mestery | You'd be surprised how many people are not aware FPF exists. | 13:57 |
| openstackgerrit | Pranav Salunke proposed openstack/neutron: Add non-isolated network metadata server route https://review.openstack.org/159580 | 13:57 |
| *** gongysh_ has quit IRC | 13:58 | |
| jschwarz | mestery, surprisingly it's written on the board behind me but I don't look there anymore | 13:58 |
| jschwarz | *cough* amuller *cough* | 13:58 |
| mestery | rofl | 13:58 |
| jschwarz | kilo-3 is 2 weeks off, right? | 13:59 |
| amuller | *sad face* That's the last thing I wrote on our board John and you don't even look :( | 13:59 |
| jschwarz | amuller, it depresses me too much!! :< | 13:59 |
| mestery | pc_m: What's the status of this one: https://review.openstack.org/#/c/144391/ | 14:00 |
| mestery | pc_m: Is that likely to be mergable soon? | 14:00 |
| *** jckasper has joined #openstack-neutron | 14:01 | |
| *** aix has joined #openstack-neutron | 14:02 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info https://review.openstack.org/152512 | 14:03 |
| mestery | ajo: Did you figure out why the tempest tests were failing on ^^^^ ??? | 14:03 |
| ajo | mestery, I think I did, yes... :) programmers make stupid mistakes :D | 14:04 |
| mestery | ajo: lol ;) | 14:04 |
| *** rotbeard has quit IRC | 14:04 | |
| ajo | mestery, at some point I was overwriting a defaultdict, with a non-defaultdict (received from RPC)... | 14:04 |
| mestery | yikes | 14:05 |
| ajo | and... I expected it to behave as a defaultdict.. | 14:05 |
| *** dboik has joined #openstack-neutron | 14:05 | |
| ajo | so I hope it will be ok now :) | 14:05 |
| ajo | and also, I wrote all the tests I wanted | 14:05 |
| * ajo crosses fingers O:) | 14:05 | |
| pc_m | mestery: hi. I think it can. The latest changes look fine to me. Hung up due to grenade failures. Badly need additional reviewers. | 14:05 |
| mestery | pc_m: Are the grenade failures due to the patch itself? Is there a grenade review out there which fixes them? | 14:06 |
| *** manishg has joined #openstack-neutron | 14:06 | |
| pc_m | mestery: No, due to the breakage AFAIK. But still waiting for test results due to latest patch. | 14:07 |
| pc_m | mestery: There several commits for VPN needing reviewers. | 14:08 |
| *** criptoverso1 has joined #openstack-neutron | 14:08 | |
| *** criptoverso has quit IRC | 14:08 | |
| *** rotbeard has joined #openstack-neutron | 14:08 | |
| *** shwetaap has joined #openstack-neutron | 14:08 | |
| mestery | pc_m: I see them there, I'm taking a look this morning, feel free to ping me if you're getting stuck! | 14:08 |
| *** dane_leblanc has quit IRC | 14:08 | |
| pc_m | ping | 14:08 |
| pc_m | :) | 14:09 |
| ajo | mestery, we're keeping the QoS discussion offline lately, but I believe it would be good to assemble a subgroup for it, and have meetings, how could I organize that? | 14:09 |
| *** kbringard has joined #openstack-neutron | 14:09 | |
| ajo | I want to be in shape, to be able to have an updated spec before the summit, | 14:10 |
| pc_m | mestery: A few things... I'm waiting for 147741 to merge (it failed tests and I did a recheck), once done, I need to get review for 160983 through (to fix the resulting UT failures) | 14:10 |
| ajo | NFV people is asking for that kind of stuff :) | 14:10 |
| *** manishg has quit IRC | 14:10 | |
| mestery | ajo: Happy to help there! Should we look at having a working session in Vancouver for QoS? | 14:11 |
| mestery | pc_m: Keep me in hte loop for reviews today, I'll be around. | 14:11 |
| ajo | mestery, IMHO, defintely! | 14:11 |
| mestery | ajo: Maybe a weekly meeting on IRC once Kilo-3 is done and we're more in hand there? | 14:11 |
| *** banix has joined #openstack-neutron | 14:11 | |
| ajo | mestery, that would be awesome, I would be happy to host/lead the group | 14:12 |
| ajo | the meeting | 14:12 |
| ajo | whatever :D | 14:12 |
| *** blues-man has joined #openstack-neutron | 14:12 | |
| *** blues-man has quit IRC | 14:12 | |
| *** blues-man has joined #openstack-neutron | 14:12 | |
| pc_m | mestery: I have a refactoring review out that needs reviewers, https://review.openstack.org/160179. I already have the second part of that ready (due to the grenade testing delay). Wondering if I should merge the two? | 14:13 |
| mestery | ajo: Ack, sounds good, lets connect soon on this, now back to FPF wrangling for me :) | 14:13 |
| ajo | sure, priorities are important :) | 14:13 |
| *** yfauser has joined #openstack-neutron | 14:13 | |
| pc_m | mestery: Also, looks like coverage tests for VPN repo are no longer running. I'm not sure what has changed in testr in the past month, but it no longer runs. | 14:13 |
| *** yfauser has left #openstack-neutron | 14:14 | |
| *** anish__ has quit IRC | 14:14 | |
| mestery | pc_m: 160179 looks good to me, lets see if we can get carl_baldwin and/or armax to quickly merge it once they arrive this morning. | 14:15 |
| mestery | pc_m: Weird on the coverage tests. | 14:15 |
| *** gongysh_ has joined #openstack-neutron | 14:15 | |
| pc_m | mestery: IT's saying option --coverage-package-name is unknown. | 14:16 |
| pc_m | mestery: Without it, then it doesn't know about the neutron-vpnaas import. | 14:16 |
| mestery | pc_m: That's no good. I'll BRB. | 14:17 |
| pc_m | mestery: On 2/9, I did a commit that added that option in (per someone's advice), so that it would work (otherwise it says neutron-vpnaas was never imported and doesn't do anything). | 14:17 |
| pc_m | mestery: Now that option is being rejected. Something must have changed with testr | 14:17 |
| *** anshul has quit IRC | 14:18 | |
| *** radez_g0n3 is now known as radez | 14:18 | |
| *** rotbeard has quit IRC | 14:21 | |
| *** johnbelamaric has joined #openstack-neutron | 14:22 | |
| *** doude_ has quit IRC | 14:22 | |
| *** doude has joined #openstack-neutron | 14:22 | |
| amuller | mestery: https://review.openstack.org/#/c/155058/ is the first patch in the bp/report-ha-router-state series | 14:22 |
| amuller | currently sitting pretty with one +2 | 14:22 |
| amuller | Just throwing that fact ou there :) | 14:23 |
| *** amotoki has joined #openstack-neutron | 14:23 | |
| *** yfauser has joined #openstack-neutron | 14:24 | |
| *** karimb has joined #openstack-neutron | 14:24 | |
| *** Marga_ has quit IRC | 14:25 | |
| *** yfauser1 has joined #openstack-neutron | 14:25 | |
| *** yfauser1 has quit IRC | 14:26 | |
| *** yfauser has quit IRC | 14:27 | |
| *** rotbeard has joined #openstack-neutron | 14:28 | |
| *** shwetaap1 has joined #openstack-neutron | 14:28 | |
| *** johnbelamaric has quit IRC | 14:28 | |
| *** tiswanso has joined #openstack-neutron | 14:29 | |
| *** zzzeek has joined #openstack-neutron | 14:29 | |
| *** shwetaap has quit IRC | 14:31 | |
| *** johnbelamaric has joined #openstack-neutron | 14:32 | |
| *** shaohe_feng has joined #openstack-neutron | 14:33 | |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: [WIP]: Add reset methods to WsgiWorker and RPCWorker https://review.openstack.org/161732 | 14:33 |
| openstackgerrit | Merged openstack/neutron: Updated from global requirements https://review.openstack.org/161577 | 14:36 |
| *** vthapar has joined #openstack-neutron | 14:38 | |
| *** jgrimmn has quit IRC | 14:38 | |
| *** dane_leblanc has joined #openstack-neutron | 14:39 | |
| *** cference_ has joined #openstack-neutron | 14:39 | |
| *** absubram has joined #openstack-neutron | 14:39 | |
| *** jgrimm has joined #openstack-neutron | 14:39 | |
| *** annegentle has joined #openstack-neutron | 14:39 | |
| *** busterswt has joined #openstack-neutron | 14:40 | |
| openstackgerrit | Merged openstack/neutron: Refactor management of namespaces in the L3 Agent https://review.openstack.org/147744 | 14:40 |
| mestery | amuller: I was looking at that earlier, looking now. | 14:42 |
| *** anish__ has joined #openstack-neutron | 14:43 | |
| *** jgrimm has quit IRC | 14:44 | |
| *** Murali has quit IRC | 14:44 | |
| *** rushil has joined #openstack-neutron | 14:44 | |
| *** mrawlings has joined #openstack-neutron | 14:45 | |
| *** jgrimm has joined #openstack-neutron | 14:45 | |
| *** r-daneel has joined #openstack-neutron | 14:46 | |
| *** chuckC_ has joined #openstack-neutron | 14:46 | |
| *** chuckC_ has quit IRC | 14:48 | |
| *** manishg has joined #openstack-neutron | 14:49 | |
| *** thomasem has joined #openstack-neutron | 14:50 | |
| *** thomasem has quit IRC | 14:51 | |
| amuller | HenryG: Thanks for approving that patch | 14:51 |
| HenryG | amuller: bit of a no-brainer :) | 14:52 |
| *** zzzeek has quit IRC | 14:52 | |
| amuller | HenryG: heh try the next one in the series then ;-) | 14:52 |
| *** yamamoto has quit IRC | 14:53 | |
| jschwarz | no! it's a trap! ;-) | 14:53 |
| *** mrawlings has quit IRC | 14:54 | |
| HenryG | amuller: yeah, I am aware of it. o_O | 14:54 |
| *** kawa2014 has quit IRC | 14:54 | |
| *** yog__ has quit IRC | 14:55 | |
| *** thomasem has joined #openstack-neutron | 14:55 | |
| haleyb | dane_leblanc: can you take a look at https://bugs.launchpad.net/neutron/+bug/1428305 especially the comment I just added? Ammunition for getting your IPv6 patches merged | 14:55 |
| openstack | Launchpad bug 1428305 in neutron "Floating IP namespace not created when DVR enabled" [Undecided,New] - Assigned to Brian Haley (brian-haley) | 14:55 |
| *** radez is now known as radez_g0n3 | 14:56 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: device driver and agent refactoring https://review.openstack.org/160179 | 14:56 |
| *** kawa2014 has joined #openstack-neutron | 14:56 | |
| *** jckasper has quit IRC | 14:57 | |
| *** carl_baldwin has joined #openstack-neutron | 14:57 | |
| mestery | amuller: HenryG beat me to that patch :) | 14:58 |
| pc_m | mestery: I combined part 1 and 2 for this ^^ | 14:58 |
| amuller | mestery: :) | 14:58 |
| mestery | pc_m: Looking | 14:58 |
| *** mlavalle has joined #openstack-neutron | 14:58 | |
| *** moshele has quit IRC | 14:59 | |
| mestery | amuller: I got sidetracked on 4 things after you pinged me. o_O | 14:59 |
| *** nmagnezi has quit IRC | 14:59 | |
| HenryG | mestery: I was already reviewing it before amuller pinged you | 14:59 |
| mestery | HenryG: Thanks! :) | 14:59 |
| *** yamamoto has joined #openstack-neutron | 14:59 | |
| *** krtaylor has quit IRC | 14:59 | |
| * mestery walks behind HenryG into the trap the next patch in the series has set for us ... | 14:59 | |
| *** jckasper has joined #openstack-neutron | 15:00 | |
| *** karimb has quit IRC | 15:02 | |
| *** mrawlings has joined #openstack-neutron | 15:03 | |
| *** karimb has joined #openstack-neutron | 15:03 | |
| dane_leblanc | haleyb: Interesting | 15:04 |
| dane_leblanc | haleyb: Do you know if just the dual-stack patch fixes the problem, or both patches are needed? | 15:05 |
| *** baoli has quit IRC | 15:05 | |
| haleyb | dane_leblanc: both are needed from my testing | 15:06 |
| *** amrith has joined #openstack-neutron | 15:08 | |
| *** banix has quit IRC | 15:08 | |
| *** prad has joined #openstack-neutron | 15:09 | |
| *** baoli__ has joined #openstack-neutron | 15:10 | |
| dane_leblanc | haleyb: I can see why the first is necessary, since the "FloatingIP agent gateway port received from the plugin" has multiple subnets in the fixed_ips list | 15:10 |
| *** mrawlings has quit IRC | 15:10 | |
| haleyb | dane_leblanc: yes, i thought it would work by itself with that change, but it didn't. I can retry to double-check that | 15:12 |
| dane_leblanc | haleyb: Are you using devstack with "IP_VERSION=4+6"? | 15:12 |
| haleyb | yes, dual-stack | 15:12 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: device driver and agent refactoring https://review.openstack.org/160179 | 15:12 |
| dane_leblanc | haleyb: Yeah, the dual-stack DevStack won't work without at least the first patch for dual-stack on gateway ports | 15:13 |
| *** mrawlings has joined #openstack-neutron | 15:13 | |
| haleyb | it used to before the FIP namespace re-factoring | 15:16 |
| *** manishg has quit IRC | 15:18 | |
| *** crose has joined #openstack-neutron | 15:18 | |
| *** ChuckC has quit IRC | 15:19 | |
| *** yamahata has joined #openstack-neutron | 15:20 | |
| *** tsg has quit IRC | 15:20 | |
| *** insequent has joined #openstack-neutron | 15:21 | |
| dane_leblanc | haleyb: dboik is also going to look at the second patch to see why it's required to fix this, but we can't see why that patch is required. | 15:22 |
| dane_leblanc | haleyb: I'm surprised the dual-stack DevStack worked before the re-factoring without the first patch. | 15:23 |
| *** kawa2014 has quit IRC | 15:23 | |
| *** kawa2014 has joined #openstack-neutron | 15:24 | |
| *** banix has joined #openstack-neutron | 15:24 | |
| *** krtaylor has joined #openstack-neutron | 15:25 | |
| *** ajmiller_ is now known as ajmiller | 15:26 | |
| *** marun has joined #openstack-neutron | 15:26 | |
| haleyb | dane_leblanc: well, devstack is doing a little bit of config to make things work, like adding an IPv6 address an route to the router manually | 15:27 |
| *** spandhe has joined #openstack-neutron | 15:27 | |
| openstackgerrit | Vijay Chundury proposed openstack/neutron: Remove references to 0.0.0.0/0 in iptable rules https://review.openstack.org/161214 | 15:27 |
| *** moha_hunt has quit IRC | 15:28 | |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: Decompose the VMware plugin https://review.openstack.org/160463 | 15:29 |
| dane_leblanc | haleyb: Yes, the current DevStack script will do some things manually like add a V6 subnet on the public, but this should fail to update the gateway port (another bug that's in review), so v6 external access will fail, but v4 should work fine | 15:30 |
| dane_leblanc | haleyb: But the multiple subnets in the fixed_ips for the gateway port should cause problems without the first patch | 15:31 |
| *** armax has quit IRC | 15:33 | |
| haleyb | dane_leblanc: perhaps it's the "not update the external port" that made it work, since that seems to be causing the current problem | 15:34 |
| *** armax has joined #openstack-neutron | 15:34 | |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 15:35 |
| haleyb | dane_leblanc: i just confirmed just your dual-stack patch doesn't fix this, so i didn't screw that up :) | 15:38 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Fix unit test breakage https://review.openstack.org/160983 | 15:38 |
| *** rotbeard has quit IRC | 15:38 | |
| *** garyk has quit IRC | 15:40 | |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 15:40 |
| *** insequent has quit IRC | 15:40 | |
| *** insequent has joined #openstack-neutron | 15:43 | |
| *** crose has quit IRC | 15:45 | |
| *** insequent has quit IRC | 15:46 | |
| *** krtaylor has quit IRC | 15:50 | |
| amuller | gongysh_: ping about https://review.openstack.org/#/c/159772/10 | 15:50 |
| *** rkukura has joined #openstack-neutron | 15:50 | |
| amuller | it's no longer rebased on my patch, why is that? | 15:50 |
| openstackgerrit | Merged openstack/neutron: Change metadata driver unit tests to use monitored spawn https://review.openstack.org/155058 | 15:51 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook https://review.openstack.org/161714 | 15:51 |
| *** _cerberus_ has joined #openstack-neutron | 15:51 | |
| *** jprovazn has joined #openstack-neutron | 15:53 | |
| openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 15:54 |
| *** crose has joined #openstack-neutron | 15:56 | |
| *** bquan has joined #openstack-neutron | 15:57 | |
| *** baoli__ has quit IRC | 16:01 | |
| *** spandhe has quit IRC | 16:01 | |
| *** krtaylor has joined #openstack-neutron | 16:02 | |
| *** dboik_ has joined #openstack-neutron | 16:02 | |
| *** baoli has joined #openstack-neutron | 16:02 | |
| *** dboik has quit IRC | 16:05 | |
| *** manishg has joined #openstack-neutron | 16:05 | |
| *** shwetaap1 has quit IRC | 16:05 | |
| *** pcm_ has joined #openstack-neutron | 16:06 | |
| *** manishg_ has joined #openstack-neutron | 16:07 | |
| *** pc_m has quit IRC | 16:07 | |
| *** carl_baldwin has quit IRC | 16:08 | |
| *** trinaths has joined #openstack-neutron | 16:09 | |
| *** tiswanso has quit IRC | 16:09 | |
| *** manishg has quit IRC | 16:10 | |
| *** tiswanso has joined #openstack-neutron | 16:10 | |
| *** jpena has quit IRC | 16:11 | |
| anteaya | kevinbenton: I just want to confirm you only want master and juno branches on your new repo, yes? | 16:11 |
| *** trinaths1 has joined #openstack-neutron | 16:13 | |
| *** xgerman has joined #openstack-neutron | 16:13 | |
| *** trinaths has quit IRC | 16:14 | |
| *** rkukura has quit IRC | 16:15 | |
| anteaya | armax: where did you get the BRANCH variable from: https://review.openstack.org/#/c/161569/ I can't confirm it | 16:16 |
| armax | anteaya: wild guess, I am looking for confirmation by the infra top dogs | 16:17 |
| *** ronis has quit IRC | 16:17 | |
| armax | anteaya: I was hoping you could help | 16:17 |
| *** ChuckC has joined #openstack-neutron | 16:17 | |
| *** imcsk8 has quit IRC | 16:17 | |
| *** imcsk8 has joined #openstack-neutron | 16:17 | |
| *** ChuckC has quit IRC | 16:18 | |
| *** vthapar has quit IRC | 16:18 | |
| *** ChuckC has joined #openstack-neutron | 16:18 | |
| *** shwetaap has joined #openstack-neutron | 16:21 | |
| anteaya | great | 16:21 |
| anteaya | well I can confirm that I can't find that variable | 16:21 |
| anteaya | not sure if that makes me a top dog or not | 16:21 |
| anteaya | so we need to find some variable that devstack-gate already knows about | 16:21 |
| anteaya | are you able to dig around and see if you can find something that exists that will suit? | 16:22 |
| openstackgerrit | Michael Smith proposed openstack/neutron: WIP delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 16:23 |
| anteaya | the other option is to introduce a new variable into devstack-gate | 16:24 |
| anteaya | but that will take longer and you have to be able to demonstrate that you have evaluated all current variables and none exist for your use case | 16:24 |
| anteaya | plus that your use case is valid | 16:24 |
| *** ihrachyshka has quit IRC | 16:25 | |
| *** jpena has joined #openstack-neutron | 16:26 | |
| *** Marga_ has joined #openstack-neutron | 16:27 | |
| *** evgenyf has quit IRC | 16:28 | |
| *** garyk has joined #openstack-neutron | 16:28 | |
| ZZelle | jschwarz, your concern on VethBridgeFixture is the more the implementation or the name | 16:28 |
| jschwarz | ZZelle, implementation | 16:28 |
| jschwarz | ZZelle, I think both classes should be merged into one, and then the naming will also change | 16:28 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 16:29 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 16:29 |
| jschwarz | ZZelle, you actually caught me on my way out :( | 16:29 |
| *** salv-orlando has quit IRC | 16:29 | |
| ZZelle | jschwarz, go! go! | 16:29 |
| *** chuckC_ has joined #openstack-neutron | 16:29 | |
| jschwarz | ZZelle, power rangers? | 16:29 |
| *** bquan has quit IRC | 16:29 | |
| ZZelle | jschwarz, :| | 16:29 |
| *** chuckC_ has quit IRC | 16:29 | |
| jschwarz | ZZelle, XD | 16:30 |
| jschwarz | ZZelle, see you on Monday :) | 16:30 |
| ZZelle | on next monday! | 16:30 |
| ZZelle | in 9 days | 16:30 |
| jschwarz | you're away next week? | 16:30 |
| ZZelle | jschwarz, i go skiing | 16:30 |
| *** Bsony_ has quit IRC | 16:30 | |
| jschwarz | ZZelle, skiing ftw :) have fun! | 16:30 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Fix unit test breakage https://review.openstack.org/160983 | 16:31 |
| ZZelle | jschwarz, thx | 16:31 |
| *** bquan has joined #openstack-neutron | 16:31 | |
| *** tidwellr has joined #openstack-neutron | 16:31 | |
| openstackgerrit | Vijay Chundury proposed openstack/neutron: Replace IPv4 and IPv6 default addresses with constants https://review.openstack.org/160782 | 16:34 |
| *** rkukura has joined #openstack-neutron | 16:34 | |
| *** rkukura has quit IRC | 16:34 | |
| *** trad511 has joined #openstack-neutron | 16:34 | |
| *** trinaths1 has quit IRC | 16:34 | |
| *** kbringard has quit IRC | 16:34 | |
| armax | obondarev_: ping | 16:35 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 16:35 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 16:35 |
| *** jschwarz has quit IRC | 16:35 | |
| obondarev_ | armax: hi | 16:36 |
| *** obondarev_ is now known as obondarev | 16:36 | |
| armax | obondarev_: hi | 16:36 |
| armax | obondarev: re https://review.openstack.org/#/c/159679/ | 16:36 |
| enikanorov | markmcclain: hi, need your help in bumping oslo.db version in global requirements for neutron: https://review.openstack.org/#/c/160731/ | 16:36 |
| armax | obondarev: I think you and I have a bit of a momentary lapse of the reason with https://review.openstack.org/#/c/159679/2/neutron/tests/unit/db/test_l3_dvr_db.py | 16:36 |
| *** gridinv has joined #openstack-neutron | 16:36 | |
| armax | obondarev: the test is pretty pointless | 16:37 |
| armax | obondarev: :) | 16:37 |
| obondarev | armax: let me check | 16:37 |
| armax | obondarev: I am filing a fix shortly, please let me know what you think | 16:37 |
| *** kickinz1 is now known as kickinz1|afk | 16:37 | |
| armax | obondarev: it looks to me that we are asserting on the mock instance | 16:38 |
| armax | obondarev: without running any actual code | 16:38 |
| obondarev | armax: got it, funny) | 16:38 |
| armax | obondarev: I take the blame for not looking this more deeply | 16:38 |
| obondarev | armax: me too | 16:39 |
| *** salv-orlando has joined #openstack-neutron | 16:39 | |
| armax | obondarev: in the end you gotta trust no-one :) | 16:39 |
| *** kickinz1|afk is now known as kickinz1 | 16:39 | |
| markmcclain | enikanorov: done.. I've pinged another core for requirements too | 16:39 |
| armax | obondarev: not even yourself :) | 16:39 |
| obondarev | armax: indeed) | 16:39 |
| enikanorov | markmcclain: thanks a lot | 16:39 |
| armax | obondarev: posting fix shortly | 16:39 |
| armax | obondarev: just writing the commit message | 16:39 |
| obondarev | armax: cool, thanks for noticing | 16:39 |
| obondarev | armax: and letting me know | 16:39 |
| armax | obondarev: sure | 16:39 |
| armax | obondarev: no problem I am gonna tag you and swami (the test offender) :) | 16:40 |
| *** spandhe has joined #openstack-neutron | 16:40 | |
| obondarev | armax: will review the fix | 16:40 |
| *** scheuran has quit IRC | 16:40 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy to log with nobody user/group https://review.openstack.org/161494 | 16:41 |
| *** gridinv has quit IRC | 16:41 | |
| *** baoli has quit IRC | 16:42 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Fix test tautology for DVR https://review.openstack.org/161772 | 16:42 |
| armax | obondarev: ^^^ | 16:42 |
| *** rwsu-afk is now known as rwsu | 16:43 | |
| obondarev | armax: yep, thanks | 16:43 |
| *** spandhe has quit IRC | 16:44 | |
| dane_leblanc | haleyb: Ping | 16:44 |
| *** carl_baldwin has joined #openstack-neutron | 16:45 | |
| *** aepifanov has quit IRC | 16:45 | |
| *** tsg has joined #openstack-neutron | 16:46 | |
| *** krtaylor has quit IRC | 16:49 | |
| *** yamahata has quit IRC | 16:50 | |
| *** insequent has joined #openstack-neutron | 16:50 | |
| *** baoli has joined #openstack-neutron | 16:51 | |
| *** achanda has joined #openstack-neutron | 16:52 | |
| haleyb | dane_leblanc: pong | 16:52 |
| *** doude has quit IRC | 16:52 | |
| *** amotoki has quit IRC | 16:53 | |
| *** doude has joined #openstack-neutron | 16:53 | |
| openstackgerrit | Tom Holtzen proposed openstack/neutron: Vendor decomposition to move CSR1000v support to the networking-cisco repo https://review.openstack.org/159253 | 16:53 |
| *** kbringard has joined #openstack-neutron | 16:54 | |
| armax | amuller: yo | 16:55 |
| dane_leblanc | haleyb: I think I know why the 2 multi prefix patches fix the problem | 16:56 |
| amuller | armax: indubitably | 16:56 |
| * haleyb moves to edge of his seat | 16:56 | |
| amuller | yo | 16:56 |
| armax | amuller: looking at the pylint snafus | 16:56 |
| dane_leblanc | haleyb: And there could be a simple fix to the bug you filed | 16:56 |
| amuller | ok | 16:56 |
| armax | amuller: I am a bit baffled as to what’s going on to be honest, but I’ll figure it out | 16:57 |
| *** doude has quit IRC | 16:57 | |
| haleyb | dane_leblanc: great | 16:57 |
| amuller | armax: I was hoping Angus could maybe figure it out also | 16:57 |
| amuller | I'd guess it's something to do with how pylint works | 16:57 |
| dane_leblanc | haleyb: If you look in l3_db.py | 16:57 |
| amuller | and not how you're calling it | 16:58 |
| *** tiswanso has quit IRC | 16:58 | |
| armax | amuller: actually I found another issue | 16:58 |
| armax | amuller: thing is: if you check HEAD~3 for instance | 16:58 |
| haleyb | dane_leblanc: looking... | 16:58 |
| dane_leblanc | method _populate_subnet_for_ports | 16:58 |
| armax | amuller: you’d do that on the HEAD’s .pylintrc | 16:59 |
| amuller | uh huh | 16:59 |
| amuller | I see | 16:59 |
| armax | amuller: so…you understand where I am getting at? | 16:59 |
| amuller | yeah | 16:59 |
| armax | amuller: so it sounds like the check should be used with care | 16:59 |
| *** Bsony has joined #openstack-neutron | 16:59 | |
| haleyb | dane_leblanc: that looks familiar | 17:00 |
| amuller | armax: I guess we have the same problem with ./run_tests -8 | 17:00 |
| armax | amuller: I don’t think we should really go on about pulling the HEAD~? version of pylintrc | 17:00 |
| armax | amuller: ya | 17:00 |
| dane_leblanc | haleyb: There's a check for more than 1 addresses | 17:00 |
| amuller | maybe that's the reason I've found it to be unreliable and stopped using it | 17:00 |
| amuller | armax: honestly I think you're spending too much energy on this | 17:00 |
| armax | amuller: well, then revert your -1 :) | 17:01 |
| armax | amuller: no, seriously, let me figure it out really quickly why it’s breaking | 17:01 |
| *** krtaylor has joined #openstack-neutron | 17:01 | |
| amuller | ok | 17:01 |
| armax | amuller: called through tox rather than directly | 17:01 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 17:01 |
| haleyb | dane_leblanc: if len(fixed_ips) > 1: ? | 17:01 |
| armax | amuller: but then, I suspect we should let it go | 17:01 |
| dane_leblanc | haleyb: len(fixed_ips) > 1 | 17:01 |
| armax | amuller: and figure it out on the go | 17:01 |
| amuller | armax: I think that if it's unreliable people won't use it so there's no point in merging it | 17:01 |
| armax | amuller: HEAD~1 I think it’s the one that counts the most | 17:02 |
| dane_leblanc | haleyb: Yes, if that turns out to be true, it exits sort of silently (yecch!) | 17:02 |
| armax | amuller: and that is reliable | 17:02 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 17:02 |
| armax | amuller: that’s what I mean | 17:02 |
| dane_leblanc | And does not get to the place where the subnet info is filled in. | 17:02 |
| amuller | armax: Wouldn't you expect HEAD~1 to break depending on the contents of HEAD~1? | 17:02 |
| armax | amuller: people shouldn’t run on HEAD~100 willy nilly | 17:02 |
| haleyb | dane_leblanc: i do see that error in the q-svc.log file | 17:03 |
| amuller | I mean like how HEAD~2 doesn't work for whatever reason | 17:03 |
| *** zzzeek has joined #openstack-neutron | 17:03 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 17:03 |
| armax | amuller: that I’ll figure it out | 17:03 |
| armax | amuller: bear with me | 17:03 |
| dane_leblanc | haleyb: If you see that error, then that confirms the theory | 17:03 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy to log with nobody user/group https://review.openstack.org/161494 | 17:04 |
| *** kawa2014 has quit IRC | 17:04 | |
| *** jlanoux has quit IRC | 17:04 | |
| haleyb | dane_leblanc: 84 occurrences in the log | 17:04 |
| *** yamamoto has quit IRC | 17:04 | |
| *** nlahouti has joined #openstack-neutron | 17:05 | |
| ZZelle | garyk, hi, i updated https://review.openstack.org/161494 commit message | 17:05 |
| dane_leblanc | haleyb: The problem is when that error occurs, the 'subnet' entry is not filled in for the port | 17:05 |
| *** tiswanso has joined #openstack-neutron | 17:05 | |
| *** achanda has quit IRC | 17:05 | |
| dane_leblanc | haleyb: And since the 'subnet' info is not filled in, you see the secondary error: "Missing subnet/agent_gateway_port" | 17:06 |
| *** shivharis has joined #openstack-neutron | 17:06 | |
| *** vishwanathj has joined #openstack-neutron | 17:06 | |
| dane_leblanc | haleyb: If you just remove the > 1 condition, I think it should just work. | 17:06 |
| *** vishwanathj has quit IRC | 17:06 | |
| haleyb | dane_leblanc: i now see that being removed in andrew's change. I can give it a quick test. I'm assuming that won't break something else | 17:07 |
| mrawlings | all. newbie here so be gentle. I'm trying to run the python-neutronclient unit tests. I can get them to execute but not against an update that I have made. | 17:07 |
| mrawlings | I have made an edit to neutronclient/v2_0/client.py, but that files doesn't seem to be unit tested. The file neutronclient/client.py *is* unit tested. I've put breakages into both files to prove my hypothesis. | 17:07 |
| mrawlings | Is there something I am missing from my 'tox' invocation that includes the v2_0/client.py in the unit test? | 17:07 |
| mrawlings | Thanks. | 17:07 |
| *** jistr has quit IRC | 17:08 | |
| *** baoli has quit IRC | 17:08 | |
| garyk | ZZelle: k. thanks | 17:08 |
| *** luqas__ has quit IRC | 17:08 | |
| *** nlahouti has quit IRC | 17:09 | |
| *** baoli has joined #openstack-neutron | 17:09 | |
| *** bquan has quit IRC | 17:09 | |
| *** jaypipes has joined #openstack-neutron | 17:09 | |
| dane_leblanc | haleyb: I don't think it will break anything, except maybe some UTs. | 17:10 |
| *** insequent has quit IRC | 17:11 | |
| amuller | russellb: about the RPC backwards compat thingie | 17:11 |
| *** insequent has joined #openstack-neutron | 17:11 | |
| amuller | russellb: I want to see this issue resolved... It wasn't clear to me if this is something you wanted to push a fix for | 17:12 |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: [WIP]: Add reset methods to WsgiWorker and RPCWorker https://review.openstack.org/161732 | 17:12 |
| *** yamahata has joined #openstack-neutron | 17:12 | |
| haleyb | dane_leblanc: thanks for looking, i'll give that a try and see | 17:12 |
| shivharis | mestery: hi | 17:13 |
| *** _cjones_ has joined #openstack-neutron | 17:13 | |
| mestery | shivharis: yo | 17:13 |
| armax | amuller: I think I got it | 17:13 |
| armax | amuller: the pylint check was swallowing this output: | 17:13 |
| shivharis | mestery: i need to get voting privileges to the Brocade VDX CI, I realized that we have been voting for some time but can only do '0' vote | 17:13 |
| *** _cjones_ has quit IRC | 17:14 | |
| armax | amuller: pkg_resources.DistributionNotFound: oslo.config>=1.9.0 | 17:14 |
| *** bquan has joined #openstack-neutron | 17:14 | |
| armax | amuller: can you try to clean your venv? | 17:14 |
| armax | amuller: before checking again? | 17:14 |
| shivharis | mestery: talked to anteaya, she pointed to u as the point person | 17:14 |
| mestery | shivharis: Do you have a history of succesful voting on patches you can share? | 17:14 |
| *** _cjones_ has joined #openstack-neutron | 17:15 | |
| dane_leblanc | haleyb: No problem, let me know if it works. Thanks! | 17:15 |
| shivharis | mestery: we have a very long history on CI - but was disabled for some time due to decomposition - but we are voting again now | 17:15 |
| amuller | armax: when it failed for me I got specific pylint errors | 17:16 |
| shivharis | mestery: not getting the privilege was an oversight on my part | 17:16 |
| armax | amuller: oh well, can you provide more input? as we discussed earlier, the check is done using the latest pylintrc | 17:16 |
| mestery | shivharis: Ack, please ping me post 3PM CST and I'll help you out | 17:16 |
| shivharis | mestery: thanks, will do | 17:17 |
| shivharis | mestery: ttyl | 17:17 |
| *** _cjones_ has quit IRC | 17:17 | |
| amuller | armax: unrelated note is that if you give postargs I think people would expect to run flake8 on the input | 17:18 |
| *** _cjones_ has joined #openstack-neutron | 17:18 | |
| amuller | currently the patch runs flake8 on everything but pylint only on the supplied input | 17:18 |
| *** yamahata has quit IRC | 17:18 | |
| *** devvesa has quit IRC | 17:18 | |
| *** yamahata has joined #openstack-neutron | 17:19 | |
| armax | amuller: we can enahnce that in a follow up patch, imo | 17:19 |
| amuller | I agree | 17:19 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add ability to run pylint check on modified files only https://review.openstack.org/159341 | 17:19 |
| armax | amuller: ^^^ | 17:19 |
| armax | amuller: we can take care of flake8 in due course | 17:20 |
| *** igordcard has quit IRC | 17:20 | |
| amuller | armax: I removed the .tox dir, checked out your latest PS and playing around with it now | 17:22 |
| *** Longgeek has quit IRC | 17:22 | |
| armax | amuller: ok, I also moved the code checks closer together, as it makes more sense | 17:22 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add ability to run pylint check on modified files only https://review.openstack.org/159341 | 17:24 |
| shivharis | armax: hi | 17:25 |
| *** vishwanathj has joined #openstack-neutron | 17:26 | |
| *** sripriya has joined #openstack-neutron | 17:26 | |
| *** sbfox has joined #openstack-neutron | 17:27 | |
| amuller | armax: Ok I feel like I'm in the twilight zone | 17:27 |
| *** yfauser has joined #openstack-neutron | 17:28 | |
| amuller | tox -e pep8 passes | 17:28 |
| amuller | however, check this out | 17:28 |
| *** nlahouti has joined #openstack-neutron | 17:28 | |
| amuller | http://pastebin.com/uD3xfhyR | 17:28 |
| amuller | that's tox -e pep8 HEAD~3 | 17:28 |
| amuller | which is a patch of mine | 17:28 |
| haleyb | dane_leblanc: it worked, and didn't even break a UT :) | 17:28 |
| amuller | now, those errors that pylint found are real | 17:28 |
| dane_leblanc | haleyb: Woohoo! Ship it! | 17:28 |
| *** jobewan has joined #openstack-neutron | 17:28 | |
| *** yfauser has left #openstack-neutron | 17:28 | |
| *** annegentle has quit IRC | 17:29 | |
| armax | amuller: ? | 17:29 |
| amuller | armax: at least the first one | 17:29 |
| amuller | armax: if you follow the error it outputed about ha_device_cidr unused in the dict | 17:29 |
| amuller | that is true... | 17:30 |
| amuller | but just running tox -e pep8 doesn't find it?? | 17:30 |
| armax | amuller how did that sneak in? | 17:30 |
| *** sbfox1 has joined #openstack-neutron | 17:30 | |
| amuller | I don't know, I don't understand why running pylint on the entire tree doesn't find an error and running pylint on specific files does? | 17:30 |
| amuller | really odd | 17:30 |
| armax | amuller: but you can confirm that at least the random failure disappeared? | 17:31 |
| amuller | what do you mean? | 17:31 |
| *** rkukura has joined #openstack-neutron | 17:31 | |
| armax | amuller: as in you get consistent errors regardless you run using tox or coding-checks directly? | 17:31 |
| amuller | oh right | 17:31 |
| *** krtaylor has quit IRC | 17:31 | |
| armax | amuller: which is sensible as the error was in tox and it was masked | 17:31 |
| *** sbfox has quit IRC | 17:32 | |
| *** blues-man has quit IRC | 17:32 | |
| amuller | hmm coding_checks.sh is not executable for me | 17:32 |
| *** spandhe has joined #openstack-neutron | 17:32 | |
| armax | amuller: you gotta do sh ./tools/cooding_checks.sh | 17:33 |
| amuller | no I mean I have no problem running it, just a note that it's not executable while other shell scripts in that directory are | 17:33 |
| armax | amuller: not all of them | 17:33 |
| armax | amuller: misch-sanity-checks.sh isn’t, let’s not start being anal about this :) | 17:34 |
| amuller | wasn't being anal, just making a note | 17:34 |
| *** matrohon has quit IRC | 17:34 | |
| armax | amuller: I know | 17:34 |
| amuller | so running coding-checks directly gives the same output as using it through tox | 17:34 |
| *** pask81 has quit IRC | 17:34 | |
| armax | amuller: ya | 17:34 |
| armax | amuller: so now we gotta figure out why running it selectively on a subset of files make it barf compared to the whole thing | 17:35 |
| openstackgerrit | Kyle Mestery proposed openstack/neutron: Move Neutron Policy pages into the tree https://review.openstack.org/161799 | 17:35 |
| amuller | armax: but barf in a positive way, it's finding actual issues | 17:35 |
| armax | amuller: yes, which is great, but why? | 17:35 |
| armax | :) | 17:35 |
| amuller | hehe | 17:35 |
| *** krtaylor has joined #openstack-neutron | 17:37 | |
| *** ygbo has quit IRC | 17:37 | |
| *** shivharis has quit IRC | 17:38 | |
| armax | amuller: that’s interesting | 17:38 |
| *** carl_baldwin_ has joined #openstack-neutron | 17:38 | |
| armax | amuller: try this: pylint --rcfile=.pylintrc --output-format=colorized neutron.tests | 17:39 |
| armax | amuller: and get ready to be scared | 17:39 |
| amuller | I just tried running: pylint --rcfile=.pylintrc --output-format=colorized neutron | 17:39 |
| amuller | and it's ignoring the exceptions in .pylintrc? | 17:39 |
| armax | amuller: and? | 17:39 |
| amuller | it's finding stuff we've put on the exceptions list | 17:40 |
| *** carl_baldwin has quit IRC | 17:40 | |
| armax | amuller: that would be bizzarre to say the least | 17:40 |
| armax | amuller: if I run pylint --rcfile=.pylintrc --output-format=colorized neutron, it’s working for me, in the sense that finds nothing | 17:41 |
| amuller | hold on I mislead you, for me it's finding a lot of 'maybe-no-member' | 17:41 |
| amuller | and I actually removed that from the exceptions in a patch some time ago | 17:42 |
| *** aranjan has joined #openstack-neutron | 17:42 | |
| *** carl_baldwin has joined #openstack-neutron | 17:42 | |
| amuller | and it's not finding anything for you? | 17:42 |
| amuller | oookkkk... | 17:42 |
| armax | amuller: nope | 17:42 |
| armax | amuller: if I ran pylint --rcfile=.pylintrc --output-format=colorized neutron | 17:43 |
| armax | amuller: it finds nothing | 17:43 |
| armax | amuller: if I run pylint --rcfile=.pylintrc --output-format=colorized neutron.tests | 17:43 |
| amuller | I have pylint-1.2-6.fc20.noarch | 17:43 |
| amuller | from yum | 17:43 |
| armax | amuller: it finds a ton of violations | 17:43 |
| *** carl_baldwin_ has quit IRC | 17:43 | |
| amuller | it finds violations for neutron.tests for me too | 17:43 |
| *** mlavalle has quit IRC | 17:43 | |
| amuller | hmm but I also have pylint installed from pip | 17:44 |
| armax | amuller: I got 1.4.1 | 17:44 |
| amuller | maybe something broken on my end | 17:44 |
| *** safchain has quit IRC | 17:44 | |
| armax | amuller: so what is surprising is that for whatever reason checking the entire neutron module yield no errors | 17:44 |
| armax | amuller: whereas checking the tests module directly does yield errors | 17:44 |
| amuller | there's a lot of surprises here =D | 17:44 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add get event to OVSDB monitor https://review.openstack.org/161806 | 17:45 |
| armax | amuller: I wonder if we checked each module individually what we’ll end up learning | 17:45 |
| armax | amuller: but at least we can safely say there’s nothing wrong with my patch :) | 17:45 |
| armax | amuller: we only need to figure out why pylint is not barfing when it should | 17:46 |
| *** aranjan has quit IRC | 17:46 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 17:47 |
| amuller | armax: so I removed pylint from yum and installed 1.4.1 from pip. Now running pylint --rcfile=.pylintrc --output-format=colorized neutron gives no errors while running it on neutron/tests gives a lot of errors, so I'm in the same boat as you are now | 17:48 |
| *** leenheer has joined #openstack-neutron | 17:48 | |
| armax | amuller: ok that’s good | 17:48 |
| amuller | previously it was finding errors on neutron for me | 17:48 |
| amuller | with an older version | 17:48 |
| *** baoli has quit IRC | 17:49 | |
| amuller | armax: Do you remember Angus' IRC nick? | 17:49 |
| amuller | maybe he knows something about this | 17:49 |
| armax | amuller: gus it is | 17:49 |
| amuller | gus: howdy | 17:49 |
| openstackgerrit | Merged openstack/neutron: Fix test tautology for DVR https://review.openstack.org/161772 | 17:49 |
| *** flwang has quit IRC | 17:50 | |
| *** ronis has joined #openstack-neutron | 17:51 | |
| *** kawa2014 has joined #openstack-neutron | 17:52 | |
| armax | amuller: we’re idiots | 17:52 |
| amuller | almost always | 17:52 |
| armax | amuller: https://github.com/openstack/neutron/blob/master/.pylintrc#L9 | 17:52 |
| armax | but now... | 17:53 |
| amuller | gotcha | 17:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/161815 | 17:53 |
| amuller | hmm why would we not check the tests subdir | 17:53 |
| armax | amuller: but now we gott afigure out why the ignore is ignored | 17:53 |
| armax | :) | 17:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/160569 | 17:53 |
| armax | amuller: unless giving an explicit file makes the ignore be ignored | 17:53 |
| amuller | probably just whoever put in pylint originally didnt have the energy to fix everything in the tests dir | 17:53 |
| *** kawa2014 has quit IRC | 17:53 | |
| amuller | armax: yeah I'd assume that | 17:54 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/160571 | 17:54 |
| amuller | ok this explains the HEAD~3 thing as well | 17:54 |
| armax | amuller: so I think we solved the last bit of mistery | 17:54 |
| amuller | it was finding issues in the tests dir | 17:54 |
| armax | amuller: right | 17:54 |
| amuller | there's not *that* many errors in the tests dir | 17:54 |
| amuller | I could probably push a patch to clean it up | 17:54 |
| armax | so I guess next step is to clear the ‘tests’ from the ignore config | 17:54 |
| amuller | alright let's merge your patch | 17:55 |
| armax | amuller: thank you sir | 17:55 |
| armax | amuller: appreciate your help | 17:55 |
| amuller | armax: I didn't actually do much :) I think this was more of a duck-debugging exercise for you | 17:55 |
| armax | amuller: well at least you kept me on my toes | 17:55 |
| *** leenheer has quit IRC | 17:57 | |
| *** apuimedo has quit IRC | 17:58 | |
| *** harlowja_away is now known as harlowja | 17:58 | |
| amuller | armax: added this to my to-do list (removing tests exclusion), going out to lunch now | 17:58 |
| *** leenheer has joined #openstack-neutron | 17:58 | |
| amuller | armax: Thanks for your time | 17:58 |
| armax | amuller: thank you! | 17:58 |
| *** ivar-lazzaro has joined #openstack-neutron | 17:59 | |
| *** lazy_prif has joined #openstack-neutron | 17:59 | |
| *** lazy_prif is now known as lazy_prince | 17:59 | |
| *** ivar-lazzaro has quit IRC | 17:59 | |
| *** killer_prince has quit IRC | 17:59 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:59 | |
| *** rushil has quit IRC | 18:03 | |
| *** jpena has quit IRC | 18:03 | |
| *** baoli has joined #openstack-neutron | 18:05 | |
| *** baoli has quit IRC | 18:06 | |
| *** jorgem has joined #openstack-neutron | 18:06 | |
| *** sbalukoff has quit IRC | 18:06 | |
| *** dboik_ has quit IRC | 18:06 | |
| *** dboik has joined #openstack-neutron | 18:07 | |
| *** baoli has joined #openstack-neutron | 18:07 | |
| *** achanda has joined #openstack-neutron | 18:08 | |
| *** baoli_ has joined #openstack-neutron | 18:08 | |
| *** radez_g0n3 is now known as radez | 18:09 | |
| *** vivek-ebay has joined #openstack-neutron | 18:09 | |
| *** anish__ has quit IRC | 18:11 | |
| *** baoli has quit IRC | 18:11 | |
| *** sthillma has joined #openstack-neutron | 18:12 | |
| *** lizk_ has quit IRC | 18:12 | |
| pcm_ | marun: ping | 18:13 |
| marun | pcm_: pong | 18:14 |
| pcm_ | marun: For neutron-vpnaas repo, I'm trying to setup a second functional job for strongswan, which needs different devstack setup. | 18:14 |
| pcm_ | marun: Talked to clarkb, about how to have it setup to have some test modules that are shared for both jobs. | 18:15 |
| *** SridharRamaswamy has joined #openstack-neutron | 18:15 | |
| pcm_ | marun: On thought was to have three subdirectories; strongswan, openswan, common. | 18:15 |
| pcm_ | marun: Do you know if OS_TEST_PATH can accept multiple paths? | 18:16 |
| marun | pcm_: not that I'm aware of | 18:16 |
| *** karimb has quit IRC | 18:16 | |
| marun | pcm_: it's possible to get around that by using load_tests, though | 18:16 |
| marun | https://review.openstack.org/#/c/128259/30/neutron/tests/functional/__init__.py | 18:16 |
| *** signed8bit has joined #openstack-neutron | 18:17 | |
| *** karimb has joined #openstack-neutron | 18:17 | |
| pcm_ | marun: Interesting... digesting... | 18:17 |
| *** karimb has quit IRC | 18:18 | |
| pcm_ | marun: So, I could put that at top level and add tests based on the IPSEC_PACKAGE environment variable? | 18:19 |
| marun | pcm_: I think it's more likely that you'd put that in each of strongswan and openswan and add the common path | 18:19 |
| marun | pcm_: assuming your intent is to run the common tests along with either of strongswan or openswan's tests | 18:19 |
| *** mrawlings has left #openstack-neutron | 18:20 | |
| marun | pcm_: so that you could still vary the OS_TEST_PATH by venv | 18:20 |
| marun | pcm_: tbh I'm not sure if there's a functional difference between those two approaches | 18:20 |
| marun | pcm_: although, if OS_TEST_PATH is the toplevel, I'm not sure what's to stop testr from pulling everything in regardless | 18:21 |
| pcm_ | marun: Let;s see if I can summarize... | 18:21 |
| *** Marga_ has quit IRC | 18:22 | |
| pcm_ | marun: Option A) each job, in post_test_hook, would call out a different tox command, which sets OS_TEST_PATH to a subdir, and the __init__.py would load the common subdir. | 18:22 |
| marun | pcm_: I think that's the only viable option | 18:23 |
| marun | pcm_: setting OS_TEST_PATH to the top of the test tree is likely to pull everything in | 18:23 |
| marun | pcm_: hmm | 18:23 |
| pcm_ | marun: So you think that, if the top level __init__.py only calls out two sub dirs, it'll still do the current dir and all under? | 18:23 |
| *** prad has quit IRC | 18:24 | |
| *** SumitNaiksatam has quit IRC | 18:24 | |
| marun | pcm_: I'm probably wrong on that | 18:24 |
| *** prad has joined #openstack-neutron | 18:24 | |
| marun | pcm_: load_tests presumably prevents automatic discovery | 18:25 |
| marun | pcm_: so, continue. option b? | 18:25 |
| pcm_ | marun: Option B) post_test_hook just runs existing tox command, in __init__.py add tests from common and look at env variable and either add openswan/ or strongswan/ | 18:26 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 18:27 |
| marun | pcm_: yeah, either way would do it | 18:27 |
| marun | pcm_: I'm not sure there's a technical reason to do either/or[. | 18:27 |
| marun | pcm_: though maybe it's easier to understand what's going on if OS_TEST_PATH varies in tox.ini | 18:28 |
| *** SumitNaiksatam has joined #openstack-neutron | 18:28 | |
| marun | pcm_: rather than being set to the same thing across jobs | 18:28 |
| *** jorgem has quit IRC | 18:29 | |
| pcm_ | marun: OK. Let me try A. Thanks for the help! | 18:29 |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 18:29 |
| marun | pcm_: np :) | 18:29 |
| *** Zemeio has joined #openstack-neutron | 18:30 | |
| *** kbringard has quit IRC | 18:30 | |
| *** annegentle has joined #openstack-neutron | 18:31 | |
| *** tsg is now known as tsg_away | 18:32 | |
| pcm_ | mestery: carl_baldwin: Can you guys review and approve https://review.openstack.org/#/c/160983/ which fixes VPN UT breakage? | 18:33 |
| *** zzzeek has quit IRC | 18:33 | |
| *** kbringard has joined #openstack-neutron | 18:33 | |
| carl_baldwin | pcm_: I looked this morning and saw that UTs had failed. I see now that you have fixed those. | 18:34 |
| mestery | pcm_: done | 18:34 |
| pcm_ | carl_baldwin: Yeah, there was a mock needed. | 18:34 |
| carl_baldwin | pcm_: +A | 18:34 |
| pcm_ | mestery: Thanks! | 18:34 |
| pcm_ | carl_baldwin: Thanks! | 18:34 |
| *** rpothier has joined #openstack-neutron | 18:34 | |
| *** bjornar_ has joined #openstack-neutron | 18:35 | |
| pcm_ | carl_baldwin: The refactoring code has passed tests too... https://review.openstack.org/#/c/160179/ | 18:35 |
| openstackgerrit | Isaku Yamahata proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 18:36 |
| carl_baldwin | pcm_: I’ll stick it in my queue for the day. | 18:36 |
| pcm_ | carl_baldwin: Thank you. | 18:37 |
| *** aepifanov has joined #openstack-neutron | 18:37 | |
| *** crose has quit IRC | 18:38 | |
| *** pritesh has joined #openstack-neutron | 18:38 | |
| *** aix has quit IRC | 18:40 | |
| *** claudiub has joined #openstack-neutron | 18:41 | |
| *** rushil has joined #openstack-neutron | 18:42 | |
| claudiub | armax: hello | 18:42 |
| *** SumitNaiksatam has quit IRC | 18:43 | |
| *** sthillma has quit IRC | 18:43 | |
| *** dboik_ has joined #openstack-neutron | 18:44 | |
| *** tflynn has joined #openstack-neutron | 18:44 | |
| *** dboik_ has quit IRC | 18:45 | |
| *** dboik_ has joined #openstack-neutron | 18:45 | |
| *** krtaylor has quit IRC | 18:45 | |
| *** amrith is now known as _amrith_ | 18:46 | |
| *** dboik has quit IRC | 18:46 | |
| *** SumitNaiksatam has joined #openstack-neutron | 18:48 | |
| *** _amrith_ is now known as amrith | 18:49 | |
| *** crc32 has joined #openstack-neutron | 18:50 | |
| *** dboik_ has quit IRC | 18:50 | |
| *** sambetts has quit IRC | 18:51 | |
| *** dboik has joined #openstack-neutron | 18:51 | |
| *** carl_baldwin_ has joined #openstack-neutron | 18:52 | |
| *** sambetts has joined #openstack-neutron | 18:52 | |
| *** carl_baldwin has quit IRC | 18:53 | |
| *** carl_baldwin_ is now known as carl_baldwin | 18:53 | |
| *** vmtrooper has joined #openstack-neutron | 18:53 | |
| *** dane_leblanc has quit IRC | 18:56 | |
| *** leenheer has quit IRC | 18:57 | |
| *** krtaylor has joined #openstack-neutron | 18:58 | |
| openstackgerrit | Rajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources https://review.openstack.org/153422 | 18:59 |
| *** mlavalle has joined #openstack-neutron | 18:59 | |
| openstackgerrit | Rajeev Grover proposed openstack/neutron: Prevent race conditions on DVR FIP resources https://review.openstack.org/153422 | 19:00 |
| *** jorgem has joined #openstack-neutron | 19:00 | |
| *** dims_ has joined #openstack-neutron | 19:01 | |
| *** lori is now known as lori|away | 19:02 | |
| *** _cjones_ has quit IRC | 19:02 | |
| *** jasonsb has joined #openstack-neutron | 19:03 | |
| *** achanda has quit IRC | 19:04 | |
| *** dims has quit IRC | 19:04 | |
| *** ivar-lazzaro has quit IRC | 19:05 | |
| *** sthillma has joined #openstack-neutron | 19:06 | |
| *** SumitNaiksatam has quit IRC | 19:06 | |
| openstackgerrit | Merged openstack/python-neutronclient: Fixed pool and health monitor create bugs https://review.openstack.org/159743 | 19:07 |
| *** jp_at_hp has quit IRC | 19:08 | |
| *** SumitNaiksatam has joined #openstack-neutron | 19:08 | |
| *** Marga_ has joined #openstack-neutron | 19:11 | |
| *** carl_baldwin_ has joined #openstack-neutron | 19:11 | |
| *** dane_leblanc has joined #openstack-neutron | 19:11 | |
| *** baoli_ has quit IRC | 19:13 | |
| *** shivharis has joined #openstack-neutron | 19:13 | |
| *** carl_baldwin has quit IRC | 19:13 | |
| *** carl_baldwin_ is now known as carl_baldwin | 19:13 | |
| *** crc32 has quit IRC | 19:13 | |
| *** aranjan has joined #openstack-neutron | 19:13 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 19:16 |
| *** jprovazn has quit IRC | 19:17 | |
| *** sbalukoff has joined #openstack-neutron | 19:17 | |
| *** achanda has joined #openstack-neutron | 19:24 | |
| *** ijw has joined #openstack-neutron | 19:24 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 19:25 |
| *** nyechiel__ has joined #openstack-neutron | 19:25 | |
| *** tsg_away is now known as tsg | 19:27 | |
| *** jorgem has quit IRC | 19:28 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 19:28 |
| *** ivar-lazzaro has joined #openstack-neutron | 19:28 | |
| openstackgerrit | Merged openstack/neutron: Refactor DVR _arp_entry methods https://review.openstack.org/158433 | 19:33 |
| *** rm_work|away is now known as rm_work | 19:34 | |
| *** annegentle has quit IRC | 19:35 | |
| *** annegentle has joined #openstack-neutron | 19:35 | |
| pcm_ | marun: ping | 19:37 |
| *** aranjan has quit IRC | 19:38 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: VPNaaS: Fix unit test breakage https://review.openstack.org/160983 | 19:42 |
| *** dims_ has quit IRC | 19:44 | |
| *** jckasper has quit IRC | 19:44 | |
| *** dims has joined #openstack-neutron | 19:45 | |
| *** jckasper has joined #openstack-neutron | 19:45 | |
| *** sthillma has quit IRC | 19:46 | |
| *** gridinv has joined #openstack-neutron | 19:47 | |
| openstackgerrit | Yanping Qu proposed openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv https://review.openstack.org/161019 | 19:47 |
| *** vishwanathj has quit IRC | 19:48 | |
| openstackgerrit | Ronak Shah proposed openstack/neutron: Adding DB model changes for Nuage Plugin post decomposition https://review.openstack.org/159281 | 19:49 |
| russellb | amuller: if you're willing to work on it, that would be great | 19:51 |
| *** nyechiel__ has quit IRC | 19:52 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: device driver and agent refactoring https://review.openstack.org/160179 | 19:52 |
| *** Sukhdev has joined #openstack-neutron | 19:53 | |
| *** aranjan has joined #openstack-neutron | 19:55 | |
| *** melwitt has joined #openstack-neutron | 19:55 | |
| *** cbouch has quit IRC | 19:56 | |
| *** kbringard has quit IRC | 19:58 | |
| *** rkukura has quit IRC | 19:58 | |
| Sukhdev | shivharis: I am back on-line now - how can I help? | 20:02 |
| *** flwang has joined #openstack-neutron | 20:03 | |
| *** vishwanathj has joined #openstack-neutron | 20:03 | |
| *** leenheer has joined #openstack-neutron | 20:03 | |
| shivharis | Sukhdev: wrt to https://review.openstack.org/#/c/152383/ | 20:04 |
| *** jorgem has joined #openstack-neutron | 20:04 | |
| shivharis | Sukhdev: the mechanism driver does not have any code, please see my comments | 20:04 |
| shivharis | Sukhdev: i pulled everything out | 20:04 |
| Sukhdev | shivharis: looking - hang on | 20:05 |
| *** aranjan has quit IRC | 20:07 | |
| marun | pcm_: pong | 20:08 |
| HenryG | armax: mestery: ping, vendor decomp question | 20:09 |
| mestery | HenryG: yo | 20:09 |
| armax | HenryG: shoot | 20:09 |
| HenryG | mestery: The n1kv monolithic plugin. It is due to be deprecated in Liberty. Can it stay in neutron (non-decomposed) for Kilo? | 20:11 |
| *** fifieldt has quit IRC | 20:11 | |
| mestery | HenryG: My personal opinion would be that's ok. What do you think armax? | 20:11 |
| armax | HenryG: deprecated as in, you’ll remove it in L? | 20:11 |
| *** fifieldt_ has joined #openstack-neutron | 20:11 | |
| HenryG | armax: yes. So decomposing is just pointless busy-work | 20:12 |
| armax | HenryG: then I see no reason to work on it in K | 20:12 |
| armax | HenryG: so to be clear, the plugin is already depreated in K | 20:12 |
| armax | HenryG: to be removed in L | 20:12 |
| HenryG | armax: Correct | 20:12 |
| armax | HenryG: then we’re good | 20:12 |
| HenryG | armax: thanks | 20:13 |
| pcm_ | marun: Just FYI, the load_tests needed to specify the current directory, in addition to me adding the common area. | 20:13 |
| armax | HenryG: thank you | 20:13 |
| shivharis | armax, mestery: is deprecation a formal process? | 20:13 |
| armax | shivharis: what do you mean? | 20:13 |
| *** trad511 has quit IRC | 20:13 | |
| shivharis | as in deprecation in K and removal in L | 20:13 |
| mestery | shivharis: Yes, please wear a suit and tie | 20:13 |
| mestery | ;) | 20:13 |
| Sukhdev | shivharis: You are correct | 20:13 |
| armax | shivharis: deprecation is usually a two cycle process | 20:14 |
| pcm_ | marun: Not sure if it would have added sub-dirs automatically. | 20:14 |
| armax | shivharis: but it can take longer and a decision for removal is taken on a case by case basis | 20:14 |
| armax | look how long it’s taking to deprecate/remove nova-network :) | 20:14 |
| Sukhdev | shivharis: my answer was to your patch - not part of this communication with armax and mestery :-) | 20:14 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Enable StrongSwan in gate hook https://review.openstack.org/161714 | 20:14 |
| marun | pcm_: I guess that forces the issue | 20:14 |
| *** lori|away is now known as lori | 20:14 | |
| mestery | lol | 20:14 |
| armax | shivharis: now if you’re referring to the decomp efforts | 20:14 |
| *** jorgem has quit IRC | 20:15 | |
| *** aranjan has joined #openstack-neutron | 20:15 | |
| armax | shivharis: the spec spells out the strategy for deprecation/removal of plugins that do not comply with the decomp approach | 20:15 |
| armax | shivharis: we’re not looking to remove anything soon | 20:15 |
| shivharis | armax: no we have a monolithic plugin that needs to be deprecated like HenryG | 20:15 |
| pcm_ | marun: Just pushed up the change ^^. If you get a chance, take a peek and see if I need to alter anything. | 20:15 |
| marun | pcm_: sure | 20:15 |
| pcm_ | marun: I tested it out locally and looks OK. | 20:15 |
| pcm_ | marun: Thank you! | 20:15 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 20:16 |
| armax | shivharis: so if you communicated that the plugin is deprecated for Kilo | 20:16 |
| claudiub | armax: hello. I've addressed your comment on the review: https://review.openstack.org/#/c/153259/ | 20:16 |
| HenryG | shivharis: Announce the deprecation in the release notes for Kilo, then remove it in Lemming | 20:16 |
| armax | shivharis: and you’re conformable that users have moved over, then you can get rid of it in L | 20:16 |
| armax | claudiub: looking | 20:16 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 20:16 |
| claudiub | armax: thanks! :D | 20:16 |
| armax | claudiub: thank you! | 20:17 |
| shivharis | HenryG, armax: yes no customers for that, all moved to ML2 | 20:17 |
| armax | shivharis: super, nuke it then | 20:17 |
| shivharis | armax: love to do that :-) | 20:17 |
| armax | shivharis: I bet you do | 20:17 |
| armax | :) | 20:17 |
| shivharis | armax, HenryG, mestry: thanks, release note in K, then kill in L | 20:18 |
| armax | shivharis: ya | 20:18 |
| shivharis | Sukhdev: ping again, sorry | 20:18 |
| Sukhdev | shivharis: no worries - did not want to confuse your communication with the others | 20:19 |
| Sukhdev | shivharis: So, I looked at your patch and went to look at your repository as well - you are correct | 20:19 |
| Sukhdev | shivharis: You do not need UTs - so, you are good there | 20:19 |
| shivharis | Sukhdev: so only item 2 needs work, 1 is taken care of | 20:20 |
| armax | claudiub: done | 20:20 |
| armax | claudiub: sorry I didn’t spot it before | 20:20 |
| Sukhdev | shivharis: but, you have to fix your voting | 20:20 |
| Sukhdev | shivharis: correct | 20:20 |
| shivharis | Sukhdev: that was item 2, you are correct, that i will handle | 20:20 |
| *** Marga_ has quit IRC | 20:20 | |
| *** Marga_ has joined #openstack-neutron | 20:21 | |
| Sukhdev | as soon as it posts a vote let me know and we can have armax to vote on it as well | 20:21 |
| amuller | russellb: Yeah I can do it | 20:21 |
| shivharis | Sukhdev: thanks will do, i will ping you when that is done | 20:21 |
| Sukhdev | shivharis: no worries - thanks | 20:22 |
| claudiub | armax: no problem. :) | 20:23 |
| shivharis | Sukhdev: ttyl | 20:23 |
| *** Longgeek has joined #openstack-neutron | 20:23 | |
| *** coolsvap_ is now known as coolsvap | 20:24 | |
| *** flwang has quit IRC | 20:24 | |
| *** leenheer has quit IRC | 20:25 | |
| *** vmtrooper has quit IRC | 20:25 | |
| *** openstackgerrit has quit IRC | 20:25 | |
| *** openstackgerrit has joined #openstack-neutron | 20:26 | |
| amuller | marun: HenryG: mestery: Armando's pylint change is ready to merge I think: https://review.openstack.org/#/c/159341/9 | 20:26 |
| *** rkukura has joined #openstack-neutron | 20:27 | |
| *** Longgeek has quit IRC | 20:27 | |
| *** rook has joined #openstack-neutron | 20:27 | |
| mestery | amuller: Thanks for the heads up. | 20:28 |
| mestery | amuller: I'm waiting for you or armax to propose the removal of run_tests.sh :) | 20:28 |
| *** kbringard has joined #openstack-neutron | 20:28 | |
| *** sthillma has joined #openstack-neutron | 20:28 | |
| armax | mestery: that ain’t coming from me :) | 20:28 |
| openstackgerrit | Yanping Qu proposed openstack/neutron-fwaas: Adds FWaaS driver for Cisco CSR1kv https://review.openstack.org/161019 | 20:28 |
| mestery | armax: Maybe we can rely on dougwig for that ;) | 20:29 |
| armax | mestery: I am done bikeshedding for a while | 20:29 |
| *** afazekas has quit IRC | 20:29 | |
| amuller | mestery: not yet... I want to see a follow up that will add the ability for flake8 to run on specific files as well | 20:29 |
| amuller | then run_tests.sh can die | 20:29 |
| armax | amuller: oh shoot I forgot | 20:29 |
| haleyb | i think i was the one that -1'd the last try to kill run_tests.sh because i use -8, but if there is a tox equivalent i won't hold things up | 20:29 |
| armax | amuller: did I commit to that? | 20:29 |
| amuller | armax: I can do it | 20:29 |
| armax | amuller: well, if you beat me to it, then tag me along and I’ll be happy to -1 your change | 20:30 |
| amuller | lol sounds great | 20:30 |
| mestery | amuller: ++ | 20:30 |
| armax | amuller: sorry I mean, +1, or did I? | 20:30 |
| armax | amuller: um... | 20:30 |
| amuller | -1 in your case is a nice gesture, at least we're not talking about -2 | 20:30 |
| armax | amuller: time will tell… | 20:30 |
| *** leenheer has joined #openstack-neutron | 20:31 | |
| *** aranjan has quit IRC | 20:32 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 20:33 |
| *** gridinv has quit IRC | 20:33 | |
| Sukhdev | HenryG: Is the L release Lemming or Liberty? | 20:34 |
| banix | Sukhdev: Liberty | 20:34 |
| Sukhdev | look what HenryG called it - [12:16:28] <HenryG> shivharis: Announce the deprecation in the release notes for Kilo, then remove it in Lemming | 20:35 |
| Sukhdev | banix: that is what I thought but, HenryG confused me :-):-) | 20:35 |
| banix | Sukhdev: https://wiki.openstack.org/wiki/Releases | 20:36 |
| *** carl_baldwin_ has joined #openstack-neutron | 20:36 | |
| openstackgerrit | vishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 20:37 |
| banix | Sukhdev: check this out: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055342.html | 20:38 |
| *** achanda has quit IRC | 20:38 | |
| *** carl_baldwin has quit IRC | 20:38 | |
| *** carl_baldwin_ is now known as carl_baldwin | 20:38 | |
| *** ivar-laz_ has joined #openstack-neutron | 20:38 | |
| HenryG | Sukhdev: it was a weak attempt at an inside joke. banix provided the context ^^ | 20:39 |
| openstackgerrit | vishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 20:39 |
| Sukhdev | HenryG: ah ha… it makes sense now - you managed to confuse me for a sec :-):-) | 20:40 |
| banix | rkukura: when you get a chance please have a look at https://review.openstack.org/#/c/156318/ where changes to ml2 plugin are being proposed wrt mtu | 20:40 |
| marun | amuller: hi | 20:40 |
| banix | Sukhdev: see the review above if you get a chance ^^^ | 20:40 |
| amuller | marun: yo | 20:40 |
| marun | amuller: I'm seeing strange behavior with AsyncProcess when running the neutron server | 20:41 |
| rkukura | banix: thanks - will do | 20:41 |
| Sukhdev | banix: will do - thanks | 20:41 |
| amuller | marun: this is in full stack testing? | 20:41 |
| marun | amuller: the command line that is looked for in the check for being active isn't matching what appears in the commandline for the pid | 20:41 |
| marun | amuller: yeah | 20:41 |
| *** achanda has joined #openstack-neutron | 20:41 | |
| marun | amuller: the pid's commandline includes /usr/bin/python | 20:41 |
| *** ivar-lazzaro has quit IRC | 20:41 | |
| marun | amuller: have you seen anything like that? Or is nothing else using async process to manage python processes yet? | 20:42 |
| amuller | marun: the only usage of asyncprocess so far I think was the ovsdb monitor | 20:42 |
| amuller | and ip monitor now | 20:43 |
| *** ajmiller has quit IRC | 20:43 | |
| marun | amuller: I think it was used in testing too, but I guess not for python commands | 20:43 |
| marun | amuller: ok, thanks. | 20:43 |
| amuller | marun: so the function that gets the cmd for pid is returning a /usr/bin/python prefix, and what method are you using to get the cmd for the pid? | 20:44 |
| amuller | or how are you invoking the process? | 20:44 |
| *** matrohon has joined #openstack-neutron | 20:44 | |
| *** david-lyle_afk has joined #openstack-neutron | 20:45 | |
| *** ZZelle_ has joined #openstack-neutron | 20:46 | |
| *** _cjones_ has joined #openstack-neutron | 20:46 | |
| marun | amuller: the process being invoked is '/usr/local/bin/neutron-server' | 20:46 |
| *** david-lyle_afk has quit IRC | 20:47 | |
| marun | amuller: the cmdline reported for the pid is '/usr/bin/python', '/usr/local/bin/neutron-server' | 20:47 |
| *** busterswt has quit IRC | 20:47 | |
| *** david-lyle_afk has joined #openstack-neutron | 20:47 | |
| amuller | marun: I think the quick fix of conditionally ignoring the first binary if it's python is reasonable | 20:47 |
| marun | amuller: yeah, that's the obvious fix. | 20:48 |
| marun | amuller: the other option would be explicitly passing in python | 20:48 |
| marun | amuller: so that the command being invoked would match without hackery | 20:48 |
| amuller | not everyone will remember to do that in the future | 20:49 |
| marun | amuller: we could document it though | 20:49 |
| amuller | it's not an error to invoke the command without the python binary so I think the utility checker should know to handle both cases | 20:49 |
| marun | amuller: I dunno, the same problem could impact us for other languages | 20:49 |
| marun | amuller: bash, perl, etc | 20:49 |
| amuller | hmm | 20:49 |
| marun | amuller: anything with #! | 20:49 |
| amuller | I don't know if that's really an issue | 20:49 |
| *** sbfox1 has quit IRC | 20:49 | |
| marun | amuller: the other option is to check that the command ends with that reported for the pid | 20:51 |
| marun | amuller: so that any prefix would be safe | 20:51 |
| amuller | sure that'd work | 20:51 |
| *** coolsvap is now known as coolsvap_ | 20:52 | |
| amuller | marun: maybe there's a case where it would report a false positive but I don't think so | 20:52 |
| marun | it would seem pretty unlikely in the cases we're using it in | 20:52 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 20:53 |
| *** rkukura has quit IRC | 20:53 | |
| *** rkukura has joined #openstack-neutron | 20:55 | |
| dougwig | mestery: i tried to nuke runtests. it has fans, it turns out. i'd say go look at the abandoned review, but they are too hard to find, apparently. :) | 20:55 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Move Neutron Policy pages into the tree https://review.openstack.org/161799 | 20:55 |
| *** Zemeio has quit IRC | 20:55 | |
| mestery | HenryG: WTH? https://review.openstack.org/#/c/161799/ | 20:55 |
| mestery | HenryG: You pushed a new version of my patch again? | 20:56 |
| mestery | HenryG: :P | 20:56 |
| carl_baldwin | dane_leblanc: ping | 20:56 |
| HenryG | mestery: I ran it through a spell checker, and rather than parrot the things it found I jsut pushed up the corrections. | 20:56 |
| mestery | HenryG: Ah, good man, though you had me worried there for a moment :)( | 20:57 |
| ZZelle_ | marun, hi | 20:57 |
| mestery | HenryG: BTW, most ofhte spell problems were salv-orlando's from his wiki page I bet :P | 20:57 |
| HenryG | mestery: Sorry I should have asked you first before surprising you | 20:57 |
| mestery | HenryG: No worries | 20:57 |
| dane_leblanc | carl_baldwin: pong | 20:57 |
| HenryG | mestery: It's always salv-orlando's fault | 20:58 |
| mestery | HenryG: so true. ;) | 20:58 |
| carl_baldwin | dane_leblanc: I was getting ready to look at https://review.openstack.org/#/c/161799/ | 20:58 |
| *** ivar-laz_ has quit IRC | 20:58 | |
| *** salv-orlando has left #openstack-neutron | 20:59 | |
| *** ivar-lazzaro has joined #openstack-neutron | 20:59 | |
| carl_baldwin | dane_leblanc: The merge conflicts don’t look too bad. Do you want me to take a crack at them? | 20:59 |
| dane_leblanc | carl_baldwin: IS that number right? | 20:59 |
| *** ihate-salv-orlan has joined #openstack-neutron | 20:59 | |
| amuller | dougwig: The fans (Myself included, and Brian Haley I remember) are using run_tests.sh -8, but I plan to fold that into another file and kill run_tests.sh) | 21:00 |
| *** ivar-lazzaro has quit IRC | 21:00 | |
| carl_baldwin | mestery: At least HenryG didn’t rebase the patch too. :) | 21:00 |
| mestery | carl_baldwin: lol :) | 21:00 |
| *** ivar-lazzaro has joined #openstack-neutron | 21:00 | |
| ihate-salv-orlan | HenryG, mestery: well said. That salv-orlando guy is only trouble. | 21:00 |
| carl_baldwin | dane_leblanc: Sorry. You’re right. I copied the wrong URL. | 21:00 |
| haleyb | amuller: i'd be fine with that, or anything i can easily alias to get it | 21:00 |
| dane_leblanc | 160799 | 21:00 |
| carl_baldwin | dane_leblanc: https://review.openstack.org/#/c/149068/ | 21:00 |
| mestery | ihate-salv-orlan: lol :) | 21:00 |
| *** ihate-salv-orlan is now known as salv-orlando | 21:00 | |
| dane_leblanc | carl_baldwin: Oooh, for that one, baoli has some AWESOME suggestions on how to change things so that the patch will reduce by quite a bit. | 21:01 |
| carl_baldwin | dane_leblanc: Shall I hold off for a bit? | 21:01 |
| dane_leblanc | carl_baldwin: Can you let me hack this up tonight? | 21:01 |
| *** tiswanso has quit IRC | 21:02 | |
| dane_leblanc | carl_baldwin: Yes, let's sync up tomorrow | 21:02 |
| *** boris-42 has quit IRC | 21:02 | |
| dougwig | amuller: others also use it still for getting stuff in the debugger. | 21:02 |
| *** sbfox has joined #openstack-neutron | 21:02 | |
| carl_baldwin | dane_leblanc: Okay, keep me posted. | 21:02 |
| dougwig | the flake8 one worries me less, because it's pretty damn quick even scanning everything. | 21:02 |
| dougwig | and if it's not quick, go buy an SSD already. :) | 21:03 |
| shivharis | mestery: ping | 21:03 |
| *** raildo is now known as raildo_away | 21:03 | |
| *** bandarji has joined #openstack-neutron | 21:03 | |
| *** david-lyle_afk is now known as david-lyle | 21:03 | |
| mestery | shivharis: You're like clockwork man! ;) | 21:03 |
| mestery | shivharis: Whats up? | 21:03 |
| dane_leblanc | carl_baldwin: Will do, thanks! | 21:03 |
| shivharis | as you said, pinging again on the privilige of voting for my CI | 21:03 |
| shivharis | s/on/for/ | 21:04 |
| * carl_baldwin sets his watch to shivharis ’ ping | 21:04 | |
| shivharis | mestery: Brocade VDX CI, to be precise | 21:04 |
| mestery | shivharis: So you want to have voting enabled for this right? | 21:04 |
| mestery | shivharis: I'm fine with that, let me take care of it now. | 21:04 |
| *** nihilifer has joined #openstack-neutron | 21:05 | |
| shivharis | mestery: yup | 21:05 |
| *** crc32 has joined #openstack-neutron | 21:05 | |
| shivharis | mestery: thanks | 21:05 |
| *** ajo has quit IRC | 21:05 | |
| mestery | shivharis: Specifically, the "Brocade VDX CI" account in gerrit? That has yoru email address on it? | 21:05 |
| mestery | shivharis: That's the one? | 21:05 |
| mestery | There are many other Brocade CI accounts there | 21:05 |
| mestery | shivharis: Brocade VDX CI added with voting rights | 21:06 |
| shivharis | Yes but only one i am refering to is "Brocade VDX CI" | 21:06 |
| mestery | have fun | 21:06 |
| mestery | shivharis: it's now voting | 21:06 |
| shivharis | mestery: it is a joy to enfranchised !! | 21:07 |
| shivharis | mestery: thanks | 21:07 |
| mestery | shivharis: lol, have fun! :) | 21:07 |
| * mestery goes away for a bit | 21:08 | |
| *** nlahouti has quit IRC | 21:08 | |
| *** achanda has quit IRC | 21:09 | |
| *** catohornet has joined #openstack-neutron | 21:10 | |
| *** cipcosma has quit IRC | 21:10 | |
| *** Marga_ has quit IRC | 21:10 | |
| *** Marga_ has joined #openstack-neutron | 21:11 | |
| *** aranjan has joined #openstack-neutron | 21:13 | |
| *** shwetaap has quit IRC | 21:13 | |
| *** mwagner_lap has quit IRC | 21:14 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 21:14 |
| openstackgerrit | Merged openstack/neutron: Add InvalidIpForNetwork and InvalidIpForSubnet exception https://review.openstack.org/121767 | 21:15 |
| *** shwetaap has joined #openstack-neutron | 21:15 | |
| *** flwang has joined #openstack-neutron | 21:15 | |
| *** kickinz1 is now known as kickinz1|afk | 21:17 | |
| *** boris-42 has joined #openstack-neutron | 21:19 | |
| openstackgerrit | Merged openstack/neutron: Add ability to run pylint check on modified files only https://review.openstack.org/159341 | 21:19 |
| *** carl_baldwin_ has joined #openstack-neutron | 21:23 | |
| *** carl_baldwin has quit IRC | 21:24 | |
| *** carl_baldwin_ is now known as carl_baldwin | 21:24 | |
| *** promethe1nfire is now known as prometheanfire | 21:26 | |
| *** achanda has joined #openstack-neutron | 21:30 | |
| *** aranjan has quit IRC | 21:31 | |
| *** achanda has quit IRC | 21:32 | |
| *** baoli has joined #openstack-neutron | 21:32 | |
| *** achanda has joined #openstack-neutron | 21:34 | |
| *** baoli has quit IRC | 21:36 | |
| *** baoli has joined #openstack-neutron | 21:36 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 21:37 |
| *** achanda has quit IRC | 21:37 | |
| *** achanda has joined #openstack-neutron | 21:39 | |
| openstackgerrit | Merged openstack/neutron: Decompose the VMware plugin https://review.openstack.org/160463 | 21:44 |
| matrohon | marun : hi | 21:46 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove BaseLinuxTestCase https://review.openstack.org/161913 | 21:46 |
| *** cference_ has quit IRC | 21:46 | |
| *** aranjan has joined #openstack-neutron | 21:47 | |
| *** SumitNaiksatam has quit IRC | 21:48 | |
| *** baoli has quit IRC | 21:48 | |
| *** SumitNaiksatam has joined #openstack-neutron | 21:50 | |
| *** Marga_ has quit IRC | 21:50 | |
| *** Marga_ has joined #openstack-neutron | 21:50 | |
| *** nlahouti has joined #openstack-neutron | 21:51 | |
| openstackgerrit | Merged openstack/neutron: Add cisco decomposition progress to chart https://review.openstack.org/160175 | 21:52 |
| *** _cjones_ has quit IRC | 21:53 | |
| *** Sukhdev has quit IRC | 21:53 | |
| *** nyechiel__ has joined #openstack-neutron | 21:53 | |
| *** ivar-lazzaro has quit IRC | 21:53 | |
| *** wverdugo500 has joined #openstack-neutron | 21:55 | |
| *** wverdugo500 has left #openstack-neutron | 21:55 | |
| *** baoli has joined #openstack-neutron | 21:56 | |
| *** jamielennox|away is now known as jamielennox | 21:56 | |
| openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 21:56 |
| *** ronis has quit IRC | 21:57 | |
| *** ivar-lazzaro has joined #openstack-neutron | 21:57 | |
| *** _cjones_ has joined #openstack-neutron | 21:57 | |
| *** aranjan has quit IRC | 22:00 | |
| *** ijw has quit IRC | 22:00 | |
| *** ivar-lazzaro has quit IRC | 22:01 | |
| openstackgerrit | Tom Holtzen proposed openstack/neutron: Vendor decomposition to move CSR1000v support to the networking-cisco repo https://review.openstack.org/159253 | 22:02 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Clean neutron.tests.functional.agent.linux.base https://review.openstack.org/161913 | 22:02 |
| sambetts | I am seeing "VxlanNetworkUnsupported: VXLAN Network unsupported." on master when I run the UTs even after I complete recreate of the tox env, the rest of the logs that surrounding the error are "Option "proxy" must be supported by command "ip link add type vxlan" to enable VXLAN MCAST mode", I didn't used to need it but do I now need to have some kinda of specfic support on my development envir | 22:03 |
| sambetts | onment for this? | 22:04 |
| *** aranjan has joined #openstack-neutron | 22:05 | |
| *** Bsony has quit IRC | 22:07 | |
| *** jaypipes has quit IRC | 22:08 | |
| *** sbfox has quit IRC | 22:09 | |
| amuller | sambetts: Haven't encountered this but it looks like some test is not mocking out ip link | 22:10 |
| amuller | and it's actually running that command on your system | 22:10 |
| sambetts | amuller: thats what I assumed too, but I thought I would check that there wasn't some new dependency that I had missed | 22:11 |
| *** ijw has joined #openstack-neutron | 22:12 | |
| *** nyechiel__ has quit IRC | 22:12 | |
| *** mriedem has joined #openstack-neutron | 22:12 | |
| mriedem | anyone here resonably familar with the nova/neutron vif plug/unplug callback code? | 22:12 |
| mriedem | since arosen isn't around | 22:13 |
| mriedem | basically wondering why we don't get events back to nova when doing a migration to a different host | 22:13 |
| mriedem | b/c we unplug vifs in that case in nova | 22:13 |
| *** Sukhdev has joined #openstack-neutron | 22:13 | |
| mestery | mriedem: arosen is the dude you want, looks like he's not on IRC ATM | 22:14 |
| mriedem | is it because nothing changed in neutron in that case? | 22:14 |
| mestery | mriedem: I would expect you should get events back on a migration (not having looked at the code, helpful I know) | 22:15 |
| mriedem | yeah...he's my obi-wan | 22:15 |
| salv-orlando | mriedem: something might be changed in the neutron l2 agent and the status transition in the neutorn port might go undected | 22:15 |
| *** aranjan has quit IRC | 22:15 | |
| salv-orlando | this is the only thing that comes up to my mind. If that happens nova receives no vif plug event because neutron believes the vif was never unplugged in the first place | 22:16 |
| *** oomichi has joined #openstack-neutron | 22:16 | |
| mriedem | salv-orlando: but doesn't neutron get the request to unplug the vif in the first place? | 22:16 |
| mriedem | keep in mind the vif driver stuff in libvirt in nova is the heart of darkness to me | 22:17 |
| openstackgerrit | Michael Smith proposed openstack/neutron: WIP delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 22:17 |
| *** sbfox has joined #openstack-neutron | 22:18 | |
| salv-orlando | mriedem: neutron gets a request to delete a port (I think). That is decoupled from the transition which leads to sending network-vif-plugged event | 22:18 |
| salv-orlando | of course what I'm saying is just speculation because I know nothing about this failures. There could be multiple failure modes leading to migration issues. | 22:19 |
| salv-orlando | I guess the problem does manifest as migration timing out because the vif on the destination host did not come up? | 22:19 |
| mriedem | salv-orlando: well, i'm looking at our old friend https://bugs.launchpad.net/nova/+bug/1323658 | 22:20 |
| openstack | Launchpad bug 1323658 in OpenStack Compute (nova) "Nova resize/restart results in guest ending up in inconsistent state with Neutron" [Critical,Confirmed] | 22:20 |
| *** rpothier has quit IRC | 22:20 | |
| mriedem | ok, i think i get it, when the nova libvirt driver does unplug_vifs, it's calling it's own vif code which has types that are associated to types in neutron, | 22:21 |
| mriedem | so for ovs hybrid, nova's vif driver is actually doing the local delif, status=down, delbr, and delete the port in ovs | 22:21 |
| amuller | armax: salv-orlando: Need some opinions on this change: https://review.openstack.org/#/c/147032/10//COMMIT_MSG | 22:22 |
| amuller | related to agents and admin_state_up | 22:22 |
| mriedem | salv-orlando: is that about right? | 22:22 |
| mriedem | and the neutron L2 agent is running on the compute node and ovs is running with it | 22:22 |
| HenryG | sambetts: Just ran full py27 suite locally with no issue. cc: amuller | 22:23 |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 22:23 |
| salv-orlando | mriedem I reckon the analysis is correct. | 22:24 |
| sambetts | HenryG: if you do "ip link add type vxlan help" on your system does your system support proxy as an option?? | 22:24 |
| salv-orlando | nova plays with ovs through the vif driver, the neutron l2 agent detects the changes and does wiring/unwiring, then it tells neutron server that it's done | 22:24 |
| *** ivar-lazzaro has joined #openstack-neutron | 22:24 | |
| mriedem | salv-orlando: ok, gross :) | 22:24 |
| salv-orlando | and finally neutron server sends the network-vif-plugged to nova api | 22:25 |
| *** vivek-ebay has quit IRC | 22:25 | |
| salv-orlando | mriedem: I might argue that is a point where the compute and network services are forcibly decoupled | 22:25 |
| *** vivek-ebay has joined #openstack-neutron | 22:25 | |
| dboik | sambetts: What does 'modinfo vxlan' show? This could possibly be happening now because of https://github.com/openstack/neutron/commit/b7a56fd1b44649daa1f768157e68a135b9e01dd1 | 22:25 |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add L3 router plugin shim for Brocade MLX https://review.openstack.org/161489 | 22:25 |
| HenryG | sambetts: yes | 22:25 |
| sambetts | dboik, HenryG: ERROR: Module vxlan not found. | 22:26 |
| openstackgerrit | Maru Newby proposed openstack/neutron: WIP Prototype of fullstack framework https://review.openstack.org/128259 | 22:26 |
| sambetts | HenryG: Mine does not show proxy as a possible option | 22:26 |
| sambetts | HenryG: but UTs shouldn't be dependant on the underlying system running them surly? | 22:27 |
| HenryG | sambetts: correct, they should not | 22:27 |
| *** vivek-ebay has quit IRC | 22:27 | |
| *** vivek-ebay has joined #openstack-neutron | 22:27 | |
| openstackgerrit | Angela Smith proposed openstack/neutron: Add ML2 VLAN mechanism driver for Brocade MLX and ICX switches. https://review.openstack.org/154709 | 22:29 |
| HenryG | sambetts: Unit tests should not execute any system commands at all | 22:30 |
| dboik | sambetts: Is it the test_lb_neutron_agent that is failing? I think when LinuxBridgeManager is initialized for those tests, it does a check for Vxlan support. I didn't see the calls mocked out anywhere. cc: HenryG | 22:30 |
| sambetts | dboik, HenryG: Yes, its the test_lb_neutron_agent tests | 22:31 |
| *** ivar-lazzaro has quit IRC | 22:32 | |
| amuller | marun: Why did you check the function that checks that the server is ready? The previous implementation checked that all of the agents are up as well | 22:32 |
| marun | amuller: the previous implementation was doing too much | 22:33 |
| HenryG | sambetts: dboik: definitely try mocking the execute call for those tests. There are examples in other tests. | 22:33 |
| marun | amuller: checking of agents can be added in subsequent patches | 22:33 |
| amuller | marun: We absolutely know that need it though | 22:33 |
| amuller | that we* | 22:33 |
| marun | amuller: what I've proposed is a minimally viable change | 22:33 |
| amuller | marun: You talked to John before right? | 22:34 |
| marun | amuller: and I would expect subsequent patches to add incrementally and minimize the kind of coupling the previous patches were showing | 22:34 |
| marun | amuller: I tried | 22:34 |
| marun | amuller: He didn't understand what I was asking for | 22:34 |
| marun | amuller: So we agreed that I'd show him. | 22:34 |
| marun | with code | 22:34 |
| amuller | ok | 22:34 |
| openstackgerrit | Brian Haley proposed openstack/neutron: Don't restrict port subnet info based on number of fixed IPs https://review.openstack.org/161928 | 22:35 |
| *** apuimedo has joined #openstack-neutron | 22:36 | |
| *** achanda has quit IRC | 22:36 | |
| *** rushil has quit IRC | 22:36 | |
| *** achanda has joined #openstack-neutron | 22:40 | |
| *** pcm_ has quit IRC | 22:43 | |
| *** Marga_ has quit IRC | 22:45 | |
| *** Marga_ has joined #openstack-neutron | 22:45 | |
| *** vivek-ebay has quit IRC | 22:46 | |
| *** vivek-ebay has joined #openstack-neutron | 22:47 | |
| mriedem | dims: sdague: some spring cleaning from today's bug dive: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1323658,n,z | 22:48 |
| *** dboik_ has joined #openstack-neutron | 22:48 | |
| ZZelle_ | marun, hi | 22:49 |
| marun | ZZelle_: hi | 22:49 |
| *** leenheer has quit IRC | 22:49 | |
| *** nlahouti has quit IRC | 22:49 | |
| *** bjornar_ has quit IRC | 22:49 | |
| ZZelle_ | marun, some questions about your remark on https://review.openstack.org/143412 | 22:49 |
| dims | mriedem: wrong channel :) | 22:50 |
| mriedem | gdi | 22:50 |
| mriedem | well, related :) | 22:50 |
| ChuckC | armax: last week you mentioned using 'probe interfaces' in a functional test. Did you have something specific in mind? (http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2015-02-28.log, toward the bottom) | 22:50 |
| dims | :) | 22:50 |
| *** nlahouti has joined #openstack-neutron | 22:51 | |
| ZZelle_ | marun, fixture can be used as context | 22:52 |
| *** dboik has quit IRC | 22:52 | |
| marun | ZZelle_: I don't think that'll be relevant | 22:52 |
| marun | ZZelle_: er, scratch that | 22:52 |
| ZZelle_ | marun, according to your remark you need to create/delete FakeMachine during the test? | 22:52 |
| marun | ZZelle_: are you familiar with how tempest scenario tests are written? | 22:53 |
| *** dboik_ has quit IRC | 22:53 | |
| ZZelle_ | marun, not really | 22:53 |
| *** annegentle has quit IRC | 22:53 | |
| marun | ZZelle_: it's not too complicated, really. | 22:54 |
| mestery | pritesh: Thanks for the reviews :) | 22:54 |
| marun | ZZelle_: the retargetable thing makes it a bit more so, though. | 22:54 |
| pritesh | mestery: Thanks to you as well :) | 22:54 |
| mestery | pritesh: I see you spinning the VLAN transparency patch, are you hitting issues with something? Just curious ;) | 22:55 |
| marun | ZZelle_: I realize my original comment wasn't entirely accurate... | 22:55 |
| *** mriedem is now known as mriedem_away | 22:55 | |
| marun | ZZelle_: the tests never create vm's themselves, they always rely on helper methods. | 22:55 |
| *** yfried has quit IRC | 22:55 | |
| *** banix has quit IRC | 22:56 | |
| ZZelle_ | marun, i hope so, otherwise we need a running nova | 22:56 |
| pritesh | mestery: yep jenkins was failing on merges, i think it is good now. | 22:56 |
| *** jaypipes has joined #openstack-neutron | 22:57 | |
| mestery | pritesh: Sweet! Nice work BTW, glad to see you upstream more :) | 22:57 |
| pritesh | mestery: Thanks :) | 22:57 |
| marun | ZZelle_: I think you're right that the implementation as fixture doesn't preclude the type of usage I can imagine for in-tree scenario testing. | 22:57 |
| *** gildub has joined #openstack-neutron | 22:58 | |
| ZZelle_ | marun, i should expect if you need to create a resource without cleaning it (or the opposite) | 22:58 |
| ZZelle_ | s/i/it/ | 22:58 |
| marun | ZZelle_: Cleanup would always be required | 22:58 |
| ZZelle_ | marun, it should except if you need to create a resource without cleaning it (or the opposite) | 22:58 |
| ZZelle_ | marun, ok so it should work | 22:59 |
| lifeless | well thats possible | 22:59 |
| ZZelle_ | marun, the aim ot retargetable tests is to test both a live neutron and a neutron with a mocked backend? | 22:59 |
| *** aranjan has joined #openstack-neutron | 22:59 | |
| marun | ZZelle_: I was worrying about whether making FakeMachine a fixture would make manual cleanup harder if we were doing it outside of test cases | 22:59 |
| marun | ZZelle_: but whether we manage non-fixtures vs fixtures outside of a testcase the work would be similar | 22:59 |
| *** aepifanov has quit IRC | 22:59 | |
| ZZelle_ | you can still use a fixture outside of a testcase ... as a context or by calling setUp/cleanUp | 23:00 |
| marun | ZZelle_: right | 23:00 |
| anteaya | mestery: we didn't solve the brocade gerrit account thing, shivharis has to do some internal investigating, he will get back to you when he has _one_ VDX account he wants in the voting group | 23:00 |
| mestery | anteaya: lol, thanks for the help and lets hope shivharis figures it out ;) | 23:00 |
| marun | ZZelle_: the approach you're taking feels right to me | 23:01 |
| openstackgerrit | Merged openstack/neutron-vpnaas: Decouple L3 and VPN service plugins during router operations https://review.openstack.org/159793 | 23:01 |
| marun | ZZelle_: I'll try to make time to review tomorrow. | 23:01 |
| openstackgerrit | Yanping Qu proposed openstack/neutron-fwaas: Provide Cisco FWaaS functional tests https://review.openstack.org/161938 | 23:02 |
| marun | ZZelle_: if you have a chance, I'd appreciate your feedback on my rework of John's patch: https://review.openstack.org/#/c/128259/ | 23:03 |
| ZZelle_ | marun, i will be on holidays for 7/9 days so if there is only small nits don't hesitate to update changes | 23:03 |
| ZZelle_ | will do | 23:03 |
| marun | ZZelle_: ah, ok. | 23:03 |
| *** shwetaap has quit IRC | 23:03 | |
| anteaya | mestery: well he can't vote until he does, so there's some incentive | 23:03 |
| marun | ZZelle_: Are you around tomorrow? | 23:03 |
| *** prad has quit IRC | 23:03 | |
| *** jckasper has quit IRC | 23:04 | |
| marun | lifeless: I like how fixtures are best used via composition. I was tempted at one point to use inheritance but it's obvious how messy that makes things in a hurry. | 23:05 |
| ZZelle_ | marun, not at us time | 23:05 |
| *** achanda has quit IRC | 23:05 | |
| marun | ZZelle_: how early? | 23:05 |
| ZZelle_ | till 11 UTC | 23:06 |
| marun | ZZelle_: ok then. hope you have a nice holiday! | 23:06 |
| ZZelle_ | after i don't know where i will be | 23:06 |
| ZZelle_ | marun, thx | 23:07 |
| *** hichihara has joined #openstack-neutron | 23:07 | |
| sambetts | Goodnight HenryG, see you tomorrow :) | 23:07 |
| sambetts | Night all! | 23:07 |
| *** achanda has joined #openstack-neutron | 23:08 | |
| *** _cjones_ has quit IRC | 23:09 | |
| *** lori is now known as lori|away | 23:11 | |
| *** yuanying has joined #openstack-neutron | 23:11 | |
| blogan | marun: regarding the plan to put neutron tempest tests in neutron's tree, do you plan on chasing tempest's trunk and fixing issues as they arise or having a co-gate in tempest? | 23:12 |
| *** aranjan has quit IRC | 23:12 | |
| *** gildub has quit IRC | 23:12 | |
| *** kbringard has quit IRC | 23:13 | |
| marun | blogan: neither :) | 23:13 |
| blogan | marun: lol can you enlighten me on the plan :) | 23:13 |
| marun | blogan: the proposed change adds the tempest tests to neutron | 23:13 |
| marun | blogan: once they've proven stable, development will be frozen in tempest | 23:14 |
| blogan | forever? | 23:14 |
| *** sthillma has quit IRC | 23:14 | |
| marun | blogan: yes. I'd like to remove them asap, but mtreinish wants to retain some of them for defcore/refstack purposes in the near term iirc | 23:15 |
| marun | blogan: if we can convince him and others to consume the tests directly from the neutron tree, then maybe removal can happen sooner | 23:15 |
| blogan | oh development on neutron tests will be frozen in tempest | 23:16 |
| marun | blogan: right | 23:16 |
| marun | blogan: and then work can proceed in the neutron tree | 23:16 |
| blogan | so the in tree tempest tests will not actually import tempest, they're importing everything they need from tempest and pulling it in tree | 23:16 |
| blogan | i see now | 23:16 |
| blogan | if what i said is right | 23:16 |
| marun | blogan: yes | 23:16 |
| marun | blogan: the only way we'll be able to gate is if we don't depend on a repo we don't control and who doesn't/won't gate on us | 23:17 |
| blogan | okay makes sense now! we may have to shamelessy steal your copy script | 23:17 |
| *** _cjones_ has joined #openstack-neutron | 23:17 | |
| blogan | yep that was just the issue i was thinking about | 23:17 |
| marun | blogan: we're still wondering at what do to about tempest-lib, though. | 23:17 |
| marun | blogan: it sounds like they don't want to gate on us either and most of the functionality we're going to copy is scheduled to move to tempest-lib | 23:18 |
| marun | blogan: we're contemplating pinning either to git hashes or a released version, but that is problematic/impossible under current policy | 23:18 |
| blogan | yeah, i can see the issues with that and the negotiations that will need to take place to achieve balance | 23:18 |
| *** ChuckC is now known as ChuckC_ | 23:19 | |
| *** jasonsb has quit IRC | 23:19 | |
| *** cference_ has joined #openstack-neutron | 23:19 | |
| marun | blogan: I think pinning + periodic jobs to evaluate new versions is the way forward, but it's probably going to be complicated/messy to get there | 23:19 |
| openstackgerrit | Merged openstack/neutron: contribute.rst: Fill in the current status of ofagent https://review.openstack.org/160523 | 23:19 |
| marun | blogan: we're likely to encounter the same problem with neutron-lib | 23:19 |
| openstackgerrit | Merged openstack/neutron: ofagent: Vendor code decomposition https://review.openstack.org/149941 | 23:19 |
| blogan | maurn: yeah going to be complicated issues, but still all of this would be boring without these complexities | 23:20 |
| marun | blogan: part of me agrees with you... | 23:20 |
| marun | blogan: the other part thinks - we could be solving *real* problems with all this time and energy instead of ones we're creating for ourselves! | 23:21 |
| blogan | marun: lol im bipolar on it too | 23:21 |
| *** dboik_ has joined #openstack-neutron | 23:21 | |
| marun | ;) | 23:21 |
| blogan | who wants to solve world hunger when we can solve in tree tempest tests! | 23:21 |
| marun | heh | 23:21 |
| blogan | marun: thanks for the info | 23:21 |
| marun | blogan: np | 23:22 |
| *** _cjones_ has quit IRC | 23:22 | |
| *** aranjan has joined #openstack-neutron | 23:24 | |
| *** marun has quit IRC | 23:25 | |
| *** aranjan has quit IRC | 23:26 | |
| *** ChuckC_ is now known as ChuckC | 23:26 | |
| *** sbfox has quit IRC | 23:27 | |
| *** _cjones_ has joined #openstack-neutron | 23:28 | |
| *** jaypipes has quit IRC | 23:28 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Stop sending gratuitous arp when ip version is 6 https://review.openstack.org/160799 | 23:29 |
| *** _cjones_ has quit IRC | 23:29 | |
| *** vivek-eb_ has joined #openstack-neutron | 23:32 | |
| *** vivek-ebay has quit IRC | 23:32 | |
| *** claired has joined #openstack-neutron | 23:34 | |
| *** claired has quit IRC | 23:35 | |
| gus | amuller,armax: I put the ignore for neutron.tests in there (see ,pylintrc->MASTER->ignore). | 23:36 |
| amuller | gus: yeah, I'm working through pylint violations in the tests dir now. Most are false positivies but I found some real issues. | 23:36 |
| gus | amuller,armax: it currently ignores neutron.tests and neutron.openstack.common, since we don't hold tests to the same standard as the rest of our codebase | 23:36 |
| armax | gus: yup, we figured it out | 23:37 |
| shivharis | mestery: hi | 23:37 |
| gus | .. and we don't modify openstack.common code via neutron, so there was no point finding issues there. | 23:37 |
| amuller | yep | 23:37 |
| gus | (I was trying to reflect the sorts of things that would / would not get a -1 in a review. It was clear that plenty of coding practices that were not ok in the main code were acceptable in tests) | 23:38 |
| armax | gus: yes, openstack-common makes sense | 23:38 |
| armax | gus: I would be happy with the line you drew | 23:38 |
| gus | oh, and annoyingly the ignore directive isn't fully-qualified modules so it's a bit clumsy exactly what you can ignore. | 23:39 |
| *** matrohon has quit IRC | 23:40 | |
| *** thomasem has quit IRC | 23:41 | |
| *** SumitNaiksatam has quit IRC | 23:41 | |
| *** cference_ has quit IRC | 23:41 | |
| *** SumitNaiksatam has joined #openstack-neutron | 23:43 | |
| *** _cjones_ has joined #openstack-neutron | 23:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/161815 | 23:44 |
| anteaya | shivharis: so the brocade group is gone now from: https://review.openstack.org/#/admin/groups/510,members | 23:44 |
| *** trad511 has joined #openstack-neutron | 23:44 | |
| anteaya | so if mestery adds brocade vdx to the list is should be the one with the brocade.com email address | 23:44 |
| anteaya | and you should be able to verify patches now | 23:44 |
| shivharis | anteaya: it still seems to show up as before | 23:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/160571 | 23:45 |
| anteaya | oh you are right | 23:45 |
| anteaya | okay so he needs to remove that one and add the one with teh brocade.com email address | 23:45 |
| anteaya | mestery: ^^ | 23:45 |
| shivharis | anteaya: agreed | 23:45 |
| *** gildub has joined #openstack-neutron | 23:46 | |
| *** SumitNaiksatam has quit IRC | 23:47 | |
| *** jobewan has quit IRC | 23:48 | |
| openstackgerrit | Itsuro Oda proposed openstack/neutron: Make floatingip reachable from the same network https://review.openstack.org/161947 | 23:48 |
| openstackgerrit | Baodong (Robert) Li proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 23:49 |
| *** SumitNaiksatam has joined #openstack-neutron | 23:50 | |
| otherwiseguy | haleyb: yeah, sorry to pretty much repeat exactly what you said. I went from reading the commit message directly to freaking out and saying 'no! no! no!' :) | 23:50 |
| *** openstack has joined #openstack-neutron | 23:53 | |
| -sendak.freenode.net- [freenode-info] if you're at a conference and other people are having trouble connecting, please mention it to staff: http://freenode.net/faq.shtml#gettinghelp | 23:53 | |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: VMware NSX: Update decomposition progress table https://review.openstack.org/161949 | 23:54 |
| *** sedwards has left #openstack-neutron | 23:56 | |
| *** baoli has quit IRC | 23:57 | |
| *** ivar-lazzaro has joined #openstack-neutron | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!