)]}'
{"id":"openstack%2Fnova~781079","triplet_id":"openstack%2Fnova~master~I8835d36a48d8f74377a5f1f63d5fd538f62173ef","project":"openstack/nova","branch":"master","topic":"bug/1917498","hashtags":[],"change_id":"I8835d36a48d8f74377a5f1f63d5fd538f62173ef","subject":"Check volume encryption keys prior to resize and migration","status":"NEW","created":"2021-03-17 11:08:01.000000000","updated":"2021-04-12 18:59:41.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":9,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d9c21002dc0298aba2ba019f4fe776c58abbb7bf","_number":781079,"virtual_id_number":781079,"owner":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-03-25 21:32:00.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-03-30 08:46:35.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":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]}],"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-03-17 11:42:16.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-03-17 12:17:18.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"},{"updated":"2021-03-17 13:02:18.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-03-17 13:27:08.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-03-17 13:32:07.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":"fa28949814bfd3c1bd974ef944ce453148f6bf6f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2021-03-17 11:08:01.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6ae1988a1c8e22a705b3b5eeb415e79840b35ce9","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 11:42:16.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/485e71700e664d17aaf04d78daa811f0 : SUCCESS in 32m 39s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/7d74b95358254ca4baf8d89a8e6daa5f : SUCCESS in 22m 14s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"d35f1a960bd22de3a52e1f98f38b93b3f0108877","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 12:17:18.000000000","message":"Patch Set 1: Code-Review-1\n\nThanks for working on this Mark.\n\nAs discussed the check itself is valid and LGTM however I\u0027d like to see if we can move this into the compute API allowing a synchronous failure to be returned to the caller instead of a 202 and later fault.","accounts_in_message":[],"_revision_number":1},{"id":"e26b6798791ac830202826d93daa32137bdb34b8","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-17 12:19:53.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e Thanks for working on this Mark.\n\u003e \n\u003e As discussed the check itself is valid and LGTM however I\u0027d like to see if we can move this into the compute API allowing a synchronous failure to be returned to the caller instead of a 202 and later fault.\n\nTBH I can see there being a need for this in multiple flows so we might also consider turning the check into a generic decorator like check_instance_state, something like check_volume_secret_access? Anyway that can come later once this fix has been backported.","accounts_in_message":[],"_revision_number":1},{"id":"cc620870832e7c6c167214079dbbc774c7b59e67","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-17 13:02:18.000000000","message":"Patch Set 1: Verified-1\n\n(4 comments)\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/b05dddd4cb1c40fb8afbb77cab2a3009 : SUCCESS in 1h 27m 07s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/3ff3915c1a8b403bb2abf23bd5a83563 : SUCCESS in 20m 23s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/001ec8a6811b4717a97204b33c941806 : SUCCESS in 14m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b6b9ef2c99974a3a8538e5c0c24b5c96 : FAILURE in 11m 31s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/590efdfa0aee408e954cb6e31ac084df : SUCCESS in 13m 37s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/8d410c8ab55949cdac619d264069c49a : SUCCESS in 13m 50s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c7653b2b16e0489b90a118821ef44e0e : SUCCESS in 14m 06s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f29e756c7ab74a6589c7c351b44a5f71 : SUCCESS in 11m 02s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/a11d561374d946a1a2580b9f2666499a : SUCCESS in 15m 51s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/c9c4e5e6059145b69fca9c88be7df0ed : SUCCESS in 1h 10m 35s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/0c5deabec9f74106a01785ad4a2605b5 : SUCCESS in 1h 28m 58s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/ab1d997e84b94cdda27a5ded3bbae6b7 : SUCCESS in 1h 15m 21s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/98da6a51cf81412eac53f63fb60a9aa7 : SUCCESS in 55m 54s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/d15dd764da874b3dad21615f7f669d26 : SUCCESS in 1h 32m 18s\n- nova-next https://zuul.opendev.org/t/openstack/build/12ba6d7b520c45e78bf9a13852d19a59 : SUCCESS in 1h 46m 09s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/e33891f4230d4a8bbea869333d5690e7 : SUCCESS in 19m 46s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/20f562a14f5648bdb256ca9c3f1d9240 : SUCCESS in 19m 41s (non-voting)\n- grenade https://zuul.opendev.org/t/openstack/build/34d8d6ea97e9437baa027d2459a5bac8 : SUCCESS in 1h 12m 21s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/d7713f6ccd5f4b2bbedc1c3dcf79b69e : SUCCESS in 1h 08m 31s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/f2795e3312154f3e825b896bb16976e2 : SUCCESS in 58m 56s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/0e29182363524993a0128066bf44a258 : SUCCESS in 36m 51s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/29d4a3b2e76e4df5909d3097b8a66d30 : SUCCESS in 46m 14s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f4b7542dfdbd4756e4d197420ffbb2197cf6a872","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-03-17 13:27:08.000000000","message":"Patch Set 1:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/781079/781079/1/check-not-vote/ext-nova-zuul/59256bb : SUCCESS in 1h 02m 59s","accounts_in_message":[],"_revision_number":1},{"id":"f7c8ad429a653b3af30f1c8f2eb31a4e99121369","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-03-17 13:32:07.000000000","message":"Patch Set 1:\n\nBuild succeeded. 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/79/781079/1/check/tempest-dsvm-full-focal-py3/09e53b7/ : SUCCESS in 2h 04m 37s","accounts_in_message":[],"_revision_number":1},{"id":"f49dbd39169a78f52d0d05eb2869554bf9ed4ce4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2021-03-25 14:24:05.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"70347785d25ef1198203ff4934077d61dc9d4692","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 15:11:29.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/ea59ac93238d4236a803c8548afd2009 : FAILURE in 35m 50s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/14f083d0dbb44651936fe1bc51d88f93 : FAILURE in 40m 11s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c8f480d46ae981f849d7d814362b2320dfa0e719","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 16:27:25.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/f5d034afc7844b9e807545e7d70ffc0a : SUCCESS in 1h 22m 35s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1c6f03b4a2494cf79ec1524e96e55782 : FAILURE in 15m 10s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/d4a7f45211b1468ba204ede3472b70ad : FAILURE in 14m 29s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c8acbc8c5fc94c379a01d356ae546700 : SUCCESS in 9m 04s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d4a616acabb84221b3faa5c430a543c2 : FAILURE in 13m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/1bed99f77e1e4238a2acfe8bfeba0543 : FAILURE in 19m 17s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/5fdd87c9e39c488a8907f6a56d89ca6a : FAILURE in 13m 53s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9a55db4db9504d8d88bfe320ddb38de8 : SUCCESS in 8m 21s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/746d210c36a346fb980801b753bc48d6 : SUCCESS in 14m 34s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/f30e757244634662b39dc46396779ee7 : SUCCESS in 1h 20m 33s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/141762c4671344869b265fe155927fbb : SUCCESS in 1h 23m 34s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/52d18cd5a2f34fefa3cc504388cd937f : SUCCESS in 1h 17m 48s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/1fabf482156a4c45a105d6d31ff282fb : SUCCESS in 52m 53s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/de7cbf88f43f4db79c634a2b9ea66784 : FAILURE in 1h 38m 56s\n- nova-next https://zuul.opendev.org/t/openstack/build/46dcb1b7bc054a6aad2d6c97d53af9f1 : SUCCESS in 1h 45m 39s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/d5a20b3a7361475191de344c84f8bc1d : SUCCESS in 21m 00s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/f3bfc1e328b9453bbe5761f2fbf57a89 : SUCCESS in 19m 55s (non-voting)\n- grenade https://zuul.opendev.org/t/openstack/build/9dcdbbc58a974301aed52e61db2d05e6 : SUCCESS in 1h 02m 47s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/bfa3037e967e433189335352096cfa5c : SUCCESS in 58m 25s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/2de7c7665c3b498c8d07d1bc2f4811f7 : SUCCESS in 59m 10s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/bf4d2973a27749c1ac916b5e1570bf2f : SUCCESS in 31m 48s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/542c483a05aa4343a2eece9d5c734428 : SUCCESS in 41m 06s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"e5a26c6d67c885c7c62314ee36d6c6979bf8d30c","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-03-25 18:24:59.000000000","message":"Patch Set 2:\n\nBuild succeeded. 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/79/781079/2/check/tempest-dsvm-full-focal-py3/116e882/ : SUCCESS in 2h 13m 55s","accounts_in_message":[],"_revision_number":2},{"id":"344d040318587fc186715bf1ba288bdb156a42cc","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2021-03-25 19:48:21.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"170c2a26dc45873c227ca7e08e5bdd94511d8f9e","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 20:31:01.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/26de186593b84ff4ac1cf080d82c310e : SUCCESS in 18m 47s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/6700553a2e904d86acb958c310ab04a0 : SUCCESS in 37m 36s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"ba80f0ec70032ae0a31325c6be7d4a9c77caa55b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 21:32:00.000000000","message":"Patch Set 3: 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/0ab3834d9ae447d9ab6ccf3a1e528993 : SUCCESS in 1h 35m 57s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/84897b87e3f74d2e895812f20993e658 : SUCCESS in 16m 48s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/75c04786337e427aa73e4621adbff078 : SUCCESS in 15m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f38bd33ef565408aac06e5971e17a90c : SUCCESS in 7m 50s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/ba32fecf9a88423cbca09829c07ac6dc : SUCCESS in 12m 18s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/b96f86a77b5141c2adbe32bd01b7c82d : SUCCESS in 13m 17s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/68089cbd1a2049e1956ae43f9e872a1d : SUCCESS in 12m 31s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f29fd4a83c1a4d7cb85c3a3e1b2c619e : SUCCESS in 10m 28s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/fa731988dc35438c990a1761c5d818e6 : SUCCESS in 15m 28s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/af99c1c45d7848a5bf77b9d6a65c26b7 : SUCCESS in 1h 21m 54s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/05eaf2df36c947ef95f7c7c61e494230 : SUCCESS in 1h 09m 10s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/eaac483238694a56a036f1ceba6cf257 : SUCCESS in 1h 15m 34s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/97c3c102d0b84bbb9f9f7bf3d3fb2502 : FAILURE in 56m 31s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/158f0c842f0147b6a30584a6ff86bf03 : SUCCESS in 1h 21m 00s\n- nova-next https://zuul.opendev.org/t/openstack/build/8312a15251984c28a6615f3302c364d1 : SUCCESS in 1h 39m 50s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/b50fc299278243e8b805d87d48c3ced3 : SUCCESS in 18m 10s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/a6313f6dbee94899b5559a6313f62b62 : SUCCESS in 21m 40s (non-voting)\n- grenade https://zuul.opendev.org/t/openstack/build/29ba9ff4be134992b86759b703ef9fe0 : SUCCESS in 1h 29m 38s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/75e91f8da64d4b84bb25453c117f056b : SUCCESS in 56m 33s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/5ecf634449f84673b4f93294c7977d14 : SUCCESS in 55m 29s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/9a0d569e3df04325b35c54919c55658a : SUCCESS in 46m 11s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/e4c8413e92604026a1312b79494842fe : SUCCESS in 40m 50s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"bb7365c25b6f5dfa4cc1992022534bddfc714c1c","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-03-26 00:00:14.000000000","message":"Patch Set 3:\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/2051bdfbe1e84b7f9f27c271c94555a6 : POST_FAILURE in 2h 33m 19s","accounts_in_message":[],"_revision_number":3},{"id":"93cfc44242920f92e44fd82dd43cfe4aa7d6f4ad","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-30 08:46:35.000000000","message":"Patch Set 3: Code-Review-1\n\n(2 comments)\n\n-1 as I\u0027d like us to use raise a generic exception for this and catch it up in the API so we can reuse this decorator elsewhere.\n\nOtherwise this LGTM, I\u0027ve got a silly nit but that\u0027s something we could address in a fup with a new method in the BlockDeviceMapping object.","accounts_in_message":[],"_revision_number":3},{"id":"b3ad998f9a96f2988bf6cb206d24dea16818bc81","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2021-03-30 09:07:09.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"a7368eb4d7117d2c7232c86c1c92c189b1a32064","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-03-30 20:26:28.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"d9c21002dc0298aba2ba019f4fe776c58abbb7bf","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-04-12 18:59:41.000000000","message":"Patch Set 3:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/781079/781079/3/check-not-vote/ext-nova-zuul/f994407 : FAILURE in 15m 49s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"f362e53eac0b104cb299152a2d9e173fcad3a19a","revisions":{"dcb26768ac0b2bf6296e4df5d9227c75b9ed4df9":{"kind":"REWORK","_number":1,"created":"2021-03-17 11:08:01.000000000","uploader":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"ref":"refs/changes/79/781079/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/79/781079/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/79/781079/1"}}},"commit":{"parents":[{"commit":"bde59951a94812f9920fb6448fc82825fec3d739","subject":"Merge \"libvirt: Deprecate `live_migration_tunnelled`\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bde59951a94812f9920fb6448fc82825fec3d739"}]}],"author":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-02-25 11:43:51.000000000","tz":0},"committer":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-03-17 11:07:41.000000000","tz":0},"subject":"Check volume encryption keys prior to resize","message":"Check volume encryption keys prior to resize\n\nIf an attempted resize or cold migration of an instance with encrypted\nvolumes fails when accessing encryption keys, the instance will be left\nin an ERROR state, and the volumes in the attaching state, without the\nattachment record in Cinder.\n\nThis happens because volume connections are terminated on the source\nhost before being recreated on the destination host. If the attachment\nfails on the destination host, rollback to the source host will also\nfail for the same reason. The volume attachment record still exists in\nNova, but is lost in Cinder.\n\nThis change handles the error more gracefully by checking whether\nvolumes are encrypted, and if so, checking that the encryption key can\nbe accessed, earlier in the resize/migration process, prior to\ntermination of the volume connections on the source host.\n\nWith the fix in place, the resize or cold migration will still fail, but\nthe instance and volumes will be unaffected. This behaviour matches that\nof live migration, since in the live migration case the volume\nconnections are not terminated on the source until after the migration\nis complete.\n\nCloses-Bug: #1917498\n\nChange-Id: I8835d36a48d8f74377a5f1f63d5fd538f62173ef\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dcb26768ac0b2bf6296e4df5d9227c75b9ed4df9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dcb26768ac0b2bf6296e4df5d9227c75b9ed4df9"}]},"branch":"refs/heads/master"},"6e75aa95cbd587cf0c1e7713de664586fddc0049":{"kind":"REWORK","_number":2,"created":"2021-03-25 14:24:05.000000000","uploader":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"ref":"refs/changes/79/781079/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/79/781079/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/79/781079/2"}}},"commit":{"parents":[{"commit":"572ae578bb228cd2dd62608c2e601af49f95fbbb","subject":"Merge \"Bump the Compute RPC API to version 6.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/572ae578bb228cd2dd62608c2e601af49f95fbbb"}]}],"author":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-02-25 11:43:51.000000000","tz":0},"committer":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-03-25 14:23:42.000000000","tz":0},"subject":"Check volume encryption keys prior to resize and migration","message":"Check volume encryption keys prior to resize and migration\n\nIf an attempted resize or cold migration of an instance with encrypted\nvolumes fails when accessing encryption keys, the instance will be left\nin an ERROR state, and the volumes in the attaching state, without the\nattachment record in Cinder.\n\nThis happens because volume connections are terminated on the source\nhost before being recreated on the destination host. If the attachment\nfails on the destination host, rollback to the source host will also\nfail for the same reason. The volume attachment record still exists in\nNova, but is lost in Cinder.\n\nFor live migration, the instance is not left in an error state because\nthe volume connection on the source host remains active. The migration\nstill fails however.\n\nThis change handles the error more gracefully by checking whether\nvolumes are encrypted, and if so, checking that the encryption key can\nbe accessed, earlier in the resize/migration process, in the nova API.\n\nWith the fix in place, a resize or cold migration will still fail, but\nthe instance and volumes will be unaffected. The user experience for\nresize, cold and live migration is improved, by failing the request\nimmediately.\n\nCloses-Bug: #1917498\n\nChange-Id: I8835d36a48d8f74377a5f1f63d5fd538f62173ef\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6e75aa95cbd587cf0c1e7713de664586fddc0049"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6e75aa95cbd587cf0c1e7713de664586fddc0049"}]},"branch":"refs/heads/master"},"f362e53eac0b104cb299152a2d9e173fcad3a19a":{"kind":"REWORK","_number":3,"created":"2021-03-25 19:48:21.000000000","uploader":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"ref":"refs/changes/79/781079/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/79/781079/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/79/781079/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/79/781079/3"}}},"commit":{"parents":[{"commit":"572ae578bb228cd2dd62608c2e601af49f95fbbb","subject":"Merge \"Bump the Compute RPC API to version 6.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/572ae578bb228cd2dd62608c2e601af49f95fbbb"}]}],"author":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-02-25 11:43:51.000000000","tz":0},"committer":{"name":"Mark Goddard","email":"mark@stackhpc.com","date":"2021-03-25 19:47:26.000000000","tz":0},"subject":"Check volume encryption keys prior to resize and migration","message":"Check volume encryption keys prior to resize and migration\n\nIf an attempted resize or cold migration of an instance with encrypted\nvolumes fails when accessing encryption keys, the instance will be left\nin an ERROR state, and the volumes in the attaching state, without the\nattachment record in Cinder.\n\nThis happens because volume connections are terminated on the source\nhost before being recreated on the destination host. If the attachment\nfails on the destination host, rollback to the source host will also\nfail for the same reason. The volume attachment record still exists in\nNova, but is lost in Cinder.\n\nFor live migration, the instance is not left in an error state because\nthe volume connection on the source host remains active. The migration\nstill fails however.\n\nThis change handles the error more gracefully by checking whether\nvolumes are encrypted, and if so, checking that the encryption key can\nbe accessed, earlier in the resize/migration process, in the nova API.\n\nWith the fix in place, a resize or cold migration will still fail, but\nthe instance and volumes will be unaffected. The user experience for\nresize, cold and live migration is improved, by failing the request\nimmediately.\n\nCloses-Bug: #1917498\n\nChange-Id: I8835d36a48d8f74377a5f1f63d5fd538f62173ef\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f362e53eac0b104cb299152a2d9e173fcad3a19a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f362e53eac0b104cb299152a2d9e173fcad3a19a"}]},"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":""}}}]}
