)]}'
{"id":"openstack%2Fwatcher~979125","triplet_id":"openstack%2Fwatcher~stable%2F2024.2~I05f8d66c6a79e0cff356ac83b88b28942f14a9a9","project":"openstack/watcher","branch":"stable/2024.2","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":"2026-04-30 09:32:46.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I05f8d66c6a79e0cff356ac83b88b28942f14a9a9","subject":"Fix zone migration dst pool and type selection","status":"ABANDONED","created":"2026-03-06 09:09:46.000000000","updated":"2026-04-30 09:32:46.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6503d5fca337a2e5e5a3c7c75b0eff1923355644","_number":979125,"virtual_id_number":979125,"owner":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2026-03-06 10:01:14.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"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-03-06 10:01:14.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:32:46.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":"f9949dee9c79e1197a25f9cc31c87e9dce29f6d6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2026-03-06 09:09:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b9680c5ab47dd32c7f22d1f9677b9a9f53edacae","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-03-06 10:01:14.000000000","message":"Patch Set 1: Verified-1\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/df7caa66ccc741ae9684bfdb1ab55beb\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/888fcde057394d1b9e45e23b62081144 : FAILURE in 3m 45s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f3831c94e3864f42991e3b7c451866e5 : FAILURE in 2m 33s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/e76198382c7543db8918e53e0d89f7a5 : FAILURE in 3m 38s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/b5d79687f1644f3aa8a5e51a2fe66aed : FAILURE in 6m 57s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/a4e21955a16b4569a227cd765ce9509b : FAILURE in 2m 28s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1cb20e955fcb4f0d8c1191dac75064a1 : FAILURE in 3m 55s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/6beb02d4c4fd4fcaa77eadb803b59c80 : SUCCESS in 2m 30s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/0e5cf203907f4540830ccccd74a0b9d1 : SUCCESS in 48m 28s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/34c5fc68e1894a779636b1ea1e22f062 : SUCCESS in 37m 43s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/59a7f05a5efe4b40ab0471e1e8637ef4 : SUCCESS in 28m 58s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/6ae783b6ac8443508b848e2e09a88e55 : SUCCESS in 33m 11s","accounts_in_message":[],"_revision_number":1},{"id":"6503d5fca337a2e5e5a3c7c75b0eff1923355644","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2026-04-30 09:32:46.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":1}],"current_revision_number":1,"current_revision":"f8b69e06a1db9b1670d9f5edcc61a5683d15e9ba","revisions":{"f8b69e06a1db9b1670d9f5edcc61a5683d15e9ba":{"kind":"REWORK","_number":1,"created":"2026-03-06 09:09:46.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/25/979125/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/25/979125/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/25/979125/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/25/979125/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/25/979125/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/25/979125/1"}}},"commit":{"parents":[{"commit":"6312a34dbf91dafe3543c013049fd4b4c8de7698","subject":"Add tests for get_dst_pool_and_type method","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6312a34dbf91dafe3543c013049fd4b4c8de7698"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-10-23 17:16:02.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2026-03-06 09:09:37.000000000","tz":60},"subject":"Fix zone migration dst pool and type selection","message":"Fix zone migration dst pool and type selection\n\nThe get_dst_pool_and_type method was always selecting the first\nmatching storage pool based only on src_pool, ignoring the\nsrc_type parameter. This caused incorrect volume migrations when\nthe user supplied multiple values for storage_pools with the same\nsource pool but different source types.\n\nThis change implements a two-pass matching approach:\n1. First pass attempts to match both src_pool and src_type for\n   exact matches\n2. Second pass falls back to matching just src_pool if no exact\n   match is found\n\nTests are updated to verify the correct behavior including the more\nrestrictive matching logic.\n\nGenerated-By: claude-code (clause-sonnet-4.5)\nCloses-Bug: 2129692\nChange-Id: I05f8d66c6a79e0cff356ac83b88b28942f14a9a9\nSigned-off-by: jgilaber \u003cjgilaber@redhat.com\u003e\n(cherry picked from commit 16446385dd840b5cc072e3453e271e726aa38a72)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f8b69e06a1db9b1670d9f5edcc61a5683d15e9ba"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f8b69e06a1db9b1670d9f5edcc61a5683d15e9ba"}]},"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":""}}}]}