*** armax has joined #openstack-lbaas | 00:36 | |
*** sapd1 has joined #openstack-lbaas | 00:42 | |
johnsom | sorrison Try this in a depends-on: https://review.opendev.org/#/c/697211/ | 01:02 |
---|---|---|
openstackgerrit | Ann Taraday proposed openstack/octavia master: Convert health_monitor flows to use provider models https://review.opendev.org/679779 | 01:08 |
openstackgerrit | Ann Taraday proposed openstack/octavia master: Transition l7policy flows to dicts https://review.opendev.org/665977 | 01:08 |
openstackgerrit | Ann Taraday proposed openstack/octavia master: Transition l7rule flows to dicts https://review.opendev.org/668173 | 01:08 |
openstackgerrit | Ann Taraday proposed openstack/octavia master: Jobboard based controller https://review.opendev.org/647406 | 01:08 |
openstackgerrit | Sam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ https://review.opendev.org/695349 | 01:08 |
openstackgerrit | Sam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ https://review.opendev.org/695349 | 01:10 |
sorrison | *take 2 (copied the wrong change first time!) | 01:10 |
johnsom | sorrison That may need to be "Depends-On:" to work, not sure | 01:12 |
johnsom | I always use Depends-On: | 01:12 |
sorrison | ah whoops, typo there too | 01:13 |
sorrison | will it still work do you know? | 01:13 |
sorrison | don't think so having a read | 01:14 |
openstackgerrit | Sam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ https://review.opendev.org/695349 | 01:14 |
*** ramishra has joined #openstack-lbaas | 01:21 | |
openstackgerrit | Ann Taraday proposed openstack/octavia master: Testing https://review.opendev.org/697213 | 01:22 |
*** sapd1 has quit IRC | 01:38 | |
*** sorrison has quit IRC | 01:42 | |
*** sorrison has joined #openstack-lbaas | 01:44 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix multi-listener LB with missing certificate https://review.opendev.org/692208 | 01:46 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix multi-listener LB client auth/re-encryption https://review.opendev.org/693586 | 01:47 |
johnsom | cgoncalves rm_work Would you mind checking out my comments on https://review.opendev.org/692208 ? Thanks | 01:47 |
*** KeithMnemonic has quit IRC | 01:52 | |
sorrison | johnsom: no luck with that dib patch although I'm unsure if the octavia job uses the DIB from the depends on or not | 02:11 |
openstackgerrit | Brian Haley proposed openstack/octavia-tempest-plugin master: Create python3 Centos-7 scenario job https://review.opendev.org/697198 | 02:12 |
johnsom | sorrison Darn, ok, let me look at the job. It *should* pick it up, but maybe that one is not configured correctly | 02:13 |
sorrison | it's still running in zuul but the job in question is finished | 02:13 |
johnsom | Yeah, ok, it's pulling from git directly | 02:16 |
johnsom | Our zuul job definition doesn't tell it to pick up the version zuul checked out. | 02:17 |
johnsom | hmmm, digging deeper, it does imply it is using the right one. Looking some more. | 02:27 |
johnsom | Yeah, 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 |
johnsom | I will have to look at this tomorrow. I'm not sure why it didn't work. | 02:31 |
sorrison | ok thanks for your help | 02:32 |
*** armax has quit IRC | 03:07 | |
*** ataraday_ has quit IRC | 03:08 | |
openstackgerrit | yuanrunsen proposed openstack/octavia master: Fix create loadbalancer error. https://review.opendev.org/697224 | 03:25 |
*** ataraday_ has joined #openstack-lbaas | 04:02 | |
openstackgerrit | Merged openstack/octavia master: Convert listener flows to use provider models https://review.opendev.org/660236 | 04:11 |
openstackgerrit | Merged openstack/octavia master: Convert pool flows to use dicts https://review.opendev.org/665381 | 04:22 |
openstackgerrit | Merged openstack/octavia master: Transition member flows to use dicts https://review.opendev.org/657842 | 04:22 |
openstackgerrit | Merged openstack/octavia-lib master: Add availability_zone to the LoadBalancer model https://review.opendev.org/694707 | 04:22 |
rm_work | woot | 04:26 |
sorrison | rm_work: nice! So I should be able to recheck on my patch now right? | 04:28 |
rm_work | uhh, might need a rebase | 04:28 |
rm_work | the jobboard changes mess with some code in the same areas | 04:28 |
openstackgerrit | Sam Morrison proposed openstack/octavia master: Support creating an LB in a specified availability-zone https://review.opendev.org/693762 | 04:38 |
sorrison | rebased cleanly so let's see how it goes | 04:38 |
*** dasp has quit IRC | 04:41 | |
*** dasp has joined #openstack-lbaas | 04:41 | |
sorrison | rm_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.0 | 05:02 |
rm_work | yes | 05:03 |
sorrison | I'm assuming that isn't something I can do? | 05:07 |
johnsom | Found my mistake in the DIB patch, giving it another try | 05:39 |
*** ataraday_ has quit IRC | 06:00 | |
*** pcaruana has joined #openstack-lbaas | 06:01 | |
*** maciejjozefczyk has joined #openstack-lbaas | 06:33 | |
*** sorrison has quit IRC | 06:39 | |
*** sapd1 has joined #openstack-lbaas | 06:48 | |
*** sorrison has joined #openstack-lbaas | 06:52 | |
*** sorrison has quit IRC | 07:06 | |
*** rcernin has quit IRC | 07:09 | |
*** sorrison has joined #openstack-lbaas | 07:09 | |
*** sorrison has quit IRC | 07:14 | |
*** sorrison has joined #openstack-lbaas | 07:27 | |
*** sorrison has quit IRC | 07:56 | |
*** sorrison has joined #openstack-lbaas | 07:59 | |
*** tesseract has joined #openstack-lbaas | 08:00 | |
*** gcheresh has joined #openstack-lbaas | 08:12 | |
*** strobert1 has quit IRC | 08:13 | |
*** tkajinam has quit IRC | 08:17 | |
*** JayF has quit IRC | 08:23 | |
*** tesseract has quit IRC | 08:24 | |
*** pcaruana has quit IRC | 08:24 | |
*** abaindur has quit IRC | 08:24 | |
*** jamesdenton has quit IRC | 08:24 | |
*** numans has quit IRC | 08:24 | |
*** irclogbot_0 has quit IRC | 08:24 | |
*** sapd1_ has quit IRC | 08:24 | |
*** dosaboy has quit IRC | 08:24 | |
*** eandersson has quit IRC | 08:24 | |
*** larsks has quit IRC | 08:24 | |
*** jmccrory has quit IRC | 08:24 | |
*** strigazi has quit IRC | 08:24 | |
*** sorrison has quit IRC | 08:24 | |
*** maciejjozefczyk has quit IRC | 08:24 | |
*** dasp has quit IRC | 08:24 | |
*** vishalmanchanda has quit IRC | 08:24 | |
*** dmellado has quit IRC | 08:24 | |
*** mugsie has quit IRC | 08:24 | |
*** fyx has quit IRC | 08:24 | |
*** andrein has quit IRC | 08:24 | |
*** beisner has quit IRC | 08:24 | |
*** rpittau|afk has quit IRC | 08:24 | |
*** gregwork has quit IRC | 08:24 | |
*** frickler has quit IRC | 08:24 | |
*** trident has quit IRC | 08:24 | |
*** openstackgerrit has quit IRC | 08:24 | |
*** brtknr has quit IRC | 08:24 | |
*** logan- has quit IRC | 08:24 | |
*** mloza has quit IRC | 08:24 | |
*** falencastro has quit IRC | 08:24 | |
*** haleyb has quit IRC | 08:24 | |
*** amotoki has quit IRC | 08:24 | |
*** coreycb has quit IRC | 08:24 | |
*** gcheresh has quit IRC | 08:24 | |
*** devfaz has quit IRC | 08:24 | |
*** lxkong has quit IRC | 08:24 | |
*** xgerman has quit IRC | 08:24 | |
*** ltomasbo has quit IRC | 08:24 | |
*** gmann has quit IRC | 08:24 | |
*** sapd1 has quit IRC | 08:24 | |
*** ramishra has quit IRC | 08:24 | |
*** gthiemonge has quit IRC | 08:24 | |
*** bcafarel has quit IRC | 08:24 | |
*** pck has quit IRC | 08:24 | |
*** ataraday has quit IRC | 08:24 | |
*** cgoncalves has quit IRC | 08:24 | |
*** tobberydberg has quit IRC | 08:24 | |
*** baffle has quit IRC | 08:24 | |
*** dtruong has quit IRC | 08:24 | |
*** johnsom has quit IRC | 08:24 | |
*** andy_ has quit IRC | 08:24 | |
*** ccamposr__ has quit IRC | 08:24 | |
*** vesper11 has quit IRC | 08:24 | |
*** f0o has quit IRC | 08:24 | |
*** kklimonda has quit IRC | 08:24 | |
*** jrosser has quit IRC | 08:24 | |
*** dulek has quit IRC | 08:24 | |
*** goldyfruit___ has quit IRC | 08:24 | |
*** guilhermesp has quit IRC | 08:24 | |
*** dougwig has quit IRC | 08:24 | |
*** squarebracket has quit IRC | 08:24 | |
*** generalfuzz has quit IRC | 08:24 | |
*** mnaser has quit IRC | 08:24 | |
*** NobodyCam has quit IRC | 08:24 | |
*** cjloader has quit IRC | 08:24 | |
*** rm_work has quit IRC | 08:24 | |
*** gcheresh has joined #openstack-lbaas | 08:24 | |
*** tesseract has joined #openstack-lbaas | 08:24 | |
*** sorrison has joined #openstack-lbaas | 08:24 | |
*** sapd1 has joined #openstack-lbaas | 08:24 | |
*** maciejjozefczyk has joined #openstack-lbaas | 08:24 | |
*** pcaruana has joined #openstack-lbaas | 08:24 | |
*** dasp has joined #openstack-lbaas | 08:24 | |
*** ramishra has joined #openstack-lbaas | 08:24 | |
*** abaindur has joined #openstack-lbaas | 08:24 | |
*** ccamposr__ has joined #openstack-lbaas | 08:24 | |
*** vesper11 has joined #openstack-lbaas | 08:24 | |
*** jamesdenton has joined #openstack-lbaas | 08:24 | |
*** vishalmanchanda has joined #openstack-lbaas | 08:24 | |
*** trident has joined #openstack-lbaas | 08:24 | |
*** dmellado has joined #openstack-lbaas | 08:24 | |
*** gthiemonge has joined #openstack-lbaas | 08:24 | |
*** f0o has joined #openstack-lbaas | 08:24 | |
*** mugsie has joined #openstack-lbaas | 08:24 | |
*** kklimonda has joined #openstack-lbaas | 08:24 | |
*** jrosser has joined #openstack-lbaas | 08:24 | |
*** fyx has joined #openstack-lbaas | 08:24 | |
*** andrein has joined #openstack-lbaas | 08:24 | |
*** numans has joined #openstack-lbaas | 08:24 | |
*** dulek has joined #openstack-lbaas | 08:24 | |
*** frickler has joined #openstack-lbaas | 08:24 | |
*** gregwork has joined #openstack-lbaas | 08:24 | |
*** rpittau has joined #openstack-lbaas | 08:24 | |
*** beisner has joined #openstack-lbaas | 08:24 | |
*** logan- has joined #openstack-lbaas | 08:24 | |
*** JayF has joined #openstack-lbaas | 08:24 | |
*** goldyfruit___ has joined #openstack-lbaas | 08:24 | |
*** baffle has joined #openstack-lbaas | 08:24 | |
*** bcafarel has joined #openstack-lbaas | 08:24 | |
*** pck has joined #openstack-lbaas | 08:24 | |
*** mloza has joined #openstack-lbaas | 08:24 | |
*** openstackgerrit has joined #openstack-lbaas | 08:24 | |
*** falencastro has joined #openstack-lbaas | 08:24 | |
*** devfaz has joined #openstack-lbaas | 08:24 | |
*** irclogbot_0 has joined #openstack-lbaas | 08:24 | |
*** sapd1_ has joined #openstack-lbaas | 08:24 | |
*** haleyb has joined #openstack-lbaas | 08:24 | |
*** ltomasbo has joined #openstack-lbaas | 08:24 | |
*** lxkong has joined #openstack-lbaas | 08:24 | |
*** brtknr has joined #openstack-lbaas | 08:24 | |
*** guilhermesp has joined #openstack-lbaas | 08:24 | |
*** dougwig has joined #openstack-lbaas | 08:24 | |
*** xgerman has joined #openstack-lbaas | 08:24 | |
*** coreycb has joined #openstack-lbaas | 08:24 | |
*** amotoki has joined #openstack-lbaas | 08:24 | |
*** dosaboy has joined #openstack-lbaas | 08:24 | |
*** eandersson has joined #openstack-lbaas | 08:24 | |
*** ataraday has joined #openstack-lbaas | 08:24 | |
*** generalfuzz has joined #openstack-lbaas | 08:24 | |
*** jmccrory has joined #openstack-lbaas | 08:24 | |
*** cgoncalves has joined #openstack-lbaas | 08:24 | |
*** dtruong has joined #openstack-lbaas | 08:24 | |
*** larsks has joined #openstack-lbaas | 08:24 | |
*** tobberydberg has joined #openstack-lbaas | 08:24 | |
*** strigazi has joined #openstack-lbaas | 08:24 | |
*** andy_ has joined #openstack-lbaas | 08:24 | |
*** gmann has joined #openstack-lbaas | 08:24 | |
*** NobodyCam has joined #openstack-lbaas | 08:24 | |
*** mnaser has joined #openstack-lbaas | 08:24 | |
*** rm_work has joined #openstack-lbaas | 08:24 | |
*** cjloader has joined #openstack-lbaas | 08:24 | |
*** squarebracket has joined #openstack-lbaas | 08:24 | |
*** johnsom has joined #openstack-lbaas | 08:24 | |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Discover LB algorithm used for test https://review.opendev.org/672264 | 08:35 |
*** ivve has joined #openstack-lbaas | 08:36 | |
*** rcernin has joined #openstack-lbaas | 08:51 | |
*** sapd1 has quit IRC | 09:28 | |
*** abaindur has quit IRC | 09:48 | |
*** rcernin has quit IRC | 09:53 | |
*** gcheresh has quit IRC | 10:10 | |
*** gcheresh has joined #openstack-lbaas | 10:10 | |
*** rpittau is now known as rpittau|bbl | 11:27 | |
openstackgerrit | Federico Ressi proposed openstack/python-octaviaclient master: Add DevStack CI functional jobs https://review.opendev.org/696967 | 12:08 |
*** servagem has joined #openstack-lbaas | 12:08 | |
dulek | Is 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.txt | 12:20 |
cgoncalves | dulek, https://github.com/openstack/octavia-tempest-plugin/blob/master/zuul.d/jobs.yaml#L166 | 12:29 |
cgoncalves | you may also find L167-168 useful for troubleshooting | 12:29 |
*** goldyfruit___ has quit IRC | 12:32 | |
dulek | cgoncalves: Oh wow, we need this. Thanks! | 12:34 |
*** gcheresh has quit IRC | 12:40 | |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Discover LB algorithm used for test https://review.opendev.org/672264 | 12:40 |
*** gcheresh has joined #openstack-lbaas | 12:44 | |
*** gcheresh has quit IRC | 12:56 | |
*** gcheresh has joined #openstack-lbaas | 12:57 | |
*** vishalmanchanda has quit IRC | 13:04 | |
*** ccamposr has joined #openstack-lbaas | 13:06 | |
*** ccamposr__ has quit IRC | 13:09 | |
*** devfaz has quit IRC | 13:31 | |
*** devfaz has joined #openstack-lbaas | 13:32 | |
*** rpittau|bbl is now known as rpittau | 13:36 | |
openstackgerrit | Federico Ressi proposed openstack/python-octaviaclient master: Add DevStack CI functional jobs https://review.opendev.org/696967 | 13:59 |
*** JayF has quit IRC | 14:04 | |
*** isakgicu has joined #openstack-lbaas | 14:08 | |
*** JayF has joined #openstack-lbaas | 14:09 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images https://review.opendev.org/697128 | 14:13 |
*** JayF has quit IRC | 14:13 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images https://review.opendev.org/697128 | 14:14 |
*** JayF has joined #openstack-lbaas | 14:19 | |
*** vishalmanchanda has joined #openstack-lbaas | 14:27 | |
*** TrevorV has joined #openstack-lbaas | 14:33 | |
*** goldyfruit has joined #openstack-lbaas | 14:35 | |
openstackgerrit | Stefan Nica proposed openstack/octavia master: Accept oslopolicy-policy-generator path arguments https://review.opendev.org/697296 | 14:44 |
*** gcheresh has quit IRC | 14:54 | |
*** dasp has quit IRC | 15:02 | |
*** gcheresh has joined #openstack-lbaas | 15:02 | |
*** dasp has joined #openstack-lbaas | 15:03 | |
*** ivve has quit IRC | 15:33 | |
*** goldyfruit_ has joined #openstack-lbaas | 15:38 | |
*** goldyfruit has quit IRC | 15:41 | |
*** salmankhan has joined #openstack-lbaas | 15:55 | |
*** maciejjozefczyk is now known as mjozefcz|brb | 15:57 | |
*** mjozefcz|brb has quit IRC | 16:00 | |
johnsom | #startmeeting Octavia | 16:01 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
*** openstack changes topic to " (Meeting topic: Octavia)" | 16:01 | |
openstack | The meeting name has been set to 'octavia' | 16:01 |
johnsom | rm_work Ping | 16:01 |
cgoncalves | hi | 16:01 |
gthiemonge | Hi | 16:01 |
haleyb | hi | 16:02 |
johnsom | #link https://wiki.openstack.org/wiki/Octavia/Weekly_Meeting_Agenda#Meeting_2019-12-04 | 16:02 |
*** ataraday_ has joined #openstack-lbaas | 16:02 | |
johnsom | I did put together a quick agenda today | 16:02 |
ataraday_ | hi | 16:02 |
johnsom | #topic Announcements | 16:03 |
*** openstack changes topic to "Announcements (Meeting topic: Octavia)" | 16:03 | |
johnsom | I guess by default I will run today's meeting | 16:03 |
johnsom | The one announcement I am aware of is about some students that may be helping us out. | 16:03 |
johnsom | The OpenStack foundation has had students from NDSU work on OpenStack projects before. | 16:04 |
cgoncalves | sweet! | 16:04 |
johnsom | rm_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 |
johnsom | As the TLS protocols and ciphers stories are pretty complete it was proposed that they could work on those tasks. | 16:05 |
johnsom | It sounds like it's about four students in their senior year. | 16:05 |
johnsom | Any questions / concerns about this? | 16:06 |
cgoncalves | how can we help them onboarding? | 16:06 |
johnsom | Otherwise I hope we welcome them and give them a good OpenStack experience. | 16:06 |
johnsom | I think Kendall will do the initial onboarding (setup gerrit and OSF accounts, etc. I will volunteer to give the Octavia intros. | 16:07 |
cgoncalves | thank you | 16:07 |
*** armax has joined #openstack-lbaas | 16:08 | |
johnsom | That was all I had for announcements today. Pretty sure I'm forgetting something. Does anyone else have anything? | 16:08 |
johnsom | There are some proposed infra team changes: | 16:09 |
johnsom | #link http://lists.openstack.org/pipermail/openstack-infra/2019-December/006537.html | 16:09 |
johnsom | But that is mostly an FYI and should not impact day-to-day | 16:09 |
johnsom | #topic Brief progress reports / bugs needing review | 16:10 |
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)" | 16:10 | |
johnsom | I 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 IRC | 16:11 | |
ataraday_ | Big thatnks to rm_work and johnsom for review of refactor jobboard changes! | 16:11 |
johnsom | We 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 |
johnsom | I 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 already | 16:13 |
ataraday_ | #link https://review.opendev.org/#/c/685337/ | 16:13 |
ataraday_ | could be starting point for students as well | 16:13 |
johnsom | Yep, it doesn't look like that is on the priority review list. I will add it. | 16:13 |
johnsom | Yes, having a default is a head start on that one for sure. | 16:14 |
cgoncalves | yeah, 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-lbaas | 16:16 | |
cgoncalves | I posted a patch to build centos 8 amps. it seems to be working well on a couple of rechecks and maybe even faster than centos7 | 16:16 |
johnsom | Is someone looking into that? There should be no problem continuing to build py2 centos 7 images for stable | 16:16 |
johnsom | Faster would be super good | 16:17 |
cgoncalves | the py2 libs were removed so... | 16:17 |
cgoncalves | haleyb started working on centos7-py3 support | 16:17 |
cgoncalves | while we don't have centos8 controllers | 16:18 |
johnsom | Ok | 16:18 |
haleyb | cgoncalves: the building of the py2 libs? that was a setup.cfg change we can maybe put back | 16:18 |
cgoncalves | it would be nice if we could have a centos7 controller + centos8 amp job for now | 16:18 |
haleyb | the centos7 py3 job is not happy, https://review.opendev.org/#/c/697198/ - seems python3 isn't installed by default | 16:18 |
cgoncalves | last 3 centos8 amp scenario jobs completed in 1h49, 1h40, 2h07 | 16:19 |
haleyb | so i either start digging into devstack or we get centos-8 working | 16:19 |
cgoncalves | centos 8 controller isn't available yet. I know of some folks working on that, there are WIP patches in Gerrit | 16:20 |
haleyb | cgoncalves: 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 |
cgoncalves | haleyb, octavia at least. | 16:20 |
johnsom | I don't think py2 removal has merged in octavia-lib yet | 16:21 |
cgoncalves | kuryr folks reported the amphora-agent started crashing on centos 7. I'd have to scroll up to find the exact reason | 16:21 |
haleyb | we 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 this | 16:21 |
johnsom | This 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 patch | 16:23 |
cgoncalves | FYI, I kept compatibility with centos 7 amphorae | 16:23 |
haleyb | so is there anyone/anything consuming octavia as a library? if so starting python2 removal (like six library) should wait until next cycle | 16:24 |
johnsom | Frankly, some py2/py3 combinations running DIB will break until this merges: | 16:24 |
johnsom | #link https://review.opendev.org/697211 | 16:24 |
johnsom | haleyb No | 16:24 |
johnsom | Only octavia-lib, Octavia should never be imported | 16:25 |
haleyb | burn the boats i say then :) | 16:25 |
cgoncalves | http://codesearch.openstack.org/?q=octavia%3E%3D&i=nope&files=&repos= | 16:25 |
haleyb | "should never" != "is not" | 16:26 |
johnsom | I think they were/are fixing that | 16:26 |
johnsom | haleyb Sadly true | 16:26 |
johnsom | I don't think that is a blocker for us though, we should still move forward | 16:27 |
cgoncalves | +1 | 16:27 |
haleyb | ack, i'll try and split it into many patches to make it reviewable (and revertable) | 16:28 |
johnsom | Any other updates today? | 16:28 |
johnsom | So this next topic I wish our PTL was present.... | 16:30 |
johnsom | #topic OVN driver under Octavia project governance | 16:30 |
*** openstack changes topic to "OVN driver under Octavia project governance (Meeting topic: Octavia)" | 16:30 | |
johnsom | There were a few patches proposed this week to move the OVN provider under the Octavia project. | 16:30 |
johnsom | I don't think that was proposed or discussed with the Octavia team prior to those patches. | 16:31 |
cgoncalves | I don't remember that being proposed or discussed either | 16:32 |
johnsom | There 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 |
haleyb | I can apologize for that, think it was just a decision based on the progress of the networking-ovn code movement | 16:33 |
johnsom | haleyb Was there any push back from the neutron team in retaining ownership of that code? | 16:34 |
haleyb | it 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 well | 16:34 |
johnsom | If there is, maybe we should add this to next week's agenda and make sure that rm_work can join. | 16:34 |
haleyb | i 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 company | 16:36 |
haleyb | in the end i don't think the neutron team will not accept it | 16:37 |
johnsom | I propose we table this topic until rm_work (PTL) can be present. Is that ok or is this time sensitive? | 16:37 |
haleyb | we can wait until rm_work is around, i think having an answer in a week is good enough | 16:38 |
johnsom | Ok, thanks. I'm just concerned that we don't have quorum at the moment. | 16:38 |
johnsom | #topic Open Discussion | 16:39 |
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)" | 16:39 | |
johnsom | Other topics this week? | 16:39 |
johnsom | I 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 |
johnsom | It will be nice to have jobboard in early in Ussuri as it was one of our priorities for Ussuri from the PTG. | 16:42 |
cgoncalves | ataraday_, thanks a lot for all the great work and sticking around despite low review attention in past months | 16:43 |
johnsom | +100 | 16:43 |
ataraday_ | cgoncalves, johnsom Thanks! Though it is not finished yet :) | 16:44 |
johnsom | If there are not any more topics this week I will close out the meeting. | 16:44 |
johnsom | #endmeeting | 16:44 |
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews" | 16:44 | |
openstack | Meeting ended Wed Dec 4 16:44:41 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:44 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.html | 16:44 |
johnsom | Thanks everyone! | 16:44 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.txt | 16:44 |
openstack | Log: http://eavesdrop.openstack.org/meetings/octavia/2019/octavia.2019-12-04-16.01.log.html | 16:44 |
openstackgerrit | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add an option to pass an existing subnet as VIP IPv6 subnet https://review.opendev.org/697318 | 16:46 |
cgoncalves | thanks for running the meeting, michjohn! | 16:47 |
johnsom | Sure, NP | 16:48 |
johnsom | sorrison 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 IRC | 17:07 | |
cgoncalves | btw, octavia gate is broken. we need an octavia-lib release | 17:11 |
johnsom | Ugh | 17:11 |
cgoncalves | http://zuul.openstack.org/builds?job_name=octavia-tox-functional-py37-tips | 17:11 |
johnsom | Hmm, this first one, py36 test, looks like it might be a py2/py3 issue | 17:15 |
johnsom | The oslo policy enforcer check is failing | 17:15 |
johnsom | Will need to dig more to confirm | 17:16 |
johnsom | Oh, that might just be that patch. lol | 17:18 |
johnsom | Yeah, ok, that was just a bad patch | 17:18 |
johnsom | I can propose a octavia-lib release. I thought rm_work was going to do that last night. | 17:19 |
johnsom | We should also talk about how to untangle this... non-voting might be the right answer | 17:19 |
johnsom | FYI, octavia-lib release patch: https://review.opendev.org/697330 | 17:24 |
cgoncalves | thank you | 17:28 |
openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Add support for CentOS 8 amphora images https://review.opendev.org/697128 | 17:49 |
cgoncalves | ^ ready to be reviewed | 17:49 |
*** rpittau is now known as rpittau|afk | 17:50 | |
*** tesseract has quit IRC | 18:04 | |
johnsom | Nice, looking | 18:06 |
johnsom | cgoncalves Is there a tempest patch that uses that image? | 18:09 |
johnsom | I see previous runs had a job, but this one only has the publish. | 18:12 |
*** maciejjozefczyk has quit IRC | 18:16 | |
cgoncalves | johnsom, previous patch sets included a scenario centos8 job to test the image | 18:18 |
cgoncalves | the job needs to be created in octavia-tempest-plugin | 18:18 |
cgoncalves | I was delaying creating the job until we had a centos7 controller job functional but ETA for that is unknown | 18:20 |
cgoncalves | so... ubuntu controller + centos8 amp? | 18:20 |
johnsom | Should work... | 18:20 |
cgoncalves | it does work. the job in previous patch sets ran against that configuration | 18:21 |
cgoncalves | octavia-v2-dsvm-scenario-centos-8 or should we make it explicit it's a mix-in? | 18:22 |
johnsom | Well, we could just go with that and "fix" it to be pure centos-8 when we are ready | 18:23 |
cgoncalves | +1 | 18:23 |
*** vishalmanchanda has quit IRC | 18:24 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Add octavia-v2-dsvm-scenario-centos-8 https://review.opendev.org/697336 | 18:26 |
*** ccamposr__ has joined #openstack-lbaas | 18:40 | |
*** ccamposr has quit IRC | 18:42 | |
rm_work | Yep, sounds good (for the cent8 thing) | 18:43 |
rm_work | Sorry forgot to check my calendar when I passed out last night T_T | 18:43 |
*** salmankhan has quit IRC | 19:06 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix multi-listener LB with missing certificate https://review.opendev.org/692208 | 19:18 |
johnsom | A rebasing we go.... | 19:18 |
rm_work | ah i forgot we don't +2 the release patches T_T | 19:21 |
johnsom | Yeah, out of our hands. Though the PTL +1 helps.... hint, hint | 19:22 |
rm_work | yeah i did it :D | 19:22 |
johnsom | Yeah, you got it already | 19:22 |
rm_work | but carlos is the releasemanager he and did +1 :) | 19:22 |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Fix multi-listener LB client auth/re-encryption https://review.opendev.org/693586 | 19:26 |
johnsom | Ok, rebases done, waiting on some gate jobs to vote, so time for lunch. | 19:28 |
rm_work | ah thought the lib release was blocking our gates? | 19:36 |
rm_work | cgoncalves: 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 does | 19:40 |
rm_work | we probably want that gate disabled for as little time as possible | 19:40 |
rm_work | johnsom: so after the lib change merges we need to update g-r? or no | 19:41 |
rm_work | we can update the lower-constraint in the patch we need the new version, but will the requirements-check jobs complain? | 19:41 |
rm_work | IIRC they do? but maybe that changed | 19:41 |
rm_work | also: I'm +2 all the way up through l7rule on the jobboard stuff | 19:44 |
rm_work | so, 5 more that could +W | 19:44 |
*** gcheresh has joined #openstack-lbaas | 19:54 | |
johnsom | The release should post the G-R patch (upper really, but details) | 20:03 |
rm_work | ah | 20:05 |
rm_work | didn't realize that was automated | 20:05 |
johnsom | rm_work https://review.opendev.org/697359 | 20:10 |
rm_work | johnsom: see dib channel | 20:11 |
rm_work | also: bunch of jobboard patches waiting for a second +2 still :D | 20:14 |
cgoncalves | johnsom, does https://review.opendev.org/#/c/694826/ really depend on what is set? | 20:17 |
*** abaindur has joined #openstack-lbaas | 20:17 | |
*** abaindur has quit IRC | 20:17 | |
cgoncalves | should have been rebased on top of said dependency patch, or was there a reason to use Depends-On? | 20:18 |
johnsom | cgoncalves no, I did that just because the queens gate is random failing and I thought the depends-on was the answer | 20:18 |
*** abaindur has joined #openstack-lbaas | 20:18 | |
*** abaindur has quit IRC | 20:18 | |
johnsom | Ah, yeah, mistake there. | 20:18 |
openstackgerrit | Michael Johnson proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs https://review.opendev.org/694826 | 20:19 |
*** abaindur has joined #openstack-lbaas | 20:19 | |
johnsom | Sadly you all need to re-vote that | 20:19 |
openstackgerrit | Adam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs https://review.opendev.org/694826 | 20:19 |
cgoncalves | ok, just wanted to check I was not missing some important detail | 20:19 |
johnsom | since I removed the depends-on | 20:19 |
rm_work | lol | 20:19 |
rm_work | i clicked rebase at the same time you changed the commit | 20:19 |
rm_work | let me undo that | 20:19 |
johnsom | Great, it reverted my change too | 20:19 |
rm_work | yeah lol | 20:20 |
cgoncalves | whooops | 20:20 |
rm_work | excellent threading | 20:20 |
johnsom | Guess that falls into the "don't do that" category | 20:20 |
cgoncalves | johnsom, https://review.opendev.org/#/c/691757/ | 20:21 |
rm_work | ok so ... i'm fixing it? or are you? :D | 20:21 |
cgoncalves | no depends-on but on top of the flush patch | 20:21 |
rm_work | i am ready to click the button to fix it, heh | 20:21 |
johnsom | No, skip the flush patch, it doesn't work | 20:22 |
openstackgerrit | Adam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs https://review.opendev.org/694826 | 20:22 |
openstackgerrit | Adam Harwell proposed openstack/octavia stable/queens: Fix a potential race condition with certs-ramfs https://review.opendev.org/694826 | 20:22 |
rm_work | ok there, back on master and no depends-on | 20:23 |
johnsom | +1 | 20:23 |
cgoncalves | https://review.opendev.org/#/q/project:openstack/octavia+status:open+NOT+branch:master | 20:23 |
cgoncalves | once those two merge, we could make some stable releases | 20:24 |
johnsom | I 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 conflict | 20:24 |
rm_work | should that other one also not depends-on? | 20:24 |
rm_work | or rather | 20:24 |
rm_work | rebase to HEAD? | 20:24 |
cgoncalves | rm_work, please | 20:24 |
openstackgerrit | Adam Harwell proposed openstack/octavia stable/queens: Fix issues with unavailable secrets https://review.opendev.org/691757 | 20:24 |
rm_work | k | 20:25 |
cgoncalves | danke | 20:25 |
rm_work | still waiting on +Ws on the the remaining intermediate jobboard patches :D | 20:27 |
rm_work | or will those break without the lib update? | 20:27 |
rm_work | ahh yeah i think i see | 20:28 |
rm_work | maybe we need to pull out the update to those functions that makes things pass | 20:29 |
rm_work | so that can merge quickly, then we can continue to work on the actual driver for the AZ stuff | 20:29 |
rm_work | which ... i can do once this g-r change merges | 20:29 |
rm_work | ah i can just do it now | 20:30 |
beisner | o/ hi, all. does anyone have time to advise on story #2006901? (https://bugs.launchpad.net/ubuntu/+source/octavia/+bug/1852599) | 20:35 |
openstack | Launchpad bug 1852599 in octavia (Ubuntu) "Cannot delete TLS Terminated listener if container has been deleted in barbican" [Undecided,New] | 20:35 |
johnsom | beisner 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 story | 20:37 |
beisner | https://storyboard.openstack.org/#!/story/2006901 | 20:37 |
johnsom | That is not it | 20:37 |
johnsom | looks like a duplicate | 20:37 |
beisner | hi johnsom - ah, a dup of a different story? | 20:38 |
johnsom | yes | 20:38 |
* beisner browses | 20:39 | |
johnsom | This one: https://storyboard.openstack.org/#!/story/2006676 | 20:39 |
johnsom | I think the delete case has merged all the way back to queens already | 20:40 |
beisner | ack, thanks johnsom | 20:41 |
johnsom | beisner Just looked at: https://review.opendev.org/#/q/7fcef817eca3ad177460ee6976633f3dbd078cf2 looks like queens is still pending merge | 20:42 |
johnsom | queens https://review.opendev.org/#/c/691757 | 20:43 |
beisner | johnsom: ok cool. so, we'll work on stable release updates in Ubuntu and the Ubuntu Cloud Archive. tyvm | 20:44 |
johnsom | Sure, NP. I think upstream stable releases will happen soon as well if those haven't already been released. | 20:45 |
coreycb | johnsom: hi, regarding 691757, do you know of octavia has upcoming stable point releases planned that will include it? | 20:57 |
johnsom | coreycb Yeah, just commented on that. I think it is planned this week. | 20:58 |
coreycb | johnsom: great! thanks. we'll keep an eye out for them. | 20:58 |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Adjust for new octavia-lib LB signature https://review.opendev.org/697365 | 21:00 |
rm_work | ^^ we can fast-track that I think | 21:03 |
rm_work | then take our time on the real AZ oatch | 21:03 |
rm_work | *patch | 21:03 |
johnsom | +2's on the two centos 8 patches. Scenario passed | 21:04 |
rm_work | nice | 21:04 |
rm_work | are our gates working? | 21:04 |
rm_work | i thought the lib change would break us | 21:04 |
rm_work | (tips) | 21:04 |
rm_work | ah yeah, it did | 21:05 |
rm_work | once reqs change merges, then we can merge my change there, and then gates will be good | 21:05 |
rm_work | but the reqs change failed briefly | 21:05 |
*** servagem has quit IRC | 21:17 | |
*** salmankhan has joined #openstack-lbaas | 21:25 | |
openstackgerrit | Adam Harwell proposed openstack/octavia master: Support creating an LB in a specified availability-zone https://review.opendev.org/693762 | 21:28 |
rm_work | sorrison: rebased on top of the reqs change patch | 21:28 |
rm_work | let's see | 21:28 |
rm_work | hopefully more stuff will pass | 21:28 |
rm_work | though i still need to do the spares-pool fixes | 21:28 |
sorrison | ok nice | 21:29 |
sorrison | if you're busy I should have some time today if you want me to sort it | 21:29 |
openstackgerrit | Adam Harwell proposed openstack/octavia-lib master: Return tips jobs to voting https://review.opendev.org/694895 | 21:30 |
rm_work | if you think you can do it easily | 21:30 |
rm_work | I just know I have all the context on that :D | 21:30 |
rm_work | but i can point you at the right stuff | 21:30 |
rm_work | johnsom / cgoncalves https://review.opendev.org/#/c/694895/ updated to point it at a better dep | 21:31 |
rm_work | needs revote ^^ | 21:31 |
sorrison | Is 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 IRC | 21:32 | |
rm_work | yeah | 21:32 |
rm_work | so like | 21:32 |
rm_work | https://review.opendev.org/#/c/558962/32/octavia/controller/worker/v1/tasks/amphora_driver_tasks.py | 21:32 |
rm_work | that will be necessary I think | 21:32 |
rm_work | and 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_work | but, 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 believe | 21:34 |
rm_work | probably this: https://review.opendev.org/#/c/558962/32/octavia/db/repositories.py | 21:35 |
rm_work | trying to find the bit where it actually manages spares | 21:35 |
sorrison | yeah ok, makes sense. | 21:35 |
rm_work | oh i think it's the FIRST change in: https://review.opendev.org/#/c/558962/32/octavia/controller/worker/v1/flows/amphora_flows.py | 21:37 |
rm_work | `get_create_amphora_flow` is what we use for spares, I believe | 21:37 |
rm_work | and you'll need slightly different code for making sure the `_get_create_amp_for_lb_subflow` picks the right zone amp | 21:38 |
rm_work | ah probably it would be that same change, but with new logic in amphora_driver_tasks.PickAmphoraZones | 21:39 |
rm_work | anyway, welcome to give it a shot :D | 21:39 |
rm_work | but if you're like "wtf", i can try to look tomorrow | 21:39 |
sorrison | yeah 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 moments | 21:43 |
*** gcheresh has quit IRC | 21:45 | |
*** salmankhan has quit IRC | 21:57 | |
rm_work | yeah my brain is even having some stuttering on this right now, need to get back into it | 22:07 |
rm_work | it's been too long | 22:07 |
rm_work | and now we've heard from yet more people who want that patch working as well | 22:08 |
rm_work | so i'm going to have to fix it up once we get this done | 22:08 |
rm_work | but it'll be simplified a lot if we're including a bunch of the code from it ;) | 22:08 |
*** goldyfruit___ has joined #openstack-lbaas | 22:09 | |
*** strobert1 has joined #openstack-lbaas | 22:09 | |
*** goldyfruit_ has quit IRC | 22:12 | |
*** goldyfruit_ has joined #openstack-lbaas | 22:12 | |
*** goldyfruit___ has quit IRC | 22:14 | |
*** rcernin has joined #openstack-lbaas | 22:19 | |
*** TrevorV has quit IRC | 22:33 | |
*** tkajinam has joined #openstack-lbaas | 23:05 | |
johnsom | haleyb octavia-ovn-provider? Killing me here. | 23:09 |
johnsom | That driver is all OVN native code writing directly to the ovsdb... | 23:12 |
openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Introduce a simple HTTPS server for TLS test. https://review.opendev.org/628891 | 23:27 |
johnsom | FYI, I just updated the priority review list with our progress. | 23:36 |
*** goldyfruit_ has quit IRC | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!