)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0b03cfed_27c58b9d","updated":"2025-06-06 15:15:55.000000000","message":"I did a quick pass and left a few suggestions, will take a better look next week, but this is already a great improvement, thanks!","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"0076ce04aabdb669b6aae13df1f4e7dfd99f378a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5de90911_5b875fb9","updated":"2025-06-06 17:51:55.000000000","message":"Nice, waiting Joan\u0027s input here, since he is working with zone migration and may have more noted to add about supported and not supported operations.\ntks Ronelle","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8648a61a_9519beac","updated":"2025-06-10 10:16:48.000000000","message":"left a few small suggestions","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a82164db_2445beae","updated":"2025-06-27 16:13:43.000000000","message":"i think the content is more or less ok but can you fix the line wrapping and note syntax.\n\nwe do not need to wrap urls just the other text.","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"46d35cfb8886d2843c379eb03c2bcf75e35ab818","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1c53a484_b9053aab","updated":"2025-06-11 10:23:13.000000000","message":"lgtm!","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"924d8bb569ffe1cb79277db5904784076bec57ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"72be3c00_c24665a7","updated":"2025-06-10 22:09:15.000000000","message":"lgtm, thanks","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"f087dd76598147cfaebc5d17131008d73dab5dbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c575c246_9a155f09","updated":"2025-06-10 15:29:54.000000000","message":"thanks for the changes! I think this doc is a good representation of the strategy current status","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"212cc59fadc6ca495103646a3993724add6c2db9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d574b738_c4571a8a","updated":"2025-06-30 12:42:02.000000000","message":"LGTM, thanks\noutput is looking good too [1]\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_848/openstack/8485cf521ff947d896f6fec22b8ec017/docs/strategies/zone_migration.html","commit_id":"de9eb2cd80075d41b1668332288cf35bb6e2a28e"}],"doc/source/strategies/zone_migration.rst":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":75,"context_line":"                                                     storage pool."},{"line_number":76,"context_line":"``priority``             object   None     Optional  List prioritizes instances"},{"line_number":77,"context_line":"                                                     and volumes."},{"line_number":78,"context_line":"``with_attached_volume`` boolean  False    Required  False: Instances will"},{"line_number":79,"context_line":"                                                     migrate after all volumes"},{"line_number":80,"context_line":"                                                     migrate."},{"line_number":81,"context_line":"                                                     True: An instance will"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1054a700_a8e3b248","line":78,"updated":"2025-06-06 15:15:55.000000000","message":"this parameter and all the `parallel*` ones are optional, they have a default, so the user can create an audit without specifying them just fine","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                                                     storage pool."},{"line_number":76,"context_line":"``priority``             object   None     Optional  List prioritizes instances"},{"line_number":77,"context_line":"                                                     and volumes."},{"line_number":78,"context_line":"``with_attached_volume`` boolean  False    Required  False: Instances will"},{"line_number":79,"context_line":"                                                     migrate after all volumes"},{"line_number":80,"context_line":"                                                     migrate."},{"line_number":81,"context_line":"                                                     True: An instance will"}],"source_content_type":"text/x-rst","patch_set":5,"id":"f99569ee_f46d9a1e","line":78,"in_reply_to":"1054a700_a8e3b248","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":84,"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\u003d\u003d"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"Note that ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"},{"line_number":87,"context_line":"parameters however, if they are passed they **require** the parameters"},{"line_number":88,"context_line":"in the tables below."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"The elements of compute_nodes array are:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"31a73899_4c2ca2a9","line":87,"updated":"2025-06-06 15:15:55.000000000","message":"```suggestion\nparameters, however, if they are passed they **require** the parameters\n```","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":84,"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\u003d\u003d"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"Note that ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"},{"line_number":87,"context_line":"parameters however, if they are passed they **require** the parameters"},{"line_number":88,"context_line":"in the tables below."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"The elements of compute_nodes array are:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9a28de69_0db71a1b","line":87,"in_reply_to":"31a73899_4c2ca2a9","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":90,"context_line":"The elements of compute_nodes array are:"},{"line_number":91,"context_line":""},{"line_number":92,"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"},{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"33d78b80_34076070","line":93,"updated":"2025-06-06 15:15:55.000000000","message":"```suggestion\nparameter     type    default   description\n```\nimo the `default` header we have in the table above looks better","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":90,"context_line":"The elements of compute_nodes array are:"},{"line_number":91,"context_line":""},{"line_number":92,"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"},{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"9c17e4b6_a8f3b82e","line":93,"updated":"2025-06-06 15:15:55.000000000","message":"it would be a good enhancement to add a `required` column for this fields as well","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":90,"context_line":"The elements of compute_nodes array are:"},{"line_number":91,"context_line":""},{"line_number":92,"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"},{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"5600131f_f4a16539","line":93,"in_reply_to":"33d78b80_34076070","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":90,"context_line":"The elements of compute_nodes array are:"},{"line_number":91,"context_line":""},{"line_number":92,"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"},{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"49b045f7_e0288f1c","line":93,"in_reply_to":"9c17e4b6_a8f3b82e","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"0076ce04aabdb669b6aae13df1f4e7dfd99f378a","unresolved":true,"context_lines":[{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."},{"line_number":97,"context_line":"``dst_node``  string    None          Compute node to which"},{"line_number":98,"context_line":"                                      instances migrate."},{"line_number":99,"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"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bd87346d_169474f3","line":96,"range":{"start_line":96,"start_character":56,"end_line":96,"end_character":65},"updated":"2025-06-06 17:51:55.000000000","message":"maybe we could also include a required columns here, as the above table, instead of keeping the info in the descrition","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":93,"context_line":"parameter     type    default Value   description"},{"line_number":94,"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"},{"line_number":95,"context_line":"``src_node``  string    None          Compute node from which"},{"line_number":96,"context_line":"                                      instances migrate(mandatory)."},{"line_number":97,"context_line":"``dst_node``  string    None          Compute node to which"},{"line_number":98,"context_line":"                                      instances migrate."},{"line_number":99,"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"}],"source_content_type":"text/x-rst","patch_set":5,"id":"0aaf8e10_18203e58","line":96,"range":{"start_line":96,"start_character":56,"end_line":96,"end_character":65},"in_reply_to":"bd87346d_169474f3","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":101,"context_line":"The elements of storage_pools array are:"},{"line_number":102,"context_line":""},{"line_number":103,"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"},{"line_number":104,"context_line":"parameter     type    default Value   description"},{"line_number":105,"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"},{"line_number":106,"context_line":"``src_pool``  string    None          Storage pool from which"},{"line_number":107,"context_line":"                                      volumes migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"67a5d218_fa636850","line":104,"updated":"2025-06-06 15:15:55.000000000","message":"same point as above about `default Value` and `required` column","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":101,"context_line":"The elements of storage_pools array are:"},{"line_number":102,"context_line":""},{"line_number":103,"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"},{"line_number":104,"context_line":"parameter     type    default Value   description"},{"line_number":105,"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"},{"line_number":106,"context_line":"``src_pool``  string    None          Storage pool from which"},{"line_number":107,"context_line":"                                      volumes migrate(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"454e1acd_c537ecb2","line":104,"in_reply_to":"67a5d218_fa636850","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"0076ce04aabdb669b6aae13df1f4e7dfd99f378a","unresolved":true,"context_lines":[{"line_number":104,"context_line":"parameter     type    default Value   description"},{"line_number":105,"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"},{"line_number":106,"context_line":"``src_pool``  string    None          Storage pool from which"},{"line_number":107,"context_line":"                                      volumes migrate(mandatory)."},{"line_number":108,"context_line":"``dst_pool``  string    None          Storage pool to which"},{"line_number":109,"context_line":"                                      volumes migrate."},{"line_number":110,"context_line":"``src_type``  string    None          Source volume type(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"e3dfe72b_9f43bc1d","line":107,"range":{"start_line":107,"start_character":54,"end_line":107,"end_character":63},"updated":"2025-06-06 17:51:55.000000000","message":"same","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":104,"context_line":"parameter     type    default Value   description"},{"line_number":105,"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"},{"line_number":106,"context_line":"``src_pool``  string    None          Storage pool from which"},{"line_number":107,"context_line":"                                      volumes migrate(mandatory)."},{"line_number":108,"context_line":"``dst_pool``  string    None          Storage pool to which"},{"line_number":109,"context_line":"                                      volumes migrate."},{"line_number":110,"context_line":"``src_type``  string    None          Source volume type(mandatory)."}],"source_content_type":"text/x-rst","patch_set":5,"id":"9ebbd6ec_d9618cb0","line":107,"range":{"start_line":107,"start_character":54,"end_line":107,"end_character":63},"in_reply_to":"e3dfe72b_9f43bc1d","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"66881cd987286aba375b578b7d25b5b9cccd664b","unresolved":true,"context_lines":[{"line_number":115,"context_line":"The elements of priority object are:"},{"line_number":116,"context_line":""},{"line_number":117,"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"},{"line_number":118,"context_line":"parameter        type    default Value   description"},{"line_number":119,"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"},{"line_number":120,"context_line":"``project``      array   None            Project names."},{"line_number":121,"context_line":"``compute_node`` array   None            Compute node names."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7c181b13_3fc875a5","line":118,"updated":"2025-06-06 15:15:55.000000000","message":"ditto","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7deda859ac2ca33c38366d15048311fad19523ca","unresolved":false,"context_lines":[{"line_number":115,"context_line":"The elements of priority object are:"},{"line_number":116,"context_line":""},{"line_number":117,"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"},{"line_number":118,"context_line":"parameter        type    default Value   description"},{"line_number":119,"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"},{"line_number":120,"context_line":"``project``      array   None            Project names."},{"line_number":121,"context_line":"``compute_node`` array   None            Compute node names."}],"source_content_type":"text/x-rst","patch_set":5,"id":"6e62e9f8_b7ea1d04","line":118,"in_reply_to":"7c181b13_3fc875a5","updated":"2025-06-06 20:29:51.000000000","message":"Done","commit_id":"dbd26c7a5b057217090deb792394cfcec93948e2"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"**goal**: ``hardware_maintenance``"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. watcher-term:: watcher.decision_engine.strategy.strategies.zone_migration.ZoneMigration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Requirements"},{"line_number":15,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"a466f2a0_d01de40b","line":12,"updated":"2025-06-10 10:16:48.000000000","message":"I think we should add a note here to make clear that the `Zone` in the strategy name is not a reference to openstack availability zones https://docs.openstack.org/nova/latest/admin/availability-zones.html but rather a user-defined set of compute nodes and storage pools. Also it might be worth noting that migrations across actual availability zones is not tested and might not work depending on the cluster configuration","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"**goal**: ``hardware_maintenance``"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. watcher-term:: watcher.decision_engine.strategy.strategies.zone_migration.ZoneMigration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Requirements"},{"line_number":15,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"a3648aec_278deb3f","line":12,"in_reply_to":"a466f2a0_d01de40b","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":69,"context_line":"                                                     total."},{"line_number":70,"context_line":"``parallel_per_node``    integer  2        Optional  The number of actions to"},{"line_number":71,"context_line":"                                                     be run in parallel per"},{"line_number":72,"context_line":"                                                     compute node."},{"line_number":73,"context_line":"``parallel_per_pool``    integer  2        Optional  The number of actions to"},{"line_number":74,"context_line":"                                                     be run in parallel per"},{"line_number":75,"context_line":"                                                     storage pool."}],"source_content_type":"text/x-rst","patch_set":7,"id":"a1cbc7f6_a24ea8e2","line":72,"updated":"2025-06-10 10:16:48.000000000","message":"```suggestion\n                                                     compute node in one action plan.\n```\nI think we should add the disclaimer to all `parallel_*` parameters that what they do is not so much dealing with concurrency, but limiting the amount of actions to be added to the action plan","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                                                     total."},{"line_number":70,"context_line":"``parallel_per_node``    integer  2        Optional  The number of actions to"},{"line_number":71,"context_line":"                                                     be run in parallel per"},{"line_number":72,"context_line":"                                                     compute node."},{"line_number":73,"context_line":"``parallel_per_pool``    integer  2        Optional  The number of actions to"},{"line_number":74,"context_line":"                                                     be run in parallel per"},{"line_number":75,"context_line":"                                                     storage pool."}],"source_content_type":"text/x-rst","patch_set":7,"id":"5dc7aa1f_9287fca0","line":72,"in_reply_to":"a1cbc7f6_a24ea8e2","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":148,"context_line":"* ``live_migrate_instance_count`` is shown as"},{"line_number":149,"context_line":"  ``The number of instances actually live migrated`` in Horizon"},{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."}],"source_content_type":"text/x-rst","patch_set":7,"id":"8526c2fc_492e061b","line":151,"updated":"2025-06-10 10:16:48.000000000","message":"```suggestion\n  ``The number of instances planned to live migrate`` in Horizon\n```","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":148,"context_line":"* ``live_migrate_instance_count`` is shown as"},{"line_number":149,"context_line":"  ``The number of instances actually live migrated`` in Horizon"},{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."}],"source_content_type":"text/x-rst","patch_set":7,"id":"e6f35879_f24a5860","line":151,"in_reply_to":"8526c2fc_492e061b","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":149,"context_line":"  ``The number of instances actually live migrated`` in Horizon"},{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."},{"line_number":155,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"deb1a8c9_41c59791","line":152,"updated":"2025-06-10 10:16:48.000000000","message":"```suggestion\n* ``planned_live_migrate_instance_count`` refers to the instances planned to live migrate in the action plan.\n```\n`planned_live_count` and `live_count` below are variable names used in the strategy implementation, they are not exposed to the user","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":149,"context_line":"  ``The number of instances actually live migrated`` in Horizon"},{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."},{"line_number":155,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"2c54364a_058870b1","line":152,"in_reply_to":"deb1a8c9_41c59791","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5f81d7c3_b3e5de08","line":153,"updated":"2025-06-10 10:16:48.000000000","message":"```suggestion\n* ``live_migrate_instance_count`` tracks all the instances that could be migrated according\n```","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":150,"context_line":"* ``planned_live_migrate_instance_count`` is  shown as"},{"line_number":151,"context_line":"  ``The number of instances planned to live migrate`` in Horzon"},{"line_number":152,"context_line":"* ``planned_live_count`` refers to the instances planned to live migrate."},{"line_number":153,"context_line":"* ``live_count`` tracks all the instances that could be migrated according"},{"line_number":154,"context_line":"  to the audit input."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"a30460ef_c26dec6c","line":153,"in_reply_to":"5f81d7c3_b3e5de08","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ff96905b7042578508dc1f1af7750aaf289105f2","unresolved":true,"context_lines":[{"line_number":178,"context_line":"  will be included in the action plan."},{"line_number":179,"context_line":"* The Cinder model collector is not enabled by default."},{"line_number":180,"context_line":"  If the Cinder model collector is not enabled while deploying Watcher, the"},{"line_number":181,"context_line":"  model will become outdated and cause errors eventually"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"Support caveats"},{"line_number":184,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"fc367bac_794bcf67","line":181,"updated":"2025-06-10 10:16:48.000000000","message":"I think it would be heplful to add a link here to the configuration option to enable the storage collector https://docs.openstack.org/watcher/latest/configuration/watcher.html#collector.collector_plugins","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f010f9303bbeae2a6ef40b2a2c8b7ccc27f6f36a","unresolved":false,"context_lines":[{"line_number":178,"context_line":"  will be included in the action plan."},{"line_number":179,"context_line":"* The Cinder model collector is not enabled by default."},{"line_number":180,"context_line":"  If the Cinder model collector is not enabled while deploying Watcher, the"},{"line_number":181,"context_line":"  model will become outdated and cause errors eventually"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"Support caveats"},{"line_number":184,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3e59ef73_bfae0236","line":181,"in_reply_to":"fc367bac_794bcf67","updated":"2025-06-10 14:52:21.000000000","message":"Done","commit_id":"8e14a5280f754144b39a2f1781bff320d818625f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. watcher-term:: watcher.decision_engine.strategy.strategies.zone_migration.ZoneMigration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"**Note** that the term ``Zone`` in the strategy name is not a reference to `Openstack availability zones \u003chttps://docs.openstack.org/nova/latest/admin/availability-zones.html\u003e`_ but rather a user-defined set of Compute nodes and storage pools. Currently, migrations across actual availability zones is not fully tested and might not work in all cluster configurations."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Requirements"},{"line_number":17,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"769d4c85_5dbc0c3b","line":14,"updated":"2025-06-27 16:13:43.000000000","message":"can we not have this all on one line?\n\nsphinx will wrap this when rendering but since i spend a lot of my time reading doc\nin raw from i also care about readablity in gerrit or in my text editor.\n\nhere and below notes shoudl use the note directive\n`.. note::`","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"5410c8ab3b9468012c436b1dc6078790ce530be8","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. watcher-term:: watcher.decision_engine.strategy.strategies.zone_migration.ZoneMigration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"**Note** that the term ``Zone`` in the strategy name is not a reference to `Openstack availability zones \u003chttps://docs.openstack.org/nova/latest/admin/availability-zones.html\u003e`_ but rather a user-defined set of Compute nodes and storage pools. Currently, migrations across actual availability zones is not fully tested and might not work in all cluster configurations."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Requirements"},{"line_number":17,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"49204097_078736f1","line":14,"in_reply_to":"769d4c85_5dbc0c3b","updated":"2025-06-27 20:12:17.000000000","message":"Done","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":false,"context_lines":[{"line_number":66,"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\u003d\u003d"},{"line_number":67,"context_line":"``compute_nodes``        array    None     Optional  Compute nodes to migrate."},{"line_number":68,"context_line":"``storage_pools``        array    None     Optional  Storage pools to migrate."},{"line_number":69,"context_line":"``parallel_total``       integer  6        Optional  The number of actions to"},{"line_number":70,"context_line":"                                                     be run in parallel in"},{"line_number":71,"context_line":"                                                     total."},{"line_number":72,"context_line":"``parallel_per_node``    integer  2        Optional  The number of actions to"}],"source_content_type":"text/x-rst","patch_set":8,"id":"faeca0e0_61a0d889","line":69,"range":{"start_line":69,"start_character":33,"end_line":69,"end_character":36},"updated":"2025-06-27 16:13:43.000000000","message":"nit: so if this was a config option we would normaly use \n\n:oslo.config:option:`default_parallel_total` or something like that to reder it properly and keep the value in the do and the actula config option linked.\n\nright now we dont haves omething to keep thsoe in sync but we may want to think about that going forward\n\nhttps://github.com/openstack/nova/blob/master/doc/ext/extra_specs.py\n\n\nat some point we will want to link the json schema defieitons to this.\ninstead of doing this by hand.\nwe woudl do htat by wrting a sphinx docs exteion like https://github.com/openstack/nova/blob/master/doc/ext/extra_specs.py or https://github.com/openstack/nova/blob/master/doc/ext/feature_matrix.py\nto do that\n\nthis is just a future todo, there is nothign actionable about this comment today.","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":true,"context_lines":[{"line_number":86,"context_line":"                                                     attached volumes migrate."},{"line_number":87,"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\u003d\u003d"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Note that:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3ec3870b_9a067f2d","line":89,"updated":"2025-06-27 16:13:43.000000000","message":"`.. note::`","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"5410c8ab3b9468012c436b1dc6078790ce530be8","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                                                     attached volumes migrate."},{"line_number":87,"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\u003d\u003d"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Note that:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"}],"source_content_type":"text/x-rst","patch_set":8,"id":"b67b6d9d_cbddaad2","line":89,"in_reply_to":"3ec3870b_9a067f2d","updated":"2025-06-27 20:12:17.000000000","message":"Done","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":true,"context_lines":[{"line_number":89,"context_line":"Note that:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"},{"line_number":93,"context_line":"* ``parallel_*`` parameters are not in reference to concurrency,"},{"line_number":94,"context_line":"  but rather on limiting the amount of actions to be added to the action plan"},{"line_number":95,"context_line":"* ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"}],"source_content_type":"text/x-rst","patch_set":8,"id":"a6f6865f_ea6f5370","line":92,"updated":"2025-06-27 16:13:43.000000000","message":"they can but we proably shoudl block that in the api.\n\nthere is nothign actionalble for the audit when you do that in thsi case.\n\nso while the schema shoudl not block it the api validation for this type should.\n\nwe can work on that as a ux bug but it likely should have a new api microversion.","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3737d461ee7bebb21ebae7920447deb2bdfa7e01","unresolved":false,"context_lines":[{"line_number":89,"context_line":"Note that:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"},{"line_number":93,"context_line":"* ``parallel_*`` parameters are not in reference to concurrency,"},{"line_number":94,"context_line":"  but rather on limiting the amount of actions to be added to the action plan"},{"line_number":95,"context_line":"* ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"}],"source_content_type":"text/x-rst","patch_set":8,"id":"4f3e3f94_3635e716","line":92,"in_reply_to":"241db6b8_7ed3a4e5","updated":"2025-06-27 21:41:38.000000000","message":"Acknowledged","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"5410c8ab3b9468012c436b1dc6078790ce530be8","unresolved":true,"context_lines":[{"line_number":89,"context_line":"Note that:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"},{"line_number":93,"context_line":"* ``parallel_*`` parameters are not in reference to concurrency,"},{"line_number":94,"context_line":"  but rather on limiting the amount of actions to be added to the action plan"},{"line_number":95,"context_line":"* ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"}],"source_content_type":"text/x-rst","patch_set":8,"id":"241db6b8_7ed3a4e5","line":92,"in_reply_to":"a6f6865f_ea6f5370","updated":"2025-06-27 20:12:17.000000000","message":"Adding caution that nothing actionable for the audit with defaults","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":true,"context_lines":[{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"},{"line_number":93,"context_line":"* ``parallel_*`` parameters are not in reference to concurrency,"},{"line_number":94,"context_line":"  but rather on limiting the amount of actions to be added to the action plan"},{"line_number":95,"context_line":"* ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"},{"line_number":96,"context_line":"  parameters, however, if they are passed they **require** the parameters"},{"line_number":97,"context_line":"  in the tables below:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"de44bfd3_750c5e5b","line":94,"updated":"2025-06-27 16:13:43.000000000","message":"taht is so not intuitive. thanks for adding this.","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"5410c8ab3b9468012c436b1dc6078790ce530be8","unresolved":false,"context_lines":[{"line_number":91,"context_line":"* All parameters in the table above have defaults and therefore the"},{"line_number":92,"context_line":"  user can create an audit without specifying a value"},{"line_number":93,"context_line":"* ``parallel_*`` parameters are not in reference to concurrency,"},{"line_number":94,"context_line":"  but rather on limiting the amount of actions to be added to the action plan"},{"line_number":95,"context_line":"* ``compute_nodes``, ``storage_pools``, and ``priority`` are optional"},{"line_number":96,"context_line":"  parameters, however, if they are passed they **require** the parameters"},{"line_number":97,"context_line":"  in the tables below:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"14963d21_9b5538b6","line":94,"in_reply_to":"de44bfd3_750c5e5b","updated":"2025-06-27 20:12:17.000000000","message":"Acknowledged","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"805858241b86097c1ac2a50bb3bb00af285b3e09","unresolved":true,"context_lines":[{"line_number":177,"context_line":"    $ openstack optimize audit create -a at1 \\"},{"line_number":178,"context_line":"      -p compute_nodes\u003d\u0027[{\"src_node\": \"s01\", \"dst_node\": \"d01\"}]\u0027"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"**Notes on usage**:"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* Currently, the strategy will not generate both volume migration and instance"},{"line_number":183,"context_line":"  migrations in the same audit. If both are requested, only volume migrations"}],"source_content_type":"text/x-rst","patch_set":8,"id":"7339337f_97eed5b0","line":180,"updated":"2025-06-27 16:13:43.000000000","message":"`.. note::`","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"5410c8ab3b9468012c436b1dc6078790ce530be8","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    $ openstack optimize audit create -a at1 \\"},{"line_number":178,"context_line":"      -p compute_nodes\u003d\u0027[{\"src_node\": \"s01\", \"dst_node\": \"d01\"}]\u0027"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"**Notes on usage**:"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"* Currently, the strategy will not generate both volume migration and instance"},{"line_number":183,"context_line":"  migrations in the same audit. If both are requested, only volume migrations"}],"source_content_type":"text/x-rst","patch_set":8,"id":"4acfb711_17030ec2","line":180,"in_reply_to":"7339337f_97eed5b0","updated":"2025-06-27 20:12:17.000000000","message":"Done","commit_id":"ccb68fc684ca670200c19f4ba20a29798803c2ba"}]}