)]}'
{"id":"openstack%2Fwatcher~961459","triplet_id":"openstack%2Fwatcher~stable%2F2024.2~Ie1894311b0e384ab52b1b3dfe0eb50618eef6c9f","project":"openstack/watcher","branch":"stable/2024.2","hashtags":[],"change_id":"Ie1894311b0e384ab52b1b3dfe0eb50618eef6c9f","subject":"Improve unit tests for zone migration strategy","status":"ABANDONED","created":"2025-09-17 08:22:53.000000000","updated":"2026-04-30 09:36:37.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"e0ae3bd10d0f601b8d9c4cd85eabc835df2a3f45","_number":961459,"virtual_id_number":961459,"owner":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2026-01-20 15:29:09.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Standard Change","+1":"Important Change","+2":"High Priority Change"},"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-17 09:54:32.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2026-04-30 09:36:37.000000000","updated_by":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"reviewer":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"state":"CC"}],"messages":[{"id":"ab190551060947e27f1676ee88acaff2f005902f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-09-17 08:22:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"133b63d86f37ec95ebbd6bbdf7bed8166490a8fb","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-17 09:54:32.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/7aae16c11ed94a8cb3bf2a1ee80f3918\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/6fb273a08d6e4195af52b914540a3cb3 : SUCCESS in 5m 23s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d0a74170b4b342a0bc54303e463facea : SUCCESS in 3m 37s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/2f019e31e8674ca88099f08cc9292c1c : SUCCESS in 4m 13s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/f9eed8cd47fc4c309cd6da57359b5685 : SUCCESS in 5m 11s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/e6126d5f21db49ff87e24ab5041d71a7 : SUCCESS in 5m 23s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f8f9cd6e43104d9b81740a51b274a614 : SUCCESS in 5m 47s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/b08a785d1ad44f4387183f73bd77054f : SUCCESS in 27m 01s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/bc626de112ed4c61a246d10314695031 : SUCCESS in 39m 40s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/b612fb540bc34ba3a7c556043a28636a : SUCCESS in 31m 27s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/95ec14ac7ba84c5f9de3b61efb29f077 : SUCCESS in 23m 24s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/e7e27e2e3e6e418b895a24fa70996ca6 : SUCCESS in 23m 47s","accounts_in_message":[],"_revision_number":1},{"id":"24fbd2813dbfd94879f4dcff3ff5f15b5155088c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2026-01-20 15:14:48.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":2},{"id":"a80cd8de25428ba5ce3ed7b0adf5619e1a18d901","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-01-20 15:29:09.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/8e730b4149914dd0a2a9347faba69a4e\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/4d6223c949d945c7a350694da8db345f : SUCCESS in 5m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4fa4ca3721be446ca53a4e5cfb18e49c : SUCCESS in 4m 14s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/d295cec8e1d647e7ae3868e44e67af95 : SUCCESS in 5m 41s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/6b634eb4b4d84025a1acd2060aab1b55 : SUCCESS in 5m 09s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/f44f12e79ef04b2cb4243579d746db97 : SUCCESS in 5m 40s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ee38e656bfb5409197dcac128c7de749 : SUCCESS in 6m 47s","accounts_in_message":[],"_revision_number":2},{"id":"e0ae3bd10d0f601b8d9c4cd85eabc835df2a3f45","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2026-04-30 09:36:37.000000000","message":"Abandoned\n\nstable/2024.2 branch of openstack/watcher transitioned to End of Life and is about to be deleted. To be able to do that, all open patches need to be abandoned.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"2fefb61fc9835af5bfec8b5a2f956e576895e81a","revisions":{"da0fae05a7af57aec639ca610e659b2388341741":{"kind":"REWORK","_number":1,"created":"2025-09-17 08:22:53.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/59/961459/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/59/961459/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/59/961459/1"}}},"commit":{"parents":[{"commit":"669fec0440c734b233828cc660db5a9ec4ffdfd4","subject":"Support zone migration audit without compute_nodes","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/669fec0440c734b233828cc660db5a9ec4ffdfd4"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-07-08 14:59:25.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-09-17 08:22:44.000000000","tz":120},"subject":"Improve unit tests for zone migration strategy","message":"Improve unit tests for zone migration strategy\n\nThe unit tests were mocking part of the Zone Migration strategy class,\nwhich could hide possible bugs. This patch removes this mocking, leaving\nmocked only other classes that are used by the zone migration one.\n\nAdditionally, it includes improved suggestions as follow-up from the\nreview of previous patches, like more explicit comments and additional\nasserts of mocked functions.\n\nAssisted-By: Cursor (Claude-4-sonnet)\nChange-Id: Ie1894311b0e384ab52b1b3dfe0eb50618eef6c9f\nSigned-off-by: jgilaber \u003cjgilaber@redhat.com\u003e\n(cherry picked from commit 821147547816888bf8e63ab8c03635792a0f564f)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/da0fae05a7af57aec639ca610e659b2388341741"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/da0fae05a7af57aec639ca610e659b2388341741"}]},"branch":"refs/heads/stable/2024.2"},"2fefb61fc9835af5bfec8b5a2f956e576895e81a":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2026-01-20 15:14:48.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/59/961459/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/59/961459/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/59/961459/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/59/961459/2"}}},"commit":{"parents":[{"commit":"51a323625a8e58ebb8a4e8cfc4a5f47545811fd8","subject":"Support zone migration audit without compute_nodes","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/51a323625a8e58ebb8a4e8cfc4a5f47545811fd8"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-07-08 14:59:25.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2026-01-20 15:13:33.000000000","tz":60},"subject":"Improve unit tests for zone migration strategy","message":"Improve unit tests for zone migration strategy\n\nThe unit tests were mocking part of the Zone Migration strategy class,\nwhich could hide possible bugs. This patch removes this mocking, leaving\nmocked only other classes that are used by the zone migration one.\n\nAdditionally, it includes improved suggestions as follow-up from the\nreview of previous patches, like more explicit comments and additional\nasserts of mocked functions.\n\nAssisted-By: Cursor (Claude-4-sonnet)\nChange-Id: Ie1894311b0e384ab52b1b3dfe0eb50618eef6c9f\nSigned-off-by: jgilaber \u003cjgilaber@redhat.com\u003e\n(cherry picked from commit 821147547816888bf8e63ab8c03635792a0f564f)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/2fefb61fc9835af5bfec8b5a2f956e576895e81a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/2fefb61fc9835af5bfec8b5a2f956e576895e81a"}]},"branch":"refs/heads/stable/2024.2"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}