)]}'
{"id":"openstack%2Fwatcher~960768","triplet_id":"openstack%2Fwatcher~stable%2F2024.1~I305c58e47093c4a884e86f1d91fdc15ef2a1cfba","project":"openstack/watcher","branch":"stable/2024.1","attention_set":{},"removed_from_attention_set":{"34452":{"account":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"last_update":"2025-10-31 13:54:41.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I305c58e47093c4a884e86f1d91fdc15ef2a1cfba","subject":"Handle missing dst_pool parameter in zone_migration","status":"ABANDONED","created":"2025-09-12 09:40:23.000000000","updated":"2025-10-31 13:54:41.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2a79eac9a662978005e03fbed7a7c8b62247ad04","_number":960768,"virtual_id_number":960768,"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":"2025-09-12 11:32:17.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-12 10:31:22.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":"2025-10-31 13:54:41.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":"3f7ee1dfc1235ef9d01281a28b5746fd561454b7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-09-12 09:40:23.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1805b33469804aa5abe5c200af462f6ad6548c77","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-12 10:31:22.000000000","message":"Patch Set 1: Verified-1\n\n(2 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/3233263b2d434098b3dbd65c6af439bf\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/15ab57c924024e16a2403d0969121151 : SUCCESS in 5m 33s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/86771a7549fb49548f493378dfd8d46f : FAILURE in 4m 28s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/1855f236a45641ff90a9b04ccaf1951d : SUCCESS in 4m 16s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/e1c895bb01c34a9ebbfcdd7d20cd8a2c : SUCCESS in 5m 01s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/fa6e4518fe524af9800cb6164f86c0f6 : SUCCESS in 6m 28s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/f581db8008aa4a41b5dab4caee3fa84f : SUCCESS in 4m 01s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/3bf8a3a4102b4a9d9e1fabcaaaf2f10e : SUCCESS in 24m 33s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/bdb4f27e0d7844978fae47e94f5755be : SUCCESS in 44m 34s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/497d57217e2f4f6a91e07f757df1f2c3 : SUCCESS in 38m 07s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/183486d1cba6483fbe8e121b7be9f6fe : SUCCESS in 40m 29s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/559f4964e1e04af18c1b83e4874ddc93 : SUCCESS in 29m 39s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/2438533437404a5887d1e795f311d024 : SUCCESS in 24m 17s","accounts_in_message":[],"_revision_number":1},{"id":"22b56430e9ecae266cd0601f446cab075bb2f955","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-09-12 10:49:38.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Verified-1\n","accounts_in_message":[],"_revision_number":2},{"id":"46306d7bccaa45784630c06c170f2ced54e60034","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-12 11:32:17.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/6de952df0b964b47bba4ae3f0e656a15\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/f929e340efb24d45a21b40a1a129ea6f : SUCCESS in 3m 00s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f334b359afd545d88d937ae7c4f0d25f : SUCCESS in 3m 30s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a07b7526be234e12a2979a9d821a0588 : SUCCESS in 4m 28s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/eb368e6b13dd42d9becbbc6fa75a7687 : SUCCESS in 4m 58s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f50011ae2daa475b991381cf3d7a96fa : SUCCESS in 6m 31s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/c4a41cf704404af6ac7a97496d8bfcfa : SUCCESS in 2m 32s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/b649c1675aad4e849f63df6212b55884 : SUCCESS in 26m 54s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/f9c26db5f2884473ac0e64edd791f6cb : SUCCESS in 24m 06s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/828b47266a374cbc81addfa667c7730e : SUCCESS in 24m 13s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/45434118c6584c31804e625553e83e20 : SUCCESS in 35m 36s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/561256f524a1400683327c10c3abfb69 : SUCCESS in 12m 47s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/7ed9dd450f8647b287d9cffb4f41c35f : SUCCESS in 28m 53s","accounts_in_message":[],"_revision_number":2},{"id":"2a79eac9a662978005e03fbed7a7c8b62247ad04","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2025-10-31 13:54:41.000000000","message":"Abandoned\n\nstable/2024.1 branch of openstack/watcher is about to be deleted. To be able to do that, all open patches need to be abandoned. Please cherry pick the patch to unmaintained/2024.1 if you want to further work on this patch.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"5c07af4d625beb062f6a5c1135b2312279ab205c","revisions":{"b0020b9ed44c6ef58b62ddcc46679fb9d2a96e40":{"kind":"REWORK","_number":1,"created":"2025-09-12 09:40:23.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/68/960768/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/68/960768/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/68/960768/1"}}},"commit":{"parents":[{"commit":"f7287cc2696fa733d973a425ecab78d84cdd97c9","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f7287cc2696fa733d973a425ecab78d84cdd97c9"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-16 11:46:57.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-09-12 09:40:14.000000000","tz":120},"subject":"Handle missing dst_pool parameter in zone_migration","message":"Handle missing dst_pool parameter in zone_migration\n\nUnlike Nova, Cinder does not support calling the \u0027os-migrate_volume\u0027[1]\naction without a host or a cluster. For volume migrations of type\n\u0027migrate\u0027 in watcher the dst_pool is required, but for other migrations\nthat migrate the volumes to different types is not needed. This\nchange checks if the dst_pool is defined and prevents some migrations\nwhen it\u0027s misssing information.\n\nAdds testing for creating audits with the Zone Migration status,\nvalidating the schema changes.\n\n[1] https://docs.openstack.org/api-ref/block-storage/v3/index.html#migrate-a-volume\n\nCloses-Bug: 2108988\n\nChange-Id: I305c58e47093c4a884e86f1d91fdc15ef2a1cfba\nSigned-off-by: jgilaber \u003cjgilaber@redhat.com\u003e\n(cherry picked from commit fe56660c444216bf235b2404ddeeba5b91c9cb8c)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/b0020b9ed44c6ef58b62ddcc46679fb9d2a96e40"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/b0020b9ed44c6ef58b62ddcc46679fb9d2a96e40"}]},"branch":"refs/heads/stable/2024.1"},"5c07af4d625beb062f6a5c1135b2312279ab205c":{"kind":"REWORK","_number":2,"created":"2025-09-12 10:49:38.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/68/960768/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/68/960768/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/68/960768/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/68/960768/2"}}},"commit":{"parents":[{"commit":"f7287cc2696fa733d973a425ecab78d84cdd97c9","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f7287cc2696fa733d973a425ecab78d84cdd97c9"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-16 11:46:57.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-09-12 10:49:28.000000000","tz":120},"subject":"Handle missing dst_pool parameter in zone_migration","message":"Handle missing dst_pool parameter in zone_migration\n\nUnlike Nova, Cinder does not support calling the \u0027os-migrate_volume\u0027[1]\naction without a host or a cluster. For volume migrations of type\n\u0027migrate\u0027 in watcher the dst_pool is required, but for other migrations\nthat migrate the volumes to different types is not needed. This\nchange checks if the dst_pool is defined and prevents some migrations\nwhen it\u0027s misssing information.\n\nAdds testing for creating audits with the Zone Migration status,\nvalidating the schema changes.\n\n[1] https://docs.openstack.org/api-ref/block-storage/v3/index.html#migrate-a-volume\n\nCloses-Bug: 2108988\n\nChange-Id: I305c58e47093c4a884e86f1d91fdc15ef2a1cfba\nSigned-off-by: jgilaber \u003cjgilaber@redhat.com\u003e\n(cherry picked from commit fe56660c444216bf235b2404ddeeba5b91c9cb8c)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/5c07af4d625beb062f6a5c1135b2312279ab205c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/5c07af4d625beb062f6a5c1135b2312279ab205c"}]},"branch":"refs/heads/stable/2024.1"}},"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":{}}},{"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":{}}},{"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":{}}}]}