)]}'
{"id":"openstack%2Fnova~806293","triplet_id":"openstack%2Fnova~master~I277ba12a676fe8ecd77b25456d1a08c81360d64a","project":"openstack/nova","branch":"master","topic":"bug/1836754","hashtags":[],"change_id":"I277ba12a676fe8ecd77b25456d1a08c81360d64a","subject":"Revert \"Consumer gen support for delete instance allocations\"","status":"ABANDONED","created":"2021-08-27 01:32:54.000000000","updated":"2021-08-27 17:00:15.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a22110dd9871b7c4e80b96b857d0eec711184fb6","_number":806293,"virtual_id_number":806293,"owner":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-08-27 03:03:55.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":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-08-27 02:01:39.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":"CC"},{"updated":"2021-08-27 03:03:55.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-08-27 03:04:40.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-08-27 03:44:20.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-27 07:06:03.000000000","updated_by":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"reviewer":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"state":"CC"}],"messages":[{"id":"d15bbbaa0e776d902f9aaa9161135ac4b026097b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-08-27 01:32:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"0aa3db28b4364d351981af230c4825c6734e921d","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-27 02:01:39.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/49599647c85045ecb78e869aa9eb7a8b : SUCCESS in 24m 13s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/f2b77fc5a4544d7ea77db0a0bdbf68ae : SUCCESS in 26m 51s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a77d8d1ae98c81e6e4e3aa63ba87edf9a68d8fb0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-27 03:03:55.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/a91c271072724b70939475b97ae458e1 : SUCCESS in 1h 11m 48s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/3e4eb74811364b9bbe182d783ea87c50 : SUCCESS in 18m 25s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f3aafa5461564b91aa1597272aa24ccb : SUCCESS in 19m 40s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/91502f9307154ac592648cd354c0255a : SUCCESS in 7m 32s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5cc2bf33cd94485ea71c84d9005b22a4 : SUCCESS in 14m 42s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/3c4f9207734a4dbf822fa6434378cf41 : SUCCESS in 16m 48s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/50479f38fba748f1b1f29c484ff50fd6 : SUCCESS in 16m 28s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ff3d879b3daa4c64af904d780f0e2ed0 : SUCCESS in 10m 22s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/30352aaf4cc94611920c4f21ae2be405 : SUCCESS in 1h 12m 00s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/8c6b6db5507f40509a02bf6baba95939 : SUCCESS in 56m 52s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/b97828d060e74cfb8dafdc6b06feb6bb : SUCCESS in 53m 44s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/6e0bef4124be43f0947085371581ed7f : SUCCESS in 51m 52s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/573050fc150d45a7a785148075de395a : SUCCESS in 1h 18m 58s\n- nova-next https://zuul.opendev.org/t/openstack/build/85c4e681f1ce406fb1018fb01ac18cfc : SUCCESS in 1h 23m 12s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/048b7d538ac8487c8a32bcfb4bf1cc94 : SUCCESS in 5m 58s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/24a4c5d6926942ecb5f79b836553aa11 : SUCCESS in 20m 16s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/a54175be3ca544ef97ed5a343e6e58b1 : SUCCESS in 19m 29s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3db78848c9894195a3d17bb41cbd0bc4 : SUCCESS in 16m 37s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/19eebebcb7e643cfb732459f45fd1775 : SUCCESS in 1h 28m 17s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/3ae7be3852084a9fae8092f25f0e8c70 : SUCCESS in 56m 30s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/eaac682ab740451fa2fc3639dd3b9a9f : SUCCESS in 40m 25s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/1435e18d5b774735bd3513c53869ece1 : SUCCESS in 37m 17s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/475ff214ba584c8d9b24fb6b3670348e : SUCCESS in 48m 46s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3be616e12896288ea1d9906af8d9e293f120000c","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-08-27 03:04:40.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA http://publiclogs.emc.com/93/806293/1/check/EMC_PowerFlex_NOVA/22b86a6/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":1},{"id":"26e0b96a2f0915543b88347481a17e64f8e1c878","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-27 03:44: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/806293/1/iscsi : SUCCESS in 2h 09m 20s","accounts_in_message":[],"_revision_number":1},{"id":"a22110dd9871b7c4e80b96b857d0eec711184fb6","tag":"autogenerated:gerrit:abandon","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-08-27 17:00:15.000000000","message":"Abandoned\n\nConsensus was to use https://review.opendev.org/c/openstack/nova/+/688802","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"c9a0b17423bbbf03213fe6f2aa2d7e9c1eee8808","revisions":{"c9a0b17423bbbf03213fe6f2aa2d7e9c1eee8808":{"kind":"REWORK","_number":1,"created":"2021-08-27 01:32:54.000000000","uploader":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"ref":"refs/changes/93/806293/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/93/806293/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/93/806293/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/93/806293/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/93/806293/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/93/806293/1"}}},"commit":{"parents":[{"commit":"9da8604f36e3360343fc0c29cae8c9341ee45285","subject":"Revert \"consumer gen: more tests for delete allocation cases\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9da8604f36e3360343fc0c29cae8c9341ee45285"}]}],"author":{"name":"melanie witt","email":"melwittt@gmail.com","date":"2021-08-26 23:49:57.000000000","tz":0},"committer":{"name":"melanie witt","email":"melwittt@gmail.com","date":"2021-08-27 01:29:36.000000000","tz":0},"subject":"Revert \"Consumer gen support for delete instance allocations\"","message":"Revert \"Consumer gen support for delete instance allocations\"\n\nThis reverts commit 6f1a1f5e8ead9ceb145e13927cd9ae7fdca9add4.\n\nConflicts:\n    doc/source/user/placement.rst\n    nova/api/openstack/compute/servers.py\n    nova/scheduler/client/report.py\n    nova/tests/functional/test_servers.py\n    nova/tests/unit/compute/test_compute.py\n    nova/tests/unit/scheduler/client/test_report.py\n\nNOTE(melwitt): The conflicts are because:\n\n  * Placement docs moved to the openstack/placement repo around the\n    time of I218a6d11fea934e8991e41b4b36203c6ba3e3dbf (Use links to\n    placement docs in nova docs)\n\n  * I1dd6abcc2be17ff76f108e7ff3771314f33259c6 (Stop handling\n    \u0027InstanceUnknownCell\u0027 exception) landed after the patch we are\n    reverting\n\n  * I1dea4472b232d6c054879ebda2536658d9769053 (Fix wrong consumer type\n    in logging) landed after the patch we are reverting\n\n  * I456f685f480b8de71014cf232a8f08c731605ad8 (Create a fixture around\n    fake_notifier) landed after the patch we are reverting\n\n  * I6daea47988181dfa6dde3d9c42004c0ecf6ae87a (functional: Add and use\n    \u0027GlanceFixture\u0027) landed after the patch we are reverting\n\n  * Iad2e4da4546b80a016e477577d23accb2606a6e4 (db: Unify \u0027nova.db.api\u0027,\n    \u0027nova.db.sqlalchemy.api\u0027) landed after the patch we are reverting\n\n  * I680f0bab766693b8cc6f0f912f0b7987a9debec9 (trivial: Remove unused\n    \u0027_instance_to_allocations_dict\u0027 function) landed after the patch we\n    are reverting\n\nNOTE(melwitt): In nova/tests/unit/compute/test_compute_mgr.py,\ntest_confirm_snapshot_based_resize_at_source_delete_alloc_fails was\nalso removed because change I0851e2d54a1fdc82fe3291fb7e286e790f121e92\n(Drop legacy cold migrate allocation compat code) that landed after the\npatch we are reverting added the test and the test is no longer valid\nbecause delete_allocation_for_instance can no longer fail with this\nrevert.\n\nCloses-Bug: #1836754\n\nChange-Id: I277ba12a676fe8ecd77b25456d1a08c81360d64a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c9a0b17423bbbf03213fe6f2aa2d7e9c1eee8808"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c9a0b17423bbbf03213fe6f2aa2d7e9c1eee8808"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}