)]}'
{"id":"openstack%2Fnova~840835","triplet_id":"openstack%2Fnova~stable%2Fxena~If9ab424cc7375a1f0d41b03f01c4a823216b3eb8","project":"openstack/nova","branch":"stable/xena","topic":"bug/1969496","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":"2024-03-05 19:13:41.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"If9ab424cc7375a1f0d41b03f01c4a823216b3eb8","subject":"Remove unavailable but not reported PCI devices at startup","status":"ABANDONED","created":"2022-05-06 08:40:46.000000000","updated":"2024-03-05 19:13:41.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7eb3c1281899944eb6f316395df53d46bdb06718","_number":840835,"virtual_id_number":840835,"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":"2022-05-06 10:19:33.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"},{"_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":"2022-05-06 09:45:10.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":"2022-05-06 09:54:38.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":"2022-05-06 10:19:33.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":"2022-05-06 10:57:51.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":"2022-05-06 12:51:06.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":"2024-03-05 19:13:41.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":"9fa86ca2b44a121abdf60abe9387df5aadc00e86","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2022-05-06 08:40:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6d39fec7d16acbf5967909acd14f7c8b1474c28b","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2022-05-06 09:45:10.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 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/35/840835/1/check/tempest-dsvm-full-focal-py3/e2a05c4/ : FAILURE in 49m 12s","accounts_in_message":[],"_revision_number":1},{"id":"d0a01f05e8685ebbd95bdec135fb00ef14f0213f","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-05-06 09:54:38.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/2cf3dd167e9d4528ab819f82314fe8b7 : SUCCESS in 23m 57s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/805b1cf6707348fd93e762d780207f12 : SUCCESS in 34m 45s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3e06fa9163ff73d27071456e13c5a30fc29b8cb0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-05-06 10:19:33.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-skip-level https://zuul.opendev.org/t/openstack/build/b8fa9a5501744332bd1ae34950410e90 : SUCCESS in 1h 11m 49s (non-voting)\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/76031ff9fd29413e8cc22b82bdea7404 : SUCCESS in 1h 18m 40s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/0efd1c080be54f83b28386d0344c4559 : FAILURE in 1h 20m 34s (non-voting)\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/cc423386a574432ea611fb03bda542a9 : SUCCESS in 53m 28s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/67cc6c1682be403fb83cfeb661cb3a03 : SUCCESS in 16m 13s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/42bf0203778c427b83cc5f18bf6799e0 : SUCCESS in 8m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/eb2c97827752448ead820bc474bd1a44 : SUCCESS in 11m 40s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/b117735024444b2ab52c84077443c959 : SUCCESS in 11m 35s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/a75a096533e748968137aa24691522b7 : SUCCESS in 11m 24s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e18355ebf39941d5a4d4596e82345c9a : SUCCESS in 9m 05s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/b49d58484083480696162666fdc61164 : SUCCESS in 50m 51s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/08ee0b7ab6f14b979b34a3dd60cd65eb : SUCCESS in 1h 09m 16s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/d833a838908a4d92b9ce417d3387628d : SUCCESS in 52m 30s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/d3f5ee56274d428faf2362785c4a6092 : SUCCESS in 52m 36s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/5711b1e41cd44ec18214c8761a2c6e38 : SUCCESS in 1h 32m 05s\n- nova-next https://zuul.opendev.org/t/openstack/build/f9c682e78ab342628d4cabcecba1a931 : SUCCESS in 1h 24m 03s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/1f5d006bbc7743e7aa9b3c81b22b9572 : SUCCESS in 56m 22s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/2981bae856b84895a5eabd40c11663a4 : FAILURE in 5m 49s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/e3499eba90354dd69019c9d8326fcf4b : FAILURE in 4m 51s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/e3af24849faf43c1b10a63126b717fc1 : SUCCESS in 18m 00s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/b42eb7ea4748435ea02ae8d9f549a909 : SUCCESS in 19m 39s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/583e69d54bfb4e9bb81dc9959f162c10 : SUCCESS in 1h 14m 12s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e7c2f7171fa04712a8ca45778d705080 : SUCCESS in 53m 43s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/03758e8785d64f94a2ce2915bd04cdcd : SUCCESS in 37m 00s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/4bb89500e01a44099e359559a019b195 : SUCCESS in 36m 45s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e56c1ad596d479296f27fafe1f958832a857430e","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":"2022-05-06 10:57:51.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/840835/1/iscsi : SUCCESS in 2h 11m 22s","accounts_in_message":[],"_revision_number":1},{"id":"435fd11c284346e410a4fe9143c9ca7576e978d0","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":34420,"name":"openstack lightbits","display_name":"Lightbits CI","email":"openstack@lightbitslabs.com","username":"openstack_lightbits"},"date":"2022-05-06 12:51:06.000000000","message":"Patch Set 1:\n\nBuild FAILED. log path: https://github.com/LightBitsLabs/logs/tree/main/06_05_2022_12_42_19\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":"7eb3c1281899944eb6f316395df53d46bdb06718","tag":"autogenerated:gerrit:abandon","author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"date":"2024-03-05 19:13:41.000000000","message":"Abandoned\n\nstable/xena 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/xena if you want to further work on this patch.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"580a4b14ae367e327229482decca0cb6cdf6e9d8","revisions":{"580a4b14ae367e327229482decca0cb6cdf6e9d8":{"kind":"REWORK","_number":1,"created":"2022-05-06 08:40:46.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/35/840835/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/840835/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/840835/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/840835/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/840835/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/840835/1"}}},"commit":{"parents":[{"commit":"1023580f6d9008f5e56f6d3905b6e227bd2bff67","subject":"Isolate PCI tracker unit tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1023580f6d9008f5e56f6d3905b6e227bd2bff67"}]}],"author":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-04-19 15:50:34.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"gibi@redhat.com","date":"2022-05-06 08:41:01.000000000","tz":120},"subject":"Remove unavailable but not reported PCI devices at startup","message":"Remove unavailable but not reported PCI devices at startup\n\nWe saw in the field that the pci_devices table can end up in\ninconsistent state after a compute node HW failure and re-deployment.\nThere could be dependent devices where the parent PF is in available\nstate while the children VFs are in unavailable state. (Before the HW\nfault the PF was allocated hence the VFs was marked unavailable).\n\nIn this state this PF is still schedulable but during the\nPCI claim the handling of dependent devices in the PCI tracker fill fail\nwith the error: \"Attempt to consume PCI device XXX from empty pool\".\n\nThe reason of the failure is that when the PF is claimed, all the\nchildren VFs are marked unavailable. But if the VF is already\nunavailable such step fails.\n\nOne way the deployer might try to recover from this state is to remove\nthe VFs from the hypervisor and restart the compute agent. The compute\nstartup already has a logic to delete PCI devices that are unused and\nnot reported by the hypervisor. However this logic only removed devices\nin \u0027available\u0027 state and ignored devices in \u0027unavailable\u0027 state.\n\nIf a device is unused and the hypervisor is not reporting the device any\nmore then it is safe to delete that device from the PCI tracker. So this\npatch extends the logic to allow deleting \u0027unavailable\u0027 devices. There\nis a small window when dependent PCI device is in \u0027unclaimable\u0027 state.\nFrom cleanup perspective this is an analogous state. So it is also\nadded to the cleanup logic.\n\nRelated-Bug: #1969496\nChange-Id: If9ab424cc7375a1f0d41b03f01c4a823216b3eb8\n(cherry picked from commit 284ea72e96604bdf16d1c5c4db47247334841b2f)\n(cherry picked from commit d7bca631fea348fb56cc46c5680643bacd82513e)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/580a4b14ae367e327229482decca0cb6cdf6e9d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/580a4b14ae367e327229482decca0cb6cdf6e9d8"}]},"branch":"refs/heads/stable/xena"}},"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":{}}}]}