| opendevreview | Alexander Shishebarov proposed openstack/octavia master: Add set error to LB on memever revert https://review.opendev.org/c/openstack/octavia/+/951956 | 09:44 |
|---|---|---|
| mchevreux | Hello 👋 | 10:01 |
| mchevreux | I've noticed a bug in octavia when creating a fully populated lb, if the listeners or the pool config is wrong, we end up with a leftover port in neutron that isn't even in the octavia.vip database table. After looking, it seems it was already referenced in https://bugs.launchpad.net/octavia/+bug/2015320 do you know if there is any patch planed for this ? The bug seems to be still present in the latest version looking at the code | 11:42 |
| mchevreux | To patch this we just need to move up the pool & listener creation by the _graph_create function in the post function of the loadbalancer controller but I'd like to know if this solution seems good for you before I prepare everything to contribute upstream | 11:44 |
| mchevreux | i.e move this if: https://github.com/openstack/octavia/blob/master/octavia/api/v2/controllers/load_balancer.py#L580 to line 528 | 11:48 |
| tobias-urdin | johnsom: iirc you said backporting would be fine for https://review.opendev.org/c/openstack/octavia/+/949503 in ptg – can you check if you still agree? thx :) | 11:54 |
| opendevreview | Alexander Shishebarov proposed openstack/octavia master: Add set error to LB on memever revert https://review.opendev.org/c/openstack/octavia/+/951956 | 12:10 |
| johnsom | FYI, it looks like some recent change (after June 4th) has broken the image Octavia uses. I don't see any cloud-init out put in the console and the VM doesn't get an IP. | 16:28 |
| rm_work | oof | 18:19 |
| rm_work | I'm trying to research implementing Envoy as an amp driver (upstream), looking for any thoughts on feasibility/interest, just starting to look so honestly it might be dumb but people are telling me they want it | 18:20 |
| johnsom | https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/2113797 | 21:32 |
| johnsom | cloud-init package is broken on Jammy, but seems fine on noble. If I rollback the cloud-init package it boots fine. | 21:32 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!