)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"37a6ceb3196d47638e493c9d9261e89997946463","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"feat(spec): disable cold/live migration for host maintenance strategy"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Spec to add disable cold and live migraiton parameters to host maintenance strategy"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic0291509f66b5a7bed016f0670206a39f27445ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a3a8086_b51b3407","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":83},"updated":"2025-04-09 18:38:30.000000000","message":"It is not only allowing the disable of them, but including a new action to stop the instance too.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"ac80cb7040d01e7130fe9d6e3ac2e1a58223148f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"feat(spec): disable cold/live migration for host maintenance strategy"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Spec to add disable cold and live migraiton parameters to host maintenance strategy"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic0291509f66b5a7bed016f0670206a39f27445ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7794d51b_db851804","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":83},"in_reply_to":"1a3a8086_b51b3407","updated":"2025-04-11 04:01:53.000000000","message":"Acknowledged","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"cd3729566184a2aea6986d185e11a3deeeaf18a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fbe4581c_e06e3039","updated":"2025-06-05 16:46:06.000000000","message":"Approving this spec since there is no more questions about the design, and further details can be discussed in the code proposed.\nThanks for proposing it.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"aa0b1decd1ec62ed49999c6153192a75a437fe1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4b69dbe7_864af6df","updated":"2025-04-25 02:14:10.000000000","message":"Hi Sean,\n\nCan I get some feedback from you to know if there any blocker for us to approved this spec?","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"2af4fbdb517ad75754f49ee6101483f99315ec04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"24533a76_1b57a15b","updated":"2025-06-05 19:11:05.000000000","message":"I missed that detail, you need to propose this spec for 2025.2 release.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a909bbbd75ad97114ac05a9bcbc45bafb40957ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"60cc2642_6f14b6f0","updated":"2025-05-21 21:04:12.000000000","message":"going to check that again, but looks good","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"3aa833fd235a1cb1660627d101f7db82ab42bdc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"28a24bd9_bf40e758","updated":"2025-05-12 09:30:24.000000000","message":"lgtm","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"51c7c0dddd6c1a2a62e938081734198688d246b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8d4315ca_5efc6e81","updated":"2025-05-27 07:59:21.000000000","message":"lgtm, it should allow users better control on the strategy","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"99490778e49c27637bc1b1e992ec4659c3e09431","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"30743ce3_08e96cda","updated":"2025-06-06 16:41:43.000000000","message":"oh ya... i missed that too\nif this is not updated by monday by  @james.lin@canonical.com\nwe can just fix that for them\n\neither as a follow up or by updatign the patch for them an reapproving","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a723ea0a63cc8c0abb96751490ca2fff58fa1e47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ec82a1f0_90edc4d9","updated":"2025-05-22 09:34:32.000000000","message":"overall, it will be a nice improvement to host maintenance strategy.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"deb2fc27f6ecebb101933cd03544ec9b412ad433","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b524961a_c36b91f1","updated":"2025-06-03 15:35:50.000000000","message":"while this spec is shorter then i would normally expect\nthis is also not a very complicate change so i think we can proceed to the implementation review.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"e53e35869c528a2726be2fb4e21170103591f482","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8435a7de_36bce393","in_reply_to":"b524961a_c36b91f1","updated":"2025-06-09 01:18:49.000000000","message":"Acknowledged","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"e53e35869c528a2726be2fb4e21170103591f482","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"993b8fae_8ca59e70","updated":"2025-06-09 01:18:49.000000000","message":"Move the spec from 2025.1/implemented to 2025.2/approved.","commit_id":"68eab6d25f5c14e7ab9d89418b4d3e74228b3adf"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"06ed8908b60d2205f371ffa4ad2f716a89efde37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3c3444d5_51744512","updated":"2025-06-09 11:14:53.000000000","message":"Thanks for the update, lgtm","commit_id":"68eab6d25f5c14e7ab9d89418b4d3e74228b3adf"}],"specs/2025.1/approved/host-maintenance-strategy-disable-migration.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1f054ad3c74ec0e0e5ad5734eba484cb480d8521","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"----------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- As a Cloud Administrator, if live migration is not supported in my OpenStack deployment, I want to stop instance and apply cold migration to instance."},{"line_number":26,"context_line":"- As a Cloud Administrator, if cold migration is not supported in my OpenStack deployment, I want to skip the migration."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"516f15cf_c535c1d6","line":25,"range":{"start_line":25,"start_character":91,"end_line":25,"end_character":152},"updated":"2025-03-10 11:01:53.000000000","message":"why do you phase it this way\n\ndo you want to do two actions, stop then cold migrate or do  you just want to cold migrate.\n\nthe difference is by default nova will resume the previous state when you cold migrate.  so if the instance was runnign before it will started when it enters resize verify.\n\nwhere as if you stop the instance first it will be migrated to off and wether it was runing or not will be lost.\n\nin general it is not recommened to stop instnace before cold migrating them to preserve the orginal state.\n\nso i think that we would want a seperate parmater to opt into stoping the instnace.","commit_id":"9709edd27a48e36b70b2985c89fc0dd6eac76420"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"cd3729566184a2aea6986d185e11a3deeeaf18a3","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"----------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- As a Cloud Administrator, if live migration is not supported in my OpenStack deployment, I want to stop instance and apply cold migration to instance."},{"line_number":26,"context_line":"- As a Cloud Administrator, if cold migration is not supported in my OpenStack deployment, I want to skip the migration."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"b71a3070_f299cbf2","line":25,"range":{"start_line":25,"start_character":91,"end_line":25,"end_character":152},"in_reply_to":"115f94cb_4c15df38","updated":"2025-06-05 16:46:06.000000000","message":"Done","commit_id":"9709edd27a48e36b70b2985c89fc0dd6eac76420"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"43181629d9624ba462b0f3af1479efb702ae06ae","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"----------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- As a Cloud Administrator, if live migration is not supported in my OpenStack deployment, I want to stop instance and apply cold migration to instance."},{"line_number":26,"context_line":"- As a Cloud Administrator, if cold migration is not supported in my OpenStack deployment, I want to skip the migration."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"e71ae80d_824e85d8","line":25,"range":{"start_line":25,"start_character":91,"end_line":25,"end_character":152},"in_reply_to":"516f15cf_c535c1d6","updated":"2025-03-11 06:44:59.000000000","message":"So here is my approach:\n\n- live migration is available -\u003e do live migration.\n- live migration is not available, but cold migration is available -\u003e do cold migration.\n- Both migration not available -\u003e stop the instance.","commit_id":"9709edd27a48e36b70b2985c89fc0dd6eac76420"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"ac48a733ddd93f5c5e5cbdb2a7ded538ca31acaa","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"----------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- As a Cloud Administrator, if live migration is not supported in my OpenStack deployment, I want to stop instance and apply cold migration to instance."},{"line_number":26,"context_line":"- As a Cloud Administrator, if cold migration is not supported in my OpenStack deployment, I want to skip the migration."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"115f94cb_4c15df38","line":25,"range":{"start_line":25,"start_character":91,"end_line":25,"end_character":152},"in_reply_to":"80d9ac65_86d3386e","updated":"2025-03-14 05:39:32.000000000","message":"Updated on patch 3","commit_id":"9709edd27a48e36b70b2985c89fc0dd6eac76420"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1a25e67416c582d88f08fd17aa1f4aca675dcc0a","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Use Cases"},{"line_number":23,"context_line":"----------"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- As a Cloud Administrator, if live migration is not supported in my OpenStack deployment, I want to stop instance and apply cold migration to instance."},{"line_number":26,"context_line":"- As a Cloud Administrator, if cold migration is not supported in my OpenStack deployment, I want to skip the migration."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"80d9ac65_86d3386e","line":25,"range":{"start_line":25,"start_character":91,"end_line":25,"end_character":152},"in_reply_to":"e71ae80d_824e85d8","updated":"2025-03-11 09:10:19.000000000","message":"i think that precednce chain works and logically makes sense\n\ncan you update the spec to record that.","commit_id":"9709edd27a48e36b70b2985c89fc0dd6eac76420"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"d4681722cd9f68766a75962697f3d1667fc652af","unresolved":true,"context_lines":[{"line_number":32,"context_line":"  I want to apply cold migration to the instance."},{"line_number":33,"context_line":"- As a Cloud Administrator, if both live migration cold migration are"},{"line_number":34,"context_line":"  not supported in my OpenStack deployment."},{"line_number":35,"context_line":"  I want to stop the instance."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"457c08eb_ec0afd32","line":35,"updated":"2025-04-08 08:34:48.000000000","message":"What\u0027s the expected behavior after the maintenance is done and the node restarted? VMs will need to be manually started or should it be managed it automatically somehow?","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"ac80cb7040d01e7130fe9d6e3ac2e1a58223148f","unresolved":true,"context_lines":[{"line_number":32,"context_line":"  I want to apply cold migration to the instance."},{"line_number":33,"context_line":"- As a Cloud Administrator, if both live migration cold migration are"},{"line_number":34,"context_line":"  not supported in my OpenStack deployment."},{"line_number":35,"context_line":"  I want to stop the instance."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"64519f2f_5e132725","line":35,"in_reply_to":"3dc5d0cc_cde48412","updated":"2025-04-11 04:01:53.000000000","message":"\u003e What\u0027s the expected behavior after the maintenance is done and the node restarted? VMs will need to be manually started or should it be managed it automatically somehow?\n\nI believe this should be a new strategy to bring all the instances back \u0026 enable the nova-compute service.\n\nGenerate two action plans from the same strategy may not be the best option here because the effort to refactor the source code and also the generated action plan may not fit into the latest status of the node.\n\n\u003e which may be the only option for some vms anyway.\n\nIf we would like to provide this recovery strategy, I guess this will be the topic there.\n\nOverall I think the recovery is a follow up feature but not required for this change?","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6679e820a0584ceb15d89e3396364fce9a64f42a","unresolved":true,"context_lines":[{"line_number":32,"context_line":"  I want to apply cold migration to the instance."},{"line_number":33,"context_line":"- As a Cloud Administrator, if both live migration cold migration are"},{"line_number":34,"context_line":"  not supported in my OpenStack deployment."},{"line_number":35,"context_line":"  I want to stop the instance."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3dc5d0cc_cde48412","line":35,"in_reply_to":"457c08eb_ec0afd32","updated":"2025-04-08 12:27:43.000000000","message":"as spcified there i woudl assume that they vms will remain stopped and the user will have to start them again.\n\nwhich may be the only option for some vms anyway.\n\nSpecifically any vm with vtpm or encrypted volumes cannot necessarily be start by a admin after a reboot.\n\nby default, admins cannot read keys in Barbican so an admin cannot start a vm that needs an encryption key stored in Barbican, such as an instance with a vtpm or cinder-encrypted volumes.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"deb2fc27f6ecebb101933cd03544ec9b412ad433","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  I want to apply cold migration to the instance."},{"line_number":33,"context_line":"- As a Cloud Administrator, if both live migration cold migration are"},{"line_number":34,"context_line":"  not supported in my OpenStack deployment."},{"line_number":35,"context_line":"  I want to stop the instance."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Proposed change"},{"line_number":38,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"18fc4331_6c307960","line":35,"in_reply_to":"64519f2f_5e132725","updated":"2025-06-03 15:35:50.000000000","message":"i think for now we can put the post matiance activiy out of scope.\n\nwhen matiantance is doen today you could use the workload stablitation stargey to redistibute the instnaces.\n\nin the case fo stop instances it  proably would make sense to leave that to the oparotr to ask there users to restore once mantaince si complete.\n\nif the instnace use volume encryption for example or vtpm\nadmins and by extention watcher cant start the instnaces so i dont think we need to automate this or cover those details in this spec.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"37a6ceb3196d47638e493c9d9261e89997946463","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Estimated changes are going to be in the following places:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"* Host maintenance strategy"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"  * Input parameters **disable_cold_migration** and **disable_live_migration**"},{"line_number":45,"context_line":"    to disable the migration."}],"source_content_type":"text/x-rst","patch_set":3,"id":"b94bbc8d_bc3e7297","line":42,"updated":"2025-04-09 18:38:30.000000000","message":"IIUC, if the user disables both cold and live migrations, then it is going to create \"stop instance\" actions only. Is that it? There will be no mix between migrations and instances stopped?\nI think we need to detail more how \"instance_migration\" method is going to behave from now.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a909bbbd75ad97114ac05a9bcbc45bafb40957ef","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Estimated changes are going to be in the following places:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"* Host maintenance strategy"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"  * Input parameters **disable_cold_migration** and **disable_live_migration**"},{"line_number":45,"context_line":"    to disable the migration."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5cc97db0_15f3f21c","line":42,"in_reply_to":"03bb1faa_bdab4f54","updated":"2025-05-21 21:04:12.000000000","message":"Ok, sounds reasonable for me.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"aa0b1decd1ec62ed49999c6153192a75a437fe1c","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Estimated changes are going to be in the following places:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"* Host maintenance strategy"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"  * Input parameters **disable_cold_migration** and **disable_live_migration**"},{"line_number":45,"context_line":"    to disable the migration."}],"source_content_type":"text/x-rst","patch_set":3,"id":"03bb1faa_bdab4f54","line":42,"in_reply_to":"77adf0c0_22ca106c","updated":"2025-04-25 02:14:10.000000000","message":"Updated in latest patch.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"ac80cb7040d01e7130fe9d6e3ac2e1a58223148f","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Estimated changes are going to be in the following places:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"* Host maintenance strategy"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"  * Input parameters **disable_cold_migration** and **disable_live_migration**"},{"line_number":45,"context_line":"    to disable the migration."}],"source_content_type":"text/x-rst","patch_set":3,"id":"77adf0c0_22ca106c","line":42,"in_reply_to":"b94bbc8d_bc3e7297","updated":"2025-04-11 04:01:53.000000000","message":"That\u0027s a great question. So I think if both options are been provided, the operation will be:\n\n1. Since live migration is disabled, stop all the active status instances.\n2. Since the cold migration is disable, no cold migration will be triggered.\n\nAbout the mixing, I think the it will happen if you *only disable live-migration*, so:\n\n1. Since live migration is disabled, stop all the active status instances.\n2. Cold migration is enable, migrate all the SHUTOFF instances, including the instances stopped by previous operation.\n\nIf you agree with this approach, I can then update this spec document.","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"37a6ceb3196d47638e493c9d9261e89997946463","unresolved":true,"context_lines":[{"line_number":45,"context_line":"    to disable the migration."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    * If **disable_live_migration** is given,"},{"line_number":48,"context_line":"      this tell the strategy the live migration is not supported"},{"line_number":49,"context_line":"      in currently OpenStack deployment."},{"line_number":50,"context_line":"    * If **disable_cold_migration** is given,"},{"line_number":51,"context_line":"      this tell the strategy the cold migration is not supported"},{"line_number":52,"context_line":"      in currently OpenStack deployment."}],"source_content_type":"text/x-rst","patch_set":3,"id":"5b504764_f38c8488","line":49,"range":{"start_line":48,"start_character":48,"end_line":49,"end_character":40},"updated":"2025-04-09 18:38:30.000000000","message":"is more like a \"shouldn\u0027t be considered\" that not supported? since it is a strategy parameter?","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"ac80cb7040d01e7130fe9d6e3ac2e1a58223148f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    to disable the migration."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    * If **disable_live_migration** is given,"},{"line_number":48,"context_line":"      this tell the strategy the live migration is not supported"},{"line_number":49,"context_line":"      in currently OpenStack deployment."},{"line_number":50,"context_line":"    * If **disable_cold_migration** is given,"},{"line_number":51,"context_line":"      this tell the strategy the cold migration is not supported"},{"line_number":52,"context_line":"      in currently OpenStack deployment."}],"source_content_type":"text/x-rst","patch_set":3,"id":"06b409e1_66dd9c36","line":49,"range":{"start_line":48,"start_character":48,"end_line":49,"end_character":40},"in_reply_to":"5b504764_f38c8488","updated":"2025-04-11 04:01:53.000000000","message":"Acknowledged","commit_id":"4957f31a5f0bbe9d141fd6d0325f56dac8bac354"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"deb2fc27f6ecebb101933cd03544ec9b412ad433","unresolved":true,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"  * Action to stop the instance"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"346540cb_3fd60841","line":64,"updated":"2025-06-03 15:35:50.000000000","message":"i think it makes sense to start with this small enhancment.\n\nin the future i think we can have a diffent spec to allow you to annotate the instance with metadata\n\nspecific during the nova ptg session we agreed that it was ok for wathcer ot use instance metadata to encode policy\n\n\nSo we could add support for controlling this with \n\n    lifecyle:evacuatable\u003dtrue|false\n    lifecyle:live-migratable\u003dtrue|false\n    lifecyle:cold-migratable\u003dtrue|false\n    ...\n    \n \nIf we decided to do that, we should have a dedicated spec for encoding watcher policy via instance metadata\n\nlong term that is proably a better solution but we would need\nto consider how that impacts all strategies.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"deb2fc27f6ecebb101933cd03544ec9b412ad433","unresolved":true,"context_lines":[{"line_number":74,"context_line":"REST API impact"},{"line_number":75,"context_line":"---------------"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Two new input parameters for host maintenance strategy."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"Security impact"},{"line_number":80,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b2941f4c_ddeaf4ba","line":77,"updated":"2025-06-03 15:35:50.000000000","message":"ack while this will extend the input parmateres and extend teh schema it\ndoes not require an api microversiuon.\n\nso really there is no or littel impact at the rest api.","commit_id":"8421d76cc9b6990f4687fbf7a63cce9124fcf4b2"}],"specs/2025.2/approved/host-maintenance-strategy-disable-migration.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b704f3d02793cd9c6969fdbc870938e0ad5d117f","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Include the URL of your launchpad blueprint:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"https://blueprints.launchpad.net/watcher/+spec/example"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Problem description"},{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"23f471fc_8851cfed","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":54},"updated":"2025-06-16 18:27:08.000000000","message":"damb i forget ot check this was created\ni have created and appoved https://blueprints.launchpad.net/watcher/+spec/host-maintenance-strategy-disable-migration\n\nalthoug we should updat this later with teh correct link.","commit_id":"68eab6d25f5c14e7ab9d89418b4d3e74228b3adf"},{"author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"change_message_id":"d5d25f6f21ef266b6a92339d35c117dd51c3714e","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Include the URL of your launchpad blueprint:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"https://blueprints.launchpad.net/watcher/+spec/example"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Problem description"},{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"a5b75309_ce5f8d64","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":54},"in_reply_to":"23f471fc_8851cfed","updated":"2025-06-17 01:13:46.000000000","message":"Ah, thanks for the notice.\nUpdated in https://review.opendev.org/c/openstack/watcher-specs/+/952729","commit_id":"68eab6d25f5c14e7ab9d89418b4d3e74228b3adf"}]}