)]}'
{"id":"openstack%2Fwatcher~960736","triplet_id":"openstack%2Fwatcher~stable%2F2024.2~I305c58e47093c4a884e86f1d91fdc15ef2a1cfba","project":"openstack/watcher","branch":"stable/2024.2","hashtags":[],"change_id":"I305c58e47093c4a884e86f1d91fdc15ef2a1cfba","subject":"Handle missing dst_pool parameter in zone_migration","status":"ABANDONED","created":"2025-09-12 08:47:28.000000000","updated":"2026-04-30 09:34:09.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"46679dcd3fa99ef82e634b09e99637607bcf135c","_number":960736,"virtual_id_number":960736,"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 16:32:01.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":"2025-09-12 10:43:52.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:34:09.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":"987c8f6e88097b773ad39feac1a6f9f831e4b166","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-09-12 08:47:28.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"0b4879e4374d8c8bdd3c83251cccbfa6e19d180f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-09-12 08:50:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"908eb3af239c65b48a50b996a0d0a6b5805a75cc","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:27:40.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e5fda0804564f7355094f9d7b810f05360363148","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:29:26.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"1c250cfad3b82a9c012849d4a3522ae279d61c21","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:43:21.000000000","message":"Uploaded patch set 5: Patch Set 4 was rebased.","accounts_in_message":[],"_revision_number":5},{"id":"f5f88fda6163aedaa8c973c0d3a0784cf7a3a718","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-12 10:43:52.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/2a416b25dade4c90b207a43184fa6155\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/9d2b75bfbd584528993f8a1fedbb0ac8 : SUCCESS in 4m 03s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8ddf382d17554c5387aabcaf872b0ea9 : SUCCESS in 3m 01s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/3643be361b9045cba04d5f503c09698e : SUCCESS in 2m 50s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/24e9de8db4b045f9b77cdc06412e2069 : SUCCESS in 6m 23s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/f765a88c1b384c908353aef3e835478e : SUCCESS in 3m 08s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/78fd91720a8a45d4a3dcc7de35c10368 : SUCCESS in 5m 44s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/52003a516f8b42ccb5d460f4a4f43429 : SUCCESS in 4m 33s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/9be7919e32aa4b308796178b5390ddc6 : SUCCESS in 12m 49s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/7abc617160aa47589b230f1781364810 : SUCCESS in 53m 38s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/7778de15d4564cf0b9f908cf1841e97d : SUCCESS in 49m 05s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/9f1bd15f1dcb46e4b0fe6ea15ee596f5 : SUCCESS in 42m 25s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/c0c07d2c35d64daea698dcafb400f592 : SUCCESS in 28m 59s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/b8c84e06055a4b9cba838907056ca6ee : SUCCESS in 12m 49s","accounts_in_message":[],"_revision_number":5},{"id":"5651b8cb1b38aa7b100d9a24fd643240e888179f","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 6.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":6},{"id":"594aaa701a94d2f74ce834b9ca6670fc503d9b70","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-17 10:26:22.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/1f1742eda08f464398197381d2ad266e\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/6cac7d86e4674a339a2a5286f940a8fa : SUCCESS in 5m 02s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4af62e9962004a758376308ea09fd67f : SUCCESS in 3m 11s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/afadc4a877844f199b011860ab458073 : SUCCESS in 4m 01s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/f8e515d2da834e58bc12a7a003edd56a : SUCCESS in 4m 52s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/145f35a2292f4443a2c6d0b9d7ba766b : SUCCESS in 5m 06s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f7bf74da514b4d36a74cf48a9fe395d4 : SUCCESS in 7m 23s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/689083826c8149fc92dbf4128fa1812e : SUCCESS in 5m 51s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/f76e78f1826f4f0e97f52839c78c71f5 : SUCCESS in 27m 03s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/b44b862f5d7e470ab41adcbd72b43fb6 : SUCCESS in 32m 28s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/07217bb3fd4b474eb93dd65332384056 : SUCCESS in 40m 28s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/6dc3139bfe1c482884be0c8a902c7022 : SUCCESS in 43m 34s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/4f9be833dc6544d887e9b5c6fa4c7b0d : SUCCESS in 37m 22s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/74b05a3cf7af47dcb73b62d15b9efb79 : SUCCESS in 27m 10s","accounts_in_message":[],"_revision_number":6},{"id":"8085605f0c8004f368d3c46edfe0daf2b8454bf5","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 7: Patch Set 6 was rebased.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":7},{"id":"63357da5117afae3e64b8f23ca0dce614504b801","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-01-20 16:32:01.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/c57aac35736f4a15a98de098705503c1\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/ce787f1553db4158901baf012c8a4ccb : SUCCESS in 6m 09s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/6b2d40677e9c4d3394a5be025b8e3aed : SUCCESS in 3m 13s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/2dc9296c80614b6c90ad27cdc23bf99d : SUCCESS in 3m 31s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/72f244525e5e4c219f5308c7660574d8 : SUCCESS in 5m 00s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/95c773a4899445dca4f38cf1f3f39383 : SUCCESS in 4m 27s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/cc005c497e484c8d9490bdab5be1b9fb : SUCCESS in 6m 09s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/de1ab71535894b43aceee097e3aafb4e : SUCCESS in 4m 40s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/0e9c42452ea94e92bd3aeb3d5bdbd9b1 : SUCCESS in 33m 01s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/eb9b9cec6613499b93d32242da4c8c83 : SUCCESS in 1h 10m 29s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/b0988a1cc59b448c8a81b428f45b9e74 : SUCCESS in 33m 19s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/33042ab912bc4e84a263578d4319afa2 : SUCCESS in 29m 47s","accounts_in_message":[],"_revision_number":7},{"id":"46679dcd3fa99ef82e634b09e99637607bcf135c","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2026-04-30 09:34:09.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":7}],"current_revision_number":7,"current_revision":"570a53ead8e2ac02e21a0d9224be98a715f66555","revisions":{"102a57eb406dc2296244c4c4141acb63b9f7fd51":{"kind":"REWORK","_number":1,"created":"2025-09-12 08:47:28.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/36/960736/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/1"}}},"commit":{"parents":[{"commit":"945fada5a7b0836bbd7e080d1f81c3dc75c0c8a1","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/945fada5a7b0836bbd7e080d1f81c3dc75c0c8a1"}]}],"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 08:47:19.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/102a57eb406dc2296244c4c4141acb63b9f7fd51"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/102a57eb406dc2296244c4c4141acb63b9f7fd51"}]},"branch":"refs/heads/stable/2024.2"},"c4ee6e9cebccfdb2c81149cbd66a6d286e39daf3":{"kind":"REWORK","_number":2,"created":"2025-09-12 08:50:17.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/36/960736/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/2"}}},"commit":{"parents":[{"commit":"945fada5a7b0836bbd7e080d1f81c3dc75c0c8a1","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/945fada5a7b0836bbd7e080d1f81c3dc75c0c8a1"}]}],"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 08:50:07.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/c4ee6e9cebccfdb2c81149cbd66a6d286e39daf3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/c4ee6e9cebccfdb2c81149cbd66a6d286e39daf3"}]},"branch":"refs/heads/stable/2024.2"},"bde2a3b8f0a13462d4f8f74f3c3e0bd4bf5b827a":{"kind":"REWORK","_number":3,"created":"2025-09-12 09:27:40.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/36/960736/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/3"}}},"commit":{"parents":[{"commit":"8289d873fe55230162616412a8d74c1f994d4497","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8289d873fe55230162616412a8d74c1f994d4497"}]}],"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:27:03.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/bde2a3b8f0a13462d4f8f74f3c3e0bd4bf5b827a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/bde2a3b8f0a13462d4f8f74f3c3e0bd4bf5b827a"}]},"branch":"refs/heads/stable/2024.2"},"f8ee0973eca434caa333673b6e5b1baa6a42c705":{"kind":"REWORK","_number":4,"created":"2025-09-12 09:29:26.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/36/960736/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/4"}}},"commit":{"parents":[{"commit":"8289d873fe55230162616412a8d74c1f994d4497","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8289d873fe55230162616412a8d74c1f994d4497"}]}],"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:28:48.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/f8ee0973eca434caa333673b6e5b1baa6a42c705"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f8ee0973eca434caa333673b6e5b1baa6a42c705"}]},"branch":"refs/heads/stable/2024.2"},"cde9e511b2deb375eb2a31d431d10529a2a30047":{"kind":"TRIVIAL_REBASE","_number":5,"created":"2025-09-12 09:43:21.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/36/960736/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/5"}}},"commit":{"parents":[{"commit":"69c6d498b3a1cc0e83a6e7a4e107175ccca6d2e4","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/69c6d498b3a1cc0e83a6e7a4e107175ccca6d2e4"}]}],"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:43:04.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/cde9e511b2deb375eb2a31d431d10529a2a30047"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/cde9e511b2deb375eb2a31d431d10529a2a30047"}]},"branch":"refs/heads/stable/2024.2"},"b53bec6352793edc0443f406304b04d8556f7010":{"kind":"REWORK","_number":6,"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/36/960736/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/6"}}},"commit":{"parents":[{"commit":"15c1a1002ef9bc16d27b8c655584a20eb5e8e620","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/15c1a1002ef9bc16d27b8c655584a20eb5e8e620"}]}],"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-17 08:22:44.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/b53bec6352793edc0443f406304b04d8556f7010"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/b53bec6352793edc0443f406304b04d8556f7010"}]},"branch":"refs/heads/stable/2024.2"},"570a53ead8e2ac02e21a0d9224be98a715f66555":{"kind":"TRIVIAL_REBASE","_number":7,"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/36/960736/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/36/960736/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/36/960736/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/36/960736/7"}}},"commit":{"parents":[{"commit":"e7b2665ada3e20ffb8b4ae7263ba31f771973e8b","subject":"Add unit test zone migration with_attached_volume","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/e7b2665ada3e20ffb8b4ae7263ba31f771973e8b"}]}],"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":"2026-01-20 15:13:33.000000000","tz":60},"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/570a53ead8e2ac02e21a0d9224be98a715f66555"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/570a53ead8e2ac02e21a0d9224be98a715f66555"}]},"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":""}}}]}