)]}'
{"id":"openstack%2Fnova~960249","triplet_id":"openstack%2Fnova~stable%2F2024.1~I63c8fb2669a3c6b3adb77d210c0f9b39d3657c80","project":"openstack/nova","branch":"stable/2024.1","topic":"bug/2115905","attention_set":{},"removed_from_attention_set":{"9708":{"account":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"last_update":"2025-10-31 13:48:00.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I63c8fb2669a3c6b3adb77d210c0f9b39d3657c80","subject":"[PCI tracker]Remove non configured devs when freed","status":"ABANDONED","created":"2025-09-09 09:21:25.000000000","updated":"2025-10-31 13:48:00.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"477623d1b372522e78e6b1f9a653b72a99375813","_number":960249,"virtual_id_number":960249,"owner":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2025-09-09 11:01:52.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":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},{"_account_id":34420,"name":"openstack lightbits","display_name":"Lightbits CI","email":"openstack@lightbitslabs.com","username":"openstack_lightbits"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-09 11:01: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":"2025-09-09 11:07:57.000000000","updated_by":{"_account_id":34420,"name":"openstack lightbits","display_name":"Lightbits CI","email":"openstack@lightbitslabs.com","username":"openstack_lightbits"},"reviewer":{"_account_id":34420,"name":"openstack lightbits","display_name":"Lightbits CI","email":"openstack@lightbitslabs.com","username":"openstack_lightbits"},"state":"CC"},{"updated":"2025-10-31 13:48:00.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":"f8bfd4001c9ba9dc52d7ba479cd66ff5ee38e2fd","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2025-09-09 09:21:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"580cccb2a346af3b554f41d4ce5e465c521f8705","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-09 11:01:52.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/8188dba5ddd745f292963281dd502ba6\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/4e01b7b296ef49eba6e4e855f41749bc : SUCCESS in 58m 48s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/9972e1710c864ff68f51e793cf74758e : SUCCESS in 1h 22m 19s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/c7a8766b1b214e6e8f30d21cdf71ff6b : SUCCESS in 24m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/849544d623084d07b0db42c823efcb44 : SUCCESS in 18m 19s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5b9fa3454e5c47d49611088ab3f160ee : SUCCESS in 6m 10s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/85b0ee9422514f5c84c071fa5dad73f6 : SUCCESS in 14m 44s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/18fc2fb58d38401a8d68aebd2fc770e2 : SUCCESS in 12m 43s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/72976517f30142d6a8c4450eb1ca9563 : SUCCESS in 8m 06s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/1fd58ee57666443e9f36bc842f7e01d5 : SUCCESS in 41m 53s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/b0ee47729508434ca7aca109da2e474c : SUCCESS in 1h 33m 03s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/bb8902d4eb574a26b8c1a6fd37889b4f : FAILURE in 4m 27s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/2f84a518c6044495aa6f4a6bed5202e9 : SUCCESS in 26m 46s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/6db10b9a415342b096102794e423fca5 : SUCCESS in 28m 45s\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/9eaa3f232f5e43cd8a9a41ccf451a0f9 : SUCCESS in 19m 47s\n- tempest-integrated-compute-enforce-scope-new-defaults https://zuul.opendev.org/t/openstack/build/65c37a2f029a43f3b602c002006c3ef4 : SUCCESS in 1h 21m 20s\n- tempest-integrated-compute-rbac-old-defaults https://zuul.opendev.org/t/openstack/build/6f3137ca7621446fa8878a6167ced753 : SUCCESS in 59m 15s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/ad1c8cf117c74788836cb3cd0cb6f4f5 : SUCCESS in 59m 42s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/7d7ddea7554f42d997537bbd6073c768 : SUCCESS in 56m 53s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/757e3db41e224682a0ace3fe23a20316 : SUCCESS in 29m 10s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/a2e691706bbb4a858959161ec7d85c02 : SUCCESS in 33m 19s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3206a18fe76e66b42e3dbd3207efbd47b96a6ef7","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":34420,"name":"openstack lightbits","display_name":"Lightbits CI","email":"openstack@lightbitslabs.com","username":"openstack_lightbits"},"date":"2025-09-09 11:07:57.000000000","message":"Patch Set 1:\n\nBuild succeeded. log path: ${LOG_PATH}\nLeave a comment with \u0027run-Lightbits CI\u0027 to trigger a recheck. \nFor more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/LightbitsLabs_CI","accounts_in_message":[],"_revision_number":1},{"id":"9c627de21b51b5f4921efea20cf5fe9e244291c1","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-09 11:10:25.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/bddad05e8ab44f4696af0637951417ef\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/ceb1c36ad4534643a74bd74c8a6f957b : FAILURE in 33m 39s (non-voting)\n- openstack-tox-py311-arm64 https://zuul.opendev.org/t/openstack/build/6686137bfa1246479b8432a909a9305a : SUCCESS in 20m 13s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"477623d1b372522e78e6b1f9a653b72a99375813","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2025-10-31 13:48:00.000000000","message":"Abandoned\n\nstable/2024.1 branch of openstack/nova is about to be deleted. To be able to do that, all open patches need to be abandoned. Please cherry pick the patch to unmaintained/2024.1 if you want to further work on this patch.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3716be03f08b78bb0523c108503b3fc1b9dae176","revisions":{"3716be03f08b78bb0523c108503b3fc1b9dae176":{"kind":"REWORK","_number":1,"created":"2025-09-09 09:21:25.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/49/960249/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/49/960249/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/49/960249/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/49/960249/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/49/960249/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/49/960249/1"}}},"commit":{"parents":[{"commit":"0f463b8ff869ad340b45b56c52b45777d2f44f75","subject":"Reproduce bug/2115905","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0f463b8ff869ad340b45b56c52b45777d2f44f75"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-07-08 14:55:55.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2025-09-09 09:14:28.000000000","tz":120},"subject":"[PCI tracker]Remove non configured devs when freed","message":"[PCI tracker]Remove non configured devs when freed\n\nThe PCI tracker handles the case when a device spec is removed from\nthe configuration while a device is still being allocated. It keeps the\ndevice until the VM is deleted to avoid inconsistencies.\n\nHowever the full removal of such a device needs not just the VM deletion,\nbut also a nova-compute restart. The device tracker just frees the\ndevice during VM deletion but does not removed them until the next\nnova-compute startup. This allows the device to be re-allocated by\nanother VM even though the device is not allowed by a device_spec.\n\nThis change adds yet another in memory dict to the pci tracker to track\nthese devices that are only kept until they are freed. Then during\nfree() this list is consulted and if the device is in the list then the\ndevice is marked for removal as well.\n\nThis kills two birds with one stone:\n\n* We prevent the re-allocation of the device as the state of the device\n  will be set to REMOVED not AVAILABLE during VM deletion.\n\n* As PCI in Placement relies on the state of the device to decide what\n  to track in placement, this change makes sure that a device that\n  needs to be removed, is now removed from placement too. Note that we have\n  another bug that prevents this removal for now. But at least the\n  reproducers of that bug now starts to behave the same regardless of\n  how many device belongs to the same RP in placement.\n\nRelated-Bug: #2115905\nChange-Id: I63c8fb2669a3c6b3adb77d210c0f9b39d3657c80\nSigned-off-by: Balazs Gibizer \u003cgibi@redhat.com\u003e\n(cherry picked from commit f37cdf0c4182103ad81dbf39188ff39955da3850)\n(cherry picked from commit a63b4e04b35f85acfee29c2ff67bf2ef6d19c533)\n(cherry picked from commit 69ab5f4bb7d23a02dac679c35ea3933c1bd74a2f)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3716be03f08b78bb0523c108503b3fc1b9dae176"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3716be03f08b78bb0523c108503b3fc1b9dae176"}]},"branch":"refs/heads/stable/2024.1"}},"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":{}}}]}