Tuesday, 2025-10-21

opendevreviewJoan Gilabert proposed openstack/watcher master: Fix check for volume type in cinder helper migrate  https://review.opendev.org/c/openstack/watcher/+/96385708:03
opendevreviewJoan Gilabert proposed openstack/watcher-tempest-plugin master: Add test for volume migrate with zone migration  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95864408:37
opendevreviewJoan Gilabert proposed openstack/watcher-tempest-plugin master: Test zone migration volume and compute migrations  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/96270208:37
opendevreviewJoan Gilabert proposed openstack/watcher-tempest-plugin master: Add extra checks to zone migration retype test  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/96355908:37
opendevreviewJoan Gilabert proposed openstack/watcher-tempest-plugin master: Relax condition on cinder backends for migrations  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/96386008:37
opendevreviewJoan Gilabert proposed openstack/watcher master: Disable cinder volume service in subnode  https://review.opendev.org/c/openstack/watcher/+/96446213:10
jgilaberdviroel thanks for the review and links on the volume migration tests patch. I've opened ^ to change the job config to have only one c-vol service in the controller, with two backends13:10
dviroeljgilaber: nice, lets see how it goes with this config13:12
opendevreviewJoan Gilabert proposed openstack/watcher master: Fix check for volume type in cinder helper migrate  https://review.opendev.org/c/openstack/watcher/+/96385713:33
opendevreviewJoan Gilabert proposed openstack/watcher master: Disable cinder volume service in subnode  https://review.opendev.org/c/openstack/watcher/+/96446216:24
opendevreviewJoan Gilabert proposed openstack/watcher master: Disable cinder volume service in subnode  https://review.opendev.org/c/openstack/watcher/+/96446216:31
sean-k-mooneydviroel: if you have time today/tomorrwo can we proceed with https://review.opendev.org/c/openstack/watcher/+/961453/418:17
sean-k-mooneythat is the final ruff check patch that is likely to conflict with things18:17
sean-k-mooneyat least on the watcher repo18:17
sean-k-mooneythe next 3 are the dead code removal for the client but those shoudl not really conflict with anythign or would be trivila to fix18:18
sean-k-mooneyhttps://review.opendev.org/c/openstack/watcher-dashboard/+/963378/2 and the patch below it would also be nice but those  can wait till after the ptg without much issue18:21
dviroelsean-k-mooney: sure, i have to get back to those, i will do that until tomorrow then18:33
opendevreviewMerged openstack/watcher master: Fix ruff check violations  https://review.opendev.org/c/openstack/watcher/+/96145319:26
opendevreviewMerged openstack/watcher master: Remove dead neutron client code  https://review.opendev.org/c/openstack/watcher/+/96299019:40

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!