)]}'
{"id":"openstack%2Fnova~785582","triplet_id":"openstack%2Fnova~stable%2Fussuri~I212a2db8d71702d330b146dc6f871b402a309e74","project":"openstack/nova","branch":"stable/ussuri","topic":"bug/1919487","attention_set":{},"removed_from_attention_set":{"10135":{"account":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"last_update":"2024-02-07 12:54:28.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I212a2db8d71702d330b146dc6f871b402a309e74","subject":"compute: Reject requests to commit intermediary snapshot of an inactive instance","status":"ABANDONED","created":"2021-04-09 09:16:32.000000000","updated":"2024-02-07 12:54:28.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"53f7c26615887590e0a5e6fc3a13d8a322caf248","_number":785582,"virtual_id_number":785582,"owner":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-04-09 10:59: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":"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":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-04-09 10:59:01.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-04-09 11:33:31.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"},{"updated":"2021-04-09 13:09:50.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":"2024-02-07 12:54:28.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":"c40e1f17d0cc2bb9270c6a9e4d492215e0ad3256","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-04-09 09:16:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"16aa847141f794c4d5544d3c0df45baf610a08bc","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-09 10:59:01.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/990d2a76060941dcaceea3b64a965b1a : SUCCESS in 1h 18m 17s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/ec561863d4544ab99f7481cbe51b296c : SUCCESS in 19m 18s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/df0919a1a09c4d0b8bf365df8e4dd214 : SUCCESS in 15m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/286613a7a9ec4b149d9c15f8e3ecbebe : FAILURE in 8m 57s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b7d6cdda2d294592b22faf9f72605129 : SUCCESS in 15m 18s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/390bdd2341d54cfeaf54a726aafc9b3b : SUCCESS in 13m 42s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/332f96e2e8fc4b3c8ebdccb5ab125084 : SUCCESS in 11m 55s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/60a4341e26de458583334d70cc64f55f : SUCCESS in 56m 54s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/fd926f6b25d444258346b2c2db196a29 : SUCCESS in 1h 27m 52s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/8e85ce0be29a4f78ae2d726ab04dc7af : SUCCESS in 1h 11m 11s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/38578117c2474286aa534eb4bc2c95b3 : SUCCESS in 47m 04s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/06ce4ac0c6f345f38104959251ab6fc3 : SUCCESS in 1h 21m 23s\n- nova-next https://zuul.opendev.org/t/openstack/build/6a57fc9412ab42a8928826a72c388b7f : SUCCESS in 1h 35m 26s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/7563b271b9414250a2d8e90556963242 : SUCCESS in 20m 18s\n- grenade https://zuul.opendev.org/t/openstack/build/bde5bef3ec694a02a2b6ce26579202f4 : SUCCESS in 55m 02s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e3e7774150cf4dd191f6120806207c97 : SUCCESS in 52m 47s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/ef87aaf71fee4c8ab7983975c9c0499e : SUCCESS in 55m 48s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/c59044ee3f734bf0a24864cecd7b74d3 : FAILURE in 27m 30s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"9d4c6727ce4dc45ff6ad1a48e3c3d7b548e39bf0","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-04-09 11:33:31.000000000","message":"Patch Set 1:\n\nBuild succeeded. 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-bionic https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/82/785582/1/check/tempest-dsvm-full-bionic/c5830b7/ : SUCCESS in 2h 08m 46s\n- tempest-dsvm-full-bionic-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/82/785582/1/check/tempest-dsvm-full-bionic-py3/3dd7f50/ : SUCCESS in 2h 11m 15s","accounts_in_message":[],"_revision_number":1},{"id":"7bcf78c22e07e54a9db0faa369376c7d845e4e38","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-04-09 13:09:50.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/785582/1/iscsi : SUCCESS in 2h 55m 32s","accounts_in_message":[],"_revision_number":1},{"id":"e5754a24a961b501e406cefc81be3aaa58a43ec2","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2021-04-26 17:43:46.000000000","message":"Topic set to bug/1919487","accounts_in_message":[],"_revision_number":1},{"id":"53f7c26615887590e0a5e6fc3a13d8a322caf248","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2024-02-07 12:54:28.000000000","message":"Abandoned\n\nstable/ussuri branch of openstack/nova 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":"57a810c0bddb2d5fb8c8e97bba07a1c4eeb2a11a","revisions":{"57a810c0bddb2d5fb8c8e97bba07a1c4eeb2a11a":{"kind":"REWORK","_number":1,"created":"2021-04-09 09:16:32.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/82/785582/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/82/785582/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/82/785582/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/82/785582/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/82/785582/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/82/785582/1"}}},"commit":{"parents":[{"commit":"6b951f9b4d61a7c2a59c6ebbd330c0d7d4efcac0","subject":"Merge \"Add missing exception\" into stable/ussuri","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6b951f9b4d61a7c2a59c6ebbd330c0d7d4efcac0"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-03-17 17:28:05.000000000","tz":0},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-04-09 09:15:07.000000000","tz":60},"subject":"compute: Reject requests to commit intermediary snapshot of an inactive instance","message":"compute: Reject requests to commit intermediary snapshot of an inactive instance\n\nIntroduced by I76eb2e4da027a13525314bd58264f482374d270d the\nos-assisted-volume-snapshots API is only implemented by the libvirt virt\ndriver and should only be called by c-vol as part of an orchestrated\nremotefs based volume snapshot creation or deletion workflow.\n\nWhile not documented clearly in the current api-ref there are code\ncomments within the compute API suggesting that this API can be called\nagainst a volume attached to an instance that is in *any* vm_state.\n\nThis however is not correct when deleting and in turn committing an\nintermediary volume snapshot of an instance that is not running given\nthe current implementation within the libvirt driver. With a request to\nvirDomainBlockCommit being made that fails if the instance and\nunderlying domain is not running.\n\nAdding support for an offline commit isn\u0027t trivial and would be\nconsidered a new feature and not something we could backport on the\nstable branches. As such this change seeks to ensure requests to commit\nan intermediary volume snapshot from an inactive instance are rejected\nquickly and clearly by the compute API to the caller before we cast to\nthe compute.\n\n Conflicts:\n\tnova/tests/unit/compute/test_compute_api.py\n\nNOTE(lyarwood): Test conflict caused by\nI3862a06ca28b383d525bcc9dcbc6fb1d4062f193 renaming test_compute_api.py\nto test_api.py during Victoria.\n\nCloses-Bug: #1919487\nChange-Id: I212a2db8d71702d330b146dc6f871b402a309e74\n(cherry picked from commit 99409375a03e6f923a8b6b00a5dd653bab74caaf)\n(cherry picked from commit 200c743400ed3fd5240fd25d0396eb6180b3f26a)\n(cherry picked from commit 8a41605a78173207746a38350d7fb554f2701b7a)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/57a810c0bddb2d5fb8c8e97bba07a1c4eeb2a11a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/57a810c0bddb2d5fb8c8e97bba07a1c4eeb2a11a"}]},"branch":"refs/heads/stable/ussuri"}},"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":"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":{}}}]}