| opendevreview | Michael Johnson proposed openstack/octavia master: Add support for SR-IOV member ports https://review.opendev.org/c/openstack/octavia/+/920156 | 00:13 |
|---|---|---|
| opendevreview | Merged openstack/octavia-lib master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/octavia-lib/+/917645 | 00:31 |
| johnsom | I love gate runs where the tests passed just before the git review, but fail in the gates.... | 15:21 |
| tweining | it seems there are a few of those https://review.opendev.org/q/(projects:openstack/octavia+OR+project:openstack/python-octaviaclient)+status:open+label:Code-Review%253D%252B2%252Ccount%253E%253D2+-age:30d | 15:23 |
| johnsom | Well, it looks like the depends-on for the octavia-lib didn't work | 15:23 |
| johnsom | At least on mine | 15:23 |
| tweining | okay, at least one of the patches in the list seems to be dependent on https://review.opendev.org/c/openstack/octavia/+/919572 which removes the failing grenade job on stable/2023.1 | 15:26 |
| opendevreview | Takashi Kajinami proposed openstack/octavia-lib master: Use the common octavia queue https://review.opendev.org/c/openstack/octavia-lib/+/920189 | 15:28 |
| tkajinam | ^^^ johnsom this is probably what you want ? | 15:28 |
| johnsom | Hmm, interesting, thanks tkajinam | 15:30 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/2023.1: Fix example policy file system-reader role https://review.opendev.org/c/openstack/octavia/+/920135 | 15:39 |
| opendevreview | Merged openstack/octavia-lib master: Use the common octavia queue https://review.opendev.org/c/openstack/octavia-lib/+/920189 | 15:54 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed May 22 16:00:32 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 | Hey | 16:00 |
| johnsom | o/ | 16:00 |
| tweining | o/ | 16:00 |
| gthiemonge | #topic Announcements | 16:01 |
| gthiemonge | well.. no announcement from me this week | 16:02 |
| gthiemonge | do you have anything guys? | 16:02 |
| tweining | no | 16:02 |
| johnsom | nope | 16:03 |
| gthiemonge | #topic CI Status | 16:04 |
| gthiemonge | I heard that we're having some issues on the stable branches, right? | 16:04 |
| gthiemonge | or is it simply a question of rechecks? | 16:05 |
| tweining | 2023.1 is still blocked because of the failing grenade job, but you created https://review.opendev.org/c/openstack/octavia/+/919572 already | 16:05 |
| gthiemonge | ok I guess that johnsom can unblock it ^ | 16:05 |
| johnsom | Done | 16:06 |
| gthiemonge | thanks | 16:06 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:08 |
| tweining | https://bugs.launchpad.net/octavia/+bug/2065798 I worked on a fix for this new bug that prevents configuration of ciphers for TLS 1.3 in amphorae | 16:09 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/919846 this is the fix | 16:09 |
| gthiemonge | tweining: cool | 16:10 |
| gthiemonge | I worked on a change for the keystone_default_roles in octavia-tempest-plugin | 16:10 |
| johnsom | I pushed up the first draft of the SR-IOV for member ports. I still have work to do on that patch, so ignore it. | 16:10 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/919789 | 16:10 |
| johnsom | Lots of test work to do there. Next is the nftables patch. I am splitting them like I did for the VIP | 16:11 |
| gthiemonge | when enabling keystone_default_roles, tempest assigns the (default) load-balancer_* roles to the dynamic users, but these roles may not exist (because they are useless) in this RBAC mode | 16:11 |
| gthiemonge | the commit depends on https://review.opendev.org/c/openstack/octavia/+/919770 | 16:11 |
| gthiemonge | but the depends-on is only for testing, I'll remove it | 16:11 |
| johnsom | Yeah, I think the tripleo override relied on them, but that is history now | 16:12 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/920097 this patch makes "git blame" ignore the commits that only contain pyupgrade changes | 16:13 |
| gthiemonge | we can also set the member/admin/..._role to member/admin/reader in the settings, but IMHO it's redundant | 16:13 |
| tweining | I don't want to be blamed for code I did not write ;) | 16:14 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/900156 once this f-string conversion change is merged I will add that revision to this ignore list as well | 16:14 |
| tweining | because that change touches lots of code the risk for merge conflicts is high. For that reason, I suggest we integrate this one ASAP. It looks huge because it touches lots of files, but it isn't that large actually. | 16:15 |
| tweining | https://review.opendev.org/q/topic:%22mypy%22+repo:openstack/octavia thanks gthiemonge for reviewing my typing/mypy patches. I will not self-approve these patches because I did those modifications by hand. (the pyupgrade patches were generated) | 16:17 |
| johnsom | Yeah, conflict magnet, I will review this morning | 16:17 |
| tweining | thanks | 16:17 |
| gthiemonge | #topic Open Discussion | 16:20 |
| tweining | nothing else from me, I think | 16:22 |
| gthiemonge | ok! | 16:23 |
| gthiemonge | then, thank you folks! | 16:23 |
| gthiemonge | #endmeeting | 16:23 |
| opendevmeet | Meeting ended Wed May 22 16:23:20 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:23 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-22-16.00.html | 16:23 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-22-16.00.txt | 16:23 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-05-22-16.00.log.html | 16:23 |
| -opendevstatus- NOTICE: There will be a short Gerrit outage while we update to the latest 3.8 release in preparation for next weeks 3.9 upgrade. | 17:01 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Fix cipher configuration for TLSv1.3 https://review.opendev.org/c/openstack/octavia/+/919846 | 17:06 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Add pytest testenv to tox.ini https://review.opendev.org/c/openstack/octavia/+/881739 | 17:06 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Make tests work with pytest runner https://review.opendev.org/c/openstack/octavia/+/881805 | 17:06 |
| opendevreview | Merged openstack/octavia master: Fix test_prometheus_proxy.test_main https://review.opendev.org/c/openstack/octavia/+/917996 | 18:21 |
| opendevreview | Merged openstack/octavia stable/2023.1: Remove grenade jobs [stable/2023.1] https://review.opendev.org/c/openstack/octavia/+/919572 | 18:21 |
| opendevreview | Merged openstack/octavia stable/2024.1: Removing tips jobs on stable/2024.1 https://review.opendev.org/c/openstack/octavia/+/918046 | 20:15 |
| opendevreview | Merged openstack/octavia master: Fix incorrect filtering when using bool attrs https://review.opendev.org/c/openstack/octavia/+/903081 | 20:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!