| openstackgerrit | Stephanie Djajadi proposed openstack/octavia master: WIP: Refactoring amphora stats driver interface https://review.opendev.org/737111 | 00:00 |
|---|---|---|
| *** armax has joined #openstack-lbaas | 00:29 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add support for nftables https://review.opendev.org/737661 | 00:31 |
| *** ivve has quit IRC | 00:35 | |
| *** rcernin has quit IRC | 01:12 | |
| *** rcernin has joined #openstack-lbaas | 01:17 | |
| *** yamamoto has joined #openstack-lbaas | 01:40 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add support for nftables https://review.opendev.org/737661 | 01:54 |
| *** yamamoto has quit IRC | 02:04 | |
| *** shtepanie has quit IRC | 02:20 | |
| *** rcernin has quit IRC | 02:34 | |
| *** rcernin has joined #openstack-lbaas | 02:36 | |
| *** yamamoto has joined #openstack-lbaas | 02:37 | |
| *** psachin has joined #openstack-lbaas | 03:43 | |
| *** rcernin has quit IRC | 03:46 | |
| *** rcernin has joined #openstack-lbaas | 03:55 | |
| *** gthiemonge has quit IRC | 04:04 | |
| *** rcernin has quit IRC | 04:04 | |
| *** gthiemonge has joined #openstack-lbaas | 04:04 | |
| *** rcernin has joined #openstack-lbaas | 04:05 | |
| lxkong | hi team, i have a feature request from our customer. Currently, when adding members (Nova VMs) to an LB, it means the VMs need to be open to its subnet. The request is, if Octavia can provider a SG that could be referenced as source to define the SGs of the members, i.e. the customer prefer to only allow the amphorae to access the VMs. | 04:20 |
| lxkong | Is that something you've already considered in the past? May I get some suggestion? | 04:21 |
| johnsom | Yeah, we have talked about it and tried a few things. I think we keep coming down to needing changes to neutron to make ot | 04:22 |
| johnsom | It work | 04:22 |
| johnsom | Private networks and backend client authentication are currently options, but don’t always meet the need. | 04:24 |
| johnsom | I think rm_work was the last to give it a try | 04:25 |
| *** gcheresh has joined #openstack-lbaas | 04:37 | |
| *** aannuusshhkkaa has quit IRC | 04:45 | |
| *** riuzen has joined #openstack-lbaas | 05:34 | |
| *** riuzen has quit IRC | 05:34 | |
| *** gthiemonge has quit IRC | 05:49 | |
| *** gthiemonge has joined #openstack-lbaas | 05:49 | |
| *** jamesdenton has quit IRC | 05:52 | |
| *** jamesdenton has joined #openstack-lbaas | 06:00 | |
| openstackgerrit | Gregory Thiemonge proposed openstack/octavia master: WIP Fix invalid DOWN status when updating a UDP pool https://review.opendev.org/736818 | 06:40 |
| rm_work | lxkong yes it's close, there's a spec in neutron for the feature we need, and a patch in Octavia that's waiting for it | 06:48 |
| rm_work | lxkong: unfortunately I haven't been able to work on the neutron spec... If you could take that over, it'd speed things up | 06:49 |
| rm_work | That's the best way you could help | 06:49 |
| *** rpittau|afk is now known as rpittau | 06:57 | |
| *** born2bake has joined #openstack-lbaas | 06:58 | |
| *** ccamposr has joined #openstack-lbaas | 07:20 | |
| *** maciejjozefczyk has joined #openstack-lbaas | 07:29 | |
| *** vishalmanchanda has joined #openstack-lbaas | 07:33 | |
| *** rcernin has quit IRC | 07:39 | |
| *** luksky has joined #openstack-lbaas | 07:40 | |
| *** dmellado has joined #openstack-lbaas | 07:43 | |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Remove amp_ssH_access_allowed option https://review.opendev.org/737706 | 08:17 |
| rm_work | nice | 08:22 |
| cgoncalves | "ssH" oops! | 08:26 |
| *** TMM has quit IRC | 08:44 | |
| *** TMM has joined #openstack-lbaas | 08:44 | |
| *** AlexStaf has joined #openstack-lbaas | 08:51 | |
| *** AlexStaf has quit IRC | 08:52 | |
| *** AlexStaf has joined #openstack-lbaas | 08:52 | |
| *** rcernin has joined #openstack-lbaas | 09:12 | |
| *** rcernin has quit IRC | 09:17 | |
| frickler | cgoncalves: seems sdk job is still failing after your patch http://paste.openstack.org/show/795142/ https://zuul.opendev.org/t/openstack/build/31c9cf89f0784c2e93fb660b3086623f | 09:36 |
| lxkong | rm_work: may i have the link for neutron spec? | 09:52 |
| lxkong | johnsom: also thank you | 09:52 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Fix missing params in amphora base and noop driver https://review.opendev.org/737731 | 10:01 |
| cgoncalves | frickler, err, sorry about that. can you check if ^ fixes the issue by updating the depends-on? | 10:02 |
| cgoncalves | lxkong, I think it is https://review.opendev.org/#/c/724207/ | 10:02 |
| *** tkajinam has quit IRC | 10:03 | |
| cgoncalves | lxkong, also: https://review.opendev.org/#/c/730736/ , https://review.opendev.org/#/c/730737/ and https://review.opendev.org/#/c/730784/ | 10:04 |
| openstackgerrit | Carlos Goncalves proposed openstack/octavia master: Fix missing params in amphora base and noop driver https://review.opendev.org/737731 | 10:05 |
| lxkong | cgoncalves: ta | 10:05 |
| *** rcernin has joined #openstack-lbaas | 10:10 | |
| *** rpittau is now known as rpittau|bbl | 10:11 | |
| *** rcernin has quit IRC | 10:15 | |
| *** luksky has quit IRC | 10:16 | |
| frickler | cgoncalves: updated, thx | 10:19 |
| cgoncalves | thanks | 10:21 |
| *** luksky has joined #openstack-lbaas | 10:28 | |
| openstackgerrit | Merged openstack/python-octaviaclient master: Show operating_status in listing LB output https://review.opendev.org/732469 | 10:43 |
| *** psachin has quit IRC | 10:57 | |
| *** rcernin has joined #openstack-lbaas | 11:27 | |
| *** rcernin has quit IRC | 11:34 | |
| *** rcernin has joined #openstack-lbaas | 11:39 | |
| *** gcheresh has quit IRC | 11:42 | |
| *** servagem has joined #openstack-lbaas | 11:58 | |
| *** rcernin has quit IRC | 12:08 | |
| *** TMM has quit IRC | 12:08 | |
| *** TMM has joined #openstack-lbaas | 12:08 | |
| *** rpittau|bbl is now known as rpittau | 12:13 | |
| *** ataraday_ has quit IRC | 12:14 | |
| *** ccamposr__ has joined #openstack-lbaas | 12:31 | |
| *** ccamposr has quit IRC | 12:33 | |
| *** rcernin has joined #openstack-lbaas | 12:35 | |
| *** gcheresh has joined #openstack-lbaas | 12:51 | |
| *** rcernin has quit IRC | 13:29 | |
| *** TrevorV has joined #openstack-lbaas | 13:54 | |
| *** luksky has quit IRC | 14:05 | |
| *** also_stingrayza is now known as stingrayza | 14:08 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Add an optional tenant flow log offload test https://review.opendev.org/673720 | 14:08 |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Add an optional amphora admin log offloading test https://review.opendev.org/673726 | 14:08 |
| rm_work | gonna miss the meeting today T_T have fun | 14:11 |
| johnsom | Ok | 14:13 |
| *** luksky has joined #openstack-lbaas | 14:19 | |
| *** gcheresh has quit IRC | 14:37 | |
| *** also_stingrayza has joined #openstack-lbaas | 15:06 | |
| *** stingrayza has quit IRC | 15:10 | |
| *** gcheresh has joined #openstack-lbaas | 15:19 | |
| *** gcheresh has quit IRC | 15:30 | |
| *** yamamoto has quit IRC | 15:34 | |
| *** ataraday_ has joined #openstack-lbaas | 15:50 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Fix availability zone API tests https://review.opendev.org/737191 | 15:57 |
| *** aannuusshhkkaa has joined #openstack-lbaas | 15:58 | |
| *** shtepanie has joined #openstack-lbaas | 15:59 | |
| johnsom | #startmeeting Octavia | 16:00 |
| openstack | Meeting started Wed Jun 24 16:00:25 2020 UTC and is due to finish in 60 minutes. The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: Octavia)" | 16:00 | |
| openstack | The meeting name has been set to 'octavia' | 16:00 |
| cgoncalves | hi | 16:00 |
| johnsom | Hi everyone (anyone?) | 16:00 |
| gthiemonge | hi | 16:00 |
| aannuusshhkkaa | hey! | 16:01 |
| shtepanie | hi | 16:01 |
| haleyb | hi | 16:01 |
| johnsom | Pretty light agenda today | 16:01 |
| johnsom | #topic Announcements | 16:01 |
| *** openstack changes topic to "Announcements (Meeting topic: Octavia)" | 16:01 | |
| johnsom | I don't think I have any real announcements this week. | 16:02 |
| johnsom | Time flies, we are now in the MS2 phase of Victoria | 16:02 |
| johnsom | #link https://releases.openstack.org/victoria/schedule.html | 16:03 |
| johnsom | There was an e-mail to the openstack-discuss list asking for input on multi-architecture Octavia. | 16:03 |
| johnsom | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015579.html | 16:04 |
| johnsom | Please add your input | 16:04 |
| cgoncalves | just a note about the MS2, that we set to "Aspire to make AmphoraV2 the default driver at MS2 checkpoint" from the PTG | 16:04 |
| *** yamamoto has joined #openstack-lbaas | 16:04 | |
| johnsom | Right, that will be the end of July, so coming up quickly | 16:05 |
| johnsom | We had some criteria for that on the PTG etherpad: | 16:06 |
| johnsom | #link https://etherpad.opendev.org/p/octavia-virtual-V-ptg | 16:06 |
| johnsom | line 84 | 16:06 |
| cgoncalves | I agree with your reply email. I think Octavia can support multiple amphora architecture types and that can be done via Octavia flavors | 16:06 |
| johnsom | Yeah, I don't think this is a bad thing. I just don't have any cycles to help that effort. | 16:07 |
| johnsom | Any other announcements this week? | 16:07 |
| cgoncalves | maybe the multi-arch SIG in openstack can help | 16:07 |
| johnsom | Yep. I don't think it would be a lot of work to add architecture to the flavors. Just some time. | 16:08 |
| johnsom | #topic Brief progress reports / bugs needing review | 16:09 |
| *** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)" | 16:09 | |
| johnsom | I have been busy with reviews and minor patches to fix / address a few issues. | 16:09 |
| johnsom | I am also working on adding support for nftables to Octavia. | 16:09 |
| johnsom | If you are not familiar, nftables is proposed to replace iptables. Debian buster has already made the switch. | 16:10 |
| *** yamamoto has quit IRC | 16:10 | |
| johnsom | It's just a slightly different syntax. Bionic supports it, but does not use it by default. | 16:10 |
| gthiemonge | I'm working on the UDP health-monitor issues in tempest. I may have an idea, but I need to reproduce the bug with more debug messages to confirm it would work... Reproduction rate is really low, we hit that issue 3 times in the CI in the last 7 days | 16:11 |
| cgoncalves | I worked on fixing the grenade job in multiple branches, patches are up for review. these are gate blockers. | 16:11 |
| johnsom | Ah, are they good now? do you have a list? | 16:11 |
| cgoncalves | I also fixed some oopsies in the failover refactor patch, thanks to the SDK for finding and reporting them! | 16:12 |
| cgoncalves | https://review.opendev.org/#/q/topic:octavia-grenade | 16:12 |
| johnsom | #link https://review.opendev.org/#/q/topic:octavia-grenade | 16:13 |
| johnsom | Thanks! | 16:13 |
| johnsom | Yeah, I noticed the timeout_dict wasn't there, but I saw it had a default and figured it didn't matter. I should have flagged it. | 16:13 |
| cgoncalves | I took a few minutes to post a patch that adds amphora image tag capability to octavia flavors. needs is in line with the multi-arch email discussed earlier | 16:14 |
| johnsom | On the grenade jobs, I assume we need to review from the oldest to the newest but I see the queens patch failed the gates. Do these just need rechecks? | 16:14 |
| cgoncalves | order is: | 16:15 |
| cgoncalves | 1. https://review.opendev.org/#/c/736965/1 | 16:15 |
| cgoncalves | 2. https://review.opendev.org/#/c/736824/ | 16:15 |
| cgoncalves | I will follow-up then with forward-portings to newer stable branches | 16:15 |
| johnsom | Ok, thanks | 16:16 |
| cgoncalves | we can't fix grenade in rocky and queens until devstack gets fixed after the new uwsgi release | 16:16 |
| johnsom | Ah, that is still open? | 16:16 |
| cgoncalves | looks like so | 16:17 |
| cgoncalves | #link https://review.opendev.org/#/c/735615/ | 16:17 |
| cgoncalves | this was caught in stable/rocky https://review.opendev.org/#/c/736825/ | 16:17 |
| johnsom | Ugh, ok | 16:18 |
| johnsom | Any other updates? | 16:18 |
| johnsom | I think shtepanie had a stats driver patch proposed as well. | 16:19 |
| shtepanie | yup! continued to work on refactoring the driver with aannuusshhkkaa | 16:20 |
| johnsom | #link https://review.opendev.org/#/c/737111/ | 16:20 |
| shtepanie | and rm_work | 16:20 |
| johnsom | Yeah, great! I think you are looking for early stage review feedback on the approach right? | 16:20 |
| shtepanie | yes | 16:20 |
| johnsom | Excellent. | 16:21 |
| johnsom | #topic Open Discussion | 16:21 |
| *** openstack changes topic to "Open Discussion (Meeting topic: Octavia)" | 16:21 | |
| johnsom | Any other topics for today? | 16:21 |
| ataraday_ | I've got question about experimental jobs with amphorav2. Is there any guide how to create such jobs or does someone looked into it? | 16:22 |
| johnsom | Yes, let me see what I can find | 16:23 |
| cgoncalves | should be just adding an "experimental" section in zuul.d/projects.yaml like "gate", "periodic" and "check", no? | 16:25 |
| johnsom | Yeah, I think so. I thought that was discussed in the project guide, but I'm not finding it right away | 16:26 |
| ataraday_ | I'm not really familiar how jobs created, so will check zuul.d/projects.yaml | 16:28 |
| johnsom | Yeah, it would be a section at the bottom of projects.yaml. Where we have "gate" and "periodic" you would add a pipeline section for "experimental". | 16:29 |
| cgoncalves | ataraday_, https://opendev.org/openstack/neutron/src/branch/master/zuul.d/project.yaml#L111 | 16:29 |
| cgoncalves | ^ just an example from neutron | 16:29 |
| johnsom | Yeah, I was just going to look there. | 16:29 |
| ataraday_ | cgoncalves, Thanks! | 16:29 |
| johnsom | There is some mention of it here: https://docs.opendev.org/opendev/system-config/latest/zuul.html but not much | 16:30 |
| johnsom | I think to trigger it, you comment on the patch with "experimental" | 16:32 |
| cgoncalves | thought it was "check-experimental" | 16:32 |
| cgoncalves | "check experimental" actually | 16:33 |
| johnsom | Yeah, ok. Ugh, wish that was easy to find in the infra docs | 16:33 |
| johnsom | Yeah, I think "check experimental" | 16:33 |
| johnsom | Thanks! | 16:33 |
| ataraday_ | got it | 16:33 |
| johnsom | Any other topics today? | 16:35 |
| johnsom | Ok, thanks everyone. Have a great week! | 16:37 |
| johnsom | #endmeeting | 16:37 |
| *** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list: https://etherpad.openstack.org/p/octavia-priority-reviews" | 16:37 | |
| openstack | Meeting ended Wed Jun 24 16:37:37 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:37 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-06-24-16.00.html | 16:37 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-06-24-16.00.txt | 16:37 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/octavia/2020/octavia.2020-06-24-16.00.log.html | 16:37 |
| cgoncalves | thanks! | 16:37 |
| *** rpittau is now known as rpittau|afk | 16:41 | |
| *** ataraday_ has quit IRC | 17:31 | |
| openstackgerrit | Anushka Singh proposed openstack/octavia master: WIP: Refactoring amphora stats driver interface https://review.opendev.org/737111 | 17:38 |
| *** ccamposr__ has quit IRC | 17:50 | |
| *** ccamposr has joined #openstack-lbaas | 17:51 | |
| aannuusshhkkaa | Hey johnsom, did you get a chance to review our changes yet? | 18:00 |
| johnsom | https://review.opendev.org/737111? Not since we last chatted about it. Is there a specific section you would like me to look at? | 18:01 |
| aannuusshhkkaa | Yeap.. we removed the driver interface for the health updater. Did you review that already? | 18:02 |
| johnsom | No I have not. If you would like me to take a look at that again I can prioritize another review pass for this morning. I just have a patch I'm wrapping up to finish first. | 18:03 |
| mloza | Hello, I have amphoras that has status error probably cause after upgrading to train and doing failover to re-issue the certs. I can confirm that they orphan. What's the safest way to clean them in the DB? | 18:05 |
| johnsom | These are amphora that are not associated with a load balancer? | 18:06 |
| mloza | johnsom: yes | 18:06 |
| johnsom | Can you paste a "openstack loadbalancer amphora show" for one of them? | 18:06 |
| aannuusshhkkaa | johnsom: Yeah it's not urgent or anything. We just wanted to know if it looked okay and were hoping to do the release note this afternoon if everything worked out fine. | 18:07 |
| johnsom | Ok | 18:07 |
| mloza | johnsom: https://clbin.com/6GdHy | 18:08 |
| johnsom | mloza Yeah, ok, they did get hung up due to a certificate issue. We have tools to help with this coming in Victoria, but for now it would be a DB update. You would set status = DELETED, cert_busy = False, then delete the nova instance by hand. This assumes you know that the load balancer has other amps that are not in ERROR. I.e. that load balancer should have a BACKUP and MASTER amphora that are not in error. | 18:11 |
| mloza | johnsom: Got it. Thanks | 18:14 |
| *** vishalmanchanda has quit IRC | 18:17 | |
| *** gcheresh has joined #openstack-lbaas | 18:31 | |
| *** irclogbot_1 has quit IRC | 18:34 | |
| *** irclogbot_0 has joined #openstack-lbaas | 18:38 | |
| mloza | johnsom: cert_busy column has type int | 18:51 |
| johnsom | Oh, yeah, that is sqlalchemy strangeness. False is 0 | 18:51 |
| mloza | copy | 18:53 |
| *** yamamoto has joined #openstack-lbaas | 18:53 | |
| *** yamamoto has quit IRC | 18:58 | |
| *** gcheresh has quit IRC | 18:59 | |
| openstackgerrit | Merged openstack/octavia master: Fix missing params in amphora base and noop driver https://review.opendev.org/737731 | 19:37 |
| *** ccamposr has quit IRC | 20:18 | |
| *** ccamposr has joined #openstack-lbaas | 20:19 | |
| *** gcheresh has joined #openstack-lbaas | 20:30 | |
| *** gcheresh has quit IRC | 20:54 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add support for nftables https://review.opendev.org/737661 | 21:01 |
| *** gthiemonge has quit IRC | 21:04 | |
| *** gthiemonge has joined #openstack-lbaas | 21:04 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add support for nftables https://review.opendev.org/737661 | 21:05 |
| dawzon | johnsom Is there any work that still needs to be done for the minimum version patch? https://review.opendev.org/#/c/722121/ | 21:07 |
| *** maciejjozefczyk has quit IRC | 21:08 | |
| johnsom | dawzon Hi there. Frankly I lost track of that patch. It looks like I didn't have any open comments. I will try to have another look later today. | 21:08 |
| dawzon | johnsom Thanks, just wanted to make sure everything was done before I purge my devstack VM :) | 21:11 |
| johnsom | dawzon Ha, yeah, you are good. Thanks again for all of the great work! | 21:12 |
| *** servagem has quit IRC | 21:59 | |
| johnsom | aannuusshhkkaa shtepanie rm_work I have done another review on the stats patch. | 22:07 |
| *** luksky has quit IRC | 22:08 | |
| aannuusshhkkaa | johnsom, thanks! will look into those.. | 22:10 |
| openstackgerrit | Michael Johnson proposed openstack/octavia master: Add support for nftables https://review.opendev.org/737661 | 22:19 |
| *** TrevorV has quit IRC | 22:41 | |
| *** rcernin has joined #openstack-lbaas | 22:42 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Tags are strings attached to an entity with the purpose of classification into groups. Tags support API tests were added. https://review.opendev.org/735939 | 22:47 |
| *** tkajinam has joined #openstack-lbaas | 22:51 | |
| aannuusshhkkaa | johnsom: not sure if gerrit lets you know if i reply to your comment on the review.. does it? | 22:52 |
| johnsom | aannuusshhkkaa It does, but sometime it can get lost in the volume of messages I get, so if you have a question feel free to ping me here. | 22:54 |
| aannuusshhkkaa | octavia/amphorae/drivers/health/heartbeat_udp.py | 22:55 |
| aannuusshhkkaa | Line 97: | 22:55 |
| aannuusshhkkaa | So this is still a single driver call, are you going to change that in this patch or a different one? | 22:55 |
| aannuusshhkkaa | I still think, under this new model, this could become a singleton. | 22:55 |
| aannuusshhkkaa | -- What do you mean by "this could become a singleton"? Do you mean we should entirely do away with the driver call and directly incorporate the code in the handler? | 22:55 |
| johnsom | By a singleton I mean that it could load the drivers once, the first time, and then reuse that loaded set of drivers for future calls. | 22:56 |
| aannuusshhkkaa | gotcha! | 22:58 |
| johnsom | aannuusshhkkaa Something similar to this: https://github.com/openstack/octavia/blob/master/octavia/db/api.py#L31 | 22:59 |
| johnsom | Basically we should only need to load them once, then future calls will just be map calls on the loaded drivers. | 23:00 |
| aannuusshhkkaa | Yeah that makes sense. Thank you! | 23:02 |
| rm_work | I think we will make it do the "multiple driver loading" thing in a later patch | 23:03 |
| rm_work | I think we will do the singleton thing as part of that change | 23:06 |
| rm_work | just need to decide how we'll do that... i guess it could be in this patch | 23:07 |
| rm_work | looking at the options you linked from earlier still and trying to determine which is the correct approach for fanout | 23:07 |
| *** gthiemonge has quit IRC | 23:36 | |
| *** gthiemonge has joined #openstack-lbaas | 23:36 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!