opendevreview | chandan kumar proposed openstack/watcher master: [DNM] epoxy openstack-check job https://review.opendev.org/c/openstack/watcher/+/948838 | 05:20 |
---|---|---|
opendevreview | chandan kumar proposed openstack/watcher stable/2025.1: [DNM] epoxy openstack-check job https://review.opendev.org/c/openstack/watcher/+/948839 | 05:28 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/948865 | 11:29 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:29 |
opendevreview | Douglas 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/+/947271 | 11:34 |
opendevreview | Douglas Viroel proposed openstack/watcher-specs master: Adds spec for extend compute model attributes https://review.opendev.org/c/openstack/watcher-specs/+/947282 | 11:34 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:41 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/948865 | 11:46 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:48 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/948865 | 11:53 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:53 |
opendevreview | Merged openstack/watcher master: update pre-commit hook versions https://review.opendev.org/c/openstack/watcher/+/948605 | 12:02 |
opendevreview | Merged openstack/watcher-specs master: Update 2025.2 spec directory and move spec implemented in 2025.1 https://review.opendev.org/c/openstack/watcher-specs/+/947271 | 12:17 |
chandankumar | sean-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 |
chandankumar | I checked the code https://bugs.launchpad.net/watcher/+bug/2109945/comments/1 and found that | 12:37 |
chandankumar | if destination node is disabled with watcher_disabled then migration works fine. I mean it enables the node and performs the migration. | 12:37 |
chandankumar | currently there are two disable reasons in host_maintaince strategy: watcher_disabled and watcher_maintaining. | 12:39 |
chandankumar | watcher_maintaining getting added when we start the maintenance but watcher_disabled might be added by other watcher strategy during cloud optimization | 12:40 |
chandankumar | if 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 |
chandankumar | I need your opinion on this, please take a look when free, thank you! | 12:42 |
opendevreview | Merged openstack/watcher master: drop suse supprot in the devstack plugin https://review.opendev.org/c/openstack/watcher/+/948439 | 12:54 |
opendevreview | Merged openstack/watcher master: drop jammy jobs https://review.opendev.org/c/openstack/watcher/+/948440 | 12:54 |
sean-k-mooney | im not sure the logic for selecting a host that is disabel when doign matiance ever makes sense | 13:04 |
sean-k-mooney | if a host is not proveide for the desithation im inclidned to say it shoudl be left to hte nova schduler to decied instead | 13:05 |
sean-k-mooney | that or we shoudl guard the existing behavior behind a new parmater to force_enableing the hosts | 13:05 |
sean-k-mooney | chandankumar: 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 take | 13:07 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/948865 | 13:21 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst parameters in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 13:21 |
chandankumar | sean-k-mooney: thank you for the input. totally agree here with no touching the disabled hosts. | 14:16 |
chandankumar | if there is no destination node, host maintaince works perfectly by relying on nova schedular to move instance. | 14:16 |
chandankumar | For now, I will update the bug and drop the maintaince node code. | 14:17 |
opendevreview | chandan 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/+/947944 | 14:31 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for EfficacyIndicator value in mysql https://review.opendev.org/c/openstack/watcher/+/946182 | 15:07 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Set number of decimal digits in efficacy indicator https://review.opendev.org/c/openstack/watcher/+/945199 | 15:07 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Add test for EfficacyIndicator value in mysql https://review.opendev.org/c/openstack/watcher/+/946182 | 17:00 |
opendevreview | Joan Gilabert proposed openstack/watcher master: Set number of decimal digits in efficacy indicator https://review.opendev.org/c/openstack/watcher/+/945199 | 17:00 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!