)]}'
{"id":"openstack%2Fnova~816486","triplet_id":"openstack%2Fnova~stable%2Fvictoria~I30d30a772475cb82d0fd088f14a54a35646bd1dc","project":"openstack/nova","branch":"stable/victoria","topic":"bug/1882521","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":"2021-11-10 16:49:17.000000000","reason":"Removed by gibi by clicking the attention icon"}},"hashtags":[],"change_id":"I30d30a772475cb82d0fd088f14a54a35646bd1dc","subject":"Parse alias from domain hostdev","status":"ABANDONED","created":"2021-11-04 08:03:06.000000000","updated":"2021-12-08 15:03:56.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"cherry_pick_of_change":807767,"cherry_pick_of_patch_set":1,"meta_rev_id":"f2bd817558f561e5e49cf7b3a68547626f3f8ac2","_number":816486,"virtual_id_number":816486,"owner":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-11-09 16:40: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":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-11-09 16:40: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":"2021-11-09 18:50:38.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-11-09 18:51:42.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"}],"messages":[{"id":"2651867207731ff503f1f74560696e43592b09f2","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-04 08:03:06.000000000","message":"Patch Set 1: Cherry Picked from branch stable/wallaby.\n\nThe following files contain Git conflicts:\n* nova/tests/unit/virt/libvirt/test_driver.py","accounts_in_message":[],"_revision_number":1},{"id":"60752766bd98a187bac68d0780bc3efc4bad00d7","tag":"autogenerated:gerrit:abandon","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-04 08:04:35.000000000","message":"Abandoned\n\nissue is not in stable/victoria","accounts_in_message":[],"_revision_number":1},{"id":"b02677ff37c8a63c1fb7e584ac7099281267a067","tag":"autogenerated:gerrit:restore","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-09 14:29:26.000000000","message":"Restored","accounts_in_message":[],"_revision_number":1},{"id":"fe6a4c4421c1c7be86cff0ada74f900939bf9189","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-09 14:31:58.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ac444aba463fa9fe7304a8c7f8be9551693023a3","tag":"autogenerated:gerrit:setReadyForReview","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-11-09 14:37:26.000000000","message":"Set Ready For Review","accounts_in_message":[],"_revision_number":2},{"id":"4f7dcdb25f9d6d4d58a85927bc2179da9dcb4a27","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-11-09 16:40:33.000000000","message":"Patch Set 2: 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/7a0181799b1f46abaf034f71fe262cec : SUCCESS in 1h 25m 41s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/322dc909c3ee4670b88c109222b61107 : FAILURE in 18m 36s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5319cbaebd8a43a9970d194a55f2046c : SUCCESS in 19m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4f7ddabb0a814afc894d8ec1bdc7b191 : SUCCESS in 16m 09s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2308872024164aa28ca173cf4145bb53 : FAILURE in 8m 03s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/2ce9eaecde2f487baa6228cf11ff9051 : SUCCESS in 14m 37s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/e2864677655846618d847ca94e449155 : SUCCESS in 15m 38s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a78dd1c5bb44486ca676746116d81965 : SUCCESS in 14m 34s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/606aeb3abf2e46c08fb614ce2bf8e855 : SUCCESS in 1h 23m 05s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/6ee5f2e1dd794adbac28456d834ccc7a : SUCCESS in 1h 16m 31s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/0c53e63b325745a8ad41d31982351671 : FAILURE in 1h 03m 33s\n- nova-lvm https://zuul.opendev.org/t/openstack/build/b23c792e6e154b85af6e7a51d84c2a7d : SUCCESS in 1h 00m 39s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/2fb9f4e318d34217978a4ccbecaf4b69 : SUCCESS in 1h 40m 01s\n- nova-next https://zuul.opendev.org/t/openstack/build/873640c8c653419b92f871583b4ecb90 : SUCCESS in 1h 23m 55s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/ca0479035a874587b9036758e7b24bd4 : SUCCESS in 7m 50s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/94ae7dc3ef1e4993ac1309fed537b96c : SUCCESS in 20m 26s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/dfb661a8dfb54a3ba9901e7b67e0e103 : SUCCESS in 1h 14m 27s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9750521bedfd45478806aa52b0f54558 : SUCCESS in 1h 06m 13s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/13881c153ae449efb9c8557a907ac3d5 : SUCCESS in 1h 01m 24s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/5a6d54cbcdd745a29da5d4ee11c48aa7 : SUCCESS in 45m 19s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/4cca149ae1ee48a7b1e1dc4f910626f2 : FAILURE in 40m 36s (non-voting)\n\nWarning:\n  Comments left for invalid file nova/virt/libvirt/driver.py","accounts_in_message":[],"_revision_number":2},{"id":"b30eae80821cdd9a48d15ab93c77529a62e691d5","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-11-09 18:50:38.000000000","message":"Patch Set 2:\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/816486/2/iscsi : SUCCESS in 2h 26m 21s","accounts_in_message":[],"_revision_number":2},{"id":"ac18fbc1c3d2360a38e3e999b1c0aeaeb398ac94","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-11-09 18:51:42.000000000","message":"Patch Set 2:\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/86/816486/2/check/tempest-dsvm-full-focal-py3/0222f26/ : TIMED_OUT in 3h 10m 59s","accounts_in_message":[],"_revision_number":2},{"id":"f2bd817558f561e5e49cf7b3a68547626f3f8ac2","tag":"autogenerated:gerrit:abandon","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-12-08 15:03:56.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"78a55ba0df2685626fce8e68eaa8dc6b80f125f8","revisions":{"9d6b02dd3ec65296db440706cd5db334ddc147e5":{"kind":"REWORK","_number":1,"created":"2021-11-04 08:03:06.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/86/816486/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/86/816486/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/86/816486/1"}}},"commit":{"parents":[{"commit":"7f00f7be226511840747643919d167c97a021bea","subject":"Merge \"address open redirect with 3 forward slashes\" into stable/victoria","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7f00f7be226511840747643919d167c97a021bea"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-09-01 15:32:56.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-11-04 08:03:06.000000000","tz":0},"subject":"Parse alias from domain hostdev","message":"Parse alias from domain hostdev\n\nIn I16e7df6932bb7dff243706ee49338ba6b3782085 we missed that\nLibvirtConfigGuestHostdevPCI is not a child class of\nLibvirtConfigGuestInterface and therefore we missed parsing out the\nalias field from the domain xml for hostdevs. The new libvirt driver\ndevice detach logic[1] uses the alias as the identifier towards libvirt\nso now hostdevs cannot be detached. This patch parses out the alias\nfield to fix the issue.\n\nCloses-Bug: #1942345\nRelated-Bug: #1882521\n\n[1] https://review.opendev.org/q/topic:bug/1882521\n\nChange-Id: I30d30a772475cb82d0fd088f14a54a35646bd1dc\n(cherry picked from commit b67b928c381511f6101778ab50f281ecd689d283)\n(cherry picked from commit 0cd4ef2dcd482e6ac2f86a345e0874970cce0bdd)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9d6b02dd3ec65296db440706cd5db334ddc147e5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9d6b02dd3ec65296db440706cd5db334ddc147e5"}]},"branch":"refs/heads/stable/victoria"},"78a55ba0df2685626fce8e68eaa8dc6b80f125f8":{"kind":"REWORK","_number":2,"created":"2021-11-09 14:31:58.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/86/816486/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/86/816486/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/86/816486/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/86/816486/2"}}},"commit":{"parents":[{"commit":"3508f0e41175135965232fd50727a167a09095a3","subject":"Consolidate device detach error handling","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3508f0e41175135965232fd50727a167a09095a3"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-09-01 15:32:56.000000000","tz":120},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2021-11-09 13:57:37.000000000","tz":60},"subject":"Parse alias from domain hostdev","message":"Parse alias from domain hostdev\n\nIn I16e7df6932bb7dff243706ee49338ba6b3782085 we missed that\nLibvirtConfigGuestHostdevPCI is not a child class of\nLibvirtConfigGuestInterface and therefore we missed parsing out the\nalias field from the domain xml for hostdevs. The new libvirt driver\ndevice detach logic[1] uses the alias as the identifier towards libvirt\nso now hostdevs cannot be detached. This patch parses out the alias\nfield to fix the issue.\n\nCloses-Bug: #1942345\nRelated-Bug: #1882521\n\n[1] https://review.opendev.org/q/topic:bug/1882521\n\nChange-Id: I30d30a772475cb82d0fd088f14a54a35646bd1dc\n(cherry picked from commit b67b928c381511f6101778ab50f281ecd689d283)\n(cherry picked from commit 0cd4ef2dcd482e6ac2f86a345e0874970cce0bdd)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/78a55ba0df2685626fce8e68eaa8dc6b80f125f8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/78a55ba0df2685626fce8e68eaa8dc6b80f125f8"}]},"branch":"refs/heads/stable/victoria"}},"requirements":[],"submit_records":[],"submit_requirements":[]}