16:00:32 <gthiemonge> #startmeeting Octavia
16:00:32 <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:32 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:32 <opendevmeet> The meeting name has been set to 'octavia'
16:00:37 <gthiemonge> Hey
16:00:48 <johnsom> o/
16:00:48 <tweining> o/
16:01:54 <gthiemonge> #topic Announcements
16:02:06 <gthiemonge> well.. no announcement from me this week
16:02:11 <gthiemonge> do you have anything guys?
16:02:28 <tweining> no
16:03:13 <johnsom> nope
16:04:03 <gthiemonge> #topic CI Status
16:04:24 <gthiemonge> I heard that we're having some issues on the stable branches, right?
16:05:11 <gthiemonge> or is it simply a question of rechecks?
16:05:19 <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:55 <gthiemonge> ok I guess that johnsom can unblock it ^
16:06:44 <johnsom> Done
16:06:50 <gthiemonge> thanks
16:08:50 <gthiemonge> #topic Brief progress reports / bugs needing review
16:09:27 <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:55 <tweining> https://review.opendev.org/c/openstack/octavia/+/919846 this is the fix
16:10:21 <gthiemonge> tweining: cool
16:10:37 <gthiemonge> I worked on a change for the keystone_default_roles in octavia-tempest-plugin
16:10:41 <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:44 <gthiemonge> https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/919789
16:11:07 <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:10 <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:19 <gthiemonge> the commit depends on https://review.opendev.org/c/openstack/octavia/+/919770
16:11:27 <gthiemonge> but the depends-on is only for testing, I'll remove it
16:12:22 <johnsom> Yeah, I think the tripleo override relied on them, but that is history now
16:13:53 <tweining> https://review.opendev.org/c/openstack/octavia/+/920097 this patch makes "git blame" ignore the commits that only contain pyupgrade changes
16:13:54 <gthiemonge> we can also set the member/admin/..._role to member/admin/reader in the settings, but IMHO it's redundant
16:14:36 <tweining> I don't want to be blamed for code I did not write ;)
16:14:58 <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:15:53 <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:17:29 <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:36 <johnsom> Yeah, conflict magnet, I will review this morning
16:17:53 <tweining> thanks
16:20:57 <gthiemonge> #topic Open Discussion
16:22:10 <tweining> nothing else from me, I think
16:23:06 <gthiemonge> ok!
16:23:16 <gthiemonge> then, thank you folks!
16:23:20 <gthiemonge> #endmeeting