Tuesday, 2025-05-06

opendevreviewchandan kumar proposed openstack/watcher master: [DNM] epoxy openstack-check job  https://review.opendev.org/c/openstack/watcher/+/94883805:20
opendevreviewchandan kumar proposed openstack/watcher stable/2025.1: [DNM] epoxy openstack-check job  https://review.opendev.org/c/openstack/watcher/+/94883905:28
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration  https://review.opendev.org/c/openstack/watcher/+/94886511:29
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration  https://review.opendev.org/c/openstack/watcher/+/94886611:29
opendevreviewDouglas Viroel proposed openstack/watcher-specs master: Update 2025.2 spec directory and move spec implemented in 2025.1  https://review.opendev.org/c/openstack/watcher-specs/+/94727111:34
opendevreviewDouglas Viroel proposed openstack/watcher-specs master: Adds spec for extend compute model attributes  https://review.opendev.org/c/openstack/watcher-specs/+/94728211:34
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration  https://review.opendev.org/c/openstack/watcher/+/94886611:41
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration  https://review.opendev.org/c/openstack/watcher/+/94886511:46
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration  https://review.opendev.org/c/openstack/watcher/+/94886611:48
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration  https://review.opendev.org/c/openstack/watcher/+/94886511:53
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration  https://review.opendev.org/c/openstack/watcher/+/94886611:53
opendevreviewMerged openstack/watcher master: update pre-commit hook versions  https://review.opendev.org/c/openstack/watcher/+/94860512:02
opendevreviewMerged openstack/watcher-specs master: Update 2025.2 spec directory and move spec implemented in 2025.1  https://review.opendev.org/c/openstack/watcher-specs/+/94727112:17
chandankumarsean-k-mooney: Hello, I have opened a bug https://bugs.launchpad.net/watcher/+bug/2109945 regarding host maintenace strategy where it fails when migrating instance from disabled node with watcher_maintaining reason to other disabled with watcher maintaining.12:36
chandankumarI checked the code https://bugs.launchpad.net/watcher/+bug/2109945/comments/1 and found that12:37
chandankumarif destination node is disabled with watcher_disabled then migration works fine. I mean it enables the node and performs the migration.12:37
chandankumarcurrently there are two disable reasons in host_maintaince strategy: watcher_disabled and watcher_maintaining. 12:39
chandankumarwatcher_maintaining getting added when we start the maintenance but watcher_disabled might be added by other watcher strategy during cloud optimization12:40
chandankumarif I drop watcher_disabled reason then we can run host maintaince between watcher_maintaining node but it may affect deployment where user might be doing maintaince on multiple node.12:41
chandankumarI need your opinion on this, please take a look when free, thank you!12:42
opendevreviewMerged openstack/watcher master: drop suse supprot in the devstack plugin  https://review.opendev.org/c/openstack/watcher/+/94843912:54
opendevreviewMerged openstack/watcher master: drop jammy jobs  https://review.opendev.org/c/openstack/watcher/+/94844012:54
sean-k-mooneyim not sure the logic for selecting a host that is disabel when doign matiance ever makes sense13:04
sean-k-mooneyif a host is not proveide for the desithation im inclidned to say it shoudl be left to hte nova schduler to decied instead13:05
sean-k-mooneythat or we shoudl guard the existing behavior behind a new parmater to force_enableing the hosts13:05
sean-k-mooneychandankumar: my orginal comment on https://github.com/openstack/watcher/blob/master/watcher/decision_engine/strategy/strategies/host_maintenance.py#L236-L250 was i dont think enableing the node like that was the correct appproch to take13:07
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration  https://review.opendev.org/c/openstack/watcher/+/94886513:21
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration  https://review.opendev.org/c/openstack/watcher/+/94886613:21
chandankumarsean-k-mooney: thank you for the input. totally agree here with no touching the disabled hosts.14:16
chandankumarif there is no destination node, host maintaince works perfectly by relying on nova schedular to move instance.14:16
chandankumarFor now, I will update the bug and drop the maintaince node code.14:17
opendevreviewchandan kumar proposed openstack/watcher-tempest-plugin master: Add tests for host_maintenance strategy with backup node  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94794414:31
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for EfficacyIndicator value in mysql  https://review.opendev.org/c/openstack/watcher/+/94618215:07
opendevreviewJoan Gilabert proposed openstack/watcher master: Set number of decimal digits in efficacy indicator  https://review.opendev.org/c/openstack/watcher/+/94519915:07
opendevreviewJoan Gilabert proposed openstack/watcher master: Add test for EfficacyIndicator value in mysql  https://review.opendev.org/c/openstack/watcher/+/94618217:00
opendevreviewJoan Gilabert proposed openstack/watcher master: Set number of decimal digits in efficacy indicator  https://review.opendev.org/c/openstack/watcher/+/94519917:00

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