Wednesday, 2023-11-22

opendevreviewQuentin GROLLEAU proposed openstack/octavia master: Add validation for minimum number of ips when the subnet is specified  https://review.opendev.org/c/openstack/octavia/+/89880309:33
opendevreviewLê Minh Thư proposed openstack/octavia master: Add validate L7Rule containing special characters  https://review.opendev.org/c/openstack/octavia/+/90158414:17
gthiemonge#startmeeting Octavia16:00
opendevmeetMeeting started Wed Nov 22 16:00:20 2023 UTC and is due to finish in 60 minutes.  The chair is gthiemonge. 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
gthiemongehi16:00
johnsomo/16:00
tweiningo/16:00
oschwarto/16:00
gthiemonge#topic Announcements16:01
gthiemongeno announcements from me, do you have any?16:01
johnsomMilestone 1 was last week16:02
gthiemongeright16:02
tweiningnothing from me16:02
johnsomOtherwise I don't think I have anything16:02
gthiemonge#topic CI Status16:03
QGo/16:03
gthiemongeReminder, we have some patches in review for the DB deadlock issue16:04
gthiemongehttps://bugs.launchpad.net/octavia/+bug/203879816:04
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/89966216:04
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/89966316:04
opendevreviewPierre-Yves Jourel proposed openstack/octavia master: Add possibility to Resize a Load Balancer  https://review.opendev.org/c/openstack/octavia/+/89021516:05
gthiemonge#topic Brief progress reports / bugs needing review16:06
johnsomI am working on this bug: https://bugs.launchpad.net/octavia/+bug/204358216:08
gthiemonge+116:08
johnsomHandling certs with empty subject fields. I plan to have a patch today, hopefully a test as well16:08
gthiemongeI've been working on health-monitor issues16:08
gthiemongethere are 2 bugs:16:09
gthiemonge1. Bug with HTTP/HTTPS HMs on pools with ALPN16:09
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/90143516:09
gthiemonge2. Bug with TLS-HELLO HMs16:09
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/90152416:09
tweiningI assume the reason why https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/893066/16 still has V-1 is that https://review.opendev.org/q/I700c65fb17bad28b2b922e03d9c94c4716de9cbe hasn't been merged yet, right?16:10
gthiemongeyeah probably16:11
oschwartright, I was about to write about it16:11
oschwartnoop api u/s jobs fail because they could not find the new noop certificate manager16:13
pyjouAlso I respond to comments in my RFE https://review.opendev.org/c/openstack/octavia/+/88549016:14
gthiemongepyjou: thanks16:14
pyjouAnd I've made a new patchset for this change https://review.opendev.org/c/openstack/octavia/+/89021516:15
gthiemongepyjou: the flavor cannot be updated with PUT16:15
johnsomThere was some interest in this spec on the mailing list this week16:15
pyjougthiemonge: I had a discussion about PUT vs POST on this comment: https://review.opendev.org/c/openstack/octavia/+/890215/comment/60969576_6795c5f5/16:18
gthiemongepyjou: sorry I was replying to your comment in https://review.opendev.org/c/openstack/octavia/+/88549016:20
gthiemongeI'm looking for a way to revert the resize16:21
gthiemongewithout having to create flavor/flavorprofile for the default nova flavor16:21
johnsomRevert inside the flow?16:22
gthiemongenop, for instance, I resize the LB with an Octavia flavor that uses a nova flavor "amphora-big", then I change my mind, I want to use the default flavor16:23
johnsomI think my advice was to keep things a bit more simple such that if the user wanted to revert they just resize again.16:23
gthiemongehow do i switch back to my "amphora-default" flavor?16:24
johnsomThe same way you switched to amphora-big IMO16:24
gthiemongeyeah so we need octavia flavors/flavorprofiles for the default flavor16:24
gthiemongenot a huge issue if it's documentetd16:25
oschwartso a resize revert would require 2 failovers?16:25
johnsomAh, I see what you are saying, resize to flavor None16:25
pyjouA revert solution was proposed. Then I followed Johnsom's advice to remove the revert because users can just resize again to revert.16:25
gthiemongepyjou: yeah I think it's acceptable16:26
gthiemongepyjou: it would be great to have a admin doc that explains this feature16:26
gthiemongemaybe here https://docs.openstack.org/octavia/latest/admin/guides/operator-maintenance.html16:27
pyjougthiemonge: No problem at all16:27
gthiemongecool16:30
gthiemonge#topic Open Discussion16:30
QGis it the housekeeping that is supposed to renew the certificates used between the amphorae and the workers?16:31
johnsomYes16:32
johnsomHousekeeping is the periodic job engine16:32
QGif the worker certificate is renewed (before it expires) will housekeeping renew it?16:33
johnsomHousekeeping will only renew the certificates issued to the amphora16:34
gthiemongepyjou: maybe the resize feature could be described there: https://docs.openstack.org/octavia/latest/admin/flavors.html16:34
pyjougthiemonge: Ack16:34
QGbecause for some reason the certificate on the worker side have been renewed, and the worker can no longer talk to amphora, and i was thinking may be the housekeeping can renew the amphora certif when it doesn't correspond anymore to the worker one 16:36
johnsomThe control plane side needs to be manually renewed. This is usually done with the deployment tooling, so it can do a rolling update across the controllers.16:38
johnsomWe also tend to use lengthy validity periods for the control plane side.16:38
QGok thanks johnsom for the infos !16:39
johnsom#link  https://docs.openstack.org/octavia/latest/admin/guides/operator-maintenance.html#rotating-cryptographic-certificates16:39
tweininglengthy = 10 years AFAIR :)16:39
johnsomYeah, we typically do 50 for the CA, then 10 for the control plane, then 1 year for the amphora. If I remember right16:39
gthiemongeany other topics?16:43
tweiningno16:43
oschwartnothing from me16:44
opendevreviewLê Minh Thư proposed openstack/octavia master: Fix duplicate specified VIP among load balancers  https://review.opendev.org/c/openstack/octavia/+/90159516:44
tweiningdid have a look at the spec from nova about health endpoints?16:44
tweiningI forgot/had no time to read it16:44
gthiemongenop, I didn't16:45
tweiningo/16:50
gthiemongeok, I guess that's it!16:50
gthiemongethank you guys16:50
gthiemonge#endmeeting16:50
opendevmeetMeeting ended Wed Nov 22 16:50:29 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:50
opendevmeetMinutes:        https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-11-22-16.00.html16:50
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-11-22-16.00.txt16:50
opendevmeetLog:            https://meetings.opendev.org/meetings/octavia/2023/octavia.2023-11-22-16.00.log.html16:50
galleeHi, I am wondering if it is the expected behaviour to accept an HTTP_COOKIE  session persistence on a TCP pool. This has no effect on the session persistence (which is normal since HA proxy supports this for HTTP traffic)16:51
gthiemongegallee: I don't think it is expected, I know that some validation steps were missing for non-HTTP objects16:53
galleeShall I open a bug ? 16:55
johnsomIt can't hurt to open a bug. https://bugs.launchpad.net/octavia16:56
galleeok will do16:58
opendevreviewMichael Johnson proposed openstack/octavia master: Fix issue with certificates with no subject or CN  https://review.opendev.org/c/openstack/octavia/+/90168921:50
opendevreviewMichael Johnson proposed openstack/octavia master: Fix issue with certificates with no subject or CN  https://review.opendev.org/c/openstack/octavia/+/90168922:49

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