Thursday, 2018-01-04

*** slaweq_ has joined #openstack-meeting-300:01
*** slaweq_ has quit IRC00:06
*** sdague has quit IRC00:18
*** slaweq_ has joined #openstack-meeting-300:34
*** wanghao has quit IRC00:37
*** wanghao has joined #openstack-meeting-300:37
*** wanghao has quit IRC00:38
*** wanghao has joined #openstack-meeting-300:39
*** gothicmindfood has quit IRC00:47
*** gothicmindfood has joined #openstack-meeting-300:50
*** VW has joined #openstack-meeting-300:55
*** wanghao has quit IRC00:58
*** wanghao_ has joined #openstack-meeting-301:03
*** ywat has joined #openstack-meeting-301:04
*** slaweq_ has quit IRC01:07
*** slaweq_ has joined #openstack-meeting-301:16
*** Swami has quit IRC01:17
*** slaweq_ has quit IRC01:21
*** tuanla____ has joined #openstack-meeting-301:26
*** wanghao_ has quit IRC01:26
*** wanghao has joined #openstack-meeting-301:27
*** cshastri has joined #openstack-meeting-301:31
*** wanghao_ has joined #openstack-meeting-301:37
*** wanghao has quit IRC01:37
*** wanghao has joined #openstack-meeting-301:40
*** tuanla____ has quit IRC01:40
*** tuanla____ has joined #openstack-meeting-301:41
*** wanghao is now known as wang_hao01:41
*** wang_hao has quit IRC01:43
*** wang_hao_ has joined #openstack-meeting-301:43
*** wang_hao has joined #openstack-meeting-301:43
*** wanghao_ has quit IRC01:43
*** wang_ha__ has joined #openstack-meeting-301:46
*** wang_hao has quit IRC01:46
*** slaweq_ has joined #openstack-meeting-301:46
*** wang_hao_ has quit IRC01:46
*** wang_ha__ is now known as wanghao01:47
*** wanghao_ has joined #openstack-meeting-301:48
*** slaweq_ has quit IRC01:51
*** wanghao_ has quit IRC01:51
*** wanghao_ has joined #openstack-meeting-301:52
*** wanghao__ has joined #openstack-meeting-301:52
*** wanghao_ has quit IRC01:52
*** wanghao has quit IRC01:52
*** wanghao__ has quit IRC01:53
*** wanghao has joined #openstack-meeting-301:53
*** liuyulong has joined #openstack-meeting-301:55
*** liuyulong has left #openstack-meeting-301:57
*** liuyulong has joined #openstack-meeting-301:57
*** yamahata has quit IRC01:58
*** zhurong has joined #openstack-meeting-302:13
*** slaweq_ has joined #openstack-meeting-302:20
*** bobh has joined #openstack-meeting-302:36
*** VW_ has joined #openstack-meeting-302:36
*** VW has quit IRC02:39
*** bobh has quit IRC02:53
*** slaweq_ has quit IRC02:53
*** ttrifonov has quit IRC02:55
*** slaweq_ has joined #openstack-meeting-302:59
*** slaweq_ has quit IRC03:03
*** anilvenkata has joined #openstack-meeting-303:05
*** ttrifonov has joined #openstack-meeting-303:16
*** coolsvap has joined #openstack-meeting-303:26
*** slaweq_ has joined #openstack-meeting-303:33
*** slaweq_ has quit IRC03:38
*** VW_ has quit IRC03:58
*** gmann has joined #openstack-meeting-303:58
*** VW has joined #openstack-meeting-303:58
*** slaweq_ has joined #openstack-meeting-304:11
*** kiseok7 has quit IRC04:26
*** slaweq_ has quit IRC04:44
*** slaweq_ has joined #openstack-meeting-304:48
*** pgadiya has joined #openstack-meeting-304:50
*** slaweq_ has quit IRC04:53
*** slaweq_ has joined #openstack-meeting-305:23
*** slaweq_ has quit IRC05:27
*** yamahata has joined #openstack-meeting-305:27
*** VW has quit IRC05:44
*** markmcclain has quit IRC05:51
*** markmcclain has joined #openstack-meeting-305:54
*** slaweq_ has joined #openstack-meeting-305:58
*** psachin has joined #openstack-meeting-306:04
*** gmann has quit IRC06:08
*** wanghao has quit IRC06:18
*** reedip has joined #openstack-meeting-306:20
*** slaweq_ has quit IRC06:31
*** slaweq_ has joined #openstack-meeting-306:34
*** slaweq_ has quit IRC06:39
*** 07EAAP3JW has joined #openstack-meeting-306:42
*** 07EAAP3JW has quit IRC06:43
*** hongbin has joined #openstack-meeting-306:43
*** zhurong has quit IRC06:47
*** zhurong has joined #openstack-meeting-306:51
*** pgadiya has quit IRC06:57
*** kiseok7 has joined #openstack-meeting-306:59
*** marios has joined #openstack-meeting-307:03
*** shuyingya has joined #openstack-meeting-307:07
*** pgadiya has joined #openstack-meeting-307:10
*** slaweq_ has joined #openstack-meeting-307:14
*** slaweq_ has quit IRC07:18
*** benj_ has quit IRC07:22
*** gmann has joined #openstack-meeting-307:22
*** benj_ has joined #openstack-meeting-307:24
*** jrist has quit IRC07:29
*** pcaruana has joined #openstack-meeting-307:32
*** slaweq_ has joined #openstack-meeting-307:43
*** snapiri has joined #openstack-meeting-307:52
*** markvoelker has quit IRC07:55
*** diablo_rojo has quit IRC07:57
*** shuyingya has quit IRC07:58
*** cshastri has quit IRC08:36
*** MarkBaker has joined #openstack-meeting-308:38
*** shuyingya has joined #openstack-meeting-308:44
*** cshastri has joined #openstack-meeting-308:47
*** jgu has quit IRC08:48
*** hongbin has quit IRC09:03
*** shuyingya has quit IRC09:04
*** e0ne has joined #openstack-meeting-309:05
*** jrist has joined #openstack-meeting-309:12
*** stendulker has joined #openstack-meeting-309:23
*** wanghao has joined #openstack-meeting-309:27
*** wanghao has quit IRC09:28
*** wanghao has joined #openstack-meeting-309:28
*** wanghao has quit IRC09:28
*** wanghao has joined #openstack-meeting-309:29
*** wanghao has quit IRC09:29
*** slaweq_ has quit IRC09:29
*** wanghao has joined #openstack-meeting-309:30
*** slaweq has joined #openstack-meeting-309:30
*** wanghao has quit IRC09:30
*** wanghao has joined #openstack-meeting-309:30
*** wanghao has quit IRC09:31
*** wanghao has joined #openstack-meeting-309:31
*** wanghao has quit IRC09:32
*** gmann has quit IRC09:32
*** wanghao has joined #openstack-meeting-309:32
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-meeting-309:33
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-meeting-309:34
*** wanghao has quit IRC09:34
*** wanghao has joined #openstack-meeting-309:35
*** cshastri has quit IRC09:46
*** cshastri has joined #openstack-meeting-309:50
*** markvoelker has joined #openstack-meeting-309:56
*** tuanla____ has quit IRC10:00
*** pbourke has quit IRC10:10
*** pbourke has joined #openstack-meeting-310:11
*** MarkBaker has quit IRC10:15
*** MarkBaker has joined #openstack-meeting-310:15
*** yamahata has quit IRC10:16
*** toabctl has quit IRC10:16
*** toabctl has joined #openstack-meeting-310:23
*** cshastri has quit IRC10:25
*** markvoelker has quit IRC10:30
*** georgk has joined #openstack-meeting-310:39
*** markmcclain has quit IRC10:51
*** markmcclain has joined #openstack-meeting-310:58
*** sdague has joined #openstack-meeting-311:01
*** lpetrut has joined #openstack-meeting-311:15
*** georgk has quit IRC11:15
*** stendulker_ has joined #openstack-meeting-311:22
*** anilvenkata has quit IRC11:23
*** lpetrut has quit IRC11:24
*** lpetrut has joined #openstack-meeting-311:25
*** stendulker has quit IRC11:25
*** stendulker_ has quit IRC11:27
*** lpetrut has quit IRC11:27
*** markvoelker has joined #openstack-meeting-311:27
*** openstack has quit IRC11:28
*** openstack has joined #openstack-meeting-313:09
*** ChanServ sets mode: +o openstack13:09
*** psachin has joined #openstack-meeting-313:14
*** liuyulong has quit IRC13:23
*** cleong has joined #openstack-meeting-313:26
*** jtollet has joined #openstack-meeting-313:26
*** lhx_ has joined #openstack-meeting-313:32
*** jeremyfreudberg has joined #openstack-meeting-313:44
*** lhx_ has quit IRC13:46
*** lhx_ has joined #openstack-meeting-313:47
*** hungpv has joined #openstack-meeting-313:51
*** hongbin has joined #openstack-meeting-313:57
*** hungpv has left #openstack-meeting-313:57
*** bobh has joined #openstack-meeting-313:58
tellesnobrega#startmeeting sahara14:00
openstackMeeting started Thu Jan  4 14:00:04 2018 UTC and is due to finish in 60 minutes.  The chair is tellesnobrega. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: sahara)"14:00
openstackThe meeting name has been set to 'sahara'14:00
jeremyfreudbergo/14:00
tellesnobregawaiting for tosky14:01
*** tosky has joined #openstack-meeting-314:03
toskyi/14:03
toskyo/14:03
tellesnobregathere we go14:03
tellesnobrega#topic News/Updates14:03
*** openstack changes topic to "News/Updates (Meeting topic: sahara)"14:03
*** lyan has joined #openstack-meeting-314:03
tellesnobregaI'm working on image gen for MapR and dashboard part of decommission of specific node (having trouble but hopefully making progress)14:04
*** psachin has quit IRC14:04
toskyI'm helping tellesnobrega testing the MapR generation (we are starting to see the light); also testing ambari 2.4 generation with less sucess14:05
jeremyfreudbergi finished up S3 JB in openstackclient and sahara-dashboard.  finished up [keystone_authtoken] patch. now hopefully i will be able to focus on force delete (not really that hard) and APIv2 in saharaclient (harder than i want it to be)14:05
toskyand trying to clean some issues in my queue: I just opened https://bugs.launchpad.net/sahara/+bug/1741233 with a related patch14:06
openstackLaunchpad bug 1741233 in Sahara "Reduce the log messages from amqp and oslo.messaging" [Undecided,New]14:06
toskyand I'm going back to the CLI tests not working with v314:06
toskykeystonev314:06
tellesnobregajeremyfreudberg, about APIv2, maybe we should send another email on the thread so Monty can actually see, the last one probably went undetected by him14:07
toskyor ping him/nag him on IRC14:07
jeremyfreudbergyeah, i need to talk to mordred for sure14:07
jeremyfreudbergthe holiday season did not make it easy for either of us, i'm sure14:07
tellesnobregadefinitely14:08
toskyyep14:08
jeremyfreudbergwe need to get that moving soon because we have to have a final release of client library at Q-314:08
toskysame for zuulv3 updates, which I need to look at again14:08
tellesnobregayes14:08
*** bobh has quit IRC14:08
tellesnobregadeadline is Jan 2614:09
tellesnobregawe get about 3 weeks to get this done14:09
jeremyfreudbergit is definitely do-able, I just need an injection of Monty power to get it done14:09
tellesnobregasounds good14:10
tellesnobregaI'm looking for help on dashboard, I will ping horizon people to see if I can get things sped up14:11
jeremyfreudbergtellesnobrega, what's the exact problem you're facing?14:11
tellesnobregathe biggest one is I don't really understand dashboard stuff14:12
tellesnobregawhat I want to do is basically the same flow as create cluster, where you first select the plugin and version and later it goes to create cluster stuff14:13
tellesnobregaI want to do the same for scale14:13
tellesnobregayou select the count of node groups than it goes to the new tab with the instances to select14:13
jeremyfreudbergi don't even remember anything about the current scaling stuff in UI14:14
tellesnobregadon't really know how to connect the two workflows, not sure if there should be two workflows or just steps in the same workflows14:14
tellesnobregaits just the node groups, and the count of instances for each14:14
tellesnobregaso you can increase or decrease14:14
jeremyfreudbergright14:15
* mordred waves to jeremyfreudberg14:16
* jeremyfreudberg waves back to mordred14:17
mordredah - the api thing ... crap, I probably did miss the most recent message14:17
jeremyfreudbergmordred, no worries14:17
mordredthere it is!14:18
tellesnobregamordred, definitely not your fault, too close to Christmas14:18
tellesnobregaI'm just going to change topic so the talk is on the right place14:18
tellesnobrega#topic APIv214:18
*** openstack changes topic to "APIv2 (Meeting topic: sahara)"14:18
mordredjeremyfreudberg, tellesnobrega: I can definitely get you some links to some places that are using the discovery code from keystoneauth 'correctly'14:19
jeremyfreudbergmordred: that would be great14:20
mordredthere's also another option to consider that may be a *bit* crazy, but maybe isn't? ...14:20
jeremyfreudbergcrazy can be good14:21
mordredwhich is to not add v2 support to python-saharaclient at all and to just add support to openstacksdk and point people to it14:21
tellesnobregaI don't think I understand how that would work14:22
* tosky smells a bit of conflict of interests from mordred14:22
tosky:D14:22
mordredI'm not 100% sure I do either - it just occured to  me as I was thinking about what I'd link you to for discovery examples...14:22
mordredtosky: :)14:22
jeremyfreudbergright, i would assume if any project were to be doing version discovery right, it would be openstacksdk14:23
jeremyfreudbergdoes it do the endpoint manipulation stuff too?14:23
jeremyfreudbergwith the project id and such, i mean14:23
toskythe only problem is: at this point, porting to saharaclient/v2 should be fast enough (most methods will almost the same), how different is going to be the API exposed by openstacksdk?14:23
mordredthe code in the master branch does14:23
mordredtosky: oh, good point14:23
toskyI mean, we can implement it for sure and slowly move there like we did for openstackclient, I'm just not sure about the initial implementation14:24
toskythe timing for it14:24
mordredmaybe for now I'll just give you some links to some things - and then when I'm done with this pagination patch I'm working on right now, I'll make a POC patch to see if just consuming sdk in python-saharaclient would make anything easier/better14:25
*** mrtenio has joined #openstack-meeting-314:25
tellesnobregathat sounds great mordred14:25
tellesnobregathanks14:25
*** dansmith has quit IRC14:25
mordredsure nuff! seven years in to openstack you'd think we'd have a solid answer for this ... maybe in another seven we will14:26
jeremyfreudbergmordred, great14:26
jeremyfreudbergthanks!14:26
tellesnobregaI don't think there is a solid answer for much stuff in OpenStack14:26
*** dansmith has joined #openstack-meeting-314:28
*** dansmith is now known as Guest385614:28
*** hongbin_ has joined #openstack-meeting-314:28
tellesnobregaI guess we can move on14:28
jeremyfreudbergyep14:28
*** VW has joined #openstack-meeting-314:29
tellesnobrega#topic Open Discussion14:29
*** openstack changes topic to "Open Discussion (Meeting topic: sahara)"14:29
*** hongbin has quit IRC14:29
*** hongbin_ has quit IRC14:29
*** jtollet has quit IRC14:29
*** hongbin has joined #openstack-meeting-314:29
tellesnobregaDid you get a chance to read shu yingya's email?14:29
*** hongbin has quit IRC14:30
toskyabout https://bugs.launchpad.net/sahara/+bug/1741233 and https://review.openstack.org/#/c/531137/, do you think that I can go on and (if passes) backport to pike? I have an hard time debugging anything right now without14:30
openstackLaunchpad bug 1741233 in Sahara "Reduce the log messages from amqp and oslo.messaging" [Undecided,New]14:30
*** hongbin has joined #openstack-meeting-314:30
jeremyfreudbergcycle-with-intermediary¶14:30
jeremyfreudbergoops14:30
jeremyfreudbergjava.lang.NoClassDefFoundError: org/apache/commons/httpclient/HttpMethod14:30
jeremyfreudbergtosky, i think it's fine, but i'm not the expert on that kind of thing14:31
tellesnobregatosky, I think so14:31
toskyjeremyfreudberg: as discussed on https://review.openstack.org/#/c/490971/, I kept stevedore as INFO, overriding the default14:33
toskyso let's see if the gates agree14:33
jeremyfreudbergtosky, great14:33
toskyaaand also I will work again on zuulv3 jobs; the job for sahara-tests is ready, with the exception of the multinode job, but that can be fixed later14:34
toskyI will shamelessly use this space and having mordred around to check if there are some news about the new publishing method for artifacts, or if I should code using the current way14:34
toskyso to fix sahara-extras, publish the oozie tarball and remove the last bits of sahara-files14:34
tellesnobregalooks like mordred went away14:37
jeremyfreudbergi think it is rather early where he is14:37
tellesnobregaprobably14:38
*** bobh has joined #openstack-meeting-314:38
toskyyeah - and I nagged him enough already about this14:39
tellesnobregalol14:39
jeremyfreudbergwe can discuss the shu yingya hadoop-swiftfs thing for a bit14:40
jeremyfreudbergas telles suggested14:40
tellesnobregayes14:41
tellesnobregadoes anyone understands what is going on there?14:41
toskydon't steal my questions!14:41
tosky:)14:41
jeremyfreudberglol, i don't really know either, it's something with the dependencies14:41
jeremyfreudbergbuilding hadoop-swiftfs against the hadoop 2.8 libraries and things go wonky i guess14:42
toskyhe asked about building openstack-hadoop.jar, but the script in sahara-extra should do it14:42
tellesnobregayes, maybe we need a little more context from him14:42
jeremyfreudbergtosky, i think he has it building now, it just builds something that doesn't totally work14:43
*** yamamoto has quit IRC14:43
tellesnobregahum, makes sense. Maybe the new oozie version he is using removed some dependencies14:43
toskycould it be that the current openstack-hadoop.jar is not working with hadoop 2.8 and he can't build it?14:44
toskyalso, openstack-hadoop is not about oozie14:44
* tosky a bit more confused than before14:44
jeremyfreudbergin the email the job was launched by oozie but in fact the traceback was within hadoop-swiftfs14:44
tellesnobregaoh, that makes more sense14:45
mordredtosky, tellesnobrega: sorry, went away for a second14:45
mordredhowever, nothing new to report there yet, so use the current way for now14:45
toskyack, thanks!14:46
jeremyfreudberghttps://github.com/apache/hadoop/commit/62579b69a0a294ba1ea14cf76c650b640f89f331 << I wonder if shuyingya has seen that14:46
toskybut that looks like 3.0.014:47
*** Swami has joined #openstack-meeting-314:48
jeremyfreudbergtosky, that is true14:48
jeremyfreudberg(but still, removing the dependency causing the error is still nice)14:48
*** georgk has joined #openstack-meeting-314:48
jeremyfreudbergactually a handful of these commits https://github.com/apache/hadoop/commits/trunk/hadoop-tools/hadoop-openstack/pom.xml all talk about httpclient14:49
tellesnobreganext time we see shu yingya we should ask him for more details so we can pin point the problem14:50
jeremyfreudbergthat seems to be one of the few things apache folks have been doing to hadoop-swiftfs, is managing dependencies14:50
jeremyfreudbergbut regardless, we do need to more info from shu yingya, to really know what the right fix is14:51
*** georgk1 has joined #openstack-meeting-314:51
tellesnobregaagreed14:51
*** georgk1 has quit IRC14:51
*** georgk has quit IRC14:52
*** david_chou has joined #openstack-meeting-314:52
jeremyfreudberganything else?14:53
jeremyfreudbergptg stuff?14:53
toskyoh, right, PTG14:54
tellesnobregaPTG14:54
tellesnobregaI will be there, tosky as well14:54
tellesnobregaare you going to be there? jeremyfreudberg14:55
tellesnobregaI will start an etherpad this week so we can start putting topics there14:55
*** janzian has joined #openstack-meeting-314:55
tellesnobregaI asked for 2 days, just like last time, should be enough time14:55
jeremyfreudbergmost likely i will be there. of course it takes place during classes so it's always a challenge14:55
jeremyfreudbergon that note, i might want to switch our two days to Thursday and Friday14:56
jeremyfreudberginstead of the usual Wednesday and Thursday14:56
jeremyfreudbergbut i'm not sure yet14:56
tellesnobregathat should be possible, I just need to check my flight time on friday14:56
tellesnobregabut I think it is late enough14:56
jeremyfreudbergit might not be necessary14:57
jeremyfreudbergjust giving heads up14:57
jeremyfreudberghopefully i can say by next week14:57
tellesnobregathanks14:57
*** mlavalle has joined #openstack-meeting-314:57
jeremyfreudbergsince i'm sure the ptg people want to know if we are changing days too14:57
tellesnobregadefinitely14:57
tellesnobregaI will send out an email with the etherpad so we can start putting it together14:58
jeremyfreudbergcool14:58
toskywhat is the deadline for requesting the days? tellesnobrega, did you already request the basic two days as in Denver?14:59
tellesnobregaI did already, but I can try to change it14:59
tellesnobreganot sure there is an specific deadline15:00
tellesnobregawe are at the top of the hour. Thanks all for joining and lets hurry up our tasks, and start getting ready for PTG.15:00
toskythanks all o/15:00
tellesnobrega#endmeeting15:00
*** openstack changes topic to "milestone 2 (Meeting topic: monasca)"15:00
openstackMeeting ended Thu Jan  4 15:00:42 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/sahara/2018/sahara.2018-01-04-14.00.html15:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/sahara/2018/sahara.2018-01-04-14.00.txt15:00
openstackLog:            http://eavesdrop.openstack.org/meetings/sahara/2018/sahara.2018-01-04-14.00.log.html15:00
*** jeremyfreudberg has left #openstack-meeting-315:01
*** yamamoto has joined #openstack-meeting-315:01
toskyand someone please fix the "idle" channel topic15:01
*** tosky has left #openstack-meeting-315:01
mlavalle#startmeeting neutron_l315:01
openstackMeeting started Thu Jan  4 15:01:33 2018 UTC and is due to finish in 60 minutes.  The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
*** openstack changes topic to " (Meeting topic: neutron_l3)"15:01
openstackThe meeting name has been set to 'neutron_l3'15:01
*** otcnsw2 has joined #openstack-meeting-315:01
Swamihi15:01
mlavallehey15:01
mlavalleHappy New Year15:02
SwamiWish you all a Happy New Year15:02
david_chouHi, Happy New Year!15:02
* haleyb walks in late15:02
* mlavalle started the meeting a little late, waiting for the previous one to end ;-)15:03
mlavalle#topic Announcements15:03
*** openstack changes topic to "Announcements (Meeting topic: neutron_l3)"15:03
mlavalleWelcome officially to 2018!15:03
mlavalleI missed you guys the past two weeks ;-)15:04
mlavalleWe are a little more tha 2 weeks from Q-315:04
Swamimlavalle: Did you take some time off.15:05
mlavalle#link https://releases.openstack.org/queens/schedule.html15:05
mlavalleSwami: mhhhh, my son was here last week so I did a lot of things with him15:05
mlavallebut I really worked all days15:05
mlavallejust slower15:05
Swamimlavalle: great15:05
mlavallethanks for asking :-)15:06
mlavalleThe other reminder is the PTG in Ireland15:07
mlavalle#link https://www.openstack.org/ptg/15:07
*** MarkBaker has quit IRC15:07
mlavalleLess than 2 months away15:07
mlavalle361 tickest remaining as of now :-)15:07
mlavalleCheck with your companies before purchasin a ticket15:08
mlavalleyour company might be getting some tickets if they are sposoring the event15:08
mlavalleThat is how I got mine15:08
mlavalleAny other announcements?15:09
mlavalleok, moving on15:10
mlavalle#topic Bugs15:10
*** openstack changes topic to "Bugs (Meeting topic: neutron_l3)"15:10
mlavalleSwami: please fire away15:10
Swamimlavalle: thanks15:10
Swami#link https://bugs.launchpad.net/neutron/+bug/171730215:11
openstackLaunchpad bug 1717302 in neutron "Tempest floatingip scenario tests failing on DVR Multinode setup with HA" [High,Confirmed] - Assigned to Brian Haley (brian-haley)15:11
SwamiI think this bug is still unsolved.15:11
SwamiI am not sure if haleyb had a chance to take a look at it.15:11
haleybno, i haven't taken another look yet15:12
SwamiI have not looked at it for a while, but I will try to spend some time this week on this, since I am winding up some work downstream.15:12
Swami#link https://bugs.launchpad.net/neutron/+bug/171640115:12
openstackLaunchpad bug 1716401 in neutron "FWaaS: Ip tables rules do not get updated in case of distributed virtual routers (DVR)" [Undecided,New] - Assigned to Sridar Kandaswamy (skandasw)15:12
SwamiI had a chat before the break with Sridhar on this bug and briefed him about the DVR changes that happened on this area.15:13
SwamiHe mentioned that he would take a look at it and get back to me.15:13
mlavalleyeah I talked to him before the break15:13
mlavalleI asked him to ping you15:13
mlavallegood to know he did15:14
SwamiHis initial suspicion is on the Firewall code, but he will check and get back to me.15:14
mlavallehe might be around now, let me see15:14
Swamimlavalle: ok15:14
Swami#link https://bugs.launchpad.net/neutron/+bug/173606815:15
openstackLaunchpad bug 1736068 in neutron "DVR lost centralized floating ip when the ha state switched." [High,In progress] - Assigned to sunzuohua (zuohuasun)15:15
mlavalleno he is not, probably they didn't meet this week15:15
mlavallewill try next week15:15
Swamimlavalle: no problem I can check with him on the progress.15:15
SwamiThere is patch up for review on this bug.15:15
Swami#link https://review.openstack.org/52584215:15
*** MarkBaker has joined #openstack-meeting-315:16
haleybit's failing unit tests, think it just needs an update15:16
mlavallecool, haleyb has looked at it15:17
Swamihaleyb: ok I will take a look at it and if not addressed I will update it.15:17
Swamihaleyb: I also found another bug reported on this same area, I will try to find it and tag it appropriately.15:17
SwamiI did see the bug popped up a couple of days back.15:18
Swami#link https://bugs.launchpad.net/neutron/+bug/171834515:18
openstackLaunchpad bug 1718345 in neutron "ml2_distributed_port_bindings not cleared after migration from DVR" [Medium,In progress] - Assigned to Swaminathan Vasudevan (swaminathan-vasudevan)15:18
*** lhx_ has quit IRC15:18
SwamiI will have to revisit the patch since it was failing and causing some dead lock.15:19
*** lhx_ has joined #openstack-meeting-315:19
SwamiI will work on it this week.15:19
SwamiThat's all I had for the bugs this week.15:19
Swamimlavalle: back to you15:19
mlavalleSwami: thanks for the update15:20
*** gmann__ has joined #openstack-meeting-315:20
mlavalleThe only one we had was https://bugs.launchpad.net/neutron/+bug/170675015:21
openstackLaunchpad bug 1706750 in neutron "FlushError for automatic IPAllocations when subnet create is retried" [High,Fix released] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)15:21
haleybi think there was a new l3-ha bug that i need to triage15:21
mlavalleand we merged the fix https://review.openstack.org/#/c/529169/ just before the Holidays15:21
haleybhttps://bugs.launchpad.net/neutron/+bug/174045015:22
openstackLaunchpad bug 1740450 in neutron "Restarting l3 agent results in lost of centralized fip in snat ns" [Undecided,New]15:22
haleyband there is also the one regarding running containerized l3 that we talked about in the meeting yesterday15:23
mlavalleyes15:23
haleybso if you don't see me for a few days....15:24
mlavalleLOL15:24
mlavallewill you be hiding, curled with your blanket haleyb?15:25
haleybit took me a day to almost catch-up so am behind a little, and today i am having blanky-time, snow day here15:25
Swamihaleyb: may be relaxing in the beach.15:25
haleybbeach is next month :)15:26
mlavalleyeah, that's true, the cold cyclon15:26
mlavallehaleyb: how is it treating you?15:26
haleybit's just a snow storm, weather channel loves to make things bigger than they are15:26
mlavallegreat to hear15:26
mlavalleany other bugs to discuss?15:27
*** shuyingya has joined #openstack-meeting-315:27
mlavalleok, let's move on15:28
mlavalle#topic dvr_openflow15:28
*** openstack changes topic to "dvr_openflow (Meeting topic: neutron_l3)"15:28
mlavalledavid_chou: any updates this week?15:29
*** sdague has quit IRC15:29
*** sdague has joined #openstack-meeting-315:29
david_chouI tried to run base ovs-dpdk test case, but failed.15:29
david_chouThe prpose of the try is to setup base for test dvr_openflow with ovs-dpdk.15:30
david_chouStill looked into the issue. No other update.15:31
*** shuyingya has quit IRC15:32
mlavallethanks for the update. let us know if we can be of help :-)15:32
mlavalle#topic Multiple Port Bindings for Nova15:32
*** openstack changes topic to "Multiple Port Bindings for Nova (Meeting topic: neutron_l3)"15:32
mlavalleSo in reality, the work I did during the Holidays was mostly on this topic, adding the ability to ML2 to handle multiple port bindings15:34
*** lhx_ has quit IRC15:34
mlavallethe work is in this patch https://review.openstack.org/#/c/414251/15:34
mlavalleI think I ma done with what needs to be added to ML215:34
mlavalleI got a very nice review from ihrachys last night15:35
mlavalleso I will be getting that patch in shape with the help of his feedback15:35
*** lhx_ has joined #openstack-meeting-315:36
mlavalleand I will follow up with another patch that updates the rpc channel and the agents to handle the activation and de-activation of port bindings15:36
mlavalleany questions or comments?15:37
mlavalleok, moving on15:38
mlavalle#topic Port forwarding15:38
*** openstack changes topic to "Port forwarding (Meeting topic: neutron_l3)"15:38
mlavalleAs I've mentioned before, there is a spec for this topic: https://review.openstack.org/#/c/470596/15:38
mlavalleit made a lot of progress over the past few weeks15:38
mlavallein Decmber it got feedback from several people15:39
mlavalleI will look again at it today or tomorrow15:39
mlavalleif you have time, please take a look15:39
mlavalleok. moving on15:41
mlavalle#topic Open Agenda15:41
*** openstack changes topic to "Open Agenda (Meeting topic: neutron_l3)"15:41
mlavalleany other topics anyone would like to bring up today?15:41
mlavalleok, thanks for attending15:43
mlavallehave a very successful 2018!15:43
mlavalle#endmeeting15:43
*** openstack changes topic to "milestone 2 (Meeting topic: monasca)"15:43
Swamibye15:43
openstackMeeting ended Thu Jan  4 15:43:40 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:43
openstackMinutes:        http://eavesdrop.openstack.org/meetings/neutron_l3/2018/neutron_l3.2018-01-04-15.01.html15:43
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/neutron_l3/2018/neutron_l3.2018-01-04-15.01.txt15:43
openstackLog:            http://eavesdrop.openstack.org/meetings/neutron_l3/2018/neutron_l3.2018-01-04-15.01.log.html15:43
david_chouThanks, Bye.15:44
*** VW_ has joined #openstack-meeting-315:49
*** elmiko has joined #openstack-meeting-315:51
*** VW has quit IRC15:51
edleafe#startmeeting api-sig16:01
openstackMeeting started Thu Jan  4 16:01:04 2018 UTC and is due to finish in 60 minutes.  The chair is edleafe. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
*** openstack changes topic to " (Meeting topic: api-sig)"16:01
openstackThe meeting name has been set to 'api_sig'16:01
elmikoo/16:01
edleafe#chair cdent elmiko edleafe dtantsur16:01
edleafe#link https://wiki.openstack.org/wiki/Meetings/API-SIG#Agenda16:01
openstackWarning: Nick not in channel: cdent16:01
openstackWarning: Nick not in channel: dtantsur16:01
openstackCurrent chairs: cdent dtantsur edleafe elmiko16:01
*** cdent has joined #openstack-meeting-316:01
edleafelet's wait for cdent16:01
cdentsorry, lost track of time in tc office hours...16:02
edleafedtantsur said he wouldn't be able to make it16:02
edleafe#topic previous meeting action items16:02
*** openstack changes topic to "previous meeting action items (Meeting topic: api-sig)"16:02
edleafe#link http://eavesdrop.openstack.org/meetings/api_sig/2017/16:02
edleafeThere were none, so...16:02
edleafe#topic open mic and ongoing or new biz16:02
*** openstack changes topic to "open mic and ongoing or new biz (Meeting topic: api-sig)"16:02
edleafeAre any of the topics on the agenda in need of discussion?16:03
edleafeOr are they just leftover from the last meeting?16:03
elmikoi didn't add anything16:03
*** raildo has quit IRC16:04
*** jgu has joined #openstack-meeting-316:04
*** MarkBaker has quit IRC16:04
edleafecdent: anything to discuss about the foundation summary report?16:04
cdentI added a few things to the agenda, to try and use it as a way of keeping track of stuff that is sort of in progress16:05
cdentthe main new thing was the foundation summary thing, which is just that they want someone to write up a bit of a summary of 2017 in api-wg/sig16:05
cdentwhich I'll do16:05
cdent#action cdent write up foundation summary thing16:05
elmikothanks cdent16:06
cdentthe plan is to mention a few highlights (interop, service discovery, becoming a sig, sdk exploration) and leave it at that16:06
* edleafe breathes a sigh of relief16:06
elmikosounds good to me, maybe mention adding a new core too?16:07
cdentsure, makes sense16:07
edleafeOK, so anything else for new/ongoing biz?16:09
cdentnot entirely related, but did people see my post on gabbi tempest to os-dev today?16:09
*** MarkBaker has joined #openstack-meeting-316:09
cdentit's kinda cool16:09
elmikoi did not, but will now =)16:09
edleafenot yet - still getting settled after wasting 2 hours commuting16:10
cdentsorry for your luck16:10
elmiko+116:11
mugsieI added info about generating machine readible output to os-api-ref to https://review.openstack.org/52446716:13
mugsieand a patch to os-api-ref to do it - https://review.openstack.org/#/c/528801/16:13
cdentmugsie++16:13
cdentwhat's your general level of happiness with that? is it "a nice try" or "omg I love this" or "let's never look here again"?16:14
mugsiethe patch to os-api-ref is hacky as hell, but if people are interested I can clean it up - I think "a nice try"16:14
elmikowhile i definitely am on the mugsie++ bandwagon, i like this effort, i thought we backed away from advising openapi(ex. swagger) for the api definitions, did that change?16:14
cdentwe're still backed awayfrom that16:15
cdentthis is something nearby but not the same16:15
mugsieelmiko: its not OpenAPI - it looks like http://paste.openstack.org/show/629241/16:15
mugsiemicroversions are also not accounted for yet16:15
cdentsuch a bugbear16:16
elmikoahh, ok. thanks! that's what i get for skimming the pr...16:16
edleafeKinda looks like SOAP++16:16
cdentyeah, I'm very conflicted about this stuff16:17
mugsieyeah - it looks a bit like what gilles had done before16:17
cdentI can see how/why people want it16:17
cdentbut I don't think it will result in excellent apis16:17
cdentI'm not sure how (or if I even need) to resolve that conflict16:18
edleafeWhat is the use case that is driving this interest?16:18
mugsieauto generating client libs16:18
cdentgilles is already using the docs to do so, but thinks more structured will be better16:19
elmikoi agree that a structured format is the way to go, will someone need to develop tools for the generation stuff given that this is not openapi?16:20
elmikoor is gilles handling that16:20
mugsiehe has a ruby lib that does it already, but not for this exact format16:21
mugsiebut only for ruby16:21
elmikoack, thanks16:21
cdentthat's part of the issue I have: if we're going to have a structured format, we should do something standard if possible (says the guy who made gabbi's own special format...)16:21
elmikoi tend to agree16:21
elmiko(on both accounts XD)16:21
edleafethen we should just add it to gabbi16:22
mugsieI would agree, but afaik microversions have a issue being modeled by OpenAPI ?16:22
* edleafe runs16:22
mugsieedleafe: ++ /s16:22
elmikomugsie: yeah, that is the main issue as i see it for openapi in the openstack ecosystem16:22
mugsieopenapi is great for designing a new API, and modeling a lot, but it has a few built in (common sense IMHO) constraints16:23
* mugsie now ducks16:23
*** marios has quit IRC16:23
cdent(as an aside, I have a todo list for gabbi<->openapi transpiler like thing, but stalled on time)16:23
cdentmugsie: you're quite right about that too16:24
elmikoyeah, agreed mugsie16:26
edleafeready to move on?16:26
cdentis there a next step?16:26
cdentso that the ball doesn't get dropped?16:26
*** Swami has quit IRC16:27
mugsienot sure - gilles seems happy with it, but I am not sure a lot of people actually care either way?16:27
cdenta persistent problem16:27
elmikoi tend to agree, this api schema stuff is far enough into the weeds that few decide to venture this way16:27
elmikobut, it could have a big impact if we could figure it out16:27
elmikolet we = someone in the community at large16:28
mugsieyeah - there is a ML thread gilles started - let me resurect that and see what happens16:28
elmiko+116:28
edleafesounds good16:28
edleafelet's move on16:29
edleafe#topic guidelines16:29
edleafe#link https://review.openstack.org/#/q/status:open+project:openstack/api-wg,n,z16:29
*** openstack changes topic to "guidelines (Meeting topic: api-sig)"16:29
edleafe#link https://review.openstack.org/#/q/status:open+project:openstack/api-sig,n,z16:29
edleafeNothing new here16:29
edleafeAnyone want to talk about guidelines?16:30
cdentnot today16:30
edleafe#topic bug review16:30
edleafe#link https://bugs.launchpad.net/openstack-api-wg16:30
*** openstack changes topic to "bug review (Meeting topic: api-sig)"16:30
edleafeNo new bugs16:30
edleafeC'mon people, you need to start creating more bugs!!!16:31
cdentno progress on bugs either, but it's been a complex time16:31
elmikoedleafe: can i make a bug about more bugs?16:31
* edleafe cues the Inception reference16:31
elmikohaha16:32
cdentI had some talks with mordred today and yesterday about issues with pagination inconsistency and how we need to start pushing harder those sorts of things being bugs16:32
cdentmaybe we can create joint bugs with various projects that violate16:32
elmikocdent: do you mean bugs on the projects that implement?16:32
cdentyeah16:33
edleafecdent: do the offending projects' API pre-date the guidelines?16:33
cdentfrequently yes, and that's the point. We need to start driving some change.16:33
elmikoi have mixed feelings about this16:34
edleafeit seems that we need to start breaking APIs to fix them16:34
elmikomainly due to the historical idea that we are not the api police16:34
edleafecdent: what could be the carrot we could use instead of the API police stick?16:35
cdentwe don't have to cross link the bugs to us, but in general I think openstack is doing itself a disservice of not treating inconsistencies as bugs16:35
mugsiecdent: ++16:35
elmikocdent: i think that's an entirely fair position16:35
cdentso we may not be the police, but somebody needs to be the complainant16:35
elmikoi can get behind an effort to work with the projects, open bugs, and raise these issues as long as it's not the api police16:36
*** e0ne has quit IRC16:36
elmikothe point about opening bugs and not necessarily cross-linking back to the sig is very sane16:37
elmikoi'm a little troubled by the idea that we might cause major version bumps for projects that need to restructure their pagination api though16:37
elmikobut maybe that's just developer angst on my part16:38
mugsieelmiko: we shouldn't be - we should support decent version discovery, and itterate as needed16:38
cdentyou can support the old and the new in the same output16:38
cdents/can/can also/16:38
cdentbut we needn't fall in a hole here, the real goal is to make things better, how we do that remains to be discovered, that we don't know should not stop us hoping16:39
*** lhx_ has quit IRC16:39
elmikoagreed mugsie and cdent, i'm just being paranoid most liekly16:40
elmikowe should promote the forward movement and improvement of these projects16:40
* cdent didn't think elmiko livedin california16:40
* elmiko chuckles16:40
*** sdague has quit IRC16:40
*** sdague has joined #openstack-meeting-316:40
mugsiethen again I am the one deleting designate'sV1 API, so I am on the opposite side of the arguement16:41
elmikohahaha16:41
edleafemugsie: https://blog.leafe.com/api-longevity/16:42
mugsieedleafe: yeah - it has taken us a while :) - we have had it turned off by default for a year now16:44
*** slaweq has quit IRC16:44
elmikogood analogy in there edleafe +116:44
mugsieyeah - I like it16:44
edleafeSo what action will we take to handle this "encouragement" of projects to update their pagination links?16:46
*** raildo has joined #openstack-meeting-316:46
*** anilvenkata has quit IRC16:46
cdentnot sure, was just throwing it out there as an anecdote16:46
edleafeOK, I'll add it to the agenda so we can discuss next week - maybe someone will have some ideas16:47
edleafe#topic weekly newsletter16:47
edleafe#link https://etherpad.openstack.org/p/api-sig-newsletter16:47
*** openstack changes topic to "weekly newsletter (Meeting topic: api-sig)"16:47
edleafeVolunteers?16:47
cdenti have a meeting immediately after this one today16:47
elmikoi can take it16:47
cdentthanks elmiko16:48
edleafecool, thanks16:48
edleafeI can review when it's ready16:48
elmikocool, thanks16:48
edleafeWith that, let's get back to whatever we were doing before16:48
cdentcool, cool16:48
edleafe#endmeeting16:48
cdentand cool16:48
*** openstack changes topic to "milestone 2 (Meeting topic: monasca)"16:48
openstackMeeting ended Thu Jan  4 16:48:45 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:48
openstackMinutes:        http://eavesdrop.openstack.org/meetings/api_sig/2018/api_sig.2018-01-04-16.01.html16:48
cdentthanks edleafe16:48
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/api_sig/2018/api_sig.2018-01-04-16.01.txt16:48
openstackLog:            http://eavesdrop.openstack.org/meetings/api_sig/2018/api_sig.2018-01-04-16.01.log.html16:48
* cdent finds another computer16:49
*** cdent has quit IRC16:49
*** gcb has quit IRC16:49
*** gcb has joined #openstack-meeting-316:50
*** pcaruana has quit IRC16:52
*** Guest3856 is now known as dansmith16:53
*** sdague has quit IRC16:55
*** sdague has joined #openstack-meeting-316:56
*** yamahata has joined #openstack-meeting-316:56
*** ChanServ changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"17:10
*** slaweq has joined #openstack-meeting-317:10
*** slaweq has quit IRC17:15
*** spzala has joined #openstack-meeting-317:18
*** spzala has quit IRC17:18
*** MarkBaker has quit IRC17:35
*** raildo has quit IRC17:41
*** raildo has joined #openstack-meeting-317:42
*** lpetrut has quit IRC17:44
*** slaweq has joined #openstack-meeting-317:47
*** tbachman has joined #openstack-meeting-317:52
*** david-lyle has quit IRC18:00
tbachmanFYI — GBP meeting will start ~15 minutes past the hour18:01
*** david-lyle has joined #openstack-meeting-318:01
*** rkukura has joined #openstack-meeting-318:05
*** gmann__ has quit IRC18:09
*** slaweq has quit IRC18:10
*** SumitNaiksatam has joined #openstack-meeting-318:14
rkukurahi SumitNaiksatam!18:15
SumitNaiksatamrkukura: tbachman : hi18:15
tbachmanSumitNaiksatam: hi!18:15
rkukurahi tbachman!18:15
tbachmanrkukura: hi!18:15
* tbachman looks for annakk18:16
SumitNaiksatamapologies for the delay18:16
tbachmanSumitNaiksatam: no worries!18:16
SumitNaiksatamrkukura: is the heater working?18:16
* tbachman hands rkukura a hot beverage18:16
rkukuraall good18:17
SumitNaiksatamtbachman: lol18:17
SumitNaiksatamtbachman: i guess it must be fairly cold for you guys too, right?18:17
tbachmanwe can always burn our laptops to keep warm, now that they’ve been reduced to security-less rubble18:17
rkukurabut haven’t had any solar power at all since Christmas day18:17
tbachmanrkukura: wow18:17
SumitNaiksatamtbachman: lol++18:17
rkukurasnow usually slides off the panels, but I think it froze on18:18
tbachmanah18:18
tbachmanreflecting light then?18:18
rkukurareflecting/scattering/blocking18:18
tbachmanack18:18
SumitNaiksatamwow18:18
SumitNaiksatamdidnt think of that18:19
SumitNaiksatam#startmeeting networking_policy18:19
openstackMeeting started Thu Jan  4 18:19:22 2018 UTC and is due to finish in 60 minutes.  The chair is SumitNaiksatam. Information about MeetBot at http://wiki.debian.org/MeetBot.18:19
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.18:19
*** openstack changes topic to " (Meeting topic: networking_policy)"18:19
openstackThe meeting name has been set to 'networking_policy'18:19
tbachmanSumitNaiksatam: not an issue in your part of the world ;)18:19
SumitNaiksatamtbachman: :-)18:19
SumitNaiksatam(but we dont have solar panels either)18:19
SumitNaiksatami still pretty much have the one point agenda item - Pike Sync18:20
SumitNaiksatamthanks for merging the patches from the other repos18:20
SumitNaiksatamthe main server patch - #link https://review.openstack.org/51818318:20
SumitNaiksatamoh wow, i just noticed the lastest set of comments18:21
rkukuralast ones from me, I think ;)18:21
SumitNaiksatamyeah, last couple of iterations are your comments18:22
SumitNaiksatamanyway, i havent had a chance to look at these latest comments18:22
SumitNaiksatamso i will do those offline18:22
rkukuramain thing is a few remaining places where ensure_tenant calls were moved inside transactions18:23
*** slaweq has joined #openstack-meeting-318:23
SumitNaiksatam“ensure_tenant() calls in the apic_aim L3_router driver and in the GBP plugin"18:23
rkukuraright18:23
SumitNaiksatami think i already moved all the GBP plugin calls18:23
rkukuraI also noticed a couple places where ensure_tenant() calls are completely missing, but that should probably be addressed via a followon patch and backported18:24
SumitNaiksatamI can put a revisit wherever that is the case18:24
rkukuraok18:24
rkukuraI was also hoping we’d be able to use reader transactions in some places, but I think the way status gets written to the DB may be preventing that18:24
*** annakk has joined #openstack-meeting-318:25
tbachmanannakk: hi!18:25
rkukurahi annakk!18:25
annakkhi, sorry I'm late18:25
SumitNaiksatamrkukura: yes, we have to use writer there (that discussion happened in some of the earlier patchsets, it was raised by annakk)18:25
SumitNaiksatamannakk: hi, no worries, we are just getting “warmed” up18:26
*** yamahata has quit IRC18:26
* tbachman hands another hot beverage to rkukura 18:26
SumitNaiksatamtbachman: lol18:26
rkukurain general, would a writer transaction nested inside a reader transaction promote the reader transaction to a writer?18:26
tbachmancan you have a transaction inside a transaction?18:27
tbachman(can you *start* a new transaction inside a transaction)18:27
SumitNaiksatamrkukura: so i had wrapped most of the ml2plus methods in the transactions earlier, with the assumption that the post_commit for the plugin/driver combination is essentially a no-op18:27
rkukuraif so, I think it would be best if the get methods generally used readers, and we nested a writer if we need to update the status18:27
SumitNaiksatamtbachman: that depends18:27
SumitNaiksatamif you nest a writer inside a reader i think it leads to a new transaction18:27
*** slaweq has quit IRC18:27
* tbachman needs to read up on mysql18:28
tbachmansqlalchemy, that is18:28
SumitNaiksatamtbachman: this is not sqlalchemy specific since we are dealing with the oslo_db abstraction18:28
rkukuraSumitNaiksatam: it is still possible to use regular ML2 MDs, like ovs or linuxbridge, with ml2plus18:28
SumitNaiksatamrkukura: one sec18:28
SumitNaiksatamrkukura: so after removing those (per your review comment yesterday), it led to this error:18:29
SumitNaiksatam#link http://logs.openstack.org/83/518183/44/check/legacy-group-based-policy-dsvm-aim/0f7a787/logs/screen-q-svc.txt.gz?level=ERROR18:29
*** yamamoto has quit IRC18:29
SumitNaiksatami had seen this before as well (and had done the wrapping in response)18:29
SumitNaiksatami was able to get around this but adding a writer transaction block at another place inside the aim_mapping PD18:30
SumitNaiksatami will add a note in the code where i added that to explain18:30
SumitNaiksatami think the security_group DB module/handling is doing something, which causes the security_group_rule to be created in a different transaction18:31
SumitNaiksatamhowever putting the transaction block in the PD fixes that18:31
rkukuraOK, if we need to use writers to avoid this, fine, but we still should make sure the ensure_tenant() calls are before the transaction starts18:31
SumitNaiksatamso, as it stands ensure_tenants() are all outside teh transaction, and i have in fact removed the wrapping transaction from all the ml2plus methods18:32
SumitNaiksatami will move the ensure_tenant out of the l3_plugin transaction18:32
rkukurathey are inside in the apic_aim l3_router driver and in the GBP plugin - see my comments in gerrit18:32
*** yamamoto has joined #openstack-meeting-318:33
rkukurahttps://review.openstack.org/#/c/518183/45/gbpservice/neutron/services/grouppolicy/plugin.py@46718:33
SumitNaiksatam“i will move the ensure_tenant out of the l3_plugin transaction”18:33
SumitNaiksatamthe other comment was here: #link https://review.openstack.org/#/c/518183/43/gbpservice/neutron/plugins/ml2plus/drivers/apic_aim/mechanism_driver.py18:34
SumitNaiksatami dont recollect if i had to change this before or after the eager loading18:34
SumitNaiksatamchange18:34
SumitNaiksatambut at one point this was definitely not working18:35
SumitNaiksatami will try to revert and see what happens18:35
rkukuraOK, and if it doesn’t work, please add a REVISIT18:35
SumitNaiksatamokay18:36
rkukurathanks18:36
SumitNaiksatamwas planning to do that18:36
SumitNaiksatamthis comment - #link https://review.openstack.org/#/c/518183/43/gbpservice/neutron/plugins/ml2plus/patch_neutron.py18:36
SumitNaiksatami have changed the wording18:36
SumitNaiksatambut the reason i had put ml2plus is because the patching happens only when ml2plus is ussed18:36
*** numans_afk is now known as numans18:36
SumitNaiksatam*used18:36
rkukuraok18:37
SumitNaiksatamalso there are at least a couple of places in ml2plus where we send out notifications from within a transaction18:37
annakkI probably didn't look well enough, but I didn't see any keystone stuff in _ensure_tenant - just comments that its for UTs only..18:37
rkukuraThere are of course cases where neutron methods are called within GBP transactions, so we still need that monkey patch, right?18:37
*** yamamoto has quit IRC18:37
rkukura?18:37
SumitNaiksatamrkukura: the patching is not needed for resource_mapping18:38
SumitNaiksatambut more importantly, per earlier comment, its needed just for ml2plus too since we have some notifications going from within the transactions which needed to be queued18:39
rkukuraensure_tenant() calls the project_name_cache.ensure_project(), which calls keystone if the project is not already in the cache18:39
SumitNaiksatamannakk: i missed that part too18:40
SumitNaiksatamso that covers patchset 4318:40
SumitNaiksatamon patchset 45, rkukura’s latest comments18:41
* tbachman hands SumitNaiksatam an adult beverage, pats him on the back18:41
SumitNaiksatamtbachman: lol18:42
rkukurathanks tbachman ;)18:42
SumitNaiksatamwas just quickly going through the comments18:42
tbachmanI can’t remember what my personal best is for patch sets18:42
tbachmanit’s pretty high18:42
SumitNaiksatami think we covered the ensure_tenant (yes, i will move the GBP plugin as well) and the writer versus reader for the get_resources18:43
rkukuraok, great18:43
SumitNaiksatam“Should the session.begin() calls in these _ml2_md_extend_<resource>_dict()"18:43
SumitNaiksatamannakk: had raised this one earlier too18:43
SumitNaiksatamher question was whether these can be eliminated18:43
SumitNaiksatamand the issue at that time was that the DB object used in there would be wierdly if the subntransaction was not created18:44
rkukuraI have no idea if they are really needed18:44
SumitNaiksatamhowever that was before i made the earlier mentioned change to wrap the create_security_group (inside the aim_mapping PD) in a transaction18:44
SumitNaiksatamso i will revisit if we can completely eliminate the creation of the (sub)transaction here18:45
rkukuraIdeally, the extend functions would always be called inside the main API transaction, but that isn’t always the case18:45
SumitNaiksatamrkukura: yeah, thats the problem18:45
SumitNaiksatamthe thing is that these changes have to be tried independently18:46
SumitNaiksatamotherwise you dont know what is breaking what18:46
* tbachman nods18:46
SumitNaiksatamso its a new incremental patchset for each of these changes18:46
rkukuraIf it were me, I’d probably convert the begins to readers for now, and then see if they can be eliminated in a separate patch18:47
SumitNaiksatamand the gate craps out almost 2 in 3 times on the HOST_IP error18:47
tbachman:(18:47
rkukuraaren’t you able to run the UTs locally?18:47
SumitNaiksatami tried earlier to fix it, and thought i had, but its come back again18:47
SumitNaiksatamrkukura: this does not manifest in the UTs18:48
rkukuraoh18:48
SumitNaiksatamin fact i cannot even locally reproduce some of the errors even if i run devstack18:48
SumitNaiksatami think its a timing issue, since neutron is doing some expunging of DB objects18:48
tbachmannumber of threads?18:49
SumitNaiksatamsynth_db_objs]18:49
SumitNaiksatamJan 03 23:45:08.268557 ubuntu-xenial-infracloud-chocolate-0001674228 neutron-server[14315]: ERROR gbpservice.neutron.services.grouppolicy.policy_driver_manager   File "/opt/stack/new/neutron/neutron/objects/base.py", line 405, in _load_object18:49
SumitNaiksatamJan 03 23:45:08.268733 ubuntu-xenial-infracloud-chocolate-0001674228 neutron-server[14315]: ERROR gbpservice.neutron.services.grouppolicy.policy_driver_manager     context.session.expunge(obj.db_obj)18:49
SumitNaiksatamJan 03 23:45:08.268896 ubuntu-xenial-infracloud-chocolate-0001674228 neutron-server[14315]: ERROR gbpservice.neutron.services.grouppolicy.policy_driver_manager   File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1586, in expunge18:49
SumitNaiksatamJan 03 23:45:08.269072 ubuntu-xenial-infracloud-chocolate-0001674228 neutron-server[14315]: ERROR gbpservice.neutron.services.grouppolicy.policy_driver_manager     state_str(state))18:49
SumitNaiksatamJan 03 23:45:08.269255 ubuntu-xenial-infracloud-chocolate-0001674228 neutron-server[14315]: ERROR gbpservice.neutron.services.grouppolicy.policy_driver_manager InvalidRequestError: Instance <SecurityGroupRule at 0x7f0674653590> is not present in this Session18:49
SumitNaiksatamtbachman: possibly18:49
SumitNaiksatamanyway, so on patchset 45, i think most of the other comments are questions or comments which i can answer inline18:50
rkukuraagreed18:50
SumitNaiksatamrkukura: will reach out if something needs more discussion18:50
rkukurasure18:50
tbachmanSumitNaiksatam: thanks for tackling this beast!18:50
SumitNaiksatamhopefully track to knock this off immedicately after this meeting18:50
SumitNaiksatamtbachman: no worries18:51
SumitNaiksatami already created the pike branches for the other repos18:51
SumitNaiksatamsince the patches were merged18:51
rkukuranice18:51
rkukuraI’ll be ready to re-review18:51
SumitNaiksatamrkukura: thanks, i guess not the best way to keep your spirits up! :-)18:52
rkukurano problem18:52
SumitNaiksatamannakk: were you able to try this with your NSX devstack?18:52
rkukuraI admit I sometimes post my review after a few comments, without looking at everything, but I got throught everything this time18:53
annakknot yet, planning to18:53
SumitNaiksatamannakk: okay18:53
tbachmantimecheck18:53
SumitNaiksatamrkukura: np, i try to post as i go along too, since it gives the author more immediate feedback18:54
SumitNaiksatamtbachman: good point18:54
SumitNaiksatamapologies again for starting late today18:54
tbachmanSumitNaiksatam:  no worries!18:54
SumitNaiksatamtbachman: so the weather okay in your neck of the woods?18:54
tbachmancold, but not that much snow18:54
tbachmaneast of us got a lot18:54
tbachmanway east18:54
tbachman(near the ocean)18:54
SumitNaiksatamtbachman: storm/hurricane/bomb-cyclone?18:55
tbachmanheh18:55
tbachmannothing bad18:55
tbachmanIt’s all relative18:55
tbachmanjust cold ;)18:56
rkukurahere, we are getting plenty of snow (about a foot so far today) but the wind isn’t too bad18:56
SumitNaiksatamrkukura: oh good, reassuring18:56
SumitNaiksatamannakk: did you have anything else to bring up today?18:56
annakkno :)18:56
SumitNaiksatamalrighty then, thanks for making it (especially rkukura and tbachman) :-)18:57
tbachmanSumitNaiksatam: thanks for running the meeeting!18:57
SumitNaiksatamstay warm and stay safe18:57
tbachmanSumitNaiksatam: ack ;)18:57
rkukurathanks SumitNaiksatam!18:57
SumitNaiksatambye!18:57
rkukurabye18:58
SumitNaiksatam#endmeeting18:58
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"18:58
annakkbyE18:58
openstackMeeting ended Thu Jan  4 18:58:02 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:58
openstackMinutes:        http://eavesdrop.openstack.org/meetings/networking_policy/2018/networking_policy.2018-01-04-18.19.html18:58
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/networking_policy/2018/networking_policy.2018-01-04-18.19.txt18:58
openstackLog:            http://eavesdrop.openstack.org/meetings/networking_policy/2018/networking_policy.2018-01-04-18.19.log.html18:58
*** annakk has quit IRC18:58
*** tbachman has left #openstack-meeting-318:58
*** rkukura has left #openstack-meeting-318:58
*** slaweq has joined #openstack-meeting-319:01
*** yamahata has joined #openstack-meeting-319:04
*** slaweq has quit IRC19:05
*** slaweq has joined #openstack-meeting-319:14
*** slaweq has quit IRC19:19
*** slaweq has joined #openstack-meeting-319:35
*** yamamoto has joined #openstack-meeting-319:35
*** elmiko has left #openstack-meeting-319:38
*** slaweq has quit IRC19:39
*** yamamoto has quit IRC19:42
*** mlavalle has left #openstack-meeting-320:00
*** spzala has joined #openstack-meeting-320:04
*** markmcclain has quit IRC20:10
*** markmcclain has joined #openstack-meeting-320:11
*** slaweq has joined #openstack-meeting-320:13
*** SumitNaiksatam has quit IRC20:13
*** raildo has quit IRC20:14
*** slaweq_ has joined #openstack-meeting-320:15
*** slaweq has quit IRC20:18
*** chyka has joined #openstack-meeting-320:25
*** raildo has joined #openstack-meeting-320:26
*** slaweq has joined #openstack-meeting-320:50
*** lyan has quit IRC20:54
*** raildo has quit IRC20:56
*** lyan has joined #openstack-meeting-321:00
*** lyan has quit IRC21:02
*** lyan has joined #openstack-meeting-321:06
*** cleong has quit IRC21:23
*** slaweq has quit IRC21:23
*** slaweq_ has quit IRC21:29
*** slaweq has joined #openstack-meeting-321:32
*** slaweq has quit IRC21:37
*** chyka has quit IRC21:43
*** chyka has joined #openstack-meeting-321:44
*** bobh has quit IRC21:45
*** slaweq_ has joined #openstack-meeting-321:50
*** slaweq_ has quit IRC21:54
*** slaweq has joined #openstack-meeting-322:05
*** slaweq has quit IRC22:10
*** markmcclain has quit IRC22:10
*** markmcclain has joined #openstack-meeting-322:11
*** markmcclain has quit IRC22:19
*** markmcclain has joined #openstack-meeting-322:20
*** bobh has joined #openstack-meeting-322:26
*** bobh has quit IRC22:38
*** slaweq has joined #openstack-meeting-322:43
*** lyan has quit IRC22:56
*** flwang has quit IRC23:14
*** hongbin has quit IRC23:18
*** slaweq has quit IRC23:19
*** flwang has joined #openstack-meeting-323:24
*** mtreinish has quit IRC23:39
*** mtreinish has joined #openstack-meeting-323:42
*** slaweq has joined #openstack-meeting-323:48
*** slaweq has quit IRC23:53
*** lyan has joined #openstack-meeting-323:54
*** liangy has joined #openstack-meeting-323:56
*** lyan has quit IRC23:58

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