| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Return HTTP code 400 when creating an audit with wrong parameters https://review.opendev.org/c/openstack/watcher/+/949703 | 07:36 |
|---|---|---|
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Set actionplan state to FAILED if any action has failed https://review.opendev.org/c/openstack/watcher/+/949225 | 08:05 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add unit test to check action plan state when a nested action fails https://review.opendev.org/c/openstack/watcher/+/949967 | 08:05 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Set actionplan state to FAILED if any action has failed https://review.opendev.org/c/openstack/watcher/+/949225 | 08:17 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Added unit test to validate audit creation with no goal and no name https://review.opendev.org/c/openstack/watcher/+/949976 | 09:15 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Fix audit creation with no name and no goal or audit_template https://review.opendev.org/c/openstack/watcher/+/949977 | 09:15 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Fix audit creation with no name and no goal or audit_template https://review.opendev.org/c/openstack/watcher/+/949977 | 10:43 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst_node parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:49 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: [WIP] Handle missing dst_pool parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/950149 | 11:49 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing dst_node parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/948866 | 11:52 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: [WIP] Handle missing dst_pool parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/950149 | 11:52 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Fix audit creation with no name and no goal or audit_template https://review.opendev.org/c/openstack/watcher/+/949977 | 12:17 |
| opendevreview | chandan kumar proposed openstack/watcher-tempest-plugin master: Use get_host_for_server to get server host https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/949557 | 12:26 |
| amoralej | sean-k-mooney, wrt release notes https://review.opendev.org/c/openstack/watcher/+/949225/4/releasenotes/notes/fix-action-plan-state-on-failure-69e498d902ada5c5.yaml is it fine to keep the link to the bug in the text for informational purposes? | 14:15 |
| amoralej | or should i remove it? i saw some references to adding it in some contributor guides | 14:16 |
| sean-k-mooney | amoralej: yes but it shoudl not be the first thing in the note ideally | 14:25 |
| sean-k-mooney | it shoudl be somethign like this https://review.opendev.org/c/openstack/watcher/+/948866/9/releasenotes/notes/zone-migration-missing-dst-node-bd0377af1f1ed245.yaml | 14:25 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/949225/4/releasenotes/notes/fix-action-plan-state-on-failure-69e498d902ada5c5.yaml is very short and it does not really expalin to an admin why they shoudl upgrade to this or backprot this to there install | 14:26 |
| sean-k-mooney | you have explaind it better in the commit https://review.opendev.org/c/openstack/watcher/+/949225/4//COMMIT_MSG although that longer then a typic release note shoudl be | 14:27 |
| opendevreview | Joan Gilabert proposed openstack/watcher stable/2025.1: Add test for EfficacyIndicator value in mysql https://review.opendev.org/c/openstack/watcher/+/950189 | 14:28 |
| amoralej | ack, i can rephrase it | 14:28 |
| opendevreview | Joan Gilabert proposed openstack/watcher stable/2025.1: Set number of decimal digits in efficacy indicator https://review.opendev.org/c/openstack/watcher/+/950190 | 14:29 |
| sean-k-mooney | amoralej: i have provdied an example in the comment although i apprently missed a new line or two | 14:32 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/949225/4/releasenotes/notes/fix-action-plan-state-on-failure-69e498d902ada5c5.yaml | 14:32 |
| amoralej | Thanks | 14:33 |
| opendevreview | Joan Gilabert proposed openstack/watcher stable/2025.1: Set number of decimal digits in efficacy indicator https://review.opendev.org/c/openstack/watcher/+/950190 | 14:36 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Set actionplan state to FAILED if any action has failed https://review.opendev.org/c/openstack/watcher/+/949225 | 14:38 |
| opendevreview | Merged openstack/watcher-tempest-plugin master: Add tests for host_maintenance strategy with backup node https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/947944 | 15:18 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Set actionplan state to FAILED if any action has failed https://review.opendev.org/c/openstack/watcher/+/949225 | 16:49 |
| opendevreview | Merged openstack/watcher master: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/948865 | 17:10 |
| opendevreview | Merged openstack/watcher master: Migrate value column of efficacy indicator on load https://review.opendev.org/c/openstack/watcher/+/949640 | 18:16 |
| opendevreview | Merged openstack/watcher master: Drop code from Host maintenance strategy migrating instance to disabled hosts https://review.opendev.org/c/openstack/watcher/+/948967 | 18:18 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!