Wednesday, 2025-09-24

opendevreviewGregory Thiemonge proposed openstack/octavia-tempest-plugin master: DNM testing new settings for octavia-tempest  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/96216109:07
opendevreviewTakashi Kajinami proposed openstack/octavia master: Require valid volume size  https://review.opendev.org/c/openstack/octavia/+/95940414:24
*** zseguin_ is now known as zseguin14:26
opendevreviewTakashi Kajinami proposed openstack/octavia master: Validate default tls versions and protocols  https://review.opendev.org/c/openstack/octavia/+/95940614:31
tkajinamI wonder if https://review.opendev.org/c/openstack/octavia/+/934790 can be merged ?14:36
tkajinamI see the same was merged in the other octavia repos14:36
*** johnsom_ is now known as johnsom14:39
*** mnaser_ is now known as mnaser14:39
johnsom#startmeeting Octavia16:00
opendevmeetMeeting started Wed Sep 24 16:00:48 2025 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'octavia'16:00
gthiemon1eo/16:01
johnsom#topic Announcements 16:01
*** gthiemon1e is now known as gthiemonge16:01
johnsomHi everyone16:01
johnsomNext week is release week for Flamingo!16:01
johnsomAlso note, the PTG is coming up. The schedule is now open.16:02
johnsomI plan to book a couple of hours on Wednesday a bit earlier than this.16:02
johnsomDoes that work for everyone?16:02
gthiemongeyeah, that looks good16:03
johnsomI am thinking two hours earlier than this.16:03
opendevreviewMerged openstack/octavia master: reno: Update master for unmaintained/2023.1  https://review.opendev.org/c/openstack/octavia/+/93479016:03
johnsomOk, sounds good.16:03
johnsom#topic Brief progress reports / bugs needing review16:04
johnsomI have mostly been stuck doing downstream work, but I have got a few reviews in as well.16:04
johnsomI plan to focus on the rate limiting work today (finally)16:05
gthiemongeI rebased my work on the tempest tests for SCTP, I'm trying to integrate the tests in the "regular" scenario job, but now the duration is really close to the timeout value16:05
gthiemongemaybe a good topic for the PTG: can we remove/merge some tests?16:06
johnsomSigh, ping me when you think you have a setup that is stable. It would be great to get that in16:06
johnsomYeah, I agree, we should review the list of tests that are running. It's been a while16:06
gthiemongefor instance we have basic tests and hm tests, I think that the providers should support HMs now so maybe the basic tests are no longer needed16:07
johnsomHmm, yeah, I know OVN didn't support health monitors for a while. That might be an artifact of that16:08
johnsomAny other progress updates? I do have a bug I would like to discuss today16:10
gthiemongeI think that's all for me16:10
johnsom#topic Bug review16:10
johnsom#link https://bugs.launchpad.net/bugs/212557316:11
johnsomThis is an interesting one. It's a behavior in OVN that is causing problems, but raises a good question.16:11
johnsomSo in the early days (sounds so old, lol), the allowed address pairs port had to be admin down, otherwise neutron tried to pin it to one compute host.16:13
johnsomThis caused a lot of problems since it is not a "real" port, but just an IP address alias.16:13
johnsomThis bug implies this may no longer be the case with OVN.16:14
johnsomAnyone have any thoughts on this?16:14
johnsomI haven't tried creating an AAP port with admin up in many years, so I have no idea if this has changed16:15
gthiemongenot really... the VIP port should be "type: virtual" in OVN, we need to check if the admin_state=False is a condition for neutron to set this type16:15
johnsomOk, I wanted to bring it up here to see if there were any thoughts on it.16:16
johnsomI will add some comments and then add neutron to the affected project list. I think this is something to be resolved on the neutron side, but I will keep octavia on the list just in case we do need to change that behavior.16:17
gthiemonge+116:17
johnsom#topic Open Discussion16:17
johnsomAny other topics today?16:17
gthiemongeyes16:17
gthiemongeI've seen multiple patches for the infamous one-liner that moves the interfaces from the "root" network namespace to the right namespace16:19
gthiemongelike this one16:19
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/961406/4/octavia/amphorae/backends/agent/api_server/templates/amphora-netns.systemd.j216:19
gthiemongemy concern is that this line is unreadable and unmaintainable, so I would like to suggest to add a script to the amphora image, it could be shell or python, it would make it easier to maintain16:20
gthiemongeI'm sweating each time I try to understand it16:20
gthiemongeany opinion on this?16:20
gthiemonge(I already suggested it in the review but I didn't give a -1)16:21
johnsomlol, that has been a bash work of art16:21
johnsomI am ok with making it a script.16:22
johnsomI think it's only really used in the VM reboot scenario right?16:22
gthiemongeanother attempt here16:22
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/94963016:22
gthiemongeright16:22
gthiemongewhen the interfaces are already plugged when the amphora-haproxy namespace is created16:23
johnsomYeah, I have no problem with that. Frankly python might be cleaner / easier to read than doing the fun in bash16:23
danfaiAs I was touching the 949630, I was also wondering instead of that one-liner, if there is maybe a script to be executed instead of 13 lines of systemd ExecStart16:23
gthiemongedanfai: +116:25
johnsomSo there are two +1's to changing that systemd service16:25
danfaiWould it be good enough for you, if the one-lienr becomes a bash script, that just makes the one-liner a multi-liner and more readable in a script?16:26
gthiemongeI think if it becomes a script, we need to make it readable and maintainable, so basically we need to refactor it16:27
danfaiWhat I mean is, create a shell script that gets added to the amphora and this systemd file executes the script instead of the ExecStarts16:27
danfai+116:28
gthiemongeI think it can include all the execstarts16:28
danfaiIs the amphora-interface script used somewhere else?16:28
gthiemongeyeah it's called by the amphora-agent when a new interface is plugged16:29
gthiemongehttps://github.com/openstack/octavia/blob/e414abb9e596241fb110549ace8c472d63a6f34e/octavia/amphorae/backends/agent/api_server/osutils.py#L8916:29
danfaiah, ok, I see16:31
danfaiNot sure I have enough time currently, but if I find a few seconds, I can have a look at writing a bash script with a bit more comments as a start16:31
gthiemongeack16:32
johnsomThank you!16:32
johnsomAny other topics today?16:33
gthiemongeno, thanks!16:34
danfainot from me16:34
johnsomOk, thank you all and have a great week!16:35
johnsom#endmeeting16:35
opendevmeetMeeting ended Wed Sep 24 16:35:16 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:35
opendevmeetMinutes:        https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-09-24-16.00.html16:35
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-09-24-16.00.txt16:35
opendevmeetLog:            https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-09-24-16.00.log.html16:35
danfaiThanks, see you around16:35

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!