| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 02:16 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 02:21 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 02:24 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 03:04 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 03:08 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Fix listener update when using SRIOV VIP https://review.opendev.org/c/openstack/octavia/+/919546 | 03:28 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add spec for custom SGs on VIP ports https://review.opendev.org/c/openstack/octavia/+/915114 | 11:15 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add sg_id to Vip data_model https://review.opendev.org/c/openstack/octavia/+/919718 | 11:18 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add get_security_group_by_id to network driver https://review.opendev.org/c/openstack/octavia/+/919719 | 11:18 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add vip_sg_ids to the API https://review.opendev.org/c/openstack/octavia/+/919720 | 11:18 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Handle custom security groups in controller https://review.opendev.org/c/openstack/octavia/+/919721 | 11:18 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia-lib master: Add vip_sg_ids and vip_sgs to the data model https://review.opendev.org/c/openstack/octavia-lib/+/919722 | 11:19 |
| opendevreview | Merged openstack/octavia master: Fix new pylint errors https://review.opendev.org/c/openstack/octavia/+/919587 | 11:25 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add typing information to octavia.amphorae package https://review.opendev.org/c/openstack/octavia/+/905193 | 11:32 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add typing information to octavia.api package https://review.opendev.org/c/openstack/octavia/+/905194 | 11:32 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code https://review.opendev.org/c/openstack/octavia/+/905195 | 11:32 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Integrate mypy type checker https://review.opendev.org/c/openstack/octavia/+/879749 | 11:32 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add typing information to remaining octavia code https://review.opendev.org/c/openstack/octavia/+/905195 | 11:57 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Integrate mypy type checker https://review.opendev.org/c/openstack/octavia/+/879749 | 11:57 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Use built-in json module https://review.opendev.org/c/openstack/octavia/+/917274 | 12:07 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/2024.1: Pin pylint on 2024.1 https://review.opendev.org/c/openstack/octavia/+/919727 | 12:27 |
| gthiemonge | ^ unblocks the gates on 2024.1 | 12:28 |
| QG | Hi, Is it my configuration or do we not have the user_id in the octavia API logs? | 12:41 |
| gthiemonge | QG hey | 12:49 |
| gthiemonge | we should inherit it from oslo_log if it's not defined in the octavia config files | 12:49 |
| gthiemonge | https://opendev.org/openstack/oslo.log/src/branch/master/oslo_log/_options.py#L153-L158 | 12:49 |
| gthiemonge | but AFAIK the setting is also defined by the deployment tools | 12:52 |
| opendevreview | Merged openstack/octavia master: Ignore temp file created by coverage https://review.opendev.org/c/openstack/octavia/+/918204 | 14:26 |
| QG | Thanks gthiemonge ! | 14:55 |
| QG | Shouldn't we use the ciphersuites parameter instead of ciphers one when using TLSv1.3 in haproxy ? | 15:02 |
| QG | When you specify in the pool the tls_ciphers to ['TLSv1.2', 'TLSv1.3'] and the list of ciphers to : TLS_AES_128_GCM_SHA256:TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256 | 15:03 |
| johnsom | If I remember correctly, we include both in the default config and openssl knows which ones to pull out of the list for which TLS version | 15:04 |
| QG | humm because Haproxy give you an error like : [ALERT] (5360) : Proxy 'ae99a24d-5919-4bd0-966d-ddf80245ac94:d01e9f6c-62f6-4943-a467-e6707f697c2f', server 'cee5d52f-2965-4fe0-a553-302ad4b2c554' [haproxy.cfg.new-failed:44] : unable to set SSL cipher list to 'TLS_AES_128_GCM_SHA256:TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256'. | 15:05 |
| johnsom | Hmm, that seems odd. Can you open a bug for it? | 15:06 |
| johnsom | I mean, those two settings are the default in Octavia (with a few more ciphers added in. | 15:07 |
| QG | Yes sure ! | 15:08 |
| johnsom | FYI, I found that HAProxy changed a few of the status variable formats in HAProxy 2.4, so the grafana dashboard has some issues with the "status" pie charts. I will fix that in the dashboard soon-ish. There will be a octavia patch too, but it will only update the help text. | 15:10 |
| QG | ah cool ! | 15:21 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed May 15 16:00:36 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:01 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | * 2024.2 Dalmatian Release Schedule: Dalmatian-1 | 16:02 |
| gthiemonge | it's already Dalmatian-1 | 16:02 |
| johnsom | FYI: The OpenStack Summit Asia call for papers closes May 29th. | 16:02 |
| johnsom | #link https://openinfrafoundation.formstack.com/forms/openinfra_asia_summit_2024 | 16:02 |
| gthiemonge | I can't remember if we defined some internal deadlines/milestones | 16:02 |
| gthiemonge | Do you have anything for Dalmatian-1? patches, specs, ...? | 16:03 |
| gthiemonge | johnsom: +1 | 16:03 |
| QG | o/ | 16:03 |
| johnsom | I was trying to think myself. | 16:04 |
| johnsom | The SG spec is still open | 16:05 |
| johnsom | As is resize | 16:05 |
| johnsom | Though there are two of those, so confusing | 16:05 |
| gthiemonge | I updated the SG spec this morning | 16:05 |
| gthiemonge | mypy? | 16:05 |
| johnsom | It's not a spec, last time I looked it was waiting on a requirements repo dependency | 16:06 |
| gthiemonge | oh right | 16:06 |
| gthiemonge | anywys, for spec, Dalmatian-1 is not a blocking deadline, just an internal milestone | 16:07 |
| johnsom | Right, but it's great if we can land specs early in the cycle | 16:07 |
| tweining | only a friendly reminder that we wanted to integrate the typing patches at the beginning of the cycle | 16:07 |
| gthiemonge | +1 | 16:08 |
| gthiemonge | Any other announcements? | 16:10 |
| gthiemonge | our last meeting was 3 weeks ago, I probably missed a few things | 16:10 |
| tweining | I haven't even started with the rate limiting spec so far btw. | 16:11 |
| tweining | re mypy/typing the requirements dep is merged now | 16:12 |
| gthiemonge | ok | 16:13 |
| gthiemonge | #topic CI Status | 16:13 |
| gthiemonge | A few ongoing issues: | 16:13 |
| gthiemonge | pylint was/is broken on master and 2024.1 | 16:13 |
| gthiemonge | the fix on master merged earlier today (thanks tkajinam!) | 16:13 |
| gthiemonge | then I have a commit that pins pylint on 2024.1 | 16:13 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/919727 | 16:14 |
| gthiemonge | (already got W+1, should merge soon) | 16:14 |
| gthiemonge | then 2023.1 gates are also broken because of grenade | 16:14 |
| gthiemonge | IFAIK we have to disable grenade because zed is now unmaintained | 16:14 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/919572 | 16:14 |
| johnsom | +1 | 16:14 |
| gthiemonge | that's all for the CI | 16:15 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:15 |
| gthiemonge | as mentionned earlier, I have updated the spec for custom Security Groups on VIP ports | 16:16 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/915114 | 16:16 |
| gthiemonge | I received a lot of feedback on this spec (thanks!) one important point was to not limit the feature to one SG per VIP, so I changed it a little bit, now the spec allows multiple SGs per VIP | 16:16 |
| johnsom | I have been making progress on SRIOV for members. It now works in the normal create path. I have removed some old unneeded code with this change. I have also been testing nova/neutron failure cases to make sure we only roll back the member and not the LB, etc. | 16:17 |
| johnsom | I also found a nasty bug I introduced late in the SRIOV for VIPs chain and have created a patch: | 16:17 |
| gthiemonge | johnsom: nice! | 16:17 |
| johnsom | #link https://review.opendev.org/c/openstack/octavia/+/919546 | 16:17 |
| johnsom | Reviews would be appreciated as this needs to be backported to caracal and is a MAJOR bug for people trying out the SRIOV for VIPs code. | 16:18 |
| johnsom | It's bad enough I will probably propose a dot release of caracal once we land it | 16:19 |
| tweining | approved | 16:19 |
| gthiemonge | ack | 16:19 |
| johnsom | Also of note, I fired up a Ubuntu 24.04 VM and ran tox. It hits two issues, the TaskFlow issue we have a patch for, and the requirements checks. | 16:20 |
| gthiemonge | looking at the open backports for Caracal, I don't see any really urgent fixes | 16:20 |
| johnsom | One of the requirements check jobs is easy to fix, the other I have no idea what is wrong with it. | 16:20 |
| gthiemonge | does it use py312? | 16:21 |
| gthiemonge | you mention taskflow, so I assume it does | 16:22 |
| johnsom | Yes it does | 16:22 |
| gthiemonge | speaking about taskflow, Octavia is a big user of taskflow, and I have a few open patches there, I think it's good to ask reviews here too | 16:24 |
| gthiemonge | I have a fix for an issue that can happen when creating an A/S LB | 16:24 |
| gthiemonge | https://review.opendev.org/c/openstack/taskflow/+/900746 | 16:24 |
| johnsom | Also of note, I tested out the Grafana dashboard with the latest versions of everything. It loads up fine, but I found HAProxy made a breaking change to the Prometheus "status" metrics. I have already figured out a fix, I just need to take the time to implement it. | 16:24 |
| gthiemonge | (related to https://bugs.launchpad.net/octavia/+bug/2043360) | 16:25 |
| johnsom | Ok, yeah, I have taskflow reviews on my todo list | 16:25 |
| gthiemonge | and I've implemented an Etcd backedn for jobboard | 16:26 |
| gthiemonge | https://review.opendev.org/c/openstack/taskflow/+/915492 | 16:26 |
| johnsom | We are going to need to land that py3.12 patch soon I think | 16:26 |
| gthiemonge | don't know why it's V-1, I'll check that | 16:26 |
| gthiemonge | johnsom: I'll review it | 16:26 |
| gthiemonge | #topic Open Discussion | 16:29 |
| johnsom | I think I covered all of my topics | 16:31 |
| tweining | me too | 16:31 |
| gthiemonge | cool! | 16:32 |
| gthiemonge | then that's it for this week! | 16:32 |
| gthiemonge | thank you guys! | 16:32 |
| tweining | amen ;) | 16:32 |
| gthiemonge | #endmeeting | 16:32 |
| opendevmeet | Meeting ended Wed May 15 16:32:33 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-15-16.00.html | 16:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-15-16.00.txt | 16:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-15-16.00.log.html | 16:32 |
| opendevreview | Merged openstack/octavia stable/2024.1: Pin pylint on 2024.1 https://review.opendev.org/c/openstack/octavia/+/919727 | 17:32 |
| opendevreview | Merged openstack/octavia master: Fix listener update when using SRIOV VIP https://review.opendev.org/c/openstack/octavia/+/919546 | 18:08 |
| opendevreview | Michael Johnson proposed openstack/octavia stable/2024.1: Fix listener update when using SRIOV VIP https://review.opendev.org/c/openstack/octavia/+/919649 | 18:36 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Don't create load-balancer_* roles when using keystone default roles https://review.opendev.org/c/openstack/octavia/+/919770 | 20:05 |
| opendevreview | Michael Johnson proposed openstack/octavia master: Update the Grafana dashboard for HAProxy 2.4+ https://review.opendev.org/c/openstack/octavia/+/919776 | 20:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!