)]}'
{"id":"openstack%2Fnova~802237","triplet_id":"openstack%2Fnova~stable%2Fvictoria~Iac0e634e66cd4e150a50935cf635f626fc11b70e","project":"openstack/nova","branch":"stable/victoria","topic":"bug/1934770","hashtags":[],"change_id":"Iac0e634e66cd4e150a50935cf635f626fc11b70e","subject":"api: Align availability zone info with forced host","status":"ABANDONED","created":"2021-07-26 08:32:00.000000000","updated":"2023-11-13 17:15:56.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0e04218434eee3e1ea822ec5987eedecd885f781","_number":802237,"virtual_id_number":802237,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-07-26 10:00:40.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":{"approved":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"all":[{"value":2,"date":"2021-08-23 17:50:12.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"_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":2},"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-07-26 10:00:40.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":"2021-07-26 10:56:58.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"CC"},{"updated":"2021-08-23 17:50:12.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":"REVIEWER"},{"updated":"2021-08-23 18:16:11.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"CC"}],"messages":[{"id":"a424184c177c7142b4afb048a39d207ff349c6af","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-07-26 08:32:00.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5ecfb44861cdb2df89ec4d0e9d6842eae1745e35","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-26 10:00:40.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\n\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/25c4839724a944eaa159e59466313106 : SUCCESS in 1h 11m 38s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/60a0c4c64f574bf8858916a1979a0354 : SUCCESS in 17m 49s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/73cb538c552c4e129a9debc1045c1f7b : SUCCESS in 15m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c58aefdac1ba4e0cbb494130e072710a : FAILURE in 9m 34s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/fee12e2288db4705b5fa378e379bda53 : SUCCESS in 14m 51s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/0d309350a58641e1a9a7cdde41256b1c : SUCCESS in 14m 33s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dcf42f680a4b480386d8995d6a1197a0 : SUCCESS in 12m 50s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/23c0a9b06735468da751db31d06b1629 : SUCCESS in 1h 21m 16s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/03acbabc47ab472fb0043e45db817221 : SUCCESS in 1h 17m 29s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/3e208b212ae74a499f5f84b64644a766 : SUCCESS in 53m 44s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/ece06b58e7ef4b26b26f4ee406e3da75 : SUCCESS in 1h 26m 19s\n- nova-next https://zuul.opendev.org/t/openstack/build/f5fca1035b8c454fb88d5903c6a050b2 : SUCCESS in 1h 24m 47s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/95e1fae9ee814bb29726c07f1fa941c2 : SUCCESS in 17m 30s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/5a62f41a945048629c50eb73b6801f8a : SUCCESS in 1h 10m 56s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f202b6d9032c46dab89379c3fae2c225 : SUCCESS in 54m 45s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/e80c2b758c5147d296c4b2b1b8c0779e : SUCCESS in 57m 29s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/327ce2a06e464fd7ad4e8ebd4fa5e213 : FAILURE in 29m 17s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/e755cc8852fa454a81f50ae341a0ca54 : SUCCESS in 39m 58s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a7bad85baa7669b4e9796f23490973fde5ba3da3","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-07-26 10:56:58.000000000","message":"Patch Set 1:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/802237/1/iscsi : SUCCESS in 2h 23m 25s","accounts_in_message":[],"_revision_number":1},{"id":"017ada402671ca220ff63561937ee94c977c058a","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2021-08-23 17:50:12.000000000","message":"Patch Set 1: Code-Review+2\n\nClean cherry pick from Wallaby, where the patch has merged. LGTM.","accounts_in_message":[],"_revision_number":1},{"id":"bf85659b9cb88348919897d2c8de21b520511515","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2021-08-23 17:50:19.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"38094a703cac8f50d0af48b8744e96403d5d78df","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-08-23 18:16:11.000000000","message":"Patch Set 1:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 finger://zuul-v3-executor.novalocal/b618e1789d824f1ab0d6b0f1e5bcc330 : RETRY_LIMIT in 1m 00s","accounts_in_message":[],"_revision_number":1},{"id":"de0c7bf88462dcea2e4eb87c568b6257cdcb4050","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-23 19:34:51.000000000","message":"Patch Set 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\n\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/b4c32a67ef6b40c682fa64889ea3941b : SUCCESS in 1h 20m 02s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/8666ce05c30949799c18b540c696d38e : SUCCESS in 16m 45s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f900174407f44ae6bbac6c213eebaa28 : SUCCESS in 14m 09s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/904236a388454da298990fec8bc85f42 : SUCCESS in 9m 42s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bd917b524a1048cdbcf4f68b44467b91 : SUCCESS in 12m 20s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/9cddb33bc1cb42e4a519b81676582efc : SUCCESS in 14m 11s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8f30bbc41bb944a5bb03634e4c2ce443 : SUCCESS in 11m 16s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/8aaecab05f124058839d8d935d3ed8d3 : SUCCESS in 1h 08m 43s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/e57d01ae0c614c20a030d8aef68e9cc6 : SUCCESS in 1h 16m 31s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/90adc222ccea49f88819038ba30f32fd : FAILURE in 54m 29s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/37a8555917c2427aa6bfc9f668dffc91 : SUCCESS in 1h 21m 08s\n- nova-next https://zuul.opendev.org/t/openstack/build/38e86562355b4d67a27dcaafb4cc5792 : SUCCESS in 1h 41m 29s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/7686b4df40cb4f7cb4b36ab5a23d133e : SUCCESS in 6m 33s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/dee3b5f9d5a54f9c9bb9fef270ebbb1c : SUCCESS in 19m 11s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/47baec4d506e4bfab8499dfff83be6b7 : SUCCESS in 1h 14m 30s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/65fc749169f44d3aba13ecd0eb6ece4a : SUCCESS in 56m 10s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/aca89622018a47f5ac9faf790d5ba592 : SUCCESS in 53m 16s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/0848bb8939154b0eb258aa515b6623d7 : SUCCESS in 26m 53s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/aa35709ad62246fdabb52c60867d281b : SUCCESS in 40m 12s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"d1fbb2bc83f2d3692995e30d8a5d8bad90e1221a","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-08-23 22:05:20.000000000","message":"Patch Set 1:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/802237/1/iscsi : SUCCESS in 3h 05m 53s","accounts_in_message":[],"_revision_number":1},{"id":"0e04218434eee3e1ea822ec5987eedecd885f781","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2023-11-13 17:15:56.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"bf235c0ed0a1a867ab1b6e5ff34eaf7a8dee6c55","revisions":{"bf235c0ed0a1a867ab1b6e5ff34eaf7a8dee6c55":{"kind":"REWORK","_number":1,"created":"2021-07-26 08:32:00.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/37/802237/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/802237/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/802237/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/802237/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/802237/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/802237/1"}}},"commit":{"parents":[{"commit":"4135970d3c62895662bbca5055c3a5ae4099bf7d","subject":"Merge \"Stop leaking ceph df cmd in RBD utils\" into stable/victoria","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4135970d3c62895662bbca5055c3a5ae4099bf7d"}]}],"author":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-06-25 17:51:06.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-07-26 08:31:49.000000000","tz":60},"subject":"api: Align availability zone info with forced host","message":"api: Align availability zone info with forced host\n\nUsers can create a server like so:\n\n  $ openstack server create --availability-zone az:host ...\n\nThis is a historical way to request that an instance be scheduled to a\nspecific host and it causes the scheduler to be bypassed. However, no\nvalidation of this availability zone-host combo takes place. The host\ncould in fact belong to a different availability zone. If it does, we\u0027ll\nend up in a very odd situation whereby the RequestSpec record for the\ninstance will record the availability zone requested by the user at\ncreate time, but the Instance record itself will record the availability\nzone of the host on which the instance was scheduled. This leads to even\nmore confusing behavior when we attempt to do something like live\nmigrate the instance since the RequestSpec record, with its original and\npossibly invalid availability zone information, is used. The\n\u0027AvailabilityZoneFilter\u0027 will fail an error message like the following:\n\n  Availability Zone \u0027foo\u0027 requested. ... has AZs: bar\n\nbut the \u0027openstack server list --long\u0027 command will show a non-foo value\nfor the availability zone column.\n\nThe solution is simple: when given an availability zone-host combo, make\nsure the availability zone requested matches that of the host (or, more\nspecifically, the host is a member of the host aggregates that form the\navailability zone [1]). If not, simply ignore the requested availability\nzone information in favour of using the availability zone of the host,\nlogging a warning just for record keeping purposes. This is deemed\npreferable to failing with HTTP 400 (Bad Request) since what users are\nreally requesting by using this was to schedule to a specific host: the\navailability zone portion of the request is really irrelevant and just\nan artifact of this legacy mechanism to request hosts. If users wish to\ntruly validate a host-availability zone combo, they can use the \u0027host\u0027\nfield introduced in microversion 2.74 along with the \u0027availability_zone\u0027\nfield:\n\n  $ openstack server create --availability-zone az --host host ...\n\n[1] https://docs.openstack.org/nova/latest/admin/aggregates.html\n\nChange-Id: Iac0e634e66cd4e150a50935cf635f626fc11b70e\nSigned-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e\nCloses-Bug: #1934770\n(cherry picked from commit 8f21ee42bd66b62e75e14acf4e91b786d78b9168)\n(cherry picked from commit 58782403cdaad33856fd59715525cee3c63ee3cf)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bf235c0ed0a1a867ab1b6e5ff34eaf7a8dee6c55"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bf235c0ed0a1a867ab1b6e5ff34eaf7a8dee6c55"}]},"branch":"refs/heads/stable/victoria"}},"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":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"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":{}}}]}