)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e8f2ca6772c919dc6d27dba12856b65a266b73a0","unresolved":true,"context_lines":[{"line_number":10,"context_line":"migration strategy can migrate both volumes and compute instances"},{"line_number":11,"context_line":"in the same action plan."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also refactor existing tests to use shared GOAL and STRATEGY class"},{"line_number":14,"context_line":"constants instead of hardcoded strings."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Generated-By: Cursor (claude-4-sonnet)"},{"line_number":17,"context_line":"Change-Id: I748d1316356e6ad116791eb33c3451c2a9c41636"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"e5b483aa_3bfa7855","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":39},"updated":"2025-10-15 07:04:55.000000000","message":"this cr does not contains any changes related to GOAL and strategy class, I think it needs to be removed.","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"5c0416b04d698ccd946a4680f2f6d5c285cee35f","unresolved":false,"context_lines":[{"line_number":10,"context_line":"migration strategy can migrate both volumes and compute instances"},{"line_number":11,"context_line":"in the same action plan."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also refactor existing tests to use shared GOAL and STRATEGY class"},{"line_number":14,"context_line":"constants instead of hardcoded strings."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Generated-By: Cursor (claude-4-sonnet)"},{"line_number":17,"context_line":"Change-Id: I748d1316356e6ad116791eb33c3451c2a9c41636"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"73a7da64_12985d31","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":39},"in_reply_to":"e5b483aa_3bfa7855","updated":"2025-10-15 08:23:07.000000000","message":"good catch, I incorporated those changes in the previous patch in the series, I\u0027ve removed the sentence, thanks.","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"347e023a360acffeb4c3da74a9d47da32c05ce78","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace the volume migration test with a new one that verifies zone"},{"line_number":10,"context_line":"migration strategy can migrate both volumes and compute instances"},{"line_number":11,"context_line":"in the same action plan."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Generated-By: Cursor (claude-4-sonnet)"},{"line_number":14,"context_line":"Change-Id: I748d1316356e6ad116791eb33c3451c2a9c41636"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":26,"id":"b6269628_1ed6295f","line":11,"updated":"2026-03-10 16:34:28.000000000","message":"we should not replace but rahter add a new test as we are losing test coverate of zone migrations of just voluems","commit_id":"50fb76a7afd10c4d20480ba1e5062cc290f326fc"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"e3d1942527f036025763d668ed7ddbf840481472","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace the volume migration test with a new one that verifies zone"},{"line_number":10,"context_line":"migration strategy can migrate both volumes and compute instances"},{"line_number":11,"context_line":"in the same action plan."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Generated-By: Cursor (claude-4-sonnet)"},{"line_number":14,"context_line":"Change-Id: I748d1316356e6ad116791eb33c3451c2a9c41636"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":26,"id":"90996a3d_e2f3ac9b","line":11,"in_reply_to":"b6269628_1ed6295f","updated":"2026-03-10 17:02:28.000000000","message":"good point, I\u0027ve added a new test instead of replacing the volume migrations one","commit_id":"50fb76a7afd10c4d20480ba1e5062cc290f326fc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1796f418fc3c862df7c62e2e3c9442900c2a7c65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"83d15e02_8988e852","updated":"2025-10-15 11:10:37.000000000","message":"If you update the commit message, I can update my vote. thank you!","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e8f2ca6772c919dc6d27dba12856b65a266b73a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"328f5020_9bea8509","updated":"2025-10-15 07:04:55.000000000","message":"Overall it looks good, left few improvements.","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"6a373fdacc5de3fcad7e30c4704390544279655a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"b1918236_4ffdcdb6","in_reply_to":"83d15e02_8988e852","updated":"2025-10-15 11:29:30.000000000","message":"thanks, done! I thought I had pushed the patch but git-review raised an error because I needed to rebase","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b1e81eff4a520c0bd4b4acb66fa3156f63859cf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"6bd4b27e_852cfded","in_reply_to":"b1918236_4ffdcdb6","updated":"2025-10-15 13:27:04.000000000","message":"thanks!","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b1e81eff4a520c0bd4b4acb66fa3156f63859cf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"7d5cfca8_d0f2c648","updated":"2025-10-15 13:27:04.000000000","message":"lgtm!","commit_id":"b582771d9b94153c3782a3e900133dfe9e732246"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a02f53c535e6c6d2ee227af49d97734b1fe15419","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"b72920de_298e00d2","updated":"2025-10-17 13:55:59.000000000","message":"check-rdo","commit_id":"f36c13de95d19a0043bf834004c3c49a85d5afb9"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e7e5b24ae5adad81d61bdd38b096850606eca89c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"2d573d46_5f2b6709","updated":"2025-11-04 21:28:22.000000000","message":"we need to drop the watcher-tempest-functional-2024-1 jobs on master seperately","commit_id":"c7d532962ed69a84f76f2a478fe18bc82f56cd80"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"8a275c99a79a558a8f0ad5ecb80808acac8cd356","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"564a5591_fb531fda","in_reply_to":"2d573d46_5f2b6709","updated":"2025-11-05 08:40:01.000000000","message":"true,good catch, I\u0027ve opened https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/966146 to drop the job","commit_id":"c7d532962ed69a84f76f2a478fe18bc82f56cd80"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"e3d1942527f036025763d668ed7ddbf840481472","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"c6a0a179_e2e4b220","in_reply_to":"564a5591_fb531fda","updated":"2026-03-10 17:02:28.000000000","message":"Done","commit_id":"c7d532962ed69a84f76f2a478fe18bc82f56cd80"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"4cc4b273bc6463ae6a603c8bc94b716cb97dcfbc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"d4b9a6e1_931a2b2c","updated":"2026-03-06 09:33:56.000000000","message":"Dropping workflow, need to confirm with others regarding do we need to backport depends on stable branches before merging it","commit_id":"8fdf293b27ebf81874b165846b2aaa069e65ddf4"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"5597a803156777a279f310e0aa6b8e69a321218a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"485e3c36_2fa9930f","updated":"2026-01-13 10:36:21.000000000","message":"lgtm","commit_id":"8fdf293b27ebf81874b165846b2aaa069e65ddf4"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fe90015d9ca7bc4b94a1273647ea267fb3e964af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"0c7f7d60_8374d4fc","updated":"2026-03-06 09:41:51.000000000","message":"recheck","commit_id":"8fdf293b27ebf81874b165846b2aaa069e65ddf4"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3764b706fc9d4607c4c2d615d7abd601420f882d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"f23313fa_3cb685e7","updated":"2026-03-20 17:59:16.000000000","message":"lgtm, test looks correct and is passing[1]\n\ntest_execute_zone_migration_with_volume_and_compute_migration[id-9ceff861-d6cd-4a88-8a8f-0a5d659b7b38,strategy,volume_migration,zone_migration]\npass\n\n[1] https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_309/openstack/3099a50ccd01410c93c7425087c87586/testr_results.html","commit_id":"817cecb7c80a26cd27d7530a9af216873aa8543a"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"69bdc1ca96a2f0a68cfe63ead10bb3a53e3d18d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"23586719_96fc86c4","updated":"2026-03-23 07:21:31.000000000","message":"recheck failed deploying devstack","commit_id":"817cecb7c80a26cd27d7530a9af216873aa8543a"}],"watcher_tempest_plugin/tests/scenario/test_execute_zone_migration.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e8f2ca6772c919dc6d27dba12856b65a266b73a0","unresolved":true,"context_lines":[{"line_number":294,"context_line":"            audit_template[\u0027uuid\u0027], **audit_kwargs)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        action_plan, _ \u003d self.get_action_plan_and_validate_actions("},{"line_number":297,"context_line":"            audit[\u0027uuid\u0027], [\u0027volume_migrate\u0027, \u0027migrate\u0027])"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        self.assertEqual(\"RECOMMENDED\", action_plan[\u0027state\u0027])"},{"line_number":300,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"7b4f5839_aeb71774","line":297,"range":{"start_line":297,"start_character":12,"end_line":297,"end_character":57},"updated":"2025-10-15 07:04:55.000000000","message":"Since we are renaming the test to add compute also. Can we also check for change_nova_service_state action here?\n```suggestion\n            audit[\u0027uuid\u0027], [\u0027volume_migrate\u0027, \u0027migrate\u0027],\n            [\u0027change_nova_service_state\u0027, \u0027migrate\u0027])\n```","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1796f418fc3c862df7c62e2e3c9442900c2a7c65","unresolved":false,"context_lines":[{"line_number":294,"context_line":"            audit_template[\u0027uuid\u0027], **audit_kwargs)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        action_plan, _ \u003d self.get_action_plan_and_validate_actions("},{"line_number":297,"context_line":"            audit[\u0027uuid\u0027], [\u0027volume_migrate\u0027, \u0027migrate\u0027])"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        self.assertEqual(\"RECOMMENDED\", action_plan[\u0027state\u0027])"},{"line_number":300,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"b143d2c5_4999b367","line":297,"range":{"start_line":297,"start_character":12,"end_line":297,"end_character":57},"in_reply_to":"23cc5e2f_1a84e8f1","updated":"2025-10-15 11:10:37.000000000","message":"Thank you @jgilaber@redhat.com for clarifying it. I was wrong here. It migrates the instance as a part of volume migration without disabling the node.","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"5c0416b04d698ccd946a4680f2f6d5c285cee35f","unresolved":true,"context_lines":[{"line_number":294,"context_line":"            audit_template[\u0027uuid\u0027], **audit_kwargs)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        action_plan, _ \u003d self.get_action_plan_and_validate_actions("},{"line_number":297,"context_line":"            audit[\u0027uuid\u0027], [\u0027volume_migrate\u0027, \u0027migrate\u0027])"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        self.assertEqual(\"RECOMMENDED\", action_plan[\u0027state\u0027])"},{"line_number":300,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"23cc5e2f_1a84e8f1","line":297,"range":{"start_line":297,"start_character":12,"end_line":297,"end_character":57},"in_reply_to":"7b4f5839_aeb71774","updated":"2025-10-15 08:23:07.000000000","message":"I don\u0027t think that\u0027s right, zone migration does not change the compute node state, just migrates instances","commit_id":"d4cf1a62f134e80877334ba121ae89a5c0d1b385"}]}