Wednesday, 2019-09-11

*** goldyfruit___ has joined #openstack-lbaas00:11
johnsomPlease review so we can merge chains: https://review.opendev.org/#/c/673712/00:49
*** spatel has joined #openstack-lbaas01:24
*** tkajinam has quit IRC01:25
*** tkajinam has joined #openstack-lbaas01:26
*** yamamoto has joined #openstack-lbaas02:56
*** spatel has quit IRC03:03
openstackgerritMerged openstack/octavia master: Fix cleanup of expired load balancer entries  https://review.opendev.org/68040004:08
openstackgerritMichael Johnson proposed openstack/octavia master: Generate PDF documentation  https://review.opendev.org/66724904:23
*** ianychoi_ has joined #openstack-lbaas04:24
*** ianychoi has quit IRC04:27
*** pcaruana has joined #openstack-lbaas04:42
*** gcheresh_ has joined #openstack-lbaas04:46
*** gcheresh_ has quit IRC05:01
*** pcaruana has quit IRC05:12
*** gcheresh_ has joined #openstack-lbaas05:13
*** rcernin has quit IRC05:22
*** redrobot has quit IRC05:25
*** rcernin has joined #openstack-lbaas05:38
*** rcernin has quit IRC05:51
*** rpittau|afk is now known as rpittau06:01
*** yamamoto has quit IRC06:06
*** rcernin has joined #openstack-lbaas06:09
*** luksky has joined #openstack-lbaas06:19
*** pcaruana has joined #openstack-lbaas06:21
openstackgerritVishal Manchanda proposed openstack/octavia-dashboard master: Generate PDF documentation  https://review.opendev.org/67928306:35
*** ricolin has joined #openstack-lbaas06:38
*** ricolin has quit IRC06:39
*** ccamposr has joined #openstack-lbaas06:42
*** gcheresh_ has quit IRC06:43
*** ccamposr__ has quit IRC06:44
*** maciejjozefczyk has joined #openstack-lbaas06:51
openstackgerritAnn Taraday proposed openstack/octavia master: Convert listener flows to use provider models  https://review.opendev.org/66023606:51
openstackgerritAnn Taraday proposed openstack/octavia master: Convert pool flows to use dicts  https://review.opendev.org/66538106:52
openstackgerritAnn Taraday proposed openstack/octavia master: Transition member flows to use dicts  https://review.opendev.org/65784206:52
openstackgerritAnn Taraday proposed openstack/octavia master: Transition amphora flows to dicts  https://review.opendev.org/66889806:53
openstackgerritAnn Taraday proposed openstack/octavia master: Convert Lb flows to use provider dicts  https://review.opendev.org/67172506:53
*** gcheresh_ has joined #openstack-lbaas06:54
openstackgerritAnn Taraday proposed openstack/octavia master: Transition l7policy flows to dicts  https://review.opendev.org/66597706:56
*** ataraday_ has quit IRC07:02
*** ccamposr__ has joined #openstack-lbaas07:05
*** ccamposr has quit IRC07:08
*** trident has quit IRC07:08
*** ivve has joined #openstack-lbaas07:09
*** luksky has quit IRC07:10
*** tesseract has joined #openstack-lbaas07:11
*** trident has joined #openstack-lbaas07:17
*** trident has quit IRC07:22
*** luksky has joined #openstack-lbaas07:23
*** trident has joined #openstack-lbaas07:31
*** rcernin has quit IRC07:38
*** luksky has quit IRC07:51
*** luksky has joined #openstack-lbaas07:57
cgoncalvesuh, https://review.opendev.org/#/c/681242 (re: py2 drop)08:09
*** luksky has quit IRC08:21
*** tkajinam has quit IRC08:27
*** ianychoi_ has quit IRC09:09
*** rcernin has joined #openstack-lbaas09:10
openstackgerritjiasirui proposed openstack/octavia-dashboard master: fix the spelling mistakes  https://review.opendev.org/68142409:39
openstackgerritMerged openstack/octavia master: Move to using octavia-lib constants  https://review.opendev.org/67371209:40
*** rcernin has quit IRC09:41
rm_workWelcome back from vacation? :D09:50
rm_workHope it was a fun one09:50
rm_workFYI it's highly unlikely I'll make meeting time today, hopefully Michael will run it09:51
openstackgerritMaciej Józefczyk proposed openstack/octavia master: Add new algorithm SOURCE_IP_PORT  https://review.opendev.org/67246310:01
*** ccamposr has joined #openstack-lbaas10:05
openstackgerritMaciej Józefczyk proposed openstack/octavia master: Validate supported LB algorithm in Amphora provider drivers  https://review.opendev.org/67247710:07
maciejjozefczyk\o10:07
maciejjozefczykHey folks! Can we proceed with those patches? https://review.opendev.org/#/q/topic:source_ip_port_octavia_ovn+(status:open+OR+status:merged)10:07
*** ccamposr__ has quit IRC10:08
maciejjozefczykthose are after reviews :)10:09
*** salmankhan has joined #openstack-lbaas10:15
*** salmankhan has quit IRC10:15
*** salmankhan has joined #openstack-lbaas10:16
*** maciejjozefczyk has quit IRC10:24
*** maciejjozefczyk has joined #openstack-lbaas10:25
*** maciejjozefczyk has quit IRC10:28
*** maciejjozefczyk has joined #openstack-lbaas10:29
*** ianychoi has joined #openstack-lbaas10:30
*** ianychoi has quit IRC10:45
*** ianychoi has joined #openstack-lbaas10:45
openstackgerritMerged openstack/octavia master: Support create amphora instance from volume based.  https://review.opendev.org/57050511:08
*** maciejjozefczyk has quit IRC11:19
*** maciejjozefczyk has joined #openstack-lbaas11:21
*** boden has joined #openstack-lbaas11:52
*** luksky has joined #openstack-lbaas11:55
*** goldyfruit___ has quit IRC12:15
openstackgerritMerged openstack/octavia stable/rocky: Add failover logging to show the amphora details.  https://review.opendev.org/67976912:24
*** happyhemant has joined #openstack-lbaas12:30
*** maciejjozefczyk has quit IRC12:43
*** goldyfruit___ has joined #openstack-lbaas13:29
*** redrobot has joined #openstack-lbaas13:50
*** rpittau is now known as rpittau|afk13:54
*** maciejjozefczyk has joined #openstack-lbaas14:09
*** Vorrtex has joined #openstack-lbaas14:10
*** ricolin has joined #openstack-lbaas14:18
openstackgerritAnn Taraday proposed openstack/octavia master: Convert Lb flows to use provider dicts  https://review.opendev.org/67172514:39
*** ivve has quit IRC15:29
*** ataraday_ has joined #openstack-lbaas15:57
johnsom#startmeeting Octavia16:00
openstackMeeting started Wed Sep 11 16:00:29 2019 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: Octavia)"16:00
openstackThe meeting name has been set to 'octavia'16:00
johnsomHappy MS3 / Feature freeze week16:01
gthiemongeHi!16:01
ataraday_hi16:01
cgoncalveshi16:01
johnsom#topic Announcements16:02
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"16:02
johnsomIt is feature freeze week for the Train release16:02
johnsom#link https://releases.openstack.org/train/schedule.html16:02
*** spatel has joined #openstack-lbaas16:02
johnsomOh, and our new PTL could not make the meeting today, so I am running it... lol16:02
johnsomIf you have a feature patch (not docs, bug fixes, or test patches) that is not on the priority review list, speak now....16:03
johnsom#link https://etherpad.openstack.org/p/octavia-priority-reviews16:03
openstackgerritMerged openstack/octavia master: Add long-running provider agent support  https://review.opendev.org/67414016:03
ataraday_I want to ask about this bugfix16:04
ataraday_#link https://review.opendev.org/#/c/681195/16:04
ataraday_but I see it is for consideration for Train16:04
johnsomOtherwise it will likely get held until the Ussuri16:04
johnsomataraday_ Yes, that is just a bug fix, so we still have two weeks for those.16:05
colin-hello16:05
johnsomI have added it to the priority review list16:07
ataraday_johnsom, OK, thanks!16:07
*** spatel has quit IRC16:07
johnsomAny other Train feature candidates?16:07
johnsomI think it will be just a bit later in the week before we make the feature freeze release.16:08
johnsomOr maybe we will just wait for the RC1 week16:08
johnsomLikely RC1 week actually16:08
johnsomAlso I want to mention that the Ubuntu 18.04 kernel -60 is bad. If you have any NAT rules setup the kernel will panic.16:09
johnsom#link https://bugs.launchpad.net/ubuntu/+source/linux/+bug/184244716:09
openstackLaunchpad bug 1842447 in linux (Ubuntu) "Kernel Panic with linux-image-4.15.0-60-generic when specifying nameserver in docker-compose" [Undecided,Confirmed]16:09
johnsomThis was the root cause of our test jobs failing with RETRY_LIMIT.16:10
johnsomI spent a bunch of time tracking this down as zuul doesn't capture the logs from those hosts.16:10
johnsomVersion -62 that came out Monday resolves the bug.16:10
johnsomAnyway, if you are using floating IPs, skip the -60 kernel16:11
johnsomAny other announcements this week?16:11
*** gcheresh_ has quit IRC16:11
johnsom#topic Brief progress reports / bugs needing review16:12
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"16:12
johnsomI have been mostly consumed with doing reviews for features and fighting the gates to get our features merged.16:13
johnsomI have made very slow progress on the failover patch. Hopefully that will change soon.16:13
colin-shoutout to johnsom for patiently hand holding me through getting DIB to put haproxy 2.0 on a stable/stein amphora for me to test with. will be running with that this week and looking for any issues related to it16:13
johnsomAlso, I have made some progress on the PDF document goal. I think we will have one for Train, but it may have bugs. For example, there are whole sections that just don't render in the PDF.... joy16:14
johnsomcolin- Thanks. Glad you got it going.16:14
johnsomI also posted a patch for DIB to stop it from silently ignoring element scripts....16:14
johnsom#link https://review.opendev.org/68078516:15
johnsomThat would have made figuring out what we were doing wrong much easier....16:15
ataraday_All refactor changes are ready, start working closely on jobboard controller16:15
johnsomataraday_ Awesome!16:16
johnsomAny other updates this week?16:17
johnsom#topic Cycle release highlights16:17
*** openstack changes topic to "Cycle release highlights (Meeting topic: Octavia)"16:17
johnsom#link http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009137.html16:17
johnsomThe foundation collects release "highlights" used for marketing, etc.16:18
johnsomWe need to have those in by tomorrow. I plan to put a patch up today.16:18
johnsomObviously log offloading is a new Train feature. As is cinder backed amphora.16:18
johnsomI will probably add that the image size has been reduced.16:19
johnsomAny other highlights I should include?16:19
johnsomAssuming the VIP ACL patch lands, I will include that as well16:19
colin-that's the only one i'd mention, VIP ACL16:19
colin-lots of interest in that internally :)16:19
colin-so i'm guessing the same is true elsewhere16:19
johnsomYeah, it is a popular one16:20
colin-only other one*16:20
*** luksky has quit IRC16:21
johnsomOk, well let me know if you think of something. I just wanted to ask to make sure I didn't miss something.16:21
*** ricolin has quit IRC16:22
johnsom#topic Open Discussion16:22
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:22
johnsomOther topics this week?16:22
johnsomOk, well if there is nothing else this week we can call the meeting.16:24
colin-have we ever considered or proposed using anything other than keepalived for quorum between amp pairs?16:24
colin-don't have anything in mind just curious16:24
johnsomYeah, CARP was proposed at one point, but it turned out to be horrible16:24
colin-ok16:25
xgermanThere are some leader elect Lagos (Paxos, etc.) which have oython libraries — but then you need 3 or more amps...16:25
johnsomOther than that, no I don't think we have had proposals for an alternate.16:25
johnsomkeepalived is actually pretty darn efficient at what it does. If you really get in and tune it you can have sub-second failover.16:26
colin-o/ xgerman16:26
xgermanalways lurking :-)16:26
colin-good info thanks guys16:26
johnsomThere was some talk of doing MAC address migration, but that doesn't play nice with the overlays and ML2 options like DVR.16:27
johnsomMicrosofts offering does MAC migration16:27
johnsomAt least it did last time I touched a MS server. lol, that has been a while now16:28
colin-better knock on wood16:29
johnsomcolin- Oh, question for you, are you running OVS for your ML2?16:29
johnsomI am hearing rumblings that there is a duplicate MAC address bug in OVS deployments.16:29
johnsomI thought of you folks when I heard that.16:30
colin-not usinv OVS, no, using a provider network model with linuxbridge and VLANs managed on the network equipment16:31
johnsomOk16:31
johnsomIf there is nothing else, I will close out the meeting16:34
johnsom#endmeeting16:34
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews"16:34
openstackMeeting ended Wed Sep 11 16:34:31 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:34
openstackMinutes:        http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-09-11-16.00.html16:34
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-09-11-16.00.txt16:34
openstackLog:            http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-09-11-16.00.log.html16:34
johnsomThanks everyone16:34
*** ivve has joined #openstack-lbaas16:42
*** ataraday has quit IRC16:44
*** tesseract has quit IRC16:57
*** salmankhan has quit IRC17:13
*** luksky has joined #openstack-lbaas17:26
*** boden has quit IRC17:47
*** ramishra has quit IRC18:00
*** boden has joined #openstack-lbaas18:02
openstackgerritMerged openstack/octavia stable/stein: Add failover logging to show the amphora details.  https://review.opendev.org/67976818:18
*** maciejjozefczyk has quit IRC18:29
*** gcheresh_ has joined #openstack-lbaas18:55
rm_workOne of these days I'm just going to change the meeting time again, if y'all really need me to attend every week. :P19:04
rm_workOur old meeting time was much better19:04
*** gcheresh_ has quit IRC19:29
*** vishalmanchanda has quit IRC19:48
*** henriqueof has joined #openstack-lbaas19:51
xgermanrm_work: you are the PTL - it’s your job to run the meeting :-)20:29
rm_workI'm a leader -- it's my job to delegate and grow the skills of my team. :)20:29
johnsomrm_work You are saying my meeting skills need work?20:35
xgermanApparently - also by that measure johnsom you should stop submitting patches so others can step up :-)20:39
rm_workYou did an EXCELLENT job, thanks johnsom! I'm very proud of you.20:39
johnsomAgreed20:39
rm_workActually we should let cgoncalves run more meetings now that he should be back from vacation?20:40
rm_workThat way he can get a good amount of experience doing it. :)20:40
xgerman+120:40
openstackgerritMerged openstack/octavia master: Add `additive_only` parameter to Batch Member call  https://review.opendev.org/66748420:43
colin-thx for that!20:47
rm_workwoo lol21:02
rm_workwas wondering if that was ever gonna make it in :D21:02
rm_workone less thing to track21:02
rm_workand i saw we got the volume-based patch in?21:02
rm_workso we need to make sure that followup merges or our gates will break soon21:03
*** ivve has quit IRC21:05
*** ivve has joined #openstack-lbaas21:05
rm_worktrying to test it locally >_>21:06
*** Vorrtex has quit IRC21:12
*** rcernin has joined #openstack-lbaas21:15
rm_workjohnsom: i need to push another PS on that21:19
rm_work./octavia/tests/functional/api/v2/test_l7rule.py:1017:9: O323  assertEqual(True, A) or assertEqual(False, A) sentences not allowed21:19
johnsomOk21:19
rm_worknow it's passing for me locally with the updated flake8 req21:22
openstackgerritAdam Harwell proposed openstack/octavia master: Fix a few nits from the main volume-based patch  https://review.opendev.org/68114421:22
rm_workhow long was this broken <_<21:22
johnsomThree months21:24
johnsomLooks like it was some of my quality code.... sigh21:25
johnsomHow many rechecks does a requirements patch take? lol21:27
*** goldyfruit_ has joined #openstack-lbaas21:32
*** goldyfruit___ has quit IRC21:34
rm_work10000021:34
rm_workwe're well on the way21:34
johnsomrm_work Can you take a quick look at our Train release highlights patch: https://review.opendev.org/68161621:36
rm_workyep21:37
rm_workthanks for taking care of that BTW :)21:37
rm_workshould we add something about significantly improved/stabilized UDP protocol support?21:37
rm_workcause we had a TON of fixes to UDP stuff this cucle21:38
rm_work*cycle21:38
johnsomNot sure if it is press/marketing speak enough, but hey....21:38
johnsomI am not sure fixing previously released stuff is exactly a news bite....  Good for release notes, maybe not for the highlights.21:40
openstackgerritCarlos Goncalves proposed openstack/octavia master: Add VIP access control list  https://review.opendev.org/65962621:40
rm_worklol21:41
rm_worki mean, you put image size optimization21:41
rm_workbut yeah lol, fair21:41
johnsomI almost dropped that one21:41
rm_workseems fine21:42
rm_worki'm trying to think of anything else21:42
rm_workguess i should read our releasenotes21:42
johnsomCool, thanks. One more thing off the to-do list21:42
cgoncalvesadditions to the driver agent and octavia-lib for providers?21:44
johnsomThat really only matters to driver developers, so....21:44
johnsomWe also need to limit it to 3-421:45
cgoncalvesok. so we are at the limit with 4, heh21:45
cgoncalvesAFK. back tomorrow o/21:46
rm_workyeah that looks good I think21:46
johnsomI do have a note that I should fix the release notes for octavia-lib.21:47
rm_work+121:47
*** henriqueof1 has joined #openstack-lbaas21:51
*** henriqueof has quit IRC21:52
*** boden has quit IRC22:17
*** threestrands has joined #openstack-lbaas22:37
*** luksky has quit IRC22:38
openstackgerritMichael Johnson proposed openstack/octavia master: Fix 'additive_only' parameter api-ref  https://review.opendev.org/68162522:54
*** numans has quit IRC22:55
*** dtruong has quit IRC22:55
colin-gthiemonge might be able to paraphrase the UDP stuff well enough for a release note? not to put them on the spot :)22:55
*** dtruong has joined #openstack-lbaas22:55
johnsomI think he has release notes on the patches22:55
colin-ah ok, that works for me22:56
*** goldyfruit_ has quit IRC22:57
johnsomrm_work Should we just merge https://review.opendev.org/681144, the reqs patch merged finally22:58
rm_workyes I think so22:58
johnsomrm_work Why don't you +W it22:58
johnsomAlso, I posted my smallest patch of train ^^^ the api-ref was wrong for your additive patch22:59
rm_worklol was it22:59
johnsomI noticed it because Carlos's patch had the same problem.23:00
colin-"Fixed an issue when Octavia cannot reach the database" anyone have a link to this patch i missed it23:00
colin-(from stein) sorry to switch topics23:01
johnsomYou can just put that into gerrit23:01
colin-ok23:01
colin-lol "too many terms in query" maybe not _exactly_ that23:01
johnsomYeah, I see that....23:02
rm_workahh interesting23:02
rm_workneed a version now23:02
johnsomWell, this way we document the version the parameter was added to the API23:02
*** tkajinam has joined #openstack-lbaas23:04
johnsomcolin- Oh, that is a release note, not a commit message.23:04
colin-fortunately carlos used "resilient" in his phrasing so i found it:)23:04
colin-https://review.opendev.org/#/c/600876/23:04
johnsomYeah, that is it23:05

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