| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Fix OpenStackSDK connection memory leak https://review.opendev.org/c/openstack/watcher/+/1014459 | 09:04 |
|---|---|---|
| dviroel | amoralej__: sorry for going back and forth on the pin/repin proposal in the zone drain spec, LOL, but it is still an open discussion | 12:31 |
| amoralej__ | no problem | 12:39 |
| amoralej__ | it's a good discussion, there are pros and cons on both options | 12:41 |
| amoralej__ | dviroel, wrt "skipping in the Action will have another Con: a continuous audit will always proposed again the Skipped instance for new APs, even if previous APs where executed." I don't get that | 12:46 |
| amoralej__ | i just did, sorry :) | 12:46 |
| dviroel | yeah | 12:46 |
| amoralej__ | yep, that's correct | 12:47 |
| amoralej__ | actually, using skip_az_pinned_instances= true, even after all the non-pinned instances have been migrated, the audit would keep creating non-empty APs | 12:48 |
| dviroel | in the second proposal, right? | 12:51 |
| amoralej__ | yes, with the skip approach | 12:51 |
| amoralej__ | second is the skip in action, right? that one | 12:52 |
| dviroel | yes | 12:52 |
| dviroel | the skip in strategy we are calling 'skip_az_pinned_instances', while the skip in the Action i named "repin_az_pinned_instances". | 12:53 |
| dviroel | but yeah, you got it | 12:53 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: zone_migration: support unlimited as parallel parameters https://review.opendev.org/c/openstack/watcher/+/1012707 | 13:53 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Add availability zone to the compute data model https://review.opendev.org/c/openstack/watcher/+/1014524 | 13:53 |
| amoralej__ | something has broken all the deployment jobs | 16:22 |
| amoralej__ | :( | 16:23 |
| sean-k-mooney | if you mean the operator jobs that shoudl be fixed again i think | 17:09 |
| sean-k-mooney | there was a patch merged without its dep patch mergign first but that was correct 20-30 mins later | 17:10 |
| sean-k-mooney | amoralej__: ^ | 17:10 |
| sean-k-mooney | if you mean the devestack jobs | 17:11 |
| sean-k-mooney | i can take a look | 17:11 |
| sean-k-mooney | but we dont refer to those as deployment jobs | 17:12 |
| sean-k-mooney | all run-test: commands[1] | tempest run --regex 'watcher_tempest_plugin.tests\.(api|scenario)' --concurrency=1 '--exclude-regex=.*\[.*\breal_load\b.*\].*' | 17:14 |
| sean-k-mooney | 2026-11-06 14:25:36.761270 | controller | The specified regex doesn't match with anythingERROR: InvocationError for command /opt/stack/tempest/.tox/tempest/bin/tempest run --regex 'watcher_tempest_plugin.tests\.(api|scenario)' --concu | 17:14 |
| sean-k-mooney | same thing in cinder tempest plugins lets bring this to the qa channel | 17:16 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Version the parameters spec of strategies https://review.opendev.org/c/openstack/watcher/+/1014569 | 17:33 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add update_parameters() to BaseStrategy https://review.opendev.org/c/openstack/watcher/+/1014570 | 17:33 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Update goals and strategies in-place in the Syncer https://review.opendev.org/c/openstack/watcher/+/1014571 | 17:33 |
| gmaan | amoralej__: sean-k-mooney it was temp outage and should be green now, you can recheck the failing jobs | 17:40 |
| amoralej__ | good, thanks gmaan++ | 17:40 |
| amoralej__ | sean-k-mooney, i meant the devstack jobs | 17:40 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Add availability zone to the compute data model https://review.opendev.org/c/openstack/watcher/+/1014524 | 20:03 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!