)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"9d9e024f9c7054440a39108fd20e61074a1277ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dade2c6d_ab0e0fe6","updated":"2025-11-21 16:43:12.000000000","message":"New docs available in https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_404/openstack/404bbd66132a4243b6cd29267e91e631/docs/actions/index.html","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b2b336d1_a47b6b6d","updated":"2025-11-27 15:49:47.000000000","message":"this is mostly ok, I left a few small comments, the only change I think we should address before proceeding is not documenting the `swap` volume migration that was removed in the past","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"ac72f9bc413ee43bac74ff2511a8d4a174a2db32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"78412976_972a98b4","in_reply_to":"b2b336d1_a47b6b6d","updated":"2025-11-27 17:02:01.000000000","message":"Good point, I removed the swap type and applied fixes for your other comments. Thanks.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ef8d623e_22f30929","updated":"2025-11-28 08:21:24.000000000","message":"thanks for the changes Alfredo, I think this is good to go now","commit_id":"6d50609266dff689005dc3daef020b697bf25f3a"}],"doc/source/actions/change_node_power_state.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91a19118390cf896ce382fc7c5a7a83488f9c357","unresolved":true,"context_lines":[{"line_number":1,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Change Node Power State"},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"Synopsis"},{"line_number":6,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"0c0f88e4_0cbe3ef1","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":24},"updated":"2025-12-08 22:04:31.000000000","message":"nit: this should be \n\n```suggestion\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nChange Node Power State\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n```","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bb4bf4e9249704c6720753e001cb361f86a1ae32","unresolved":false,"context_lines":[{"line_number":1,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":2,"context_line":"Change Node Power State"},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"Synopsis"},{"line_number":6,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"29ca3b70_654efbf4","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":24},"in_reply_to":"0c0f88e4_0cbe3ef1","updated":"2025-12-09 10:57:03.000000000","message":"Done","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91a19118390cf896ce382fc7c5a7a83488f9c357","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**action name**: ``change_node_power_state``"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Compute node power on/off"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to turn on/off the power of a"},{"line_number":13,"context_line":"compute node."}],"source_content_type":"text/x-rst","patch_set":4,"id":"05744a47_8516884d","line":10,"updated":"2025-12-08 22:04:31.000000000","message":"so technially this isnt nessiarly true.\n\nit depend on the ironic that its talking too.\n\nin general the \"overcloud\" ironic wont be the one that is used to provision the ocmptue nodes.\n\n\nwatcher is only going to talk to the ironic that is use by nova or the overcloud so really this would pbe powering of end user workloads\n\nwe do not have the ablity to manage multiple ironic instnace today.\nhttps://docs.openstack.org/watcher/latest/configuration/watcher.html#ironic-client\n\ni know this was intended for compute node usage but that would only work if ironic was shared between provisioning the compute nodes and tenant workloads.\n\nthis ti me is another reason to revisti remvoing the current ironic integration.","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"8c3ccbd9ab3d14eac01c594f8f9ad2cf46796e3c","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**action name**: ``change_node_power_state``"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Compute node power on/off"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to turn on/off the power of a"},{"line_number":13,"context_line":"compute node."}],"source_content_type":"text/x-rst","patch_set":4,"id":"46cf00e6_afe5f8fa","line":10,"in_reply_to":"05744a47_8516884d","updated":"2025-12-09 10:27:56.000000000","message":"Rephrased to a more generic \"... node managed by the Ironic service\"","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":21,"context_line":"parameter                type   required description"},{"line_number":22,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":23,"context_line":"``resource_id``          string yes      Baremetal node id (list of available"},{"line_number":24,"context_line":"                                         ironic nodes is returned by"},{"line_number":25,"context_line":"                                         ``ironic node-list`` command)"},{"line_number":26,"context_line":"``state``                string yes      Power state: \"on\" or \"off\""}],"source_content_type":"text/x-rst","patch_set":5,"id":"8105c0d5_999819dc","line":23,"in_reply_to":"061fc8f6_a82e4aa8","updated":"2026-01-15 12:59:32.000000000","message":"done","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"}],"doc/source/actions/change_nova_service_state.rst":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"180f1d6a_13f71d5a","line":26,"updated":"2025-11-27 15:49:47.000000000","message":"according to the schema this parameter is not required https://github.com/openstack/watcher/blob/8a884e3d5166678b85eaf264e518ef24b30085e5/watcher/applier/actions/change_nova_service_state.py#L77 which looks like a bug to me, since it\u0027s used unconditionally in https://github.com/openstack/watcher/blob/8a884e3d5166678b85eaf264e518ef24b30085e5/watcher/applier/actions/change_nova_service_state.py#L116","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"acba564c_97e9ec2f","line":26,"in_reply_to":"180f1d6a_13f71d5a","updated":"2025-11-27 17:01:20.000000000","message":"Exactly, it should be required and it\u0027s a bug I\u0027d say, that\u0027s why I adjusted the documentation. Changing it to mandatory should be another patch.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a507f38_0bc8110f","line":26,"in_reply_to":"47e800c8_98d8f637","updated":"2026-01-15 12:59:32.000000000","message":"that is a bug then.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"02098bbf61a3bbea71ab4cafa5e643151cec708d","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"47e800c8_98d8f637","line":26,"in_reply_to":"5b9b4183_6351268b","updated":"2025-12-09 17:46:48.000000000","message":"We can understand this as a bug in the code as currently the action requires the resource_name https://github.com/openstack/watcher/blob/8a884e3d5166678b85eaf264e518ef24b30085e5/watcher/applier/actions/change_nova_service_state.py#L82-L83  and actually resource_id is unused. That\u0027s why i docummented the current status of the code. We could treat this as a bug but in current status, the resource_name is needed for it to work.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"e0623ad6_eb03febe","line":26,"in_reply_to":"acba564c_97e9ec2f","updated":"2025-11-28 08:21:24.000000000","message":"+1, we should not be fixing it here, I was just surprised to see it. I created a bug for the issue https://bugs.launchpad.net/watcher/+bug/2133199","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bb4bf4e9249704c6720753e001cb361f86a1ae32","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"},{"line_number":29,"context_line":"                                         have ``watcher_`` prefix, such as"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5b9b4183_6351268b","line":26,"in_reply_to":"e0623ad6_eb03febe","updated":"2025-12-09 10:57:03.000000000","message":"you shoudl be able to use either the resouce_id or the resouce_name\n\nits not requried\n\nyou can use either the legacy api \n\nhttps://docs.openstack.org/api-ref/compute/#disable-scheduling-for-a-compute-service\n\nwhich uses the service and host values\nor its replacement which is \n\nhttps://docs.openstack.org/api-ref/compute/#update-compute-service\n\nthat uses the service uuid and is the prefered api to use.\n\nso watcher schema is correct and this doc is not.\n\nim not saying there isnt a bug but the bug is not in teh schma or that this is optional.\n\nwe requrie one of the resouce_id or resouce name but really our internall code shoudl be using the uuid not the name and it does nto need to specify both.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":false,"context_lines":[{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":"parameter                type   required description"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"}],"source_content_type":"text/x-rst","patch_set":5,"id":"945dfe65_dbaf2919","line":24,"in_reply_to":"3239d8d2_76b15128","updated":"2026-01-15 12:59:32.000000000","message":"Done","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":false,"context_lines":[{"line_number":22,"context_line":"parameter                type   required description"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"``resource_id``          string yes      nova-compute service name"},{"line_number":25,"context_line":"``state``                string yes      Service state: \"enabled\" or \"disabled\""},{"line_number":26,"context_line":"``resource_name``        string yes      nova-compute service name"},{"line_number":27,"context_line":"``disabled_reason``      string no       Reason why Watcher disables this"},{"line_number":28,"context_line":"                                         nova-compute service. Value should"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5c0ed5f9_5b2ffa70","line":25,"in_reply_to":"789994f6_0498de47","updated":"2026-01-15 12:59:32.000000000","message":"no this is fine we are intentally only docuemtneid enabeld and disabeld becuase that is what we want to use and it alinges to the nova api values.","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"}],"doc/source/actions/index.rst":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":31,"context_line":"       * - :doc:`migrate`"},{"line_number":32,"context_line":"         - Migrates a server to a destination nova-compute host (live or cold)"},{"line_number":33,"context_line":"       * - :doc:`resize`"},{"line_number":34,"context_line":"         - Resizes a server with specified flavor"},{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"}],"source_content_type":"text/x-rst","patch_set":2,"id":"ab054dc5_a307c009","line":34,"updated":"2025-11-27 15:49:47.000000000","message":"```suggestion\n         - Resizes a server with the specified flavor\n```","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":false,"context_lines":[{"line_number":31,"context_line":"       * - :doc:`migrate`"},{"line_number":32,"context_line":"         - Migrates a server to a destination nova-compute host (live or cold)"},{"line_number":33,"context_line":"       * - :doc:`resize`"},{"line_number":34,"context_line":"         - Resizes a server with specified flavor"},{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"}],"source_content_type":"text/x-rst","patch_set":2,"id":"63418591_2d18fe59","line":34,"in_reply_to":"ab054dc5_a307c009","updated":"2025-11-27 17:01:20.000000000","message":"Done","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":34,"context_line":"         - Resizes a server with specified flavor"},{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"},{"line_number":38,"context_line":"         - Migrates a volume to destination node or type"},{"line_number":39,"context_line":"       * - :doc:`nop`"},{"line_number":40,"context_line":"         - Logs a message (**testing only**)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"45f2cb2a_7ef46f58","line":37,"updated":"2025-11-27 15:49:47.000000000","message":"this is a nit, but the rows in the table do not follow the same order as the index at the top of the page, `nop` and `volume_migration` are out of place","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":true,"context_lines":[{"line_number":34,"context_line":"         - Resizes a server with specified flavor"},{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"},{"line_number":38,"context_line":"         - Migrates a volume to destination node or type"},{"line_number":39,"context_line":"       * - :doc:`nop`"},{"line_number":40,"context_line":"         - Logs a message (**testing only**)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"8ddf2f0b_190228d3","line":37,"in_reply_to":"45f2cb2a_7ef46f58","updated":"2025-11-27 17:01:20.000000000","message":"I made it in purpose because I liked putting the testing-only actions at the end. As an alternative, I will separe productiong and testing-only actions in two different tables, wdyt?","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":false,"context_lines":[{"line_number":34,"context_line":"         - Resizes a server with specified flavor"},{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"},{"line_number":38,"context_line":"         - Migrates a volume to destination node or type"},{"line_number":39,"context_line":"       * - :doc:`nop`"},{"line_number":40,"context_line":"         - Logs a message (**testing only**)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"0bca9a79_f93704f7","line":37,"in_reply_to":"8ddf2f0b_190228d3","updated":"2025-11-28 08:21:24.000000000","message":"I like the idea, that way we also emphasize more the difference between the two sets of actions","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"},{"line_number":38,"context_line":"         - Migrates a volume to destination node or type"},{"line_number":39,"context_line":"       * - :doc:`nop`"},{"line_number":40,"context_line":"         - Logs a message (**testing only**)"},{"line_number":41,"context_line":"       * - :doc:`sleep`"}],"source_content_type":"text/x-rst","patch_set":2,"id":"d1c9f615_30d5cd0e","line":38,"updated":"2025-11-27 15:49:47.000000000","message":"```suggestion\n         - Migrates a volume to a destination node or type\n```","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":false,"context_lines":[{"line_number":35,"context_line":"       * - :doc:`stop`"},{"line_number":36,"context_line":"         - Stops a server instance"},{"line_number":37,"context_line":"       * - :doc:`volume_migration`"},{"line_number":38,"context_line":"         - Migrates a volume to destination node or type"},{"line_number":39,"context_line":"       * - :doc:`nop`"},{"line_number":40,"context_line":"         - Logs a message (**testing only**)"},{"line_number":41,"context_line":"       * - :doc:`sleep`"}],"source_content_type":"text/x-rst","patch_set":2,"id":"6c28c41b_fd46d92e","line":38,"in_reply_to":"d1c9f615_30d5cd0e","updated":"2025-11-27 17:01:20.000000000","message":"Done","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"}],"doc/source/actions/migrate.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"bb4bf4e9249704c6720753e001cb361f86a1ae32","unresolved":true,"context_lines":[{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":"``resource_id``          string yes      UUID of the server to migrate"},{"line_number":32,"context_line":"``migration_type``       string yes      Type of migration: \"live\" or \"cold\""},{"line_number":33,"context_line":"``source_node``          string yes      Source compute hostname"},{"line_number":34,"context_line":"``destination_node``     string no       Destination compute hostname."},{"line_number":35,"context_line":"                                         If not specified, nova-scheduler"},{"line_number":36,"context_line":"                                         will determine the destination"}],"source_content_type":"text/x-rst","patch_set":5,"id":"05fdf040_11d63ecd","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":64},"updated":"2025-12-09 10:57:03.000000000","message":"this shoudl not actually be required at all.\nit can be specified for pre check evaulation but in general we shoudl not requried it\n\nwith that said we shoudl be docuemntign the current state rather then desired stte.","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":false,"context_lines":[{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":"``resource_id``          string yes      UUID of the server to migrate"},{"line_number":32,"context_line":"``migration_type``       string yes      Type of migration: \"live\" or \"cold\""},{"line_number":33,"context_line":"``source_node``          string yes      Source compute hostname"},{"line_number":34,"context_line":"``destination_node``     string no       Destination compute hostname."},{"line_number":35,"context_line":"                                         If not specified, nova-scheduler"},{"line_number":36,"context_line":"                                         will determine the destination"}],"source_content_type":"text/x-rst","patch_set":5,"id":"598d8181_23387bb6","line":33,"range":{"start_line":33,"start_character":0,"end_line":33,"end_character":64},"in_reply_to":"05fdf040_11d63ecd","updated":"2026-01-15 12:59:32.000000000","message":"this is also a latent bug but we cna adress that when we fix the implementation.","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"}],"doc/source/actions/resize.rst":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91a19118390cf896ce382fc7c5a7a83488f9c357","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**action name**: ``resize``"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Resizes a server with specified flavor"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This action will allow you to resize a server to another flavor."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"955db65a_af52be3c","line":10,"range":{"start_line":10,"start_character":17,"end_line":10,"end_character":21},"updated":"2025-12-08 22:04:31.000000000","message":"to","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"8c3ccbd9ab3d14eac01c594f8f9ad2cf46796e3c","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"**action name**: ``resize``"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Resizes a server with specified flavor"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This action will allow you to resize a server to another flavor."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"6e5e4ed3_05b2e667","line":10,"range":{"start_line":10,"start_character":17,"end_line":10,"end_character":21},"in_reply_to":"955db65a_af52be3c","updated":"2025-12-09 10:27:56.000000000","message":"Done","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"}],"doc/source/actions/volume_migration.rst":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":10,"context_line":"Migrates a volume to destination node or type"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to migrate cinder volume."},{"line_number":13,"context_line":"Migration type \u0027swap\u0027 can only be used for migrating attached volume."},{"line_number":14,"context_line":"Migration type \u0027migrate\u0027 can be used for migrating detached volume to"},{"line_number":15,"context_line":"the pool of same volume type."},{"line_number":16,"context_line":"Migration type \u0027retype\u0027 can be used for changing volume type of"}],"source_content_type":"text/x-rst","patch_set":2,"id":"f6748a61_469e6b24","line":13,"updated":"2025-11-27 15:49:47.000000000","message":"I would not document this action type at all, since we removed its implementation and any remaining reference was left to avoid breaking exisiting code","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":true,"context_lines":[{"line_number":10,"context_line":"Migrates a volume to destination node or type"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to migrate cinder volume."},{"line_number":13,"context_line":"Migration type \u0027swap\u0027 can only be used for migrating attached volume."},{"line_number":14,"context_line":"Migration type \u0027migrate\u0027 can be used for migrating detached volume to"},{"line_number":15,"context_line":"the pool of same volume type."},{"line_number":16,"context_line":"Migration type \u0027retype\u0027 can be used for changing volume type of"}],"source_content_type":"text/x-rst","patch_set":2,"id":"f6e81483_735d2965","line":13,"in_reply_to":"f6748a61_469e6b24","updated":"2025-11-27 17:01:20.000000000","message":"Right, good point, I\u0027ll remove it.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Migrates a volume to destination node or type"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to migrate cinder volume."},{"line_number":13,"context_line":"Migration type \u0027swap\u0027 can only be used for migrating attached volume."},{"line_number":14,"context_line":"Migration type \u0027migrate\u0027 can be used for migrating detached volume to"},{"line_number":15,"context_line":"the pool of same volume type."},{"line_number":16,"context_line":"Migration type \u0027retype\u0027 can be used for changing volume type of"}],"source_content_type":"text/x-rst","patch_set":2,"id":"cafb6f90_2ddb8dce","line":13,"in_reply_to":"f6e81483_735d2965","updated":"2025-11-28 08:21:24.000000000","message":"Done","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":30,"context_line":"``destination_node``     string no       Destination block storage pool name"},{"line_number":31,"context_line":"                                         (list of available pools returned by"},{"line_number":32,"context_line":"                                         ``cinder get-pools`` command)."},{"line_number":33,"context_line":"                                         Mandatory for migrating detached"},{"line_number":34,"context_line":"                                         volume to one with same volume type"},{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7062b6af_16bced02","line":33,"updated":"2025-11-27 15:49:47.000000000","message":"this is used to migrate both attached or detached volumes","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":false,"context_lines":[{"line_number":30,"context_line":"``destination_node``     string no       Destination block storage pool name"},{"line_number":31,"context_line":"                                         (list of available pools returned by"},{"line_number":32,"context_line":"                                         ``cinder get-pools`` command)."},{"line_number":33,"context_line":"                                         Mandatory for migrating detached"},{"line_number":34,"context_line":"                                         volume to one with same volume type"},{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"}],"source_content_type":"text/x-rst","patch_set":2,"id":"14650a54_83c12dfd","line":33,"in_reply_to":"2e6bb992_e6194605","updated":"2025-11-28 08:21:24.000000000","message":"Done","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":true,"context_lines":[{"line_number":30,"context_line":"``destination_node``     string no       Destination block storage pool name"},{"line_number":31,"context_line":"                                         (list of available pools returned by"},{"line_number":32,"context_line":"                                         ``cinder get-pools`` command)."},{"line_number":33,"context_line":"                                         Mandatory for migrating detached"},{"line_number":34,"context_line":"                                         volume to one with same volume type"},{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"}],"source_content_type":"text/x-rst","patch_set":2,"id":"2e6bb992_e6194605","line":33,"in_reply_to":"7062b6af_16bced02","updated":"2025-11-27 17:01:20.000000000","message":"I\u0027ll rephrase it.","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9ab1f28d4e993e1b7224c0f2ee15cd6dcf14ff1d","unresolved":true,"context_lines":[{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"},{"line_number":37,"context_line":"                                         ``cinder type-list`` command)."},{"line_number":38,"context_line":"                                         Mandatory for migrating detached"},{"line_number":39,"context_line":"                                         volume or swapping attached volume"},{"line_number":40,"context_line":"                                         to one with different volume type"},{"line_number":41,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\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":2,"id":"7fea5ac0_054f97a0","line":38,"updated":"2025-11-27 15:49:47.000000000","message":"same point about `detached`","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df3d7bf0f06578f73ec8104eac04c85cf87b3767","unresolved":true,"context_lines":[{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"},{"line_number":37,"context_line":"                                         ``cinder type-list`` command)."},{"line_number":38,"context_line":"                                         Mandatory for migrating detached"},{"line_number":39,"context_line":"                                         volume or swapping attached volume"},{"line_number":40,"context_line":"                                         to one with different volume type"},{"line_number":41,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\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":2,"id":"f7c167d6_c4d3f141","line":38,"in_reply_to":"7fea5ac0_054f97a0","updated":"2025-11-27 17:01:20.000000000","message":"ditto","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a5ea006588927c26e9e66a60d0d0d6ab2babc726","unresolved":false,"context_lines":[{"line_number":35,"context_line":"``destination_type``     string no       Destination block storage type name"},{"line_number":36,"context_line":"                                         (list of available types returned by"},{"line_number":37,"context_line":"                                         ``cinder type-list`` command)."},{"line_number":38,"context_line":"                                         Mandatory for migrating detached"},{"line_number":39,"context_line":"                                         volume or swapping attached volume"},{"line_number":40,"context_line":"                                         to one with different volume type"},{"line_number":41,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\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":2,"id":"ddc77a23_c4e4e07e","line":38,"in_reply_to":"f7c167d6_c4d3f141","updated":"2025-11-28 08:21:24.000000000","message":"Done","commit_id":"1447627e19486786697ff1db83ef455c95c268d5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91a19118390cf896ce382fc7c5a7a83488f9c357","unresolved":true,"context_lines":[{"line_number":10,"context_line":"Migrates a volume to destination node or type"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to migrate cinder volume."},{"line_number":13,"context_line":"Migration type \u0027migrate\u0027 can be used for migrating detached volume to"},{"line_number":14,"context_line":"the pool of same volume type."},{"line_number":15,"context_line":"Migration type \u0027retype\u0027 can be used for changing volume type of"},{"line_number":16,"context_line":"detached volume."}],"source_content_type":"text/x-rst","patch_set":4,"id":"182a73b6_70acdde7","line":13,"range":{"start_line":13,"start_character":51,"end_line":13,"end_character":59},"updated":"2025-12-08 22:04:31.000000000","message":"it can be used to migrate in use volumes too.","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"8c3ccbd9ab3d14eac01c594f8f9ad2cf46796e3c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Migrates a volume to destination node or type"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"By using this action, you will be able to migrate cinder volume."},{"line_number":13,"context_line":"Migration type \u0027migrate\u0027 can be used for migrating detached volume to"},{"line_number":14,"context_line":"the pool of same volume type."},{"line_number":15,"context_line":"Migration type \u0027retype\u0027 can be used for changing volume type of"},{"line_number":16,"context_line":"detached volume."}],"source_content_type":"text/x-rst","patch_set":4,"id":"84df03f2_b6223299","line":13,"range":{"start_line":13,"start_character":51,"end_line":13,"end_character":59},"in_reply_to":"182a73b6_70acdde7","updated":"2025-12-09 10:27:56.000000000","message":"Done","commit_id":"c571417796c00c529e4fcb7a63826ba4a9adc031"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"992bb389bb56eab4a478bd8f126d0d0c29bd60de","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":"parameter                type   required description"},{"line_number":25,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":26,"context_line":"``resource_id``          string yes      UUID of cinder volume to migrate"},{"line_number":27,"context_line":"``migration_type``       string yes      Type of migration: \"migrate\" or"},{"line_number":28,"context_line":"                                         \"retype\""},{"line_number":29,"context_line":"``destination_node``     string no       Destination block storage pool name"}],"source_content_type":"text/x-rst","patch_set":5,"id":"41d07da8_68605f3f","line":26,"in_reply_to":"330a1309_daf7435f","updated":"2026-01-15 12:59:32.000000000","message":"swap has been remvoed on securty grounds so we are documenting the 2 values that sould be used\n\nswap is now a alias of migrate.","commit_id":"3c2d7a4b0f801046f92b11a741758260d51844ed"}]}
