Thursday, 2016-04-07

*** nplanel has joined #openstack-neutron00:01
*** nplanel_ has joined #openstack-neutron00:02
*** thorst has quit IRC00:02
*** akshai has joined #openstack-neutron00:03
*** john-davidge has quit IRC00:03
*** tflynn_ has joined #openstack-neutron00:04
jamielennoxhi, can i get some core eyes on https://review.openstack.org/#/c/292640/00:05
jamielennoxit's a simple change that fixes the options returned to the oslo.config generator so the sample files look right00:05
*** cgalan has joined #openstack-neutron00:06
*** tflynn has quit IRC00:07
*** tflynn_ has quit IRC00:09
*** vhosakot has joined #openstack-neutron00:09
*** akshai_ has joined #openstack-neutron00:10
openstackgerritArmando Migliaccio proposed openstack/neutron: Fix race conditions in IP availability API tests  https://review.openstack.org/29964700:12
*** akshai has quit IRC00:13
*** iyamahat has quit IRC00:17
openstackgerritRyan Tidwell proposed openstack/neutron: Compute IPAvailabilityRanges in memory during IP allocation  https://review.openstack.org/29220700:17
*** hoangcx has quit IRC00:23
*** hoangcx has joined #openstack-neutron00:23
*** thorst has joined #openstack-neutron00:25
*** brad_behle has joined #openstack-neutron00:25
*** dslevin has joined #openstack-neutron00:33
*** dslevin has quit IRC00:34
*** hoangcx has quit IRC00:34
*** dslevin has joined #openstack-neutron00:34
*** absubram has quit IRC00:34
*** hoangcx has joined #openstack-neutron00:35
*** dslevin has quit IRC00:35
*** abhiraut has quit IRC00:35
*** dslevin has joined #openstack-neutron00:35
*** dslevin has quit IRC00:36
*** dslevin has joined #openstack-neutron00:37
*** akshai_ has quit IRC00:37
*** dslevin has quit IRC00:37
*** baojg has joined #openstack-neutron00:37
*** mfuruta has joined #openstack-neutron00:38
*** abhiraut has joined #openstack-neutron00:38
*** dslevin has joined #openstack-neutron00:39
*** dslevin has quit IRC00:39
*** achanda has quit IRC00:40
*** tiswanso has quit IRC00:40
*** dslevin has joined #openstack-neutron00:41
*** dslevin has quit IRC00:42
*** sc68cal has quit IRC00:42
*** dslevin has joined #openstack-neutron00:43
*** dslevin has quit IRC00:43
*** Marga_ has quit IRC00:43
*** busterswt has quit IRC00:44
*** thorst has quit IRC00:45
*** mickeys has quit IRC00:45
*** thorst has joined #openstack-neutron00:46
*** gvrangan has quit IRC00:46
*** Marga_ has joined #openstack-neutron00:47
*** mickeys has joined #openstack-neutron00:48
*** gvrangan has joined #openstack-neutron00:48
*** tbachman has quit IRC00:50
*** tbachman has joined #openstack-neutron00:50
*** dave-mcc_ has quit IRC00:51
*** Marga_ has quit IRC00:51
*** busterswt has joined #openstack-neutron00:52
openstackgerritReedip proposed openstack/neutron: Add name and desc to networksegments table  https://review.openstack.org/29330500:52
*** mickeys has quit IRC00:52
*** dave-mccowan has joined #openstack-neutron00:54
*** thorst has quit IRC00:54
*** jonlan has joined #openstack-neutron00:55
*** iyamahat has joined #openstack-neutron00:55
*** harlowja has quit IRC01:00
*** hieulq has joined #openstack-neutron01:00
*** zhhuabj has quit IRC01:01
*** dave-mccowan has quit IRC01:03
openstackgerritReedip proposed openstack/python-neutronclient: Add possible updatable options to net-update CLI  https://review.openstack.org/28052101:03
*** harlowja has joined #openstack-neutron01:03
*** apuimedo has quit IRC01:07
*** aranjan_ has quit IRC01:07
*** shivrao has quit IRC01:07
*** rickyrem has quit IRC01:09
*** sarun has quit IRC01:09
*** Marga_ has joined #openstack-neutron01:10
*** zhhuabj has joined #openstack-neutron01:13
*** Marga_ has quit IRC01:14
*** abhiraut has quit IRC01:15
*** absubram has joined #openstack-neutron01:17
*** tiswanso has joined #openstack-neutron01:18
*** achanda has joined #openstack-neutron01:18
*** sridhar_ram1 has quit IRC01:18
reediparmax: ping01:20
*** sarun has joined #openstack-neutron01:20
reedipakamyshnikova: ping01:21
*** vhosakot has quit IRC01:21
*** stanzgy has joined #openstack-neutron01:21
openstackgerritHong Hui Xiao proposed openstack/neutron: Deprecate the functions map after segment moving out of ml2  https://review.openstack.org/29517301:21
*** reedip has quit IRC01:22
*** reedip has joined #openstack-neutron01:23
*** abhiraut has joined #openstack-neutron01:24
*** jonlan has quit IRC01:25
*** shashank_hegde has joined #openstack-neutron01:26
*** abhiraut has quit IRC01:29
*** EinstCrazy has joined #openstack-neutron01:29
openstackgerritZhaoBo proposed openstack/python-neutronclient: Do not delete dhcp port when purge tenant's resource  https://review.openstack.org/30204601:32
*** busterswt has quit IRC01:35
*** ajmiller has joined #openstack-neutron01:38
*** ajmiller has quit IRC01:39
*** julim has joined #openstack-neutron01:40
*** gvrangan has quit IRC01:40
*** shivrao has joined #openstack-neutron01:41
*** shivrao has quit IRC01:41
*** hieulq has quit IRC01:42
*** hieulq has joined #openstack-neutron01:42
*** vhosakot has joined #openstack-neutron01:44
*** baoli has joined #openstack-neutron01:44
*** baoli has quit IRC01:47
*** shashank_hegde has quit IRC01:47
*** baoli has joined #openstack-neutron01:48
*** sarun has quit IRC01:51
*** thorst has joined #openstack-neutron01:51
*** busterswt has joined #openstack-neutron01:54
*** s3wong has quit IRC01:54
*** rickyrem has joined #openstack-neutron01:57
*** thorst has quit IRC01:59
*** hoangcx has quit IRC02:02
armaxreedip: pong02:02
*** kengo_sakai has joined #openstack-neutron02:02
reediparmax: was checking out https://bugs.launchpad.net/neutron/+bug/156697202:02
openstackLaunchpad 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-neutron02:03
*** hoangcx has joined #openstack-neutron02:03
* armax looks02:03
reediparmax: cannot decide how to process it02:03
armaxreedip: this probably needs more vetting…let’s start with adding tags etc02:04
*** ljianbj has quit IRC02:05
*** shashank_hegde has joined #openstack-neutron02:05
reedipthat is where I would need some suggestion. This is impacting db, and currently looks more like an opinion02:06
*** baoli has quit IRC02:06
reedipand as it needs more information, needs-attention seems likely (??)02:07
reedipokay, you already changed it :)02:07
*** tmorin has joined #openstack-neutron02:08
*** Aish has joined #openstack-neutron02:08
*** ircuser-1 has quit IRC02:08
*** Aish has quit IRC02:09
*** wuhg has joined #openstack-neutron02:12
*** wolverin_ has joined #openstack-neutron02:14
*** wolverineav has quit IRC02:17
*** kevo has quit IRC02:17
*** anshul has joined #openstack-neutron02:21
*** namnh has joined #openstack-neutron02:21
*** gongysh has quit IRC02:22
*** gongysh has joined #openstack-neutron02:22
*** salv-orlando has quit IRC02:23
*** shashank_hegde has quit IRC02:24
openstackgerritHenry Gessau proposed openstack/neutron: Remove unused PolicyFileNotFound exception  https://review.openstack.org/30249702:26
*** iyamahat has quit IRC02:27
openstackgerritAssaf Muller proposed openstack/neutron: Fix keepalived functional tests  https://review.openstack.org/30242102:27
*** neeti has joined #openstack-neutron02:27
*** salv-orlando has joined #openstack-neutron02:28
*** tmorin has quit IRC02:29
*** minwang2 has quit IRC02:30
*** gongysh has quit IRC02:32
*** gongysh has joined #openstack-neutron02:32
*** salv-orlando has quit IRC02:33
*** EinstCrazy has quit IRC02:35
*** EinstCrazy has joined #openstack-neutron02:36
*** edmondsw has quit IRC02:36
*** yamamoto_ has joined #openstack-neutron02:37
*** tmorin has joined #openstack-neutron02:39
*** tmorin has quit IRC02:43
*** cshahani has quit IRC02:45
*** shashank_hegde has joined #openstack-neutron02:45
*** Leo_ has joined #openstack-neutron02:47
*** zhhuabj has quit IRC02:48
*** jhershbe has joined #openstack-neutron02:50
*** iyamahat has joined #openstack-neutron02:51
*** wolverin_ has quit IRC02:53
*** Leom has joined #openstack-neutron02:56
*** thorst has joined #openstack-neutron02:56
*** Leo_ has quit IRC02:59
*** mdnadeem has joined #openstack-neutron02:59
*** wolverineav has joined #openstack-neutron03:00
*** gongysh has quit IRC03:01
*** openstackstatus has quit IRC03:01
*** michchap has quit IRC03:02
*** singhj has joined #openstack-neutron03:02
*** ritesh has quit IRC03:03
*** thorst has quit IRC03:04
*** ritesh has joined #openstack-neutron03:04
*** wolverineav has quit IRC03:04
*** zhhuabj has joined #openstack-neutron03:05
*** michchap has joined #openstack-neutron03:06
*** reedip__ has quit IRC03:07
*** reedip__ has joined #openstack-neutron03:07
*** gongysh has joined #openstack-neutron03:08
*** achanda has quit IRC03:09
openstackgerritsteve proposed openstack/neutron-specs: Add support for external vxlan encapsulation to neutron router  https://review.openstack.org/28218003:15
*** neeti has quit IRC03:19
*** yamamoto_ has quit IRC03:20
*** wolverineav has joined #openstack-neutron03:22
*** singhj has quit IRC03:23
openstackgerritZhaoBo proposed openstack/python-neutronclient: Do not delete dhcp port when purge tenant's resource  https://review.openstack.org/30204603:23
*** jhershbe has quit IRC03:24
*** wolverineav has quit IRC03:26
*** kengo_sa_ has joined #openstack-neutron03:28
*** kengo_sa_ has quit IRC03:28
*** kengo_sakai_mba has joined #openstack-neutron03:28
*** kengo_sakai_mba has quit IRC03:30
*** kengo_sakai_mba has joined #openstack-neutron03:30
*** gongysh has quit IRC03:30
*** worawutchan has joined #openstack-neutron03:31
*** tiswanso has quit IRC03:32
*** gongysh has joined #openstack-neutron03:32
*** kengo_sakai_mba has quit IRC03:34
*** wolverineav has joined #openstack-neutron03:36
*** ramishra has joined #openstack-neutron03:37
worawutchanHi,Need help i want to know where can i find information about OVSvApp solution instructions guide.03:39
*** fitoduarte has joined #openstack-neutron03:41
*** shashank_hegde has quit IRC03:42
*** shivrao has joined #openstack-neutron03:44
*** annp has joined #openstack-neutron03:46
*** neelashah has joined #openstack-neutron03:49
annpHi 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-neutron03:50
*** sekrit has quit IRC03:51
*** worawutchan has quit IRC03:51
*** salv-orlando has quit IRC03:52
*** grassass has quit IRC03:52
*** tflynn has joined #openstack-neutron03:53
*** salv-orlando has joined #openstack-neutron03:53
*** tflynn has joined #openstack-neutron03:54
*** neelashah has quit IRC03:56
*** Marga_ has joined #openstack-neutron03:56
*** dave-mccowan has joined #openstack-neutron03:57
*** julim has quit IRC03:57
*** Marga_ has quit IRC03:57
*** Marga_ has joined #openstack-neutron03:57
*** brad_behle has quit IRC03:57
*** salv-orlando has quit IRC03:58
*** dave-mcc_ has joined #openstack-neutron03:58
*** jamespage has quit IRC04:00
*** jamespage has joined #openstack-neutron04:01
*** dave-mccowan has quit IRC04:01
*** Marga_ has quit IRC04:01
*** thorst has joined #openstack-neutron04:01
*** Marga_ has joined #openstack-neutron04:02
*** busterswt has quit IRC04:02
*** fedexo has joined #openstack-neutron04:02
*** cshahani has joined #openstack-neutron04:05
*** sekrit has joined #openstack-neutron04:05
*** mkolesni_ has joined #openstack-neutron04:06
*** numan_ has joined #openstack-neutron04:07
*** thorst has quit IRC04:09
*** yamamoto_ has joined #openstack-neutron04:10
*** rickyrem has quit IRC04:10
*** Sukhdev has joined #openstack-neutron04:11
*** links has joined #openstack-neutron04:12
*** absubram_ has joined #openstack-neutron04:14
*** absubram_ has quit IRC04:14
*** absubram has quit IRC04:15
*** minwang2 has joined #openstack-neutron04:17
*** tflynn has quit IRC04:17
*** shivrao has quit IRC04:18
*** tflynn has joined #openstack-neutron04:18
*** shivrao has joined #openstack-neutron04:20
*** yfried has joined #openstack-neutron04:20
*** amotoki has joined #openstack-neutron04:20
openstackgerritsteve 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/28650404:20
*** shashank_hegde has joined #openstack-neutron04:22
*** achanda has joined #openstack-neutron04:22
*** javeriak has joined #openstack-neutron04:25
*** iyamahat has quit IRC04:26
*** banix has joined #openstack-neutron04:26
*** anshul has quit IRC04:33
*** achanda has quit IRC04:37
*** irenab has quit IRC04:37
*** tflynn has quit IRC04:40
*** lujinluo has joined #openstack-neutron04:44
*** vikram has joined #openstack-neutron04:44
*** shivrao_ has joined #openstack-neutron04:50
*** jonlan has joined #openstack-neutron04:52
*** shivrao has quit IRC04:53
*** shivrao_ is now known as shivrao04:53
*** tflynn has joined #openstack-neutron04:54
*** shausy has joined #openstack-neutron04:55
*** admin0 has joined #openstack-neutron04:56
*** shivrao has quit IRC04:56
*** salv-orlando has joined #openstack-neutron04:57
*** armax has quit IRC04:59
*** abregman has joined #openstack-neutron05:00
*** yfried has quit IRC05:01
*** salv-orlando has quit IRC05:01
*** kengo_sakai_mba has joined #openstack-neutron05:02
*** aranjan has joined #openstack-neutron05:02
*** oanson has joined #openstack-neutron05:03
*** rossella_s has quit IRC05:03
*** achanda has joined #openstack-neutron05:03
*** rossella_s has joined #openstack-neutron05:03
*** thorst has joined #openstack-neutron05:06
*** admin0 has quit IRC05:08
*** jonlan has quit IRC05:08
*** asingh has quit IRC05:08
*** numan_ has quit IRC05:08
*** kawa2014 has joined #openstack-neutron05:09
*** ushkalim has quit IRC05:10
*** minwang2 has quit IRC05:10
*** salv-orlando has joined #openstack-neutron05:11
*** javeriak has quit IRC05:12
*** thorst has quit IRC05:13
*** asingh has joined #openstack-neutron05:16
*** ramishra has quit IRC05:16
*** sarun has joined #openstack-neutron05:17
*** asingh has quit IRC05:17
*** gongysh has quit IRC05:17
*** gongysh has joined #openstack-neutron05:19
*** michchap has quit IRC05:22
*** cshahani has quit IRC05:23
*** hdaniel has joined #openstack-neutron05:24
*** michchap has joined #openstack-neutron05:24
*** banix has quit IRC05:25
*** shivrao has joined #openstack-neutron05:26
openstackgerritHong Hui Xiao proposed openstack/neutron: [WIP] Enable create and delete for segments in ML2  https://review.openstack.org/28444005:29
*** aranjan has quit IRC05:29
*** itamarl has joined #openstack-neutron05:30
*** vthapar has joined #openstack-neutron05:31
*** anilvenkata has joined #openstack-neutron05:33
*** achanda has quit IRC05:33
*** gampel1 has joined #openstack-neutron05:35
*** shausy has quit IRC05:35
*** irenab has joined #openstack-neutron05:36
*** shausy has joined #openstack-neutron05:36
*** mohankumar__ has joined #openstack-neutron05:37
*** hieulq has quit IRC05:37
*** fedexo has quit IRC05:39
*** hieulq has joined #openstack-neutron05:39
*** javeriak has joined #openstack-neutron05:39
*** Sukhdev has quit IRC05:40
*** fedexo has joined #openstack-neutron05:42
*** minwang2 has joined #openstack-neutron05:43
*** admin0 has joined #openstack-neutron05:43
*** jhershbe has joined #openstack-neutron05:44
*** sarun has quit IRC05:44
*** rcernin has joined #openstack-neutron05:45
*** itzikb has joined #openstack-neutron05:45
*** garyk1 has joined #openstack-neutron05:46
*** sarun has joined #openstack-neutron05:47
*** minwang2 has quit IRC05:47
*** minwang2 has joined #openstack-neutron05:47
*** Shahid_ has joined #openstack-neutron05:49
vikramamotoki05:49
vikramamotoki: ping05:49
vikramamotoki: I tried the steps but couldn't get the result05:50
vikramamotoki: Can you try once... I have created the file_list05:50
vikramamotoki: https://github.com/VikramChoudhary/neutron-dynamic-routing/blob/master/file105:50
*** moshele has joined #openstack-neutron05:50
amotokivikram: let me push my results to my github05:51
*** admin0 has quit IRC05:51
vikramamotoki: ok05:51
vikramamotoki: i don't know what I did worng05:52
*** yfried has joined #openstack-neutron05:53
*** minwang2 has quit IRC05:57
*** sridharg has joined #openstack-neutron05:58
*** achanda has joined #openstack-neutron05:59
*** minwang2 has joined #openstack-neutron05:59
*** shivrao has quit IRC06:00
*** sarun has quit IRC06:00
*** sarun has joined #openstack-neutron06:01
*** sarun has quit IRC06:02
*** ircuser-1 has joined #openstack-neutron06:03
*** sarun has joined #openstack-neutron06:04
*** Leom has quit IRC06:08
*** eddima has joined #openstack-neutron06:08
*** numan_ has joined #openstack-neutron06:10
*** sarun has quit IRC06:11
*** thorst has joined #openstack-neutron06:11
*** gongysh has quit IRC06:13
*** gongysh has joined #openstack-neutron06:14
*** hdaniel has quit IRC06:17
*** gongysh has quit IRC06:18
*** thorst has quit IRC06:18
*** kengo_sakai_mba has quit IRC06:19
*** minwang2 has quit IRC06:19
*** jlibosva has joined #openstack-neutron06:20
*** oshvartz has joined #openstack-neutron06:21
openstackgerritMerged openstack/neutron: devref: Update ovs-firewall  https://review.openstack.org/29992606:21
*** achanda has quit IRC06:22
*** Leo_ has joined #openstack-neutron06:22
*** scheuran has joined #openstack-neutron06:22
*** minwang2 has joined #openstack-neutron06:24
*** kevo has joined #openstack-neutron06:24
*** moshele has quit IRC06:24
*** shashank_hegde has quit IRC06:25
*** pcaruana has joined #openstack-neutron06:26
*** Leom has joined #openstack-neutron06:26
*** fedexo has quit IRC06:27
*** neeti has joined #openstack-neutron06:28
*** numan_ has quit IRC06:28
*** numan_ has joined #openstack-neutron06:28
*** Leo_ has quit IRC06:29
*** gongysh has joined #openstack-neutron06:30
*** ekuris has joined #openstack-neutron06:30
*** Leom has quit IRC06:33
*** p01nt3r75 has quit IRC06:35
*** shivrao has joined #openstack-neutron06:36
openstackgerritJohn Schwarz proposed openstack/neutron: Add ALLOCATING state to routers  https://review.openstack.org/25705906:37
*** apoorv has joined #openstack-neutron06:38
*** sridharg has quit IRC06:38
*** ramishra has joined #openstack-neutron06:39
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Create standard attribute decorator  https://review.openstack.org/29282906:39
*** korzen has joined #openstack-neutron06:43
*** ushkalim has joined #openstack-neutron06:45
*** tesseract has joined #openstack-neutron06:45
*** tesseract is now known as Guest1287606:45
*** sridharg has joined #openstack-neutron06:47
*** sridharg has quit IRC06:47
*** kawa2014 has quit IRC06:50
*** sridharg has joined #openstack-neutron06:50
*** salv-orlando has quit IRC06:52
*** javeriak has quit IRC06:56
*** hdaniel has joined #openstack-neutron06:58
*** EinstCra_ has joined #openstack-neutron06:59
*** minwang2 has quit IRC07:00
*** apoorv_ has joined #openstack-neutron07:00
*** ekuris has quit IRC07:01
*** EinstCrazy has quit IRC07:02
*** kobis has joined #openstack-neutron07:03
*** Marga_ has quit IRC07:04
*** wolverineav has quit IRC07:04
*** hdaniel has quit IRC07:05
*** fzdarsky has joined #openstack-neutron07:08
*** dave-mcc_ has quit IRC07:09
*** shivrao has quit IRC07:09
*** admin0 has joined #openstack-neutron07:10
*** nyechiel_ has joined #openstack-neutron07:10
*** kickinz1 has joined #openstack-neutron07:11
*** atuvenie has joined #openstack-neutron07:11
*** admin0 has quit IRC07:11
*** nmagnezi has joined #openstack-neutron07:11
*** ramishra has quit IRC07:11
*** ihrachys has joined #openstack-neutron07:11
*** jlanoux has joined #openstack-neutron07:13
*** javeriak has joined #openstack-neutron07:14
*** Shahid_ has quit IRC07:15
*** moshele has joined #openstack-neutron07:15
*** thorst has joined #openstack-neutron07:16
*** ekuris has joined #openstack-neutron07:17
*** admin0 has joined #openstack-neutron07:19
*** tflynn has quit IRC07:19
*** kengo_sakai_mba has joined #openstack-neutron07:19
*** hdaniel has joined #openstack-neutron07:21
*** thorst has quit IRC07:24
*** baojg has quit IRC07:25
*** baojg has joined #openstack-neutron07:25
*** ramishra has joined #openstack-neutron07:27
*** kengo_sakai_mba has quit IRC07:28
*** abehl has joined #openstack-neutron07:29
*** matrohon has joined #openstack-neutron07:29
*** shivrao has joined #openstack-neutron07:29
*** yfried_ has joined #openstack-neutron07:32
*** yfried has quit IRC07:32
*** abehl has quit IRC07:33
reedipamotoki: Can you have a look if you have time: https://review.openstack.org/#/c/282184/ ?07:33
*** ekuris has quit IRC07:35
*** ramishra has quit IRC07:36
openstackgerritSayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups  https://review.openstack.org/28473807:37
openstackgerritLujin Luo proposed openstack/neutron: Add a composite unique key to floatingip table in Neutron database  https://review.openstack.org/27601807:38
*** dedery has joined #openstack-neutron07:38
*** ramishra has joined #openstack-neutron07:39
openstackgerritMerged openstack/neutron: Simplify chained comparison  https://review.openstack.org/30115607:41
openstackgerritMerged openstack/neutron: QoS DSCP use mod_flow instead of mod_flows  https://review.openstack.org/30063507:41
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Merge tag '8.0.0'  https://review.openstack.org/30257007:43
*** shivrao has quit IRC07:43
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Merge tag '8.0.0'  https://review.openstack.org/30257107:44
*** jpena|off is now known as jpean07:44
*** jpean is now known as jpena07:44
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Merge tag '8.0.0'  https://review.openstack.org/30257607:44
*** jamielennox is now known as jamielennox|away07:45
*** ekuris has joined #openstack-neutron07:48
vikramamotoki: ping07:49
amotokivikram: pong07:49
amotokireedip: will look soon07:49
vikramamotoki: got few questions while reviewing "https://review.openstack.org/#/c/299399/"07:49
vikramamotoki: can you please help07:49
reedipamotoki: Okay :)07:50
*** matrohon_ has joined #openstack-neutron07:51
*** matrohon_ has quit IRC07:51
amotokivikram: thanks for reminding. will reply in the review07:51
vikramamotoki: thanks07:52
*** hdaniel has quit IRC07:52
ihrachysrossella_s: would you mind https://review.openstack.org/#/c/301837/ ?07:54
*** salv-orlando has joined #openstack-neutron07:54
*** ramishra has quit IRC07:55
ihrachysrossella_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
openstackgerritMerged openstack/neutron: Fix race conditions in IP availability API tests  https://review.openstack.org/29964707:56
ihrachysah not really the last one since it depends on a patch with -107:57
vikramreedip: ping07:57
*** yfried_ has quit IRC07:58
reedipvikram: pongs07:58
vikramreedip: couldn't find your views @ https://review.openstack.org/#/c/299399/07:58
reedipJust check the 2 comments above yours :)07:58
korzenihrachys, rossella_s I've +1 on 29282907:58
ihrachyskorzen: thanks07:59
ihrachysrossella_s: but https://review.openstack.org/#/c/302267/2 should be good to go, for my taste07:59
openstackgerritIhar Hrachyshka proposed openstack/neutron: ovo: Update object with data from db on update()  https://review.openstack.org/30226708:00
*** haukebruno has joined #openstack-neutron08:03
*** devvesa has joined #openstack-neutron08:04
*** atuvenie has quit IRC08:04
*** wolverineav has joined #openstack-neutron08:05
*** hdaniel has joined #openstack-neutron08:05
rossella_sihrachys, I am on it08:07
*** apoorv_ has quit IRC08:08
*** apoorv has quit IRC08:08
ihrachysrossella_s: question on macaddress field. I probably miss something substential08:09
ihrachysrossella_s: why do we push strings into the field and not netaddr.EUI?08:09
*** lajos-katona has joined #openstack-neutron08:09
*** kawa2014 has joined #openstack-neutron08:09
*** wolverineav has quit IRC08:10
*** salv-orlando has quit IRC08:10
rossella_sihrachys, where do I do that?08:10
*** mickeys has joined #openstack-neutron08:11
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Create standard attribute decorator  https://review.openstack.org/29282908:11
rossella_sihrachys, maybe I miss something08:11
ihrachysrossella_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 IRC08:12
*** ygbo has joined #openstack-neutron08:12
ihrachysrossella_s: note that f.e. for obj_fields.IPNetworkField we set netaddr.IPNetwork08:12
rossella_sihrachys, good point08:13
ihrachysso that would be reasonable to have it consistent08:13
rossella_sihrachys, it was there before, I didn't notice it08:13
ihrachysok let me make a common08:13
ihrachyscomment08:13
*** rubasov has joined #openstack-neutron08:14
ihrachysrossella_s: done. there was also a small nit in the test, I posted the comment too08:14
ihrachysrossella_s: if you don't have time for that atm I can handle it08:14
rossella_sihrachys, great, I will update it later today, after the reviews08:14
rossella_sihrachys, but if you want to go on, please do08:14
ihrachysrossella_s: ok I will respin it. trying to make things going. :)08:14
rossella_sihrachys, same mood here! let's get things rolling08:15
*** ramishra has joined #openstack-neutron08:15
*** admin0 has quit IRC08:17
*** admin0 has joined #openstack-neutron08:19
*** bjornar has joined #openstack-neutron08:20
*** thorst has joined #openstack-neutron08:21
*** gampel1 has left #openstack-neutron08:24
*** mickeys has quit IRC08:25
*** mickeys has joined #openstack-neutron08:26
*** salv-orlando has joined #openstack-neutron08:26
*** ramishra has quit IRC08:26
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom OVO field type for MAC address  https://review.openstack.org/29347908:27
ihrachysrossella_s: that ^08:27
rossella_sihrachys, great!08:28
*** openstackstatus has joined #openstack-neutron08:29
*** ChanServ sets mode: +v openstackstatus08:29
*** thorst has quit IRC08:30
*** mickeys has quit IRC08:30
*** jistr has joined #openstack-neutron08:30
*** mickeys has joined #openstack-neutron08:31
rossella_sihrachys, I +2 both patches you mentioned08:33
-openstackstatus- NOTICE: jobs depending on npm are now working again08:34
*** apoorv_ has joined #openstack-neutron08:36
*** apoorv has joined #openstack-neutron08:36
ihrachysrossella_s: https://review.openstack.org/#/c/302267/ that one is the first one in the series08:36
rossella_sihrachys, I will review that too and the depending patches08:37
*** vhosakot has quit IRC08:38
*** p01nt3r75 has joined #openstack-neutron08:39
*** ramishra has joined #openstack-neutron08:45
*** rmart04 has joined #openstack-neutron08:45
jlibosvaihrachys: 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
ihrachysjlibosva: I would hug you then :P08:45
jlibosvaihrachys: so I'm pushing the one way only :D08:45
* ihrachys now realizes it may sound like a threat!08:45
ihrachysjlibosva: I am fine with a piece08:46
*** Shahid_ has joined #openstack-neutron08:47
*** EinstCra_ is now known as EinstCrazy08:51
ihrachysSam-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-neutron08:52
*** mickeys has quit IRC08:52
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods  https://review.openstack.org/30227608:53
*** mickeys has joined #openstack-neutron08:54
*** mickeys has quit IRC08:54
*** mickeys has joined #openstack-neutron08:54
*** mickeys has quit IRC08:54
*** salv-orlando has quit IRC08:55
*** rossella_s has quit IRC09:03
*** rossella_s has joined #openstack-neutron09:03
*** lujinluo has quit IRC09:04
*** javeriak has quit IRC09:05
*** nijaba has quit IRC09:05
*** sambetts|afk is now known as sambetts09:05
*** nijaba has joined #openstack-neutron09:07
*** yfried has joined #openstack-neutron09:08
*** djan has quit IRC09:11
*** mfuruta has quit IRC09:12
*** abehl has joined #openstack-neutron09:15
openstackgerritvenkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver  https://review.openstack.org/28111609:16
openstackgerritvenkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports  https://review.openstack.org/24122709:16
openstackgerritvenkata anil proposed openstack/neutron: Optimize get_ports query by filtering on subnet  https://review.openstack.org/30206209:16
*** zhhuabj has quit IRC09:18
*** zhhuabj has joined #openstack-neutron09:21
*** javeriak has joined #openstack-neutron09:22
*** hichihara has quit IRC09:23
*** shausy has quit IRC09:24
*** neiljerram has joined #openstack-neutron09:26
*** thorst has joined #openstack-neutron09:27
*** ramishra has quit IRC09:28
*** ramishra has joined #openstack-neutron09:30
*** namnh has quit IRC09:30
*** links has quit IRC09:31
*** alex_xu has quit IRC09:32
*** annp has quit IRC09:32
*** hoangcx has quit IRC09:32
*** thorst has quit IRC09:34
*** hoangcx has joined #openstack-neutron09:35
*** alex_xu has joined #openstack-neutron09:36
*** annp has joined #openstack-neutron09:40
*** yamamoto_ has quit IRC09:40
*** mvk has joined #openstack-neutron09:41
*** mkrcmari__ has quit IRC09:43
*** links has joined #openstack-neutron09:44
*** rubasov has quit IRC09:48
*** mvk_ has joined #openstack-neutron09:49
*** gongysh has quit IRC09:49
*** gongysh has joined #openstack-neutron09:50
openstackgerritvenkata anil proposed openstack/neutron: l2pop fdb flows for HA router ports  https://review.openstack.org/25523709:52
openstackgerritvenkata anil proposed openstack/neutron: HA: notify l2pop on port status update in slave  https://review.openstack.org/28287409:52
*** javeriak has quit IRC09:53
*** rubasov has joined #openstack-neutron09:53
*** mvk has quit IRC09:53
*** mhickey has joined #openstack-neutron09:54
*** achanda has joined #openstack-neutron09:56
*** obondarev has quit IRC10:00
*** achanda has quit IRC10:02
openstackgerritvenkata anil proposed openstack/neutron: [l2pop] fix agents failing to create flood flows  https://review.openstack.org/29167410:03
*** ociuhandu has quit IRC10:05
*** obondarev has joined #openstack-neutron10:06
*** wolverineav has joined #openstack-neutron10:06
*** numan_ has quit IRC10:07
*** Shahid_ has quit IRC10:07
*** baojg has quit IRC10:09
*** wolverineav has quit IRC10:11
*** obondarev has quit IRC10:12
*** Kiall has quit IRC10:14
*** Kiall has joined #openstack-neutron10:14
*** tbachman has quit IRC10:14
*** tbachman has joined #openstack-neutron10:14
*** fzdarsky has quit IRC10:15
*** hoangcx has quit IRC10:16
*** hoangcx has joined #openstack-neutron10:16
ihrachysrossella_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-neutron10:17
rossella_sihrachys, lol...am I not an author :p ?10:17
ihrachysrossella_s: you have not added yourself in a tag! :)10:18
ihrachysrossella_s: also, I don't think there is much left from the original implementation.10:18
rossella_sihrachys, got you10:19
*** sdague has joined #openstack-neutron10:20
*** singhj has joined #openstack-neutron10:20
*** numan_ has joined #openstack-neutron10:21
*** Shahid_ has joined #openstack-neutron10:22
*** numan_ has quit IRC10:22
*** numan_ has joined #openstack-neutron10:22
*** singhj has quit IRC10:24
openstackgerritMerged openstack/neutron: standard attributes: expose created_at/updated_at on models  https://review.openstack.org/30183710:27
*** EinstCrazy has quit IRC10:30
openstackgerritJakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config  https://review.openstack.org/30265310:30
*** obondarev has joined #openstack-neutron10:31
*** thorst has joined #openstack-neutron10:32
*** gongysh has quit IRC10:32
*** mhickey has quit IRC10:35
*** Shahid_ has quit IRC10:37
*** thorst has quit IRC10:39
*** links has quit IRC10:39
*** yamamoto has joined #openstack-neutron10:42
*** abregman has quit IRC10:44
*** abregman has joined #openstack-neutron10:44
*** gongysh has joined #openstack-neutron10:44
ihrachysajo: 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
ihrachysand it's simple10:47
*** Guest12876 is now known as tesseract10:47
*** mvk_ has quit IRC10:47
*** tesseract is now known as Guest7403610:48
*** mhickey has joined #openstack-neutron10:50
HenryGihrachys: https://review.openstack.org/30257010:52
HenryGihrachys: shall we merge it?10:52
*** andymaier has joined #openstack-neutron10:53
*** alex_xu has quit IRC10:55
*** sridharg has quit IRC10:55
ihrachysprolly. was there an email with the announcement for those patches?10:55
*** links has joined #openstack-neutron10:56
*** kengo_sakai_mba has joined #openstack-neutron10:56
ihrachysHenryG: ?10:56
HenryGI saw http://lists.openstack.org/pipermail/openstack-dev/2016-April/091649.html10:57
*** alex_xu has joined #openstack-neutron10:57
HenryGKnowing dhellmann's time zone, I am guessing he has been up all night o_O10:58
ihrachysthat one does not talk about those patches though10:58
HenryGright10:58
*** jlanoux has quit IRC10:58
ihrachysbut 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-neutron10:58
HenryGI remember that too10:59
HenryGChecking release channel ...10:59
*** cgalan has quit IRC10:59
ihrachysHenryG: https://review.openstack.org/#/c/302452/11:00
*** p01nt3r75 has quit IRC11:00
ihrachysHenryG: I +2'd11:00
*** annp has quit IRC11:01
*** gongysh has quit IRC11:02
*** nyechiel_ has quit IRC11:02
HenryGihrachys: ack11:02
pc_mihrachys: I just did the same for the VPN repo11:02
*** ushkalim has quit IRC11:02
*** jlanoux has joined #openstack-neutron11:03
ihrachysah right, going thru other repos11:03
pc_mhttps://review.openstack.org/#/c/302576/11:03
pc_mfyi11:03
*** achanda has quit IRC11:03
*** yamamoto has quit IRC11:04
ihrachyspc_m: +W11:04
pc_mihrachys: Thanks!11:04
ihrachysHenryG: pc_m: I guess we need someone else with +2 in *aas?11:04
ihrachysI +2'd but it's not two votes11:04
pc_mihrachys: Yeah.11:05
*** jlanoux_ has joined #openstack-neutron11:05
ihrachysor I go and abuse the rule for the benefit of everyone? :)11:05
HenryGihrachys: I don't have +2 there11:05
* ihrachys checking gerrit groups11:05
pc_mihrachys: me neither. Need FW and LB cores.11:05
ihrachyswould be weird to land neutron but not *aas11:06
*** yamamoto has joined #openstack-neutron11:06
*** ygbo has quit IRC11:06
ihrachysI will go and abuse and will prepare to explain my misbehaviour!11:06
pc_mHenryG: As drivers team member, can't you +2 *aaS?11:07
*** jlanoux has quit IRC11:07
HenryGpc_m: Nope. Only on specs.11:07
ihrachysall in gate11:08
HenryGihrachys: you rebel!11:08
* ihrachys feels good11:09
HenryG:)11:09
*** jprovazn has joined #openstack-neutron11:09
*** sridharg has joined #openstack-neutron11:10
*** sridharg has quit IRC11:10
*** sridharg has joined #openstack-neutron11:10
pc_mihrachys: I've got your back...11:10
* pc_m runs away silently11:10
*** thorst has joined #openstack-neutron11:11
*** thorst_ has joined #openstack-neutron11:11
openstackgerritMerged openstack/neutron: ovo: Update object with data from db on update()  https://review.openstack.org/30226711:13
winggundamthanyone knows about policy entry name that control add gateway interface to router?11:13
*** thorst has quit IRC11:15
*** kengo_sakai_mba has quit IRC11:15
ihrachysHenryG: does it mean we finally released Mitaka?11:16
ihrachysI guess so11:16
HenryGThat's what I remember from last cycle11:16
*** nyechiel_ has joined #openstack-neutron11:16
ihrachyswoohoo11:16
*** mvk_ has joined #openstack-neutron11:16
HenryGBut the release mgr probably waits for all project tags to merge before announcing, I guess11:17
*** ushkalim has joined #openstack-neutron11:18
HenryGAs a dev it feels like it was released weeks ago :)11:18
*** rtheis has joined #openstack-neutron11:18
*** fzdarsky has quit IRC11:21
*** tpsilva has joined #openstack-neutron11:22
openstackgerritHenry Gessau proposed openstack/neutron: Remove unused PolicyFileNotFound exception  https://review.openstack.org/30249711:23
*** zhhuabj has quit IRC11:23
*** zhhuabj has joined #openstack-neutron11:26
*** admin0 has quit IRC11:26
ihrachysHenryG: remember there was a patch to add test coverage framework for alembic scripts?11:28
ihrachysHenryG: do you have a link? can't find it :-|11:28
HenryGihrachys: https://review.openstack.org/12494611:29
ihrachysoh it's merged!11:29
*** john_bar has quit IRC11:29
ihrachysHenryG: should we force tests in scripts thru pep8 + update script templates?11:30
HenryGihrachys: I don't think we are there yet11:30
ihrachysHenryG: what's missing?11:31
HenryGihrachys: actual tests11:31
ihrachysHenryG: you mean for existing scripts?11:31
ihrachysHenryG: I suggest to enforce them for new scripts11:31
*** admin0 has joined #openstack-neutron11:31
HenryGihrachys: the framework is for functional tests11:33
jlibosvaihrachys: 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 meant11:33
HenryGihrachys: it needs mysql and postgresql backends. Won't work in pep8.11:34
ihrachysHenryG: 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 upgrades11:34
ihrachysHenryG: in pep8, we would merely check that _pre_upgrade_ is defined11:34
ihrachysHenryG: and then it will be reviewers call to make sure it makes stuff11:35
HenryGihrachys: perhaps that could be useful11:35
HenryGihrachys: 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
HenryGihrachys: this is primarily to verify scripts that change record data11:39
*** stanzgy has quit IRC11:40
*** alex_xu has quit IRC11:41
*** alex_xu has joined #openstack-neutron11:43
*** dslevin1 has joined #openstack-neutron11:44
*** dslevin1 has quit IRC11:44
ihrachysHenryG: exactly. so I would say, it's cool to dream about us going back to existing scripts and add test coverage there11:44
ihrachysHenryG: but it may not happen11:44
ihrachysHenryG: and we can enforce authors for new patches to get coverage for us at least for new stuff11:44
ajoihrachys, checking11:44
ihrachysHenryG: does it make sense?11:45
HenryGihrachys: Well, I am not sure that all scripts should require data tests.11:45
ihrachysHenryG: we just make sure they have the function defined11:45
ihrachysHenryG: then reviewers make a call11:45
*** dslevin1 has joined #openstack-neutron11:46
ihrachysHenryG: I guess once a reviewer sees the empty method, he may start asking questions11:46
HenryGihrachys: I guess it will raise awareness. And maybe annoyance. ;)11:46
ihrachysHenryG: we can even make the method name explicit enough, like '_pre_populate_for_testing' :)11:46
ihrachysHenryG: I love to annoy people11:47
ihrachysok, I will go craft some patches for that then :)_11:47
*** jhesketh has joined #openstack-neutron11:48
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack"11:48
HenryGihrachys: That's fine. I may not allow it to merge until I see an actual test implemented.11:49
*** Marga_ has joined #openstack-neutron11:49
*** banix has joined #openstack-neutron11:50
ihrachysHenryG: that's ok. I just want to make sure we have consistent coverage in Newton11:50
*** anilvenkata has quit IRC11:50
HenryGihrachys: That is a worthy goal, indeed.11:51
*** hynekm has joined #openstack-neutron11:52
*** p01nt3r75 has joined #openstack-neutron11:53
ajoihrachys, done: https://review.openstack.org/#/c/293479/811:54
ajolet me know if I'm missing something11:54
*** fzdarsky has joined #openstack-neutron11:55
*** javeriak has joined #openstack-neutron11:58
ihrachysajo: 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 IRC11:59
ihrachysrossella_s: korzen: why do we even have test_stringify_invalid?11:59
ihrachystest_coerce_bad_values should be enough to check validation11:59
ihrachysthat would allow us to kill all custom stringify() methods we have12:00
rossella_sihrachys, I raised a similar point yesterday...I guess the thing is if we want to validate when we call stringify12:00
ihrachysrossella_s: but it's not called with random values by anything, right?12:02
ihrachysrossella_s: the only place it's used is in base class for ovo12:02
ihrachysrossella_s: in __repr__12:02
rossella_sihrachys, theoretically coerce is called first12:02
ihrachyswhere it passes getattr(self, name) as a value12:02
ihrachysso coerce already happened12:02
rossella_sihrachys, right12:02
ihrachysso I go and clean that up? :)12:03
rossella_sihrachys, the convention in our custom type seems to be to validate in stringify too12:03
ihrachysrossella_s: I am not sure it's a convention or just a random mistake that got copy-pasted several times12:03
rossella_sihrachys, I agree in changing that...although that should be done for other types too, not only the MAC12:03
ihrachysrossella_s: right. I will propose a follow up and ask ajo to reconsider the vote for the MAC patch for now12:03
*** norman_ has joined #openstack-neutron12:04
rossella_sihrachys, agreed, thanks for that12:04
*** yfried_ has joined #openstack-neutron12:04
*** yfried has quit IRC12:05
*** EinstCrazy has joined #openstack-neutron12:06
*** jhesketh has quit IRC12:07
*** achanda has joined #openstack-neutron12:07
*** jhesketh has joined #openstack-neutron12:07
*** ekuris has quit IRC12:07
*** wolverineav has joined #openstack-neutron12:08
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: avoid validation in stringify for custom field types  https://review.openstack.org/30270412:09
ihrachysajo: ^12:09
ihrachysrossella_s: ^12:09
ihrachysajo: please reconsider the vote for MAC patch since I want to handle the comment in this follow up12:09
*** cgalan has joined #openstack-neutron12:10
*** davidsha has joined #openstack-neutron12:10
*** cgalan has quit IRC12:10
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods  https://review.openstack.org/30227612:10
ihrachysHenryG: oh I think I now see that the framework is not completely in line with how I thought about it12:12
*** kengo_sakai_mba has joined #openstack-neutron12:12
ihrachysHenryG: it requires test definition in functional tree, not in alembic scripts themselves?12:12
*** wolverineav has quit IRC12:12
HenryGihrachys: yes12:12
*** baoli has joined #openstack-neutron12:13
ihrachysI somehow thought you would define some callables in script and get them triggered by the framework12:13
*** ygbo has joined #openstack-neutron12:13
*** baoli_ has joined #openstack-neutron12:13
HenryGihrachys: Hah! I thought you were going to come up with some advanced hacking check for pep8. :)12:15
*** norman_ has quit IRC12:15
*** zhhuabj has quit IRC12:15
*** achanda has quit IRC12:15
*** admin0 has quit IRC12:16
ihrachysHenryG: well, it may actually still work :)12:17
*** kengo_sakai_mba has quit IRC12:17
ihrachysHenryG: I mean, some hacking check or a test for the test ;)12:17
*** neelashah has joined #openstack-neutron12:17
*** javeriak has quit IRC12:17
*** zhhuabj has joined #openstack-neutron12:17
* ihrachys hacks an evil enforcement muahaha12:17
*** baoli has quit IRC12:18
ihrachysHenryG: why do we have downgrades handling there?12:19
*** karthiks has joined #openstack-neutron12:20
*** Shahid_ has joined #openstack-neutron12:21
*** ekuris has joined #openstack-neutron12:22
*** neelashah has quit IRC12:22
*** edmondsw has joined #openstack-neutron12:23
*** mohankumar__ has quit IRC12:24
openstackgerritIhar Hrachyshka proposed openstack/neutron: tet_migrations: remove code to validate downgrades  https://review.openstack.org/30272212:26
*** admin0 has joined #openstack-neutron12:26
*** doug-fish has joined #openstack-neutron12:26
*** akshai has joined #openstack-neutron12:26
*** apoorv_ has quit IRC12:27
*** apoorv has quit IRC12:27
openstackgerritIhar Hrachyshka proposed openstack/neutron: test_migrations: always assume we want to validate scripts with data  https://review.openstack.org/30272412:28
*** salv-orl_ has quit IRC12:28
*** sakthi_chn has quit IRC12:28
*** wuhg has quit IRC12:29
openstackgerritMerged openstack/neutron: Merge tag '8.0.0'  https://review.openstack.org/30257012:29
*** nplanel has quit IRC12:29
*** amuller has joined #openstack-neutron12:30
pavel_bondargaryk: Hi! FYI, UTs has been added for https://review.openstack.org/#/c/272125/12:30
openstackgerritMerged openstack/neutron-vpnaas: Merge tag '8.0.0'  https://review.openstack.org/30257612:30
openstackgerritMerged openstack/neutron: Return oslo_config Opts to config generator  https://review.openstack.org/29264012:31
ihrachyswoot, we land the tags.12:31
*** banix has quit IRC12:31
openstackgerritPavel Bondar proposed openstack/neutron: Update ml2 delete_subnet to deallocate via ipam  https://review.openstack.org/30098412:31
*** banix has joined #openstack-neutron12:32
*** banix has quit IRC12:32
openstackgerritMerged openstack/neutron-fwaas: Merge tag '8.0.0'  https://review.openstack.org/30257112:32
HenryGihrachys: I know you backport aggressively, but I want to point out bug 154843312:32
openstackbug 1548433 in keystoneauth "neutron returns objects other than oslo_config.cfg.Opt instances from list_opts" [Undecided,Incomplete] https://launchpad.net/bugs/154843312:32
ihrachysHenryG: I *just* cherry-picked :)12:32
HenryGihrachys: good! :)12:33
ihrachysHenryG: should have gone into final, but well, it's life12:33
*** nplanel has joined #openstack-neutron12:33
*** pradk has joined #openstack-neutron12:34
*** brad_behle has joined #openstack-neutron12:34
openstackgerritJakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config  https://review.openstack.org/30265312:34
HenryGihrachys: yeah, we released Mitaka with it generating junk keystone config options. :(12:34
*** mkolesni_ has quit IRC12:35
*** jpena is now known as jpena|lunch12:35
ihrachysHenryG: way to go!12:36
openstackgerritSayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups  https://review.openstack.org/28473812:36
ihrachysHenryG: why wasn't the bug marked as a blocker?12:36
*** mohankumar__ has joined #openstack-neutron12:36
garyk1ihrachys: you understand fixtures?12:36
ajoihrachys, done, thanks!12:37
ihrachysajo: thanks12:37
*** lucasagomes has quit IRC12:37
ihrachysgaryk1: well... I think there are people who know more, like ZZelle or jlibosva12:37
ihrachysgaryk1: but you can try12:37
ihrachysHenryG: I moved it to High, it's not a Low bug for my consideration12:37
garyk1a 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.gz12:38
jlibosvalifeless knows probably the most of it ;)12:38
*** lucasagomes has joined #openstack-neutron12:38
garyk1TypeError: fake_consume_in_threads() takes exactly 1 argument (0 given)12:38
*** yfried_ has quit IRC12:38
*** tmorin has joined #openstack-neutron12:38
garyk1i have no idea what has/may caused this12:39
*** singhj has joined #openstack-neutron12:39
*** dougs has joined #openstack-neutron12:40
*** salv-orlando has joined #openstack-neutron12:40
garyk1has anyone else hit something like this?12:40
ihrachysgaryk1: it's neutron, not some dep bump?12:40
dougsAnyone know if that rate at which neutron is dumping flows is configurable?12:40
dougsneutron : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/usr/bin/neutron-rootwrap /etc/neutron/rootwrap.conf ovs-ofctl dump-flows br-int table=2312:40
dougsseems to be making calls at 10Hz12:41
dougs#journalctl --since=today | grep neutron-rootwrap | wc -w12:41
dougs27284212:41
garyk1ihrachys: the only dependedncies in the last 24 hrs that were updated were alembic and oslo.messagin12:41
*** doug-fish has quit IRC12:42
*** ababich has quit IRC12:42
*** doug-fish has joined #openstack-neutron12:42
*** admin0 has quit IRC12:43
garyk1ihrachys: 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.py12:43
*** nplanel has quit IRC12:43
*** ababich has joined #openstack-neutron12:43
ihrachysdougs: it should be controlled by polling_interval option12:44
*** ramishra has quit IRC12:44
*** nplanel has joined #openstack-neutron12:44
*** ramishra has joined #openstack-neutron12:45
dougsihrachys: got it, thanks!12:45
ihrachysgaryk1: I download the patch to play locally12:45
*** lucasagomes has quit IRC12:45
garyk1ihrachys: thanks.12:45
garyk1so the l2gw folks have hit the same issue12:46
garyk1i am not sure what is going on here.12:46
ihrachysyeah, we may better understand that instead of patching all subprojects12:46
garyk1i am leaning to what assaf wrote, lets drop all of the cores and tests…. could that be a april 7 joke12:46
*** lucasagomes has joined #openstack-neutron12:46
*** doug-fish has quit IRC12:47
*** achanda has joined #openstack-neutron12:47
ajojlibosva, I added two nit tiny comments: https://review.openstack.org/#/c/302653/212:47
openstackgerritAssaf Muller proposed openstack/neutron: Fix keepalived functional tests  https://review.openstack.org/30242112:48
amullerajo: ^12:48
amullerajo: bleh, the gerrit diff is confusing but the change is really small12:49
ihrachysgaryk1: if I replace it with mock, it works12:49
ajoamuller, ack checking12:49
ihrachysgaryk1: there was a fixtures release today12:49
ihrachys2.0.0 it is12:49
jlibosvaajo: thanks :)12:49
ihrachysI will push a patch in 5 mins12:49
amullerihrachys: oh my, neutron survived that?12:49
ihrachysamuller: neutron? probably. not subprojects ;P12:50
ihrachysamuller: or maybe neutron too, not sure12:50
ihrachysI don't care much12:50
*** julim has joined #openstack-neutron12:51
garyk1ihrachys: what to pin it? i think i have a fix12:51
garyk1def fake_consume_in_threads(*args, **kwargs):12:51
ihrachysREALLY?12:51
ihrachyswtf12:51
openstackgerritJakub Libosvar proposed openstack/neutron: ovsfw: Load vlan tag from other_config  https://review.openstack.org/30265312:52
garyk1yes, that looks like it resolves it.12:52
jlibosvaajo: updated commit message, fullstack TODO :)12:52
openstackgerritgaryk proposed openstack/neutron: Fix broken unit tests  https://review.openstack.org/30273712:53
amullerihrachys: I've seen two fullstack failures recently, one was neutron-server could not bind to socket, the other was OVS agent could not bind12:53
*** kengo_sakai has quit IRC12:54
garyk1ihrachys: ^^12:54
*** doug-fish has joined #openstack-neutron12:54
garyk1i need to maybe go and work with armax in the tomatoe field12:55
ihrachysgaryk1: I will scratch it gently12:55
*** doug-fis_ has joined #openstack-neutron12:56
*** kengo_sakai_mba has joined #openstack-neutron12:56
*** akshai has quit IRC12:56
*** dedery has quit IRC12:57
*** akshai has joined #openstack-neutron12:57
*** achanda has quit IRC12:58
*** doug-fish has quit IRC12:59
*** jonlan has joined #openstack-neutron13:00
ihrachysgaryk1: you know what's the REAL problem?13:00
ihrachysgaryk1: that subprojects don't use constraints :)13:00
amotokigaryk1: 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 IRC13:00
*** rodrigods has quit IRC13:01
*** rodrigods has joined #openstack-neutron13:01
garyk1ihrachys: sorry i do not follow13:02
ihrachysamotoki: the direction is right, but let me fix the commit message and explain it13:02
ihrachysgaryk1: if subprojects would use requirement constraints, they would not be hit by the issue right now13:02
ihrachysamuller: that's the reason neutron is not hit ^13:02
*** rossella_s has quit IRC13:03
ihrachysamuller: so we saved another fire drill for later (at least for core repos)13:03
garyk1ihrachys: due to the fact that i was at my wits end with this, i just copied the neutron requirements file(s)13:03
garyk1that did not help.13:03
amotokiihrachys: do we need to use (*args, **kwargs) for fake method?13:03
*** rossella_s has joined #openstack-neutron13:03
ihrachysamotoki: yes. since now fixtures make the method static and don't pass self13:03
ihrachysamotoki: new release13:03
ihrachysgaryk1: let me get back to you after I fix the patch13:04
garyk1ihrachys: ok, i am not sure that i know what is broken with the patch (other than the cryptic commit message )13:05
openstackgerritSayali Lunkad proposed openstack/neutron: Introduces ovo objects for security groups  https://review.openstack.org/28473813:06
openstackgerritIhar Hrachyshka proposed openstack/neutron: Fix broken unit tests  https://review.openstack.org/30273713:06
amotokiihrachys: thanks. this sounds tricky, but it is what's happening...13:07
ihrachysamotoki: garyk1: ^ I updated with explanation and bug reference13:07
ihrachysI guess we'll need to backport it till kilo..13:07
ihrachysespecially for kilo.13:07
ihrachyssince it's not guarded by constraints13:07
*** achanda has joined #openstack-neutron13:08
ihrachysgaryk1: so re constraints, I meant that subprojects should utilize: https://github.com/openstack/neutron/blob/master/tox.ini#L1113:08
ihrachysgaryk1: that would guard your gate from catching 2.0.0 right after it's released to pypi13:08
garyk1ihrachys: thanks!13:08
ihrachysthe bug would still be there, but your gate would be ok13:08
*** wolverineav has joined #openstack-neutron13:10
garyk1ihrachys: thanks!13:10
*** johnsom has quit IRC13:10
*** eric_lopez has quit IRC13:10
amotokiI 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-neutron13:11
*** johnsom has joined #openstack-neutron13:11
*** mvk_ has quit IRC13:11
*** anilvenkata has joined #openstack-neutron13:12
*** achanda has quit IRC13:13
*** mvk has joined #openstack-neutron13:13
ihrachysamotoki: yeah, probably. which is weird a bit.13:13
ihrachysamotoki: we can always fall back to mock if anything13:14
ihrachysamotoki: it's just that for now the garyk1's patch is ok13:14
*** javeriak has joined #openstack-neutron13:14
*** akshai has quit IRC13:14
Sam-I-Ammoo.13:14
ihrachysSam-I-Am: moo13:15
amotokiihrachys: understood and garyk1's patch looks good.13:15
ihrachysSam-I-Am: I suggest you review that docs patch for MTU13:15
*** wolverineav has quit IRC13:15
Sam-I-Amihrachys: looking at it13:15
ihrachysSam-I-Am: I blocked it with WIP for now, since it already has one +213:15
ihrachysI wanted you to chime in first13:15
Sam-I-Amsure13:15
*** jonlan has quit IRC13:17
ZZellegaryk, needs some help on fixtures13:17
ZZelle?13:18
garyk1ZZelle: thanks. we are sorted. now just reviews :)13:19
garyk1ZZelle: https://review.openstack.org/30273713:19
*** tiswanso has joined #openstack-neutron13:20
*** ababich has quit IRC13:20
*** ababich has joined #openstack-neutron13:21
*** oanson has quit IRC13:22
*** doug-fis_ has quit IRC13:23
*** akshai has joined #openstack-neutron13:23
*** ociuhandu has joined #openstack-neutron13:25
openstackgerritPavel Bondar proposed openstack/neutron: Always call ipam driver on subnet update  https://review.openstack.org/27212513:27
*** boden has joined #openstack-neutron13:28
amullerjlibosva: any idea of addOnException handler would be called before all fixtures are torn down or after?13:29
amullerjlibosva: if it's after, it might not be very useful to find out which ports are used by which processes13:29
*** singhj has quit IRC13:29
openstackgerritSayali Lunkad proposed openstack/neutron: Instantiate Enum() rather than subclassing  https://review.openstack.org/30275913:29
*** eddima1 has joined #openstack-neutron13:30
*** eddima has quit IRC13:31
*** eddima1 is now known as eddima13:31
*** enriquetaso has joined #openstack-neutron13:31
*** banix has joined #openstack-neutron13:34
openstackgerritVictor Morales proposed openstack/neutron: Address Scope added to OVO  https://review.openstack.org/29782113:37
*** fzdarsky has quit IRC13:38
*** bdemers has quit IRC13:40
*** yamamoto has quit IRC13:40
openstackgerritFelipe Reyes proposed openstack/neutron: Add configuration option to change keepalived globals  https://review.openstack.org/30276613:40
*** singhj has joined #openstack-neutron13:40
*** jpena|lunch is now known as jpena13:41
*** sc68cal has joined #openstack-neutron13:42
*** irenab has quit IRC13:44
*** mgoddard_ has joined #openstack-neutron13:44
*** bdemers has joined #openstack-neutron13:45
mhickeyihrachys: Hey13:45
*** Shahid_ has quit IRC13:45
scheuranrkukura, ping13:46
*** kobis has quit IRC13:46
*** links has quit IRC13:46
*** garyk1 has quit IRC13:47
*** mgoddard has quit IRC13:48
*** bdemers_ has joined #openstack-neutron13:48
*** janzian has joined #openstack-neutron13:48
openstackgerritAnn Kamyshnikova proposed openstack/neutron: WIP: New engine facade from oslo_db  https://review.openstack.org/30001713:48
*** woodburn1 has joined #openstack-neutron13:49
ihrachysmhickey: hi13:49
*** bdemers has quit IRC13:49
*** woodburn has quit IRC13:50
mhickeyihrachys: wrt comment in https://review.openstack.org/#/c/285397/5/neutron/db/extra_dhcp_opt/models.py13:50
ihrachysmhickey: I thought we discussed it already?13:50
*** cshahani has joined #openstack-neutron13:51
mhickeyihrachys: yes, just found an issue with it13:51
ihrachysmhickey: tell me13:51
*** sridharg has quit IRC13:52
mhickeyihrachys: if I roll back the changes and return the class ExtraDhcpOpt to neutron/db/extradhcpopt_db.py, I will end up with circular imports13:52
amullerihrachys: OK, it looks like the way we determine which ports are used is broken13:52
ihrachysmhickey: now *that's* a reason13:52
*** jhershbe has quit IRC13:52
ihrachysamuller: in where?13:52
amullerihrachys: "_get_source_ports_from_ss_output"13:52
mhickeyihrachys: I know! I thought you would like it! :-)13:53
amullerihrachys: it's missing rows13:53
ihrachysmhickey: if that's the case, then let's indeed move forward with what we have.13:53
*** tonytan4ever has joined #openstack-neutron13:53
mhickeyihrachys: ok, let me try and get patch in in next while.13:53
ihrachysamuller: regex consumes multiple lines?13:54
ihrachysmhickey: I believe there were other comments to address right?13:54
amullerihrachys: 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 others13:54
*** singhj has quit IRC13:55
*** ametts has joined #openstack-neutron13:56
*** busterswt has joined #openstack-neutron13:57
*** yamamoto has joined #openstack-neutron13:57
*** admin0 has joined #openstack-neutron14:00
openstackgerritQingchuanHao proposed openstack/neutron: FIP: Ensure fpr and rfp are configured correctly  https://review.openstack.org/30234014:00
amullerihrachys: btw nice fix to a functional tests issue https://review.openstack.org/#/c/302421/14:01
*** yamahata has joined #openstack-neutron14:01
* ihrachys looks14:01
mhickeyihrachys: sure; that was one beaking me and causing me some deep thinking! :-D14:02
*** yamamoto has quit IRC14:03
*** mgoddard has joined #openstack-neutron14:04
*** mgoddard_ has quit IRC14:04
ihrachysamuller: +W14:04
amullerihrachys: thanks, that's at least one cause for functional tests failures gone14:04
ihrachysamuller: one spoon at a time14:04
ihrachysamuller: 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 IRC14:06
*** eddima has joined #openstack-neutron14:06
*** korzen has quit IRC14:06
amullerihrachys: 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
amullerihrachys: so I think the next step would be to add additional info / logging that would help us next time this happens at the gate14:07
ihrachysamuller: ack. ok, in the worst scenario we could start disabling tests to see which one deadlocks us14:08
ihrachysamuller: yeah, syslog since we loose logs in timeour?14:08
ihrachys*timeout14:08
amullerihrachys: yeah, syslog is a possibility14:08
amullerihrachys: maybe also look in to why individual tests don't time out14:08
ihrachysamuller: that's actually one thing we should look at independently - why the hell time limits don't work14:08
ihrachysyea14:08
ihrachys:)14:08
amullerihrachys: if we solve that we get our logs back14:09
amullerihrachys: maybe that should be priority 114:09
*** tbachman has quit IRC14:10
*** knikolla has joined #openstack-neutron14:10
openstackgerritOleg Bondarev proposed openstack/neutron: Notify resource_versions from agents only when needed  https://review.openstack.org/30279214:10
*** neelashah has joined #openstack-neutron14:10
ihrachysajo: ^14:10
*** wolverineav has joined #openstack-neutron14:11
*** neeti has quit IRC14:12
*** sadasu has quit IRC14:15
*** wolverineav has quit IRC14:16
*** ushkalim has quit IRC14:16
*** mkolesni_ has joined #openstack-neutron14:17
*** ChrisBenson has joined #openstack-neutron14:17
*** julim has quit IRC14:19
*** singhj has joined #openstack-neutron14:20
*** thorst_ has quit IRC14:21
*** julim has joined #openstack-neutron14:22
*** thorst_ has joined #openstack-neutron14:22
*** baoli_ has quit IRC14:26
*** thorst__ has joined #openstack-neutron14:26
*** thorst_ has quit IRC14:27
*** sarun has joined #openstack-neutron14:28
*** cshahani has quit IRC14:28
*** ushkalim has joined #openstack-neutron14:28
*** asingh has joined #openstack-neutron14:28
*** yamamoto has joined #openstack-neutron14:29
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Split modify_(from|to)_db methods  https://review.openstack.org/30227614:29
*** mlavalle has joined #openstack-neutron14:29
*** jlibosva has quit IRC14:30
*** javeriak has quit IRC14:30
*** baoli has joined #openstack-neutron14:30
*** lajos-katona has quit IRC14:30
*** matrohon has quit IRC14:31
*** vhosakot has joined #openstack-neutron14:31
ajoihrachys, ouch14:31
*** matrohon has joined #openstack-neutron14:31
*** jlibosva has joined #openstack-neutron14:31
openstackgerritIhar Hrachyshka proposed openstack/neutron: test_migrations: enforce tests for new alembic scripts  https://review.openstack.org/30281314:32
ajoobondarev, thanks, looking at https://review.openstack.org/#/c/302792/14:32
*** psanchez has quit IRC14:33
ihrachysHenryG: ^ nuffsaid14:33
*** javeriak has joined #openstack-neutron14:34
*** tbachman has joined #openstack-neutron14:34
*** jhershbe has joined #openstack-neutron14:36
*** thorst__ has quit IRC14:37
openstackgerritMerged openstack/neutron: Remove unused PolicyFileNotFound exception  https://review.openstack.org/30249714:37
obondarevajo: ihrachys: thanks14:38
ajoobondarev, there's only one thing to take care of: https://github.com/openstack/neutron/blob/c007960ab8ebc56e5c3d0c8cad99c921615a1915/neutron/api/rpc/callbacks/version_manager.py#L14314:38
openstackgerritFelipe Reyes proposed openstack/neutron: Add configuration option to change keepalived globals  https://review.openstack.org/30276614:38
ajowe need to keep the compatibility to the old ovs/sriov agents which did not report versions in liberty,14:38
ajobut that need a certain version of the QoSPolicy (v1.0)14:39
*** tbachman_ has joined #openstack-neutron14:39
ajoobondarev, I did not forsee that could be so slow... :?14:39
ajoobondarev, may be we could get update_versions to return True/False, and in case of True we fanout to other servers?14:40
openstackgerritMerged openstack/neutron: DVR: Increase the link-local address pair range  https://review.openstack.org/29783914:40
*** tbachman has quit IRC14:40
*** tbachman_ is now known as tbachman14:40
obondarevajo: that might be due to debug logs, but still rpc to server is smth we want to reduce14:40
*** korzen has joined #openstack-neutron14:40
*** thorst_ has joined #openstack-neutron14:41
ajoI understand.14:41
obondarevajo: this won't be needed if agent return resource_versions onle when needed I guess14:42
*** banix has quit IRC14:42
ajoobondarev, yes, but in the case of upgrade,14:42
*** rkukura has quit IRC14:42
ajowe will have old liberty agents never returning resource_versions14:42
*** vhosakot has quit IRC14:43
ajowe won't know if they are old, or ... if they are new but not sending resource_versions at that time...14:43
obondarevajo: I see, need to think about it..14:43
*** vhosakot has joined #openstack-neutron14:43
*** davidsha has quit IRC14:44
saisrikiihrachys: 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
ihrachysajo: kinda easy path but: how about just disabling the special casing since we never got dscp in mitaka?14:44
ajoobondarev,  it's optimizable probably in the way you did, with some tweaks, let me know if you need  any help.14:44
ajohmmm14:44
*** kawa2014 has quit IRC14:44
ajoihrachys, good point ^  obondarev14:44
ajoyeah, and we're not supposed to allow two jump versions...14:45
*** numans has quit IRC14:45
ajoobondarev, there it is...14:45
ihrachyssaisriki: please take dns. qos - it may be a special case, I will need to take a look myself.14:45
obondarevajo: not sure I'm following..14:45
saisrikiihrachys: ok. Thanks.14:45
obondarevajo: so liberty agents will always need only 1.0 version14:45
ajoobondarev, QoSPolicy version was not changed in mitaka :)14:46
obondarevajo: right14:46
ajoobondarev, that means, that they started reporting their versions14:46
ajo(at start)14:46
ajoso from mitaka->newton is when that will be necessary14:46
ajoand the workaround is not needed anymore then14:46
obondarevajo: perfect14:46
ajoworkaround = https://github.com/openstack/neutron/blob/c007960ab8ebc56e5c3d0c8cad99c921615a1915/neutron/api/rpc/callbacks/version_manager.py#L14314:47
korzensaisriki, DNSNameServer is covered by me14:47
*** numan_ has quit IRC14:47
obondarevajo: can be cleaned up then?14:47
ajoobondarev, ihrachys , yes... let me do it in a separate patch :)14:47
obondarevajo: great14:48
obondarevajo: thanks14:48
*** ametts has quit IRC14:48
saisrikikorzen: ok, did you mean in https://review.openstack.org/#/c/264273/ patch?14:48
ihrachysajo: honestly, the whole code is not really needed in mitaka14:48
ihrachysajo: we could as well kill the updater thread14:48
ihrachys:)14:48
korzensaisriki, yes14:49
*** ametts has joined #openstack-neutron14:49
*** tbachman has quit IRC14:49
ajoihrachys, yes, but we can't kill it in newton.. :)14:49
saisrikikorzen: ok. Thanks for the clarification14:49
ajoihrachys, stable only patch? :)14:49
ihrachysajo: why not14:49
ihrachysajo: if it effectively reduces load14:49
ajoihrachys, do the honors :P14:49
*** javeriak_ has joined #openstack-neutron14:49
ajoihrachys, or obondarev hmm14:49
*** nmagnezi has quit IRC14:49
korzensaisriki, the DNSNameServer was a dependency for Subent so I've took it with couple other obejcts14:50
ihrachysajo: nah, I will sit here watching for the fix emerging from under your fingers14:50
ajoihrachys,  may be it's better not to have stable only patches if not absolutely necessary, later backports hurt when code has diverged14:50
ajoor we can do it in a minimalistic way14:50
* ajo thinks14:50
ihrachysajo: well if it saves performance...14:50
ihrachysajo: we can just tactically disable the updates in proper places14:50
*** javeriak has quit IRC14:50
saisrikikorzen: ok are you finished with the DNSNameServer objects work?14:50
ihrachysand leave the code unused14:51
ajoihrachys, sounds good14:51
ajoso plan14:51
ajo1) obondarev optimizes newton (as of his current patch)14:51
ajo2) I kill the corner cases not needed anymore14:51
*** tbachman has joined #openstack-neutron14:51
ajo3) I kill the update code in mitaka14:51
obondarevajo: ihrachys: I'd like to backport https://review.openstack.org/#/c/302792 to mitaka as well..14:52
*** salv-orl_ has joined #openstack-neutron14:52
ajoobondarev,14:52
obondarevdoes it fit?14:52
ajoprobably we should consider 3 as optional14:52
ajoobondarev, how did your tests look after your patch?14:52
*** salv-orl_ has quit IRC14:53
ihrachysajo: I love the 3 steps plan14:53
ajodoes it make a big difference if we disable the update mechanism completely?14:53
obondarevajo: I need to retest, I know it helper when I just commented self._update_local_agent_resource_versions(context, agent_state)14:53
ihrachysajo: we should only make sure that agents *sending* the resource versions are not breaking the new server that would ignore the data14:53
ajoihrachys, it's a dict14:54
ajoso whatever not used is ignored14:54
*** salv-orlando has quit IRC14:54
korzensaisriki, yes, the DNSServer as object is finished, I did not look into using it outside of Seubnet OVO patch14:54
ihrachysajo: ok. but we should test the scenario with old agents and new server.14:54
*** thorst__ has joined #openstack-neutron14:54
ihrachyssince the feature is not tested in gate, it's only manual14:55
*** mickeys has joined #openstack-neutron14:55
ajoihrachys, makes sense, if the perf gain is worth it14:55
saisrikikorzen: ok, thanks.14:55
*** anilvenkata has quit IRC14:55
*** ramishra has quit IRC14:55
korzensaisriki, ihrachys I need to remoev the WIP from my Subnet patch because I've seen no reviews ;)14:55
ihrachyskorzen: noted14:57
saisrikikorzen: there seems to be a Merge conflict as well?14:57
electrocucarachakorzen: you're also rebase that change right?14:57
*** ametts has quit IRC14:57
*** ramishra has joined #openstack-neutron14:57
*** javeriak has joined #openstack-neutron14:57
*** cristicalin has joined #openstack-neutron14:58
*** thorst_ has quit IRC14:58
korzenelectrocucaracha, saisriki, yes, I need to rebase it as well14:58
*** ametts has joined #openstack-neutron14:58
*** cristicalin has quit IRC14:58
*** regXboi has joined #openstack-neutron14:58
*** cristicalin has joined #openstack-neutron14:59
*** javeriak_ has quit IRC14:59
electrocucarachakorzen: what about the TODO section in the commit message?14:59
*** mgoddard_ has joined #openstack-neutron15:00
ajoobondarev, I have a meeting, and I will do a 2nd pass on your patch to make sure I didnt' miss anything15:00
*** johnbelamaric has joined #openstack-neutron15:00
*** oshvartz has quit IRC15:00
*** thorst__ is now known as thorst________15:00
*** abehl has quit IRC15:01
obondarevajo: cool, thanks!15:01
*** iyamahat has joined #openstack-neutron15:01
openstackgerritBrian Haley proposed openstack/neutron: Move all link-local cidr constants to a central location  https://review.openstack.org/30282915:01
*** tiswanso has quit IRC15:01
*** kawa2014 has joined #openstack-neutron15:02
*** tiswanso has joined #openstack-neutron15:03
*** dougs1 has joined #openstack-neutron15:03
*** mgoddard has quit IRC15:03
korzenelectrocucaracha, 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 IRC15:03
korzenelectrocucaracha, 1) is happening in parallel and 2) is in TBD in near future15:04
*** sarun has quit IRC15:05
openstackgerritOleg Bondarev proposed openstack/neutron: Notify resource_versions from agents only when needed  https://review.openstack.org/30279215:06
*** kawa2014 has quit IRC15:06
*** ajmiller has joined #openstack-neutron15:07
*** ababich has quit IRC15:07
*** j_king_ is now known as j_king15:07
*** ababich has joined #openstack-neutron15:08
*** thorst________ is now known as thorst_15:08
*** jhershbe has quit IRC15:08
*** ababich has quit IRC15:09
*** ababich has joined #openstack-neutron15:09
*** fitoduarte has quit IRC15:10
*** jlibosva1 has joined #openstack-neutron15:10
*** Guest74036 has quit IRC15:10
*** stanzgy has joined #openstack-neutron15:10
electrocucarachaquestion, BGP stands for Border Gateway P? or Big Giant Pencil...15:10
*** sarun has joined #openstack-neutron15:11
njohnstonBorder Gateway Pencil15:11
*** mkolesni_ has quit IRC15:11
*** yottatsa has joined #openstack-neutron15:11
electrocucarachahahaha15:11
*** tbachman has quit IRC15:11
*** crose has joined #openstack-neutron15:12
*** rkukura has joined #openstack-neutron15:13
*** jlibosva has quit IRC15:14
matrohonscheuran, hi15:14
*** javeriak has quit IRC15:15
openstackgerritRichard Theis proposed openstack/python-neutronclient: Devref: Newton updates for transition to OSC  https://review.openstack.org/29939915:16
*** tbachman has joined #openstack-neutron15:17
openstackgerritIhar Hrachyshka proposed openstack/neutron: Enable sorting and pagination in -api gate  https://review.openstack.org/30163415:18
*** javeriak has joined #openstack-neutron15:18
ihrachysHenryG: amuller: ^ a new version of the first api tests for sorting/pagination. would love to hear what you think.15:19
*** baoli has quit IRC15:19
*** kawa2014 has joined #openstack-neutron15:19
openstackgerritJakub Libosvar proposed openstack/neutron: ovo: Make modify_fields_(from|to)_db instance methods  https://review.openstack.org/30227615:19
*** hongbin has joined #openstack-neutron15:20
openstackgerritOleg Bondarev proposed openstack/neutron: ADDRESS_SCOPE_MARK_IDS should not be global for L3 agent  https://review.openstack.org/30168515:20
obondarevcarl_baldwin: haleyb: ^^15:20
carl_baldwinobondarev: Thanks!15:20
haleybobondarev: thanks15:20
hongbinHi 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 IRC15:21
hongbinHere 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_56715:21
hongbinI wonder if there is a recent commit in neutron that causes this error?15:21
ramishrahongbin, hi, it seems https://github.com/openstack/neutron/commit/34a328fe12950c339b8259451262470c627f2f00 has broken lbaas15:22
*** rickyrem has joined #openstack-neutron15:22
ramishraand hence heat gate is also broken15:22
hongbinramishra: ack. Thanks15:22
hongbinI guess someone is working on a fix?15:23
openstackgerritVictor Morales proposed openstack/neutron: WIP - BGP to OVO  https://review.openstack.org/30286315:23
yottatsaJust found that physical_network_mtus = physnet1:1500,physnet2:1500 will not be handled due to https://bugs.launchpad.net/neutron/+bug/156750215:24
openstackLaunchpad bug 1567502 in neutron "[ml2]/physical_network_mtus impoperly handled" [Undecided,New]15:24
*** achanda has joined #openstack-neutron15:24
scheuranmatrohon, hi15:24
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/30286515:24
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/30166915:24
*** lnicolas has quit IRC15:24
matrohonscheuran, https://review.openstack.org/#/c/297100/315:24
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/30167115:24
*** neelashah has quit IRC15:25
matrohonscheuran, great start! I gave my opinion inline15:25
*** alex_xu has quit IRC15:25
scheuranmatrohon, thanks15:25
matrohonscheuran, I though there was a related blueprint already15:25
scheuranmatrohon, regarding the failing, tests - I need to look into them15:25
scheuranmatrohon, on my devstack multinode migration was working15:26
*** minwang2 has joined #openstack-neutron15:26
matrohonscheuran, great! from ovs to linuxbridge?15:26
sc68calyottatsa: are you going to push a patch?15:26
*** davidsha has joined #openstack-neutron15:26
garykihrachys: amotoki: amuller: https://review.openstack.org/#/c/302737/ has the grumpy old mans +1. Can we please approve15:26
scheuranmatrohon, so I expect them to be test-failures15:26
*** yamahata has quit IRC15:26
scheuranmatrohon, yes, ovs to lb, lb to ovs, ovs to macvtap15:26
ihrachysgaryk: +W15:26
scheuranmatrohon, I forgot to add the blueprint link...you're right15:27
scheuranmatrohon, blueprint + spec already exists..15:28
openstackgerritOpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/30288315:28
garykihrachys: thanks! much appreciated.15:28
matrohonscheuran, great, it will help reviewer to have the context of this patch15:28
*** Swami has joined #openstack-neutron15:28
ramishraihrachys, ajo: can someone please look at https://bugs.launchpad.net/neutron/+bug/156750715:28
openstackLaunchpad bug 1567507 in neutron "neutron-lbaas broken with neutron change" [Undecided,New]15:28
*** alex_xu has joined #openstack-neutron15:29
ihrachysramishra: ouch looking15:29
*** cristicalin has quit IRC15:29
*** stanzgy has quit IRC15:29
scheuranmatrohon, great, thanks!15:29
scheuranmatrohon, and you're right, multibound ports are only network ports today15:30
matrohonscheuran, the code is pretty thin, hope it will be able to merge in newton15:30
*** armax has joined #openstack-neutron15:30
scheuranmatrohon, yes it is, I still need to make some methods public that are currently private or so, but nothing big15:31
ramishraihrachys: does it make sense to revert the change or wait for a fix?15:31
scheuranmatrohon, the problem probably is what I described in my comment, regarding external drivers...15:31
scheuranmatrohon, so some research must be done..15:31
ihrachysramishra: give me a minute to think on the stack15:32
scheuranmatrohon, The alternative is multiple bound ports15:32
scheuranmatrohon, https://review.openstack.org/#/c/189410/15:32
*** cshahani has joined #openstack-neutron15:32
*** garyk has quit IRC15:32
scheuranmatrohon, this is the multiple bind patch15:32
ramishraihrachys: np:)15:33
scheuranthe idea is to remove the 2 portbinding tables that currently exist (dvr & normal) and merge them15:33
matrohonscheuran, it's a very hold patch that didn't have the love it deserves15:33
scheuranmatrohon, if done, a new api could be established to allow multpile host binding for compute ports as well15:34
*** singhj has quit IRC15:34
scheuranmatrohon, yes I know - I failed rebasing it :/15:34
scheuranmatrohon, need to ask Bob to do so15:34
dougwigat least with the new gerrit, i get to re-sign in about every 17 minutes.15:34
*** links has joined #openstack-neutron15:36
*** sc68cal has quit IRC15:36
matrohonscheuran, you think it would improve your patch? It will only give two binding to nova, and nova will use the one for the new host15:36
*** iyamahat has quit IRC15:36
matrohonscheuran, no?15:36
*** EinstCrazy has quit IRC15:36
*** tmorin has quit IRC15:37
scheuranmatrohon, it wouldn't make a difference for the reference implementations15:37
*** achanda has quit IRC15:37
scheuranmatrohon, but probably for some external drivers, that would shut down a port and rebuild it on the new host when the host_id is changed15:38
scheuranmatrohon, for those cases it would improve the patch, yes15:38
scheuranmatrohon, nova then could just request another binding in pre_live_migration15:38
vikramarmax: ping15:38
scheuranmatrohon, and remove the old binding in post_live_migration15:38
armaxvikram: pong15:38
matrohonscheuran, yep makes sense15:38
ihrachysramishra: let's revert15:39
*** moshele has quit IRC15:39
vikramarmax: need your approval for "https://review.openstack.org/#/c/268726/11"15:39
armaxI have it on my stack of things to review15:39
vikramarmax: thanks ;)15:40
matrohonscheuran, 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 release15:40
*** ababich has quit IRC15:40
vikramarmax: One more "https://review.openstack.org/#/c/302120/"  ;)15:40
*** ababich1 has joined #openstack-neutron15:40
openstackgerritIhar Hrachyshka proposed openstack/neutron: Revert "standard attributes: expose created_at/updated_at on models"  https://review.openstack.org/30290415:40
ihrachysramishra: ^15:40
ihrachysjlibosva1: ^15:40
vikramarmax: I think we need to resume our decision about the Stadium project creation15:41
jlibosva1=-O15:41
scheuranmatrohon, yes, probably. I think we need to discuss this with a wider audience..15:41
*** jlibosva1 is now known as jlibosva15:41
scheuranmatrohon, I plan to do some of the discussions during the summit..15:41
scheuranmatrohon, will you attend?15:41
armaxvikram: no need for me to review that one15:41
armaxvikram: as for the stadium, yes I know that we do, thanks for the kind reminder15:41
vikramarmax: ok15:41
matrohonscheuran, great!15:41
openstackgerritNate Johnston proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies  https://review.openstack.org/21709215:42
matrohonscheuran, yes15:42
vikramarmax: thanks for your help15:42
matrohonscheuran, I saw you submitted a design session about it15:42
electrocucarachaihrachys: what should be the topic for this one -> https://review.openstack.org/#/c/301634/15:42
matrohonscheuran, hope it will be accepted15:42
matrohonscheuran, but it also needs to be blessed by nova guys15:43
matrohonscheuran, is there any nova session about that?15:43
scheuranmatrohon, yes, I did - marked it as nova-neutron session15:43
scheuranmatrohon, I started attending the weekly nova live migration meetings :)15:43
matrohonscheuran, great!15:43
scheuranmatrohon, so those guys are up to date...15:44
ihrachyselectrocucaracha: it is not strictly ovo, it's just related, so no idea and I don't care that much :)15:44
*** sarun has quit IRC15:44
*** afranc has joined #openstack-neutron15:44
scheuranmatrohon, 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 IRC15:45
davidshaajo: ping15:45
ajodavidsha, pong15:45
davidshaajo: good evening15:46
*** mickeys has quit IRC15:46
electrocucarachaihrachys: yes, I know that it doesn't matter but just curiosity...15:46
davidshaajo: Did you get njohnstons email?15:46
ajodavidsha, for the presentation?, yes :)15:46
*** eddima has quit IRC15:46
ajodavidsha, I just resent it to me to look at it tomorrow :)15:46
ajoas I ran out of time today15:46
ajoon the high level it was looking good, beutiful in fact15:47
ajobeautiful15:47
njohnstonajo: Thanks!15:47
*** scheuran has quit IRC15:47
davidshaajo: Ok, no problem!15:47
ajogood work, I should start doing the same for our other presentation :)15:47
ihrachysramishra: how do I reproduce locally?15:47
*** ababich1 has quit IRC15:48
openstackgerritMerged openstack/neutron: Fix keepalived functional tests  https://review.openstack.org/30242115:48
*** links has quit IRC15:48
*** ababich has joined #openstack-neutron15:48
*** ramishra has quit IRC15:48
*** baoli has joined #openstack-neutron15:49
davidshanjohnston: actually, I was wondering should you do slide 32/33 as I wasn't involved in this part?15:49
*** asingh has quit IRC15:49
ihrachysjlibosva: can you craft the partial revert?15:49
*** asingh has joined #openstack-neutron15:49
*** matrohon has quit IRC15:49
*** admin0 has quit IRC15:49
ihrachysjlibosva: I want to test something different in next 10 mins till a meeting15:49
jlibosvaihrachys: I can try but not sure I'll make it. I have one patch in progress that I'd like to push15:50
*** ababich has quit IRC15:50
*** ababich has joined #openstack-neutron15:50
njohnstondavidsha: OK, so that's mostly the "what was omitted from the reference implementation" slide?  I can definitely go over that bit, yes.15:51
davidshanjohnston: yup that's the one, Thanks!15:52
ihrachysjlibosva: I see.15:52
jlibosvaihrachys: I'm on it15:53
openstackgerritMartin Hickey proposed openstack/neutron: Integrate the Extra Dhcp Opt VersionedObject in Neutron  https://review.openstack.org/28539715:53
ihrachysthanks15:53
*** pcaruana has quit IRC15:53
*** vthapar has quit IRC15:53
ihrachysjlibosva: 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-neutron15:54
jlibosvaihrachys: when reading from proxy?15:54
ihrachysjlibosva: yeah15:54
jlibosvaihrachys: even when it was obtained from db?15:54
jlibosvaihrachys: even when it was obtained from db?15:55
ihrachysjlibosva: it is retrieved from there in the test15:55
jlibosvasorry15:55
ihrachysI refetched15:55
ihrachysjlibosva: and why is it just lbaas, not neutron gate?15:56
openstackgerritJakub Libosvar proposed openstack/neutron: Fix regexp for ss output  https://review.openstack.org/30291315:58
*** sridhar_ram has joined #openstack-neutron15:58
*** ygbo has quit IRC15:59
*** garyk has joined #openstack-neutron15:59
*** minwang2 has quit IRC16:00
*** nmagnezi has joined #openstack-neutron16:01
openstackgerritJakub Libosvar proposed openstack/neutron: Fetch timestamps via standard attribute model  https://review.openstack.org/30290416:01
jlibosvaihrachys: ^^16:01
*** ametts has quit IRC16:01
*** javeriak has joined #openstack-neutron16:02
*** armax has quit IRC16:02
*** armax has joined #openstack-neutron16:02
jlibosvaihrachys: do you have a link with the failure in lbaas?16:03
ihrachysjlibosva: not but there is magnum gate broken too16:03
*** dougs has joined #openstack-neutron16:03
ihrachysjlibosva: see the openstack-dev16:03
*** dave-mccowan has joined #openstack-neutron16:03
*** yottatsa has quit IRC16:04
ihrachysjlibosva: http://logs.openstack.org/91/301891/1/check/gate-functional-dsvm-magnum-api/ea0d4ba/logs/screen-q-lbaas.txt.gz16:04
jlibosvaihrachys: thanks16:05
ihrachysthe 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_49516:05
*** dougs1 has quit IRC16:05
*** julim has quit IRC16:05
*** itamarl has quit IRC16:06
*** yamamoto has quit IRC16:06
*** mgoddard_ has quit IRC16:06
*** marios_ has joined #openstack-neutron16:06
*** yamahata has joined #openstack-neutron16:08
*** akshai has quit IRC16:08
*** mgoddard has joined #openstack-neutron16:08
*** marios has quit IRC16:08
*** gvrangan has joined #openstack-neutron16:09
*** rickyrem has quit IRC16:09
*** marios_ is now known as marios16:10
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/30286516:10
openstackgerritRossella Sblendido proposed openstack/neutron: SUPER WIP OVO port object  https://review.openstack.org/25364116:11
openstackgerritRossella Sblendido proposed openstack/neutron: Port allowed address pairs to OVO  https://review.openstack.org/26827416:12
*** wolverineav has joined #openstack-neutron16:13
*** ekuris has quit IRC16:13
*** ababich has quit IRC16:13
*** rickyrem has joined #openstack-neutron16:14
*** ababich has joined #openstack-neutron16:14
*** wolverineav has quit IRC16:17
*** ametts has joined #openstack-neutron16:17
*** ramishra has joined #openstack-neutron16:17
*** mhickey has quit IRC16:17
*** nplanel has quit IRC16:18
*** jlanoux_ has quit IRC16:19
*** tbachman_ has joined #openstack-neutron16:23
*** asingh has quit IRC16:24
*** tbachman has quit IRC16:25
*** tbachman_ is now known as tbachman16:25
*** davidsha has quit IRC16:26
*** ekuris has joined #openstack-neutron16:26
*** shashank_hegde has joined #openstack-neutron16:27
*** daneyon_ has quit IRC16:28
*** daneyon has joined #openstack-neutron16:29
*** neiljerram has quit IRC16:31
*** john-davidge has joined #openstack-neutron16:31
*** minwang2 has joined #openstack-neutron16:31
*** sridhar_ram has quit IRC16:32
*** moshele has joined #openstack-neutron16:34
*** devvesa has quit IRC16:34
*** Leo_ has joined #openstack-neutron16:35
*** haukebruno has quit IRC16:35
*** slaweq__ has joined #openstack-neutron16:36
*** david-lyle has quit IRC16:36
*** mfranc213 has joined #openstack-neutron16:36
*** haplo37 has joined #openstack-neutron16:37
*** slaweq__ is now known as slaweq_16:37
armaxihrachys:16:38
armaxping16:38
ihrachysarmax: pong16:38
armaxregarding https://review.openstack.org/#/c/302904/16:38
ihrachysarmax: shoot16:38
armaxI am confused16:38
*** sarun has joined #openstack-neutron16:39
*** aranjan has joined #openstack-neutron16:39
armaxhow is it that we haven’t caught this up before?16:39
armaxand how does it show up?16:39
ihrachysarmax: that's a question I also wonder. we have api tests for timestamps.16:39
ihrachysarmax: it seems it shows up in lbaas gates only (?)16:39
ihrachysarmax: 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_49516:39
*** ushkalim has quit IRC16:40
armaxic16:40
ihrachysarmax: it seems that when server fetches .created_at from the model itself (thru proxy), it gets string.16:40
armaxgood thing is that this isn’t mitaka16:40
ihrachysfor some reason16:40
armaxphew16:40
ihrachysbut I can't repro it with a unit test.16:40
armaxI almost had a heath attach16:40
armaxattack16:40
ihrachysarmax: 8.0.0 tags are merged.16:40
*** neiljerram has joined #openstack-neutron16:40
ihrachysarmax: so you can relax anyway :)16:40
*** dslevin1 has quit IRC16:41
armaxwell I figured we had merged them whilst being broken!16:41
ihrachysarmax: we released mitaka without https://review.openstack.org/#/c/302728/ though16:41
ihrachyswhich sucks, but well...16:41
ihrachysshould have been a reelase blocker16:42
ihrachysnot sure why it was identified as Low priority16:42
*** ekuris has quit IRC16:42
armaxblame HenryG16:42
*** shashank_hegde has quit IRC16:43
*** javeriak_ has joined #openstack-neutron16:44
ihrachysme gets ready to blame!16:44
armaxihrachys: though I am not quite sure what the failure is16:44
ihrachysarmax: in the patch I posted?16:44
*** johnbelamaric has left #openstack-neutron16:44
armaxihrachys: no, associated to 154843316:44
armaxbug 154843316:44
openstackbug 1548433 in keystoneauth "neutron returns objects other than oslo_config.cfg.Opt instances from list_opts" [Undecided,Incomplete] https://launchpad.net/bugs/154843316:44
ihrachysarmax: right. the bug shows up as a broken config file (auth section)16:45
*** javeriak has quit IRC16:45
ihrachysneutron still works, but the config file is not ideal16:45
*** Leo_ has quit IRC16:45
ihrachysarmax: example snippet from bad config file at: https://review.openstack.org/#/c/292640/16:46
*** Leo_ has joined #openstack-neutron16:48
*** kickinz1 is now known as kickinz1|eod16:48
armaxihrachys: ic16:48
*** Leom has joined #openstack-neutron16:49
amullerZZelle: fullstack fix of the day: https://review.openstack.org/#/c/302913/16:50
*** SteffanW has joined #openstack-neutron16:51
armaxihrachys: when do those warning get emitted precisely?16:51
amullerhmm, check queue is very full today16:51
*** asingh has joined #openstack-neutron16:51
armaxihrachys: shouldn’t they show up in the conf files that we generate during the CI job runs?16:51
ihrachysarmax: I suspect they are there. let me check.16:52
armaxI couldn’t find any16:52
*** rcernin has quit IRC16:52
armaxihrachys: but I didn’t look hard enough16:52
*** mickeys has joined #openstack-neutron16:52
*** Leo_ has quit IRC16:52
ihrachysarmax: 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.gz16:53
*** julim has joined #openstack-neutron16:53
*** marios_ has joined #openstack-neutron16:53
armaxihrachys: how about that16:54
armaxihrachys: 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.gz16:54
slaweq_ajo: ping16:54
armaxihrachys: see? I wasn’t lying16:54
ihrachysarmax: it's master?16:54
*** marios has quit IRC16:54
armaxihrachys: yes16:54
ihrachysarmax: well master is fixed now16:54
*** marios_ is now known as marios16:55
ihrachysanyhow16:55
*** ZZelle_ has joined #openstack-neutron16:55
armaxihrachys: indeed16:55
* ihrachys tries to understand how lbaas is broken by that innocent change16:55
armaxihrachys: I thought it was a couple of days old16:55
armaxihrachys: thanks16:55
ihrachysarmax: also in case you are not aware: https://review.openstack.org/#/c/302737/16:56
ihrachysarmax: new fixtures broke us16:56
ihrachysbut it's under control now16:56
armaxihrachys: I saw16:56
*** mickeys has quit IRC16:56
ihrachysok16:56
armaxihrachys: thanks16:56
armaxihrachys: it’s been a great awaking this morning16:56
ihrachys:D16:56
ihrachysbut see how constraints guarded us in this case!16:57
*** mickeys has joined #openstack-neutron16:57
*** abregman has quit IRC16:57
*** hdaniel has quit IRC16:58
*** mohankumar__ has quit IRC16:59
*** mohankumar__ has joined #openstack-neutron16:59
*** s3wong has joined #openstack-neutron17:00
*** admin0 has joined #openstack-neutron17:00
*** oanson has joined #openstack-neutron17:01
*** asingh has quit IRC17:01
*** mickeys has quit IRC17:01
*** jistr has quit IRC17:01
*** asingh has joined #openstack-neutron17:01
*** neelashah has joined #openstack-neutron17:02
*** rossella_s has quit IRC17:03
*** rossella_s has joined #openstack-neutron17:03
*** jpena is now known as jpena|off17:03
*** iyamahat has joined #openstack-neutron17:03
garykihrachys: you around?17:04
*** tmorin has joined #openstack-neutron17:05
ihrachysgaryk: sure17:05
*** azbiswas has joined #openstack-neutron17:05
*** ramishra has quit IRC17:05
*** yamahata has quit IRC17:06
*** david-lyle has joined #openstack-neutron17:06
*** oshvartz has joined #openstack-neutron17:06
*** oanson has quit IRC17:06
*** ametts has quit IRC17:07
*** ramu_ramamurthy has joined #openstack-neutron17:07
*** yamamoto has joined #openstack-neutron17:07
*** rmart04 has quit IRC17:07
*** pcaruana has joined #openstack-neutron17:08
ihrachysarmax: huh. upper-constraints just bumped fixtures: https://review.openstack.org/#/c/302539/2/upper-constraints.txt17:09
ihrachysso now the gate is officially broken17:09
ihrachysdims: ^17:09
ihrachysI started getting first -1s from Jenkins17:10
armaxlovely17:10
ihrachysyeah17:10
*** david-lyle has quit IRC17:11
*** bjornar has quit IRC17:11
armaxihrachys: looks like garyk’ change didn’t go deep enough?17:11
armaxihrachys: http://logs.openstack.org/37/302737/2/gate/gate-neutron-python34/dd413b3/testr_results.html.gz17:11
amullerWell you know what they say, it's not a Thursday without a gate breakage17:11
armaxamuller: or Friday, or Wednesday17:11
amullerSame for all other days as well17:11
amullerarmax: heh yeah17:11
ihrachysarmax: pff17:12
ihrachysarmax: I guess I will just switch to mock17:12
ihrachysit's so much more sane17:12
armaxihrachys: we shall just switch jobs17:12
armaxihrachys: I hear the tomato picking industry is booming17:12
ihrachysarmax: tomatoes?17:12
ihrachysYES17:12
*** salv-orlando has joined #openstack-neutron17:12
ihrachyscrude tomatoes17:13
armaxihrachys: under the scorching sun17:13
*** wolverineav has joined #openstack-neutron17:13
*** kevo has quit IRC17:13
*** oanson has joined #openstack-neutron17:14
ihrachysstill better17:14
garykarmax: the python 3 tests passed and then failed. i have no idea why.17:14
ihrachysgaryk: magic17:14
ihrachysI am crafting a switch to mock17:14
armaxgaryk: it’s worth seeing whether it’s an order issue17:14
*** jlibosva has quit IRC17:15
garykarmax: yes, it could be. something is fishy17:15
*** salv-orlando has quit IRC17:15
*** p01nt3r75 has quit IRC17:15
*** yamamoto has quit IRC17:16
*** baoli has quit IRC17:16
*** baoli has joined #openstack-neutron17:17
*** fitoduarte has joined #openstack-neutron17:17
dimsihrachys : y, those are bot proposed updates17:17
*** david-lyle has joined #openstack-neutron17:17
garykone thing for sure is that the code in the neutron lib copying this stuff is also broken.17:17
garyki will address that too.17:17
garykis that project still alive or can we kill it :)17:17
ihrachysdims: yeah, but aren't we supposed to at least give a sec waiting before  landing17:17
ihrachysdims: should we add neutron unit tests into constraints gate?17:18
ihrachysarmax: garyk: https://review.openstack.org/302997 mock version17:18
*** Marga_ has quit IRC17:19
garykok, thanks.17:19
ihrachysgaryk: huh, your fix was first time gated against OLD fixtures17:19
dimsihrachys : explain please? bot proposes updates, review runs some ci jobs, person eyeballs the results and merges..that's the protocol with requirements bot17:19
ihrachysgaryk: now that constraints bumped, your gate run caught the new one17:19
ihrachysand failed on it17:19
garyk:)17:20
ihrachysdims: ok, but it then shows there is gating lacking for those version bumps17:20
*** fitoduarte has quit IRC17:20
dimslifeless : fixtures 2.0.0 broke neutron jobs...fyi17:20
ihrachysdims: if neutron unit tests would run in the gate, you would see the CI failing17:20
*** ametts has joined #openstack-neutron17:20
*** Prem_ has joined #openstack-neutron17:21
dimsihrachys : i'll let you take that up with lifeless :)17:21
ihrachysdims: that's known, the release is explicitly breaking api17:21
dimsihrachys : then you can propose a blacklist or a cap in global-requirements17:22
dimsagain, that's the work flow. u-c/bot does not catch everything17:22
ihrachysdims: 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 IRC17:23
ihrachysI naively thought that we will pick up a representative list of jobs from other projects to gate against for constraints bumps.17:23
ihrachyswasn't it the plan?17:23
*** david-lyle has quit IRC17:23
dimsihrachys : the test matrix needs work, but i don't know if unit tests would make the cut17:25
*** Marga_ has joined #openstack-neutron17:26
*** lnicolas has joined #openstack-neutron17:29
*** banix has joined #openstack-neutron17:30
openstackgerritSlawek Kaplonski proposed openstack/neutron: LinuxBridge agent's QoS driver bw limit for egress traffic  https://review.openstack.org/30021017:30
*** sambetts is now known as sambetts|afk17:30
ihrachysWAT? I rebased a patch thru gerrit, and it's now empty17:31
dimsihrachys : 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-neutron17:31
ihrachysdims: I am heading out in a sec. just need to stack gate fixes properly17:31
*** abhiraut has joined #openstack-neutron17:31
dimsack ihrachys17:31
*** tbh has joined #openstack-neutron17:32
*** amuller is now known as amuller_afk17:32
openstackgerritArmando Migliaccio proposed openstack/neutron: Tempest 11.0.0 is released  https://review.openstack.org/30300517:32
armaxihrachys: so your change supersedes garyk's?17:33
bharathihrachys: 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-neutron17:33
ihrachysarmax: that's the plan17:33
*** moshele has quit IRC17:33
*** abhiraut has quit IRC17:34
openstackgerritIhar Hrachyshka proposed openstack/neutron: Fetch timestamps via standard attribute model  https://review.openstack.org/30290417:35
openstackgerritIhar Hrachyshka proposed openstack/neutron: Switched from fixtures to mock to mock out starting RPC consumers  https://review.openstack.org/30299717:35
*** fitoduarte has joined #openstack-neutron17: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
ihrachysarmax: gerrit screwed patch rebase somehow, so I needed to respin ^ :-|17:36
armaxihrachys: are you sure that the switch to mock doesn’t have its own side effects?17:37
*** abhiraut has joined #openstack-neutron17:38
ihrachysarmax: I tested it today, it seemed to work. you never know. I am ok if someone pursues fixtures version in parallel17:38
*** wolverineav has quit IRC17:38
ihrachysarmax: sorry, I need to run home, it's late here.17:38
armaxihrachys: it’s the ‘never know’ attitude that keeps us in this mess though :)17:38
armaxihrachys: no worries, thanks17:38
armaxZZelle: is there any chance you can take over ihrachys’ change?17:39
armax^17:39
armaxZZelle: https://review.openstack.org/#/c/302997/17:39
*** mickeys has joined #openstack-neutron17:39
ihrachystwo changes actually, one general gate breakage, another one for lbaas17:39
ZZelle_armax, looking17:39
ihrachyslbaas https://review.openstack.org/#/c/30290417:39
ihrachysI stacked them17:39
*** shivrao has joined #openstack-neutron17:40
armaxwe should have kevinbenton review that one17:40
armaxhttps://review.openstack.org/#/c/302904/ I mean17:40
ihrachysarmax: yeah. maybe he even has an idea of the culprit.17:41
ihrachysok, now off17:41
*** ihrachys has quit IRC17:41
*** lezbar has joined #openstack-neutron17:42
*** lezbar__ has quit IRC17:42
garykarmax: if ihars passes then we need to abandon mine17:44
*** mvk has quit IRC17:45
*** shashank_hegde has joined #openstack-neutron17:46
*** mickeys has quit IRC17:46
*** kevo has joined #openstack-neutron17:47
*** mickeys has joined #openstack-neutron17:47
*** hongbin has left #openstack-neutron17:48
*** aranjan has quit IRC17:49
*** Leom has quit IRC17:50
*** davidsha has joined #openstack-neutron17:50
*** SumitNaiksatam has joined #openstack-neutron17:51
*** rmart04 has joined #openstack-neutron17:52
*** tflynn has joined #openstack-neutron17:53
*** mickeys has quit IRC17:54
kevinbentonarmax: you want me to look at 302904?17:55
*** mickeys has joined #openstack-neutron17:55
*** david-lyle has joined #openstack-neutron17:56
*** david-lyle has quit IRC17: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-neutron17:57
*** david-lyle has joined #openstack-neutron17:58
*** wolverineav has joined #openstack-neutron17:58
*** tbachman has quit IRC17:58
*** vikram has quit IRC18:00
*** jprovazn has quit IRC18:00
*** bjornar has joined #openstack-neutron18:01
*** aranjan has joined #openstack-neutron18:01
manjeetsis there any specific version requirement of setuptools when some body uses linux bridge ?18:03
*** julim has quit IRC18:04
manjeetsI am trying that even though its getting installed but linuxbridge is not working properly18:04
manjeetshttp://paste.openstack.org/show/493382/18:04
*** ociuhandu has quit IRC18:04
manjeetsalso seeing this warning for lb container http://paste.openstack.org/show/493385/18:05
*** wolverineav has quit IRC18:06
*** wolverineav has joined #openstack-neutron18:07
*** davidsha has left #openstack-neutron18:07
*** haplo37 has quit IRC18:08
*** jaimguer has joined #openstack-neutron18:08
openstackgerritgaryk proposed openstack/neutron: Fix broken unit tests  https://review.openstack.org/30273718:10
*** ankur-gupta-f has joined #openstack-neutron18:10
*** mfranc21_ has joined #openstack-neutron18:12
*** sridhar_ram has joined #openstack-neutron18:14
*** mfranc213 has quit IRC18:15
*** aranjan has quit IRC18:16
*** aranjan has joined #openstack-neutron18:17
bharathneutron-vpnaas tox api are failing with tempest.lib.NotFound . Is it know issue?18:17
*** javeriak_ has quit IRC18:17
*** efoley has joined #openstack-neutron18:17
*** admin0 has quit IRC18:17
*** amuller_afk is now known as amuller18:19
openstackgerritArtur Korzeniewski proposed openstack/neutron: Introducing the Subnet OVO  https://review.openstack.org/26427318:20
*** ociuhandu has joined #openstack-neutron18:21
*** Marga_ has quit IRC18:24
*** oanson has quit IRC18:24
*** korzen has quit IRC18:25
*** doug-fish has joined #openstack-neutron18:26
kevinbentonarmax: ah, it's because lbaas calls get_port, changes the relevant fields, and then sends the entire dict into an update_port call18:26
*** mohankumar__ has quit IRC18:26
kevinbentonarmax: so the association proxy thinks updated_at and created_at are being set18:27
*** gvrangan has quit IRC18:28
*** s3wong has quit IRC18:32
*** salv-orlando has joined #openstack-neutron18:32
*** yfried has joined #openstack-neutron18:34
*** tonytan4ever has quit IRC18:34
*** tonytan4ever has joined #openstack-neutron18:34
*** salv-orlando has quit IRC18:38
*** nihilifer has quit IRC18:39
*** saggi has quit IRC18:40
*** admin0 has joined #openstack-neutron18:40
*** akshai has joined #openstack-neutron18:42
*** sambetts|afk has quit IRC18:42
*** abhiraut has quit IRC18:45
*** Leo_ has joined #openstack-neutron18:45
*** sambetts has joined #openstack-neutron18:46
*** mickeys has quit IRC18:46
*** abhiraut has joined #openstack-neutron18:46
*** mickeys has joined #openstack-neutron18:47
*** hynekm has quit IRC18:49
*** gvrangan has joined #openstack-neutron18:49
*** njohnston has quit IRC18:49
*** SumitNaiksatam has left #openstack-neutron18:51
*** mickeys has quit IRC18:51
*** salv-orlando has joined #openstack-neutron18:52
adreznecmestery: 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 possible18:55
openstackLaunchpad 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 IRC18:56
*** neelashah has joined #openstack-neutron18:56
*** janzian has quit IRC18:59
*** sambetts has quit IRC19:00
*** kawa2014 has quit IRC19:00
*** salv-orlando has quit IRC19:01
*** emagana has joined #openstack-neutron19:02
*** abhiraut has quit IRC19:02
mesteryadreznec: I've assigned that to ihar, he'll get to it tomorrow I imagine19:03
adreznecmestery: Sure, not a problem. Thanks!19:03
*** sambetts has joined #openstack-neutron19:03
blogankevinbenton: does lbaas code need to change?19:04
kevinbentonblogan: no19:05
kevinbentonblogan: the timestamp code should be able to handle this19:05
*** admin0 has quit IRC19:06
*** sarun has quit IRC19:06
*** sarun has joined #openstack-neutron19:07
openstackgerritKevin Benton proposed openstack/neutron: Ignore timestamp fields in model update  https://review.openstack.org/30304219:10
*** rickyrem has quit IRC19:10
kevinbentonblogan: ^^19:10
kevinbentonarmax: ^^~119:10
*** janzian has joined #openstack-neutron19:12
blogankevinbenton: i can't seem to find where the failure was19:12
bloganin the review armax linked19:12
*** david-lyle_ has joined #openstack-neutron19:13
*** rickyrem has joined #openstack-neutron19:14
*** salv-orlando has joined #openstack-neutron19:14
*** Marga_ has joined #openstack-neutron19:14
*** tbh has quit IRC19:14
*** david-lyle has quit IRC19:15
*** SteffanW has quit IRC19:17
*** iyamahat has quit IRC19:18
*** tmorin has quit IRC19:18
*** hdaniel has joined #openstack-neutron19:18
*** Sukhdev has joined #openstack-neutron19:18
*** rickyrem has quit IRC19:18
*** david-lyle_ has quit IRC19:19
kevinbentonblogan: i think it's sort of skirting around the issue19:21
*** shivrao has quit IRC19:24
*** ihrachys has joined #openstack-neutron19:24
*** tbachman has joined #openstack-neutron19:25
ihrachysarmax: so we don't land fixtures patch?19:25
ihrachysarmax: if not, we need to rebase the one fixing lbaas gate.19:26
*** janzian has quit IRC19:26
*** tmorin has joined #openstack-neutron19:26
armaxihrachys: if the g-r cap is in place19:26
armaxihrachys: I think it’s probably best let this bake a bit more?19:27
armaxihrachys: and figure out a way to continue to use fixtures?19:27
ihrachysarmax: 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 fragile19:27
ihrachysarmax: let me untangle the lbaas patch19:28
kevinbentonmanjeets: what's the rest of that linux bridge traceback?19:28
kevinbentonmanjeets: that paste doesn't have the error19:29
openstackgerritIhar Hrachyshka proposed openstack/neutron: Fetch timestamps via standard attribute model  https://review.openstack.org/30290419:29
ihrachysarmax: untangled ^19:29
ihrachyskevinbenton: would really love you checking the bug and the patch ^ maybe you will know the culprit from the first spot19:29
*** david-lyle has joined #openstack-neutron19:29
kevinbentonihrachys: https://review.openstack.org/30304219:29
kevinbentonihrachys: culprit is that update_port is setting the timestamp fields to the strings passed into it19:29
*** zhhuabj has quit IRC19:30
*** bharath has quit IRC19:30
ihrachyskevinbenton: oh nice. looking.19:30
kevinbentonajo: still around?19:30
*** david-lyle has quit IRC19:31
blogankevinbenton: your patch won't ever allow the timestamps to be updated in code?19:31
*** rickyrem has joined #openstack-neutron19:31
kevinbentonblogan: not by passing in a dict to port19:31
ihrachyskevinbenton: oh that's a great explanation of what we see19:31
ihrachysblogan: you are not supposed to update it yourself19:31
*** david-lyle has joined #openstack-neutron19:31
ihrachysonly thru events19:31
*** minwang2 has quit IRC19:31
bloganah okay, just wondering if there was a scenario where that'd be needed19:32
ihrachyskevinbenton: have we validated the fix by some 'do not merge' patch in e.g. magnum?19:32
*** clickboom has joined #openstack-neutron19:32
*** mvk has joined #openstack-neutron19:32
*** panda has quit IRC19:33
*** shivrao has joined #openstack-neutron19:33
*** julim has joined #openstack-neutron19:33
kevinbentonihrachys: no, i wasn't sure where the failures where19:33
kevinbentonihrachys: lbaas was in the example19:33
ihrachyskevinbenton: ok, lemme craft one19:33
kevinbentonihrachys: but i wasn't sure if that was current lbaas gate19:33
*** panda has joined #openstack-neutron19:33
*** clickboom is now known as aabradsh19:35
*** tonytan_brb has joined #openstack-neutron19:35
*** aabradsh has quit IRC19:35
*** mvk_ has joined #openstack-neutron19:35
kevinbentonblogan: yeah, updating with code will be difficult even without this. because the db event would override it19:35
ihrachyskevinbenton: I pushed https://review.openstack.org/#/c/303046/19:36
kevinbentonblogan: if someone wants to update the timestamp in code, it will have to be with a manual SQL statement19:36
lifelessihrachys: dims: need to get unit tests voting on requirements changes to avoid this19:36
*** tonytan4ever has quit IRC19:36
dimslifeless : which projects? :)19:36
ihrachyslifeless: fully agreed. neutron would be happy to be the one whose unit tests are to be executed ;)19:36
*** banix has quit IRC19:36
*** janzian has joined #openstack-neutron19:37
ihrachysdims: some huge/core/representative ;)19:37
dimsihrachys : why just neutron? :)19:37
ihrachysdims: because we are cool!19:37
dimsLOL, yes you are19:37
*** david-lyle has quit IRC19:37
ihrachysdims: well, there may be an assumption that most breaking cases affect the whole type of a job.19:37
ihrachysdims: like fixtures release broke neutron and nova unit tests19:38
ihrachysor new hacking may break all pep8 jobs19:38
openstackgerritMerged openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/30288319:38
ihrachysdims: also, neutron was the first project to adopt constraints ;)19:38
*** mvk has quit IRC19:39
lifelessdims: the ones that don't want to suffer the slings and arrows of outrageous fortune19:39
*** tonytan_brb has quit IRC19:39
dimsihrachys : sold!19:39
*** baoli has joined #openstack-neutron19:39
* ihrachys crafts a straw man for project-config19:39
*** tonytan4ever has joined #openstack-neutron19:39
*** tmorin has quit IRC19:40
*** iyamahat has joined #openstack-neutron19:41
*** minwang2 has joined #openstack-neutron19:42
*** abhiraut has joined #openstack-neutron19:42
*** zhhuabj has joined #openstack-neutron19:43
*** clickboom has joined #openstack-neutron19:43
*** abhiraut has quit IRC19:44
*** mfranc21_ has quit IRC19:45
*** bharath has joined #openstack-neutron19:48
*** efoley has quit IRC19:50
*** mfranc213 has joined #openstack-neutron19:52
*** alex_xu has quit IRC19:54
*** ivase has joined #openstack-neutron19:54
*** alex_xu has joined #openstack-neutron19:55
*** tonytan_brb has joined #openstack-neutron19:55
*** vhosakot has quit IRC19:58
ZZelle_ihrachys, hi19:58
ihrachysZZelle_: hi19:58
ZZelle_ihrachys, about https://review.openstack.org/#/c/302913/1/neutron/tests/common/net_helpers.py19:58
*** tmorin has joined #openstack-neutron19:58
*** vhosakot has joined #openstack-neutron19:58
*** john-davidge has quit IRC19:58
*** tonytan4ever has quit IRC19:59
ZZelle_ihrachys, why not using: r'^.*\s+\d+\s+.*:(?P<port>\d+)\s'19:59
*** yamahata has joined #openstack-neutron20:00
*** hdaniel has quit IRC20:00
ZZelle_we don't really care about peer information20:01
blogankevinbenton: thats what i figured, so the update timestamp being updated just relies on events20:01
ihrachysZZelle_: well yeah, it could also work. I don't mind.20:01
*** ametts has quit IRC20:01
*** vhosakot has quit IRC20:01
kevinbentonWHO HAS BEEN SNEAKING REGEX INTO NEUTRON!20:02
kevinbenton:)20:02
armaxregXboi: did I recall correctly or you own me an answer?20:02
*** tbachman_ has joined #openstack-neutron20:02
*** tbachman has quit IRC20:02
*** tbachman_ is now known as tbachman20:02
* regXboi reads scrollback20:02
regXboikevinbenton: you could accuse me... but I'm innocent on this one :)20:03
armaxregXboi: I mean from last week20:03
ihrachyskevinbenton: relax it's just tests. those irrelevant things that make the whole project down20:03
*** baoli has quit IRC20:03
*** baoli has joined #openstack-neutron20:04
regXboiarmax: yes I owe you an answer, but I've not gotten an answer back20:04
*** tbachman has quit IRC20:04
regXboiarmax: at this point, I'd say propose the patch to remove it and see who screams :)20:05
*** hdaniel has joined #openstack-neutron20:05
armaxregXboi: ack20:06
manjeetskevinbenton http://paste.openstack.org/show/493412/20:06
manjeetssorry my previous trace missed last few lines20:07
*** mkrcmari__ has joined #openstack-neutron20:07
kevinbentonmanjeets: sounds like kolla may be missing a dependency20:07
kevinbentonmanjeets: ebtables20:07
*** jpena|off is now known as jpena20:07
*** wolverin_ has joined #openstack-neutron20:07
manjeetsok is it gettable via pip20:08
*** jpena is now known as jpena|off20:08
openstackgerritSlawek Kaplonski proposed openstack/neutron: Change name of QoS bandwidth limit burst parameter to kilobits  https://review.openstack.org/29163320:09
kevinbentonmanjeets: it's not a python package20:09
*** dane_leblanc has quit IRC20:09
*** jpena|off is now known as jpena20:09
*** tmorin has quit IRC20:09
kevinbentonmanjeets: its a distro package20:09
manjeetskevinbenton thanks ill get that see if had same issue20:10
manjeetsthanks very much20:10
manjeetskevinbenton20:10
*** dane_leblanc has joined #openstack-neutron20:10
*** mvk_ has quit IRC20:10
kevinbentonmanjeets: no prob. kolla manages including system dependencies doesn't it?20:10
kevinbentonmanjeets: if so, a bug needs to be filed because they forgot to include it20:11
manjeetsit did really well for ovs  may be it missed something20:11
manjeetswill do that20:11
*** wolverineav has quit IRC20:11
*** ramu_ramamurthy has quit IRC20:13
*** efoley has joined #openstack-neutron20:13
openstackgerritAssaf Muller proposed openstack/neutron: WIP: Fix functional tests logging  https://review.openstack.org/30305320:13
amullerHenryG: ^20:13
HenryGamuller: wat?!20:14
amullerHenryG: :)20:14
amullerHenryG: turns out there were 2 different patches that disabled logging for functional tests20:14
*** vishwanathj has quit IRC20:14
amullerHenryG: The power of collaboration!20:15
*** gvrangan has quit IRC20:15
*** abhiraut has joined #openstack-neutron20:16
kfox1111has 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
blogankevinbenton: you wanna abandon this guy: https://review.openstack.org/#/c/29287320:16
HenryGamuller: do you have gerrit review links to the culprit patches?20:16
*** tbachman has joined #openstack-neutron20:16
amullerHenryG: they're in the bug report20:17
*** ametts has joined #openstack-neutron20:17
kfox1111its not mtu related. I've tested that bit. huge pings work.20:17
* HenryG grumbles at extra steps to get from hashes to reviews20:17
kevinbentonblogan: done20:18
amullerHenryG: I figured it's better to use commit hashes in commit messages because we might switch from Gerrit at some point in the future20:18
HenryGamuller: never!20:18
amullerHenryG: or maybe the URLs would change20:18
kevinbentonHenryG: if you're using gerrit you just click on the hash20:19
blogandoes gertty do the same?20:19
amullerkevinbenton: but I was cute and I used prefixes and not the full commit hash20:19
amullerkevinbenton: maybe I should use the full hashes =D20:19
HenryGkevinbenton: gerrit is not generating links for the short hashes in https://review.openstack.org/30305320:20
*** azbiswas has quit IRC20:20
kevinbentonamuller: full hashes!20:20
kevinbentonamuller: 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-neutron20:20
*** johnbelamaric has joined #openstack-neutron20:21
ihrachyslifeless: dims: armax: amuller: https://review.openstack.org/#/c/303054/ I *hope* it will work20:21
kevinbentonerror: short SHA1 2255 is ambiguous.20:21
kevinbentonwe've already had collisions on 4 chars :)20:21
kevinbentonoooh, even 5 chars error: short SHA1 22554 is ambiguous.20:22
blogandoesn't gerrit link across all the openstack projects for a hash too?20:23
*** Leo_ has quit IRC20:24
*** Leom has joined #openstack-neutron20:24
*** john-davidge has joined #openstack-neutron20:24
amullerkevinbenton: we could not be more off topic =p20:25
*** tonytan_brb is now known as tonytan4ever20:26
*** tmorin has quit IRC20:26
*** praveen has joined #openstack-neutron20:27
*** numan_ has quit IRC20:28
*** john-davidge has quit IRC20:29
*** tongli has joined #openstack-neutron20:29
*** vhosakot has joined #openstack-neutron20:31
*** tbachman has quit IRC20:31
*** tbachman has joined #openstack-neutron20:32
*** enriquetaso has quit IRC20:32
*** jpena is now known as jpena|off20:33
*** mickeys has joined #openstack-neutron20:34
*** Leom has quit IRC20:35
*** mickeys has quit IRC20:36
*** julim has quit IRC20:36
*** mickeys has joined #openstack-neutron20:36
*** bharath has quit IRC20:37
*** Leom has joined #openstack-neutron20:38
*** tmorin has joined #openstack-neutron20:38
*** john-davidge has joined #openstack-neutron20:39
*** kbringard has joined #openstack-neutron20:39
*** ametts has quit IRC20:41
*** baoli has quit IRC20:41
*** hdaniel has quit IRC20:41
*** jaimguer has quit IRC20:41
*** jaimguer has joined #openstack-neutron20:42
*** baoli has joined #openstack-neutron20:45
*** baoli_ has joined #openstack-neutron20:46
*** mvk_ has joined #openstack-neutron20:46
*** wolverin_ has quit IRC20:47
*** vishwana_ has joined #openstack-neutron20:47
*** Sukhdev has quit IRC20:47
*** gvrangan has joined #openstack-neutron20:47
*** wolverineav has joined #openstack-neutron20:47
lifelessihrachys: we're about to do a 2.1 - reducing the strictness; alaski is working on it20:48
ihrachyslifeless: I am scared enough already :P20:49
nmagnezifollowing to armax's mail about the drivers meeting, is it canceled for today or is it happening anyway?20:49
ihrachysnmagnezi: cancelled today.20:49
ihrachysso go have some sleep20:50
*** mkrcmari__ has quit IRC20:50
*** baoli has quit IRC20:50
nmagneziihrachys, :)20:50
nmagneziihrachys, it should do the same :)20:50
nmagneziyou*20:50
*** mkrcmari__ has joined #openstack-neutron20:50
nmagnezi(passed my working hours..)20:50
*** baoli_ has quit IRC20:50
*** baoli has joined #openstack-neutron20:51
*** ramu_ramamurthy has joined #openstack-neutron20:52
*** mvk_ has quit IRC20:54
*** nihilifer has joined #openstack-neutron20:54
openstackgerritAssaf Muller proposed openstack/neutron: Fix Windows IPDevice.device_has_ip racefulness  https://review.openstack.org/30307120:55
*** mvk_ has joined #openstack-neutron20:55
amullerihrachys: armax: more functional tests fixes ^20:56
*** david-lyle has joined #openstack-neutron20:56
amullerthis time for windows, kill me20:56
*** admin0 has joined #openstack-neutron20:56
kevinbentonamuller: that seems extreme20:56
*** abregman has joined #openstack-neutron20:57
*** marios has quit IRC20:57
kevinbentonpavel_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 deadlock20:59
*** mkrcmari__ has quit IRC20:59
kevinbentoneven if we change the current allocation behavior to a random choice, i don't think we would back-port that20:59
kevinbentonWhile 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 worker21:01
ihrachysamuller: you casted Berserk magic, right?21:01
kevinbentonsorry, i mean so only 1 allocation per server at a time21:01
*** pcaruana has quit IRC21:01
amullerihrachys: is that a magic the gathering reference?21:01
*** davideagnello has quit IRC21:02
*** davideagnello has joined #openstack-neutron21:02
*** rossella_s has quit IRC21:03
*** lucasagomes has quit IRC21:03
ihrachysamuller: http://heroes.thelazy.net/wiki/Berserk21:03
*** rossella_s has joined #openstack-neutron21:03
ihrachys"It makes the target of the spell to attack the nearest troop wheter it is an ally or enemy"21:03
amullerihrachys: omg heroes 321:04
amullerihrachys: wow this brings back memories21:04
*** fitoduarte has quit IRC21:04
ihrachysamuller: it's avail from steam btw21:04
amullerihrachys: for linux?21:04
ihrachysamuller: also ios if you are that kind of person21:04
ihrachysamuller: I think win32 only21:05
ihrachyshttp://store.steampowered.com/app/297000/21:05
amullerihrachys: my windows gaming pc is on another continent21:05
ihrachysit's offtopic day ;)21:05
carl_baldwinkevinbenton:  Are you seeing the deadlock besides when pluggable IPAM is turned on in the gate?21:06
*** clickboo_ has joined #openstack-neutron21:06
kevinbentoncarl_baldwin: not in gate21:07
kevinbentoncarl_baldwin: just for pluggable ipam and multi-writer galera21:07
openstackgerritCedric Brandily proposed openstack/neutron: Avoid multiple gw ports on the same router WIP  https://review.openstack.org/27402121:09
ZZelle_salv-orlando, hi21:09
carl_baldwinkevinbenton: 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 IRC21:10
*** tbachman has quit IRC21:10
kevinbentoncarl_baldwin: right, i think that's the only place we can get it to hit in the gate21:10
carl_baldwinkevinbenton: Is anyone actually asking for a backported fix?  (i.e. is anyone using the ref implementation of pluggable IPAM yet?21:10
carl_baldwinkevinbenton: I thought it was seen without galera.21:10
kevinbentoncarl_baldwin: there are two scenarios21:11
kevinbentoncarl_baldwin: one is pluggable ipam21:11
ZZelle_amuller, "my windows gaming pc", if you want minesweeper is also available on Linux :)21:11
kevinbentoncarl_baldwin: one it multi-writer galera with either ipam system21:11
*** lucasagomes has joined #openstack-neutron21:11
carl_baldwinkevinbenton: Ok, I see.21:11
*** nyechiel_ has quit IRC21:11
*** thorst_ has quit IRC21:12
carl_baldwinkevinbenton: Was that reported as a different issue or wrapped in with bug 1543094 ?21:12
openstackbug 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
kevinbentoncarl_baldwin: for example, fuel right now just deploys galera with only one active writer to avoid these kind of issues21:12
kevinbentoni don't think it was reported21:13
carl_baldwinI just hadn't noticed it reported specifically using multi-writer.21:13
carl_baldwinOh, okay.21:13
*** Devon has joined #openstack-neutron21:14
*** Devon has left #openstack-neutron21:14
kevinbentoncarl_baldwin: i don't think many people deploy multi-writer right now, so it might not be worth fixing21:14
*** clickboo_ has quit IRC21:15
tidwellrcarl_baldwin kevinbenton: I agree we wouldn't backport the randomized IP selection, I think the idea of a semaphore inside create_port is worth exploring21:15
kevinbentoncarl_baldwin, tidwellr: i did verify that it will at least unblock us in the gate as well21:16
kevinbentoni'll push up the patch to let people discuss there21:16
carl_baldwinkevinbenton: 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-orlandohi ZZelle_21:16
*** njohnston has joined #openstack-neutron21:17
carl_baldwinkevinbenton: 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-neutron21:17
*** tongli has quit IRC21:17
tidwellrcarl_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 minds21:17
*** shakamunyi has joined #openstack-neutron21:17
salv-orlandoZZelle_: sure sir21:17
ZZelle_salv-orlando, thx21:18
carl_baldwinkevinbenton: So, if it unblocks the move to pluggable IPAM in the gate, that could make it worth it.21:18
*** thorst_ has joined #openstack-neutron21:18
kevinbentontidwellr: yes, people do like their numerology21:18
*** efoley has quit IRC21:18
*** ivase has quit IRC21:18
*** salv-orl_ has joined #openstack-neutron21:18
tidwellrkevinbenton: the sheer number of tests I've been adjusting is a little unnerving21:18
*** abhiraut has quit IRC21:19
ihrachyskevinbenton: btw any comments on https://review.openstack.org/#/c/302089/ ? does it make sense?21:20
*** njohnsto_ has joined #openstack-neutron21:20
amullerZZelle_: :)21:21
*** thorst__ has joined #openstack-neutron21:21
*** abhiraut has joined #openstack-neutron21:22
*** thorst_ has quit IRC21:22
*** salv-orlando has quit IRC21:22
*** shakamunyi has quit IRC21:24
*** Sukhdev has joined #openstack-neutron21:25
openstackgerritKevin Benton proposed openstack/neutron: Add semaphore to ML2 create_port db operation  https://review.openstack.org/30308521:25
*** DevonBoatwright has joined #openstack-neutron21:25
*** thorst__ has quit IRC21:25
*** DevonBoatwright has left #openstack-neutron21:26
*** tflynn has quit IRC21:26
tidwellrkevinbenton: 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
kevinbentontidwellr: just to unlock us21:28
nmagnezikevinbenton, hey, when I asked dougwig about this patch he asked me to show you as well: https://review.openstack.org/#/c/299998/21:28
kevinbentontidwellr: until we can figure out how to do ipam in a less racey way21:28
tidwellrkevinbenton: ack21:28
kevinbentonihrachys: just left a comment21:28
kevinbentonihrachys: that config value does serve a slightly separate purpose21:28
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Check for snat_namespace existence before add or delete  https://review.openstack.org/30035821:28
*** tflynn has joined #openstack-neutron21:28
armaxihrachys: we might have another one21:29
armaxhttp://logs.openstack.org/73/297673/1/gate/gate-neutron-dsvm-api/4a178ab/console.html#_2016-04-07_20_10_03_30721:29
armaxplease shoot me now21:29
kevinbentonihrachys: check out neutron/plugins/ml2/drivers/type_tunnel.py get_mtu21:29
*** tmorin1 has joined #openstack-neutron21:29
* ihrachys laughs21:29
* dougwig points a carrot at armax and says, "bang!"21:29
* armax lies flat21:29
dougwigopenstack has claimed another innocent life.21:30
kevinbentonarmax: it looks like that config option doesn't exist21:30
armaxwe try too hard21:30
kevinbentonarmax: am i being helpful?21:30
kevinbentonnmagnezi: looking21:30
ihrachyskevinbenton: there, super() returns segment_mtu already21:30
armaxkevinbenton: very21:30
*** rtheis has quit IRC21:31
nmagnezikevinbenton, thanks21:31
armaxI am this close from pulling the rest of hair out21:31
*** njohnston has quit IRC21:31
kevinbentonihrachys: yes, but path_mtu is taken into account21:31
armax*my21:31
kevinbentonihrachys: path_mtu can be set to something different21:31
ihrachyskevinbenton: yeah, but it will have the same value, ain't it?21:31
ihrachyskevinbenton: why would it be different?21:31
kevinbentonihrachys: e.g. if your tunnel networks are going across interfaces that don't have jumbo frames21:31
*** njohnston has joined #openstack-neutron21:31
ihrachyskevinbenton: at least from reading option help texts, it does seem the same thing21:32
*** johnbelamaric has quit IRC21:32
*** tmorin has quit IRC21:32
armaxihrachys: sh!iesse21:32
ihrachyskevinbenton: 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
armaxlet’s stop merging21:33
armaxwe’re broken in stable/mitaka too21:33
ihrachyssh*t I pushed some there lately21:33
armaxyes, you did21:34
amullerarmax: kevinbenton: https://review.openstack.org/#/c/301167/9/tempest/config.py21:34
ihrachysthought we were safe there.21:34
*** iyamahat has quit IRC21:34
kevinbentonihrachys: 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 routing21:34
armaxamuller: looks we ain't21:34
kevinbentonihrachys: so it's wonky, but you can set global_physnet_mtu to e.g. 800021:35
kevinbentonihrachys: then path_mtu to 150021:35
amullerarmax: kevinbenton: Although Doug set the deprecated_name option, doesn't oslo_config register both the new and deprecated names?21:35
ihrachyskevinbenton: ok, I haven't thought we consider that kind of special cases for support. but ok.21:35
kevinbentonihrachys: and your vxlan networks will get 1500 (minus overhead)21:35
armaxnot quite sure what’s happening yet21:35
kevinbentonihrachys: 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 MTU21:36
amullerarmax: kevinbenton: I'll send a patch to rename tenant to project, that will probably fix it21:36
kevinbentonthey do all serve a different purpose21:36
armaxamuller: I wonder if we should cherry pick this21:36
kevinbentonwe just need to emphasize that for the majority of cases, global_physnet_mtu will be enough21:36
ihrachysarmax: I will bump all mitaka patches out of gate with rebas421:36
*** tiswanso has quit IRC21:37
armaxihrachys: ack21:37
armaxamuller: https://review.openstack.org/#/c/300648/21:37
kevinbentonamuller: no21:37
nmagnezikevinbenton, thank you for the comment :) I will take that into account.21:37
kevinbentonamuller: well sort of21:37
armaxthis one was not easy to cherry pick and I didn’t bother21:37
amullerarmax: Although, we use Tempest from pypi, not source, so something must be configuring tempest.conf differently21:37
*** lucasagomes has quit IRC21:37
tonytan4everrossella_s: ihrachys: Hi I am a developer from Rackspace, and I would like to participate in OVO initiative.21:37
armaxamuller: not in stable/mitaka21:37
*** baoli has quit IRC21:37
*** sridhar_ram has quit IRC21:37
kevinbentonamuller: it does set it up, but it doesn't work with the old name for things like set_override and whatnot used in tests21:37
amullerarmax: ahh so only stable/mitaka is broken by this? not master?21:38
armaxamuller: that’s negligence on my part21:38
kevinbentonamuller: so if you are moving an option, all of the code needs to be updated to work with the new one21:38
*** baoli has joined #openstack-neutron21:38
tonytan4everwhen you guys have a chance I would like to talk to you.21:38
armaxI should have anticipated the clusterf!ck21:38
armaxbesides it makes sense to keep all branches inline21:38
*** crose has quit IRC21:38
kevinbentontonytan4ever: what about?21:38
armaxamuller: leave it with me, you focus on the functional/fullstack21:38
tonytan4everAbout what kind of "objects" I can start to contribute to.21:39
tonytan4everkevinbenton: ^^, with regard to OVO bp21:39
armaxamuller: and no, I don’t think we’re broken in master as far as the API is conerned21:39
armaxjob I mean21:39
kevinbentontonytan4ever: ah, you'll have to sync up with ihrachys on that one i think21:39
amullerarmax: so we need to fix master and backport21:39
tonytan4everkevinbenton: cool21:39
kevinbentontonytan4ever: i haven't been following the OVO stuff very closely21:39
armaxamuller: there’s nothing to fix in master21:40
armaxwe have to backport my fix from Friday21:40
armaxlast Friday21:40
amullerarmax: oh you want stable branches on tempest from pypi as well21:40
armaxamuller: ya21:40
armaxunless I am talking nosnense?21:40
tonytan4everihrachys: When you get a chance.21:40
ihrachystonytan4ever: howdy. what's your timezone?21:41
tonytan4everUS EST21:41
amullerarmax: 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 anyway21:41
amullerarmax: worth asking for mtreinish's opinion here21:41
*** njohnston has quit IRC21:41
*** asingh has quit IRC21:41
armaxamuller: well, now I would be with you if you gave me the sacrosanct patch, now would I not?21:41
tonytan4everhowdy ihrachys21:41
*** iyamahat has joined #openstack-neutron21:41
ihrachystonytan4ever: 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-neutron21:42
amullerarmax: no idea what you're talking about =p21:42
armaxamuller: I do, and I should probably strip you from all your badges as retaliation21:42
openstackgerritKevin Benton proposed openstack/neutron: Add semaphore to ML2 create_port db operation  https://review.openstack.org/30308521:43
armaxhow about that? :)21:43
amullerarmax: still no idea what you're on about21:43
ihrachyskevinbenton: I also have docs update for MTU stuff at: https://review.openstack.org/#/c/302165/21:43
*** tmorin1 has quit IRC21:43
armaxamuller: https://bugs.launchpad.net/neutron/+bug/155535621:43
openstackLaunchpad bug 1555356 in neutron "Neutron too coupled with Tempest" [High,In progress] - Assigned to Assaf Muller (amuller)21:43
ihrachyskevinbenton: I guess I will need to expand it with path_mtu if we see the option staying21:43
armaxamuller: does that ring a bell now?21:43
kevinbentonihrachys: ok. cool. i'll try to take a look21:43
amullerarmax: what does that have to do with this new failure?21:44
*** thorst_ has joined #openstack-neutron21:44
armaxamuller: forget it21:44
amullerarmax: we would have been broken if we were still forked21:44
*** lucasagomes has joined #openstack-neutron21:45
amullerarmax: 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
armaxamuller: I am not complaining about decisions made in the past21:47
armaxamuller: those are hurting but not as much as lack of progress to resolve the current situation21:48
*** thorst_ has quit IRC21:48
amullerarmax: 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 plugin21:49
amullerarmax: If you think that's my fault I have news for you =p21:49
openstackgerritMartin Roy proposed openstack/neutron: Typo on side_effect rendering the test useless  https://review.openstack.org/30308921:49
armaxamuller: I am not interested in the blame game21:49
armaxI have never have and never will21:50
armaxI just look forward to get sh!t done21:50
*** lucasagomes has quit IRC21:50
armaxand when that ain’t happen I get frustrated21:50
amullerarmax: I'll start ignoring you when I see you're frustrated then21:51
amullermeanwhile I'll send a patch to fix this new thing21:51
armaxamuller: great response!21:51
*** jaimguer has quit IRC21:52
*** jaimguer has joined #openstack-neutron21:52
*** Sukhdev has quit IRC21:52
*** anshul has joined #openstack-neutron21:53
*** anshul has quit IRC21:54
*** lucasagomes has joined #openstack-neutron21:56
ihrachyskevinbenton: meh. magnum test patch failed in -api, but that's due to the tempest issue with the option we just hit21:57
*** regXboi has quit IRC21:58
*** admin0 has quit IRC21:59
*** enriquetaso has joined #openstack-neutron22:01
*** johnbelamaric has joined #openstack-neutron22:02
*** nmagnezi has quit IRC22:02
*** ihrachys has quit IRC22:05
*** efoley_ has quit IRC22:06
*** dave-mccowan has quit IRC22:06
openstackgerritAssaf Muller proposed openstack/neutron: Fix broken Tempest conf options in API tests  https://review.openstack.org/30309622:08
*** jonlan has joined #openstack-neutron22:08
amullerarmax: reproduced locally by installing tempest from git, removing it from the .tox venv and enabling sitepackages in tox.ini22:08
amullerarmax: that patch is confirmed to fix the issue, and even passes pep8 :)22:09
armaxamuller: ack22:09
armaxI am considering putting the api job to be non-voting until we get ourselves to a better place22:09
*** neiljerram has quit IRC22:10
amullerarmax: the only thing we could have done to prevent something like this is to not use CONF options defined in Tempest22:10
*** abhiraut has quit IRC22:11
amullerarmax: we could create our own CONF options and use our gate hook to configure tempest.conf with our new keys22:11
armaxack22:12
amullerarmax: that'd make it harder to run the tests locally and by other downstream consumers22:12
mtreinishoh, the config option rename broke things for you22:12
openstackgerritMartin Roy proposed openstack/neutron: Typo on side_effect rendering the test useless  https://review.openstack.org/30308922:13
mtreinishwe tried to maintain bacwards compat by deprecating the old opt name. Didn't think about external code consumers22:13
*** NightKhaos has quit IRC22:13
*** s3wong has joined #openstack-neutron22:13
armaxamuller, mtreinish: https://review.openstack.org/#/c/303100/22:13
*** asingh has quit IRC22:14
*** ntpttr_ has joined #openstack-neutron22:14
*** busterswt has quit IRC22:15
amullerarmax: I'm not sure about that22:16
armaxamuller: I am22:16
armaxamuller: we need to make changes to the gate hooks anyway and we can’t be 100% sure we got them right22:17
*** jonlan has quit IRC22:17
armaxamuller: thus we can’t keep the job voting whilst we move things aroud22:17
armaxaround22:17
*** saisrikiran has joined #openstack-neutron22:17
*** avozza has joined #openstack-neutron22:18
*** shakamunyi has joined #openstack-neutron22:18
*** dave-mccowan has joined #openstack-neutron22:18
amullerarmax: 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
amullerarmax: mtreinish: So if you want to disable the api job, you better have a plan22:19
*** knikolla has quit IRC22:19
armaxamuller: if we used the plugin interface would we still have that tempest code in tree? the one you just modified?22:20
amullerarmax: yes22:20
amullerarmax: the tests remain in our tree22:20
armaxand if so, then that should move under tempest/lib?22:20
*** neelashah has quit IRC22:21
armaxamuller: right, but not https://review.openstack.org/#/c/303096/1/neutron/tests/api/base.py22:21
*** Sukhdev_ has joined #openstack-neutron22:21
*** winggundamth has quit IRC22:21
amullerarmax: 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-neutron22:22
*** doug-fish has quit IRC22:23
*** shakamunyi has quit IRC22:24
armaxamuller: I think the long term solution is using both the plugin interface and the pypi released version of Tempest22:24
amullerarmax: That was my line of thinking as well, I don't think any other project does that on the other hand22:25
amullerarmax: Which would make us special, and we could be affected by issues with Devstack patches like yesterday22:26
*** boden has quit IRC22:26
amullerarmax: projects that use the tempest plugin interface currently use tempest from git22:26
*** Sukhdev_ has quit IRC22:26
*** vhosakot has quit IRC22:26
*** hdaniel has joined #openstack-neutron22:26
armaxamuller: I can’t see the point in having to use a stable interface when pulling from source though22:26
*** johnsom has quit IRC22:26
armaxamuller: I must be missing something22:27
armaxbut I always got the impression that the move to pypi for tempest was done to avoid pulling from source?22:27
mtreinisharmax, 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 git22:28
mtreinishbut there is nothing binding you to that22:28
amullermtreinish: Are you recommending projects to move to grab Tempest from pypi?22:28
mtreinishwell except for the lack of gate job to set it up for running from release22:28
mtreinishamuller: tbh, I hadn't thought about it before22:28
amullermtreinish: That would have prevented today's neutron issue but not yesterday's22:29
amullermtreinish: So TBH I'm not sure what is better for us22:29
armaxmtreinish: I was always under the impression that pulling from source was frowned upon22:29
*** abhiraut has joined #openstack-neutron22:29
*** rcernin has quit IRC22:29
openstackgerritBrian Haley proposed openstack/neutron: DVR: Ensure fpr and rfp devices are configured correctly  https://review.openstack.org/30234022:30
*** winggundamth has joined #openstack-neutron22:30
armaxdevstack is kind of a special case in that it itself pulls all openstack deps from source22:30
armaxand that might need more thinking22:30
mtreinisharmax: well it is for libs and stuff, but also tempest wasn't published on pypi until like a month ago22:31
armaxmtreinish: right, I meant the services22:31
*** rmart04 has quit IRC22:32
*** hdaniel has quit IRC22:33
*** avozza has quit IRC22:35
mtreinisharmax: 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 IRC22:35
*** saisrikiran has quit IRC22:35
*** tmorin has joined #openstack-neutron22:36
*** tpsilva has quit IRC22:37
amullerarmax: mtreinish: dmellado has a patch for a Neutron Tempest plugin: https://review.openstack.org/#/c/274023/22:39
amullercurrently in merge conflict, but latest patchset passed22:39
amullerhowever, the plugin is essentially unused. The neutron api job still uses tox -e api from the neutron dir.22:39
armaxamuller: nice22:39
armaxamuller: right, that’s the part on project-config that we’d need to tweak22:39
amullerWe want the Neutron API job set up like other projects that use the plugin interface22:40
*** ntpttr_ has joined #openstack-neutron22:40
amullerarmax: looking at that patch as it is, it doesn't buy us any stability... Am I missing something mtreinish?22:40
amullerit's still a good thing, it just doesn't prevent any of the issues we've been having22:40
mtreinishamuller: we din't think about the code consumers when we changed those config options22:40
amullermtreinish: That problem is in the past :)22:41
mtreinishamuller: we've said that config should be stable, but I think we only really considered the config file consumers22:41
amullermtreinish: armax and I are looking ahead and asking if the plugin interface will actually help us22:41
mtreinishamuller: but that's the break you're worried about right?22:41
*** tonytan4ever has quit IRC22:41
*** minwang2 has left #openstack-neutron22:41
amullermtreinish: that too22:42
*** saisrikiran has joined #openstack-neutron22:43
mtreinishamuller: 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 suite22:44
armaxamuller: so let’s continue chipping away at this22:44
amullerarmax: ^ That's what I've been saying for the past month...22:44
armaxamuller: and I understood22:44
armaxamuller: no need to rub it in22:44
amullerarmax: But it feels good :(22:45
amullerarmax: today's sucked, let me have that22:45
armaxamuller: why? I never contradicted you!22:45
*** abregman has quit IRC22:45
armaxamuller: in the meantime I am gonna flip the stable branches to use the released version of tempest, you onboard with that?22:46
amullerarmax: yes22:46
armaxamuller: ack22:46
amullerarmax: unit tests still broken I understand?22:47
armaxamuller: ya22:47
amullerarmax: ok, if both unit and api tests are broken we have to disable one or the other22:47
armaxmtreinish, amuller to me the extra stability comes from the a versioned plugin interface22:47
amullerarmax: the fix for the api job can't go in cause of the unit tests failure...22:48
mtreinisharmax: for the stable branch this is just for the api job right?22:48
armaxamuller: the unit test fix is coming from g-r cap I think22:48
armaxmtreinish: ack22:48
mtreinishok22:48
armaxmtreinish: yes22:48
armaxamuller: so once that’s sorted yours should go in no probs22:48
amullerarmax: yeah I see https://review.openstack.org/#/c/303034/22:48
amullerarmax: ok I'll recheck it later22:49
amullerarmax: that patch failed also =D22:49
amullerarmax: a perfect storm of failures22:49
armaxit did?22:49
armaxI did suggest to ihar switching to tomato picking farm22:49
armaxyou in?22:49
armaxI am starting Monday morning22:50
armax5am22:50
amullerarmax: I always wanted to learn carpentry22:50
amullerarmax: the g-r patch failed due to a glance test22:50
armaxamuller: that doesn’t suit you22:50
amuller... in the postres job22:50
*** abhiraut has quit IRC22:51
*** baoli has quit IRC22:52
*** abhiraut has joined #openstack-neutron22:52
amullerarmax: alright I think I'm going offline to go cry in a corner or something. I'll recheck my shit later.22:53
armaxamuller: no need22:53
armaxamuller: cheer up, we got this handled22:53
armaxamuller: at least we know the root causes22:53
armaxamuller: imagine if we hadn’t22:53
amullerarmax: Then I would have had a longer work day =D22:54
*** rickyrem has quit IRC22:54
amullerarmax: https://review.openstack.org/#/c/303054/ is very interesting btw22:54
*** p01nt3r75 has joined #openstack-neutron22:54
amullertoodles22:55
*** amuller has quit IRC22:55
*** sdague has quit IRC22:57
*** shivrao has quit IRC22:57
*** Leom has quit IRC22:58
*** Leom has joined #openstack-neutron22:58
*** tflynn_ has joined #openstack-neutron22:59
*** jaimguer has quit IRC22:59
*** tflynn__ has joined #openstack-neutron23:00
*** tflynn_ has quit IRC23:00
*** tflynn__ has quit IRC23:00
*** shivrao has joined #openstack-neutron23:01
*** tmorin has quit IRC23:01
*** tflynn has quit IRC23:02
*** Leom has quit IRC23:03
openstackgerritArmando Migliaccio proposed openstack/neutron: Tempest 11.0.0 is released  https://review.openstack.org/30300523:05
*** johnsom has joined #openstack-neutron23:06
*** shivrao has quit IRC23:07
*** tflynn has joined #openstack-neutron23:08
*** mickeys has quit IRC23:09
*** mickeys has joined #openstack-neutron23:09
*** djan has joined #openstack-neutron23:09
*** akshai has quit IRC23:09
*** tflynn has quit IRC23:10
*** banix has joined #openstack-neutron23:11
*** tflynn has joined #openstack-neutron23:11
*** tbachman has joined #openstack-neutron23:14
*** busterswt has joined #openstack-neutron23:14
*** reedip__ has quit IRC23:16
openstackgerritArmando Migliaccio proposed openstack/neutron-lib: Revert "Add Collector Agent type string"  https://review.openstack.org/30311023:19
*** mickeys has quit IRC23:19
*** vjmorale has joined #openstack-neutron23:20
*** mickeys has joined #openstack-neutron23:21
openstackgerritArmando Migliaccio proposed openstack/neutron-lib: Revert "Add Collector Agent type string"  https://review.openstack.org/30311023:21
openstackgerritArmando Migliaccio proposed openstack/neutron: Revert "Add instrumentation devref, Part I"  https://review.openstack.org/30311123:22
*** saisrikiran has quit IRC23:22
*** vjmorale has quit IRC23:22
*** ntpttr_ has quit IRC23:22
*** salv-orl_ has quit IRC23:24
openstackgerritArmando Migliaccio proposed openstack/neutron: Revert "Add instrumentation devref, Part I"  https://review.openstack.org/30311123:24
*** banix has quit IRC23:27
*** markvoelker has quit IRC23:29
*** sridhar_ram has joined #openstack-neutron23:31
*** abhiraut has quit IRC23:32
*** panda has quit IRC23:33
*** panda has joined #openstack-neutron23:34
*** abhiraut has joined #openstack-neutron23:36
*** jamielennox|away is now known as jamielennox23:36
*** ZZelle_ has quit IRC23:41
*** akshai has joined #openstack-neutron23:48
*** vhosakot has joined #openstack-neutron23:51
*** busterswt has quit IRC23:52
*** akshai_ has joined #openstack-neutron23:54
*** lujinluo has joined #openstack-neutron23:56
*** Sukhdev_ has joined #openstack-neutron23:57
*** akshai has quit IRC23:58
*** markvoelker has joined #openstack-neutron23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!