)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"30c4a8185adb6a69f64d1003395426be60364555","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5b303fcd_37cb67e9","updated":"2025-12-15 10:39:54.000000000","message":"lgtm","commit_id":"7182ddc60c4776676a940a28f64ffcf36f76f1f8"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"5fbf077e4436492b9b789654315802c78929778e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1d33b99d_aa09501f","updated":"2026-01-15 15:16:05.000000000","message":"recheck","commit_id":"15a4f31a52b1c4aac0ec3e8270f65b3215c0cc2a"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"b6ce664a21aeb5714d019c8d65543f2130468c9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c3a1db83_91d0cdb1","updated":"2026-01-15 18:38:22.000000000","message":"recheck","commit_id":"15a4f31a52b1c4aac0ec3e8270f65b3215c0cc2a"}],"doc/source/actions/stop.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3f16ff756843e096a55f673c25d98887f0605c0c","unresolved":true,"context_lines":[{"line_number":29,"context_line":"the following cases:"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"- The server does not exist"},{"line_number":32,"context_line":"- The server is already stopped"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5da42fa1_0bce2241","line":32,"updated":"2026-01-15 13:12:53.000000000","message":"i think technically nova will not reject stop in stopped but ya skipping makes sense too.","commit_id":"7182ddc60c4776676a940a28f64ffcf36f76f1f8"}],"watcher/applier/actions/stop.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3f16ff756843e096a55f673c25d98887f0605c0c","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        current_state \u003d instance.status"},{"line_number":160,"context_line":"        LOG.debug(\"Instance %s pre-condition check: state\u003d%s\","},{"line_number":161,"context_line":"                  self.instance_uuid, current_state)"},{"line_number":162,"context_line":"        if current_state \u003d\u003d \u0027SHUTOFF\u0027:"},{"line_number":163,"context_line":"            raise exception.ActionSkipped("},{"line_number":164,"context_line":"                _(\"Instance %s is already stopped\") % self.instance_uuid)"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac9092c_cb8769b3","line":162,"in_reply_to":"cb7d8b89_defdaef2","updated":"2026-01-15 13:12:53.000000000","message":"\u003e Missing handling of potential transient states other than SHUTOFF\n\u003e \n\u003e **Severity**: WARNING | **Confidence**: 0.7\n\u003e \n\u003e **Impact**: Instances in intermediate states might not be properly handled\n\u003e \n\u003e **Suggestion**:\n\u003e Consider handling other power states that should skip the stop action (e.g., SUSPENDED, PAUSED)\n\nwe can stop form both states i belvie so no we can ignore those.\nskiping is an optimisaton but we dont tneed to skip every possible place we could skip\n\npaused and suspended instase are not stopped even if there cpus are not executign so we shoudl nto thread it as a stoped state.","commit_id":"7182ddc60c4776676a940a28f64ffcf36f76f1f8"}]}