)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"b54d9f48a03bd5f722fcb222e8157bd6466d1f7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"14dfdff8_d6cb560c","updated":"2025-07-14 09:07:33.000000000","message":"check-rdo","commit_id":"b3d00524ff2dcca8938dd140ad7af6cda11fec6f"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"a3b6a7355e78069b8eb1aa23244099b67c6f8b4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0fe52318_5d8bcdde","updated":"2025-08-18 10:58:51.000000000","message":"recheck","commit_id":"4375368aefbcccd365313bcd09ab37220ba3f09b"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ec9c5aefa5ff61b85355a5f864958cfe64be2e1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"2304aa0d_6869a077","updated":"2025-08-19 05:23:49.000000000","message":"recheck","commit_id":"4375368aefbcccd365313bcd09ab37220ba3f09b"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"bc2de7e05c52c0f829057316cf6e1e81820e3c4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a47bde47_24a93f12","updated":"2025-08-18 15:46:24.000000000","message":"recheck","commit_id":"4375368aefbcccd365313bcd09ab37220ba3f09b"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"5ea93bd4e20acca88ca07464304f5be6e5a59276","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"db48bb55_3b634522","updated":"2025-08-18 12:57:29.000000000","message":"recheck","commit_id":"4375368aefbcccd365313bcd09ab37220ba3f09b"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"c8dab4c360498139f19154b930e818affb827c1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"0db302e4_cd96abc1","updated":"2025-08-19 18:08:44.000000000","message":"recheck","commit_id":"27dbbe1244f23e859af921a86898a213bab74aed"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"d29706bfa24197d9f79f25033830cd22854b8228","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"dd0a5903_247e86b2","updated":"2025-08-19 13:21:36.000000000","message":"recheck","commit_id":"27dbbe1244f23e859af921a86898a213bab74aed"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"e742e52f0080ba32fa8445344c39f408548b923a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"5496fe9f_7d827fcd","updated":"2025-08-25 17:26:16.000000000","message":"recheck","commit_id":"b4ec283ffe4e8b3528a2631f7fd1f0e643fdf495"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9e1e0545b56a21b64f6728a6fcefbc42aa1cfa62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"85633702_fe21bfc8","updated":"2025-08-26 07:46:35.000000000","message":"recheck","commit_id":"b4ec283ffe4e8b3528a2631f7fd1f0e643fdf495"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e1338a53330872a944f81dce0c8ac7dc47ae463e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"6b0679d9_db30d434","updated":"2025-09-09 12:36:17.000000000","message":"This is lgtm, since it cover a missing scenario for zone_migration.","commit_id":"2ac0e9f03518ff9b4fd7b6c07fcbefadde83737e"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"c2fcadbac19259946c3dd84efccbee5aab245c67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"322d39ec_798aacf5","updated":"2025-09-02 07:45:07.000000000","message":"recheck depends-on updated","commit_id":"2ac0e9f03518ff9b4fd7b6c07fcbefadde83737e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6df9a52c8787755c86059e0b5d6672a634ac653e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"390b05ab_4e6e2557","updated":"2025-09-22 11:49:14.000000000","message":"lgtm! https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_321/openstack/3219b12f3e18450a9ff1e6f97119763c/testr_results.html \n\nthank you for improving the tempest coverage around volume type","commit_id":"4e697c300b674986a6f47dd84b79deb3fe34f4a6"}],"watcher_tempest_plugin/tests/scenario/test_execute_zone_migration.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e1338a53330872a944f81dce0c8ac7dc47ae463e","unresolved":true,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        audit_parameters \u003d {"},{"line_number":170,"context_line":"            \"storage_pools\": ["},{"line_number":171,"context_line":"                {\"src_pool\": src_pool, \"src_type\": src_type,"},{"line_number":172,"context_line":"                 \"dst_type\": volume_type[\u0027name\u0027]}"},{"line_number":173,"context_line":"                for src_pool in src_pools"},{"line_number":174,"context_line":"                ]"}],"source_content_type":"text/x-python","patch_set":19,"id":"415172ed_a204f701","line":171,"range":{"start_line":171,"start_character":51,"end_line":171,"end_character":59},"updated":"2025-09-09 12:36:17.000000000","message":"ok, so src_type is the same for both, since it is the default one, in create_volume.","commit_id":"2ac0e9f03518ff9b4fd7b6c07fcbefadde83737e"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e1338a53330872a944f81dce0c8ac7dc47ae463e","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        strategy_name \u003d \"zone_migration\""},{"line_number":179,"context_line":"        audit_kwargs \u003d {\"parameters\": audit_parameters}"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        LOG.debug(\"Executing strategy\")"},{"line_number":182,"context_line":"        self.execute_strategy(goal_name, strategy_name,"},{"line_number":183,"context_line":"                              expected_actions\u003d[\u0027volume_migrate\u0027],"},{"line_number":184,"context_line":"                              **audit_kwargs)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"        # check that the available volume was retyped"},{"line_number":187,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":19,"id":"6bf09475_ae192df0","line":184,"range":{"start_line":181,"start_character":0,"end_line":184,"end_character":45},"updated":"2025-09-09 12:36:17.000000000","message":"so I guess that this would be a simple retype. Is there a more complex retype that we can validate as a follow up? where a retype would move the volume to a different pool?","commit_id":"2ac0e9f03518ff9b4fd7b6c07fcbefadde83737e"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ca1ff516883ce75f35eee5782c046b80175a2cb8","unresolved":true,"context_lines":[{"line_number":178,"context_line":"        strategy_name \u003d \"zone_migration\""},{"line_number":179,"context_line":"        audit_kwargs \u003d {\"parameters\": audit_parameters}"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        LOG.debug(\"Executing strategy\")"},{"line_number":182,"context_line":"        self.execute_strategy(goal_name, strategy_name,"},{"line_number":183,"context_line":"                              expected_actions\u003d[\u0027volume_migrate\u0027],"},{"line_number":184,"context_line":"                              **audit_kwargs)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"        # check that the available volume was retyped"},{"line_number":187,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":19,"id":"33f1aedb_b208452d","line":184,"range":{"start_line":181,"start_character":0,"end_line":184,"end_character":45},"in_reply_to":"6bf09475_ae192df0","updated":"2025-09-18 12:05:35.000000000","message":"yes, there is another scenario like you mention where we might want to retype and then migrated to a different pool, but we don\u0027t have the code for that operator in the Watcher actions yet","commit_id":"2ac0e9f03518ff9b4fd7b6c07fcbefadde83737e"}]}