| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Support for Jobboard etcd backend https://review.opendev.org/c/openstack/octavia/+/915834 | 14:33 |
|---|---|---|
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Update amphorav2/jobboard doc https://review.opendev.org/c/openstack/octavia/+/923763 | 14:33 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: DNM/WIP temporary workaround for etcd3gw https://review.opendev.org/c/openstack/octavia/+/923764 | 14:34 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add spec for rate limiting https://review.opendev.org/c/openstack/octavia/+/923318 | 14:36 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add spec for rate limiting https://review.opendev.org/c/openstack/octavia/+/923318 | 14:39 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add spec for rate limiting https://review.opendev.org/c/openstack/octavia/+/923318 | 15:21 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Jul 10 16:00:39 2024 UTC and is due to finish in 60 minutes. The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'octavia' | 16:00 |
| gthiemonge | o/ | 16:00 |
| johnsom | o/ | 16:00 |
| tweining | o/ | 16:00 |
| damiandabrowski | hi! | 16:00 |
| gthiemonge | #topic Announcements | 16:01 |
| gthiemonge | well, no annoucements from me this week | 16:01 |
| johnsom | Nothing from me either | 16:01 |
| tweining | let's move on then | 16:02 |
| gthiemonge | we passed Dalmatian-2, the next milestone is the final release for non-client libraries (Aug 19) then the week after is the Feature Freeze | 16:02 |
| gthiemonge | I'll skip CI status too, no update | 16:03 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:03 |
| gthiemonge | FYI the new etcd backend for taskflow jobboard merged earlier this week | 16:03 |
| gthiemonge | now we need a plugin in octavia to use it | 16:03 |
| johnsom | I have been working on reviews. | 16:03 |
| gthiemonge | I'm working on it but I found a bug in etcd3gw (I proposed a fix), so it means that we will need a release of taskflow (for the etcd backend) and etcd3gw (for the fix) before approving my patch | 16:04 |
| gthiemonge | thanks for the reviews guys | 16:04 |
| johnsom | I also posted some patches for issues in the Octavia devstack plugin when running on Ubuntu 24.04 (related to some SR-IOV testing work I have) | 16:04 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/923318 I updated the rate limiting spec. Thanks for your comments. It really helps to have different views. | 16:05 |
| johnsom | Yeah, I got to a few of the specs, I still have two more on the review pile | 16:06 |
| tweining | I have one question about the "notifications impact" section. Is it about event notifications https://docs.openstack.org/octavia/latest/admin/event-notifications.html | 16:06 |
| johnsom | Yes | 16:07 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/923318/comment/6b6187bb_28443eb1/ | 16:07 |
| gthiemonge | johnsom: do you expect new notifications with this feature? | 16:07 |
| gthiemonge | right now we only have notifications for the creation/update/deletion of the LBs | 16:08 |
| johnsom | Well, it adds two new API endpoints with Create/Update/Delete methods. So I would expect notifications to be generated for them, meaning there will be new notifications | 16:08 |
| johnsom | I don't know how much special code there would be for that, but at a minimum we will be sending new notifications. | 16:09 |
| tweining | hmm, I will need to look into that. It wasn't on my radar until now. | 16:09 |
| gthiemonge | it's not a huge task | 16:10 |
| johnsom | Hmm, do we only send them for LB create/update/delete? I thought we sent them for all of the API endpoints. | 16:10 |
| gthiemonge | nop | 16:10 |
| gthiemonge | https://opendev.org/openstack/octavia/src/branch/master/octavia/controller/worker/v2/tasks/notification_tasks.py | 16:10 |
| johnsom | Ok, then I was remembering it wrong. Disregard my notifications comment. | 16:11 |
| gthiemonge | I think we can specify the notifications in the spec, but I feel that we don't need to implement them now | 16:11 |
| tweining | ok | 16:11 |
| johnsom | (designate sends them for everything, so I am probably just getting the projects mixed) | 16:11 |
| gthiemonge | yeah notifications are limited | 16:11 |
| gthiemonge | haha | 16:11 |
| gthiemonge | we can improve the notifications in Octavia ;-) | 16:12 |
| johnsom | I have also looked into the IPv6 test failures. This is an interesting ordering thing. | 16:13 |
| johnsom | Does anyone remember why we had to remove the gateway address from the lb-mgmt-net when the ML2 is OVN? | 16:13 |
| noonedeadpunk | o/ | 16:14 |
| johnsom | I seem to remember it was some OVN bug, but a quick search didn't bring it up. | 16:14 |
| gthiemonge | johnsom: I remember something about ARP requests sent to an non-existant gateway | 16:14 |
| johnsom | #link https://github.com/openstack/neutron/blob/master/devstack/lib/octavia#L14 | 16:14 |
| gthiemonge | johnsom: maybe related to https://bugzilla.redhat.com/show_bug.cgi?id=1961162 | 16:15 |
| johnsom | So, if we remove the gateway all of the time to address this ordering bug, we have to remove the router from the network too, which means SLAAC and probably DHCP don't work anymore. | 16:16 |
| johnsom | Unless OVN doesn't have that limitation and it was only OVS/linux bridge | 16:17 |
| gthiemonge | I need to check but the IP address of the management interface might be static in most of cases | 16:17 |
| noonedeadpunk | I can distantly recall some backport in neutro regarding broadcast flood for OVN... but really vaguely, so might be not related... | 16:18 |
| johnsom | Yeah, I kind of thing so too. I may go down that path of removing the router too. My patch yesterday removes the DHCP from the o-hm0 interface. | 16:18 |
| johnsom | cloud-init should "do the right thing" in the amp instances without DHCP | 16:19 |
| noonedeadpunk | maybe it was https://bugs.launchpad.net/neutron/+bug/2069625... but not sure | 16:19 |
| noonedeadpunk | sorry folks, would there be an open discussion? As I failed to find a meeting agenda recently and have something to raise | 16:21 |
| gthiemonge | noonedeadpunk: please go ahead | 16:22 |
| noonedeadpunk | yeah, so that;'s the topic I've raised couple of PTGs ago and you pretty much aware about - AZs :D | 16:22 |
| gthiemonge | yeah i've seen some updates in gerrit :) | 16:22 |
| noonedeadpunk | I tried to make a different approach and target changes to nova scheduling groups | 16:23 |
| noonedeadpunk | and then arddennis also proposed the blueprint: https://review.opendev.org/c/openstack/octavia/+/923571 | 16:23 |
| noonedeadpunk | as during implementation it appears that multiple options are possible | 16:23 |
| noonedeadpunk | so I thought that it's best to have it written for futher discussion | 16:23 |
| noonedeadpunk | basically, we're having implementation mostly ready, except unsure about couple of points described in the blueprint | 16:24 |
| noonedeadpunk | frankly - I kinda like doing thing on a nova side. Though there was quite some level of complexity in how scheduler works with regards to AZs | 16:25 |
| noonedeadpunk | or better say ordering of data, as AZ was passed to filters directly... | 16:25 |
| noonedeadpunk | TL;DR - once you have a spec review day - ping me or arddennis for some discussion around this | 16:26 |
| arddennis | Maybe the main point. If compute_tasks is a good place to distribute amphora instances across AZ. Maybe there should be AZ defined for each amphora inside worker? | 16:27 |
| noonedeadpunk | as there're outstanding questions in spec | 16:27 |
| johnsom | It is one of the next specs on my list to review | 16:27 |
| gthiemonge | noonedeadpunk: ack, I will review it in the next days (maybe this week) | 16:27 |
| johnsom | That and re-review the resize one | 16:27 |
| * noonedeadpunk very sorry for us taking _that_ long to get any progress | 16:28 | |
| gthiemonge | BTW I like to read the rendered doc, but it doesn't look alright https://d2cee1cf1f9910cb6506-7163d5c589c56aef778972593b9aaf7c.ssl.cf2.rackcdn.com/923571/7/check/openstack-tox-docs/350a8d5/docs/contributor/specs/version15.0/multiaz_amphora_distribution.html | 16:28 |
| noonedeadpunk | ah, yes, will fix that right away | 16:28 |
| gthiemonge | thanks | 16:29 |
| gthiemonge | noonedeadpunk: are the unclear points highlighted in the spec? | 16:30 |
| noonedeadpunk | arddennis: ^ | 16:31 |
| arddennis | It is described in Alternatives | 16:31 |
| gthiemonge | (I added it to our priority review etherpad: https://etherpad.opendev.org/p/octavia-priority-reviews) | 16:32 |
| arddennis | It looks like controller_worker is a better place than compute_tasks. But I am deeply unsure if it is a good path either | 16:32 |
| gthiemonge | ok I see | 16:33 |
| noonedeadpunk | gthiemonge: I _think_ we can also propose our WIP on top of the spec. | 16:33 |
| noonedeadpunk | potentially it might be easier for you to grasp some spec parts with it... | 16:34 |
| noonedeadpunk | (with code implementing that) | 16:34 |
| noonedeadpunk | s/that/spec | 16:34 |
| gthiemonge | yeah that sounds good to me | 16:34 |
| noonedeadpunk | ok, agred then | 16:36 |
| gthiemonge | great! | 16:36 |
| noonedeadpunk | we'll try to do that asap so you could have a review later this week if have time for that :D | 16:37 |
| gthiemonge | anyways, I can take a look at the spec in its current form | 16:38 |
| noonedeadpunk | sure, thanks! sorry for hijacking the meeting | 16:39 |
| gthiemonge | hey np | 16:40 |
| tweining | I have nothing else to talk about today | 16:40 |
| gthiemonge | any other topics for today? | 16:40 |
| gthiemonge | ok! | 16:41 |
| gthiemonge | then, Have a good week! | 16:41 |
| gthiemonge | thank you folks | 16:41 |
| gthiemonge | #endmeeting | 16:41 |
| opendevmeet | Meeting ended Wed Jul 10 16:41:33 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:41 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-07-10-16.00.html | 16:41 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-07-10-16.00.txt | 16:41 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-07-10-16.00.log.html | 16:41 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add spec for rate limiting https://review.opendev.org/c/openstack/octavia/+/923318 | 16:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/octavia master: Add spec for multiple AZ amphora placement https://review.opendev.org/c/openstack/octavia/+/923571 | 16:55 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Stop using DHCP for o-hm0 interface in devstack https://review.opendev.org/c/openstack/octavia/+/923792 | 17:33 |
| opendevreview | Michael Johnson proposed openstack/octavia master: WIP Testing gateway none in devstack plugin https://review.opendev.org/c/openstack/octavia/+/907199 | 17:42 |
| noonedeadpunk | this looks better now :) https://6e2b41c04dce55eaa7bd-95baeb13f84d1cb0954af27128e2e856.ssl.cf2.rackcdn.com/923571/8/check/openstack-tox-docs/df0eb54/docs/contributor/specs/version15.0/multiaz_amphora_distribution.html | 18:00 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Stop using DHCP for o-hm0 interface in devstack https://review.opendev.org/c/openstack/octavia/+/923792 | 18:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!