)]}'
{"id":"openstack%2Fnova~797725","triplet_id":"openstack%2Fnova~master~Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf","project":"openstack/nova","branch":"master","topic":"bug/1899835","attention_set":{},"removed_from_attention_set":{"10135":{"account":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"last_update":"2026-06-17 15:36:58.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf","subject":"WIP compute: Avoid calling detach with src connection_info during LM rollback","status":"ABANDONED","created":"2021-06-23 17:19:12.000000000","updated":"2026-06-17 15:36:58.000000000","total_comment_count":2,"unresolved_comment_count":2,"has_review_started":true,"meta_rev_id":"2582d858a92d9f424cb878207185a00fea867ce5","_number":797725,"virtual_id_number":797725,"owner":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-08-12 15:42:19.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_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"]}],"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":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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"]},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-23 17:33:37.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"CC"},{"updated":"2021-06-23 17:59:19.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-06-23 19:34:36.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-06-23 20:16:53.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-06-23 20:27:41.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-06-24 20:09:50.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"}],"messages":[{"id":"25d6ad643b549f04e8b91aee2064024715392e25","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-23 17:19:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a13912242356e9ec5413dde821dfafbd4f70deb6","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-06-23 17:33:37.000000000","message":"Patch Set 1:\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"d5348949ab780dc912a509cb023e71e02e8355c4","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-23 17:59:19.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/0583e4e193d34d2682b56255400ac58c : SUCCESS in 17m 40s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/5c0e2c0fb25a4141b5a04d99c36fc060 : SUCCESS in 27m 20s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a0c998e6f973789a39d6ae39be8cd3ecaeb55c3c","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-06-23 19:34:36.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/797725/1/iscsi : FAILURE in 2h 13m 50s","accounts_in_message":[],"_revision_number":1},{"id":"d8428b72662275523d1c7467679ff78dc2cddd3a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-23 20:16:53.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/7434e56e3cd74f5ab6ed1498d623141b : SUCCESS in 1h 19m 44s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/68668ab316174dfda51586011488843a : SUCCESS in 19m 26s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/e5dd18e2f14442768b775f7bf5d39783 : SUCCESS in 16m 26s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/345b19141b7146efa0370443a4e104c5 : SUCCESS in 8m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/87bf872257944ae194cfcba82954b517 : SUCCESS in 15m 11s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/88601fa6114a49a18fbd3b89b9408c5d : SUCCESS in 13m 47s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/043dd36c17174d22ae76d569a5142451 : SUCCESS in 14m 10s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1f591f0374494c7982d57152bd8bb174 : SUCCESS in 10m 49s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/06936bd9c3c442bb9f87631a8d8450c5 : SUCCESS in 1h 13m 04s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/d8f83895ce3c49d990c824e95fe15a04 : SUCCESS in 1h 16m 44s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/c4dacf9a845a45e88bba0459befcd495 : POST_FAILURE in 21m 11s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/555f4ab2bed645c693472970b45aa3fc : FAILURE in 52m 04s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/4f93492e5b454fd8a8e74fb0ac854dbd : SUCCESS in 1h 19m 09s\n- nova-next https://zuul.opendev.org/t/openstack/build/a31e826550c843d4aff00bec98f47f26 : SUCCESS in 2h 00m 16s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/ca7f751559244628ab7f52df3c269fa8 : SUCCESS in 6m 50s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/848ee9b08f3c4caa97e29d8febfc6ad5 : SUCCESS in 20m 29s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/4b3b81e0f19544a4a0ac7d6be673d924 : SUCCESS in 23m 10s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/5ba7075a3865495980f14922cd22b181 : FAILURE in 1h 26m 56s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/d641deba6d254e0bbf5b18a95c00dc9a : SUCCESS in 57m 11s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/298bbd7569cb473bbb34e60bf3a2507f : SUCCESS in 47m 34s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/41c79a2eccab4edf8e18ff194ca3fe7c : FAILURE in 31m 04s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/da2739acf3ee422d82746edfee67aba0 : SUCCESS in 43m 16s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"edf006273a5a42bd400c009b0d6e6e04f262430d","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-23 20:27:41.000000000","message":"Patch Set 1:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/25/797725/1/check/tempest-dsvm-full-focal-py3/abe39ba/ : FAILURE in 50m 40s","accounts_in_message":[],"_revision_number":1},{"id":"48f418d3b1a272d4df6c20d34d1e801378075517","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-24 09:38:35.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"8b8758f4faa96bd977f9c7490e31acf3e4d1bee9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-24 09:42:16.000000000","message":"Uploaded patch set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"8a550a817f65d5174ce3c9373108751c741ba1f3","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-24 10:18:52.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/a16eaf6bac9d469db1bb74228a814284 : SUCCESS in 28m 59s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/54662a876e73412082d454fb7f9feedc : SUCCESS in 31m 01s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"a1d39e4c3b08f125dfec2380b8423564b66e60e5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-24 11:16:42.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/3ebf1c9805a04c54bf9ecb3b7ab8e14e : SUCCESS in 1h 13m 44s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/69067b4e56144dc69a181fa8453cdfe7 : SUCCESS in 16m 25s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/5c07467653244660bbc34587a7efc804 : SUCCESS in 13m 45s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d890b14d18dd4986b55c5f34830b2278 : SUCCESS in 9m 10s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a1fb317e5a22418baa4e724266153a9c : SUCCESS in 20m 49s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/76bc1b3bbde647d189a84afb8221e439 : SUCCESS in 13m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1803e0b75952402fb9df804b11a764ed : SUCCESS in 14m 19s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9c890fbf50a740c18bff345c9522b532 : SUCCESS in 12m 18s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/a5b17f8932a34523a93f4ccd8b6f4ffa : SUCCESS in 1h 13m 44s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/7b5584293bf24c5fb99fcc415f38f067 : SUCCESS in 1h 12m 04s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/9a3f37fd80e2486995bb4d5c0dd56e03 : FAILURE in 54m 09s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/85ca15bc15074b7fa5285df2ffe26bee : FAILURE in 1h 02m 38s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/ea8961a0ec6b48ae8967c7998f352a6d : SUCCESS in 1h 20m 49s\n- nova-next https://zuul.opendev.org/t/openstack/build/ff7a6c602a5d473db9c0f2bcd8412787 : SUCCESS in 1h 26m 35s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/8cb6325181994e4c909efc9248db6b85 : SUCCESS in 7m 28s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/33377a7678544e34a22b3eac3bc2aedf : SUCCESS in 20m 53s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/83f1a3f27b494bc980be12b098f362bd : SUCCESS in 22m 27s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/e07b16335c9842d49b216ffcace6626f : SUCCESS in 1h 15m 11s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f5096d2d97cc4ad5ae06ff37d98c5d5e : SUCCESS in 54m 04s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/03f7fe2171dd4fcbbbb95cbbb58c9a8c : SUCCESS in 49m 03s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/5aefd9d6b39540b99037bc5827601ac9 : FAILURE in 26m 37s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/fefe9d2c75a2436eac47f94f41c72eb1 : SUCCESS in 39m 10s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"0f59c97933e47622780485a87c4d8e9cea33256f","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-06-24 12:01:06.000000000","message":"Patch Set 3:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/797725/3/iscsi : FAILURE in 2h 17m 42s","accounts_in_message":[],"_revision_number":3},{"id":"b66af750a7f9f935efb53ed666558f56118cfb60","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-24 13:00:46.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/ef492a14f1bc436ab87216c9c572652a : POST_FAILURE in 1h 53m 48s","accounts_in_message":[],"_revision_number":3},{"id":"10a39c24eb04c5a371d9e1be726f9c11b7ed15a2","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-24 20:09:50.000000000","message":"Patch Set 3:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/797725/797725/3/check-not-vote/ext-nova-zuul/fd1a88b : SUCCESS in 59m 58s","accounts_in_message":[],"_revision_number":3},{"id":"df0607614b647d09f8fbd29f448a5e919a3ef1c5","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-25 08:01:23.000000000","message":"Patch Set 3:\n\nhttps://zuul.opendev.org/t/openstack/build/9a3f37fd80e2486995bb4d5c0dd56e03/log/compute1/logs/screen-n-cpu.txt#9526\n\nThis isn\u0027t great, seems we haven\u0027t been updating the connection_info passed to the destination?","accounts_in_message":[],"_revision_number":3},{"id":"c5e0efdb8d040b81bb1c81e37b6f56e2965ddf8f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-25 09:00:02.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"0969aa8dc49d192c3962d44f4be10ee909af553f","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-25 09:08:01.000000000","message":"Patch Set 4:\n\n\u003e Patch Set 3:\n\u003e \n\u003e https://zuul.opendev.org/t/openstack/build/9a3f37fd80e2486995bb4d5c0dd56e03/log/compute1/logs/screen-n-cpu.txt#9526\n\u003e \n\u003e This isn\u0027t great, seems we haven\u0027t been updating the connection_info passed to the destination?\n\nThis turned out to be because of the following optimisation in the DriverVolumeBlockDevice class:\n\nhttps://github.com/openstack/nova/blob/e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1/nova/virt/block_device.py#L678-L680","accounts_in_message":[],"_revision_number":4},{"id":"d1197fba87af05ba2c2e6c3113779a6444f41feb","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-25 09:36:46.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/a2c8978567014bffaaed981d7a162aa3 : SUCCESS in 21m 30s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/4a59117312d74f6e825f2f32518f78f4 : SUCCESS in 34m 33s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"2547c398fdb258ce5218798c5863c76da892b84d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-25 11:09:24.000000000","message":"Patch Set 4: 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/3dd4317fa7044643bc72ae49b2660cd0 : SUCCESS in 1h 16m 58s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/7f038346ba034a45a5b248421d793347 : SUCCESS in 17m 49s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/8075f6502cef4bc9912655c85be64196 : SUCCESS in 24m 25s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/3bdb39ee9b954890974b3c8e140f1238 : SUCCESS in 15m 07s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/62fd6b5fa1264068928599778ca0c60a : SUCCESS in 21m 14s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/ce795699449a41f7b7a1e187a1efa675 : SUCCESS in 15m 06s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/54c574e05db94ff189e9e7e53b392721 : SUCCESS in 15m 46s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d60b0335b7e74a99a032cbfbabe90871 : SUCCESS in 11m 51s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/9e200433fade4106b851fb00e79aab59 : SUCCESS in 1h 22m 07s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/a10690bb97f94a62bf8b7f97959b49af : SUCCESS in 1h 12m 24s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/fb9e560dfac64e329be7a22b4ca842b4 : SUCCESS in 55m 42s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/a9567777cbd34e21b051a31537e7e661 : SUCCESS in 58m 33s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/3dd5ac8f3f1e45e0bb31764bee2e4ed7 : SUCCESS in 1h 33m 56s\n- nova-next https://zuul.opendev.org/t/openstack/build/0bc80ec30d264d6f92010547d8ca255d : SUCCESS in 1h 29m 13s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/cd8c54626b884636b4a2a56bc43e043f : SUCCESS in 7m 44s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/19f9ffa97e444c569e35f1aecc2be962 : SUCCESS in 22m 46s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3e793071cda6416584ffeb98064a137b : SUCCESS in 20m 40s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/a499c33aaed94dcfb2800eae21b1aba6 : FAILURE in 1h 13m 30s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4102a1150ad14b6789b7a9eab02ba39d : SUCCESS in 53m 25s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/5d821ecf283f4a8285ef2ed6420309e5 : SUCCESS in 50m 35s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/91734cd87df8485a9283eb3ee52341e6 : SUCCESS in 31m 27s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/2cfd195b61504b7587f24c40d3581f0e : SUCCESS in 36m 39s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"a2a5163034f8d9a76ff7c3740fd46a0187fc5885","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-06-25 11:39:25.000000000","message":"Patch Set 4: Workflow-1\n\nArgh and this breaks N to N-1 migrations as the destination doesn\u0027t refresh the connection_info and fails because it\u0027s still set to None when we try to connect during pre_live_migration.\n\nI guess the only way to handle this is to *only* unset connection_info on the destination when we know both hosts are new enough to handle it.","accounts_in_message":[],"_revision_number":4},{"id":"2c22d6ee1c9cbdfa6c843c8b2c0cfb0a212780ed","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-06-25 13:13:47.000000000","message":"Patch Set 4:\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/797725/4/iscsi : SUCCESS in 2h 18m 43s","accounts_in_message":[],"_revision_number":4},{"id":"1aec1cdd17f4b11c04d8c40c59e0f048820dbd58","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-25 16:30:31.000000000","message":"Patch Set 4:\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/32ede84234b940119bf076f866508044 : POST_FAILURE in 1h 55m 37s","accounts_in_message":[],"_revision_number":4},{"id":"4bf240b031f74f1427776320b22addf2ec8ebcba","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-25 17:51:56.000000000","message":"Patch Set 4:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/797725/797725/4/check-not-vote/ext-nova-zuul/54b0ffb : SUCCESS in 1h 10m 29s","accounts_in_message":[],"_revision_number":4},{"id":"98a9d25698cf0f236b70dd6d0fadf99478bcbd81","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-12 14:06:36.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"f6f7fecb58f19dd8610f4ae9d569f76c8c7b04ed","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 14:35:31.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/5d91383f904640bb946c70a07ff82091 : FAILURE in 23m 17s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/bcfdaca7b9234b6bb477b0e3c9d09ac2 : FAILURE in 26m 55s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"725418b92d5720a4ae671abd5cd863612244f74d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 15:42:19.000000000","message":"Patch Set 5: 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/d7b51e7fe87c4ec6a657c1a6e842688b : SUCCESS in 1h 12m 36s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1cbf204132c34c22a7fe796772721a60 : FAILURE in 13m 51s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/632095a87c8341e6a61c94f26e2eac3f : FAILURE in 15m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/38c04032658d460d81cdfb0bd4ecca0d : SUCCESS in 9m 13s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f70ed7598dd2459ba171a27b07ed5792 : FAILURE in 12m 08s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/33cc20e8e61f40569d51ae2b658192d8 : FAILURE in 13m 48s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/f320c7c9b9c7436b9196239a4647fb56 : FAILURE in 13m 50s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6826dfa39cac4b788e49b61798a445aa : SUCCESS in 11m 09s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/f665a60b3e2d46c2887f6cb6597ee948 : SUCCESS in 1h 16m 41s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/68492200388149b79dd053a9ecde6f7e : SUCCESS in 1h 20m 58s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/b5e95d795cf44ebab48f9dcf9b1aea60 : SUCCESS in 50m 48s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/084021ff71844c6d92dabd1f4dc7941f : SUCCESS in 53m 36s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/2a1a19faa3714fa2b3235b38574b6a02 : SUCCESS in 1h 00m 50s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/3f8c2cd965ba44238e14932bad8e0e9b : SUCCESS in 1h 14m 28s\n- nova-next https://zuul.opendev.org/t/openstack/build/c824d002576d4706b241158931cba62e : SUCCESS in 1h 31m 08s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/75fb58a210b5414aaee9f2951fe21f03 : SUCCESS in 6m 03s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/820277cc66bc460d92787f96a0e018eb : FAILURE in 20m 39s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/a20345ababba47f09b7517367216419d : FAILURE in 19m 12s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/68326aa5b51e4f5d87b20e308a3dcfd6 : FAILURE in 18m 41s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/fb33dee984694322aa70a67819974376 : SUCCESS in 1h 12m 02s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/906193bf4d264c829c4fc47ba49b710b : SUCCESS in 51m 23s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/10df9a151c7c4ff6944a6e01d8e6c331 : SUCCESS in 45m 45s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/84b0d0002136432cbedb8d31ad71525d : SUCCESS in 31m 00s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/42b277434c98468da8e269476df38662 : SUCCESS in 36m 50s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"99ae8dacfec963a2ede507a6fe89759e52935ea5","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-12 16:22:36.000000000","message":"Patch Set 5:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/797725/5/iscsi : FAILURE in 2h 11m 30s","accounts_in_message":[],"_revision_number":5},{"id":"048668c63ad4838c3f6458ac623994d331d69df8","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-08-12 16:29:49.000000000","message":"Patch Set 5:\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/25/797725/5/check/tempest-dsvm-full-focal-py3/901fc7f/ : FAILURE in 2h 01m 48s","accounts_in_message":[],"_revision_number":5},{"id":"5a0c7640ca6a7e6511b5173a3a5d7e4cd9e68054","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-08-18 09:25:28.000000000","message":"Topic set to bug/1899835","accounts_in_message":[],"_revision_number":5},{"id":"f2e78662882dcddd72dc55711b8e765c6754b3ac","tag":"autogenerated:gerrit:abandon","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-12-03 12:08:14.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":5},{"id":"d3b5af8009b4aa63fbc6055281134217b1116ca9","tag":"autogenerated:gerrit:restore","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2023-07-12 15:45:04.000000000","message":"Restored","accounts_in_message":[],"_revision_number":5},{"id":"e0b79fe0a7b1c9e7de18e8faf61176765867ac88","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-07-12 15:56:35.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/fdcf9c9c84324c4e87a5addb2997b3ea\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/0ffd2c61ca23415c83e34bd6cceadab2 : FAILURE in 10m 33s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/5928ffb5440a4a6b878794fd14ef32d2 : FAILURE in 10m 25s (non-voting)\n- openstack-tox-py310-arm64 https://zuul.opendev.org/t/openstack/build/934e12e314c642d0917ae3b86354a90c : FAILURE in 9m 57s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"8ac6173ff73dd89fa4ee64deca8d5eaff4118e09","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-07-12 18:15:42.000000000","message":"Patch Set 5:\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\nhttps://zuul.opendev.org/t/openstack/buildset/7820563e6a72477a94ee1731ec7357e1\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/ee935db70b0945cd9cff94ff607b2f25 : SUCCESS in 58m 29s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/a22b60598dce409a8b34ebf5bfa54289 : SUCCESS in 1h 49m 58s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/14b709093aba4d16998b8712d27b7534 : SUCCESS in 1h 12m 56s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b1305349f448461c884043dc03b9cd14 : FAILURE in 22m 41s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c474b8cb9297433e9156cde6b2c2049a : SUCCESS in 10m 34s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/df1df71e481544d9870ac161cbbd142e : FAILURE in 14m 25s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/fe3ddb8d718e473ead565fb798f6710a : FAILURE in 19m 49s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/46ca7ca033494999ba7c5c73351d95a5 : FAILURE in 13m 53s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dcfb62c8c14a4b07836bf23ff1175b7c : SUCCESS in 12m 01s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/9757415c16564d4f88058e5870c14977 : SUCCESS in 58m 14s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/85eb43fd7f784758bf1e65c730598682 : SUCCESS in 1h 57m 08s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/b950b2a2bd1941d0aeadbd1a7fb88486 : FAILURE in 1h 21m 43s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/0496566c90354be7b37b7d592d2c0ec0 : FAILURE in 58m 54s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/d74e4776e0d145f89d882aef14948810 : SUCCESS in 1h 13m 26s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/0803a073f41b443cba5b1989c24259e7 : SUCCESS in 2h 01m 34s\n- nova-next https://zuul.opendev.org/t/openstack/build/69f6440921924f44ad3723d3f22821c1 : FAILURE in 2h 12m 07s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/9b3d108931064700a2bcc468436ec022 : SUCCESS in 54m 06s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/55a7b2e75fd3492986064c36b482463d : SUCCESS in 5m 38s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/69979204543b45bd875c7deeb0345e7b : FAILURE in 24m 30s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/affadb94b49d4defae6b03cf6e808a2b : FAILURE in 30m 46s\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/810b14ccdac3410a83c33f292c36ea32 : FAILURE in 28m 49s\n- tempest-integrated-compute-enforce-scope-new-defaults https://zuul.opendev.org/t/openstack/build/f9049913fbc546efb4e970b9a4563d28 : SUCCESS in 1h 29m 53s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/837b22e01f5a4e329438f7d0f39f5364 : SUCCESS in 1h 08m 01s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/aa6207214fb849ccac3558a3f1ab99d7 : FAILURE in 2h 13m 56s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/01c890d99ddb468992647f3ffe210b5e : SUCCESS in 21m 34s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/ac350bf7bcea49e69c425d188fa2f737 : SUCCESS in 30m 10s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"2582d858a92d9f424cb878207185a00fea867ce5","tag":"autogenerated:gerrit:abandon","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2026-06-17 15:36:58.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"723f357c059c6075f38d339b586f5322987c91b7","revisions":{"d8975b80433373da71bc0d506dfd64bd62e3da69":{"kind":"REWORK","_number":1,"created":"2021-06-23 17:19:12.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/25/797725/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/25/797725/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/25/797725/1"}}},"commit":{"parents":[{"commit":"e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1","subject":"Merge \"Fix max cpu topologies with numa affinity\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 16:47:51.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 17:17:11.000000000","tz":60},"subject":"WIP compute: Avoid calling detach with src connection_info during LM rollback","message":"WIP compute: Avoid calling detach with src connection_info during LM rollback\n\nWhen rolling back from an early pre_live_migration failure Nova would\npreviously attempt to detach volumes from the destination using the\nconnection_info associated with the source.\n\nThis can lead to failures being logged (and ignored since ) by certain\nvirt drivers as this connection_info could point to a different\nunderlying volume on the destination.\n\nThis change seeks to avoid such attempts by nulling out the\nconnection_info of the block device mapping records *after* stashing a\ncopy to use if we rollback but *before* calling pre_live_migration on\nthe destination. This allows us to later check if any connection_info is\nset in the block device mapping record before attempting to detach\nduring a rollback, before the original connection info is then restored.\n\nThe previously written functional test is extended to assert that\nthe original connection_info is restored after the rollback.\n\nCloses-Bug: #1899835\nChange-Id: Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d8975b80433373da71bc0d506dfd64bd62e3da69"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d8975b80433373da71bc0d506dfd64bd62e3da69"}]},"branch":"refs/heads/master"},"c29817c162babe30b9a6dc14bde6ffbd407f09e9":{"kind":"REWORK","_number":2,"created":"2021-06-24 09:38:35.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/25/797725/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/25/797725/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/25/797725/2"}}},"commit":{"parents":[{"commit":"e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1","subject":"Merge \"Fix max cpu topologies with numa affinity\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 16:47:51.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-24 09:37:57.000000000","tz":60},"subject":"compute: Avoid calling detach with src connection_info during LM rollback","message":"compute: Avoid calling detach with src connection_info during LM rollback\n\nWhen rolling back from an early pre_live_migration failure Nova would\npreviously attempt to detach volumes from the destination using the\nconnection_info associated with the source.\n\nThis can lead to failures being logged (and ignored since ) by certain\nvirt drivers as this connection_info could point to a different\nunderlying volume on the destination.\n\nThis change seeks to avoid such attempts by nulling out the\nconnection_info of the block device mapping records *after* stashing a\ncopy to use if we rollback but *before* calling pre_live_migration on\nthe destination. This allows us to later check if any connection_info is\nset in the block device mapping record before attempting to detach\nduring a rollback, before the original connection info is then restored.\n\nThe previously written functional test is extended to assert that\nthe original connection_info is restored after the rollback.\n\nCloses-Bug: #1899835\nChange-Id: Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c29817c162babe30b9a6dc14bde6ffbd407f09e9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c29817c162babe30b9a6dc14bde6ffbd407f09e9"}]},"branch":"refs/heads/master"},"f80832e2608b0e02a90fce98c7d275d87f4d74b3":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2021-06-24 09:42:16.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/25/797725/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/25/797725/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/25/797725/3"}}},"commit":{"parents":[{"commit":"e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1","subject":"Merge \"Fix max cpu topologies with numa affinity\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 16:47:51.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-24 09:41:08.000000000","tz":60},"subject":"compute: Avoid calling detach with src connection_info during LM rollback","message":"compute: Avoid calling detach with src connection_info during LM rollback\n\nWhen rolling back from an early pre_live_migration failure Nova would\npreviously attempt to detach volumes from the destination using the\nconnection_info associated with the source.\n\nThis can lead to failures being logged (and ignored since\nI6bc73e8c8f98d9955f33f309beb8a7c56981b553) by certain virt drivers and\nvolume backends as this connection_info could either point to a\ndifferent underlying volume on the destination when compared to the\nsource or nothing at all on the destination.\n\nThis change seeks to avoid such attempts by nulling out the\nconnection_info of the block device mapping records *after* stashing a\ncopy to use if we rollback but *before* calling pre_live_migration on\nthe destination. This allows us to later check if any connection_info is\nset in the block device mapping record before attempting to detach on\nthe destination during a rollback, before the original connection info\nis then restored for use on the source.\n\nThe previously written functional test is extended to assert that\nthe original connection_info is restored after the rollback.\n\nCloses-Bug: #1899835\nChange-Id: Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f80832e2608b0e02a90fce98c7d275d87f4d74b3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f80832e2608b0e02a90fce98c7d275d87f4d74b3"}]},"branch":"refs/heads/master"},"c18c01b041b0eb47ac7f36e0d9cd39845ebcd571":{"kind":"REWORK","_number":4,"created":"2021-06-25 09:00:02.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/25/797725/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/25/797725/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/25/797725/4"}}},"commit":{"parents":[{"commit":"e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1","subject":"Merge \"Fix max cpu topologies with numa affinity\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2f3d8f5ec27fdbd81509bd3f6b348003c1566e1"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 16:47:51.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-25 08:59:36.000000000","tz":60},"subject":"compute: Avoid calling detach with src connection_info during LM rollback","message":"compute: Avoid calling detach with src connection_info during LM rollback\n\nWhen rolling back from an early pre_live_migration failure Nova would\npreviously attempt to detach volumes from the destination using the\nconnection_info associated with the source.\n\nThis can lead to failures being logged (and ignored since\nI6bc73e8c8f98d9955f33f309beb8a7c56981b553) by certain virt drivers and\nvolume backends as this connection_info could either point to a\ndifferent underlying volume on the destination when compared to the\nsource or nothing at all on the destination.\n\nThis change seeks to avoid such attempts by nulling out the\nconnection_info of the block device mapping records *after* stashing a\ncopy to use if we rollback but *before* calling pre_live_migration on\nthe destination. This allows us to later check if any connection_info is\nset in the block device mapping record before attempting to detach on\nthe destination during a rollback, before the original connection_info\nis then restored for use on the source.\n\nTo allow this we need to remove an old untested optimisation added by\nIf961329061e5413824d891eada645dd8dd9cb6dd within the\nrefresh_connection_info method of the DriverVolumeBlockDevice class that\nwould skip a refresh when connection_info is unset. We obviously need\nthis to happen still during pre_live_migration to ensure the fresh\nconnection_info for the destination is used to initially connect any\nvolumes. The removal of this optimisation shouldn\u0027t have any knock on\neffects as all callers expect connection_info to already be set aside\nfrom pre_live_migration with the newly introduced logic above.\n\nFinally, the previously written functional test is extended to assert\nthat the original connection_info is restored after the rollback.\n\nCloses-Bug: #1899835\nChange-Id: Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c18c01b041b0eb47ac7f36e0d9cd39845ebcd571"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c18c01b041b0eb47ac7f36e0d9cd39845ebcd571"}]},"branch":"refs/heads/master"},"723f357c059c6075f38d339b586f5322987c91b7":{"kind":"REWORK","_number":5,"created":"2021-08-12 14:06:36.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/25/797725/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/25/797725/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/25/797725/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/25/797725/5"}}},"commit":{"parents":[{"commit":"e0cacaa397f8414871f1e2cd589928f9afc27dc7","subject":"Merge \"tox: Encode specific Python versions\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e0cacaa397f8414871f1e2cd589928f9afc27dc7"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-06-23 16:47:51.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2021-08-12 14:04:32.000000000","tz":60},"subject":"WIP compute: Avoid calling detach with src connection_info during LM rollback","message":"WIP compute: Avoid calling detach with src connection_info during LM rollback\n\nWhen rolling back from an early pre_live_migration failure Nova would\npreviously attempt to detach volumes from the destination using the\nconnection_info associated with the source.\n\nThis can lead to failures being logged (and ignored since\nI6bc73e8c8f98d9955f33f309beb8a7c56981b553) by certain virt drivers and\nvolume backends as this connection_info could either point to a\ndifferent underlying volume on the destination when compared to the\nsource or nothing at all on the destination.\n\nThis change seeks to avoid such attempts by nulling out the\nconnection_info of the block device mapping records *after* stashing a\ncopy to use if we rollback but *before* calling pre_live_migration on\nthe destination. This allows us to later check if any connection_info is\nset in the block device mapping record before attempting to detach on\nthe destination during a rollback, before the original connection_info\nis then restored for use on the source.\n\nTo allow this we need to remove an old untested optimisation added by\nIf961329061e5413824d891eada645dd8dd9cb6dd within the\nrefresh_connection_info method of the DriverVolumeBlockDevice class that\nwould skip a refresh when connection_info is unset. We obviously need\nthis to happen still during pre_live_migration to ensure the fresh\nconnection_info for the destination is used to initially connect any\nvolumes. The removal of this optimisation shouldn\u0027t have any knock on\neffects as all callers expect connection_info to already be set aside\nfrom pre_live_migration with the newly introduced logic above.\n\nLibvirtLiveMigrateData is extended to allow the destination host of a\nlive migration to indicate if it supports the above logic, otherwise the\nconnection_info will not be unset and the previous behaviour will be\nused. This is required in order to support live migrations between N and\nN-1 computes where the latter doesn\u0027t support the new behaviour.\n\nFinally, the previously written functional test is extended to assert\nthat the original connection_info is restored after the rollback.\n\nCloses-Bug: #1899835\nChange-Id: Ifef629d1c79a4fd2b4c864080bd7a37c2fbd69bf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/723f357c059c6075f38d339b586f5322987c91b7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/723f357c059c6075f38d339b586f5322987c91b7"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"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":""}}}]}