| *** tobias-urdin|pto is now known as tobias-urdin | 06:31 | |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 11:10 |
|---|---|---|
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 11:45 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Add spec for custom SGs on VIP ports https://review.opendev.org/c/openstack/octavia/+/915114 | 11:47 |
| opendevreview | Tom Weininger proposed openstack/octavia-tempest-plugin master: Add scenario tests for failover circuit breaker https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 | 12:28 |
| sorbal | Hello, I am looking at the get_update_listener_flow() (listener's TaskFlow) and more specifically at how ListenersUpdate is executed before UpdateListenerInDB but the first one uses the database state of the loadbalancer in order to update the listener config in the Amphorae | 16:49 |
| sorbal | How is it possible for ListenersUpdate to have the new updated listener if it only requires the loadbalancer id | 16:50 |
| sorbal | I am trying to find more information at the docs or commit history but I can't | 16:51 |
| sorbal | I am probably misunderstanding the purpose of UpdateListenerInDB | 16:58 |
| johnsom | Yeah, that is an interesting find. | 20:15 |
| johnsom | sorbal Orignally we didn't update the database until the amphora configuration was fully updated, i.e. don't reflect the change in the DB until it has actually changed. However, somewhere along the line we changed the API behavior to commit it to the DB when the driver accepts the update. | 20:16 |
| johnsom | So, UpdateListenerInDB may not be a necessary task anymore. | 20:16 |
| johnsom | The API commits it here: https://github.com/openstack/octavia/blob/master/octavia/api/v2/controllers/listener.py#L665 | 20:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!