| skraynev | @gthiemonge : hi, I have updated PR: https://review.opendev.org/c/openstack/octavia/+/943985 with release note, could you please review it again it? | 05:12 |
|---|---|---|
| gthiemonge | skraynev: done | 08:36 |
| skraynev | @gthiemonge thank you | 08:36 |
| tkajinam | gthiemonge, do you have a min to merge this ? https://review.opendev.org/c/openstack/octavia-dashboard/+/944705 | 09:58 |
| gthiemonge | tkajinam: done, approved | 11:40 |
| tkajinam | gthiemonge, thx ! | 11:45 |
| opendevreview | Merged openstack/octavia-dashboard master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/octavia-dashboard/+/944705 | 11:51 |
| opendevreview | Arkady Shtempler proposed openstack/octavia-tempest-plugin master: Add LB test coverage for listener without pool https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/755281 | 13:28 |
| opendevreview | Takashi Kajinami proposed openstack/octavia-tempest-plugin master: Replace UPPER_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/947491 | 13:35 |
| johnsom | #startmeeting Octavia | 16:13 |
| opendevmeet | Meeting started Wed Apr 16 16:13:53 2025 UTC and is due to finish in 60 minutes. The chair is johnsom. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:13 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:13 |
| opendevmeet | The meeting name has been set to 'octavia' | 16:13 |
| johnsom | Sorry we are starting late | 16:13 |
| gthiemonge | \o/ | 16:14 |
| johnsom | #topic Announcements | 16:14 |
| johnsom | Welcome to the Flamingo cycle and thank you for a successful PTG! | 16:15 |
| gthiemonge | indeed, that was a good PTG! | 16:15 |
| johnsom | I don't have any announcement today. Anyone else? | 16:16 |
| gthiemonge | nop | 16:16 |
| johnsom | #topic CI Status | 16:16 |
| johnsom | I have been spending a lot of time on this recently. The stable branches all had issues (mostly due to the RBAC switch). | 16:17 |
| johnsom | I have worked through 2023.2 and 2024.1 and requested releases for those. I am still working on 2024.2. | 16:17 |
| johnsom | The 2023.2 release is quite large, which is not ideal, but it will get us caught up for when the unmaintained tag is applied | 16:18 |
| johnsom | Otherwise I think the CI is doing ok. | 16:18 |
| johnsom | #topic Brief progress reports / bugs needing review | 16:19 |
| johnsom | See above for my progress report.... | 16:19 |
| gthiemonge | regarding the CI tkajinam has proposed 2 patches to update the octavia-tempest-plugin config: | 16:19 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/947110 | 16:20 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/947491 | 16:20 |
| johnsom | Yes, he beat me to it on the new job. I have been working my way to that branch. | 16:20 |
| gthiemonge | I've investigated this launchpad: https://bugs.launchpad.net/octavia/+bug/2097532 (it was discussed during the PTG) | 16:21 |
| johnsom | Hmmm, on that second one, there seems to be an existing open patch: https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/844733?usp=search | 16:22 |
| johnsom | I will check those two and mark one as duplicate as appropriate. | 16:23 |
| gthiemonge | so, i think the title is not accurate, the problem is that if some rules prevent the user to create a port (with or without specifying an IP address), Octavia bypasses them, it only checks that the user has access to the network and then creates the port with the admin role | 16:23 |
| johnsom | I am glad you are taking a look at that. I have been looking at the duplicate IP error message bug that was also raised at the PTG | 16:24 |
| gthiemonge | for instance in devstack, the demo user cannot create a port with an ip address on the shared network, but the user can create a LB on this network with a vip_address | 16:24 |
| johnsom | Basically when we added provider driver support we ended up trashing the exception raised from neutron. | 16:25 |
| johnsom | That is interesting, since the AAP port is owned by the project ID of the user, I would have expected neutron to reject it. | 16:26 |
| gthiemonge | yeah the rules apply to the user and not the tenant | 16:26 |
| gthiemonge | I'll add more detail to the launchpad | 16:27 |
| johnsom | Ok | 16:27 |
| gthiemonge | so far, i don't have a solution | 16:27 |
| johnsom | Maybe we need to add an additional RBAC check to neutron | 16:27 |
| johnsom | I will read through your comments on the launchpad. The initial description was a bit confusing. | 16:29 |
| johnsom | Any other progress updates? | 16:30 |
| gthiemonge | nop, that's all i have | 16:30 |
| johnsom | #topic Open Discussion | 16:30 |
| johnsom | Great, any other topics this week? | 16:30 |
| johnsom | Ok then, thanks for joining! | 16:31 |
| johnsom | #endmeeting | 16:32 |
| opendevmeet | Meeting ended Wed Apr 16 16:32:02 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-04-16-16.13.html | 16:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-04-16-16.13.txt | 16:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2025/octavia.2025-04-16-16.13.log.html | 16:32 |
| gthiemonge | thanks johnsom | 16:32 |
| opendevreview | Arkady Shtempler proposed openstack/octavia-tempest-plugin master: Add LB test coverage for listener without pool https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/755281 | 17:30 |
| opendevreview | Merged openstack/octavia master: Ignore load_balancer graph in task results to avoid break write data in jobboard DB https://review.opendev.org/c/openstack/octavia/+/943985 | 17:49 |
| opendevreview | Merged openstack/octavia-tempest-plugin master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/844733 | 18:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!