Comment 21 for bug 2007919

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Thanks for the review Robie. I appreciate your thorough reviews and keeping Ubuntu stable!

(Note: The following reply is taken from my reply to a related thread: https://lists.ubuntu.com/archives/ubuntu-release/2023-September/005765.html)

For the behavioral change [1], I think there is minimal regression potential as the change is limited to an exception path where sleeps and reconnects were determined to be unnecessary. I'd have appreciated it if they would have provided a bug reference for the commit to provide more context, however with that said, it seems they are making a code improvement here for performance reasons and they found it to be useful enough to backport to stable branches.
[1] https://opendev.org/openstack/ovsdbapp/commit/ab3e0cb0d0865417efbf103f44954573a5ba92ac

It is certainly not ideal to have missing unit tests. It's important to consider that we perform testing above and beyond the unit tests that are run as part of our package builds. The regression testing that we have in place for OpenStack is run for all stable releases. For that we deploy a full OpenStack cloud and then run tempest integration tests that exercise all of the packages and the cloud solution as a whole.

I think we have a solid history of limiting regression potential for OpenStack users in our stable releases and that remains very important to us.