)]}'
{"id":"openstack%2Fplacement~762987","triplet_id":"openstack%2Fplacement~master~I569043dd9f1c964faf7d4fcbb901a93f9569bf3d","project":"openstack/placement","branch":"master","hashtags":[],"change_id":"I569043dd9f1c964faf7d4fcbb901a93f9569bf3d","subject":"placement-status: check only consumers in allocation table","status":"MERGED","created":"2020-11-17 10:41:08.000000000","updated":"2021-07-30 19:50:08.000000000","submitted":"2021-07-30 19:48:50.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":7,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"762987","meta_rev_id":"78c70cc1ff0a6bc9bdef443706cb0ebcf2c97543","_number":762987,"virtual_id_number":762987,"owner":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-07-30 19:48:50.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"all":[{"value":2,"date":"2021-07-30 18:23:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},{"value":2,"date":"2021-07-30 17:05:00.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_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":{"approved":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"all":[{"value":1,"date":"2021-07-30 18:23:04.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_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,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_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":{"REVIEWER":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-11-18 17:51:06.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":"2020-11-19 08:33:05.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2021-07-30 17:04:33.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"}],"messages":[{"id":"5314ef3646484b91fe47bae23637d9dfca323fc5","author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"date":"2020-11-17 10:41:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"677dda11c809ad515c9cb43f89bf2c414a4834fe","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-17 13:51:22.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/c68cb5125a50449d82d097f517b8bc82 : SUCCESS in 58m 38s\n- tempest-integrated-placement https://zuul.opendev.org/t/openstack/build/67b09de4fa9d4f2388636b640ba5dd67 : SUCCESS in 1h 20m 29s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/9bc80b0f59ea4edaac8794e8b91f83bd : SUCCESS in 10m 08s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/97f00919ea4c4c82bce5a0ff2fe789f4 : SUCCESS in 6m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0c3819b7450347d780513ec37b48b9b8 : SUCCESS in 4m 46s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d083dd6fe0bb4607a5ea5c31e1e61f0a : SUCCESS in 5m 30s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/46b57a63e43b4adb97506a0d2b8ae3da : SUCCESS in 5m 16s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/955f40fb30f64c718acf23f5b63e1fa1 : SUCCESS in 6m 08s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ccf59828da6843dbb1825b1b0264234e : SUCCESS in 7m 39s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/e5eeed27c7b14532a2091766c21ceead : SUCCESS in 8m 10s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/6569026fd583497ba12d3d030ace2ba7 : SUCCESS in 6m 25s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/ed2ca2ff5ae54119945cbc2a2b155a13 : SUCCESS in 7m 30s\n- placement-nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/d701a911ecbc4b5591d72b78a70aba33 : SUCCESS in 13m 33s\n- placement-nested-perfload https://zuul.opendev.org/t/openstack/build/fc9cd6930c714244b97f996bda397d9a : SUCCESS in 24m 36s (non-voting)\n- placement-perfload https://zuul.opendev.org/t/openstack/build/3a90056223bc422189ebb0d89b57ebdd : SUCCESS in 11m 51s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/b099b16e02604c2888fe6c6c999d3db6 : SUCCESS in 58m 45s","accounts_in_message":[],"_revision_number":1},{"id":"d498d789eb58722aec3da7c78e2d2a7f55a06ea8","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2020-11-17 22:08:20.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"009fbd83f1f923a5e4e0f6558b2129d769d3b390","author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"date":"2020-11-18 11:15:58.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"99324d68110646c5155e61fdf7605fe5d2a6c349","author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"date":"2020-11-18 11:18:52.000000000","message":"Patch Set 2:\n\nMelanie,\n\nThanks for review. There is test_check_incomplete_consumers, but the edge case was missed indeed. Updated.","accounts_in_message":[],"_revision_number":2},{"id":"3f324e26662a4550a38eef2a40ef17239dc3831c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-18 17:51:06.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/6f1cc9b647594691ba0e24ef80f41f46 : SUCCESS in 1h 02m 21s\n- tempest-integrated-placement https://zuul.opendev.org/t/openstack/build/c36d6b726b0a4a7a8d75df8153370c3f : SUCCESS in 1h 25m 27s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/d0bc02081a6245c3a9318a8f86f65951 : SUCCESS in 5m 59s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/67e93ec1e7e3487a8c966ecac82d1d9a : SUCCESS in 6m 41s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4e8f396672a94f9982924275f9d56e21 : SUCCESS in 5m 16s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0b4c0b3995ce443c9026058197d690cd : SUCCESS in 4m 56s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/b73eac9e63cc4e55b952dfcd20a83090 : SUCCESS in 5m 49s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/3b195f1bcbe54ee48f3703ef2a83b2c6 : SUCCESS in 6m 46s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bf8e959e2eb845fd92b54a9d3dce44bc : SUCCESS in 8m 05s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/98403ee10ef1463484b94336e18f5a7b : SUCCESS in 5m 50s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/00537b74eb4c43219e071c810bff5f14 : SUCCESS in 5m 52s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/14670dea2b8f4fa9a98f29816cd4fb2f : SUCCESS in 6m 23s\n- placement-nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/9be4b3e4f4554f9fbee61e4977484668 : SUCCESS in 14m 33s\n- placement-nested-perfload https://zuul.opendev.org/t/openstack/build/caede7ebe236428fac40a3fc88280493 : SUCCESS in 21m 59s (non-voting)\n- placement-perfload https://zuul.opendev.org/t/openstack/build/6084820d10004182b48e35a262747adf : SUCCESS in 10m 24s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4ee7589a497c4c6c8d902675b227ff64 : SUCCESS in 1h 04m 13s","accounts_in_message":[],"_revision_number":2},{"id":"eae48c8d5dc385b14b74072899485f7579e79f4b","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2020-11-19 01:17:41.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)\n\nThanks Andrey but I think you added the edge case to test_create_incomplete_consumers instead of test_check_incomplete_consumers by mistake.","accounts_in_message":[],"_revision_number":2},{"id":"f603f8a07a94410eb18546189daefc4f5d42e865","author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"date":"2020-11-19 07:20:00.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"c9d001059060fe7df0450816075caf9f71534bcd","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2020-11-19 08:33:05.000000000","message":"Patch Set 2: Code-Review+1\n\n(1 comment)\n\nWell, I\u0027m dumb and was running the wrong test. The correct test is working well, fails without the fix and passes with the fix. This LGTM.","accounts_in_message":[],"_revision_number":2},{"id":"59411a691a4c135ae854dced2ee1b6a8f0a5cb9f","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2020-11-19 20:06:04.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"ff1724f27110967707f67f58609b756ce7b81f18","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-04-20 18:47:16.000000000","message":"Patch Set 2: Code-Review+2\n\nUpgrading to +2 since I now have +2 permission on this repo.","accounts_in_message":[],"_revision_number":2},{"id":"7fd0632d012e2c3dd6013c65eae7b07f7fcf03b9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-07-30 17:04:33.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"8756eb05274f31403eca6fc20ce14b395a6b6243","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-07-30 17:05:00.000000000","message":"Patch Set 3: Code-Review+2\n\nResolved a trivial merge conflict due to [1]. This fix seems sensible to me\n\n[1] https://review.opendev.org/c/openstack/placement/+/801419","accounts_in_message":[],"_revision_number":3},{"id":"2604b1c425a5e84241ce71031911efeb6b90e97a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 18:07:57.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/9b779e98f2ae4d5eb1349762b0bb7535 : SUCCESS in 1h 00m 57s\n- tempest-integrated-placement https://zuul.opendev.org/t/openstack/build/6f160cc49c6a4ce6911d1ea759333c4f : SUCCESS in 59m 19s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/79e912d51dec4e0ca4c167f42ca31896 : SUCCESS in 8m 36s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/bd0619d94ea54803ba79f7c83d060d1e : SUCCESS in 7m 48s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1c59e4b1ce4e49f5a635151ddef03e5d : SUCCESS in 6m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/310e8d836fc74eae8c12e3b38209eebe : SUCCESS in 7m 12s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/63be07064dc94c22be99f5338942d138 : SUCCESS in 5m 16s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/4a63bbfa656b440fbf4c4e112261253a : SUCCESS in 5m 36s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c411f466f4794d1c87c62e7d618f90c3 : SUCCESS in 10m 45s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/70132dd1033d4cd283e6d4169bd31c28 : SUCCESS in 5m 33s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/7faf09b38400417487d677020b393bf9 : SUCCESS in 6m 16s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/7e381576fd6c47368c1f44652b7e69eb : SUCCESS in 6m 14s\n- openstack-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/d52394d63372463d97c5dbe3941f58c3 : SUCCESS in 6m 53s (non-voting)\n- placement-nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/d5b0fb82acf34f9fa9de609e8ba9a578 : SUCCESS in 14m 06s\n- placement-nested-perfload https://zuul.opendev.org/t/openstack/build/ab13849d759f444aa05fa8785cc41197 : SUCCESS in 24m 27s (non-voting)\n- placement-perfload https://zuul.opendev.org/t/openstack/build/b111fac73d9443eaaec718d7e0bbd25b : SUCCESS in 13m 34s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/1c110b46642e489f9fd09613e63944ba : SUCCESS in 46m 38s","accounts_in_message":[],"_revision_number":3},{"id":"48a04d82c177063a484bb2e40c564d815408a0dd","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-07-30 18:23:04.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\nMerge conflict resolution is good, LGTM","accounts_in_message":[],"_revision_number":3},{"id":"4e6af345d7d21934bc683b8e7707abfb41cd7e75","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 18:23:37.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"fbd5fabddd1df8f4abb2ba87b574199276cc381c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 19:48:50.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/2cb519ff298e49e697dbbb6b43224758 : SUCCESS in 1h 04m 49s\n- tempest-integrated-placement https://zuul.opendev.org/t/openstack/build/4bda131889284095a549b8e507762ac8 : SUCCESS in 1h 11m 36s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/00b5b07f815c456aa869e41de73a0bae : SUCCESS in 6m 59s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1ee9ecfb639445b4bb1b897f2c04a58d : SUCCESS in 5m 57s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/37c88d1ca7cb4a9b81dfe6c3c1fb0386 : SUCCESS in 5m 34s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d20c3b8eec7244fd9fcca484c1b34a7e : SUCCESS in 11m 15s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/0ccc1cebc0324465a35ec13f8f584086 : SUCCESS in 9m 30s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/0be1020c6b7540a7a7b3ca706d0bdecd : SUCCESS in 6m 52s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/e0d60ce4fcf54a22942d59c441b613df : SUCCESS in 10m 08s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/8742c570a3fd4b18ae88bbad9fcb3fbb : SUCCESS in 11m 21s\n- placement-nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/1d9efcb5d2e144039d3b74638b6954d6 : SUCCESS in 18m 34s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/a8bbcea08c694cfc91b83a9d5e5d58e2 : SUCCESS in 50m 46s","accounts_in_message":[],"_revision_number":3},{"id":"b47fe98d01ecb7c92ace3ec2552be6088a61210f","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 19:48:50.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"78c70cc1ff0a6bc9bdef443706cb0ebcf2c97543","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 19:50:08.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/67a26b97fb764611acd682cc88319f68 : SUCCESS in 56s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"3253e2efc05836b0b5cefafbf2b2656e9e1a50c7","revisions":{"b36e6e63779fc6de9a02ff91119f4fa77c174b63":{"kind":"REWORK","_number":1,"created":"2020-11-17 10:41:08.000000000","uploader":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"ref":"refs/changes/87/762987/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/87/762987/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/87/762987/1"}}},"commit":{"parents":[{"commit":"c4fb76511f94c76e63a911925e4d0bab5fbab093","subject":"Merge \"Adds py38 functional tests to gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/c4fb76511f94c76e63a911925e4d0bab5fbab093"}]}],"author":{"name":"Andrey Volkov","email":"avolkov@mirantis.com","date":"2020-11-17 08:19:44.000000000","tz":180},"committer":{"name":"Andrey Volkov","email":"avolkov@mirantis.com","date":"2020-11-17 10:38:19.000000000","tz":180},"subject":"placement-status: check only consumers in allocation table","message":"placement-status: check only consumers in allocation table\n\nInitially, _check_incomplete_consumers was intended to find the case\nwhen consumers are mentioned in allocation but do not exist in the\nconsumer table. Due to a bug it\u0027s possible to have rows in the\nconsumer table those are not in the allocation table. The second case\nit seems was not expected and it can be tricky to automate a cleanup\nwith zero touch upgrade.\n\nThis patch modifies the check such way it counts only consumers\npresented in the allocation table.\n\nrelated-bug: #1780799\nChange-Id: I569043dd9f1c964faf7d4fcbb901a93f9569bf3d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/b36e6e63779fc6de9a02ff91119f4fa77c174b63"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/b36e6e63779fc6de9a02ff91119f4fa77c174b63"}]},"branch":"refs/heads/master"},"da8b06b68f37bbe169d2ba40d79c990a1da50cac":{"kind":"REWORK","_number":2,"created":"2020-11-18 11:15:58.000000000","uploader":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"ref":"refs/changes/87/762987/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/87/762987/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/87/762987/2"}}},"commit":{"parents":[{"commit":"c4fb76511f94c76e63a911925e4d0bab5fbab093","subject":"Merge \"Adds py38 functional tests to gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/c4fb76511f94c76e63a911925e4d0bab5fbab093"}]}],"author":{"name":"Andrey Volkov","email":"avolkov@mirantis.com","date":"2020-11-17 08:19:44.000000000","tz":180},"committer":{"name":"Andrey Volkov","email":"avolkov@mirantis.com","date":"2020-11-18 11:13:48.000000000","tz":180},"subject":"placement-status: check only consumers in allocation table","message":"placement-status: check only consumers in allocation table\n\nInitially, _check_incomplete_consumers was intended to find the case\nwhen consumers are mentioned in allocation but do not exist in the\nconsumer table. Due to a bug it\u0027s possible to have rows in the\nconsumer table those are not in the allocation table. The second case\nit seems was not expected and it can be tricky to automate a cleanup\nwith zero touch upgrade.\n\nThis patch modifies the check such way it counts only consumers\npresented in the allocation table.\n\nrelated-bug: #1780799\nChange-Id: I569043dd9f1c964faf7d4fcbb901a93f9569bf3d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/da8b06b68f37bbe169d2ba40d79c990a1da50cac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/da8b06b68f37bbe169d2ba40d79c990a1da50cac"}]},"branch":"refs/heads/master"},"3253e2efc05836b0b5cefafbf2b2656e9e1a50c7":{"kind":"REWORK","_number":3,"created":"2021-07-30 17:04:33.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/87/762987/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/87/762987/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/87/762987/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/87/762987/3"}}},"commit":{"parents":[{"commit":"3c51c6413bfc316cad8bb9d50b6fbffc83069b79","subject":"Merge \"Fix SQL query counting the number of individual consumers having allocations by only selecting the aggregated consumer_id column.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/3c51c6413bfc316cad8bb9d50b6fbffc83069b79"}]}],"author":{"name":"Andrey Volkov","email":"avolkov@mirantis.com","date":"2020-11-17 08:19:44.000000000","tz":180},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-07-30 17:00:55.000000000","tz":60},"subject":"placement-status: check only consumers in allocation table","message":"placement-status: check only consumers in allocation table\n\nInitially, _check_incomplete_consumers was intended to find the case\nwhen consumers are mentioned in allocation but do not exist in the\nconsumer table. Due to a bug it\u0027s possible to have rows in the\nconsumer table those are not in the allocation table. The second case\nit seems was not expected and it can be tricky to automate a cleanup\nwith zero touch upgrade.\n\nThis patch modifies the check such way it counts only consumers\npresented in the allocation table.\n\nRelated-bug: #1780799\nChange-Id: I569043dd9f1c964faf7d4fcbb901a93f9569bf3d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/3253e2efc05836b0b5cefafbf2b2656e9e1a50c7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/3253e2efc05836b0b5cefafbf2b2656e9e1a50c7"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}