Wednesday, 2019-12-04

*** armax has joined #openstack-lbaas00:36
*** sapd1 has joined #openstack-lbaas00:42
johnsomsorrison Try this in a depends-on: https://review.opendev.org/#/c/697211/01:02
openstackgerritAnn Taraday proposed openstack/octavia master: Convert health_monitor flows to use provider models  https://review.opendev.org/67977901:08
openstackgerritAnn Taraday proposed openstack/octavia master: Transition l7policy flows to dicts  https://review.opendev.org/66597701:08
openstackgerritAnn Taraday proposed openstack/octavia master: Transition l7rule flows to dicts  https://review.opendev.org/66817301:08
openstackgerritAnn Taraday proposed openstack/octavia master: Jobboard based controller  https://review.opendev.org/64740601:08
openstackgerritSam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534901:08
openstackgerritSam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534901:10
sorrison*take 2 (copied the wrong change first time!)01:10
johnsomsorrison That may need to be "Depends-On:" to work, not sure01:12
johnsomI always use Depends-On:01:12
sorrisonah whoops, typo there too01:13
sorrisonwill it still work do you know?01:13
sorrisondon't think so having a read01:14
openstackgerritSam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534901:14
*** ramishra has joined #openstack-lbaas01:21
openstackgerritAnn Taraday proposed openstack/octavia master: Testing  https://review.opendev.org/69721301:22
*** sapd1 has quit IRC01:38
*** sorrison has quit IRC01:42
*** sorrison has joined #openstack-lbaas01:44
openstackgerritMichael Johnson proposed openstack/octavia master: Fix multi-listener LB with missing certificate  https://review.opendev.org/69220801:46
openstackgerritMichael Johnson proposed openstack/octavia master: Fix multi-listener LB client auth/re-encryption  https://review.opendev.org/69358601:47
johnsomcgoncalves rm_work Would you mind checking out my comments on https://review.opendev.org/692208 ? Thanks01:47
*** KeithMnemonic has quit IRC01:52
sorrisonjohnsom: no luck with that dib patch although I'm unsure if the octavia job uses the DIB from the depends on or not02:11
openstackgerritBrian Haley proposed openstack/octavia-tempest-plugin master: Create python3 Centos-7 scenario job  https://review.opendev.org/69719802:12
johnsomsorrison Darn, ok, let me look at the job. It *should* pick it up, but maybe that one is not configured correctly02:13
sorrisonit's still running in zuul but the job in question is finished02:13
johnsomYeah, ok, it's pulling from git directly02:16
johnsomOur zuul job definition doesn't tell it to pick up the version zuul checked out.02:17
johnsomhmmm, digging deeper, it does imply it is using the right one.  Looking some more.02:27
johnsomYeah, hmmm, it does look like it picked up the patch version. I wonder if the environment is trashed by then or I made some kind of mistake.02:29
johnsomI will have to look at this tomorrow. I'm not sure why it didn't work.02:31
sorrisonok thanks for your help02:32
*** armax has quit IRC03:07
*** ataraday_ has quit IRC03:08
openstackgerrityuanrunsen proposed openstack/octavia master: Fix create loadbalancer error.  https://review.opendev.org/69722403:25
*** ataraday_ has joined #openstack-lbaas04:02
openstackgerritMerged openstack/octavia master: Convert listener flows to use provider models  https://review.opendev.org/66023604:11
openstackgerritMerged openstack/octavia master: Convert pool flows to use dicts  https://review.opendev.org/66538104:22
openstackgerritMerged openstack/octavia master: Transition member flows to use dicts  https://review.opendev.org/65784204:22
openstackgerritMerged openstack/octavia-lib master: Add availability_zone to the LoadBalancer model  https://review.opendev.org/69470704:22
rm_workwoot04:26
sorrisonrm_work: nice! So I should be able to recheck on my patch now right?04:28
rm_workuhh, might need a rebase04:28
rm_workthe jobboard changes mess with some code in the same areas04:28
openstackgerritSam Morrison proposed openstack/octavia master: Support creating an LB in a specified availability-zone  https://review.opendev.org/69376204:38
sorrisonrebased cleanly so let's see how it goes04:38
*** dasp has quit IRC04:41
*** dasp has joined #openstack-lbaas04:41
sorrisonrm_work: I think we need to make a new version of octavia-lib and then update the global upper-constraints.txt so my patch can work? Currently it's locked at 1.4.005:02
rm_workyes05:03
sorrisonI'm assuming that isn't something I can do?05:07
johnsomFound my mistake in the DIB patch, giving it another try05:39
*** ataraday_ has quit IRC06:00
*** pcaruana has joined #openstack-lbaas06:01
*** maciejjozefczyk has joined #openstack-lbaas06:33
*** sorrison has quit IRC06:39
*** sapd1 has joined #openstack-lbaas06:48
*** sorrison has joined #openstack-lbaas06:52
*** sorrison has quit IRC07:06
*** rcernin has quit IRC07:09
*** sorrison has joined #openstack-lbaas07:09
*** sorrison has quit IRC07:14
*** sorrison has joined #openstack-lbaas07:27
*** sorrison has quit IRC07:56
*** sorrison has joined #openstack-lbaas07:59
*** tesseract has joined #openstack-lbaas08:00
*** gcheresh has joined #openstack-lbaas08:12
*** strobert1 has quit IRC08:13
*** tkajinam has quit IRC08:17
*** JayF has quit IRC08:23
*** tesseract has quit IRC08:24
*** pcaruana has quit IRC08:24
*** abaindur has quit IRC08:24
*** jamesdenton has quit IRC08:24
*** numans has quit IRC08:24
*** irclogbot_0 has quit IRC08:24
*** sapd1_ has quit IRC08:24
*** dosaboy has quit IRC08:24
*** eandersson has quit IRC08:24
*** larsks has quit IRC08:24
*** jmccrory has quit IRC08:24
*** strigazi has quit IRC08:24
*** sorrison has quit IRC08:24
*** maciejjozefczyk has quit IRC08:24
*** dasp has quit IRC08:24
*** vishalmanchanda has quit IRC08:24
*** dmellado has quit IRC08:24
*** mugsie has quit IRC08:24
*** fyx has quit IRC08:24
*** andrein has quit IRC08:24
*** beisner has quit IRC08:24
*** rpittau|afk has quit IRC08:24
*** gregwork has quit IRC08:24
*** frickler has quit IRC08:24
*** trident has quit IRC08:24
*** openstackgerrit has quit IRC08:24
*** brtknr has quit IRC08:24
*** logan- has quit IRC08:24
*** mloza has quit IRC08:24
*** falencastro has quit IRC08:24
*** haleyb has quit IRC08:24
*** amotoki has quit IRC08:24
*** coreycb has quit IRC08:24
*** gcheresh has quit IRC08:24
*** devfaz has quit IRC08:24
*** lxkong has quit IRC08:24
*** xgerman has quit IRC08:24
*** ltomasbo has quit IRC08:24
*** gmann has quit IRC08:24
*** sapd1 has quit IRC08:24
*** ramishra has quit IRC08:24
*** gthiemonge has quit IRC08:24
*** bcafarel has quit IRC08:24
*** pck has quit IRC08:24
*** ataraday has quit IRC08:24
*** cgoncalves has quit IRC08:24
*** tobberydberg has quit IRC08:24
*** baffle has quit IRC08:24
*** dtruong has quit IRC08:24
*** johnsom has quit IRC08:24
*** andy_ has quit IRC08:24
*** ccamposr__ has quit IRC08:24
*** vesper11 has quit IRC08:24
*** f0o has quit IRC08:24
*** kklimonda has quit IRC08:24
*** jrosser has quit IRC08:24
*** dulek has quit IRC08:24
*** goldyfruit___ has quit IRC08:24
*** guilhermesp has quit IRC08:24
*** dougwig has quit IRC08:24
*** squarebracket has quit IRC08:24
*** generalfuzz has quit IRC08:24
*** mnaser has quit IRC08:24
*** NobodyCam has quit IRC08:24
*** cjloader has quit IRC08:24
*** rm_work has quit IRC08:24
*** gcheresh has joined #openstack-lbaas08:24
*** tesseract has joined #openstack-lbaas08:24
*** sorrison has joined #openstack-lbaas08:24
*** sapd1 has joined #openstack-lbaas08:24
*** maciejjozefczyk has joined #openstack-lbaas08:24
*** pcaruana has joined #openstack-lbaas08:24
*** dasp has joined #openstack-lbaas08:24
*** ramishra has joined #openstack-lbaas08:24
*** abaindur has joined #openstack-lbaas08:24
*** ccamposr__ has joined #openstack-lbaas08:24
*** vesper11 has joined #openstack-lbaas08:24
*** jamesdenton has joined #openstack-lbaas08:24
*** vishalmanchanda has joined #openstack-lbaas08:24
*** trident has joined #openstack-lbaas08:24
*** dmellado has joined #openstack-lbaas08:24
*** gthiemonge has joined #openstack-lbaas08:24
*** f0o has joined #openstack-lbaas08:24
*** mugsie has joined #openstack-lbaas08:24
*** kklimonda has joined #openstack-lbaas08:24
*** jrosser has joined #openstack-lbaas08:24
*** fyx has joined #openstack-lbaas08:24
*** andrein has joined #openstack-lbaas08:24
*** numans has joined #openstack-lbaas08:24
*** dulek has joined #openstack-lbaas08:24
*** frickler has joined #openstack-lbaas08:24
*** gregwork has joined #openstack-lbaas08:24
*** rpittau has joined #openstack-lbaas08:24
*** beisner has joined #openstack-lbaas08:24
*** logan- has joined #openstack-lbaas08:24
*** JayF has joined #openstack-lbaas08:24
*** goldyfruit___ has joined #openstack-lbaas08:24
*** baffle has joined #openstack-lbaas08:24
*** bcafarel has joined #openstack-lbaas08:24
*** pck has joined #openstack-lbaas08:24
*** mloza has joined #openstack-lbaas08:24
*** openstackgerrit has joined #openstack-lbaas08:24
*** falencastro has joined #openstack-lbaas08:24
*** devfaz has joined #openstack-lbaas08:24
*** irclogbot_0 has joined #openstack-lbaas08:24
*** sapd1_ has joined #openstack-lbaas08:24
*** haleyb has joined #openstack-lbaas08:24
*** ltomasbo has joined #openstack-lbaas08:24
*** lxkong has joined #openstack-lbaas08:24
*** brtknr has joined #openstack-lbaas08:24
*** guilhermesp has joined #openstack-lbaas08:24
*** dougwig has joined #openstack-lbaas08:24
*** xgerman has joined #openstack-lbaas08:24
*** coreycb has joined #openstack-lbaas08:24
*** amotoki has joined #openstack-lbaas08:24
*** dosaboy has joined #openstack-lbaas08:24
*** eandersson has joined #openstack-lbaas08:24
*** ataraday has joined #openstack-lbaas08:24
*** generalfuzz has joined #openstack-lbaas08:24
*** jmccrory has joined #openstack-lbaas08:24
*** cgoncalves has joined #openstack-lbaas08:24
*** dtruong has joined #openstack-lbaas08:24
*** larsks has joined #openstack-lbaas08:24
*** tobberydberg has joined #openstack-lbaas08:24
*** strigazi has joined #openstack-lbaas08:24
*** andy_ has joined #openstack-lbaas08:24
*** gmann has joined #openstack-lbaas08:24
*** NobodyCam has joined #openstack-lbaas08:24
*** mnaser has joined #openstack-lbaas08:24
*** rm_work has joined #openstack-lbaas08:24
*** cjloader has joined #openstack-lbaas08:24
*** squarebracket has joined #openstack-lbaas08:24
*** johnsom has joined #openstack-lbaas08:24
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Discover LB algorithm used for test  https://review.opendev.org/67226408:35
*** ivve has joined #openstack-lbaas08:36
*** rcernin has joined #openstack-lbaas08:51
*** sapd1 has quit IRC09:28
*** abaindur has quit IRC09:48
*** rcernin has quit IRC09:53
*** gcheresh has quit IRC10:10
*** gcheresh has joined #openstack-lbaas10:10
*** rpittau is now known as rpittau|bbl11:27
openstackgerritFederico Ressi proposed openstack/python-octaviaclient master: Add DevStack CI functional jobs  https://review.opendev.org/69696712:08
*** servagem has joined #openstack-lbaas12:08
dulekIs it possible to check why dib fails on the gate? Talking about this run, we're switching stable/train to do Amp builds: https://779fc7d34db68cc285ea-24c1b4d0bad6f112f42872ab6905686e.ssl.cf1.rackcdn.com/697266/1/check/kuryr-kubernetes-tempest/2a8a098/job-output.txt12:20
cgoncalvesdulek, https://github.com/openstack/octavia-tempest-plugin/blob/master/zuul.d/jobs.yaml#L16612:29
cgoncalvesyou may also find L167-168 useful for troubleshooting12:29
*** goldyfruit___ has quit IRC12:32
dulekcgoncalves: Oh wow, we need this. Thanks!12:34
*** gcheresh has quit IRC12:40
openstackgerritMaciej Józefczyk proposed openstack/octavia-tempest-plugin master: Discover LB algorithm used for test  https://review.opendev.org/67226412:40
*** gcheresh has joined #openstack-lbaas12:44
*** gcheresh has quit IRC12:56
*** gcheresh has joined #openstack-lbaas12:57
*** vishalmanchanda has quit IRC13:04
*** ccamposr has joined #openstack-lbaas13:06
*** ccamposr__ has quit IRC13:09
*** devfaz has quit IRC13:31
*** devfaz has joined #openstack-lbaas13:32
*** rpittau|bbl is now known as rpittau13:36
openstackgerritFederico Ressi proposed openstack/python-octaviaclient master: Add DevStack CI functional jobs  https://review.opendev.org/69696713:59
*** JayF has quit IRC14:04
*** isakgicu has joined #openstack-lbaas14:08
*** JayF has joined #openstack-lbaas14:09
openstackgerritCarlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images  https://review.opendev.org/69712814:13
*** JayF has quit IRC14:13
openstackgerritCarlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images  https://review.opendev.org/69712814:14
*** JayF has joined #openstack-lbaas14:19
*** vishalmanchanda has joined #openstack-lbaas14:27
*** TrevorV has joined #openstack-lbaas14:33
*** goldyfruit has joined #openstack-lbaas14:35
openstackgerritStefan Nica proposed openstack/octavia master: Accept oslopolicy-policy-generator path arguments  https://review.opendev.org/69729614:44
*** gcheresh has quit IRC14:54
*** dasp has quit IRC15:02
*** gcheresh has joined #openstack-lbaas15:02
*** dasp has joined #openstack-lbaas15:03
*** ivve has quit IRC15:33
*** goldyfruit_ has joined #openstack-lbaas15:38
*** goldyfruit has quit IRC15:41
*** salmankhan has joined #openstack-lbaas15:55
*** maciejjozefczyk is now known as mjozefcz|brb15:57
*** mjozefcz|brb has quit IRC16:00
johnsom#startmeeting Octavia16:01
openstackMeeting started Wed Dec  4 16:01:03 2019 UTC and is due to finish in 60 minutes.  The chair is johnsom. 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: Octavia)"16:01
openstackThe meeting name has been set to 'octavia'16:01
johnsomrm_work Ping16:01
cgoncalveshi16:01
gthiemongeHi16:01
haleybhi16:02
johnsom#link https://wiki.openstack.org/wiki/Octavia/Weekly_Meeting_Agenda#Meeting_2019-12-0416:02
*** ataraday_ has joined #openstack-lbaas16:02
johnsomI did put together a quick agenda today16:02
ataraday_hi16:02
johnsom#topic Announcements16:03
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"16:03
johnsomI guess by default I will run today's meeting16:03
johnsomThe one announcement I am aware of is about some students that may be helping us out.16:03
johnsomThe OpenStack foundation has had students from NDSU work on OpenStack projects before.16:04
cgoncalvessweet!16:04
johnsomrm_work had a conversation about it with Kendall at the Summit/PTG and we may be getting some students to help us out.16:04
johnsomAs the TLS protocols and ciphers stories are pretty complete it was proposed that they could work on those tasks.16:05
johnsomIt sounds like it's about four students in their senior year.16:05
johnsomAny questions / concerns about this?16:06
cgoncalveshow can we help them onboarding?16:06
johnsomOtherwise I hope we welcome them and give them a good OpenStack experience.16:06
johnsomI think Kendall will do the initial onboarding (setup gerrit and OSF accounts, etc. I will volunteer to give the Octavia intros.16:07
cgoncalvesthank you16:07
*** armax has joined #openstack-lbaas16:08
johnsomThat was all I had for announcements today. Pretty sure I'm forgetting something. Does anyone else have anything?16:08
johnsomThere are some proposed infra team changes:16:09
johnsom#link http://lists.openstack.org/pipermail/openstack-infra/2019-December/006537.html16:09
johnsomBut that is mostly an FYI and should not impact day-to-day16:09
johnsom#topic Brief progress reports / bugs needing review16:10
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"16:10
johnsomI was out for a few days due to the US holiday. Otherwise it has been working on reviews, bug fixes, and making progress on the failover work.16:11
*** gcheresh has quit IRC16:11
ataraday_Big thatnks to rm_work and johnsom for review of refactor jobboard changes!16:11
johnsomWe are starting to see a number of issues with the transition between python2 and python3 in the master gates. We have been trying to work through those as we see them.16:11
johnsomI fixed a DIB issue last night for example.16:12
ataraday_But if we spoke of ciphers I will highlight the one that we have on review already16:13
ataraday_#link https://review.opendev.org/#/c/685337/16:13
ataraday_could be starting point for students as well16:13
johnsomYep, it doesn't look like that is on the priority review list. I will add it.16:13
johnsomYes, having a default is a head start on that one for sure.16:14
cgoncalvesyeah, now with py2 jobs removed we started seeing issues also in centos 7 amphorae. kuryr team reported the amp no longer works.16:15
*** maciejjozefczyk has joined #openstack-lbaas16:16
cgoncalvesI posted a patch to build centos 8 amps. it seems to be working well on a couple of rechecks and maybe even faster than centos716:16
johnsomIs someone looking into that? There should be no problem continuing to build py2 centos 7 images for stable16:16
johnsomFaster would be super good16:17
cgoncalvesthe py2 libs were removed so...16:17
cgoncalveshaleyb started working on centos7-py3 support16:17
cgoncalveswhile we don't have centos8 controllers16:18
johnsomOk16:18
haleybcgoncalves: the building of the py2 libs?  that was a setup.cfg change we can maybe put back16:18
cgoncalvesit would be nice if we could have a centos7 controller + centos8 amp job for now16:18
haleybthe centos7 py3 job is not happy, https://review.opendev.org/#/c/697198/ - seems python3 isn't installed by default16:18
cgoncalveslast 3 centos8 amp scenario jobs completed in 1h49, 1h40, 2h0716:19
haleybso i either start digging into devstack or we get centos-8 working16:19
cgoncalvescentos 8 controller isn't available yet. I know of some folks working on that, there are WIP patches in Gerrit16:20
haleybcgoncalves: getting back to the py2 libs being removed - that was octavia or octavia-lib ?16:20
cgoncalves#link https://review.opendev.org/#/c/688614/16:20
cgoncalveshaleyb, octavia at least.16:20
johnsomI don't think py2 removal has merged in octavia-lib yet16:21
cgoncalveskuryr folks reported the amphora-agent started crashing on centos 7. I'd have to scroll up to find the exact reason16:21
haleybwe could have been too aggressive - if there was a py2 consumer we shouldn't have broken them.  i learned the upstream goal was to remove the testing this cycle, more than that could have caused this16:21
johnsomThis is the master branch, so I think some transition issues are expected.16:22
cgoncalves+1, as long as it happens early in the cycle which is the case here ;)16:22
johnsom+1 to that!16:23
cgoncalves#link https://review.opendev.org/#/c/697128/16:23
cgoncalves^ centos 8 amphora patch16:23
cgoncalvesFYI, I kept compatibility with centos 7 amphorae16:23
haleybso is there anyone/anything consuming octavia as a library?  if so starting python2 removal (like six library) should wait until next cycle16:24
johnsomFrankly, some py2/py3 combinations running DIB will break until this merges:16:24
johnsom#link https://review.opendev.org/69721116:24
johnsomhaleyb No16:24
johnsomOnly octavia-lib, Octavia should never be imported16:25
haleybburn the boats i say then :)16:25
cgoncalveshttp://codesearch.openstack.org/?q=octavia%3E%3D&i=nope&files=&repos=16:25
haleyb"should never" != "is not"16:26
johnsomI think they were/are fixing that16:26
johnsomhaleyb Sadly true16:26
johnsomI don't think that is a blocker for us though, we should still move forward16:27
cgoncalves+116:27
haleyback, i'll try and split it into many patches to make it reviewable (and revertable)16:28
johnsomAny other updates today?16:28
johnsomSo this next topic I wish our PTL was present....16:30
johnsom#topic OVN driver under Octavia project governance16:30
*** openstack changes topic to "OVN driver under Octavia project governance (Meeting topic: Octavia)"16:30
johnsomThere were a few patches proposed this week to move the OVN provider under the Octavia project.16:30
johnsomI don't think that was proposed or discussed with the Octavia team prior to those patches.16:31
cgoncalvesI don't remember that being proposed or discussed either16:32
johnsomThere were a number of -1 votes on those. I think there are a number of concerns about that and it should be discussed, but without the PTL here I'm not sure we should jump into that.16:32
haleybI can apologize for that, think it was just a decision based on the progress of the networking-ovn code movement16:33
johnsomhaleyb Was there any push back from the neutron team in retaining ownership of that code?16:34
haleybit was basically an orphan, and we can hold any merge until we're able to discuss more, think it's on the neutron-drivers menu this week as well16:34
johnsomIf there is, maybe we should  add this to next week's agenda and make sure that rm_work can join.16:34
haleybi know there are no in-tree drivers, seemed odd to consider this a third-party driver since it is associated with neutron and not a third-party company16:36
haleybin the end i don't think the neutron team will not accept it16:37
johnsomI propose we table this topic until rm_work (PTL) can be present. Is that ok or is this time sensitive?16:37
haleybwe can wait until rm_work is around, i think having an answer in a week is good enough16:38
johnsomOk, thanks. I'm just concerned that we don't have quorum at the moment.16:38
johnsom#topic Open Discussion16:39
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:39
johnsomOther topics this week?16:39
johnsomI think the two patch chains folks are focused on reviewing this week are the AZ patches and the jobboard chain. Thanks for all of the help reviewing those! Great progress on both fronts.16:41
johnsomIt will be nice to have jobboard in early in Ussuri as it was one of our priorities for Ussuri from the PTG.16:42
cgoncalvesataraday_, thanks a lot for all the great work and sticking around despite low review attention in past months16:43
johnsom+10016:43
ataraday_cgoncalves, johnsom Thanks! Though it is not finished yet :)16:44
johnsomIf there are not any more topics this week I will close out the meeting.16:44
johnsom#endmeeting16:44
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews"16:44
openstackMeeting ended Wed Dec  4 16:44:41 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:44
openstackMinutes:        http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.html16:44
johnsomThanks everyone!16:44
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.txt16:44
openstackLog:            http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.log.html16:44
openstackgerritGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add an option to pass an existing subnet as VIP IPv6 subnet  https://review.opendev.org/69731816:46
cgoncalvesthanks for running the meeting, michjohn!16:47
johnsomSure, NP16:48
johnsomsorrison I don't know if you are online yet, but my fix last night to the DIB patch fixed your tempest patch svc-map issue. I had a typo in my DIB patch where the path was pointing to the file not the directory.17:05
*** isakgicu has quit IRC17:07
cgoncalvesbtw, octavia gate is broken. we need an octavia-lib release17:11
johnsomUgh17:11
cgoncalveshttp://zuul.openstack.org/builds?job_name=octavia-tox-functional-py37-tips17:11
johnsomHmm, this first one, py36 test, looks like it might be a py2/py3 issue17:15
johnsomThe oslo policy enforcer check is failing17:15
johnsomWill need to dig more to confirm17:16
johnsomOh, that might just be that patch. lol17:18
johnsomYeah, ok, that was just a bad patch17:18
johnsomI can propose a octavia-lib release. I thought rm_work was going to do that last night.17:19
johnsomWe should also talk about how to untangle this...  non-voting might be the right answer17:19
johnsomFYI, octavia-lib release patch: https://review.opendev.org/69733017:24
cgoncalvesthank you17:28
openstackgerritCarlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images  https://review.opendev.org/69712817:49
cgoncalves^ ready to be reviewed17:49
*** rpittau is now known as rpittau|afk17:50
*** tesseract has quit IRC18:04
johnsomNice, looking18:06
johnsomcgoncalves Is there a tempest patch that uses that image?18:09
johnsomI see previous runs had a job, but this one only has the publish.18:12
*** maciejjozefczyk has quit IRC18:16
cgoncalvesjohnsom, previous patch sets included a scenario centos8 job to test the image18:18
cgoncalvesthe job needs to be created in octavia-tempest-plugin18:18
cgoncalvesI was delaying creating the job until we had a centos7 controller job functional but ETA for that is unknown18:20
cgoncalvesso... ubuntu controller + centos8 amp?18:20
johnsomShould work...18:20
cgoncalvesit does work. the job in previous patch sets ran against that configuration18:21
cgoncalvesoctavia-v2-dsvm-scenario-centos-8 or should we make it explicit it's a mix-in?18:22
johnsomWell, we could just go with that and "fix" it to be pure centos-8 when we are ready18:23
cgoncalves+118:23
*** vishalmanchanda has quit IRC18:24
openstackgerritCarlos Goncalves proposed openstack/octavia-tempest-plugin master: Add octavia-v2-dsvm-scenario-centos-8  https://review.opendev.org/69733618:26
*** ccamposr__ has joined #openstack-lbaas18:40
*** ccamposr has quit IRC18:42
rm_workYep, sounds good (for the cent8 thing)18:43
rm_workSorry forgot to check my calendar when I passed out last night T_T18:43
*** salmankhan has quit IRC19:06
openstackgerritMichael Johnson proposed openstack/octavia master: Fix multi-listener LB with missing certificate  https://review.opendev.org/69220819:18
johnsomA rebasing we go....19:18
rm_workah i forgot we don't +2 the release patches T_T19:21
johnsomYeah, out of our hands. Though the PTL +1 helps.... hint, hint19:22
rm_workyeah i did it :D19:22
johnsomYeah, you got it already19:22
rm_workbut carlos is the releasemanager he and did +1 :)19:22
openstackgerritMichael Johnson proposed openstack/octavia master: Fix multi-listener LB client auth/re-encryption  https://review.opendev.org/69358619:26
johnsomOk, rebases done, waiting on some gate jobs to vote, so time for lunch.19:28
rm_workah thought the lib release was blocking our gates?19:36
rm_workcgoncalves: if you want to go ahead and pre+2 https://review.opendev.org/#/c/694895/ that'll make sure it merges once the dependent patch does19:40
rm_workwe probably want that gate disabled for as little time as possible19:40
rm_workjohnsom: so after the lib change merges we need to update g-r? or no19:41
rm_workwe can update the lower-constraint in the patch we need the new version, but will the requirements-check jobs complain?19:41
rm_workIIRC they do? but maybe that changed19:41
rm_workalso: I'm +2 all the way up through l7rule on the jobboard stuff19:44
rm_workso, 5 more that could +W19:44
*** gcheresh has joined #openstack-lbaas19:54
johnsomThe release should post the G-R patch (upper really, but details)20:03
rm_workah20:05
rm_workdidn't realize that was automated20:05
johnsomrm_work https://review.opendev.org/69735920:10
rm_workjohnsom: see dib channel20:11
rm_workalso: bunch of jobboard patches waiting for a second +2 still :D20:14
cgoncalvesjohnsom, does https://review.opendev.org/#/c/694826/ really depend on what is set?20:17
*** abaindur has joined #openstack-lbaas20:17
*** abaindur has quit IRC20:17
cgoncalvesshould have been rebased on top of said dependency patch, or was there a reason to use Depends-On?20:18
johnsomcgoncalves no, I did that just because the queens gate is random failing and I thought the depends-on was the answer20:18
*** abaindur has joined #openstack-lbaas20:18
*** abaindur has quit IRC20:18
johnsomAh, yeah, mistake there.20:18
openstackgerritMichael Johnson proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs  https://review.opendev.org/69482620:19
*** abaindur has joined #openstack-lbaas20:19
johnsomSadly you all need to re-vote that20:19
openstackgerritAdam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs  https://review.opendev.org/69482620:19
cgoncalvesok, just wanted to check I was not missing some important detail20:19
johnsomsince I removed the depends-on20:19
rm_worklol20:19
rm_worki clicked rebase at the same time you changed the commit20:19
rm_worklet me undo that20:19
johnsomGreat, it reverted my change too20:19
rm_workyeah lol20:20
cgoncalveswhooops20:20
rm_workexcellent threading20:20
johnsomGuess that falls into the "don't do that" category20:20
cgoncalvesjohnsom, https://review.opendev.org/#/c/691757/20:21
rm_workok so ... i'm fixing it? or are you? :D20:21
cgoncalvesno depends-on but on top of the flush patch20:21
rm_worki am ready to click the button to fix it, heh20:21
johnsomNo, skip the flush patch, it doesn't work20:22
openstackgerritAdam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs  https://review.opendev.org/69482620:22
openstackgerritAdam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs  https://review.opendev.org/69482620:22
rm_workok there, back on master and no depends-on20:23
johnsom+120:23
cgoncalveshttps://review.opendev.org/#/q/project:openstack/octavia+status:open+NOT+branch:master20:23
cgoncalvesonce those two merge, we could make some stable releases20:24
johnsomI have no idea what is going on with that queens image and those RTLINK issues. I tried flushing the addresses and flushing the routes, but it still claims a conflict20:24
rm_workshould that other one also not depends-on?20:24
rm_workor rather20:24
rm_workrebase to HEAD?20:24
cgoncalvesrm_work, please20:24
openstackgerritAdam Harwell proposed openstack/octavia stable/queens: Fix issues with unavailable secrets  https://review.opendev.org/69175720:24
rm_workk20:25
cgoncalvesdanke20:25
rm_workstill waiting on +Ws on the the remaining intermediate jobboard patches :D20:27
rm_workor will those break without the lib update?20:27
rm_workahh yeah i think i see20:28
rm_workmaybe we need to pull out the update to those functions that makes things pass20:29
rm_workso that can merge quickly, then we can continue to work on the actual driver for the AZ stuff20:29
rm_workwhich ... i can do once this g-r change merges20:29
rm_workah i can just do it now20:30
beisnero/ hi, all.  does anyone have time to advise on story #2006901?  (https://bugs.launchpad.net/ubuntu/+source/octavia/+bug/1852599)20:35
openstackLaunchpad bug 1852599 in octavia (Ubuntu) "Cannot delete TLS Terminated listener if container has been deleted in barbican" [Undecided,New]20:35
johnsombeisner FYI: Octavia doesn't use launchpad for bugs any longer. It was migrated to storyboard. That said, this had been fixed in a series of patches already. Let me find the storyboard story20:37
beisnerhttps://storyboard.openstack.org/#!/story/200690120:37
johnsomThat is not it20:37
johnsomlooks like a duplicate20:37
beisnerhi johnsom - ah, a dup of a different story?20:38
johnsomyes20:38
* beisner browses20:39
johnsomThis one: https://storyboard.openstack.org/#!/story/200667620:39
johnsomI think the delete case has merged all the way back to queens already20:40
beisnerack, thanks johnsom20:41
johnsombeisner Just looked at: https://review.opendev.org/#/q/7fcef817eca3ad177460ee6976633f3dbd078cf2 looks like queens is still pending merge20:42
johnsomqueens https://review.opendev.org/#/c/69175720:43
beisnerjohnsom:  ok cool.  so, we'll work on stable release updates in Ubuntu and the Ubuntu Cloud Archive.  tyvm20:44
johnsomSure, NP. I think upstream stable releases will happen soon as well if those haven't already been released.20:45
coreycbjohnsom: hi, regarding 691757, do you know of octavia has upcoming stable point releases planned that will include it?20:57
johnsomcoreycb Yeah, just commented on that. I think it is planned this week.20:58
coreycbjohnsom: great! thanks. we'll keep an eye out for them.20:58
openstackgerritAdam Harwell proposed openstack/octavia master: Adjust for new octavia-lib LB signature  https://review.opendev.org/69736521:00
rm_work^^ we can fast-track that I think21:03
rm_workthen take our time on the real AZ oatch21:03
rm_work*patch21:03
johnsom+2's on the two centos 8 patches. Scenario passed21:04
rm_worknice21:04
rm_workare our gates working?21:04
rm_worki thought the lib change would break us21:04
rm_work(tips)21:04
rm_workah yeah, it did21:05
rm_workonce reqs change merges, then we can merge my change there, and then gates will be good21:05
rm_workbut the reqs change failed briefly21:05
*** servagem has quit IRC21:17
*** salmankhan has joined #openstack-lbaas21:25
openstackgerritAdam Harwell proposed openstack/octavia master: Support creating an LB in a specified availability-zone  https://review.opendev.org/69376221:28
rm_worksorrison: rebased on top of the reqs change patch21:28
rm_worklet's see21:28
rm_workhopefully more stuff will pass21:28
rm_workthough i still need to do the spares-pool fixes21:28
sorrisonok nice21:29
sorrisonif you're busy I should have some time today if you want me to sort it21:29
openstackgerritAdam Harwell proposed openstack/octavia-lib master: Return tips jobs to voting  https://review.opendev.org/69489521:30
rm_workif you think you can do it easily21:30
rm_workI just know I have all the context on that :D21:30
rm_workbut i can point you at the right stuff21:30
rm_workjohnsom / cgoncalves https://review.opendev.org/#/c/694895/ updated to point it at a better dep21:31
rm_workneeds revote ^^21:31
sorrisonIs the idea that if spare count is set to say 3 then it would have 3 spares in each AZ?21:31
*** pcaruana has quit IRC21:32
rm_workyeah21:32
rm_workso like21:32
rm_workhttps://review.opendev.org/#/c/558962/32/octavia/controller/worker/v1/tasks/amphora_driver_tasks.py21:32
rm_workthat will be necessary I think21:32
rm_workand some changes in https://review.opendev.org/#/c/558962/32/octavia/controller/worker/v1/tasks/database_tasks.py like that? i think (though not exactly those changes...)21:33
rm_workbut, it has to be able to pick a spare amp correctly -- as the change is now, it will BOOT a new amp correctly, but not pick an existing one I believe21:34
rm_workprobably this: https://review.opendev.org/#/c/558962/32/octavia/db/repositories.py21:35
rm_worktrying to find the bit where it actually manages spares21:35
sorrisonyeah ok, makes sense.21:35
rm_workoh i think it's the FIRST change in: https://review.opendev.org/#/c/558962/32/octavia/controller/worker/v1/flows/amphora_flows.py21:37
rm_work`get_create_amphora_flow` is what we use for spares, I believe21:37
rm_workand you'll need slightly different code for making sure the `_get_create_amp_for_lb_subflow` picks the right zone amp21:38
rm_workah probably it would be that same change, but with new logic in amphora_driver_tasks.PickAmphoraZones21:39
rm_workanyway, welcome to give it a shot :D21:39
rm_workbut if you're like "wtf", i can try to look tomorrow21:39
sorrisonyeah cool, I'll take a look in a few hours. want to get to know that part of the codebase anyway and will defer to you for wtf moments21:43
*** gcheresh has quit IRC21:45
*** salmankhan has quit IRC21:57
rm_workyeah my brain is even having some stuttering on this right now, need to get back into it22:07
rm_workit's been too long22:07
rm_workand now we've heard from yet more people who want that patch working as well22:08
rm_workso i'm going to have to fix it up once we get this done22:08
rm_workbut it'll be simplified a lot if we're including a bunch of the code from it ;)22:08
*** goldyfruit___ has joined #openstack-lbaas22:09
*** strobert1 has joined #openstack-lbaas22:09
*** goldyfruit_ has quit IRC22:12
*** goldyfruit_ has joined #openstack-lbaas22:12
*** goldyfruit___ has quit IRC22:14
*** rcernin has joined #openstack-lbaas22:19
*** TrevorV has quit IRC22:33
*** tkajinam has joined #openstack-lbaas23:05
johnsomhaleyb octavia-ovn-provider? Killing me here.23:09
johnsomThat driver is all OVN native code writing directly to the ovsdb...23:12
openstackgerritMichael Johnson proposed openstack/octavia-tempest-plugin master: Introduce a simple HTTPS server for TLS test.  https://review.opendev.org/62889123:27
johnsomFYI, I just updated the priority review list with our progress.23:36
*** goldyfruit_ has quit IRC23:52

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