)]}'
{"id":"openstack%2Fnova~804230","triplet_id":"openstack%2Fnova~master~Iea8896682fc28e3a5cd25afa45238272bee745e1","project":"openstack/nova","branch":"master","topic":"bug/1939545","hashtags":[],"change_id":"Iea8896682fc28e3a5cd25afa45238272bee745e1","subject":"compute: Ensure updates to bdms during pre_live_migration are saved","status":"MERGED","created":"2021-08-11 12:19:22.000000000","updated":"2021-08-20 15:27:01.000000000","submitted":"2021-08-20 15:23:57.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":8,"unresolved_comment_count":2,"has_review_started":true,"submission_id":"804230-bug/1939545","meta_rev_id":"4538e847be240274f31b6603b64928fe2d06bb1e","_number":804230,"virtual_id_number":804230,"owner":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-08-20 15:23:57.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"all":[{"value":2,"date":"2021-08-18 09:26:44.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"date":"2021-08-11 18:55:35.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":2,"date":"2021-08-20 00:19:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"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":{"approved":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"all":[{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":1,"date":"2021-08-20 00:19:19.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"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":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":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"_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-08-11 12:56:51.000000000","updated_by":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"reviewer":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"state":"CC"},{"updated":"2021-08-11 13:00:06.000000000","updated_by":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"reviewer":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"state":"REVIEWER"},{"updated":"2021-08-11 13:03:26.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-08-11 13:45:43.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-08-11 14:31:43.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-08-11 20:18:12.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-08-11 22:12:38.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-08-18 09:18:59.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"CC"},{"updated":"2021-08-18 09:26:44.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"REVIEWER"},{"updated":"2021-08-20 00:19:19.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2021-08-20 10:01:15.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"}],"messages":[{"id":"a7a20649c82e6cad8c2036a7cc99255c8de28cec","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-11 12:19:22.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6e6f89b31a762453da2f043a5636fdc66d45fa11","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-08-11 12:56:51.000000000","message":"Patch Set 1:\n\n(1 comment)\n\n-1 for question visibility","accounts_in_message":[],"_revision_number":1},{"id":"6742a281513fdf88d00c6a36e51545adfaee8345","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-08-11 13:00:06.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"308426bbc6020531983b01715abfefc70a2b360c","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 13:03:26.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/76878c78f9684cacb2fa456d2ef0e182 : SUCCESS in 25m 43s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/4407e0c9d86e425c8354dbc8dc09bb21 : SUCCESS in 25m 36s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"05c2b3ec1eae77dc624f2a7e6a0d01b0e63bd913","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-11 13:26:30.000000000","message":"Patch Set 1: Workflow-1\n\n(1 comment)\n\nLooks like some func tests are failing as they are using pure dicts instead of driver BDMs, I\u0027ll respin shortly.","accounts_in_message":[],"_revision_number":1},{"id":"17100aa09c5cc05446927a74a8234e40cc05d00e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 13:45:43.000000000","message":"Patch Set 1: 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/2f860de1bfa8403b807917b6ba455923 : SUCCESS in 1h 02m 08s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/37be372438914a56accb8350ad84478f : SUCCESS in 16m 48s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/efbdc5915dd3452a91b904f39f535440 : SUCCESS in 13m 21s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e47372d036414ecba8f104b0984205e2 : SUCCESS in 7m 54s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/29f0fac09db347028adaf7b3481c60ef : SUCCESS in 12m 47s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d9e1473b5db0474baf9c444b1e2e009f : SUCCESS in 13m 46s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1ab3c914d12344d694e0e5f9ba95931a : SUCCESS in 13m 03s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d85ad56d899b4a8f8c6e871d7d0a341c : SUCCESS in 9m 08s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/5096fa66a1c84f7393f4811b389598b5 : SUCCESS in 1h 20m 27s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/988e5b1a0e9143ddb5d44cb17ed1c689 : SUCCESS in 1h 20m 37s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/72a54d07d4e547a5abc34597a00f9f61 : SUCCESS in 50m 22s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/9bee7c101d2b44a697fe373badc24e49 : SUCCESS in 52m 35s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/d268d4f62ec3455caa13b247db3c8420 : SUCCESS in 1h 18m 20s\n- nova-next https://zuul.opendev.org/t/openstack/build/982a12af502c477bba94f9ec27f76336 : SUCCESS in 1h 20m 37s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/41ef3b4609154c888f0920013d012262 : SUCCESS in 6m 37s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/84e03400c0d94fa0a6a0cd1d4f14d911 : FAILURE in 18m 47s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/6c15fc12ca5a4dd1a3fd27510a270e7d : FAILURE in 17m 16s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/e6c074689a1b4aeb88fbe8fcceac13aa : FAILURE in 20m 07s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/4aa55b3f7e4c409496125a75e48da475 : SUCCESS in 1h 14m 13s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/41eaa7a1197746d59980f11edcd180f8 : SUCCESS in 47m 25s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/fc73ea90568e4fb0a60cc282d90ab56b : SUCCESS in 41m 30s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/37b8f3e6d5ae40448c42ffe817cd509d : SUCCESS in 26m 48s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/7415201561f346b28faf962188e2cc50 : SUCCESS in 37m 23s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"9f2b69628f0bd3628e8f79255246ada7d2fba3d7","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-08-11 14:31:43.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/804230/1/iscsi : SUCCESS in 2h 09m 56s","accounts_in_message":[],"_revision_number":1},{"id":"587916e03c24d6574558aa1c0e4afd74a5e31fa3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-11 17:42:53.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"8f14fc5888105f3aaab156e188cddc414864afbf","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 18:16:57.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/fe6a6c5c38bf40dbabe7af0222e6a75f : FAILURE in 20m 29s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/6fb67791db0c4bc2ab7a0e4e2ba61a72 : FAILURE in 23m 28s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"6203fa4c1dc6627c6e51c9faa3de5f21c016011c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-11 18:52:53.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.","accounts_in_message":[],"_revision_number":3},{"id":"e851c20b28e54a340e16791644225718517ab98e","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-08-11 18:55:35.000000000","message":"Patch Set 3: -Code-Review","accounts_in_message":[],"_revision_number":3},{"id":"ec26c2ec514fad5210704c02d1f3cff82d09953d","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 19:25:14.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/781782bf60774044b4fef746e2b2ece5 : SUCCESS in 28m 26s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/45163353277b4fcaa550f4146e1744b6 : SUCCESS in 30m 16s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"5d6d4ba96004d60a8d3b1c3d8e98938e691a7d5f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-11 20:17:34.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/f7088d22958042a89310ddaf9eb0cbae : SUCCESS in 1h 07m 34s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5a9391fc683b4182a5c965d316952fab : SUCCESS in 16m 03s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b4ede01f60624527a5e877bc52c40657 : SUCCESS in 15m 06s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4c60706bfd2c4771a0fb3dceefd9bfad : SUCCESS in 9m 21s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0c8b819b94104d65b911d26275657ffc : SUCCESS in 13m 07s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/2e59e4392961423d86a40d85fa27be38 : SUCCESS in 12m 54s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/3b7e2abf5aeb4542a601453f7a89fbbd : SUCCESS in 13m 16s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/21c2a0b2738b4f4ab6a9955fe987cf55 : SUCCESS in 10m 16s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/d127f988eb294c10b2aa052df69d97d5 : SUCCESS in 1h 11m 14s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/9b7ef1f3b4184060bed2878eba303c65 : SUCCESS in 1h 09m 28s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/ba9edbd3273e4c1fbfb85d3857ca53b3 : SUCCESS in 53m 31s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/02269f037697417ea725d2f842ebe629 : SUCCESS in 47m 38s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/1ac2e89858c24243a794e69a6872180b : SUCCESS in 1h 13m 40s\n- nova-next https://zuul.opendev.org/t/openstack/build/25c4aefa344a41068f1d1deda7777c57 : SUCCESS in 1h 20m 17s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/1b6debb469d2415aa858217892e3a8fd : SUCCESS in 6m 03s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/c26e5dd9fd0a420287ccdada9b6dddf9 : SUCCESS in 17m 26s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/a2f558a637284cae80e79818adf03141 : SUCCESS in 18m 32s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/d6822319b8574a70a20a7950fb1d9448 : SUCCESS in 16m 49s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/acc746eaa3d4481abf22b4d5695585b7 : SUCCESS in 1h 15m 19s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2851b94fb24d40e2aca5f5c105948ae6 : SUCCESS in 44m 01s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/7011069634244944a15556988e2b04ac : SUCCESS in 42m 50s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/b09480494dbf43f6a14dd69425d6dee0 : SUCCESS in 24m 19s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/483fb8ca0e494c54862c8a2d1c0ccc2d : SUCCESS in 30m 30s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"8e0ed3eec489a8dba042a2105aaccce470ff413c","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-08-11 20:18:12.000000000","message":"Patch Set 3:\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/30/804230/3/check/EMC_PowerFlex_NOVA/e6af121/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":3},{"id":"4114ec79365f05274313af7d7b545b9b03249388","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-08-11 21:12:20.000000000","message":"Patch Set 3:\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/804230/3/iscsi : SUCCESS in 2h 14m 41s","accounts_in_message":[],"_revision_number":3},{"id":"bacb4eb668b462ff6c81c39ba9ca26a795860ae5","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-08-11 22:12:38.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 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/30/804230/3/check/tempest-dsvm-full-focal-py3/95507b1/ : FAILURE in 2h 07m 39s","accounts_in_message":[],"_revision_number":3},{"id":"7e1983027e25d108edb45e779ce707ba18ee9644","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-12 10:34:18.000000000","message":"Topic set to bug/1939545","accounts_in_message":[],"_revision_number":3},{"id":"169077c5a29b63c6954f83fbb55694574c9a75d5","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-08-18 09:18:59.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"9699ae7b82862d828ce2e6193ad50f6b626107c8","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-18 09:22:59.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"4a777e2d33aed3ef6b2ba7636f0186f2882be3b2","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-08-18 09:26:44.000000000","message":"Patch Set 3: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"d519b5bb0f22fe058127d2a0f90b0337f781dce6","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-08-20 00:19:19.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\n(2 comments)\n\nLooks OK to me","accounts_in_message":[],"_revision_number":3},{"id":"ba2a3120bb82c63caa346cc47c380829658a7321","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 00:20:49.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"41ee055fe526345775db63578bf5dc5225bdd341","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 01:40:48.000000000","message":"Patch Set 3: Verified-2\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":3},{"id":"10a6ff20fd2629fe43a2dc0d6124de9758ada946","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-08-20 10:01:15.000000000","message":"Patch Set 3:\n\nrecheck","accounts_in_message":[],"_revision_number":3},{"id":"29656829ade0c3c8539bb33303bca083d7a488cb","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 10:28:40.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/45103e9d2e3145198bf3ab399fc954f7 : SUCCESS in 23m 52s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/5cdae6584a924905ade2679a5dfdfd70 : SUCCESS in 25m 32s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"cb3c8437ff2816937d95b03f14ff4bdf031c4720","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 11:42:30.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/c910ec9533b5401d8d3c53cfda15a947 : SUCCESS in 1h 16m 19s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/d15d55d24ee844d5a56d3cc4fc32a55c : SUCCESS in 16m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/31c79011dd9b4e8bbd34a5765483c7b8 : SUCCESS in 13m 35s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fbc067182005470592d4f8b866d2ff04 : SUCCESS in 8m 52s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/65a292bd9fe24a5596220acb0a2d4600 : SUCCESS in 13m 09s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/24d0692c4a6d4476ac31dedd84dba9a8 : SUCCESS in 12m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c879d32f4b854651910207a58cc84340 : SUCCESS in 12m 21s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c23f9b8391634f63aaa9a02dee16741a : SUCCESS in 9m 56s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/acf0f330ff884014ba1bc46ab0fc83e1 : SUCCESS in 1h 09m 12s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/3ce360c2d2564f29bfcd6987af97b48d : SUCCESS in 1h 06m 42s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/6d470a6b6d494d29bde518b142ec91b5 : FAILURE in 49m 23s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/0336e3b2b8f6408da9bf5fcd215445c6 : SUCCESS in 51m 35s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/6991f50d29c24cdcb49edc2f86ebcbb9 : SUCCESS in 1h 32m 35s\n- nova-next https://zuul.opendev.org/t/openstack/build/a5449e0e56ee4c5ea1d5dce9770e1205 : SUCCESS in 1h 18m 33s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/f7966e048e134508aa816a0c4cd6740b : SUCCESS in 6m 02s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/2e084aa824d34ba7934b9734f2b0aefb : SUCCESS in 19m 15s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/b884ee1fbaf246f7994c6aea6bb68548 : SUCCESS in 16m 51s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/ca40170b823b43799cadef64393149ae : SUCCESS in 17m 54s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/a50067f806384d05a39b305f9ff595ae : SUCCESS in 1h 34m 49s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e3f88e18cbc64d3abf026e608a5dcd75 : SUCCESS in 49m 17s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/0b6957c54e63414fa7be94eb7f253b94 : SUCCESS in 42m 56s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/b665e9ed88104bbc9d4a99006eee37a5 : SUCCESS in 29m 45s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/ca26f1a49a5b4ae6be934061ee30c872 : SUCCESS in 35m 50s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"b94858bc51644510917c17c80a725744b05f65fd","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-08-20 11:42:51.000000000","message":"Patch Set 3:\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/30/804230/3/check/EMC_PowerFlex_NOVA/96c3d04/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":3},{"id":"adf80984b97fd1276468d284b25008a103183ad7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 11:43:13.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"b39565b5b837b50c0987c4473c79fdb38818fd48","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-08-20 12:05:37.000000000","message":"Patch Set 3:\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/30/804230/3/check/tempest-dsvm-full-focal-py3/07bb96f/ : SUCCESS in 1h 58m 20s","accounts_in_message":[],"_revision_number":3},{"id":"48a79861fbeb90f0bf18bf7c9c6c3bc2e058ff51","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-08-20 12:52:05.000000000","message":"Patch Set 3:\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/804230/3/iscsi : SUCCESS in 2h 47m 28s","accounts_in_message":[],"_revision_number":3},{"id":"ee01e9154948efb849db74c32f02d4fec2bebf37","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 15:23:57.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/4af1d186858d40b180a64bdb2166587b : SUCCESS in 1h 13m 06s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ec7737efaaae4bf5a8227a9787cc3c78 : SUCCESS in 13m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8a596363795d45d8b6ad767a7f1f30c6 : SUCCESS in 10m 15s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4f43a8f5e8d84b4d860fab73ead1b57d : SUCCESS in 13m 15s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/ecac33b823ce408f88bede86b7f7969a : SUCCESS in 15m 12s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f6656d11c2ea4dccae0654eca5ef9381 : SUCCESS in 10m 39s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/cd1d7e9f38524e02baf8db59780945af : SUCCESS in 1h 03m 15s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/f234ace2a0034e75a5fdfc1d0b50c4a8 : POST_FAILURE in 58m 02s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/144a8b1171a54c75ad225021b81b0885 : SUCCESS in 17m 45s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/27a98f5a6bc3485f85ade8f8ec0ae4d4 : SUCCESS in 24m 15s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/62a78b537f1b4b9982b1f67c41c7ae48 : SUCCESS in 1h 27m 26s\n- nova-next https://zuul.opendev.org/t/openstack/build/4f45a67b4e704900867385e9996a99aa : SUCCESS in 1h 22m 23s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/6612c84983ce4fc8b201d503a262d4ac : SUCCESS in 6m 36s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/fb12cb58f7514292a1f227c40a4f6c5a : SUCCESS in 1h 14m 56s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/fc1ca0fa2f7b4f98b2e0eecfc82313d2 : SUCCESS in 1h 10m 27s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/112cedbd74cc439c8d7b4a7c67774db7 : SUCCESS in 46m 07s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/c34b1b147fc5413490965aff23f10be3 : SUCCESS in 46m 16s","accounts_in_message":[],"_revision_number":3},{"id":"1c2536ff46a63b0e6cb6331fc3a219be9201b9b0","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 15:23:57.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"4538e847be240274f31b6603b64928fe2d06bb1e","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 15:27:01.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/086f2c7051ec456788b17e835f315628 : SUCCESS in 1m 14s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"962eda94d52321c3237da870c3d0455c6f0e851b","revisions":{"dd63600d45711887a74a53fd14a351139c0185fd":{"kind":"REWORK","_number":1,"created":"2021-08-11 12:19:22.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/30/804230/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/30/804230/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/30/804230/1"}}},"commit":{"parents":[{"commit":"0db5d332f000a9b8d62c92a83fc35e13cc510b23","subject":"Add a regression test for bug 1939545","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0db5d332f000a9b8d62c92a83fc35e13cc510b23"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 11:04:11.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 11:45:08.000000000","tz":60},"subject":"compute: Ensure updates to bdms during pre_live_migration are saved","message":"compute: Ensure updates to bdms during pre_live_migration are saved\n\nWhen connecting volumes to the underlying host the virt drivers can\nattempt to stash additional metadata returned from os-brick into the\nconnection_info associated with a bdm. This pretty janky behaviour\nrelies on someone later calling .save() against the underlying\nBlockDeviceMapping object to persist these changes into the database as\nhappens in the driver block device layer during a standard volume\nattach.\n\nHowever during pre_live_migration no call was made to .save() resulting\nin the changes made to the connection_info being lost. This change\nsimply introduces this call at the end of the pre_live_migration method\non the destination via the driver bdm objects we provide in\nblock_device_info.\n\nCloses-Bug: #1939545\nChange-Id: Iea8896682fc28e3a5cd25afa45238272bee745e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dd63600d45711887a74a53fd14a351139c0185fd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dd63600d45711887a74a53fd14a351139c0185fd"}]},"branch":"refs/heads/master"},"3b4a6994b910cf156c3be03b3ca916191c455d0c":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2021-08-11 17:42:53.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/30/804230/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/30/804230/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/30/804230/2"}}},"commit":{"parents":[{"commit":"4375cd19496f00a59b519c9a5b3218ada5008851","subject":"Add a regression test for bug 1939545","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4375cd19496f00a59b519c9a5b3218ada5008851"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 11:04:11.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 17:42:42.000000000","tz":60},"subject":"compute: Ensure updates to bdms during pre_live_migration are saved","message":"compute: Ensure updates to bdms during pre_live_migration are saved\n\nWhen connecting volumes to the underlying host the virt drivers can\nattempt to stash additional metadata returned from os-brick into the\nconnection_info associated with a bdm. This pretty janky behaviour\nrelies on someone later calling .save() against the underlying\nBlockDeviceMapping object to persist these changes into the database as\nhappens in the driver block device layer during a standard volume\nattach.\n\nHowever during pre_live_migration no call was made to .save() resulting\nin the changes made to the connection_info being lost. This change\nsimply introduces this call at the end of the pre_live_migration method\non the destination via the driver bdm objects we provide in\nblock_device_info.\n\nCloses-Bug: #1939545\nChange-Id: Iea8896682fc28e3a5cd25afa45238272bee745e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3b4a6994b910cf156c3be03b3ca916191c455d0c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3b4a6994b910cf156c3be03b3ca916191c455d0c"}]},"branch":"refs/heads/master"},"962eda94d52321c3237da870c3d0455c6f0e851b":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2021-08-11 18:52:53.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/30/804230/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/30/804230/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/30/804230/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/30/804230/3"}}},"commit":{"parents":[{"commit":"268b7169554f871d3813d78e79861934d086484d","subject":"Add a regression test for bug 1939545","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/268b7169554f871d3813d78e79861934d086484d"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 11:04:11.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-11 18:52:12.000000000","tz":60},"subject":"compute: Ensure updates to bdms during pre_live_migration are saved","message":"compute: Ensure updates to bdms during pre_live_migration are saved\n\nWhen connecting volumes to the underlying host the virt drivers can\nattempt to stash additional metadata returned from os-brick into the\nconnection_info associated with a bdm. This pretty janky behaviour\nrelies on someone later calling .save() against the underlying\nBlockDeviceMapping object to persist these changes into the database as\nhappens in the driver block device layer during a standard volume\nattach.\n\nHowever during pre_live_migration no call was made to .save() resulting\nin the changes made to the connection_info being lost. This change\nsimply introduces this call at the end of the pre_live_migration method\non the destination via the driver bdm objects we provide in\nblock_device_info.\n\nCloses-Bug: #1939545\nChange-Id: Iea8896682fc28e3a5cd25afa45238272bee745e1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/962eda94d52321c3237da870c3d0455c6f0e851b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/962eda94d52321c3237da870c3d0455c6f0e851b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
