)]}'
{"id":"openstack%2Fnova~798968","triplet_id":"openstack%2Fnova~master~Ic6116b18a1586864dbea3cedef1c4b15a75b6b00","project":"openstack/nova","branch":"master","topic":"bug/1934203","hashtags":[],"change_id":"Ic6116b18a1586864dbea3cedef1c4b15a75b6b00","subject":"Fix volume_id in connection info","status":"NEW","created":"2021-07-01 01:31:43.000000000","updated":"2021-07-06 00:12:22.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c3236f86fe864ffdc3c49feaef66ca6959c83938","_number":798968,"virtual_id_number":798968,"owner":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-07-01 03:19:34.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":{"disliked":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-07-02 12:34:58.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_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":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-07-01 02:09:12.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-07-01 02:35:40.000000000","updated_by":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-07-01 03:19:34.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-07-01 03:20:42.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-07-01 04:12: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-07-01 04:54:27.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":"2021-07-01 05:09:44.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-07-01 13:41:14.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":"CC"},{"updated":"2021-07-01 13:41:14.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"}],"messages":[{"id":"2044a12d914793dc7d934c219721aeea0a03a0b7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-07-01 01:31:43.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"59a003545282d2b3463eca786352d08b81ba0f37","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-01 02:09:12.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/c578485847764f2b8124886fe2f11118 : SUCCESS in 29m 18s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/c8c603ff772f446f8cd5226602b6d5e4 : SUCCESS in 35m 38s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"7030591397bd48b5fddf0c5390baf9e72208b3c4","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2021-07-01 02:35:40.000000000","message":"Patch Set 1:\n\nBuild Successful \n\n* SRIOV-nova-direct http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-direct : SUCCESS\n\n* SRIOV-nova-macvtap http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-macvtap : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"ae32dfe4e01658f3d14d7541ad65fbfb5198addf","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-01 03:19:34.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/27048625719642148feded32ea908877 : SUCCESS in 1h 10m 49s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/18f76270446d40e38852e27f6274f581 : SUCCESS in 18m 31s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0432673df44743958f2139759a185642 : SUCCESS in 14m 56s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/08c27fd49a06494090114dbc0a61eb30 : SUCCESS in 8m 45s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d091e62cb71a480d98ad368c503ccde6 : SUCCESS in 14m 09s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a3c4427015a54c3b9bcc3299752f851b : SUCCESS in 13m 12s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1c823a5fb1fa480ab31ff173978a5fe5 : SUCCESS in 20m 39s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/11f1d0e2f3aa475aa03776694e7dc8f2 : SUCCESS in 11m 13s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/813899d892a14ccf82ff80771b5f6fe9 : SUCCESS in 1h 23m 28s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/d1afa22a91f64383a8387c9310db1f1d : SUCCESS in 1h 40m 57s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/7031e6889e1c46089b6e5ac0b705a191 : SUCCESS in 53m 42s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/dfa2faa675604863a794c6ced0e0bc06 : SUCCESS in 53m 32s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/25b0881a06984c42920ef22945746e2d : SUCCESS in 1h 25m 24s\n- nova-next https://zuul.opendev.org/t/openstack/build/6ac81673fdd54ba2839224dcd33e10ce : SUCCESS in 1h 25m 34s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/0053e2a7a9354de6ae66cf4eec30eea7 : SUCCESS in 6m 08s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/9986ad70a99e4d20a4b11cab80b001b9 : SUCCESS in 23m 31s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/5a3788f306584479a4b9b08fdd1fd132 : SUCCESS in 18m 17s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/0c753f71f4e8406f968eb04620a6bbe1 : SUCCESS in 26m 10s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/3152c83ee29343bc9a74776992fea1c5 : SUCCESS in 1h 17m 54s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/be08d34de7c2405085220c189339c211 : SUCCESS in 48m 01s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/dc6d56818d2445dbaf6e71006c1c6d74 : SUCCESS in 43m 53s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/29aa98dbd6074bb78098fd553145f592 : SUCCESS in 31m 41s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/3a2fa9790915469195f8f1125a525763 : SUCCESS in 49m 01s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6b1284685adbf60b921c4d1d9c2156de2f162919","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-07-01 03:20:42.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/68/798968/1/check/EMC_PowerFlex_NOVA/fcb54e9/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":"7c762b2da685fb956c127d0f8b89113e98370782","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-07-01 04:12: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/798968/1/iscsi : SUCCESS in 2h 39m 20s","accounts_in_message":[],"_revision_number":1},{"id":"78b623cce2f54f99fb9c9d792fbf2dd16c0aed25","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2021-07-01 04:21:35.000000000","message":"Patch Set 1:\n\nBuild Successful \n\n* SRIOV-nova-direct http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-direct : SUCCESS\n\n* SRIOV-nova-macvtap http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-macvtap : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"9292eb8688e60b09040cea32c0eb694b9293d28e","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-07-01 04:54:27.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 finger://zuul-v3-executor.novalocal/d399581732794da59029de4357795981 : POST_FAILURE in 1h 27m 24s","accounts_in_message":[],"_revision_number":1},{"id":"0a52ba2c4c069f9fd666b2395a9e1ff7b95deb4c","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-07-01 05:09:44.000000000","message":"Patch Set 1:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/798968/798968/1/check-not-vote/ext-nova-zuul/ea1a0ad : SUCCESS in 1h 06m 48s","accounts_in_message":[],"_revision_number":1},{"id":"59ede392626de05af4124b66a063763931b12c2c","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2021-07-01 06:05:37.000000000","message":"Patch Set 1:\n\nBuild Successful \n\n* SRIOV-nova-direct http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-direct : SUCCESS\n\n* SRIOV-nova-macvtap http://13.74.249.42/refs/changes/68/798968/1/SRIOV-nova-macvtap : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"75e854a78e991192c16308145679c25293862de8","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-07-02 12:34:58.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nThis isn\u0027t required as Nova should always stash the volume_id as the serial within the connection_info. This is in turn used as a fallback by nova.virt.block_device.get_volume_id *if* \u0027volume_id\u0027 isn\u0027t present in the provided connection_info from cinder for whatever reason.","accounts_in_message":[],"_revision_number":1},{"id":"b245bc0947999d6dc2ec001df715e8f9778d65c9","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-07-04 11:25:13.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e (1 comment)\n\u003e \n\u003e This isn\u0027t required as Nova should always stash the volume_id as the serial within the connection_info. This is in turn used as a fallback by nova.virt.block_device.get_volume_id *if* \u0027volume_id\u0027 isn\u0027t present in the provided connection_info from cinder for whatever reason.\n\nhmm.. imo looks like nova.virt.block_device.get_volume_id will always use volume_id rather than serial if volume id exists connection info. the problem of this is during volume swap, new connection will point to a volume id while will be switched back to the origin volume_id eventually, which makes previous one stale. \n\nand if serial is better than volume_id then I suggest change nova.virt.block_device.get_volume_id and use serial whenever it is available...","accounts_in_message":[],"_revision_number":1},{"id":"41b6308f2ec867763e09fa8e0dbb16759fc5c2c7","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-07-05 04:27:57.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"423e857acdd1b78113ffd7ad2256b1c5a5343763","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-07-05 16:33:09.000000000","message":"Patch Set 1:\n\n(1 comment)\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \n\u003e \u003e (1 comment)\n\u003e \u003e \n\u003e \u003e This isn\u0027t required as Nova should always stash the volume_id as the serial within the connection_info. This is in turn used as a fallback by nova.virt.block_device.get_volume_id *if* \u0027volume_id\u0027 isn\u0027t present in the provided connection_info from cinder for whatever reason.\n\u003e \n\u003e hmm.. imo looks like nova.virt.block_device.get_volume_id will always use volume_id rather than serial if volume id exists connection info. the problem of this is during volume swap, new connection will point to a volume id while will be switched back to the origin volume_id eventually, which makes previous one stale. \n\u003e \n\u003e and if serial is better than volume_id then I suggest change nova.virt.block_device.get_volume_id and use serial whenever it is available...","accounts_in_message":[],"_revision_number":1},{"id":"c3236f86fe864ffdc3c49feaef66ca6959c83938","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-07-06 00:12:22.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e (1 comment)\n\u003e \n\u003e \u003e Patch Set 1:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \u003e \n\u003e \u003e \u003e (1 comment)\n\u003e \u003e \u003e \n\u003e \u003e \u003e This isn\u0027t required as Nova should always stash the volume_id as the serial within the connection_info. This is in turn used as a fallback by nova.virt.block_device.get_volume_id *if* \u0027volume_id\u0027 isn\u0027t present in the provided connection_info from cinder for whatever reason.\n\u003e \u003e \n\u003e \u003e hmm.. imo looks like nova.virt.block_device.get_volume_id will always use volume_id rather than serial if volume id exists connection info. the problem of this is during volume swap, new connection will point to a volume id while will be switched back to the origin volume_id eventually, which makes previous one stale. \n\u003e \u003e \n\u003e \u003e and if serial is better than volume_id then I suggest change nova.virt.block_device.get_volume_id and use serial whenever it is available...\n\nfirst of all, connection info is eventually populated by volume drivers, so behavior might be different across different drivers, take rbd driver as an example, please look at https://github.com/openstack/cinder/blob/a7e98dba5b0bc84a2642952b66962d2e645c64f3/cinder/volume/drivers/rbd.py#L1458. this method is called after updating volume attachment through rest API. and you can see volume_id is being populated. this volume id is OK to use as long as volume has not been retyped...\n\nanother way to verify this is to query volume_attachment table from cinder db, I believe you can identify volume_id available in connection_info dict for at least rbd drivers...","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"e0f640e7580afb87544d23eb2a4eea1ca53fa9af","revisions":{"e0f640e7580afb87544d23eb2a4eea1ca53fa9af":{"kind":"REWORK","_number":1,"created":"2021-07-01 01:31:43.000000000","uploader":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"ref":"refs/changes/68/798968/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/68/798968/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/798968/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/798968/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/798968/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/68/798968/1"}}},"commit":{"parents":[{"commit":"d5ed968826895d362f4f2aa21decfdebb9b1fd84","subject":"zuul: Skip swap_volume tests as part of nova-next","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d5ed968826895d362f4f2aa21decfdebb9b1fd84"}]}],"author":{"name":"ushen","email":"yshxxsjt715@gmail.com","date":"2021-07-01 01:29:33.000000000","tz":480},"committer":{"name":"ushen","email":"yshxxsjt715@gmail.com","date":"2021-07-01 01:29:33.000000000","tz":480},"subject":"Fix volume_id in connection info","message":"Fix volume_id in connection info\n\nSome drivers appear to obtain volume_id\nfrom connection info to determine if volume\nis multi-attach enabled. so we need to make\nsure volume_id is correct.\n\nChange-Id: Ic6116b18a1586864dbea3cedef1c4b15a75b6b00\nCloses-Bug: 1934203\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e0f640e7580afb87544d23eb2a4eea1ca53fa9af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e0f640e7580afb87544d23eb2a4eea1ca53fa9af"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"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":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"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":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"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":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}