*** nplanel has joined #openstack-neutron | 00:01 | |
*** nplanel_ has joined #openstack-neutron | 00:02 | |
*** thorst has quit IRC | 00:02 | |
*** akshai has joined #openstack-neutron | 00:03 | |
*** john-davidge has quit IRC | 00:03 | |
*** tflynn_ has joined #openstack-neutron | 00:04 | |
jamielennox | hi, can i get some core eyes on https://review.openstack.org/#/c/292640/ | 00:05 |
---|---|---|
jamielennox | it's a simple change that fixes the options returned to the oslo.config generator so the sample files look right | 00:05 |
*** cgalan has joined #openstack-neutron | 00:06 | |
*** tflynn has quit IRC | 00:07 | |
*** tflynn_ has quit IRC | 00:09 | |
*** vhosakot has joined #openstack-neutron | 00:09 | |
*** akshai_ has joined #openstack-neutron | 00:10 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Fix race conditions in IP availability API tests https://review.openstack.org/299647 | 00:12 |
*** akshai has quit IRC | 00:13 | |
*** iyamahat has quit IRC | 00:17 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Compute IPAvailabilityRanges in memory during IP allocation https://review.openstack.org/292207 | 00:17 |
*** hoangcx has quit IRC | 00:23 | |
*** hoangcx has joined #openstack-neutron | 00:23 | |
*** thorst has joined #openstack-neutron | 00:25 | |
*** brad_behle has joined #openstack-neutron | 00:25 | |
*** dslevin has joined #openstack-neutron | 00:33 | |
*** dslevin has quit IRC | 00:34 | |
*** hoangcx has quit IRC | 00:34 | |
*** dslevin has joined #openstack-neutron | 00:34 | |
*** absubram has quit IRC | 00:34 | |
*** hoangcx has joined #openstack-neutron | 00:35 | |
*** dslevin has quit IRC | 00:35 | |
*** abhiraut has quit IRC | 00:35 | |
*** dslevin has joined #openstack-neutron | 00:35 | |
*** dslevin has quit IRC | 00:36 | |
*** dslevin has joined #openstack-neutron | 00:37 | |
*** akshai_ has quit IRC | 00:37 | |
*** dslevin has quit IRC | 00:37 | |
*** baojg has joined #openstack-neutron | 00:37 | |
*** mfuruta has joined #openstack-neutron | 00:38 | |
*** abhiraut has joined #openstack-neutron | 00:38 | |
*** dslevin has joined #openstack-neutron | 00:39 | |
*** dslevin has quit IRC | 00:39 | |
*** achanda has quit IRC | 00:40 | |
*** tiswanso has quit IRC | 00:40 | |
*** dslevin has joined #openstack-neutron | 00:41 | |
*** dslevin has quit IRC | 00:42 | |
*** sc68cal has quit IRC | 00:42 | |
*** dslevin has joined #openstack-neutron | 00:43 | |
*** dslevin has quit IRC | 00:43 | |
*** Marga_ has quit IRC | 00:43 | |
*** busterswt has quit IRC | 00:44 | |
*** thorst has quit IRC | 00:45 | |
*** mickeys has quit IRC | 00:45 | |
*** thorst has joined #openstack-neutron | 00:46 | |
*** gvrangan has quit IRC | 00:46 | |
*** Marga_ has joined #openstack-neutron | 00:47 | |
*** mickeys has joined #openstack-neutron | 00:48 | |
*** gvrangan has joined #openstack-neutron | 00:48 | |
*** tbachman has quit IRC | 00:50 | |
*** tbachman has joined #openstack-neutron | 00:50 | |
*** dave-mcc_ has quit IRC | 00:51 | |
*** Marga_ has quit IRC | 00:51 | |
*** busterswt has joined #openstack-neutron | 00:52 | |
openstackgerrit | Reedip proposed openstack/neutron: Add name and desc to networksegments table https://review.openstack.org/293305 | 00:52 |
*** mickeys has quit IRC | 00:52 | |
*** dave-mccowan has joined #openstack-neutron | 00:54 | |
*** thorst has quit IRC | 00:54 | |
*** jonlan has joined #openstack-neutron | 00:55 | |
*** iyamahat has joined #openstack-neutron | 00:55 | |
*** harlowja has quit IRC | 01:00 | |
*** hieulq has joined #openstack-neutron | 01:00 | |
*** zhhuabj has quit IRC | 01:01 | |
*** dave-mccowan has quit IRC | 01:03 | |
openstackgerrit | Reedip proposed openstack/python-neutronclient: Add possible updatable options to net-update CLI https://review.openstack.org/280521 | 01:03 |
*** harlowja has joined #openstack-neutron | 01:03 | |
*** apuimedo has quit IRC | 01:07 | |
*** aranjan_ has quit IRC | 01:07 | |
*** shivrao has quit IRC | 01:07 | |
*** rickyrem has quit IRC | 01:09 | |
*** sarun has quit IRC | 01:09 | |
*** Marga_ has joined #openstack-neutron | 01:10 | |
*** zhhuabj has joined #openstack-neutron | 01:13 | |
*** Marga_ has quit IRC | 01:14 | |
*** abhiraut has quit IRC | 01:15 | |
*** absubram has joined #openstack-neutron | 01:17 | |
*** tiswanso has joined #openstack-neutron | 01:18 | |
*** achanda has joined #openstack-neutron | 01:18 | |
*** sridhar_ram1 has quit IRC | 01:18 | |
reedip | armax: ping | 01:20 |
*** sarun has joined #openstack-neutron | 01:20 | |
reedip | akamyshnikova: ping | 01:21 |
*** vhosakot has quit IRC | 01:21 | |
*** stanzgy has joined #openstack-neutron | 01:21 | |
openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Deprecate the functions map after segment moving out of ml2 https://review.openstack.org/295173 | 01:21 |
*** reedip has quit IRC | 01:22 | |
*** reedip has joined #openstack-neutron | 01:23 | |
*** abhiraut has joined #openstack-neutron | 01:24 | |
*** jonlan has quit IRC | 01:25 | |
*** shashank_hegde has joined #openstack-neutron | 01:26 | |
*** abhiraut has quit IRC | 01:29 | |
*** EinstCrazy has joined #openstack-neutron | 01:29 | |
openstackgerrit | ZhaoBo proposed openstack/python-neutronclient: Do not delete dhcp port when purge tenant's resource https://review.openstack.org/302046 | 01:32 |
*** busterswt has quit IRC | 01:35 | |
*** ajmiller has joined #openstack-neutron | 01:38 | |
*** ajmiller has quit IRC | 01:39 | |
*** julim has joined #openstack-neutron | 01:40 | |
*** gvrangan has quit IRC | 01:40 | |
*** shivrao has joined #openstack-neutron | 01:41 | |
*** shivrao has quit IRC | 01:41 | |
*** hieulq has quit IRC | 01:42 | |
*** hieulq has joined #openstack-neutron | 01:42 | |
*** vhosakot has joined #openstack-neutron | 01:44 | |
*** baoli has joined #openstack-neutron | 01:44 | |
*** baoli has quit IRC | 01:47 | |
*** shashank_hegde has quit IRC | 01:47 | |
*** baoli has joined #openstack-neutron | 01:48 | |
*** sarun has quit IRC | 01:51 | |
*** thorst has joined #openstack-neutron | 01:51 | |
*** busterswt has joined #openstack-neutron | 01:54 | |
*** s3wong has quit IRC | 01:54 | |
*** rickyrem has joined #openstack-neutron | 01:57 | |
*** thorst has quit IRC | 01:59 | |
*** hoangcx has quit IRC | 02:02 | |
armax | reedip: pong | 02:02 |
*** kengo_sakai has joined #openstack-neutron | 02:02 | |
reedip | armax: was checking out https://bugs.launchpad.net/neutron/+bug/1566972 | 02:02 |
openstack | Launchpad bug 1566972 in neutron "Neutron unit tests are failing with SQLAlchemy 1.0.11, 1.0.12 fixes the issue" [Undecided,New] | 02:02 |
*** salv-orlando has joined #openstack-neutron | 02:03 | |
*** hoangcx has joined #openstack-neutron | 02:03 | |
* armax looks | 02:03 | |
reedip | armax: cannot decide how to process it | 02:03 |
armax | reedip: this probably needs more vetting…let’s start with adding tags etc | 02:04 |
*** ljianbj has quit IRC | 02:05 | |
*** shashank_hegde has joined #openstack-neutron | 02:05 | |
reedip | that is where I would need some suggestion. This is impacting db, and currently looks more like an opinion | 02:06 |
*** baoli has quit IRC | 02:06 | |
reedip | and as it needs more information, needs-attention seems likely (??) | 02:07 |
reedip | okay, you already changed it :) | 02:07 |
*** tmorin has joined #openstack-neutron | 02:08 | |
*** Aish has joined #openstack-neutron | 02:08 | |
*** ircuser-1 has quit IRC | 02:08 | |
*** Aish has quit IRC | 02:09 | |
*** wuhg has joined #openstack-neutron | 02:12 | |
*** wolverin_ has joined #openstack-neutron | 02:14 | |
*** wolverineav has quit IRC | 02:17 | |
*** kevo has quit IRC | 02:17 | |
*** anshul has joined #openstack-neutron | 02:21 | |
*** namnh has joined #openstack-neutron | 02:21 | |
*** gongysh has quit IRC | 02:22 | |
*** gongysh has joined #openstack-neutron | 02:22 | |
*** salv-orlando has quit IRC | 02:23 | |
*** shashank_hegde has quit IRC | 02:24 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Remove unused PolicyFileNotFound exception https://review.openstack.org/302497 | 02:26 |
*** iyamahat has quit IRC | 02:27 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Fix keepalived functional tests https://review.openstack.org/302421 | 02:27 |
*** neeti has joined #openstack-neutron | 02:27 | |
*** salv-orlando has joined #openstack-neutron | 02:28 | |
*** tmorin has quit IRC | 02:29 | |
*** minwang2 has quit IRC | 02:30 | |
*** gongysh has quit IRC | 02:32 | |
*** gongysh has joined #openstack-neutron | 02:32 | |
*** salv-orlando has quit IRC | 02:33 | |
*** EinstCrazy has quit IRC | 02:35 | |
*** EinstCrazy has joined #openstack-neutron | 02:36 | |
*** edmondsw has quit IRC | 02:36 | |
*** yamamoto_ has joined #openstack-neutron | 02:37 | |
*** tmorin has joined #openstack-neutron | 02:39 | |
*** tmorin has quit IRC | 02:43 | |
*** cshahani has quit IRC | 02:45 | |
*** shashank_hegde has joined #openstack-neutron | 02:45 | |
*** Leo_ has joined #openstack-neutron | 02:47 | |
*** zhhuabj has quit IRC | 02:48 | |
*** jhershbe has joined #openstack-neutron | 02:50 | |
*** iyamahat has joined #openstack-neutron | 02:51 | |
*** wolverin_ has quit IRC | 02:53 | |
*** Leom has joined #openstack-neutron | 02:56 | |
*** thorst has joined #openstack-neutron | 02:56 | |
*** Leo_ has quit IRC | 02:59 | |
*** mdnadeem has joined #openstack-neutron | 02:59 | |
*** wolverineav has joined #openstack-neutron | 03:00 | |
*** gongysh has quit IRC | 03:01 | |
*** openstackstatus has quit IRC | 03:01 | |
*** michchap has quit IRC | 03:02 | |
*** singhj has joined #openstack-neutron | 03:02 | |
*** ritesh has quit IRC | 03:03 | |
*** thorst has quit IRC | 03:04 | |
*** ritesh has joined #openstack-neutron | 03:04 | |
*** wolverineav has quit IRC | 03:04 | |
*** zhhuabj has joined #openstack-neutron | 03:05 | |
*** michchap has joined #openstack-neutron | 03:06 | |
*** reedip__ has quit IRC | 03:07 | |
*** reedip__ has joined #openstack-neutron | 03:07 | |
*** gongysh has joined #openstack-neutron | 03:08 | |
*** achanda has quit IRC | 03:09 | |
openstackgerrit | steve proposed openstack/neutron-specs: Add support for external vxlan encapsulation to neutron router https://review.openstack.org/282180 | 03:15 |
*** neeti has quit IRC | 03:19 | |
*** yamamoto_ has quit IRC | 03:20 | |
*** wolverineav has joined #openstack-neutron | 03:22 | |
*** singhj has quit IRC | 03:23 | |
openstackgerrit | ZhaoBo proposed openstack/python-neutronclient: Do not delete dhcp port when purge tenant's resource https://review.openstack.org/302046 | 03:23 |
*** jhershbe has quit IRC | 03:24 | |
*** wolverineav has quit IRC | 03:26 | |
*** kengo_sa_ has joined #openstack-neutron | 03:28 | |
*** kengo_sa_ has quit IRC | 03:28 | |
*** kengo_sakai_mba has joined #openstack-neutron | 03:28 | |
*** kengo_sakai_mba has quit IRC | 03:30 | |
*** kengo_sakai_mba has joined #openstack-neutron | 03:30 | |
*** gongysh has quit IRC | 03:30 | |
*** worawutchan has joined #openstack-neutron | 03:31 | |
*** tiswanso has quit IRC | 03:32 | |
*** gongysh has joined #openstack-neutron | 03:32 | |
*** kengo_sakai_mba has quit IRC | 03:34 | |
*** wolverineav has joined #openstack-neutron | 03:36 | |
*** ramishra has joined #openstack-neutron | 03:37 | |
worawutchan | Hi,Need help i want to know where can i find information about OVSvApp solution instructions guide. | 03:39 |
*** fitoduarte has joined #openstack-neutron | 03:41 | |
*** shashank_hegde has quit IRC | 03:42 | |
*** shivrao has joined #openstack-neutron | 03:44 | |
*** annp has joined #openstack-neutron | 03:46 | |
*** neelashah has joined #openstack-neutron | 03:49 | |
annp | Hi all, if you have time , can you review this: https://review.openstack.org/#/c/203509/. Thanks in advance. | 03:49 |
*** salv-orlando has joined #openstack-neutron | 03:50 | |
*** sekrit has quit IRC | 03:51 | |
*** worawutchan has quit IRC | 03:51 | |
*** salv-orlando has quit IRC | 03:52 | |
*** grassass has quit IRC | 03:52 | |
*** tflynn has joined #openstack-neutron | 03:53 | |
*** salv-orlando has joined #openstack-neutron | 03:53 | |
*** tflynn has joined #openstack-neutron | 03:54 | |
*** neelashah has quit IRC | 03:56 | |
*** Marga_ has joined #openstack-neutron | 03:56 | |
*** dave-mccowan has joined #openstack-neutron | 03:57 | |
*** julim has quit IRC | 03:57 | |
*** Marga_ has quit IRC | 03:57 | |
*** Marga_ has joined #openstack-neutron | 03:57 | |
*** brad_behle has quit IRC | 03:57 | |
*** salv-orlando has quit IRC | 03:58 | |
*** dave-mcc_ has joined #openstack-neutron | 03:58 | |
*** jamespage has quit IRC | 04:00 | |
*** jamespage has joined #openstack-neutron | 04:01 | |
*** dave-mccowan has quit IRC | 04:01 | |
*** Marga_ has quit IRC | 04:01 | |
*** thorst has joined #openstack-neutron | 04:01 | |
*** Marga_ has joined #openstack-neutron | 04:02 | |
*** busterswt has quit IRC | 04:02 | |
*** fedexo has joined #openstack-neutron | 04:02 | |
*** cshahani has joined #openstack-neutron | 04:05 | |
*** sekrit has joined #openstack-neutron | 04:05 | |
*** mkolesni_ has joined #openstack-neutron | 04:06 | |
*** numan_ has joined #openstack-neutron | 04:07 | |
*** thorst has quit IRC | 04:09 | |
*** yamamoto_ has joined #openstack-neutron | 04:10 | |
*** rickyrem has quit IRC | 04:10 | |
*** Sukhdev has joined #openstack-neutron | 04:11 | |
*** links has joined #openstack-neutron | 04:12 | |
*** absubram_ has joined #openstack-neutron | 04:14 | |
*** absubram_ has quit IRC | 04:14 | |
*** absubram has quit IRC | 04:15 | |
*** minwang2 has joined #openstack-neutron | 04:17 | |
*** tflynn has quit IRC | 04:17 | |
*** shivrao has quit IRC | 04:18 | |
*** tflynn has joined #openstack-neutron | 04:18 | |
*** shivrao has joined #openstack-neutron | 04:20 | |
*** yfried has joined #openstack-neutron | 04:20 | |
*** amotoki has joined #openstack-neutron | 04:20 | |
openstackgerrit | steve proposed openstack/python-neutronclient: Add a new option "--provider:flooding_vtep" to net-create. This option is used to configure the VTEP of VXLAN network which can't be found by neutron automatically. https://review.openstack.org/286504 | 04:20 |
*** shashank_hegde has joined #openstack-neutron | 04:22 | |
*** achanda has joined #openstack-neutron | 04:22 | |
*** javeriak has joined #openstack-neutron | 04:25 | |
*** iyamahat has quit IRC | 04:26 | |
*** banix has joined #openstack-neutron | 04:26 | |
*** anshul has quit IRC | 04:33 | |
*** achanda has quit IRC | 04:37 | |
*** irenab has quit IRC | 04:37 | |
*** tflynn has quit IRC | 04:40 | |
*** lujinluo has joined #openstack-neutron | 04:44 | |
*** vikram has joined #openstack-neutron | 04:44 | |
*** shivrao_ has joined #openstack-neutron | 04:50 | |
*** jonlan has joined #openstack-neutron | 04:52 | |
*** shivrao has quit IRC | 04:53 | |
*** shivrao_ is now known as shivrao | 04:53 | |
*** tflynn has joined #openstack-neutron | 04:54 | |
*** shausy has joined #openstack-neutron | 04:55 | |
*** admin0 has joined #openstack-neutron | 04:56 | |
*** shivrao has quit IRC | 04:56 | |
*** salv-orlando has joined #openstack-neutron | 04:57 | |
*** armax has quit IRC | 04:59 | |
*** abregman has joined #openstack-neutron | 05:00 | |
*** yfried has quit IRC | 05:01 | |
*** salv-orlando has quit IRC | 05:01 | |
*** kengo_sakai_mba has joined #openstack-neutron | 05:02 | |
*** aranjan has joined #openstack-neutron | 05:02 | |
*** oanson has joined #openstack-neutron | 05:03 | |
*** rossella_s has quit IRC | 05:03 | |
*** achanda has joined #openstack-neutron | 05:03 | |
*** rossella_s has joined #openstack-neutron | 05:03 | |
*** thorst has joined #openstack-neutron | 05:06 | |
*** admin0 has quit IRC | 05:08 | |
*** jonlan has quit IRC | 05:08 | |
*** asingh has quit IRC | 05:08 | |
*** numan_ has quit IRC | 05:08 | |
*** kawa2014 has joined #openstack-neutron | 05:09 | |
*** ushkalim has quit IRC | 05:10 | |
*** minwang2 has quit IRC | 05:10 | |
*** salv-orlando has joined #openstack-neutron | 05:11 | |
*** javeriak has quit IRC | 05:12 | |
*** thorst has quit IRC | 05:13 | |
*** asingh has joined #openstack-neutron | 05:16 | |
*** ramishra has quit IRC | 05:16 | |
*** sarun has joined #openstack-neutron | 05:17 | |
*** asingh has quit IRC | 05:17 | |
*** gongysh has quit IRC | 05:17 | |
*** gongysh has joined #openstack-neutron | 05:19 | |
*** michchap has quit IRC | 05:22 | |
*** cshahani has quit IRC | 05:23 | |
*** hdaniel has joined #openstack-neutron | 05:24 | |
*** michchap has joined #openstack-neutron | 05:24 | |
*** banix has quit IRC | 05:25 | |
*** shivrao has joined #openstack-neutron | 05:26 | |
openstackgerrit | Hong Hui Xiao proposed openstack/neutron: [WIP] Enable create and delete for segments in ML2 https://review.openstack.org/284440 | 05:29 |
*** aranjan has quit IRC | 05:29 | |
*** itamarl has joined #openstack-neutron | 05:30 | |
*** vthapar has joined #openstack-neutron | 05:31 | |
*** anilvenkata has joined #openstack-neutron | 05:33 | |
*** achanda has quit IRC | 05:33 | |
*** gampel1 has joined #openstack-neutron | 05:35 | |
*** shausy has quit IRC | 05:35 | |
*** irenab has joined #openstack-neutron | 05:36 | |
*** shausy has joined #openstack-neutron | 05:36 | |
*** mohankumar__ has joined #openstack-neutron | 05:37 | |
*** hieulq has quit IRC | 05:37 | |
*** fedexo has quit IRC | 05:39 | |
*** hieulq has joined #openstack-neutron | 05:39 | |
*** javeriak has joined #openstack-neutron | 05:39 | |
*** Sukhdev has quit IRC | 05:40 | |
*** fedexo has joined #openstack-neutron | 05:42 | |
*** minwang2 has joined #openstack-neutron | 05:43 | |
*** admin0 has joined #openstack-neutron | 05:43 | |
*** jhershbe has joined #openstack-neutron | 05:44 | |
*** sarun has quit IRC | 05:44 | |
*** rcernin has joined #openstack-neutron | 05:45 | |
*** itzikb has joined #openstack-neutron | 05:45 | |
*** garyk1 has joined #openstack-neutron | 05:46 | |
*** sarun has joined #openstack-neutron | 05:47 | |
*** minwang2 has quit IRC | 05:47 | |
*** minwang2 has joined #openstack-neutron | 05:47 | |
*** Shahid_ has joined #openstack-neutron | 05:49 | |
vikram | amotoki | 05:49 |
vikram | amotoki: ping | 05:49 |
vikram | amotoki: I tried the steps but couldn't get the result | 05:50 |
vikram | amotoki: Can you try once... I have created the file_list | 05:50 |
vikram | amotoki: https://github.com/VikramChoudhary/neutron-dynamic-routing/blob/master/file1 | 05:50 |
*** moshele has joined #openstack-neutron | 05:50 | |
amotoki | vikram: let me push my results to my github | 05:51 |
*** admin0 has quit IRC | 05:51 | |
vikram | amotoki: ok | 05:51 |
vikram | amotoki: i don't know what I did worng | 05:52 |
*** yfried has joined #openstack-neutron | 05:53 | |
*** minwang2 has quit IRC | 05:57 | |
*** sridharg has joined #openstack-neutron | 05:58 | |
*** achanda has joined #openstack-neutron | 05:59 | |
*** minwang2 has joined #openstack-neutron | 05:59 | |
*** shivrao has quit IRC | 06:00 | |
*** sarun has quit IRC | 06:00 | |
*** sarun has joined #openstack-neutron | 06:01 | |
*** sarun has quit IRC | 06:02 | |
*** ircuser-1 has joined #openstack-neutron | 06:03 | |
*** sarun has joined #openstack-neutron | 06:04 | |
*** Leom has quit IRC | 06:08 | |
*** eddima has joined #openstack-neutron | 06:08 | |
*** numan_ has joined #openstack-neutron | 06:10 | |
*** sarun has quit IRC | 06:11 | |
*** thorst has joined #openstack-neutron | 06:11 | |
*** gongysh has quit IRC | 06:13 | |
*** gongysh has joined #openstack-neutron | 06:14 | |
*** hdaniel has quit IRC | 06:17 | |
*** gongysh has quit IRC | 06:18 | |
*** thorst has quit IRC | 06:18 | |
*** kengo_sakai_mba has quit IRC | 06:19 | |
*** minwang2 has quit IRC | 06:19 | |
*** jlibosva has joined #openstack-neutron | 06:20 | |
*** oshvartz has joined #openstack-neutron | 06:21 | |
openstackgerrit | Merged openstack/neutron: devref: Update ovs-firewall https://review.openstack.org/299926 | 06:21 |
*** achanda has quit IRC | 06:22 | |
*** Leo_ has joined #openstack-neutron | 06:22 | |
*** scheuran has joined #openstack-neutron | 06:22 | |
*** minwang2 has joined #openstack-neutron | 06:24 | |
*** kevo has joined #openstack-neutron | 06:24 | |
*** moshele has quit IRC | 06:24 | |
*** shashank_hegde has quit IRC | 06:25 | |
*** pcaruana has joined #openstack-neutron | 06:26 | |
*** Leom has joined #openstack-neutron | 06:26 | |
*** fedexo has quit IRC | 06:27 | |
*** neeti has joined #openstack-neutron | 06:28 | |
*** numan_ has quit IRC | 06:28 | |
*** numan_ has joined #openstack-neutron | 06:28 | |
*** Leo_ has quit IRC | 06:29 | |
*** gongysh has joined #openstack-neutron | 06:30 | |
*** ekuris has joined #openstack-neutron | 06:30 | |
*** Leom has quit IRC | 06:33 | |
*** p01nt3r75 has quit IRC | 06:35 | |
*** shivrao has joined #openstack-neutron | 06:36 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add ALLOCATING state to routers https://review.openstack.org/257059 | 06:37 |
*** apoorv has joined #openstack-neutron | 06:38 | |
*** sridharg has quit IRC | 06:38 | |
*** ramishra has joined #openstack-neutron | 06:39 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Create standard attribute decorator https://review.openstack.org/292829 | 06:39 |
*** korzen has joined #openstack-neutron | 06:43 | |
*** ushkalim has joined #openstack-neutron | 06:45 | |
*** tesseract has joined #openstack-neutron | 06:45 | |
*** tesseract is now known as Guest12876 | 06:45 | |
*** sridharg has joined #openstack-neutron | 06:47 | |
*** sridharg has quit IRC | 06:47 | |
*** kawa2014 has quit IRC | 06:50 | |
*** sridharg has joined #openstack-neutron | 06:50 | |
*** salv-orlando has quit IRC | 06:52 | |
*** javeriak has quit IRC | 06:56 | |
*** hdaniel has joined #openstack-neutron | 06:58 | |
*** EinstCra_ has joined #openstack-neutron | 06:59 | |
*** minwang2 has quit IRC | 07:00 | |
*** apoorv_ has joined #openstack-neutron | 07:00 | |
*** ekuris has quit IRC | 07:01 | |
*** EinstCrazy has quit IRC | 07:02 | |
*** kobis has joined #openstack-neutron | 07:03 | |
*** Marga_ has quit IRC | 07:04 | |
*** wolverineav has quit IRC | 07:04 | |
*** hdaniel has quit IRC | 07:05 | |
*** fzdarsky has joined #openstack-neutron | 07:08 | |
*** dave-mcc_ has quit IRC | 07:09 | |
*** shivrao has quit IRC | 07:09 | |
*** admin0 has joined #openstack-neutron | 07:10 | |
*** nyechiel_ has joined #openstack-neutron | 07:10 | |
*** kickinz1 has joined #openstack-neutron | 07:11 | |
*** atuvenie has joined #openstack-neutron | 07:11 | |
*** admin0 has quit IRC | 07:11 | |
*** nmagnezi has joined #openstack-neutron | 07:11 | |
*** ramishra has quit IRC | 07:11 | |
*** ihrachys has joined #openstack-neutron | 07:11 | |
*** jlanoux has joined #openstack-neutron | 07:13 | |
*** javeriak has joined #openstack-neutron | 07:14 | |
*** Shahid_ has quit IRC | 07:15 | |
*** moshele has joined #openstack-neutron | 07:15 | |
*** thorst has joined #openstack-neutron | 07:16 | |
*** ekuris has joined #openstack-neutron | 07:17 | |
*** admin0 has joined #openstack-neutron | 07:19 | |
*** tflynn has quit IRC | 07:19 | |
*** kengo_sakai_mba has joined #openstack-neutron | 07:19 | |
*** hdaniel has joined #openstack-neutron | 07:21 | |
*** thorst has quit IRC | 07:24 | |
*** baojg has quit IRC | 07:25 | |
*** baojg has joined #openstack-neutron | 07:25 | |
*** ramishra has joined #openstack-neutron | 07:27 | |
*** kengo_sakai_mba has quit IRC | 07:28 | |
*** abehl has joined #openstack-neutron | 07:29 | |
*** matrohon has joined #openstack-neutron | 07:29 | |
*** shivrao has joined #openstack-neutron | 07:29 | |
*** yfried_ has joined #openstack-neutron | 07:32 | |
*** yfried has quit IRC | 07:32 | |
*** abehl has quit IRC | 07:33 | |
reedip | amotoki: Can you have a look if you have time: https://review.openstack.org/#/c/282184/ ? | 07:33 |
*** ekuris has quit IRC | 07:35 | |
*** ramishra has quit IRC | 07:36 | |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups https://review.openstack.org/284738 | 07:37 |
openstackgerrit | Lujin Luo proposed openstack/neutron: Add a composite unique key to floatingip table in Neutron database https://review.openstack.org/276018 | 07:38 |
*** dedery has joined #openstack-neutron | 07:38 | |
*** ramishra has joined #openstack-neutron | 07:39 | |
openstackgerrit | Merged openstack/neutron: Simplify chained comparison https://review.openstack.org/301156 | 07:41 |
openstackgerrit | Merged openstack/neutron: QoS DSCP use mod_flow instead of mod_flows https://review.openstack.org/300635 | 07:41 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Merge tag '8.0.0' https://review.openstack.org/302570 | 07:43 |
*** shivrao has quit IRC | 07:43 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Merge tag '8.0.0' https://review.openstack.org/302571 | 07:44 |
*** jpena|off is now known as jpean | 07:44 | |
*** jpean is now known as jpena | 07:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Merge tag '8.0.0' https://review.openstack.org/302576 | 07:44 |
*** jamielennox is now known as jamielennox|away | 07:45 | |
*** ekuris has joined #openstack-neutron | 07:48 | |
vikram | amotoki: ping | 07:49 |
amotoki | vikram: pong | 07:49 |
amotoki | reedip: will look soon | 07:49 |
vikram | amotoki: got few questions while reviewing "https://review.openstack.org/#/c/299399/" | 07:49 |
vikram | amotoki: can you please help | 07:49 |
reedip | amotoki: Okay :) | 07:50 |
*** matrohon_ has joined #openstack-neutron | 07:51 | |
*** matrohon_ has quit IRC | 07:51 | |
amotoki | vikram: thanks for reminding. will reply in the review | 07:51 |
vikram | amotoki: thanks | 07:52 |
*** hdaniel has quit IRC | 07:52 | |
ihrachys | rossella_s: would you mind https://review.openstack.org/#/c/301837/ ? | 07:54 |
*** salv-orlando has joined #openstack-neutron | 07:54 | |
*** ramishra has quit IRC | 07:55 | |
ihrachys | rossella_s: also I believe https://review.openstack.org/#/c/292829/ is good to go. there is a comment from korzen but I think it's not valid. | 07:55 |
openstackgerrit | Merged openstack/neutron: Fix race conditions in IP availability API tests https://review.openstack.org/299647 | 07:56 |
ihrachys | ah not really the last one since it depends on a patch with -1 | 07:57 |
vikram | reedip: ping | 07:57 |
*** yfried_ has quit IRC | 07:58 | |
reedip | vikram: pongs | 07:58 |
vikram | reedip: couldn't find your views @ https://review.openstack.org/#/c/299399/ | 07:58 |
reedip | Just check the 2 comments above yours :) | 07:58 |
korzen | ihrachys, rossella_s I've +1 on 292829 | 07:58 |
ihrachys | korzen: thanks | 07:59 |
ihrachys | rossella_s: but https://review.openstack.org/#/c/302267/2 should be good to go, for my taste | 07:59 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: ovo: Update object with data from db on update() https://review.openstack.org/302267 | 08:00 |
*** haukebruno has joined #openstack-neutron | 08:03 | |
*** devvesa has joined #openstack-neutron | 08:04 | |
*** atuvenie has quit IRC | 08:04 | |
*** wolverineav has joined #openstack-neutron | 08:05 | |
*** hdaniel has joined #openstack-neutron | 08:05 | |
rossella_s | ihrachys, I am on it | 08:07 |
*** apoorv_ has quit IRC | 08:08 | |
*** apoorv has quit IRC | 08:08 | |
ihrachys | rossella_s: question on macaddress field. I probably miss something substential | 08:09 |
ihrachys | rossella_s: why do we push strings into the field and not netaddr.EUI? | 08:09 |
*** lajos-katona has joined #openstack-neutron | 08:09 | |
*** kawa2014 has joined #openstack-neutron | 08:09 | |
*** wolverineav has quit IRC | 08:10 | |
*** salv-orlando has quit IRC | 08:10 | |
rossella_s | ihrachys, where do I do that? | 08:10 |
*** mickeys has joined #openstack-neutron | 08:11 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Create standard attribute decorator https://review.openstack.org/292829 | 08:11 |
rossella_s | ihrachys, maybe I miss something | 08:11 |
ihrachys | rossella_s: https://review.openstack.org/#/c/293479/7/neutron/tests/unit/objects/test_common_types.py get_random_mac returns a string right? | 08:11 |
*** rubasov has quit IRC | 08:12 | |
*** ygbo has joined #openstack-neutron | 08:12 | |
ihrachys | rossella_s: note that f.e. for obj_fields.IPNetworkField we set netaddr.IPNetwork | 08:12 |
rossella_s | ihrachys, good point | 08:13 |
ihrachys | so that would be reasonable to have it consistent | 08:13 |
rossella_s | ihrachys, it was there before, I didn't notice it | 08:13 |
ihrachys | ok let me make a common | 08:13 |
ihrachys | comment | 08:13 |
*** rubasov has joined #openstack-neutron | 08:14 | |
ihrachys | rossella_s: done. there was also a small nit in the test, I posted the comment too | 08:14 |
ihrachys | rossella_s: if you don't have time for that atm I can handle it | 08:14 |
rossella_s | ihrachys, great, I will update it later today, after the reviews | 08:14 |
rossella_s | ihrachys, but if you want to go on, please do | 08:14 |
ihrachys | rossella_s: ok I will respin it. trying to make things going. :) | 08:14 |
rossella_s | ihrachys, same mood here! let's get things rolling | 08:15 |
*** ramishra has joined #openstack-neutron | 08:15 | |
*** admin0 has quit IRC | 08:17 | |
*** admin0 has joined #openstack-neutron | 08:19 | |
*** bjornar has joined #openstack-neutron | 08:20 | |
*** thorst has joined #openstack-neutron | 08:21 | |
*** gampel1 has left #openstack-neutron | 08:24 | |
*** mickeys has quit IRC | 08:25 | |
*** mickeys has joined #openstack-neutron | 08:26 | |
*** salv-orlando has joined #openstack-neutron | 08:26 | |
*** ramishra has quit IRC | 08:26 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom OVO field type for MAC address https://review.openstack.org/293479 | 08:27 |
ihrachys | rossella_s: that ^ | 08:27 |
rossella_s | ihrachys, great! | 08:28 |
*** openstackstatus has joined #openstack-neutron | 08:29 | |
*** ChanServ sets mode: +v openstackstatus | 08:29 | |
*** thorst has quit IRC | 08:30 | |
*** mickeys has quit IRC | 08:30 | |
*** jistr has joined #openstack-neutron | 08:30 | |
*** mickeys has joined #openstack-neutron | 08:31 | |
rossella_s | ihrachys, I +2 both patches you mentioned | 08:33 |
-openstackstatus- NOTICE: jobs depending on npm are now working again | 08:34 | |
*** apoorv_ has joined #openstack-neutron | 08:36 | |
*** apoorv has joined #openstack-neutron | 08:36 | |
ihrachys | rossella_s: https://review.openstack.org/#/c/302267/ that one is the first one in the series | 08:36 |
rossella_s | ihrachys, I will review that too and the depending patches | 08:37 |
*** vhosakot has quit IRC | 08:38 | |
*** p01nt3r75 has joined #openstack-neutron | 08:39 | |
*** ramishra has joined #openstack-neutron | 08:45 | |
*** rmart04 has joined #openstack-neutron | 08:45 | |
jlibosva | ihrachys: should I do the thingy about calling classmethod from instance method to modify fields for both direction even though it's need only to "from db" ? | 08:45 |
ihrachys | jlibosva: I would hug you then :P | 08:45 |
jlibosva | ihrachys: so I'm pushing the one way only :D | 08:45 |
* ihrachys now realizes it may sound like a threat! | 08:45 | |
ihrachys | jlibosva: I am fine with a piece | 08:46 |
*** Shahid_ has joined #openstack-neutron | 08:47 | |
*** EinstCra_ is now known as EinstCrazy | 08:51 | |
ihrachys | Sam-I-Am: you prolly want to chime in before https://review.openstack.org/#/c/302165/ | 08:52 |
ihrachys | ...lands. | 08:52 |
*** salv-orl_ has joined #openstack-neutron | 08:52 | |
*** mickeys has quit IRC | 08:52 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods https://review.openstack.org/302276 | 08:53 |
*** mickeys has joined #openstack-neutron | 08:54 | |
*** mickeys has quit IRC | 08:54 | |
*** mickeys has joined #openstack-neutron | 08:54 | |
*** mickeys has quit IRC | 08:54 | |
*** salv-orlando has quit IRC | 08:55 | |
*** rossella_s has quit IRC | 09:03 | |
*** rossella_s has joined #openstack-neutron | 09:03 | |
*** lujinluo has quit IRC | 09:04 | |
*** javeriak has quit IRC | 09:05 | |
*** nijaba has quit IRC | 09:05 | |
*** sambetts|afk is now known as sambetts | 09:05 | |
*** nijaba has joined #openstack-neutron | 09:07 | |
*** yfried has joined #openstack-neutron | 09:08 | |
*** djan has quit IRC | 09:11 | |
*** mfuruta has quit IRC | 09:12 | |
*** abehl has joined #openstack-neutron | 09:15 | |
openstackgerrit | venkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver https://review.openstack.org/281116 | 09:16 |
openstackgerrit | venkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports https://review.openstack.org/241227 | 09:16 |
openstackgerrit | venkata anil proposed openstack/neutron: Optimize get_ports query by filtering on subnet https://review.openstack.org/302062 | 09:16 |
*** zhhuabj has quit IRC | 09:18 | |
*** zhhuabj has joined #openstack-neutron | 09:21 | |
*** javeriak has joined #openstack-neutron | 09:22 | |
*** hichihara has quit IRC | 09:23 | |
*** shausy has quit IRC | 09:24 | |
*** neiljerram has joined #openstack-neutron | 09:26 | |
*** thorst has joined #openstack-neutron | 09:27 | |
*** ramishra has quit IRC | 09:28 | |
*** ramishra has joined #openstack-neutron | 09:30 | |
*** namnh has quit IRC | 09:30 | |
*** links has quit IRC | 09:31 | |
*** alex_xu has quit IRC | 09:32 | |
*** annp has quit IRC | 09:32 | |
*** hoangcx has quit IRC | 09:32 | |
*** thorst has quit IRC | 09:34 | |
*** hoangcx has joined #openstack-neutron | 09:35 | |
*** alex_xu has joined #openstack-neutron | 09:36 | |
*** annp has joined #openstack-neutron | 09:40 | |
*** yamamoto_ has quit IRC | 09:40 | |
*** mvk has joined #openstack-neutron | 09:41 | |
*** mkrcmari__ has quit IRC | 09:43 | |
*** links has joined #openstack-neutron | 09:44 | |
*** rubasov has quit IRC | 09:48 | |
*** mvk_ has joined #openstack-neutron | 09:49 | |
*** gongysh has quit IRC | 09:49 | |
*** gongysh has joined #openstack-neutron | 09:50 | |
openstackgerrit | venkata anil proposed openstack/neutron: l2pop fdb flows for HA router ports https://review.openstack.org/255237 | 09:52 |
openstackgerrit | venkata anil proposed openstack/neutron: HA: notify l2pop on port status update in slave https://review.openstack.org/282874 | 09:52 |
*** javeriak has quit IRC | 09:53 | |
*** rubasov has joined #openstack-neutron | 09:53 | |
*** mvk has quit IRC | 09:53 | |
*** mhickey has joined #openstack-neutron | 09:54 | |
*** achanda has joined #openstack-neutron | 09:56 | |
*** obondarev has quit IRC | 10:00 | |
*** achanda has quit IRC | 10:02 | |
openstackgerrit | venkata anil proposed openstack/neutron: [l2pop] fix agents failing to create flood flows https://review.openstack.org/291674 | 10:03 |
*** ociuhandu has quit IRC | 10:05 | |
*** obondarev has joined #openstack-neutron | 10:06 | |
*** wolverineav has joined #openstack-neutron | 10:06 | |
*** numan_ has quit IRC | 10:07 | |
*** Shahid_ has quit IRC | 10:07 | |
*** baojg has quit IRC | 10:09 | |
*** wolverineav has quit IRC | 10:11 | |
*** obondarev has quit IRC | 10:12 | |
*** Kiall has quit IRC | 10:14 | |
*** Kiall has joined #openstack-neutron | 10:14 | |
*** tbachman has quit IRC | 10:14 | |
*** tbachman has joined #openstack-neutron | 10:14 | |
*** fzdarsky has quit IRC | 10:15 | |
*** hoangcx has quit IRC | 10:16 | |
*** hoangcx has joined #openstack-neutron | 10:16 | |
ihrachys | rossella_s: the mac type, I can't vote for that since now I am an author, but you can: https://review.openstack.org/#/c/293479/ :) | 10:17 |
*** fzdarsky has joined #openstack-neutron | 10:17 | |
rossella_s | ihrachys, lol...am I not an author :p ? | 10:17 |
ihrachys | rossella_s: you have not added yourself in a tag! :) | 10:18 |
ihrachys | rossella_s: also, I don't think there is much left from the original implementation. | 10:18 |
rossella_s | ihrachys, got you | 10:19 |
*** sdague has joined #openstack-neutron | 10:20 | |
*** singhj has joined #openstack-neutron | 10:20 | |
*** numan_ has joined #openstack-neutron | 10:21 | |
*** Shahid_ has joined #openstack-neutron | 10:22 | |
*** numan_ has quit IRC | 10:22 | |
*** numan_ has joined #openstack-neutron | 10:22 | |
*** singhj has quit IRC | 10:24 | |
openstackgerrit | Merged openstack/neutron: standard attributes: expose created_at/updated_at on models https://review.openstack.org/301837 | 10:27 |
*** EinstCrazy has quit IRC | 10:30 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config https://review.openstack.org/302653 | 10:30 |
*** obondarev has joined #openstack-neutron | 10:31 | |
*** thorst has joined #openstack-neutron | 10:32 | |
*** gongysh has quit IRC | 10:32 | |
*** mhickey has quit IRC | 10:35 | |
*** Shahid_ has quit IRC | 10:37 | |
*** thorst has quit IRC | 10:39 | |
*** links has quit IRC | 10:39 | |
*** yamamoto has joined #openstack-neutron | 10:42 | |
*** abregman has quit IRC | 10:44 | |
*** abregman has joined #openstack-neutron | 10:44 | |
*** gongysh has joined #openstack-neutron | 10:44 | |
ihrachys | ajo: may I ask you to review https://review.openstack.org/#/c/293479/ ? it's one of ovo related patches that would save us some stacking. | 10:46 |
ihrachys | and it's simple | 10:47 |
*** Guest12876 is now known as tesseract | 10:47 | |
*** mvk_ has quit IRC | 10:47 | |
*** tesseract is now known as Guest74036 | 10:48 | |
*** mhickey has joined #openstack-neutron | 10:50 | |
HenryG | ihrachys: https://review.openstack.org/302570 | 10:52 |
HenryG | ihrachys: shall we merge it? | 10:52 |
*** andymaier has joined #openstack-neutron | 10:53 | |
*** alex_xu has quit IRC | 10:55 | |
*** sridharg has quit IRC | 10:55 | |
ihrachys | prolly. was there an email with the announcement for those patches? | 10:55 |
*** links has joined #openstack-neutron | 10:56 | |
*** kengo_sakai_mba has joined #openstack-neutron | 10:56 | |
ihrachys | HenryG: ? | 10:56 |
HenryG | I saw http://lists.openstack.org/pipermail/openstack-dev/2016-April/091649.html | 10:57 |
*** alex_xu has joined #openstack-neutron | 10:57 | |
HenryG | Knowing dhellmann's time zone, I am guessing he has been up all night o_O | 10:58 |
ihrachys | that one does not talk about those patches though | 10:58 |
HenryG | right | 10:58 |
*** jlanoux has quit IRC | 10:58 | |
ihrachys | but I guess we should be ok to merge it. seems legit, I remember we did similar thing for Liberty. | 10:58 |
*** achanda has joined #openstack-neutron | 10:58 | |
HenryG | I remember that too | 10:59 |
HenryG | Checking release channel ... | 10:59 |
*** cgalan has quit IRC | 10:59 | |
ihrachys | HenryG: https://review.openstack.org/#/c/302452/ | 11:00 |
*** p01nt3r75 has quit IRC | 11:00 | |
ihrachys | HenryG: I +2'd | 11:00 |
*** annp has quit IRC | 11:01 | |
*** gongysh has quit IRC | 11:02 | |
*** nyechiel_ has quit IRC | 11:02 | |
HenryG | ihrachys: ack | 11:02 |
pc_m | ihrachys: I just did the same for the VPN repo | 11:02 |
*** ushkalim has quit IRC | 11:02 | |
*** jlanoux has joined #openstack-neutron | 11:03 | |
ihrachys | ah right, going thru other repos | 11:03 |
pc_m | https://review.openstack.org/#/c/302576/ | 11:03 |
pc_m | fyi | 11:03 |
*** achanda has quit IRC | 11:03 | |
*** yamamoto has quit IRC | 11:04 | |
ihrachys | pc_m: +W | 11:04 |
pc_m | ihrachys: Thanks! | 11:04 |
ihrachys | HenryG: pc_m: I guess we need someone else with +2 in *aas? | 11:04 |
ihrachys | I +2'd but it's not two votes | 11:04 |
pc_m | ihrachys: Yeah. | 11:05 |
*** jlanoux_ has joined #openstack-neutron | 11:05 | |
ihrachys | or I go and abuse the rule for the benefit of everyone? :) | 11:05 |
HenryG | ihrachys: I don't have +2 there | 11:05 |
* ihrachys checking gerrit groups | 11:05 | |
pc_m | ihrachys: me neither. Need FW and LB cores. | 11:05 |
ihrachys | would be weird to land neutron but not *aas | 11:06 |
*** yamamoto has joined #openstack-neutron | 11:06 | |
*** ygbo has quit IRC | 11:06 | |
ihrachys | I will go and abuse and will prepare to explain my misbehaviour! | 11:06 |
pc_m | HenryG: As drivers team member, can't you +2 *aaS? | 11:07 |
*** jlanoux has quit IRC | 11:07 | |
HenryG | pc_m: Nope. Only on specs. | 11:07 |
ihrachys | all in gate | 11:08 |
HenryG | ihrachys: you rebel! | 11:08 |
* ihrachys feels good | 11:09 | |
HenryG | :) | 11:09 |
*** jprovazn has joined #openstack-neutron | 11:09 | |
*** sridharg has joined #openstack-neutron | 11:10 | |
*** sridharg has quit IRC | 11:10 | |
*** sridharg has joined #openstack-neutron | 11:10 | |
pc_m | ihrachys: I've got your back... | 11:10 |
* pc_m runs away silently | 11:10 | |
*** thorst has joined #openstack-neutron | 11:11 | |
*** thorst_ has joined #openstack-neutron | 11:11 | |
openstackgerrit | Merged openstack/neutron: ovo: Update object with data from db on update() https://review.openstack.org/302267 | 11:13 |
winggundamth | anyone knows about policy entry name that control add gateway interface to router? | 11:13 |
*** thorst has quit IRC | 11:15 | |
*** kengo_sakai_mba has quit IRC | 11:15 | |
ihrachys | HenryG: does it mean we finally released Mitaka? | 11:16 |
ihrachys | I guess so | 11:16 |
HenryG | That's what I remember from last cycle | 11:16 |
*** nyechiel_ has joined #openstack-neutron | 11:16 | |
ihrachys | woohoo | 11:16 |
*** mvk_ has joined #openstack-neutron | 11:16 | |
HenryG | But the release mgr probably waits for all project tags to merge before announcing, I guess | 11:17 |
*** ushkalim has joined #openstack-neutron | 11:18 | |
HenryG | As a dev it feels like it was released weeks ago :) | 11:18 |
*** rtheis has joined #openstack-neutron | 11:18 | |
*** fzdarsky has quit IRC | 11:21 | |
*** tpsilva has joined #openstack-neutron | 11:22 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Remove unused PolicyFileNotFound exception https://review.openstack.org/302497 | 11:23 |
*** zhhuabj has quit IRC | 11:23 | |
*** zhhuabj has joined #openstack-neutron | 11:26 | |
*** admin0 has quit IRC | 11:26 | |
ihrachys | HenryG: remember there was a patch to add test coverage framework for alembic scripts? | 11:28 |
ihrachys | HenryG: do you have a link? can't find it :-| | 11:28 |
HenryG | ihrachys: https://review.openstack.org/124946 | 11:29 |
ihrachys | oh it's merged! | 11:29 |
*** john_bar has quit IRC | 11:29 | |
ihrachys | HenryG: should we force tests in scripts thru pep8 + update script templates? | 11:30 |
HenryG | ihrachys: I don't think we are there yet | 11:30 |
ihrachys | HenryG: what's missing? | 11:31 |
HenryG | ihrachys: actual tests | 11:31 |
ihrachys | HenryG: you mean for existing scripts? | 11:31 |
ihrachys | HenryG: I suggest to enforce them for new scripts | 11:31 |
*** admin0 has joined #openstack-neutron | 11:31 | |
HenryG | ihrachys: the framework is for functional tests | 11:33 |
jlibosva | ihrachys: can you please look at my reply at https://review.openstack.org/#/c/302276/3/neutron/objects/base.py ? I don't get what you meant | 11:33 |
HenryG | ihrachys: it needs mysql and postgresql backends. Won't work in pep8. | 11:34 |
ihrachys | HenryG: ok maybe I don't follow the goal of it. but I suppose it allows us to pre-populate db with models before testing script upgrades | 11:34 |
ihrachys | HenryG: in pep8, we would merely check that _pre_upgrade_ is defined | 11:34 |
ihrachys | HenryG: and then it will be reviewers call to make sure it makes stuff | 11:35 |
HenryG | ihrachys: perhaps that could be useful | 11:35 |
HenryG | ihrachys: the goal is to upgrade to the revision before, then put actual records (data) in the DB, then upgrade to the new revision, with data verification. | 11:38 |
HenryG | ihrachys: this is primarily to verify scripts that change record data | 11:39 |
*** stanzgy has quit IRC | 11:40 | |
*** alex_xu has quit IRC | 11:41 | |
*** alex_xu has joined #openstack-neutron | 11:43 | |
*** dslevin1 has joined #openstack-neutron | 11:44 | |
*** dslevin1 has quit IRC | 11:44 | |
ihrachys | HenryG: exactly. so I would say, it's cool to dream about us going back to existing scripts and add test coverage there | 11:44 |
ihrachys | HenryG: but it may not happen | 11:44 |
ihrachys | HenryG: and we can enforce authors for new patches to get coverage for us at least for new stuff | 11:44 |
ajo | ihrachys, checking | 11:44 |
ihrachys | HenryG: does it make sense? | 11:45 |
HenryG | ihrachys: Well, I am not sure that all scripts should require data tests. | 11:45 |
ihrachys | HenryG: we just make sure they have the function defined | 11:45 |
ihrachys | HenryG: then reviewers make a call | 11:45 |
*** dslevin1 has joined #openstack-neutron | 11:46 | |
ihrachys | HenryG: I guess once a reviewer sees the empty method, he may start asking questions | 11:46 |
HenryG | ihrachys: I guess it will raise awareness. And maybe annoyance. ;) | 11:46 |
ihrachys | HenryG: we can even make the method name explicit enough, like '_pre_populate_for_testing' :) | 11:46 |
ihrachys | HenryG: I love to annoy people | 11:47 |
ihrachys | ok, I will go craft some patches for that then :)_ | 11:47 |
*** jhesketh has joined #openstack-neutron | 11:48 | |
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack" | 11:48 | |
HenryG | ihrachys: That's fine. I may not allow it to merge until I see an actual test implemented. | 11:49 |
*** Marga_ has joined #openstack-neutron | 11:49 | |
*** banix has joined #openstack-neutron | 11:50 | |
ihrachys | HenryG: that's ok. I just want to make sure we have consistent coverage in Newton | 11:50 |
*** anilvenkata has quit IRC | 11:50 | |
HenryG | ihrachys: That is a worthy goal, indeed. | 11:51 |
*** hynekm has joined #openstack-neutron | 11:52 | |
*** p01nt3r75 has joined #openstack-neutron | 11:53 | |
ajo | ihrachys, done: https://review.openstack.org/#/c/293479/8 | 11:54 |
ajo | let me know if I'm missing something | 11:54 |
*** fzdarsky has joined #openstack-neutron | 11:55 | |
*** javeriak has joined #openstack-neutron | 11:58 | |
ihrachys | ajo: I think you are right, that's only because we have test_stringify_invalid test that calls stringify() with random values. | 11:59 |
*** andymaier has quit IRC | 11:59 | |
ihrachys | rossella_s: korzen: why do we even have test_stringify_invalid? | 11:59 |
ihrachys | test_coerce_bad_values should be enough to check validation | 11:59 |
ihrachys | that would allow us to kill all custom stringify() methods we have | 12:00 |
rossella_s | ihrachys, I raised a similar point yesterday...I guess the thing is if we want to validate when we call stringify | 12:00 |
ihrachys | rossella_s: but it's not called with random values by anything, right? | 12:02 |
ihrachys | rossella_s: the only place it's used is in base class for ovo | 12:02 |
ihrachys | rossella_s: in __repr__ | 12:02 |
rossella_s | ihrachys, theoretically coerce is called first | 12:02 |
ihrachys | where it passes getattr(self, name) as a value | 12:02 |
ihrachys | so coerce already happened | 12:02 |
rossella_s | ihrachys, right | 12:02 |
ihrachys | so I go and clean that up? :) | 12:03 |
rossella_s | ihrachys, the convention in our custom type seems to be to validate in stringify too | 12:03 |
ihrachys | rossella_s: I am not sure it's a convention or just a random mistake that got copy-pasted several times | 12:03 |
rossella_s | ihrachys, I agree in changing that...although that should be done for other types too, not only the MAC | 12:03 |
ihrachys | rossella_s: right. I will propose a follow up and ask ajo to reconsider the vote for the MAC patch for now | 12:03 |
*** norman_ has joined #openstack-neutron | 12:04 | |
rossella_s | ihrachys, agreed, thanks for that | 12:04 |
*** yfried_ has joined #openstack-neutron | 12:04 | |
*** yfried has quit IRC | 12:05 | |
*** EinstCrazy has joined #openstack-neutron | 12:06 | |
*** jhesketh has quit IRC | 12:07 | |
*** achanda has joined #openstack-neutron | 12:07 | |
*** jhesketh has joined #openstack-neutron | 12:07 | |
*** ekuris has quit IRC | 12:07 | |
*** wolverineav has joined #openstack-neutron | 12:08 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: avoid validation in stringify for custom field types https://review.openstack.org/302704 | 12:09 |
ihrachys | ajo: ^ | 12:09 |
ihrachys | rossella_s: ^ | 12:09 |
ihrachys | ajo: please reconsider the vote for MAC patch since I want to handle the comment in this follow up | 12:09 |
*** cgalan has joined #openstack-neutron | 12:10 | |
*** davidsha has joined #openstack-neutron | 12:10 | |
*** cgalan has quit IRC | 12:10 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods https://review.openstack.org/302276 | 12:10 |
ihrachys | HenryG: oh I think I now see that the framework is not completely in line with how I thought about it | 12:12 |
*** kengo_sakai_mba has joined #openstack-neutron | 12:12 | |
ihrachys | HenryG: it requires test definition in functional tree, not in alembic scripts themselves? | 12:12 |
*** wolverineav has quit IRC | 12:12 | |
HenryG | ihrachys: yes | 12:12 |
*** baoli has joined #openstack-neutron | 12:13 | |
ihrachys | I somehow thought you would define some callables in script and get them triggered by the framework | 12:13 |
*** ygbo has joined #openstack-neutron | 12:13 | |
*** baoli_ has joined #openstack-neutron | 12:13 | |
HenryG | ihrachys: Hah! I thought you were going to come up with some advanced hacking check for pep8. :) | 12:15 |
*** norman_ has quit IRC | 12:15 | |
*** zhhuabj has quit IRC | 12:15 | |
*** achanda has quit IRC | 12:15 | |
*** admin0 has quit IRC | 12:16 | |
ihrachys | HenryG: well, it may actually still work :) | 12:17 |
*** kengo_sakai_mba has quit IRC | 12:17 | |
ihrachys | HenryG: I mean, some hacking check or a test for the test ;) | 12:17 |
*** neelashah has joined #openstack-neutron | 12:17 | |
*** javeriak has quit IRC | 12:17 | |
*** zhhuabj has joined #openstack-neutron | 12:17 | |
* ihrachys hacks an evil enforcement muahaha | 12:17 | |
*** baoli has quit IRC | 12:18 | |
ihrachys | HenryG: why do we have downgrades handling there? | 12:19 |
*** karthiks has joined #openstack-neutron | 12:20 | |
*** Shahid_ has joined #openstack-neutron | 12:21 | |
*** ekuris has joined #openstack-neutron | 12:22 | |
*** neelashah has quit IRC | 12:22 | |
*** edmondsw has joined #openstack-neutron | 12:23 | |
*** mohankumar__ has quit IRC | 12:24 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tet_migrations: remove code to validate downgrades https://review.openstack.org/302722 | 12:26 |
*** admin0 has joined #openstack-neutron | 12:26 | |
*** doug-fish has joined #openstack-neutron | 12:26 | |
*** akshai has joined #openstack-neutron | 12:26 | |
*** apoorv_ has quit IRC | 12:27 | |
*** apoorv has quit IRC | 12:27 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: test_migrations: always assume we want to validate scripts with data https://review.openstack.org/302724 | 12:28 |
*** salv-orl_ has quit IRC | 12:28 | |
*** sakthi_chn has quit IRC | 12:28 | |
*** wuhg has quit IRC | 12:29 | |
openstackgerrit | Merged openstack/neutron: Merge tag '8.0.0' https://review.openstack.org/302570 | 12:29 |
*** nplanel has quit IRC | 12:29 | |
*** amuller has joined #openstack-neutron | 12:30 | |
pavel_bondar | garyk: Hi! FYI, UTs has been added for https://review.openstack.org/#/c/272125/ | 12:30 |
openstackgerrit | Merged openstack/neutron-vpnaas: Merge tag '8.0.0' https://review.openstack.org/302576 | 12:30 |
openstackgerrit | Merged openstack/neutron: Return oslo_config Opts to config generator https://review.openstack.org/292640 | 12:31 |
ihrachys | woot, we land the tags. | 12:31 |
*** banix has quit IRC | 12:31 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Update ml2 delete_subnet to deallocate via ipam https://review.openstack.org/300984 | 12:31 |
*** banix has joined #openstack-neutron | 12:32 | |
*** banix has quit IRC | 12:32 | |
openstackgerrit | Merged openstack/neutron-fwaas: Merge tag '8.0.0' https://review.openstack.org/302571 | 12:32 |
HenryG | ihrachys: I know you backport aggressively, but I want to point out bug 1548433 | 12:32 |
openstack | bug 1548433 in keystoneauth "neutron returns objects other than oslo_config.cfg.Opt instances from list_opts" [Undecided,Incomplete] https://launchpad.net/bugs/1548433 | 12:32 |
ihrachys | HenryG: I *just* cherry-picked :) | 12:32 |
HenryG | ihrachys: good! :) | 12:33 |
ihrachys | HenryG: should have gone into final, but well, it's life | 12:33 |
*** nplanel has joined #openstack-neutron | 12:33 | |
*** pradk has joined #openstack-neutron | 12:34 | |
*** brad_behle has joined #openstack-neutron | 12:34 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config https://review.openstack.org/302653 | 12:34 |
HenryG | ihrachys: yeah, we released Mitaka with it generating junk keystone config options. :( | 12:34 |
*** mkolesni_ has quit IRC | 12:35 | |
*** jpena is now known as jpena|lunch | 12:35 | |
ihrachys | HenryG: way to go! | 12:36 |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups https://review.openstack.org/284738 | 12:36 |
ihrachys | HenryG: why wasn't the bug marked as a blocker? | 12:36 |
*** mohankumar__ has joined #openstack-neutron | 12:36 | |
garyk1 | ihrachys: you understand fixtures? | 12:36 |
ajo | ihrachys, done, thanks! | 12:37 |
ihrachys | ajo: thanks | 12:37 |
*** lucasagomes has quit IRC | 12:37 | |
ihrachys | garyk1: well... I think there are people who know more, like ZZelle or jlibosva | 12:37 |
ihrachys | garyk1: but you can try | 12:37 |
ihrachys | HenryG: I moved it to High, it's not a Low bug for my consideration | 12:37 |
garyk1 | a change in neutron over the last 12 hrs has broken our plugins unit tests - http://logs.openstack.org/13/293513/8/check/gate-vmware-nsx-python27/48c99df/testr_results.html.gz | 12:38 |
jlibosva | lifeless knows probably the most of it ;) | 12:38 |
*** lucasagomes has joined #openstack-neutron | 12:38 | |
garyk1 | TypeError: fake_consume_in_threads() takes exactly 1 argument (0 given) | 12:38 |
*** yfried_ has quit IRC | 12:38 | |
*** tmorin has joined #openstack-neutron | 12:38 | |
garyk1 | i have no idea what has/may caused this | 12:39 |
*** singhj has joined #openstack-neutron | 12:39 | |
*** dougs has joined #openstack-neutron | 12:40 | |
*** salv-orlando has joined #openstack-neutron | 12:40 | |
garyk1 | has anyone else hit something like this? | 12:40 |
ihrachys | garyk1: it's neutron, not some dep bump? | 12:40 |
dougs | Anyone know if that rate at which neutron is dumping flows is configurable? | 12:40 |
dougs | neutron : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/usr/bin/neutron-rootwrap /etc/neutron/rootwrap.conf ovs-ofctl dump-flows br-int table=23 | 12:40 |
dougs | seems to be making calls at 10Hz | 12:41 |
dougs | #journalctl --since=today | grep neutron-rootwrap | wc -w | 12:41 |
dougs | 272842 | 12:41 |
garyk1 | ihrachys: the only dependedncies in the last 24 hrs that were updated were alembic and oslo.messagin | 12:41 |
*** doug-fish has quit IRC | 12:42 | |
*** ababich has quit IRC | 12:42 | |
*** doug-fish has joined #openstack-neutron | 12:42 | |
*** admin0 has quit IRC | 12:43 | |
garyk1 | ihrachys: i see the same issue in l2gw project - https://review.openstack.org/#/c/302613/2/networking_l2gw/tests/unit/services/l2gateway/service_drivers/test_rpc_l2gw.py | 12:43 |
*** nplanel has quit IRC | 12:43 | |
*** ababich has joined #openstack-neutron | 12:43 | |
ihrachys | dougs: it should be controlled by polling_interval option | 12:44 |
*** ramishra has quit IRC | 12:44 | |
*** nplanel has joined #openstack-neutron | 12:44 | |
*** ramishra has joined #openstack-neutron | 12:45 | |
dougs | ihrachys: got it, thanks! | 12:45 |
ihrachys | garyk1: I download the patch to play locally | 12:45 |
*** lucasagomes has quit IRC | 12:45 | |
garyk1 | ihrachys: thanks. | 12:45 |
garyk1 | so the l2gw folks have hit the same issue | 12:46 |
garyk1 | i am not sure what is going on here. | 12:46 |
ihrachys | yeah, we may better understand that instead of patching all subprojects | 12:46 |
garyk1 | i am leaning to what assaf wrote, lets drop all of the cores and tests…. could that be a april 7 joke | 12:46 |
*** lucasagomes has joined #openstack-neutron | 12:46 | |
*** doug-fish has quit IRC | 12:47 | |
*** achanda has joined #openstack-neutron | 12:47 | |
ajo | jlibosva, I added two nit tiny comments: https://review.openstack.org/#/c/302653/2 | 12:47 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Fix keepalived functional tests https://review.openstack.org/302421 | 12:48 |
amuller | ajo: ^ | 12:48 |
amuller | ajo: bleh, the gerrit diff is confusing but the change is really small | 12:49 |
ihrachys | garyk1: if I replace it with mock, it works | 12:49 |
ajo | amuller, ack checking | 12:49 |
ihrachys | garyk1: there was a fixtures release today | 12:49 |
ihrachys | 2.0.0 it is | 12:49 |
jlibosva | ajo: thanks :) | 12:49 |
ihrachys | I will push a patch in 5 mins | 12:49 |
amuller | ihrachys: oh my, neutron survived that? | 12:49 |
ihrachys | amuller: neutron? probably. not subprojects ;P | 12:50 |
ihrachys | amuller: or maybe neutron too, not sure | 12:50 |
ihrachys | I don't care much | 12:50 |
*** julim has joined #openstack-neutron | 12:51 | |
garyk1 | ihrachys: what to pin it? i think i have a fix | 12:51 |
garyk1 | def fake_consume_in_threads(*args, **kwargs): | 12:51 |
ihrachys | REALLY? | 12:51 |
ihrachys | wtf | 12:51 |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config https://review.openstack.org/302653 | 12:52 |
garyk1 | yes, that looks like it resolves it. | 12:52 |
jlibosva | ajo: updated commit message, fullstack TODO :) | 12:52 |
openstackgerrit | garyk proposed openstack/neutron: Fix broken unit tests https://review.openstack.org/302737 | 12:53 |
amuller | ihrachys: I've seen two fullstack failures recently, one was neutron-server could not bind to socket, the other was OVS agent could not bind | 12:53 |
*** kengo_sakai has quit IRC | 12:54 | |
garyk1 | ihrachys: ^^ | 12:54 |
*** doug-fish has joined #openstack-neutron | 12:54 | |
garyk1 | i need to maybe go and work with armax in the tomatoe field | 12:55 |
ihrachys | garyk1: I will scratch it gently | 12:55 |
*** doug-fis_ has joined #openstack-neutron | 12:56 | |
*** kengo_sakai_mba has joined #openstack-neutron | 12:56 | |
*** akshai has quit IRC | 12:56 | |
*** dedery has quit IRC | 12:57 | |
*** akshai has joined #openstack-neutron | 12:57 | |
*** achanda has quit IRC | 12:58 | |
*** doug-fish has quit IRC | 12:59 | |
*** jonlan has joined #openstack-neutron | 13:00 | |
ihrachys | garyk1: you know what's the REAL problem? | 13:00 |
ihrachys | garyk1: that subprojects don't use constraints :) | 13:00 |
amotoki | garyk1: I am not sure your test fix is the right direction. isn't it a bug in fixtures? | 13:00 |
*** kengo_sakai_mba has quit IRC | 13:00 | |
*** rodrigods has quit IRC | 13:01 | |
*** rodrigods has joined #openstack-neutron | 13:01 | |
garyk1 | ihrachys: sorry i do not follow | 13:02 |
ihrachys | amotoki: the direction is right, but let me fix the commit message and explain it | 13:02 |
ihrachys | garyk1: if subprojects would use requirement constraints, they would not be hit by the issue right now | 13:02 |
ihrachys | amuller: that's the reason neutron is not hit ^ | 13:02 |
*** rossella_s has quit IRC | 13:03 | |
ihrachys | amuller: so we saved another fire drill for later (at least for core repos) | 13:03 |
garyk1 | ihrachys: due to the fact that i was at my wits end with this, i just copied the neutron requirements file(s) | 13:03 |
garyk1 | that did not help. | 13:03 |
amotoki | ihrachys: do we need to use (*args, **kwargs) for fake method? | 13:03 |
*** rossella_s has joined #openstack-neutron | 13:03 | |
ihrachys | amotoki: yes. since now fixtures make the method static and don't pass self | 13:03 |
ihrachys | amotoki: new release | 13:03 |
ihrachys | garyk1: let me get back to you after I fix the patch | 13:04 |
garyk1 | ihrachys: ok, i am not sure that i know what is broken with the patch (other than the cryptic commit message ) | 13:05 |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups https://review.openstack.org/284738 | 13:06 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Fix broken unit tests https://review.openstack.org/302737 | 13:06 |
amotoki | ihrachys: thanks. this sounds tricky, but it is what's happening... | 13:07 |
ihrachys | amotoki: garyk1: ^ I updated with explanation and bug reference | 13:07 |
ihrachys | I guess we'll need to backport it till kilo.. | 13:07 |
ihrachys | especially for kilo. | 13:07 |
ihrachys | since it's not guarded by constraints | 13:07 |
*** achanda has joined #openstack-neutron | 13:08 | |
ihrachys | garyk1: so re constraints, I meant that subprojects should utilize: https://github.com/openstack/neutron/blob/master/tox.ini#L11 | 13:08 |
ihrachys | garyk1: that would guard your gate from catching 2.0.0 right after it's released to pypi | 13:08 |
garyk1 | ihrachys: thanks! | 13:08 |
ihrachys | the bug would still be there, but your gate would be ok | 13:08 |
*** wolverineav has joined #openstack-neutron | 13:10 | |
garyk1 | ihrachys: thanks! | 13:10 |
*** johnsom has quit IRC | 13:10 | |
*** eric_lopez has quit IRC | 13:10 | |
amotoki | I think i understand now. if we need to monkeypatch to instance method, we need to do it against not class but instance. | 13:11 |
*** eric_lopez has joined #openstack-neutron | 13:11 | |
*** johnsom has joined #openstack-neutron | 13:11 | |
*** mvk_ has quit IRC | 13:11 | |
*** anilvenkata has joined #openstack-neutron | 13:12 | |
*** achanda has quit IRC | 13:13 | |
*** mvk has joined #openstack-neutron | 13:13 | |
ihrachys | amotoki: yeah, probably. which is weird a bit. | 13:13 |
ihrachys | amotoki: we can always fall back to mock if anything | 13:14 |
ihrachys | amotoki: it's just that for now the garyk1's patch is ok | 13:14 |
*** javeriak has joined #openstack-neutron | 13:14 | |
*** akshai has quit IRC | 13:14 | |
Sam-I-Am | moo. | 13:14 |
ihrachys | Sam-I-Am: moo | 13:15 |
amotoki | ihrachys: understood and garyk1's patch looks good. | 13:15 |
ihrachys | Sam-I-Am: I suggest you review that docs patch for MTU | 13:15 |
*** wolverineav has quit IRC | 13:15 | |
Sam-I-Am | ihrachys: looking at it | 13:15 |
ihrachys | Sam-I-Am: I blocked it with WIP for now, since it already has one +2 | 13:15 |
ihrachys | I wanted you to chime in first | 13:15 |
Sam-I-Am | sure | 13:15 |
*** jonlan has quit IRC | 13:17 | |
ZZelle | garyk, needs some help on fixtures | 13:17 |
ZZelle | ? | 13:18 |
garyk1 | ZZelle: thanks. we are sorted. now just reviews :) | 13:19 |
garyk1 | ZZelle: https://review.openstack.org/302737 | 13:19 |
*** tiswanso has joined #openstack-neutron | 13:20 | |
*** ababich has quit IRC | 13:20 | |
*** ababich has joined #openstack-neutron | 13:21 | |
*** oanson has quit IRC | 13:22 | |
*** doug-fis_ has quit IRC | 13:23 | |
*** akshai has joined #openstack-neutron | 13:23 | |
*** ociuhandu has joined #openstack-neutron | 13:25 | |
openstackgerrit | Pavel Bondar proposed openstack/neutron: Always call ipam driver on subnet update https://review.openstack.org/272125 | 13:27 |
*** boden has joined #openstack-neutron | 13:28 | |
amuller | jlibosva: any idea of addOnException handler would be called before all fixtures are torn down or after? | 13:29 |
amuller | jlibosva: if it's after, it might not be very useful to find out which ports are used by which processes | 13:29 |
*** singhj has quit IRC | 13:29 | |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: Instantiate Enum() rather than subclassing https://review.openstack.org/302759 | 13:29 |
*** eddima1 has joined #openstack-neutron | 13:30 | |
*** eddima has quit IRC | 13:31 | |
*** eddima1 is now known as eddima | 13:31 | |
*** enriquetaso has joined #openstack-neutron | 13:31 | |
*** banix has joined #openstack-neutron | 13:34 | |
openstackgerrit | Victor Morales proposed openstack/neutron: Address Scope added to OVO https://review.openstack.org/297821 | 13:37 |
*** fzdarsky has quit IRC | 13:38 | |
*** bdemers has quit IRC | 13:40 | |
*** yamamoto has quit IRC | 13:40 | |
openstackgerrit | Felipe Reyes proposed openstack/neutron: Add configuration option to change keepalived globals https://review.openstack.org/302766 | 13:40 |
*** singhj has joined #openstack-neutron | 13:40 | |
*** jpena|lunch is now known as jpena | 13:41 | |
*** sc68cal has joined #openstack-neutron | 13:42 | |
*** irenab has quit IRC | 13:44 | |
*** mgoddard_ has joined #openstack-neutron | 13:44 | |
*** bdemers has joined #openstack-neutron | 13:45 | |
mhickey | ihrachys: Hey | 13:45 |
*** Shahid_ has quit IRC | 13:45 | |
scheuran | rkukura, ping | 13:46 |
*** kobis has quit IRC | 13:46 | |
*** links has quit IRC | 13:46 | |
*** garyk1 has quit IRC | 13:47 | |
*** mgoddard has quit IRC | 13:48 | |
*** bdemers_ has joined #openstack-neutron | 13:48 | |
*** janzian has joined #openstack-neutron | 13:48 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP: New engine facade from oslo_db https://review.openstack.org/300017 | 13:48 |
*** woodburn1 has joined #openstack-neutron | 13:49 | |
ihrachys | mhickey: hi | 13:49 |
*** bdemers has quit IRC | 13:49 | |
*** woodburn has quit IRC | 13:50 | |
mhickey | ihrachys: wrt comment in https://review.openstack.org/#/c/285397/5/neutron/db/extra_dhcp_opt/models.py | 13:50 |
ihrachys | mhickey: I thought we discussed it already? | 13:50 |
*** cshahani has joined #openstack-neutron | 13:51 | |
mhickey | ihrachys: yes, just found an issue with it | 13:51 |
ihrachys | mhickey: tell me | 13:51 |
*** sridharg has quit IRC | 13:52 | |
mhickey | ihrachys: if I roll back the changes and return the class ExtraDhcpOpt to neutron/db/extradhcpopt_db.py, I will end up with circular imports | 13:52 |
amuller | ihrachys: OK, it looks like the way we determine which ports are used is broken | 13:52 |
ihrachys | mhickey: now *that's* a reason | 13:52 |
*** jhershbe has quit IRC | 13:52 | |
ihrachys | amuller: in where? | 13:52 |
amuller | ihrachys: "_get_source_ports_from_ss_output" | 13:52 |
mhickey | ihrachys: I know! I thought you would like it! :-) | 13:53 |
amuller | ihrachys: it's missing rows | 13:53 |
ihrachys | mhickey: if that's the case, then let's indeed move forward with what we have. | 13:53 |
*** tonytan4ever has joined #openstack-neutron | 13:53 | |
mhickey | ihrachys: ok, let me try and get patch in in next while. | 13:53 |
ihrachys | amuller: regex consumes multiple lines? | 13:54 |
ihrachys | mhickey: I believe there were other comments to address right? | 13:54 |
amuller | ihrachys: I don't know yet what is the specific issue, but I'm trying it locally and I noticed it's returning some ports but not others | 13:54 |
*** singhj has quit IRC | 13:55 | |
*** ametts has joined #openstack-neutron | 13:56 | |
*** busterswt has joined #openstack-neutron | 13:57 | |
*** yamamoto has joined #openstack-neutron | 13:57 | |
*** admin0 has joined #openstack-neutron | 14:00 | |
openstackgerrit | QingchuanHao proposed openstack/neutron: FIP: Ensure fpr and rfp are configured correctly https://review.openstack.org/302340 | 14:00 |
amuller | ihrachys: btw nice fix to a functional tests issue https://review.openstack.org/#/c/302421/ | 14:01 |
*** yamahata has joined #openstack-neutron | 14:01 | |
* ihrachys looks | 14:01 | |
mhickey | ihrachys: sure; that was one beaking me and causing me some deep thinking! :-D | 14:02 |
*** yamamoto has quit IRC | 14:03 | |
*** mgoddard has joined #openstack-neutron | 14:04 | |
*** mgoddard_ has quit IRC | 14:04 | |
ihrachys | amuller: +W | 14:04 |
amuller | ihrachys: thanks, that's at least one cause for functional tests failures gone | 14:04 |
ihrachys | amuller: one spoon at a time | 14:04 |
ihrachys | amuller: how do we go with deadlocks? I guess you don't want to land revert? and we have no idea of the cause? | 14:05 |
*** eddima has quit IRC | 14:06 | |
*** eddima has joined #openstack-neutron | 14:06 | |
*** korzen has quit IRC | 14:06 | |
amuller | ihrachys: no idea of the cause. The issue with a revert is that this issue seems very rare and I can't reproduce locally. | 14:07 |
amuller | ihrachys: so I think the next step would be to add additional info / logging that would help us next time this happens at the gate | 14:07 |
ihrachys | amuller: ack. ok, in the worst scenario we could start disabling tests to see which one deadlocks us | 14:08 |
ihrachys | amuller: yeah, syslog since we loose logs in timeour? | 14:08 |
ihrachys | *timeout | 14:08 |
amuller | ihrachys: yeah, syslog is a possibility | 14:08 |
amuller | ihrachys: maybe also look in to why individual tests don't time out | 14:08 |
ihrachys | amuller: that's actually one thing we should look at independently - why the hell time limits don't work | 14:08 |
ihrachys | yea | 14:08 |
ihrachys | :) | 14:08 |
amuller | ihrachys: if we solve that we get our logs back | 14:09 |
amuller | ihrachys: maybe that should be priority 1 | 14:09 |
*** tbachman has quit IRC | 14:10 | |
*** knikolla has joined #openstack-neutron | 14:10 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Notify resource_versions from agents only when needed https://review.openstack.org/302792 | 14:10 |
*** neelashah has joined #openstack-neutron | 14:10 | |
ihrachys | ajo: ^ | 14:10 |
*** wolverineav has joined #openstack-neutron | 14:11 | |
*** neeti has quit IRC | 14:12 | |
*** sadasu has quit IRC | 14:15 | |
*** wolverineav has quit IRC | 14:16 | |
*** ushkalim has quit IRC | 14:16 | |
*** mkolesni_ has joined #openstack-neutron | 14:17 | |
*** ChrisBenson has joined #openstack-neutron | 14:17 | |
*** julim has quit IRC | 14:19 | |
*** singhj has joined #openstack-neutron | 14:20 | |
*** thorst_ has quit IRC | 14:21 | |
*** julim has joined #openstack-neutron | 14:22 | |
*** thorst_ has joined #openstack-neutron | 14:22 | |
*** baoli_ has quit IRC | 14:26 | |
*** thorst__ has joined #openstack-neutron | 14:26 | |
*** thorst_ has quit IRC | 14:27 | |
*** sarun has joined #openstack-neutron | 14:28 | |
*** cshahani has quit IRC | 14:28 | |
*** ushkalim has joined #openstack-neutron | 14:28 | |
*** asingh has joined #openstack-neutron | 14:28 | |
*** yamamoto has joined #openstack-neutron | 14:29 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods https://review.openstack.org/302276 | 14:29 |
*** mlavalle has joined #openstack-neutron | 14:29 | |
*** jlibosva has quit IRC | 14:30 | |
*** javeriak has quit IRC | 14:30 | |
*** baoli has joined #openstack-neutron | 14:30 | |
*** lajos-katona has quit IRC | 14:30 | |
*** matrohon has quit IRC | 14:31 | |
*** vhosakot has joined #openstack-neutron | 14:31 | |
ajo | ihrachys, ouch | 14:31 |
*** matrohon has joined #openstack-neutron | 14:31 | |
*** jlibosva has joined #openstack-neutron | 14:31 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: test_migrations: enforce tests for new alembic scripts https://review.openstack.org/302813 | 14:32 |
ajo | obondarev, thanks, looking at https://review.openstack.org/#/c/302792/ | 14:32 |
*** psanchez has quit IRC | 14:33 | |
ihrachys | HenryG: ^ nuffsaid | 14:33 |
*** javeriak has joined #openstack-neutron | 14:34 | |
*** tbachman has joined #openstack-neutron | 14:34 | |
*** jhershbe has joined #openstack-neutron | 14:36 | |
*** thorst__ has quit IRC | 14:37 | |
openstackgerrit | Merged openstack/neutron: Remove unused PolicyFileNotFound exception https://review.openstack.org/302497 | 14:37 |
obondarev | ajo: ihrachys: thanks | 14:38 |
ajo | obondarev, there's only one thing to take care of: https://github.com/openstack/neutron/blob/c007960ab8ebc56e5c3d0c8cad99c921615a1915/neutron/api/rpc/callbacks/version_manager.py#L143 | 14:38 |
openstackgerrit | Felipe Reyes proposed openstack/neutron: Add configuration option to change keepalived globals https://review.openstack.org/302766 | 14:38 |
ajo | we need to keep the compatibility to the old ovs/sriov agents which did not report versions in liberty, | 14:38 |
ajo | but that need a certain version of the QoSPolicy (v1.0) | 14:39 |
*** tbachman_ has joined #openstack-neutron | 14:39 | |
ajo | obondarev, I did not forsee that could be so slow... :? | 14:39 |
ajo | obondarev, may be we could get update_versions to return True/False, and in case of True we fanout to other servers? | 14:40 |
openstackgerrit | Merged openstack/neutron: DVR: Increase the link-local address pair range https://review.openstack.org/297839 | 14:40 |
*** tbachman has quit IRC | 14:40 | |
*** tbachman_ is now known as tbachman | 14:40 | |
obondarev | ajo: that might be due to debug logs, but still rpc to server is smth we want to reduce | 14:40 |
*** korzen has joined #openstack-neutron | 14:40 | |
*** thorst_ has joined #openstack-neutron | 14:41 | |
ajo | I understand. | 14:41 |
obondarev | ajo: this won't be needed if agent return resource_versions onle when needed I guess | 14:42 |
*** banix has quit IRC | 14:42 | |
ajo | obondarev, yes, but in the case of upgrade, | 14:42 |
*** rkukura has quit IRC | 14:42 | |
ajo | we will have old liberty agents never returning resource_versions | 14:42 |
*** vhosakot has quit IRC | 14:43 | |
ajo | we won't know if they are old, or ... if they are new but not sending resource_versions at that time... | 14:43 |
obondarev | ajo: I see, need to think about it.. | 14:43 |
*** vhosakot has joined #openstack-neutron | 14:43 | |
*** davidsha has quit IRC | 14:44 | |
saisriki | ihrachys: rosella_s: Quick question: Is anyone working on DNS, QOS objects for Ports? I see korzen working on Subnet objects - DNSNameServer and others in link: https://review.openstack.org/#/c/264273/ ?? | 14:44 |
ihrachys | ajo: kinda easy path but: how about just disabling the special casing since we never got dscp in mitaka? | 14:44 |
ajo | obondarev, it's optimizable probably in the way you did, with some tweaks, let me know if you need any help. | 14:44 |
ajo | hmmm | 14:44 |
*** kawa2014 has quit IRC | 14:44 | |
ajo | ihrachys, good point ^ obondarev | 14:44 |
ajo | yeah, and we're not supposed to allow two jump versions... | 14:45 |
*** numans has quit IRC | 14:45 | |
ajo | obondarev, there it is... | 14:45 |
ihrachys | saisriki: please take dns. qos - it may be a special case, I will need to take a look myself. | 14:45 |
obondarev | ajo: not sure I'm following.. | 14:45 |
saisriki | ihrachys: ok. Thanks. | 14:45 |
obondarev | ajo: so liberty agents will always need only 1.0 version | 14:45 |
ajo | obondarev, QoSPolicy version was not changed in mitaka :) | 14:46 |
obondarev | ajo: right | 14:46 |
ajo | obondarev, that means, that they started reporting their versions | 14:46 |
ajo | (at start) | 14:46 |
ajo | so from mitaka->newton is when that will be necessary | 14:46 |
ajo | and the workaround is not needed anymore then | 14:46 |
obondarev | ajo: perfect | 14:46 |
ajo | workaround = https://github.com/openstack/neutron/blob/c007960ab8ebc56e5c3d0c8cad99c921615a1915/neutron/api/rpc/callbacks/version_manager.py#L143 | 14:47 |
korzen | saisriki, DNSNameServer is covered by me | 14:47 |
*** numan_ has quit IRC | 14:47 | |
obondarev | ajo: can be cleaned up then? | 14:47 |
ajo | obondarev, ihrachys , yes... let me do it in a separate patch :) | 14:47 |
obondarev | ajo: great | 14:48 |
obondarev | ajo: thanks | 14:48 |
*** ametts has quit IRC | 14:48 | |
saisriki | korzen: ok, did you mean in https://review.openstack.org/#/c/264273/ patch? | 14:48 |
ihrachys | ajo: honestly, the whole code is not really needed in mitaka | 14:48 |
ihrachys | ajo: we could as well kill the updater thread | 14:48 |
ihrachys | :) | 14:48 |
korzen | saisriki, yes | 14:49 |
*** ametts has joined #openstack-neutron | 14:49 | |
*** tbachman has quit IRC | 14:49 | |
ajo | ihrachys, yes, but we can't kill it in newton.. :) | 14:49 |
saisriki | korzen: ok. Thanks for the clarification | 14:49 |
ajo | ihrachys, stable only patch? :) | 14:49 |
ihrachys | ajo: why not | 14:49 |
ihrachys | ajo: if it effectively reduces load | 14:49 |
ajo | ihrachys, do the honors :P | 14:49 |
*** javeriak_ has joined #openstack-neutron | 14:49 | |
ajo | ihrachys, or obondarev hmm | 14:49 |
*** nmagnezi has quit IRC | 14:49 | |
korzen | saisriki, the DNSNameServer was a dependency for Subent so I've took it with couple other obejcts | 14:50 |
ihrachys | ajo: nah, I will sit here watching for the fix emerging from under your fingers | 14:50 |
ajo | ihrachys, may be it's better not to have stable only patches if not absolutely necessary, later backports hurt when code has diverged | 14:50 |
ajo | or we can do it in a minimalistic way | 14:50 |
* ajo thinks | 14:50 | |
ihrachys | ajo: well if it saves performance... | 14:50 |
ihrachys | ajo: we can just tactically disable the updates in proper places | 14:50 |
*** javeriak has quit IRC | 14:50 | |
saisriki | korzen: ok are you finished with the DNSNameServer objects work? | 14:50 |
ihrachys | and leave the code unused | 14:51 |
ajo | ihrachys, sounds good | 14:51 |
ajo | so plan | 14:51 |
ajo | 1) obondarev optimizes newton (as of his current patch) | 14:51 |
ajo | 2) I kill the corner cases not needed anymore | 14:51 |
*** tbachman has joined #openstack-neutron | 14:51 | |
ajo | 3) I kill the update code in mitaka | 14:51 |
obondarev | ajo: ihrachys: I'd like to backport https://review.openstack.org/#/c/302792 to mitaka as well.. | 14:52 |
*** salv-orl_ has joined #openstack-neutron | 14:52 | |
ajo | obondarev, | 14:52 |
obondarev | does it fit? | 14:52 |
ajo | probably we should consider 3 as optional | 14:52 |
ajo | obondarev, how did your tests look after your patch? | 14:52 |
*** salv-orl_ has quit IRC | 14:53 | |
ihrachys | ajo: I love the 3 steps plan | 14:53 |
ajo | does it make a big difference if we disable the update mechanism completely? | 14:53 |
obondarev | ajo: I need to retest, I know it helper when I just commented self._update_local_agent_resource_versions(context, agent_state) | 14:53 |
ihrachys | ajo: we should only make sure that agents *sending* the resource versions are not breaking the new server that would ignore the data | 14:53 |
ajo | ihrachys, it's a dict | 14:54 |
ajo | so whatever not used is ignored | 14:54 |
*** salv-orlando has quit IRC | 14:54 | |
korzen | saisriki, yes, the DNSServer as object is finished, I did not look into using it outside of Seubnet OVO patch | 14:54 |
ihrachys | ajo: ok. but we should test the scenario with old agents and new server. | 14:54 |
*** thorst__ has joined #openstack-neutron | 14:54 | |
ihrachys | since the feature is not tested in gate, it's only manual | 14:55 |
*** mickeys has joined #openstack-neutron | 14:55 | |
ajo | ihrachys, makes sense, if the perf gain is worth it | 14:55 |
saisriki | korzen: ok, thanks. | 14:55 |
*** anilvenkata has quit IRC | 14:55 | |
*** ramishra has quit IRC | 14:55 | |
korzen | saisriki, ihrachys I need to remoev the WIP from my Subnet patch because I've seen no reviews ;) | 14:55 |
ihrachys | korzen: noted | 14:57 |
saisriki | korzen: there seems to be a Merge conflict as well? | 14:57 |
electrocucaracha | korzen: you're also rebase that change right? | 14:57 |
*** ametts has quit IRC | 14:57 | |
*** ramishra has joined #openstack-neutron | 14:57 | |
*** javeriak has joined #openstack-neutron | 14:57 | |
*** cristicalin has joined #openstack-neutron | 14:58 | |
*** thorst_ has quit IRC | 14:58 | |
korzen | electrocucaracha, saisriki, yes, I need to rebase it as well | 14:58 |
*** ametts has joined #openstack-neutron | 14:58 | |
*** cristicalin has quit IRC | 14:58 | |
*** regXboi has joined #openstack-neutron | 14:58 | |
*** cristicalin has joined #openstack-neutron | 14:59 | |
*** javeriak_ has quit IRC | 14:59 | |
electrocucaracha | korzen: what about the TODO section in the commit message? | 14:59 |
*** mgoddard_ has joined #openstack-neutron | 15:00 | |
ajo | obondarev, I have a meeting, and I will do a 2nd pass on your patch to make sure I didnt' miss anything | 15:00 |
*** johnbelamaric has joined #openstack-neutron | 15:00 | |
*** oshvartz has quit IRC | 15:00 | |
*** thorst__ is now known as thorst________ | 15:00 | |
*** abehl has quit IRC | 15:01 | |
obondarev | ajo: cool, thanks! | 15:01 |
*** iyamahat has joined #openstack-neutron | 15:01 | |
openstackgerrit | Brian Haley proposed openstack/neutron: Move all link-local cidr constants to a central location https://review.openstack.org/302829 | 15:01 |
*** tiswanso has quit IRC | 15:01 | |
*** kawa2014 has joined #openstack-neutron | 15:02 | |
*** tiswanso has joined #openstack-neutron | 15:03 | |
*** dougs1 has joined #openstack-neutron | 15:03 | |
*** mgoddard has quit IRC | 15:03 | |
korzen | electrocucaracha, THe TODOs are 1) description handled by jlibosva in other patch, and the second is remove the workaround for SQL type decorator... | 15:03 |
*** dougs has quit IRC | 15:03 | |
korzen | electrocucaracha, 1) is happening in parallel and 2) is in TBD in near future | 15:04 |
*** sarun has quit IRC | 15:05 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Notify resource_versions from agents only when needed https://review.openstack.org/302792 | 15:06 |
*** kawa2014 has quit IRC | 15:06 | |
*** ajmiller has joined #openstack-neutron | 15:07 | |
*** ababich has quit IRC | 15:07 | |
*** j_king_ is now known as j_king | 15:07 | |
*** ababich has joined #openstack-neutron | 15:08 | |
*** thorst________ is now known as thorst_ | 15:08 | |
*** jhershbe has quit IRC | 15:08 | |
*** ababich has quit IRC | 15:09 | |
*** ababich has joined #openstack-neutron | 15:09 | |
*** fitoduarte has quit IRC | 15:10 | |
*** jlibosva1 has joined #openstack-neutron | 15:10 | |
*** Guest74036 has quit IRC | 15:10 | |
*** stanzgy has joined #openstack-neutron | 15:10 | |
electrocucaracha | question, BGP stands for Border Gateway P? or Big Giant Pencil... | 15:10 |
*** sarun has joined #openstack-neutron | 15:11 | |
njohnston | Border Gateway Pencil | 15:11 |
*** mkolesni_ has quit IRC | 15:11 | |
*** yottatsa has joined #openstack-neutron | 15:11 | |
electrocucaracha | hahaha | 15:11 |
*** tbachman has quit IRC | 15:11 | |
*** crose has joined #openstack-neutron | 15:12 | |
*** rkukura has joined #openstack-neutron | 15:13 | |
*** jlibosva has quit IRC | 15:14 | |
matrohon | scheuran, hi | 15:14 |
*** javeriak has quit IRC | 15:15 | |
openstackgerrit | Richard Theis proposed openstack/python-neutronclient: Devref: Newton updates for transition to OSC https://review.openstack.org/299399 | 15:16 |
*** tbachman has joined #openstack-neutron | 15:17 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Enable sorting and pagination in -api gate https://review.openstack.org/301634 | 15:18 |
*** javeriak has joined #openstack-neutron | 15:18 | |
ihrachys | HenryG: amuller: ^ a new version of the first api tests for sorting/pagination. would love to hear what you think. | 15:19 |
*** baoli has quit IRC | 15:19 | |
*** kawa2014 has joined #openstack-neutron | 15:19 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: ovo: Make modify_fields_(from|to)_db instance methods https://review.openstack.org/302276 | 15:19 |
*** hongbin has joined #openstack-neutron | 15:20 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: ADDRESS_SCOPE_MARK_IDS should not be global for L3 agent https://review.openstack.org/301685 | 15:20 |
obondarev | carl_baldwin: haleyb: ^^ | 15:20 |
carl_baldwin | obondarev: Thanks! | 15:20 |
haleyb | obondarev: thanks | 15:20 |
hongbin | Hi all. I am from Magnum team. Our gate recently broke with this error: AttributeError: 'str' object has no attribute 'strftime' | 15:21 |
*** itzikb has quit IRC | 15:21 | |
hongbin | Here is the full log: http://logs.openstack.org/91/301891/1/check/gate-functional-dsvm-magnum-api/ea0d4ba/logs/screen-q-lbaas.txt.gz#_2016-04-07_14_18_11_567 | 15:21 |
hongbin | I wonder if there is a recent commit in neutron that causes this error? | 15:21 |
ramishra | hongbin, hi, it seems https://github.com/openstack/neutron/commit/34a328fe12950c339b8259451262470c627f2f00 has broken lbaas | 15:22 |
*** rickyrem has joined #openstack-neutron | 15:22 | |
ramishra | and hence heat gate is also broken | 15:22 |
hongbin | ramishra: ack. Thanks | 15:22 |
hongbin | I guess someone is working on a fix? | 15:23 |
openstackgerrit | Victor Morales proposed openstack/neutron: WIP - BGP to OVO https://review.openstack.org/302863 | 15:23 |
yottatsa | Just found that physical_network_mtus = physnet1:1500,physnet2:1500 will not be handled due to https://bugs.launchpad.net/neutron/+bug/1567502 | 15:24 |
openstack | Launchpad bug 1567502 in neutron "[ml2]/physical_network_mtus impoperly handled" [Undecided,New] | 15:24 |
*** achanda has joined #openstack-neutron | 15:24 | |
scheuran | matrohon, hi | 15:24 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/302865 | 15:24 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/301669 | 15:24 |
*** lnicolas has quit IRC | 15:24 | |
matrohon | scheuran, https://review.openstack.org/#/c/297100/3 | 15:24 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/301671 | 15:24 |
*** neelashah has quit IRC | 15:25 | |
matrohon | scheuran, great start! I gave my opinion inline | 15:25 |
*** alex_xu has quit IRC | 15:25 | |
scheuran | matrohon, thanks | 15:25 |
matrohon | scheuran, I though there was a related blueprint already | 15:25 |
scheuran | matrohon, regarding the failing, tests - I need to look into them | 15:25 |
scheuran | matrohon, on my devstack multinode migration was working | 15:26 |
*** minwang2 has joined #openstack-neutron | 15:26 | |
matrohon | scheuran, great! from ovs to linuxbridge? | 15:26 |
sc68cal | yottatsa: are you going to push a patch? | 15:26 |
*** davidsha has joined #openstack-neutron | 15:26 | |
garyk | ihrachys: amotoki: amuller: https://review.openstack.org/#/c/302737/ has the grumpy old mans +1. Can we please approve | 15:26 |
scheuran | matrohon, so I expect them to be test-failures | 15:26 |
*** yamahata has quit IRC | 15:26 | |
scheuran | matrohon, yes, ovs to lb, lb to ovs, ovs to macvtap | 15:26 |
ihrachys | garyk: +W | 15:26 |
scheuran | matrohon, I forgot to add the blueprint link...you're right | 15:27 |
scheuran | matrohon, blueprint + spec already exists.. | 15:28 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/302883 | 15:28 |
garyk | ihrachys: thanks! much appreciated. | 15:28 |
matrohon | scheuran, great, it will help reviewer to have the context of this patch | 15:28 |
*** Swami has joined #openstack-neutron | 15:28 | |
ramishra | ihrachys, ajo: can someone please look at https://bugs.launchpad.net/neutron/+bug/1567507 | 15:28 |
openstack | Launchpad bug 1567507 in neutron "neutron-lbaas broken with neutron change" [Undecided,New] | 15:28 |
*** alex_xu has joined #openstack-neutron | 15:29 | |
ihrachys | ramishra: ouch looking | 15:29 |
*** cristicalin has quit IRC | 15:29 | |
*** stanzgy has quit IRC | 15:29 | |
scheuran | matrohon, great, thanks! | 15:29 |
scheuran | matrohon, and you're right, multibound ports are only network ports today | 15:30 |
matrohon | scheuran, the code is pretty thin, hope it will be able to merge in newton | 15:30 |
*** armax has joined #openstack-neutron | 15:30 | |
scheuran | matrohon, yes it is, I still need to make some methods public that are currently private or so, but nothing big | 15:31 |
ramishra | ihrachys: does it make sense to revert the change or wait for a fix? | 15:31 |
scheuran | matrohon, the problem probably is what I described in my comment, regarding external drivers... | 15:31 |
scheuran | matrohon, so some research must be done.. | 15:31 |
ihrachys | ramishra: give me a minute to think on the stack | 15:32 |
scheuran | matrohon, The alternative is multiple bound ports | 15:32 |
scheuran | matrohon, https://review.openstack.org/#/c/189410/ | 15:32 |
*** cshahani has joined #openstack-neutron | 15:32 | |
*** garyk has quit IRC | 15:32 | |
scheuran | matrohon, this is the multiple bind patch | 15:32 |
ramishra | ihrachys: np:) | 15:33 |
scheuran | the idea is to remove the 2 portbinding tables that currently exist (dvr & normal) and merge them | 15:33 |
matrohon | scheuran, it's a very hold patch that didn't have the love it deserves | 15:33 |
scheuran | matrohon, if done, a new api could be established to allow multpile host binding for compute ports as well | 15:34 |
*** singhj has quit IRC | 15:34 | |
scheuran | matrohon, yes I know - I failed rebasing it :/ | 15:34 |
scheuran | matrohon, need to ask Bob to do so | 15:34 |
dougwig | at least with the new gerrit, i get to re-sign in about every 17 minutes. | 15:34 |
*** links has joined #openstack-neutron | 15:36 | |
*** sc68cal has quit IRC | 15:36 | |
matrohon | scheuran, you think it would improve your patch? It will only give two binding to nova, and nova will use the one for the new host | 15:36 |
*** iyamahat has quit IRC | 15:36 | |
matrohon | scheuran, no? | 15:36 |
*** EinstCrazy has quit IRC | 15:36 | |
*** tmorin has quit IRC | 15:37 | |
scheuran | matrohon, it wouldn't make a difference for the reference implementations | 15:37 |
*** achanda has quit IRC | 15:37 | |
scheuran | matrohon, but probably for some external drivers, that would shut down a port and rebuild it on the new host when the host_id is changed | 15:38 |
scheuran | matrohon, for those cases it would improve the patch, yes | 15:38 |
scheuran | matrohon, nova then could just request another binding in pre_live_migration | 15:38 |
vikram | armax: ping | 15:38 |
scheuran | matrohon, and remove the old binding in post_live_migration | 15:38 |
armax | vikram: pong | 15:38 |
matrohon | scheuran, yep makes sense | 15:38 |
ihrachys | ramishra: let's revert | 15:39 |
*** moshele has quit IRC | 15:39 | |
vikram | armax: need your approval for "https://review.openstack.org/#/c/268726/11" | 15:39 |
armax | I have it on my stack of things to review | 15:39 |
vikram | armax: thanks ;) | 15:40 |
matrohon | scheuran, but I don't think your patch should depend on it, external driver have time to accommodate with your change since we're really early in the release | 15:40 |
*** ababich has quit IRC | 15:40 | |
vikram | armax: One more "https://review.openstack.org/#/c/302120/" ;) | 15:40 |
*** ababich1 has joined #openstack-neutron | 15:40 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Revert "standard attributes: expose created_at/updated_at on models" https://review.openstack.org/302904 | 15:40 |
ihrachys | ramishra: ^ | 15:40 |
ihrachys | jlibosva1: ^ | 15:40 |
vikram | armax: I think we need to resume our decision about the Stadium project creation | 15:41 |
jlibosva1 | =-O | 15:41 |
scheuran | matrohon, yes, probably. I think we need to discuss this with a wider audience.. | 15:41 |
*** jlibosva1 is now known as jlibosva | 15:41 | |
scheuran | matrohon, I plan to do some of the discussions during the summit.. | 15:41 |
scheuran | matrohon, will you attend? | 15:41 |
armax | vikram: no need for me to review that one | 15:41 |
armax | vikram: as for the stadium, yes I know that we do, thanks for the kind reminder | 15:41 |
vikram | armax: ok | 15:41 |
matrohon | scheuran, great! | 15:41 |
openstackgerrit | Nate Johnston proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies https://review.openstack.org/217092 | 15:42 |
matrohon | scheuran, yes | 15:42 |
vikram | armax: thanks for your help | 15:42 |
matrohon | scheuran, I saw you submitted a design session about it | 15:42 |
electrocucaracha | ihrachys: what should be the topic for this one -> https://review.openstack.org/#/c/301634/ | 15:42 |
matrohon | scheuran, hope it will be accepted | 15:42 |
matrohon | scheuran, but it also needs to be blessed by nova guys | 15:43 |
matrohon | scheuran, is there any nova session about that? | 15:43 |
scheuran | matrohon, yes, I did - marked it as nova-neutron session | 15:43 |
scheuran | matrohon, I started attending the weekly nova live migration meetings :) | 15:43 |
matrohon | scheuran, great! | 15:43 |
scheuran | matrohon, so those guys are up to date... | 15:44 |
ihrachys | electrocucaracha: it is not strictly ovo, it's just related, so no idea and I don't care that much :) | 15:44 |
*** sarun has quit IRC | 15:44 | |
*** afranc has joined #openstack-neutron | 15:44 | |
scheuran | matrohon, I have to leave now, I'll keep you up to date - maybe I can start fixing the test cases tomorrow or so... | 15:45 |
*** javeriak has quit IRC | 15:45 | |
davidsha | ajo: ping | 15:45 |
ajo | davidsha, pong | 15:45 |
davidsha | ajo: good evening | 15:46 |
*** mickeys has quit IRC | 15:46 | |
electrocucaracha | ihrachys: yes, I know that it doesn't matter but just curiosity... | 15:46 |
davidsha | ajo: Did you get njohnstons email? | 15:46 |
ajo | davidsha, for the presentation?, yes :) | 15:46 |
*** eddima has quit IRC | 15:46 | |
ajo | davidsha, I just resent it to me to look at it tomorrow :) | 15:46 |
ajo | as I ran out of time today | 15:46 |
ajo | on the high level it was looking good, beutiful in fact | 15:47 |
ajo | beautiful | 15:47 |
njohnston | ajo: Thanks! | 15:47 |
*** scheuran has quit IRC | 15:47 | |
davidsha | ajo: Ok, no problem! | 15:47 |
ajo | good work, I should start doing the same for our other presentation :) | 15:47 |
ihrachys | ramishra: how do I reproduce locally? | 15:47 |
*** ababich1 has quit IRC | 15:48 | |
openstackgerrit | Merged openstack/neutron: Fix keepalived functional tests https://review.openstack.org/302421 | 15:48 |
*** links has quit IRC | 15:48 | |
*** ababich has joined #openstack-neutron | 15:48 | |
*** ramishra has quit IRC | 15:48 | |
*** baoli has joined #openstack-neutron | 15:49 | |
davidsha | njohnston: actually, I was wondering should you do slide 32/33 as I wasn't involved in this part? | 15:49 |
*** asingh has quit IRC | 15:49 | |
ihrachys | jlibosva: can you craft the partial revert? | 15:49 |
*** asingh has joined #openstack-neutron | 15:49 | |
*** matrohon has quit IRC | 15:49 | |
*** admin0 has quit IRC | 15:49 | |
ihrachys | jlibosva: I want to test something different in next 10 mins till a meeting | 15:49 |
jlibosva | ihrachys: I can try but not sure I'll make it. I have one patch in progress that I'd like to push | 15:50 |
*** ababich has quit IRC | 15:50 | |
*** ababich has joined #openstack-neutron | 15:50 | |
njohnston | davidsha: OK, so that's mostly the "what was omitted from the reference implementation" slide? I can definitely go over that bit, yes. | 15:51 |
davidsha | njohnston: yup that's the one, Thanks! | 15:52 |
ihrachys | jlibosva: I see. | 15:52 |
jlibosva | ihrachys: I'm on it | 15:53 |
openstackgerrit | Martin Hickey proposed openstack/neutron: Integrate the Extra Dhcp Opt VersionedObject in Neutron https://review.openstack.org/285397 | 15:53 |
ihrachys | thanks | 15:53 |
*** pcaruana has quit IRC | 15:53 | |
*** vthapar has quit IRC | 15:53 | |
ihrachys | jlibosva: something weird is going on there. I crafted a test that checks that values are always of datetime type. it passes. | 15:54 |
*** numan_ has joined #openstack-neutron | 15:54 | |
jlibosva | ihrachys: when reading from proxy? | 15:54 |
ihrachys | jlibosva: yeah | 15:54 |
jlibosva | ihrachys: even when it was obtained from db? | 15:54 |
jlibosva | ihrachys: even when it was obtained from db? | 15:55 |
ihrachys | jlibosva: it is retrieved from there in the test | 15:55 |
jlibosva | sorry | 15:55 |
ihrachys | I refetched | 15:55 |
ihrachys | jlibosva: and why is it just lbaas, not neutron gate? | 15:56 |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fix regexp for ss output https://review.openstack.org/302913 | 15:58 |
*** sridhar_ram has joined #openstack-neutron | 15:58 | |
*** ygbo has quit IRC | 15:59 | |
*** garyk has joined #openstack-neutron | 15:59 | |
*** minwang2 has quit IRC | 16:00 | |
*** nmagnezi has joined #openstack-neutron | 16:01 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fetch timestamps via standard attribute model https://review.openstack.org/302904 | 16:01 |
jlibosva | ihrachys: ^^ | 16:01 |
*** ametts has quit IRC | 16:01 | |
*** javeriak has joined #openstack-neutron | 16:02 | |
*** armax has quit IRC | 16:02 | |
*** armax has joined #openstack-neutron | 16:02 | |
jlibosva | ihrachys: do you have a link with the failure in lbaas? | 16:03 |
ihrachys | jlibosva: not but there is magnum gate broken too | 16:03 |
*** dougs has joined #openstack-neutron | 16:03 | |
ihrachys | jlibosva: see the openstack-dev | 16:03 |
*** dave-mccowan has joined #openstack-neutron | 16:03 | |
*** yottatsa has quit IRC | 16:04 | |
ihrachys | jlibosva: http://logs.openstack.org/91/301891/1/check/gate-functional-dsvm-magnum-api/ea0d4ba/logs/screen-q-lbaas.txt.gz | 16:04 |
jlibosva | ihrachys: thanks | 16:05 |
ihrachys | the exception in server log: http://logs.openstack.org/91/301891/1/check/gate-functional-dsvm-magnum-api/ea0d4ba/logs/screen-q-svc.txt.gz#_2016-04-07_14_18_12_495 | 16:05 |
*** dougs1 has quit IRC | 16:05 | |
*** julim has quit IRC | 16:05 | |
*** itamarl has quit IRC | 16:06 | |
*** yamamoto has quit IRC | 16:06 | |
*** mgoddard_ has quit IRC | 16:06 | |
*** marios_ has joined #openstack-neutron | 16:06 | |
*** yamahata has joined #openstack-neutron | 16:08 | |
*** akshai has quit IRC | 16:08 | |
*** mgoddard has joined #openstack-neutron | 16:08 | |
*** marios has quit IRC | 16:08 | |
*** gvrangan has joined #openstack-neutron | 16:09 | |
*** rickyrem has quit IRC | 16:09 | |
*** marios_ is now known as marios | 16:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/302865 | 16:10 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: SUPER WIP OVO port object https://review.openstack.org/253641 | 16:11 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Port allowed address pairs to OVO https://review.openstack.org/268274 | 16:12 |
*** wolverineav has joined #openstack-neutron | 16:13 | |
*** ekuris has quit IRC | 16:13 | |
*** ababich has quit IRC | 16:13 | |
*** rickyrem has joined #openstack-neutron | 16:14 | |
*** ababich has joined #openstack-neutron | 16:14 | |
*** wolverineav has quit IRC | 16:17 | |
*** ametts has joined #openstack-neutron | 16:17 | |
*** ramishra has joined #openstack-neutron | 16:17 | |
*** mhickey has quit IRC | 16:17 | |
*** nplanel has quit IRC | 16:18 | |
*** jlanoux_ has quit IRC | 16:19 | |
*** tbachman_ has joined #openstack-neutron | 16:23 | |
*** asingh has quit IRC | 16:24 | |
*** tbachman has quit IRC | 16:25 | |
*** tbachman_ is now known as tbachman | 16:25 | |
*** davidsha has quit IRC | 16:26 | |
*** ekuris has joined #openstack-neutron | 16:26 | |
*** shashank_hegde has joined #openstack-neutron | 16:27 | |
*** daneyon_ has quit IRC | 16:28 | |
*** daneyon has joined #openstack-neutron | 16:29 | |
*** neiljerram has quit IRC | 16:31 | |
*** john-davidge has joined #openstack-neutron | 16:31 | |
*** minwang2 has joined #openstack-neutron | 16:31 | |
*** sridhar_ram has quit IRC | 16:32 | |
*** moshele has joined #openstack-neutron | 16:34 | |
*** devvesa has quit IRC | 16:34 | |
*** Leo_ has joined #openstack-neutron | 16:35 | |
*** haukebruno has quit IRC | 16:35 | |
*** slaweq__ has joined #openstack-neutron | 16:36 | |
*** david-lyle has quit IRC | 16:36 | |
*** mfranc213 has joined #openstack-neutron | 16:36 | |
*** haplo37 has joined #openstack-neutron | 16:37 | |
*** slaweq__ is now known as slaweq_ | 16:37 | |
armax | ihrachys: | 16:38 |
armax | ping | 16:38 |
ihrachys | armax: pong | 16:38 |
armax | regarding https://review.openstack.org/#/c/302904/ | 16:38 |
ihrachys | armax: shoot | 16:38 |
armax | I am confused | 16:38 |
*** sarun has joined #openstack-neutron | 16:39 | |
*** aranjan has joined #openstack-neutron | 16:39 | |
armax | how is it that we haven’t caught this up before? | 16:39 |
armax | and how does it show up? | 16:39 |
ihrachys | armax: that's a question I also wonder. we have api tests for timestamps. | 16:39 |
ihrachys | armax: it seems it shows up in lbaas gates only (?) | 16:39 |
ihrachys | armax: http://logs.openstack.org/91/301891/1/check/gate-functional-dsvm-magnum-api/ea0d4ba/logs/screen-q-svc.txt.gz#_2016-04-07_14_18_12_495 | 16:39 |
*** ushkalim has quit IRC | 16:40 | |
armax | ic | 16:40 |
ihrachys | armax: it seems that when server fetches .created_at from the model itself (thru proxy), it gets string. | 16:40 |
armax | good thing is that this isn’t mitaka | 16:40 |
ihrachys | for some reason | 16:40 |
armax | phew | 16:40 |
ihrachys | but I can't repro it with a unit test. | 16:40 |
armax | I almost had a heath attach | 16:40 |
armax | attack | 16:40 |
ihrachys | armax: 8.0.0 tags are merged. | 16:40 |
*** neiljerram has joined #openstack-neutron | 16:40 | |
ihrachys | armax: so you can relax anyway :) | 16:40 |
*** dslevin1 has quit IRC | 16:41 | |
armax | well I figured we had merged them whilst being broken! | 16:41 |
ihrachys | armax: we released mitaka without https://review.openstack.org/#/c/302728/ though | 16:41 |
ihrachys | which sucks, but well... | 16:41 |
ihrachys | should have been a reelase blocker | 16:42 |
ihrachys | not sure why it was identified as Low priority | 16:42 |
*** ekuris has quit IRC | 16:42 | |
armax | blame HenryG | 16:42 |
*** shashank_hegde has quit IRC | 16:43 | |
*** javeriak_ has joined #openstack-neutron | 16:44 | |
ihrachys | me gets ready to blame! | 16:44 |
armax | ihrachys: though I am not quite sure what the failure is | 16:44 |
ihrachys | armax: in the patch I posted? | 16:44 |
*** johnbelamaric has left #openstack-neutron | 16:44 | |
armax | ihrachys: no, associated to 1548433 | 16:44 |
armax | bug 1548433 | 16:44 |
openstack | bug 1548433 in keystoneauth "neutron returns objects other than oslo_config.cfg.Opt instances from list_opts" [Undecided,Incomplete] https://launchpad.net/bugs/1548433 | 16:44 |
ihrachys | armax: right. the bug shows up as a broken config file (auth section) | 16:45 |
*** javeriak has quit IRC | 16:45 | |
ihrachys | neutron still works, but the config file is not ideal | 16:45 |
*** Leo_ has quit IRC | 16:45 | |
ihrachys | armax: example snippet from bad config file at: https://review.openstack.org/#/c/292640/ | 16:46 |
*** Leo_ has joined #openstack-neutron | 16:48 | |
*** kickinz1 is now known as kickinz1|eod | 16:48 | |
armax | ihrachys: ic | 16:48 |
*** Leom has joined #openstack-neutron | 16:49 | |
amuller | ZZelle: fullstack fix of the day: https://review.openstack.org/#/c/302913/ | 16:50 |
*** SteffanW has joined #openstack-neutron | 16:51 | |
armax | ihrachys: when do those warning get emitted precisely? | 16:51 |
amuller | hmm, check queue is very full today | 16:51 |
*** asingh has joined #openstack-neutron | 16:51 | |
armax | ihrachys: shouldn’t they show up in the conf files that we generate during the CI job runs? | 16:51 |
ihrachys | armax: I suspect they are there. let me check. | 16:52 |
armax | I couldn’t find any | 16:52 |
*** rcernin has quit IRC | 16:52 | |
armax | ihrachys: but I didn’t look hard enough | 16:52 |
*** mickeys has joined #openstack-neutron | 16:52 | |
*** Leo_ has quit IRC | 16:52 | |
ihrachys | armax: search for "Warning: Failed to format sample for auth_url" in http://logs.openstack.org/34/296534/1/check/gate-tempest-dsvm-neutron-full/b739e41/logs/etc/neutron/neutron.conf.txt.gz | 16:53 |
*** julim has joined #openstack-neutron | 16:53 | |
*** marios_ has joined #openstack-neutron | 16:53 | |
armax | ihrachys: how about that | 16:54 |
armax | ihrachys: it ain’t showing up here: http://logs.openstack.org/21/302421/3/check/gate-neutron-dsvm-api/8b44543/logs/etc/neutron/neutron.conf.txt.gz | 16:54 |
slaweq_ | ajo: ping | 16:54 |
armax | ihrachys: see? I wasn’t lying | 16:54 |
ihrachys | armax: it's master? | 16:54 |
*** marios has quit IRC | 16:54 | |
armax | ihrachys: yes | 16:54 |
ihrachys | armax: well master is fixed now | 16:54 |
*** marios_ is now known as marios | 16:55 | |
ihrachys | anyhow | 16:55 |
*** ZZelle_ has joined #openstack-neutron | 16:55 | |
armax | ihrachys: indeed | 16:55 |
* ihrachys tries to understand how lbaas is broken by that innocent change | 16:55 | |
armax | ihrachys: I thought it was a couple of days old | 16:55 |
armax | ihrachys: thanks | 16:55 |
ihrachys | armax: also in case you are not aware: https://review.openstack.org/#/c/302737/ | 16:56 |
ihrachys | armax: new fixtures broke us | 16:56 |
ihrachys | but it's under control now | 16:56 |
armax | ihrachys: I saw | 16:56 |
*** mickeys has quit IRC | 16:56 | |
ihrachys | ok | 16:56 |
armax | ihrachys: thanks | 16:56 |
armax | ihrachys: it’s been a great awaking this morning | 16:56 |
ihrachys | :D | 16:56 |
ihrachys | but see how constraints guarded us in this case! | 16:57 |
*** mickeys has joined #openstack-neutron | 16:57 | |
*** abregman has quit IRC | 16:57 | |
*** hdaniel has quit IRC | 16:58 | |
*** mohankumar__ has quit IRC | 16:59 | |
*** mohankumar__ has joined #openstack-neutron | 16:59 | |
*** s3wong has joined #openstack-neutron | 17:00 | |
*** admin0 has joined #openstack-neutron | 17:00 | |
*** oanson has joined #openstack-neutron | 17:01 | |
*** asingh has quit IRC | 17:01 | |
*** mickeys has quit IRC | 17:01 | |
*** jistr has quit IRC | 17:01 | |
*** asingh has joined #openstack-neutron | 17:01 | |
*** neelashah has joined #openstack-neutron | 17:02 | |
*** rossella_s has quit IRC | 17:03 | |
*** rossella_s has joined #openstack-neutron | 17:03 | |
*** jpena is now known as jpena|off | 17:03 | |
*** iyamahat has joined #openstack-neutron | 17:03 | |
garyk | ihrachys: you around? | 17:04 |
*** tmorin has joined #openstack-neutron | 17:05 | |
ihrachys | garyk: sure | 17:05 |
*** azbiswas has joined #openstack-neutron | 17:05 | |
*** ramishra has quit IRC | 17:05 | |
*** yamahata has quit IRC | 17:06 | |
*** david-lyle has joined #openstack-neutron | 17:06 | |
*** oshvartz has joined #openstack-neutron | 17:06 | |
*** oanson has quit IRC | 17:06 | |
*** ametts has quit IRC | 17:07 | |
*** ramu_ramamurthy has joined #openstack-neutron | 17:07 | |
*** yamamoto has joined #openstack-neutron | 17:07 | |
*** rmart04 has quit IRC | 17:07 | |
*** pcaruana has joined #openstack-neutron | 17:08 | |
ihrachys | armax: huh. upper-constraints just bumped fixtures: https://review.openstack.org/#/c/302539/2/upper-constraints.txt | 17:09 |
ihrachys | so now the gate is officially broken | 17:09 |
ihrachys | dims: ^ | 17:09 |
ihrachys | I started getting first -1s from Jenkins | 17:10 |
armax | lovely | 17:10 |
ihrachys | yeah | 17:10 |
*** david-lyle has quit IRC | 17:11 | |
*** bjornar has quit IRC | 17:11 | |
armax | ihrachys: looks like garyk’ change didn’t go deep enough? | 17:11 |
armax | ihrachys: http://logs.openstack.org/37/302737/2/gate/gate-neutron-python34/dd413b3/testr_results.html.gz | 17:11 |
amuller | Well you know what they say, it's not a Thursday without a gate breakage | 17:11 |
armax | amuller: or Friday, or Wednesday | 17:11 |
amuller | Same for all other days as well | 17:11 |
amuller | armax: heh yeah | 17:11 |
ihrachys | armax: pff | 17:12 |
ihrachys | armax: I guess I will just switch to mock | 17:12 |
ihrachys | it's so much more sane | 17:12 |
armax | ihrachys: we shall just switch jobs | 17:12 |
armax | ihrachys: I hear the tomato picking industry is booming | 17:12 |
ihrachys | armax: tomatoes? | 17:12 |
ihrachys | YES | 17:12 |
*** salv-orlando has joined #openstack-neutron | 17:12 | |
ihrachys | crude tomatoes | 17:13 |
armax | ihrachys: under the scorching sun | 17:13 |
*** wolverineav has joined #openstack-neutron | 17:13 | |
*** kevo has quit IRC | 17:13 | |
*** oanson has joined #openstack-neutron | 17:14 | |
ihrachys | still better | 17:14 |
garyk | armax: the python 3 tests passed and then failed. i have no idea why. | 17:14 |
ihrachys | garyk: magic | 17:14 |
ihrachys | I am crafting a switch to mock | 17:14 |
armax | garyk: it’s worth seeing whether it’s an order issue | 17:14 |
*** jlibosva has quit IRC | 17:15 | |
garyk | armax: yes, it could be. something is fishy | 17:15 |
*** salv-orlando has quit IRC | 17:15 | |
*** p01nt3r75 has quit IRC | 17:15 | |
*** yamamoto has quit IRC | 17:16 | |
*** baoli has quit IRC | 17:16 | |
*** baoli has joined #openstack-neutron | 17:17 | |
*** fitoduarte has joined #openstack-neutron | 17:17 | |
dims | ihrachys : y, those are bot proposed updates | 17:17 |
*** david-lyle has joined #openstack-neutron | 17:17 | |
garyk | one thing for sure is that the code in the neutron lib copying this stuff is also broken. | 17:17 |
garyk | i will address that too. | 17:17 |
garyk | is that project still alive or can we kill it :) | 17:17 |
ihrachys | dims: yeah, but aren't we supposed to at least give a sec waiting before landing | 17:17 |
ihrachys | dims: should we add neutron unit tests into constraints gate? | 17:18 |
ihrachys | armax: garyk: https://review.openstack.org/302997 mock version | 17:18 |
*** Marga_ has quit IRC | 17:19 | |
garyk | ok, thanks. | 17:19 |
ihrachys | garyk: huh, your fix was first time gated against OLD fixtures | 17:19 |
dims | ihrachys : explain please? bot proposes updates, review runs some ci jobs, person eyeballs the results and merges..that's the protocol with requirements bot | 17:19 |
ihrachys | garyk: now that constraints bumped, your gate run caught the new one | 17:19 |
ihrachys | and failed on it | 17:19 |
garyk | :) | 17:20 |
ihrachys | dims: ok, but it then shows there is gating lacking for those version bumps | 17:20 |
*** fitoduarte has quit IRC | 17:20 | |
dims | lifeless : fixtures 2.0.0 broke neutron jobs...fyi | 17:20 |
ihrachys | dims: if neutron unit tests would run in the gate, you would see the CI failing | 17:20 |
*** ametts has joined #openstack-neutron | 17:20 | |
*** Prem_ has joined #openstack-neutron | 17:21 | |
dims | ihrachys : i'll let you take that up with lifeless :) | 17:21 |
ihrachys | dims: that's known, the release is explicitly breaking api | 17:21 |
dims | ihrachys : then you can propose a blacklist or a cap in global-requirements | 17:22 |
dims | again, that's the work flow. u-c/bot does not catch everything | 17:22 |
ihrachys | dims: huh? you think it's the only way? aren't we supposed to expand constraints gate with more jobs to avoid breakages? | 17:22 |
*** baoli has quit IRC | 17:23 | |
ihrachys | I naively thought that we will pick up a representative list of jobs from other projects to gate against for constraints bumps. | 17:23 |
ihrachys | wasn't it the plan? | 17:23 |
*** david-lyle has quit IRC | 17:23 | |
dims | ihrachys : the test matrix needs work, but i don't know if unit tests would make the cut | 17:25 |
*** Marga_ has joined #openstack-neutron | 17:26 | |
*** lnicolas has joined #openstack-neutron | 17:29 | |
*** banix has joined #openstack-neutron | 17:30 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron: LinuxBridge agent's QoS driver bw limit for egress traffic https://review.openstack.org/300210 | 17:30 |
*** sambetts is now known as sambetts|afk | 17:30 | |
ihrachys | WAT? I rebased a patch thru gerrit, and it's now empty | 17:31 |
dims | ihrachys : looks like it's a bit early for lifeless. do you want to wait or drop an email to -dev? | 17:31 |
*** bharath has joined #openstack-neutron | 17:31 | |
ihrachys | dims: I am heading out in a sec. just need to stack gate fixes properly | 17:31 |
*** abhiraut has joined #openstack-neutron | 17:31 | |
dims | ack ihrachys | 17:31 |
*** tbh has joined #openstack-neutron | 17:32 | |
*** amuller is now known as amuller_afk | 17:32 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Tempest 11.0.0 is released https://review.openstack.org/303005 | 17:32 |
armax | ihrachys: so your change supersedes garyk's? | 17:33 |
bharath | ihrachys: while running tox over neutron-vpnaas , it failing with object Not Found while calling create_vpnservice. I observed its using neutron client which no longer supports vpn. How to run these tox api cases? | 17:33 |
*** rcernin has joined #openstack-neutron | 17:33 | |
ihrachys | armax: that's the plan | 17:33 |
*** moshele has quit IRC | 17:33 | |
*** abhiraut has quit IRC | 17:34 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Fetch timestamps via standard attribute model https://review.openstack.org/302904 | 17:35 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Switched from fixtures to mock to mock out starting RPC consumers https://review.openstack.org/302997 | 17:35 |
*** fitoduarte has joined #openstack-neutron | 17:36 | |
bharath | while running tox over neutron-vpnaas , it failing with object Not Found while calling create_vpnservice. I observed its using neutron client which no longer supports vpn. How to run these tox api cases? | 17:36 |
ihrachys | armax: gerrit screwed patch rebase somehow, so I needed to respin ^ :-| | 17:36 |
armax | ihrachys: are you sure that the switch to mock doesn’t have its own side effects? | 17:37 |
*** abhiraut has joined #openstack-neutron | 17:38 | |
ihrachys | armax: I tested it today, it seemed to work. you never know. I am ok if someone pursues fixtures version in parallel | 17:38 |
*** wolverineav has quit IRC | 17:38 | |
ihrachys | armax: sorry, I need to run home, it's late here. | 17:38 |
armax | ihrachys: it’s the ‘never know’ attitude that keeps us in this mess though :) | 17:38 |
armax | ihrachys: no worries, thanks | 17:38 |
armax | ZZelle: is there any chance you can take over ihrachys’ change? | 17:39 |
armax | ^ | 17:39 |
armax | ZZelle: https://review.openstack.org/#/c/302997/ | 17:39 |
*** mickeys has joined #openstack-neutron | 17:39 | |
ihrachys | two changes actually, one general gate breakage, another one for lbaas | 17:39 |
ZZelle_ | armax, looking | 17:39 |
ihrachys | lbaas https://review.openstack.org/#/c/302904 | 17:39 |
ihrachys | I stacked them | 17:39 |
*** shivrao has joined #openstack-neutron | 17:40 | |
armax | we should have kevinbenton review that one | 17:40 |
armax | https://review.openstack.org/#/c/302904/ I mean | 17:40 |
ihrachys | armax: yeah. maybe he even has an idea of the culprit. | 17:41 |
ihrachys | ok, now off | 17:41 |
*** ihrachys has quit IRC | 17:41 | |
*** lezbar has joined #openstack-neutron | 17:42 | |
*** lezbar__ has quit IRC | 17:42 | |
garyk | armax: if ihars passes then we need to abandon mine | 17:44 |
*** mvk has quit IRC | 17:45 | |
*** shashank_hegde has joined #openstack-neutron | 17:46 | |
*** mickeys has quit IRC | 17:46 | |
*** kevo has joined #openstack-neutron | 17:47 | |
*** mickeys has joined #openstack-neutron | 17:47 | |
*** hongbin has left #openstack-neutron | 17:48 | |
*** aranjan has quit IRC | 17:49 | |
*** Leom has quit IRC | 17:50 | |
*** davidsha has joined #openstack-neutron | 17:50 | |
*** SumitNaiksatam has joined #openstack-neutron | 17:51 | |
*** rmart04 has joined #openstack-neutron | 17:52 | |
*** tflynn has joined #openstack-neutron | 17:53 | |
*** mickeys has quit IRC | 17:54 | |
kevinbenton | armax: you want me to look at 302904? | 17:55 |
*** mickeys has joined #openstack-neutron | 17:55 | |
*** david-lyle has joined #openstack-neutron | 17:56 | |
*** david-lyle has quit IRC | 17:56 | |
bharath | while running tox over neutron-vpnaas , it failing with object Not Found while calling create_vpnservice. I observed its using neutron client which no longer supports vpn. How to run these tox api cases? | 17:57 |
*** tull has joined #openstack-neutron | 17:57 | |
*** david-lyle has joined #openstack-neutron | 17:58 | |
*** wolverineav has joined #openstack-neutron | 17:58 | |
*** tbachman has quit IRC | 17:58 | |
*** vikram has quit IRC | 18:00 | |
*** jprovazn has quit IRC | 18:00 | |
*** bjornar has joined #openstack-neutron | 18:01 | |
*** aranjan has joined #openstack-neutron | 18:01 | |
manjeets | is there any specific version requirement of setuptools when some body uses linux bridge ? | 18:03 |
*** julim has quit IRC | 18:04 | |
manjeets | I am trying that even though its getting installed but linuxbridge is not working properly | 18:04 |
manjeets | http://paste.openstack.org/show/493382/ | 18:04 |
*** ociuhandu has quit IRC | 18:04 | |
manjeets | also seeing this warning for lb container http://paste.openstack.org/show/493385/ | 18:05 |
*** wolverineav has quit IRC | 18:06 | |
*** wolverineav has joined #openstack-neutron | 18:07 | |
*** davidsha has left #openstack-neutron | 18:07 | |
*** haplo37 has quit IRC | 18:08 | |
*** jaimguer has joined #openstack-neutron | 18:08 | |
openstackgerrit | garyk proposed openstack/neutron: Fix broken unit tests https://review.openstack.org/302737 | 18:10 |
*** ankur-gupta-f has joined #openstack-neutron | 18:10 | |
*** mfranc21_ has joined #openstack-neutron | 18:12 | |
*** sridhar_ram has joined #openstack-neutron | 18:14 | |
*** mfranc213 has quit IRC | 18:15 | |
*** aranjan has quit IRC | 18:16 | |
*** aranjan has joined #openstack-neutron | 18:17 | |
bharath | neutron-vpnaas tox api are failing with tempest.lib.NotFound . Is it know issue? | 18:17 |
*** javeriak_ has quit IRC | 18:17 | |
*** efoley has joined #openstack-neutron | 18:17 | |
*** admin0 has quit IRC | 18:17 | |
*** amuller_afk is now known as amuller | 18:19 | |
openstackgerrit | Artur Korzeniewski proposed openstack/neutron: Introducing the Subnet OVO https://review.openstack.org/264273 | 18:20 |
*** ociuhandu has joined #openstack-neutron | 18:21 | |
*** Marga_ has quit IRC | 18:24 | |
*** oanson has quit IRC | 18:24 | |
*** korzen has quit IRC | 18:25 | |
*** doug-fish has joined #openstack-neutron | 18:26 | |
kevinbenton | armax: ah, it's because lbaas calls get_port, changes the relevant fields, and then sends the entire dict into an update_port call | 18:26 |
*** mohankumar__ has quit IRC | 18:26 | |
kevinbenton | armax: so the association proxy thinks updated_at and created_at are being set | 18:27 |
*** gvrangan has quit IRC | 18:28 | |
*** s3wong has quit IRC | 18:32 | |
*** salv-orlando has joined #openstack-neutron | 18:32 | |
*** yfried has joined #openstack-neutron | 18:34 | |
*** tonytan4ever has quit IRC | 18:34 | |
*** tonytan4ever has joined #openstack-neutron | 18:34 | |
*** salv-orlando has quit IRC | 18:38 | |
*** nihilifer has quit IRC | 18:39 | |
*** saggi has quit IRC | 18:40 | |
*** admin0 has joined #openstack-neutron | 18:40 | |
*** akshai has joined #openstack-neutron | 18:42 | |
*** sambetts|afk has quit IRC | 18:42 | |
*** abhiraut has quit IRC | 18:45 | |
*** Leo_ has joined #openstack-neutron | 18:45 | |
*** sambetts has joined #openstack-neutron | 18:46 | |
*** mickeys has quit IRC | 18:46 | |
*** abhiraut has joined #openstack-neutron | 18:46 | |
*** mickeys has joined #openstack-neutron | 18:47 | |
*** hynekm has quit IRC | 18:49 | |
*** gvrangan has joined #openstack-neutron | 18:49 | |
*** njohnston has quit IRC | 18:49 | |
*** SumitNaiksatam has left #openstack-neutron | 18:51 | |
*** mickeys has quit IRC | 18:51 | |
*** salv-orlando has joined #openstack-neutron | 18:52 | |
adreznec | mestery: Do you think you (or someone else from the neutron-release team) might have a chance to look at https://bugs.launchpad.net/networking-powervm/+bug/1563926 today? We'd like to push our subproject mitaka release to line up with neutron if possible | 18:55 |
openstack | Launchpad bug 1563926 in networking-powervm "Release networking-powervm 2.0.0 and create stable/mitaka branch" [High,Confirmed] - Assigned to Adam Reznechek (adreznec) | 18:55 |
*** neelashah has quit IRC | 18:56 | |
*** neelashah has joined #openstack-neutron | 18:56 | |
*** janzian has quit IRC | 18:59 | |
*** sambetts has quit IRC | 19:00 | |
*** kawa2014 has quit IRC | 19:00 | |
*** salv-orlando has quit IRC | 19:01 | |
*** emagana has joined #openstack-neutron | 19:02 | |
*** abhiraut has quit IRC | 19:02 | |
mestery | adreznec: I've assigned that to ihar, he'll get to it tomorrow I imagine | 19:03 |
adreznec | mestery: Sure, not a problem. Thanks! | 19:03 |
*** sambetts has joined #openstack-neutron | 19:03 | |
blogan | kevinbenton: does lbaas code need to change? | 19:04 |
kevinbenton | blogan: no | 19:05 |
kevinbenton | blogan: the timestamp code should be able to handle this | 19:05 |
*** admin0 has quit IRC | 19:06 | |
*** sarun has quit IRC | 19:06 | |
*** sarun has joined #openstack-neutron | 19:07 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Ignore timestamp fields in model update https://review.openstack.org/303042 | 19:10 |
*** rickyrem has quit IRC | 19:10 | |
kevinbenton | blogan: ^^ | 19:10 |
kevinbenton | armax: ^^~1 | 19:10 |
*** janzian has joined #openstack-neutron | 19:12 | |
blogan | kevinbenton: i can't seem to find where the failure was | 19:12 |
blogan | in the review armax linked | 19:12 |
*** david-lyle_ has joined #openstack-neutron | 19:13 | |
*** rickyrem has joined #openstack-neutron | 19:14 | |
*** salv-orlando has joined #openstack-neutron | 19:14 | |
*** Marga_ has joined #openstack-neutron | 19:14 | |
*** tbh has quit IRC | 19:14 | |
*** david-lyle has quit IRC | 19:15 | |
*** SteffanW has quit IRC | 19:17 | |
*** iyamahat has quit IRC | 19:18 | |
*** tmorin has quit IRC | 19:18 | |
*** hdaniel has joined #openstack-neutron | 19:18 | |
*** Sukhdev has joined #openstack-neutron | 19:18 | |
*** rickyrem has quit IRC | 19:18 | |
*** david-lyle_ has quit IRC | 19:19 | |
kevinbenton | blogan: i think it's sort of skirting around the issue | 19:21 |
*** shivrao has quit IRC | 19:24 | |
*** ihrachys has joined #openstack-neutron | 19:24 | |
*** tbachman has joined #openstack-neutron | 19:25 | |
ihrachys | armax: so we don't land fixtures patch? | 19:25 |
ihrachys | armax: if not, we need to rebase the one fixing lbaas gate. | 19:26 |
*** janzian has quit IRC | 19:26 | |
*** tmorin has joined #openstack-neutron | 19:26 | |
armax | ihrachys: if the g-r cap is in place | 19:26 |
armax | ihrachys: I think it’s probably best let this bake a bit more? | 19:27 |
armax | ihrachys: and figure out a way to continue to use fixtures? | 19:27 |
ihrachys | armax: well, I am ok if you think that's worth it. though using two libs for the same thing (mocking out pieces of libraries) makes us more fragile | 19:27 |
ihrachys | armax: let me untangle the lbaas patch | 19:28 |
kevinbenton | manjeets: what's the rest of that linux bridge traceback? | 19:28 |
kevinbenton | manjeets: that paste doesn't have the error | 19:29 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Fetch timestamps via standard attribute model https://review.openstack.org/302904 | 19:29 |
ihrachys | armax: untangled ^ | 19:29 |
ihrachys | kevinbenton: would really love you checking the bug and the patch ^ maybe you will know the culprit from the first spot | 19:29 |
*** david-lyle has joined #openstack-neutron | 19:29 | |
kevinbenton | ihrachys: https://review.openstack.org/303042 | 19:29 |
kevinbenton | ihrachys: culprit is that update_port is setting the timestamp fields to the strings passed into it | 19:29 |
*** zhhuabj has quit IRC | 19:30 | |
*** bharath has quit IRC | 19:30 | |
ihrachys | kevinbenton: oh nice. looking. | 19:30 |
kevinbenton | ajo: still around? | 19:30 |
*** david-lyle has quit IRC | 19:31 | |
blogan | kevinbenton: your patch won't ever allow the timestamps to be updated in code? | 19:31 |
*** rickyrem has joined #openstack-neutron | 19:31 | |
kevinbenton | blogan: not by passing in a dict to port | 19:31 |
ihrachys | kevinbenton: oh that's a great explanation of what we see | 19:31 |
ihrachys | blogan: you are not supposed to update it yourself | 19:31 |
*** david-lyle has joined #openstack-neutron | 19:31 | |
ihrachys | only thru events | 19:31 |
*** minwang2 has quit IRC | 19:31 | |
blogan | ah okay, just wondering if there was a scenario where that'd be needed | 19:32 |
ihrachys | kevinbenton: have we validated the fix by some 'do not merge' patch in e.g. magnum? | 19:32 |
*** clickboom has joined #openstack-neutron | 19:32 | |
*** mvk has joined #openstack-neutron | 19:32 | |
*** panda has quit IRC | 19:33 | |
*** shivrao has joined #openstack-neutron | 19:33 | |
*** julim has joined #openstack-neutron | 19:33 | |
kevinbenton | ihrachys: no, i wasn't sure where the failures where | 19:33 |
kevinbenton | ihrachys: lbaas was in the example | 19:33 |
ihrachys | kevinbenton: ok, lemme craft one | 19:33 |
kevinbenton | ihrachys: but i wasn't sure if that was current lbaas gate | 19:33 |
*** panda has joined #openstack-neutron | 19:33 | |
*** clickboom is now known as aabradsh | 19:35 | |
*** tonytan_brb has joined #openstack-neutron | 19:35 | |
*** aabradsh has quit IRC | 19:35 | |
*** mvk_ has joined #openstack-neutron | 19:35 | |
kevinbenton | blogan: yeah, updating with code will be difficult even without this. because the db event would override it | 19:35 |
ihrachys | kevinbenton: I pushed https://review.openstack.org/#/c/303046/ | 19:36 |
kevinbenton | blogan: if someone wants to update the timestamp in code, it will have to be with a manual SQL statement | 19:36 |
lifeless | ihrachys: dims: need to get unit tests voting on requirements changes to avoid this | 19:36 |
*** tonytan4ever has quit IRC | 19:36 | |
dims | lifeless : which projects? :) | 19:36 |
ihrachys | lifeless: fully agreed. neutron would be happy to be the one whose unit tests are to be executed ;) | 19:36 |
*** banix has quit IRC | 19:36 | |
*** janzian has joined #openstack-neutron | 19:37 | |
ihrachys | dims: some huge/core/representative ;) | 19:37 |
dims | ihrachys : why just neutron? :) | 19:37 |
ihrachys | dims: because we are cool! | 19:37 |
dims | LOL, yes you are | 19:37 |
*** david-lyle has quit IRC | 19:37 | |
ihrachys | dims: well, there may be an assumption that most breaking cases affect the whole type of a job. | 19:37 |
ihrachys | dims: like fixtures release broke neutron and nova unit tests | 19:38 |
ihrachys | or new hacking may break all pep8 jobs | 19:38 |
openstackgerrit | Merged openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/302883 | 19:38 |
ihrachys | dims: also, neutron was the first project to adopt constraints ;) | 19:38 |
*** mvk has quit IRC | 19:39 | |
lifeless | dims: the ones that don't want to suffer the slings and arrows of outrageous fortune | 19:39 |
*** tonytan_brb has quit IRC | 19:39 | |
dims | ihrachys : sold! | 19:39 |
*** baoli has joined #openstack-neutron | 19:39 | |
* ihrachys crafts a straw man for project-config | 19:39 | |
*** tonytan4ever has joined #openstack-neutron | 19:39 | |
*** tmorin has quit IRC | 19:40 | |
*** iyamahat has joined #openstack-neutron | 19:41 | |
*** minwang2 has joined #openstack-neutron | 19:42 | |
*** abhiraut has joined #openstack-neutron | 19:42 | |
*** zhhuabj has joined #openstack-neutron | 19:43 | |
*** clickboom has joined #openstack-neutron | 19:43 | |
*** abhiraut has quit IRC | 19:44 | |
*** mfranc21_ has quit IRC | 19:45 | |
*** bharath has joined #openstack-neutron | 19:48 | |
*** efoley has quit IRC | 19:50 | |
*** mfranc213 has joined #openstack-neutron | 19:52 | |
*** alex_xu has quit IRC | 19:54 | |
*** ivase has joined #openstack-neutron | 19:54 | |
*** alex_xu has joined #openstack-neutron | 19:55 | |
*** tonytan_brb has joined #openstack-neutron | 19:55 | |
*** vhosakot has quit IRC | 19:58 | |
ZZelle_ | ihrachys, hi | 19:58 |
ihrachys | ZZelle_: hi | 19:58 |
ZZelle_ | ihrachys, about https://review.openstack.org/#/c/302913/1/neutron/tests/common/net_helpers.py | 19:58 |
*** tmorin has joined #openstack-neutron | 19:58 | |
*** vhosakot has joined #openstack-neutron | 19:58 | |
*** john-davidge has quit IRC | 19:58 | |
*** tonytan4ever has quit IRC | 19:59 | |
ZZelle_ | ihrachys, why not using: r'^.*\s+\d+\s+.*:(?P<port>\d+)\s' | 19:59 |
*** yamahata has joined #openstack-neutron | 20:00 | |
*** hdaniel has quit IRC | 20:00 | |
ZZelle_ | we don't really care about peer information | 20:01 |
blogan | kevinbenton: thats what i figured, so the update timestamp being updated just relies on events | 20:01 |
ihrachys | ZZelle_: well yeah, it could also work. I don't mind. | 20:01 |
*** ametts has quit IRC | 20:01 | |
*** vhosakot has quit IRC | 20:01 | |
kevinbenton | WHO HAS BEEN SNEAKING REGEX INTO NEUTRON! | 20:02 |
kevinbenton | :) | 20:02 |
armax | regXboi: did I recall correctly or you own me an answer? | 20:02 |
*** tbachman_ has joined #openstack-neutron | 20:02 | |
*** tbachman has quit IRC | 20:02 | |
*** tbachman_ is now known as tbachman | 20:02 | |
* regXboi reads scrollback | 20:02 | |
regXboi | kevinbenton: you could accuse me... but I'm innocent on this one :) | 20:03 |
armax | regXboi: I mean from last week | 20:03 |
ihrachys | kevinbenton: relax it's just tests. those irrelevant things that make the whole project down | 20:03 |
*** baoli has quit IRC | 20:03 | |
*** baoli has joined #openstack-neutron | 20:04 | |
regXboi | armax: yes I owe you an answer, but I've not gotten an answer back | 20:04 |
*** tbachman has quit IRC | 20:04 | |
regXboi | armax: at this point, I'd say propose the patch to remove it and see who screams :) | 20:05 |
*** hdaniel has joined #openstack-neutron | 20:05 | |
armax | regXboi: ack | 20:06 |
manjeets | kevinbenton http://paste.openstack.org/show/493412/ | 20:06 |
manjeets | sorry my previous trace missed last few lines | 20:07 |
*** mkrcmari__ has joined #openstack-neutron | 20:07 | |
kevinbenton | manjeets: sounds like kolla may be missing a dependency | 20:07 |
kevinbenton | manjeets: ebtables | 20:07 |
*** jpena|off is now known as jpena | 20:07 | |
*** wolverin_ has joined #openstack-neutron | 20:07 | |
manjeets | ok is it gettable via pip | 20:08 |
*** jpena is now known as jpena|off | 20:08 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron: Change name of QoS bandwidth limit burst parameter to kilobits https://review.openstack.org/291633 | 20:09 |
kevinbenton | manjeets: it's not a python package | 20:09 |
*** dane_leblanc has quit IRC | 20:09 | |
*** jpena|off is now known as jpena | 20:09 | |
*** tmorin has quit IRC | 20:09 | |
kevinbenton | manjeets: its a distro package | 20:09 |
manjeets | kevinbenton thanks ill get that see if had same issue | 20:10 |
manjeets | thanks very much | 20:10 |
manjeets | kevinbenton | 20:10 |
*** dane_leblanc has joined #openstack-neutron | 20:10 | |
*** mvk_ has quit IRC | 20:10 | |
kevinbenton | manjeets: no prob. kolla manages including system dependencies doesn't it? | 20:10 |
kevinbenton | manjeets: if so, a bug needs to be filed because they forgot to include it | 20:11 |
manjeets | it did really well for ovs may be it missed something | 20:11 |
manjeets | will do that | 20:11 |
*** wolverineav has quit IRC | 20:11 | |
*** ramu_ramamurthy has quit IRC | 20:13 | |
*** efoley has joined #openstack-neutron | 20:13 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: WIP: Fix functional tests logging https://review.openstack.org/303053 | 20:13 |
amuller | HenryG: ^ | 20:13 |
HenryG | amuller: wat?! | 20:14 |
amuller | HenryG: :) | 20:14 |
amuller | HenryG: turns out there were 2 different patches that disabled logging for functional tests | 20:14 |
*** vishwanathj has quit IRC | 20:14 | |
amuller | HenryG: The power of collaboration! | 20:15 |
*** gvrangan has quit IRC | 20:15 | |
*** abhiraut has joined #openstack-neutron | 20:16 | |
kfox1111 | has anyone seen with dvr, packet loss between a non fip'ed vm and the snat namespace on the nn? odder still is there's no packet loss with pings. only other types of traffic. And its not 100% loss. more like 90%. some of the times, it retries enough to actually transmit data. | 20:16 |
blogan | kevinbenton: you wanna abandon this guy: https://review.openstack.org/#/c/292873 | 20:16 |
HenryG | amuller: do you have gerrit review links to the culprit patches? | 20:16 |
*** tbachman has joined #openstack-neutron | 20:16 | |
amuller | HenryG: they're in the bug report | 20:17 |
*** ametts has joined #openstack-neutron | 20:17 | |
kfox1111 | its not mtu related. I've tested that bit. huge pings work. | 20:17 |
* HenryG grumbles at extra steps to get from hashes to reviews | 20:17 | |
kevinbenton | blogan: done | 20:18 |
amuller | HenryG: I figured it's better to use commit hashes in commit messages because we might switch from Gerrit at some point in the future | 20:18 |
HenryG | amuller: never! | 20:18 |
amuller | HenryG: or maybe the URLs would change | 20:18 |
kevinbenton | HenryG: if you're using gerrit you just click on the hash | 20:19 |
blogan | does gertty do the same? | 20:19 |
amuller | kevinbenton: but I was cute and I used prefixes and not the full commit hash | 20:19 |
amuller | kevinbenton: maybe I should use the full hashes =D | 20:19 |
HenryG | kevinbenton: gerrit is not generating links for the short hashes in https://review.openstack.org/303053 | 20:20 |
*** azbiswas has quit IRC | 20:20 | |
kevinbenton | amuller: full hashes! | 20:20 |
kevinbenton | amuller: at one point we could have a collision on the prefix and then nobody could ever know which one was correct :) | 20:20 |
*** tmorin has joined #openstack-neutron | 20:20 | |
*** johnbelamaric has joined #openstack-neutron | 20:21 | |
ihrachys | lifeless: dims: armax: amuller: https://review.openstack.org/#/c/303054/ I *hope* it will work | 20:21 |
kevinbenton | error: short SHA1 2255 is ambiguous. | 20:21 |
kevinbenton | we've already had collisions on 4 chars :) | 20:21 |
kevinbenton | oooh, even 5 chars error: short SHA1 22554 is ambiguous. | 20:22 |
blogan | doesn't gerrit link across all the openstack projects for a hash too? | 20:23 |
*** Leo_ has quit IRC | 20:24 | |
*** Leom has joined #openstack-neutron | 20:24 | |
*** john-davidge has joined #openstack-neutron | 20:24 | |
amuller | kevinbenton: we could not be more off topic =p | 20:25 |
*** tonytan_brb is now known as tonytan4ever | 20:26 | |
*** tmorin has quit IRC | 20:26 | |
*** praveen has joined #openstack-neutron | 20:27 | |
*** numan_ has quit IRC | 20:28 | |
*** john-davidge has quit IRC | 20:29 | |
*** tongli has joined #openstack-neutron | 20:29 | |
*** vhosakot has joined #openstack-neutron | 20:31 | |
*** tbachman has quit IRC | 20:31 | |
*** tbachman has joined #openstack-neutron | 20:32 | |
*** enriquetaso has quit IRC | 20:32 | |
*** jpena is now known as jpena|off | 20:33 | |
*** mickeys has joined #openstack-neutron | 20:34 | |
*** Leom has quit IRC | 20:35 | |
*** mickeys has quit IRC | 20:36 | |
*** julim has quit IRC | 20:36 | |
*** mickeys has joined #openstack-neutron | 20:36 | |
*** bharath has quit IRC | 20:37 | |
*** Leom has joined #openstack-neutron | 20:38 | |
*** tmorin has joined #openstack-neutron | 20:38 | |
*** john-davidge has joined #openstack-neutron | 20:39 | |
*** kbringard has joined #openstack-neutron | 20:39 | |
*** ametts has quit IRC | 20:41 | |
*** baoli has quit IRC | 20:41 | |
*** hdaniel has quit IRC | 20:41 | |
*** jaimguer has quit IRC | 20:41 | |
*** jaimguer has joined #openstack-neutron | 20:42 | |
*** baoli has joined #openstack-neutron | 20:45 | |
*** baoli_ has joined #openstack-neutron | 20:46 | |
*** mvk_ has joined #openstack-neutron | 20:46 | |
*** wolverin_ has quit IRC | 20:47 | |
*** vishwana_ has joined #openstack-neutron | 20:47 | |
*** Sukhdev has quit IRC | 20:47 | |
*** gvrangan has joined #openstack-neutron | 20:47 | |
*** wolverineav has joined #openstack-neutron | 20:47 | |
lifeless | ihrachys: we're about to do a 2.1 - reducing the strictness; alaski is working on it | 20:48 |
ihrachys | lifeless: I am scared enough already :P | 20:49 |
nmagnezi | following to armax's mail about the drivers meeting, is it canceled for today or is it happening anyway? | 20:49 |
ihrachys | nmagnezi: cancelled today. | 20:49 |
ihrachys | so go have some sleep | 20:50 |
*** mkrcmari__ has quit IRC | 20:50 | |
*** baoli has quit IRC | 20:50 | |
nmagnezi | ihrachys, :) | 20:50 |
nmagnezi | ihrachys, it should do the same :) | 20:50 |
nmagnezi | you* | 20:50 |
*** mkrcmari__ has joined #openstack-neutron | 20:50 | |
nmagnezi | (passed my working hours..) | 20:50 |
*** baoli_ has quit IRC | 20:50 | |
*** baoli has joined #openstack-neutron | 20:51 | |
*** ramu_ramamurthy has joined #openstack-neutron | 20:52 | |
*** mvk_ has quit IRC | 20:54 | |
*** nihilifer has joined #openstack-neutron | 20:54 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Fix Windows IPDevice.device_has_ip racefulness https://review.openstack.org/303071 | 20:55 |
*** mvk_ has joined #openstack-neutron | 20:55 | |
amuller | ihrachys: armax: more functional tests fixes ^ | 20:56 |
*** david-lyle has joined #openstack-neutron | 20:56 | |
amuller | this time for windows, kill me | 20:56 |
*** admin0 has joined #openstack-neutron | 20:56 | |
kevinbenton | amuller: that seems extreme | 20:56 |
*** abregman has joined #openstack-neutron | 20:57 | |
*** marios has quit IRC | 20:57 | |
kevinbenton | pavel_bondar, carl_baldwin: i'm thinking we need to put a semaphore on create_port scoped to the network ID and backport that to mitaka/liberty to deal with the IP allocation deadlock | 20:59 |
*** mkrcmari__ has quit IRC | 20:59 | |
kevinbenton | even if we change the current allocation behavior to a random choice, i don't think we would back-port that | 20:59 |
kevinbenton | While it's a semaphore so it's only scoped to the server, it changes the probability significantly by making it so only 1 server at a time is doing allocations instead of every worker | 21:01 |
ihrachys | amuller: you casted Berserk magic, right? | 21:01 |
kevinbenton | sorry, i mean so only 1 allocation per server at a time | 21:01 |
*** pcaruana has quit IRC | 21:01 | |
amuller | ihrachys: is that a magic the gathering reference? | 21:01 |
*** davideagnello has quit IRC | 21:02 | |
*** davideagnello has joined #openstack-neutron | 21:02 | |
*** rossella_s has quit IRC | 21:03 | |
*** lucasagomes has quit IRC | 21:03 | |
ihrachys | amuller: http://heroes.thelazy.net/wiki/Berserk | 21:03 |
*** rossella_s has joined #openstack-neutron | 21:03 | |
ihrachys | "It makes the target of the spell to attack the nearest troop wheter it is an ally or enemy" | 21:03 |
amuller | ihrachys: omg heroes 3 | 21:04 |
amuller | ihrachys: wow this brings back memories | 21:04 |
*** fitoduarte has quit IRC | 21:04 | |
ihrachys | amuller: it's avail from steam btw | 21:04 |
amuller | ihrachys: for linux? | 21:04 |
ihrachys | amuller: also ios if you are that kind of person | 21:04 |
ihrachys | amuller: I think win32 only | 21:05 |
ihrachys | http://store.steampowered.com/app/297000/ | 21:05 |
amuller | ihrachys: my windows gaming pc is on another continent | 21:05 |
ihrachys | it's offtopic day ;) | 21:05 |
carl_baldwin | kevinbenton: Are you seeing the deadlock besides when pluggable IPAM is turned on in the gate? | 21:06 |
*** clickboo_ has joined #openstack-neutron | 21:06 | |
kevinbenton | carl_baldwin: not in gate | 21:07 |
kevinbenton | carl_baldwin: just for pluggable ipam and multi-writer galera | 21:07 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Avoid multiple gw ports on the same router WIP https://review.openstack.org/274021 | 21:09 |
ZZelle_ | salv-orlando, hi | 21:09 |
carl_baldwin | kevinbenton: Just saying that where I've seen the deadlock is in trying to gate the switch to pluggable IPAM in the gate. | 21:09 |
*** clickboom has quit IRC | 21:10 | |
*** tbachman has quit IRC | 21:10 | |
kevinbenton | carl_baldwin: right, i think that's the only place we can get it to hit in the gate | 21:10 |
carl_baldwin | kevinbenton: Is anyone actually asking for a backported fix? (i.e. is anyone using the ref implementation of pluggable IPAM yet? | 21:10 |
carl_baldwin | kevinbenton: I thought it was seen without galera. | 21:10 |
kevinbenton | carl_baldwin: there are two scenarios | 21:11 |
kevinbenton | carl_baldwin: one is pluggable ipam | 21:11 |
ZZelle_ | amuller, "my windows gaming pc", if you want minesweeper is also available on Linux :) | 21:11 |
kevinbenton | carl_baldwin: one it multi-writer galera with either ipam system | 21:11 |
*** lucasagomes has joined #openstack-neutron | 21:11 | |
carl_baldwin | kevinbenton: Ok, I see. | 21:11 |
*** nyechiel_ has quit IRC | 21:11 | |
*** thorst_ has quit IRC | 21:12 | |
carl_baldwin | kevinbenton: Was that reported as a different issue or wrapped in with bug 1543094 ? | 21:12 |
openstack | bug 1543094 in neutron "[Pluggable IPAM] DB exceeded retry limit (RetryRequest) on create_router call" [High,In progress] https://launchpad.net/bugs/1543094 - Assigned to Ryan Tidwell (ryan-tidwell) | 21:12 |
kevinbenton | carl_baldwin: for example, fuel right now just deploys galera with only one active writer to avoid these kind of issues | 21:12 |
kevinbenton | i don't think it was reported | 21:13 |
carl_baldwin | I just hadn't noticed it reported specifically using multi-writer. | 21:13 |
carl_baldwin | Oh, okay. | 21:13 |
*** Devon has joined #openstack-neutron | 21:14 | |
*** Devon has left #openstack-neutron | 21:14 | |
kevinbenton | carl_baldwin: i don't think many people deploy multi-writer right now, so it might not be worth fixing | 21:14 |
*** clickboo_ has quit IRC | 21:15 | |
tidwellr | carl_baldwin kevinbenton: I agree we wouldn't backport the randomized IP selection, I think the idea of a semaphore inside create_port is worth exploring | 21:15 |
kevinbenton | carl_baldwin, tidwellr: i did verify that it will at least unblock us in the gate as well | 21:16 |
kevinbenton | i'll push up the patch to let people discuss there | 21:16 |
carl_baldwin | kevinbenton: How many Neutron servers have you seen deployed (not workers)? I've seen 2, maybe 3. So, a server local semaphore could have an effect. Since we don't have reports of this, it is hard to know how many concurrent port creates it takes to get deadlock. | 21:16 |
salv-orlando | hi ZZelle_ | 21:16 |
*** njohnston has joined #openstack-neutron | 21:17 | |
carl_baldwin | kevinbenton: Ya, in the gate, there is only one server so that makes sense. | 21:17 |
ZZelle_ | salv-orlando, if you have time could you provide your feedback on https://review.openstack.org/274021 ? | 21:17 |
*** efoley_ has joined #openstack-neutron | 21:17 | |
*** tongli has quit IRC | 21:17 | |
tidwellr | carl_baldwin kevinbenton: there also seems to be an assumption (right or wrong) that neutron does sequential allocation of IP's, and that assumption codified in many many tests and in user's minds | 21:17 |
*** shakamunyi has joined #openstack-neutron | 21:17 | |
salv-orlando | ZZelle_: sure sir | 21:17 |
ZZelle_ | salv-orlando, thx | 21:18 |
carl_baldwin | kevinbenton: So, if it unblocks the move to pluggable IPAM in the gate, that could make it worth it. | 21:18 |
*** thorst_ has joined #openstack-neutron | 21:18 | |
kevinbenton | tidwellr: yes, people do like their numerology | 21:18 |
*** efoley has quit IRC | 21:18 | |
*** ivase has quit IRC | 21:18 | |
*** salv-orl_ has joined #openstack-neutron | 21:18 | |
tidwellr | kevinbenton: the sheer number of tests I've been adjusting is a little unnerving | 21:18 |
*** abhiraut has quit IRC | 21:19 | |
ihrachys | kevinbenton: btw any comments on https://review.openstack.org/#/c/302089/ ? does it make sense? | 21:20 |
*** njohnsto_ has joined #openstack-neutron | 21:20 | |
amuller | ZZelle_: :) | 21:21 |
*** thorst__ has joined #openstack-neutron | 21:21 | |
*** abhiraut has joined #openstack-neutron | 21:22 | |
*** thorst_ has quit IRC | 21:22 | |
*** salv-orlando has quit IRC | 21:22 | |
*** shakamunyi has quit IRC | 21:24 | |
*** Sukhdev has joined #openstack-neutron | 21:25 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Add semaphore to ML2 create_port db operation https://review.openstack.org/303085 | 21:25 |
*** DevonBoatwright has joined #openstack-neutron | 21:25 | |
*** thorst__ has quit IRC | 21:25 | |
*** DevonBoatwright has left #openstack-neutron | 21:26 | |
*** tflynn has quit IRC | 21:26 | |
tidwellr | kevinbenton: so, is your intent with https://review.openstack.org/303085 to provide a permanent solution or to just unlock us in the gate so we can move forward with pluggable IPAM as the default? | 21:27 |
kevinbenton | tidwellr: just to unlock us | 21:28 |
nmagnezi | kevinbenton, hey, when I asked dougwig about this patch he asked me to show you as well: https://review.openstack.org/#/c/299998/ | 21:28 |
kevinbenton | tidwellr: until we can figure out how to do ipam in a less racey way | 21:28 |
tidwellr | kevinbenton: ack | 21:28 |
kevinbenton | ihrachys: just left a comment | 21:28 |
kevinbenton | ihrachys: that config value does serve a slightly separate purpose | 21:28 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR: Check for snat_namespace existence before add or delete https://review.openstack.org/300358 | 21:28 |
*** tflynn has joined #openstack-neutron | 21:28 | |
armax | ihrachys: we might have another one | 21:29 |
armax | http://logs.openstack.org/73/297673/1/gate/gate-neutron-dsvm-api/4a178ab/console.html#_2016-04-07_20_10_03_307 | 21:29 |
armax | please shoot me now | 21:29 |
kevinbenton | ihrachys: check out neutron/plugins/ml2/drivers/type_tunnel.py get_mtu | 21:29 |
*** tmorin1 has joined #openstack-neutron | 21:29 | |
* ihrachys laughs | 21:29 | |
* dougwig points a carrot at armax and says, "bang!" | 21:29 | |
* armax lies flat | 21:29 | |
dougwig | openstack has claimed another innocent life. | 21:30 |
kevinbenton | armax: it looks like that config option doesn't exist | 21:30 |
armax | we try too hard | 21:30 |
kevinbenton | armax: am i being helpful? | 21:30 |
kevinbenton | nmagnezi: looking | 21:30 |
ihrachys | kevinbenton: there, super() returns segment_mtu already | 21:30 |
armax | kevinbenton: very | 21:30 |
*** rtheis has quit IRC | 21:31 | |
nmagnezi | kevinbenton, thanks | 21:31 |
armax | I am this close from pulling the rest of hair out | 21:31 |
*** njohnston has quit IRC | 21:31 | |
kevinbenton | ihrachys: yes, but path_mtu is taken into account | 21:31 |
armax | *my | 21:31 |
kevinbenton | ihrachys: path_mtu can be set to something different | 21:31 |
ihrachys | kevinbenton: yeah, but it will have the same value, ain't it? | 21:31 |
ihrachys | kevinbenton: why would it be different? | 21:31 |
kevinbenton | ihrachys: e.g. if your tunnel networks are going across interfaces that don't have jumbo frames | 21:31 |
*** njohnston has joined #openstack-neutron | 21:31 | |
ihrachys | kevinbenton: at least from reading option help texts, it does seem the same thing | 21:32 |
*** johnbelamaric has quit IRC | 21:32 | |
*** tmorin has quit IRC | 21:32 | |
armax | ihrachys: sh!iesse | 21:32 |
ihrachys | kevinbenton: you don't control your local routing for tunnelled traffic, so the only safe bet is to set it to the least MTU among all interfaces participating in routing, ain't it? | 21:33 |
armax | let’s stop merging | 21:33 |
armax | we’re broken in stable/mitaka too | 21:33 |
ihrachys | sh*t I pushed some there lately | 21:33 |
armax | yes, you did | 21:34 |
amuller | armax: kevinbenton: https://review.openstack.org/#/c/301167/9/tempest/config.py | 21:34 |
ihrachys | thought we were safe there. | 21:34 |
*** iyamahat has quit IRC | 21:34 | |
kevinbenton | ihrachys: no, if you have physical interfaces that are dedicated to jumbo frame stuff just for flat networks and no IP addresses are assigned, the kernel will not use them for routing | 21:34 |
armax | amuller: looks we ain't | 21:34 |
kevinbenton | ihrachys: so it's wonky, but you can set global_physnet_mtu to e.g. 8000 | 21:35 |
kevinbenton | ihrachys: then path_mtu to 1500 | 21:35 |
amuller | armax: kevinbenton: Although Doug set the deprecated_name option, doesn't oslo_config register both the new and deprecated names? | 21:35 |
ihrachys | kevinbenton: ok, I haven't thought we consider that kind of special cases for support. but ok. | 21:35 |
kevinbenton | ihrachys: and your vxlan networks will get 1500 (minus overhead) | 21:35 |
armax | not quite sure what’s happening yet | 21:35 |
kevinbenton | ihrachys: yes, the reason for all of these values is to support these mixed MTU cases. we don't want to regress on our crusade to fix MTU | 21:36 |
amuller | armax: kevinbenton: I'll send a patch to rename tenant to project, that will probably fix it | 21:36 |
kevinbenton | they do all serve a different purpose | 21:36 |
armax | amuller: I wonder if we should cherry pick this | 21:36 |
kevinbenton | we just need to emphasize that for the majority of cases, global_physnet_mtu will be enough | 21:36 |
ihrachys | armax: I will bump all mitaka patches out of gate with rebas4 | 21:36 |
*** tiswanso has quit IRC | 21:37 | |
armax | ihrachys: ack | 21:37 |
armax | amuller: https://review.openstack.org/#/c/300648/ | 21:37 |
kevinbenton | amuller: no | 21:37 |
nmagnezi | kevinbenton, thank you for the comment :) I will take that into account. | 21:37 |
kevinbenton | amuller: well sort of | 21:37 |
armax | this one was not easy to cherry pick and I didn’t bother | 21:37 |
amuller | armax: Although, we use Tempest from pypi, not source, so something must be configuring tempest.conf differently | 21:37 |
*** lucasagomes has quit IRC | 21:37 | |
tonytan4ever | rossella_s: ihrachys: Hi I am a developer from Rackspace, and I would like to participate in OVO initiative. | 21:37 |
armax | amuller: not in stable/mitaka | 21:37 |
*** baoli has quit IRC | 21:37 | |
*** sridhar_ram has quit IRC | 21:37 | |
kevinbenton | amuller: it does set it up, but it doesn't work with the old name for things like set_override and whatnot used in tests | 21:37 |
amuller | armax: ahh so only stable/mitaka is broken by this? not master? | 21:38 |
armax | amuller: that’s negligence on my part | 21:38 |
kevinbenton | amuller: so if you are moving an option, all of the code needs to be updated to work with the new one | 21:38 |
*** baoli has joined #openstack-neutron | 21:38 | |
tonytan4ever | when you guys have a chance I would like to talk to you. | 21:38 |
armax | I should have anticipated the clusterf!ck | 21:38 |
armax | besides it makes sense to keep all branches inline | 21:38 |
*** crose has quit IRC | 21:38 | |
kevinbenton | tonytan4ever: what about? | 21:38 |
armax | amuller: leave it with me, you focus on the functional/fullstack | 21:38 |
tonytan4ever | About what kind of "objects" I can start to contribute to. | 21:39 |
tonytan4ever | kevinbenton: ^^, with regard to OVO bp | 21:39 |
armax | amuller: and no, I don’t think we’re broken in master as far as the API is conerned | 21:39 |
armax | job I mean | 21:39 |
kevinbenton | tonytan4ever: ah, you'll have to sync up with ihrachys on that one i think | 21:39 |
amuller | armax: so we need to fix master and backport | 21:39 |
tonytan4ever | kevinbenton: cool | 21:39 |
kevinbenton | tonytan4ever: i haven't been following the OVO stuff very closely | 21:39 |
armax | amuller: there’s nothing to fix in master | 21:40 |
armax | we have to backport my fix from Friday | 21:40 |
armax | last Friday | 21:40 |
amuller | armax: oh you want stable branches on tempest from pypi as well | 21:40 |
armax | amuller: ya | 21:40 |
armax | unless I am talking nosnense? | 21:40 |
tonytan4ever | ihrachys: When you get a chance. | 21:40 |
ihrachys | tonytan4ever: howdy. what's your timezone? | 21:41 |
tonytan4ever | US EST | 21:41 |
amuller | armax: It makes sense for the branches to use Tempest from the same source, that's for sure, but I don't know if we want to use Tempest from pypi or source. If the goal is to use a tempest plugin, we will be using tempest from source anyway | 21:41 |
amuller | armax: worth asking for mtreinish's opinion here | 21:41 |
*** njohnston has quit IRC | 21:41 | |
*** asingh has quit IRC | 21:41 | |
armax | amuller: well, now I would be with you if you gave me the sacrosanct patch, now would I not? | 21:41 |
tonytan4ever | howdy ihrachys | 21:41 |
*** iyamahat has joined #openstack-neutron | 21:41 | |
ihrachys | tonytan4ever: you see, it's quite late here right now. how about syncing tomorrow your morning? just ping me in the channel, and I will try to accommodate for you. | 21:42 |
*** asingh has joined #openstack-neutron | 21:42 | |
amuller | armax: no idea what you're talking about =p | 21:42 |
armax | amuller: I do, and I should probably strip you from all your badges as retaliation | 21:42 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Add semaphore to ML2 create_port db operation https://review.openstack.org/303085 | 21:43 |
armax | how about that? :) | 21:43 |
amuller | armax: still no idea what you're on about | 21:43 |
ihrachys | kevinbenton: I also have docs update for MTU stuff at: https://review.openstack.org/#/c/302165/ | 21:43 |
*** tmorin1 has quit IRC | 21:43 | |
armax | amuller: https://bugs.launchpad.net/neutron/+bug/1555356 | 21:43 |
openstack | Launchpad bug 1555356 in neutron "Neutron too coupled with Tempest" [High,In progress] - Assigned to Assaf Muller (amuller) | 21:43 |
ihrachys | kevinbenton: I guess I will need to expand it with path_mtu if we see the option staying | 21:43 |
armax | amuller: does that ring a bell now? | 21:43 |
kevinbenton | ihrachys: ok. cool. i'll try to take a look | 21:43 |
amuller | armax: what does that have to do with this new failure? | 21:44 |
*** thorst_ has joined #openstack-neutron | 21:44 | |
armax | amuller: forget it | 21:44 |
amuller | armax: we would have been broken if we were still forked | 21:44 |
*** lucasagomes has joined #openstack-neutron | 21:45 | |
amuller | armax: If you seriously think we should not have stopped the fork, revert my patch. Otherwise, let it go. I absolutely think it was the right thing to do. We traded one set of gate failures for another, and it's clear which ones I prefer. | 21:45 |
armax | amuller: I am not complaining about decisions made in the past | 21:47 |
armax | amuller: those are hurting but not as much as lack of progress to resolve the current situation | 21:48 |
*** thorst_ has quit IRC | 21:48 | |
amuller | armax: any project with out of tree tests would be broken by something like this, provided they use CONF options defined in Tempest, regardless of usage of a tempest plugin | 21:49 |
amuller | armax: If you think that's my fault I have news for you =p | 21:49 |
openstackgerrit | Martin Roy proposed openstack/neutron: Typo on side_effect rendering the test useless https://review.openstack.org/303089 | 21:49 |
armax | amuller: I am not interested in the blame game | 21:49 |
armax | I have never have and never will | 21:50 |
armax | I just look forward to get sh!t done | 21:50 |
*** lucasagomes has quit IRC | 21:50 | |
armax | and when that ain’t happen I get frustrated | 21:50 |
amuller | armax: I'll start ignoring you when I see you're frustrated then | 21:51 |
amuller | meanwhile I'll send a patch to fix this new thing | 21:51 |
armax | amuller: great response! | 21:51 |
*** jaimguer has quit IRC | 21:52 | |
*** jaimguer has joined #openstack-neutron | 21:52 | |
*** Sukhdev has quit IRC | 21:52 | |
*** anshul has joined #openstack-neutron | 21:53 | |
*** anshul has quit IRC | 21:54 | |
*** lucasagomes has joined #openstack-neutron | 21:56 | |
ihrachys | kevinbenton: meh. magnum test patch failed in -api, but that's due to the tempest issue with the option we just hit | 21:57 |
*** regXboi has quit IRC | 21:58 | |
*** admin0 has quit IRC | 21:59 | |
*** enriquetaso has joined #openstack-neutron | 22:01 | |
*** johnbelamaric has joined #openstack-neutron | 22:02 | |
*** nmagnezi has quit IRC | 22:02 | |
*** ihrachys has quit IRC | 22:05 | |
*** efoley_ has quit IRC | 22:06 | |
*** dave-mccowan has quit IRC | 22:06 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Fix broken Tempest conf options in API tests https://review.openstack.org/303096 | 22:08 |
*** jonlan has joined #openstack-neutron | 22:08 | |
amuller | armax: reproduced locally by installing tempest from git, removing it from the .tox venv and enabling sitepackages in tox.ini | 22:08 |
amuller | armax: that patch is confirmed to fix the issue, and even passes pep8 :) | 22:09 |
armax | amuller: ack | 22:09 |
armax | I am considering putting the api job to be non-voting until we get ourselves to a better place | 22:09 |
*** neiljerram has quit IRC | 22:10 | |
amuller | armax: the only thing we could have done to prevent something like this is to not use CONF options defined in Tempest | 22:10 |
*** abhiraut has quit IRC | 22:11 | |
amuller | armax: we could create our own CONF options and use our gate hook to configure tempest.conf with our new keys | 22:11 |
armax | ack | 22:12 |
amuller | armax: that'd make it harder to run the tests locally and by other downstream consumers | 22:12 |
mtreinish | oh, the config option rename broke things for you | 22:12 |
openstackgerrit | Martin Roy proposed openstack/neutron: Typo on side_effect rendering the test useless https://review.openstack.org/303089 | 22:13 |
mtreinish | we tried to maintain bacwards compat by deprecating the old opt name. Didn't think about external code consumers | 22:13 |
*** NightKhaos has quit IRC | 22:13 | |
*** s3wong has joined #openstack-neutron | 22:13 | |
armax | amuller, mtreinish: https://review.openstack.org/#/c/303100/ | 22:13 |
*** asingh has quit IRC | 22:14 | |
*** ntpttr_ has joined #openstack-neutron | 22:14 | |
*** busterswt has quit IRC | 22:15 | |
amuller | armax: I'm not sure about that | 22:16 |
armax | amuller: I am | 22:16 |
armax | amuller: we need to make changes to the gate hooks anyway and we can’t be 100% sure we got them right | 22:17 |
*** jonlan has quit IRC | 22:17 | |
armax | amuller: thus we can’t keep the job voting whilst we move things aroud | 22:17 |
armax | around | 22:17 |
*** saisrikiran has joined #openstack-neutron | 22:17 | |
*** avozza has joined #openstack-neutron | 22:18 | |
*** shakamunyi has joined #openstack-neutron | 22:18 | |
*** dave-mccowan has joined #openstack-neutron | 22:18 | |
amuller | armax: mtreinish: I don't think moving to a tempest plugin will magically solve all of our issues. Today's issue would not have been prevented, for example. | 22:19 |
amuller | armax: mtreinish: So if you want to disable the api job, you better have a plan | 22:19 |
*** knikolla has quit IRC | 22:19 | |
armax | amuller: if we used the plugin interface would we still have that tempest code in tree? the one you just modified? | 22:20 |
amuller | armax: yes | 22:20 |
amuller | armax: the tests remain in our tree | 22:20 |
armax | and if so, then that should move under tempest/lib? | 22:20 |
*** neelashah has quit IRC | 22:21 | |
armax | amuller: right, but not https://review.openstack.org/#/c/303096/1/neutron/tests/api/base.py | 22:21 |
*** Sukhdev_ has joined #openstack-neutron | 22:21 | |
*** winggundamth has quit IRC | 22:21 | |
amuller | armax: I'm pretty sure that class would remain in Neutron as well, but not sure right now. Regardless we have tests using Tempest conf options. | 22:22 |
*** NightKhaos has joined #openstack-neutron | 22:22 | |
*** doug-fish has quit IRC | 22:23 | |
*** shakamunyi has quit IRC | 22:24 | |
armax | amuller: I think the long term solution is using both the plugin interface and the pypi released version of Tempest | 22:24 |
amuller | armax: That was my line of thinking as well, I don't think any other project does that on the other hand | 22:25 |
amuller | armax: Which would make us special, and we could be affected by issues with Devstack patches like yesterday | 22:26 |
*** boden has quit IRC | 22:26 | |
amuller | armax: projects that use the tempest plugin interface currently use tempest from git | 22:26 |
*** Sukhdev_ has quit IRC | 22:26 | |
*** vhosakot has quit IRC | 22:26 | |
*** hdaniel has joined #openstack-neutron | 22:26 | |
armax | amuller: I can’t see the point in having to use a stable interface when pulling from source though | 22:26 |
*** johnsom has quit IRC | 22:26 | |
armax | amuller: I must be missing something | 22:27 |
armax | but I always got the impression that the move to pypi for tempest was done to avoid pulling from source? | 22:27 |
mtreinish | armax, amuller: you can run with any version of tempest you want. Most projects using the plugin interface mirror the normal dsvm workflow and use tempest from git | 22:28 |
mtreinish | but there is nothing binding you to that | 22:28 |
amuller | mtreinish: Are you recommending projects to move to grab Tempest from pypi? | 22:28 |
mtreinish | well except for the lack of gate job to set it up for running from release | 22:28 |
mtreinish | amuller: tbh, I hadn't thought about it before | 22:28 |
amuller | mtreinish: That would have prevented today's neutron issue but not yesterday's | 22:29 |
amuller | mtreinish: So TBH I'm not sure what is better for us | 22:29 |
armax | mtreinish: I was always under the impression that pulling from source was frowned upon | 22:29 |
*** abhiraut has joined #openstack-neutron | 22:29 | |
*** rcernin has quit IRC | 22:29 | |
openstackgerrit | Brian Haley proposed openstack/neutron: DVR: Ensure fpr and rfp devices are configured correctly https://review.openstack.org/302340 | 22:30 |
*** winggundamth has joined #openstack-neutron | 22:30 | |
armax | devstack is kind of a special case in that it itself pulls all openstack deps from source | 22:30 |
armax | and that might need more thinking | 22:30 |
mtreinish | armax: well it is for libs and stuff, but also tempest wasn't published on pypi until like a month ago | 22:31 |
armax | mtreinish: right, I meant the services | 22:31 |
*** rmart04 has quit IRC | 22:32 | |
*** hdaniel has quit IRC | 22:33 | |
*** avozza has quit IRC | 22:35 | |
mtreinish | armax: well part of it is that we're testing that everything works together. Also at least for the base IaaS layer we're supposed to be supporting a CD use case, so treat every commit like a release :) | 22:35 |
*** ntpttr_ has quit IRC | 22:35 | |
*** saisrikiran has quit IRC | 22:35 | |
*** tmorin has joined #openstack-neutron | 22:36 | |
*** tpsilva has quit IRC | 22:37 | |
amuller | armax: mtreinish: dmellado has a patch for a Neutron Tempest plugin: https://review.openstack.org/#/c/274023/ | 22:39 |
amuller | currently in merge conflict, but latest patchset passed | 22:39 |
amuller | however, the plugin is essentially unused. The neutron api job still uses tox -e api from the neutron dir. | 22:39 |
armax | amuller: nice | 22:39 |
armax | amuller: right, that’s the part on project-config that we’d need to tweak | 22:39 |
amuller | We want the Neutron API job set up like other projects that use the plugin interface | 22:40 |
*** ntpttr_ has joined #openstack-neutron | 22:40 | |
amuller | armax: looking at that patch as it is, it doesn't buy us any stability... Am I missing something mtreinish? | 22:40 |
amuller | it's still a good thing, it just doesn't prevent any of the issues we've been having | 22:40 |
mtreinish | amuller: we din't think about the code consumers when we changed those config options | 22:40 |
amuller | mtreinish: That problem is in the past :) | 22:41 |
mtreinish | amuller: we've said that config should be stable, but I think we only really considered the config file consumers | 22:41 |
amuller | mtreinish: armax and I are looking ahead and asking if the plugin interface will actually help us | 22:41 |
mtreinish | amuller: but that's the break you're worried about right? | 22:41 |
*** tonytan4ever has quit IRC | 22:41 | |
*** minwang2 has left #openstack-neutron | 22:41 | |
amuller | mtreinish: that too | 22:42 |
*** saisrikiran has joined #openstack-neutron | 22:43 | |
mtreinish | amuller: but, yes the plugin interface doesn't really provide extra stability from the code consumption standpoint. It just provides hooks for registering external test code and integrating it into the tempest suite | 22:44 |
armax | amuller: so let’s continue chipping away at this | 22:44 |
amuller | armax: ^ That's what I've been saying for the past month... | 22:44 |
armax | amuller: and I understood | 22:44 |
armax | amuller: no need to rub it in | 22:44 |
amuller | armax: But it feels good :( | 22:45 |
amuller | armax: today's sucked, let me have that | 22:45 |
armax | amuller: why? I never contradicted you! | 22:45 |
*** abregman has quit IRC | 22:45 | |
armax | amuller: in the meantime I am gonna flip the stable branches to use the released version of tempest, you onboard with that? | 22:46 |
amuller | armax: yes | 22:46 |
armax | amuller: ack | 22:46 |
amuller | armax: unit tests still broken I understand? | 22:47 |
armax | amuller: ya | 22:47 |
amuller | armax: ok, if both unit and api tests are broken we have to disable one or the other | 22:47 |
armax | mtreinish, amuller to me the extra stability comes from the a versioned plugin interface | 22:47 |
amuller | armax: the fix for the api job can't go in cause of the unit tests failure... | 22:48 |
mtreinish | armax: for the stable branch this is just for the api job right? | 22:48 |
armax | amuller: the unit test fix is coming from g-r cap I think | 22:48 |
armax | mtreinish: ack | 22:48 |
mtreinish | ok | 22:48 |
armax | mtreinish: yes | 22:48 |
armax | amuller: so once that’s sorted yours should go in no probs | 22:48 |
amuller | armax: yeah I see https://review.openstack.org/#/c/303034/ | 22:48 |
amuller | armax: ok I'll recheck it later | 22:49 |
amuller | armax: that patch failed also =D | 22:49 |
amuller | armax: a perfect storm of failures | 22:49 |
armax | it did? | 22:49 |
armax | I did suggest to ihar switching to tomato picking farm | 22:49 |
armax | you in? | 22:49 |
armax | I am starting Monday morning | 22:50 |
armax | 5am | 22:50 |
amuller | armax: I always wanted to learn carpentry | 22:50 |
amuller | armax: the g-r patch failed due to a glance test | 22:50 |
armax | amuller: that doesn’t suit you | 22:50 |
amuller | ... in the postres job | 22:50 |
*** abhiraut has quit IRC | 22:51 | |
*** baoli has quit IRC | 22:52 | |
*** abhiraut has joined #openstack-neutron | 22:52 | |
amuller | armax: alright I think I'm going offline to go cry in a corner or something. I'll recheck my shit later. | 22:53 |
armax | amuller: no need | 22:53 |
armax | amuller: cheer up, we got this handled | 22:53 |
armax | amuller: at least we know the root causes | 22:53 |
armax | amuller: imagine if we hadn’t | 22:53 |
amuller | armax: Then I would have had a longer work day =D | 22:54 |
*** rickyrem has quit IRC | 22:54 | |
amuller | armax: https://review.openstack.org/#/c/303054/ is very interesting btw | 22:54 |
*** p01nt3r75 has joined #openstack-neutron | 22:54 | |
amuller | toodles | 22:55 |
*** amuller has quit IRC | 22:55 | |
*** sdague has quit IRC | 22:57 | |
*** shivrao has quit IRC | 22:57 | |
*** Leom has quit IRC | 22:58 | |
*** Leom has joined #openstack-neutron | 22:58 | |
*** tflynn_ has joined #openstack-neutron | 22:59 | |
*** jaimguer has quit IRC | 22:59 | |
*** tflynn__ has joined #openstack-neutron | 23:00 | |
*** tflynn_ has quit IRC | 23:00 | |
*** tflynn__ has quit IRC | 23:00 | |
*** shivrao has joined #openstack-neutron | 23:01 | |
*** tmorin has quit IRC | 23:01 | |
*** tflynn has quit IRC | 23:02 | |
*** Leom has quit IRC | 23:03 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Tempest 11.0.0 is released https://review.openstack.org/303005 | 23:05 |
*** johnsom has joined #openstack-neutron | 23:06 | |
*** shivrao has quit IRC | 23:07 | |
*** tflynn has joined #openstack-neutron | 23:08 | |
*** mickeys has quit IRC | 23:09 | |
*** mickeys has joined #openstack-neutron | 23:09 | |
*** djan has joined #openstack-neutron | 23:09 | |
*** akshai has quit IRC | 23:09 | |
*** tflynn has quit IRC | 23:10 | |
*** banix has joined #openstack-neutron | 23:11 | |
*** tflynn has joined #openstack-neutron | 23:11 | |
*** tbachman has joined #openstack-neutron | 23:14 | |
*** busterswt has joined #openstack-neutron | 23:14 | |
*** reedip__ has quit IRC | 23:16 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-lib: Revert "Add Collector Agent type string" https://review.openstack.org/303110 | 23:19 |
*** mickeys has quit IRC | 23:19 | |
*** vjmorale has joined #openstack-neutron | 23:20 | |
*** mickeys has joined #openstack-neutron | 23:21 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-lib: Revert "Add Collector Agent type string" https://review.openstack.org/303110 | 23:21 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revert "Add instrumentation devref, Part I" https://review.openstack.org/303111 | 23:22 |
*** saisrikiran has quit IRC | 23:22 | |
*** vjmorale has quit IRC | 23:22 | |
*** ntpttr_ has quit IRC | 23:22 | |
*** salv-orl_ has quit IRC | 23:24 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revert "Add instrumentation devref, Part I" https://review.openstack.org/303111 | 23:24 |
*** banix has quit IRC | 23:27 | |
*** markvoelker has quit IRC | 23:29 | |
*** sridhar_ram has joined #openstack-neutron | 23:31 | |
*** abhiraut has quit IRC | 23:32 | |
*** panda has quit IRC | 23:33 | |
*** panda has joined #openstack-neutron | 23:34 | |
*** abhiraut has joined #openstack-neutron | 23:36 | |
*** jamielennox|away is now known as jamielennox | 23:36 | |
*** ZZelle_ has quit IRC | 23:41 | |
*** akshai has joined #openstack-neutron | 23:48 | |
*** vhosakot has joined #openstack-neutron | 23:51 | |
*** busterswt has quit IRC | 23:52 | |
*** akshai_ has joined #openstack-neutron | 23:54 | |
*** lujinluo has joined #openstack-neutron | 23:56 | |
*** Sukhdev_ has joined #openstack-neutron | 23:57 | |
*** akshai has quit IRC | 23:58 | |
*** markvoelker has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!