| opendevreview | Ho Minh Quang Ngo proposed openstack/watcher master: Add options to disable migration in host maintenance https://review.opendev.org/c/openstack/watcher/+/952538 | 00:36 |
|---|---|---|
| opendevreview | Ho Minh Quang Ngo proposed openstack/watcher-tempest-plugin master: Add tests for disable migration in host maintenance https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954214 | 08:36 |
| amoralej | @sean-k-mooney, @dviroel wrt implementation https://specs.openstack.org/openstack/watcher-specs/specs/2025.2/approved/add-skip-actions.html . While implementing it, i realized it may be also usefull to add a field status_message to the audit that could be only modified internally when a audit failed. i.e. | 09:37 |
| amoralej | that was not covered in the spec, but given that i'm adding a new microversion to cover the status_message of action and actionplan and follows the same implementation pattern, would it be fine to include it also for audit? | 09:38 |
| amoralej | i mean, in the same patch and in the same microversion | 09:40 |
| sean-k-mooney | we can, i tought we talked about adding it to the audit during the spec review, we defintly discusssed action and action plan but i tought auit also came up | 11:25 |
| sean-k-mooney | if its not in the spec then you shoudl submit a followup patch to amend it and we can review/merge that before the actual code change assuming there is no ojections. but yes i think that woudl be prefered, one microversion to add it to all the relevent apis | 11:26 |
| sean-k-mooney | the db change, action change and api change can be split inot multiple patches but we shoudl ideally minimes the number of microversion needed | 11:27 |
| amoralej | i agree | 11:29 |
| amoralej | i think we mentioned it may be useful, but never got it into the actual spec | 11:29 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: WIP - Add scenario tests with continuous audit https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954264 | 14:46 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: WIP - Extend decision engine to support threading mode https://review.opendev.org/c/openstack/watcher/+/952257 | 14:49 |
| *** haleyb|out is now known as haleyb | 14:58 | |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: WIP - Add scenario tests with continuous audit https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954264 | 17:52 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: WIP - Add scenario tests with continuous audit https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/954264 | 19:54 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!