)]}'
{"id":"openstack%2Fnova~659338","triplet_id":"openstack%2Fnova~stable%2Frocky~I933687891abef4878de09481937d576ce5899511","project":"openstack/nova","branch":"stable/rocky","topic":"bug/1821594","hashtags":[],"change_id":"I933687891abef4878de09481937d576ce5899511","subject":"[stable-only] Delete allocations even if _confirm_resize raises (part 2)","status":"MERGED","created":"2019-05-15 16:39:44.000000000","updated":"2019-05-24 19:11:03.000000000","submitted":"2019-05-24 19:11:03.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":8,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"659338-1558725063995-ec6a912d","meta_rev_id":"b4b6bfb480a9a2512870a81f9a5d9802e2bcf8da","_number":659338,"virtual_id_number":659338,"owner":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"date":"2019-05-23 11:39:23.000000000","_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"date":"2019-05-23 18:59:36.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":2,"date":"2019-05-24 19:11:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"date":"2019-05-23 07:18:46.000000000","_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"date":"2019-05-24 15:42:42.000000000","_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"value":0,"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},{"value":0,"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"}],"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":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"all":[{"value":0,"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},{"value":2,"date":"2019-05-24 05:53:37.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},{"value":2,"date":"2019-05-24 15:31:01.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2019-05-22 20:49:59.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":1,"date":"2019-05-24 12:51:43.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"value":1,"date":"2019-05-24 05:55:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},{"value":0,"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"}],"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},{"value":1,"date":"2019-05-24 15:31:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"value":0,"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},{"value":0,"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"value":0,"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},{"value":0,"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-05-15 16:41:58.000000000","updated_by":{"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-05-15 16:51:19.000000000","updated_by":{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},"reviewer":{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},"state":"REVIEWER"},{"updated":"2019-05-15 22:40:45.000000000","updated_by":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"reviewer":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"state":"REVIEWER"},{"updated":"2019-05-15 22:45:50.000000000","updated_by":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"reviewer":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"state":"REVIEWER"},{"updated":"2019-05-22 20:49:59.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":"2019-05-23 07:18:46.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":"REVIEWER"},{"updated":"2019-05-23 11:39:23.000000000","updated_by":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-05-23 18:59:36.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":"REVIEWER"},{"updated":"2019-05-24 05:53:37.000000000","updated_by":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"reviewer":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"state":"REVIEWER"},{"updated":"2019-05-24 05:55:06.000000000","updated_by":{"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},"reviewer":{"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},"state":"REVIEWER"},{"updated":"2019-05-24 12:51:43.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-05-24 15:31:01.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2019-05-24 19:11:03.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"}],"messages":[{"id":"774e040a4959e95039a089c037ba9e0771259ced","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-15 16:39:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1666e77f24fdcf918f6e064ba8455cd82143693a","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-15 16:41:43.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"5bfdb91845aa65ec240b882ee1d3c3c4ba660409","author":{"_account_id":16128,"name":"IBM PowerVM CI","email":"powervmci@linux.vnet.ibm.com","username":"powervmci","tags":["SERVICE_USER"]},"date":"2019-05-15 16:41:58.000000000","message":"Patch Set 1:\n\nBuild failed. Comment \u0027powervm: recheck\u0027 to recheck.\n For 3rd party CI contact info: https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_PowerVM_CI\n\n- nova-out-of-tree-pvm http://184.172.12.213/38/659338/1/check/nova-out-of-tree-pvm/a341c79 : FAILURE in 6s\n- nova-in-tree-pvm http://184.172.12.213/38/659338/1/check/nova-in-tree-pvm/29a8cfd : FAILURE in 6s","accounts_in_message":[],"_revision_number":1},{"id":"9de4473ba0abb25c6ed6409070a46b625f74019c","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2019-05-15 17:56:41.000000000","message":"Patch Set 1:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/tempest-dsvm-full-xenial/56edb2a/ : SUCCESS in 1h 09m 53s\n- tempest-dsvm-full-xenial-py3 https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/tempest-dsvm-full-xenial-py3/8bca8f0/ : SUCCESS in 1h 15m 31s (non-voting)\n- grenade-dsvm-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/grenade-dsvm-xenial/77cda72/ : SUCCESS in 1h 01m 16s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a00e69d71cc7902a3dece6060cb6487063f2be01","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-05-15 18:38:21.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/659338/1 : SUCCESS in 1h 57m 15s","accounts_in_message":[],"_revision_number":1},{"id":"789a88b2be723492037e0962b472603dcea2ed8b","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2019-05-15 19:01:37.000000000","message":"Patch Set 1:\n\nTesting failed ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenial-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/659338/1/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/711b028 : FAILURE in 1h 04m 45s","accounts_in_message":[],"_revision_number":1},{"id":"8d8478553f17c71fb44111a40e1fa1f1265d5492","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-15 19:13:38.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 http://logs.openstack.org/38/659338/1/check/openstack-tox-pep8/622f7cd/ : SUCCESS in 10m 50s\n- openstack-tox-py27 http://logs.openstack.org/38/659338/1/check/openstack-tox-py27/3adf556/ : SUCCESS in 13m 02s\n- openstack-tox-py35 http://logs.openstack.org/38/659338/1/check/openstack-tox-py35/ea27c35/ : SUCCESS in 14m 26s\n- build-openstack-sphinx-docs http://logs.openstack.org/38/659338/1/check/build-openstack-sphinx-docs/e6fa1b2/html/ : SUCCESS in 7m 32s\n- tempest-full http://logs.openstack.org/38/659338/1/check/tempest-full/e1d1455/ : SUCCESS in 1h 31m 05s\n- neutron-grenade http://logs.openstack.org/38/659338/1/check/neutron-grenade/eac72be/ : SUCCESS in 53m 45s\n- tempest-full-py3 http://logs.openstack.org/38/659338/1/check/tempest-full-py3/7d104f7/ : SUCCESS in 1h 25m 17s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/38/659338/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/3fdbb69/ : FAILURE in 48m 43s (non-voting)\n- nova-cells-v1 http://logs.openstack.org/38/659338/1/check/nova-cells-v1/89b1282/ : SUCCESS in 57m 36s\n- nova-grenade-live-migration http://logs.openstack.org/38/659338/1/check/nova-grenade-live-migration/ccae357/ : SUCCESS in 1h 06m 54s (non-voting)\n- nova-live-migration http://logs.openstack.org/38/659338/1/check/nova-live-migration/1946a86/ : SUCCESS in 49m 52s\n- nova-multiattach http://logs.openstack.org/38/659338/1/check/nova-multiattach/011b27b/ : SUCCESS in 1h 17m 51s\n- nova-next http://logs.openstack.org/38/659338/1/check/nova-next/4537169/ : FAILURE in 1h 12m 46s\n- tempest-slow http://logs.openstack.org/38/659338/1/check/tempest-slow/1fc365a/ : SUCCESS in 2h 03m 23s\n- nova-tox-functional http://logs.openstack.org/38/659338/1/check/nova-tox-functional/913404a/ : SUCCESS in 15m 21s\n- nova-tox-functional-py35 http://logs.openstack.org/38/659338/1/check/nova-tox-functional-py35/f6f228c/ : SUCCESS in 19m 28s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/659338/1/check/openstack-tox-lower-constraints/09789b1/ : SUCCESS in 12m 17s\n- neutron-grenade-multinode http://logs.openstack.org/38/659338/1/check/neutron-grenade-multinode/5a1e4e5/ : SUCCESS in 1h 00m 10s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/38/659338/1/check/devstack-plugin-ceph-tempest/b2f458b/ : SUCCESS in 1h 15m 57s (non-voting)\n- tempest-multinode-full http://logs.openstack.org/38/659338/1/check/tempest-multinode-full/159dd06/ : SUCCESS in 1h 35m 37s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"ec97982360be03a1f56d2c0ca4526779b7a813ba","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-15 20:03:17.000000000","message":"Patch Set 1:\n\nrecheck test_tagged_attachment ssh","accounts_in_message":[],"_revision_number":1},{"id":"4ca837b775dd6b41973728a9cb168b7030d024e7","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2019-05-15 21:18:16.000000000","message":"Patch Set 1:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/tempest-dsvm-full-xenial/2a13b0b/ : SUCCESS in 1h 14m 18s\n- tempest-dsvm-full-xenial-py3 https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/tempest-dsvm-full-xenial-py3/fdec2ec/ : SUCCESS in 1h 13m 55s (non-voting)\n- grenade-dsvm-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/1/check/grenade-dsvm-xenial/2b099b3/ : SUCCESS in 1h 05m 00s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b73ab10331fcfc373c734ed9f2baf1f9af89e52a","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-05-15 21:54:09.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/659338/1 : SUCCESS in 1h 50m 18s","accounts_in_message":[],"_revision_number":1},{"id":"bf40df521c27e2c9ff8cdb578b42cdafcd9c5b85","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-15 22:25:00.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/38/659338/1/check/openstack-tox-pep8/4318a0f/ : SUCCESS in 11m 10s\n- openstack-tox-py27 http://logs.openstack.org/38/659338/1/check/openstack-tox-py27/9a1c156/ : SUCCESS in 13m 15s\n- openstack-tox-py35 http://logs.openstack.org/38/659338/1/check/openstack-tox-py35/2c9c371/ : SUCCESS in 15m 29s\n- build-openstack-sphinx-docs http://logs.openstack.org/38/659338/1/check/build-openstack-sphinx-docs/01cb092/html/ : SUCCESS in 6m 53s\n- tempest-full http://logs.openstack.org/38/659338/1/check/tempest-full/23a99ff/ : SUCCESS in 1h 30m 50s\n- neutron-grenade http://logs.openstack.org/38/659338/1/check/neutron-grenade/cca367d/ : SUCCESS in 53m 41s\n- tempest-full-py3 http://logs.openstack.org/38/659338/1/check/tempest-full-py3/f424aaf/ : SUCCESS in 1h 26m 39s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/38/659338/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/e4c828a/ : FAILURE in 43m 00s (non-voting)\n- nova-cells-v1 http://logs.openstack.org/38/659338/1/check/nova-cells-v1/b11e1bc/ : SUCCESS in 57m 20s\n- nova-grenade-live-migration http://logs.openstack.org/38/659338/1/check/nova-grenade-live-migration/9afe642/ : SUCCESS in 1h 17m 07s (non-voting)\n- nova-live-migration http://logs.openstack.org/38/659338/1/check/nova-live-migration/211448e/ : SUCCESS in 41m 48s\n- nova-multiattach http://logs.openstack.org/38/659338/1/check/nova-multiattach/81d1dbc/ : SUCCESS in 1h 03m 12s\n- nova-next http://logs.openstack.org/38/659338/1/check/nova-next/3f96f60/ : SUCCESS in 1h 14m 56s\n- tempest-slow http://logs.openstack.org/38/659338/1/check/tempest-slow/4a72c3e/ : SUCCESS in 2h 17m 20s\n- nova-tox-functional http://logs.openstack.org/38/659338/1/check/nova-tox-functional/b3a0e52/ : SUCCESS in 18m 24s\n- nova-tox-functional-py35 http://logs.openstack.org/38/659338/1/check/nova-tox-functional-py35/f7967fe/ : SUCCESS in 15m 14s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/659338/1/check/openstack-tox-lower-constraints/c27febb/ : SUCCESS in 12m 45s\n- neutron-grenade-multinode http://logs.openstack.org/38/659338/1/check/neutron-grenade-multinode/1a38484/ : SUCCESS in 1h 25m 04s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/38/659338/1/check/devstack-plugin-ceph-tempest/92ac891/ : FAILURE in 1h 20m 38s (non-voting)\n- tempest-multinode-full http://logs.openstack.org/38/659338/1/check/tempest-multinode-full/5915d07/ : SUCCESS in 1h 47m 17s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f1cda02e6d645bd4823428179c18e1a283023bd2","author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"date":"2019-05-15 22:40:45.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"86e73b00ef02057345a1d6c453b8f835cfdbfa87","author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"date":"2019-05-15 22:45:50.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"6c47ee4eeae05a18ea080bbef4c78fbdf810ebac","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-05-17 22:07:26.000000000","message":"Patch Set 1:\n\n(3 comments)\n\nI\u0027m pretty sure from the commit message that only one of the _delete_allocation_after_move call sites need the temporary mutation but I\u0027m asking inline anyway as a sanity check.","accounts_in_message":[],"_revision_number":1},{"id":"481e4d2c89dd995b3a0b4eaff2f817cd2015f047","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-22 20:12:48.000000000","message":"Patch Set 1:\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"4f77a99e544aa8d3e7140ba1ca029c7aefb2af39","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-05-22 20:30:34.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nThanks for confirming the other call sites don\u0027t need to be changed. This looks reasonable to me. Proxying Matt\u0027s backport as a stable core as a +2.","accounts_in_message":[],"_revision_number":1},{"id":"632638f362bc9ca467202557e398f16f57a7c549","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-22 20:31:32.000000000","message":"Patch Set 1:\n\n\u003e Proxying Matt\u0027s backport as a stable core as a +2.\n\nThis isn\u0027t really a backport so it\u0027s probably not really appropriate to proxy my change as being a +2.","accounts_in_message":[],"_revision_number":1},{"id":"b9e9651c6e91016054fcefcf5e88f4fc952d2b11","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-22 20:32:03.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"dab490dc5feea39a02a10a71b9e1ec1820d6e414","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-05-22 20:36:58.000000000","message":"Patch Set 1: -Workflow\n\nThis is stable-only, not a backport, so should not proxy a +2.","accounts_in_message":[],"_revision_number":1},{"id":"a5794defed95fd9ed628dbd2ca4379cb9dc6ed69","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-22 20:46:02.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"d103f591510ca02dbc3ddd65e5c0a9dc60bdb611","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-05-22 20:49:59.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"bbb1f88854077a08ab437b4bb1a21c500a35e270","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-22 22:36:01.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/38/659338/2/check/openstack-tox-pep8/0537303/ : SUCCESS in 10m 03s\n- openstack-tox-py27 http://logs.openstack.org/38/659338/2/check/openstack-tox-py27/a9ad076/ : SUCCESS in 13m 22s\n- openstack-tox-py35 http://logs.openstack.org/38/659338/2/check/openstack-tox-py35/78dbe4c/ : SUCCESS in 12m 46s\n- build-openstack-sphinx-docs http://logs.openstack.org/38/659338/2/check/build-openstack-sphinx-docs/a588ce6/html/ : SUCCESS in 6m 38s\n- tempest-full http://logs.openstack.org/38/659338/2/check/tempest-full/2b2dfa2/ : SUCCESS in 1h 15m 58s\n- neutron-grenade http://logs.openstack.org/38/659338/2/check/neutron-grenade/9560fe1/ : SUCCESS in 58m 27s\n- tempest-full-py3 http://logs.openstack.org/38/659338/2/check/tempest-full-py3/7400b49/ : SUCCESS in 1h 13m 40s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/38/659338/2/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/83fdd33/ : SUCCESS in 42m 22s (non-voting)\n- nova-cells-v1 http://logs.openstack.org/38/659338/2/check/nova-cells-v1/6bb4962/ : SUCCESS in 59m 20s\n- nova-grenade-live-migration http://logs.openstack.org/38/659338/2/check/nova-grenade-live-migration/887bf03/ : SUCCESS in 55m 52s (non-voting)\n- nova-live-migration http://logs.openstack.org/38/659338/2/check/nova-live-migration/1e45742/ : SUCCESS in 37m 47s\n- nova-multiattach http://logs.openstack.org/38/659338/2/check/nova-multiattach/058169e/ : SUCCESS in 51m 24s\n- nova-next http://logs.openstack.org/38/659338/2/check/nova-next/52ad9a1/ : SUCCESS in 1h 21m 47s\n- tempest-slow http://logs.openstack.org/38/659338/2/check/tempest-slow/e51a1b4/ : SUCCESS in 1h 44m 55s\n- nova-tox-functional http://logs.openstack.org/38/659338/2/check/nova-tox-functional/87896eb/ : SUCCESS in 16m 31s\n- nova-tox-functional-py35 http://logs.openstack.org/38/659338/2/check/nova-tox-functional-py35/4b750bf/ : SUCCESS in 14m 18s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/659338/2/check/openstack-tox-lower-constraints/e637886/ : SUCCESS in 14m 04s\n- neutron-grenade-multinode http://logs.openstack.org/38/659338/2/check/neutron-grenade-multinode/f9da008/ : SUCCESS in 54m 16s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/38/659338/2/check/devstack-plugin-ceph-tempest/951f549/ : SUCCESS in 1h 14m 22s (non-voting)\n- tempest-multinode-full http://logs.openstack.org/38/659338/2/check/tempest-multinode-full/ca361da/ : SUCCESS in 1h 23m 48s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"625b85c5f94f6837045422ba8ee3dc1e78ddd20e","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-05-23 07:18:46.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/659338/2 : SUCCESS in 1h 58m 12s","accounts_in_message":[],"_revision_number":2},{"id":"c20dd47344f98e9e59f8a2158acb57bca1f86c4d","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2019-05-23 11:39:23.000000000","message":"Patch Set 2:\n\nTesting failed ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenial-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/659338/2/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/fe39e10 : FAILURE in 1h 30m 24s","accounts_in_message":[],"_revision_number":2},{"id":"57b26c4f6224c76857f924d4c769f7972de1bcee","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2019-05-23 18:59:36.000000000","message":"Patch Set 2:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/2/check/tempest-dsvm-full-xenial/184ab60/ : SUCCESS in 1h 09m 44s\n- tempest-dsvm-full-xenial-py3 https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/2/check/tempest-dsvm-full-xenial-py3/cf50c13/ : FAILURE in 1h 20m 12s (non-voting)\n- grenade-dsvm-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/38/659338/2/check/grenade-dsvm-xenial/87d01c3/ : SUCCESS in 1h 02m 57s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"a8a7e05e1abb9b04dedb08aa6032f749d4b25f3e","author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"date":"2019-05-24 05:53:37.000000000","message":"Patch Set 2: Code-Review+2\n\nLooks good to me but as this is stable only I want (at least) Dan to look at it","accounts_in_message":[],"_revision_number":2},{"id":"bf7f949f1492cc0a517168a8218c78122d56d98e","author":{"_account_id":27679,"name":"gaoruyue","email":"gaory@awcloud.com","username":"gaoruyue"},"date":"2019-05-24 05:55:06.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"ffd43c87f50213ca0a34c22ec317ac726f27c34c","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-05-24 12:51:43.000000000","message":"Patch Set 2: Code-Review+1\n\nLGTM. Functional test in the next patch fails in the expected way with the fix reverted. Unfortunate that we had to do it this way, but it\u0027s not tech debt as it\u0027s \"already\" resolved in subsequent releases.","accounts_in_message":[],"_revision_number":2},{"id":"936a01c25d9c50d10ea8f048d03d95feb6cee076","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-05-24 15:31:01.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"5bac2a5a7f985e40c4de7bcf326d8f3bb1afec5f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-24 15:31:42.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"edb3c2177c2ad3f52c70fc39d61432b802eab2a4","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-05-24 15:42:42.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"d95c667026d05af3a0faf15a5cce9d9649a006ac","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-24 19:11:03.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/38/659338/2/gate/openstack-tox-pep8/e0100f9/ : SUCCESS in 11m 20s\n- openstack-tox-py27 http://logs.openstack.org/38/659338/2/gate/openstack-tox-py27/843db25/ : SUCCESS in 12m 29s\n- openstack-tox-py35 http://logs.openstack.org/38/659338/2/gate/openstack-tox-py35/9eac48a/ : SUCCESS in 12m 23s\n- build-openstack-sphinx-docs http://logs.openstack.org/38/659338/2/gate/build-openstack-sphinx-docs/ad4f567/html/ : SUCCESS in 7m 42s\n- tempest-full http://logs.openstack.org/38/659338/2/gate/tempest-full/f5b7360/ : SUCCESS in 1h 21m 27s\n- neutron-grenade http://logs.openstack.org/38/659338/2/gate/neutron-grenade/c856539/ : SUCCESS in 46m 40s\n- tempest-full-py3 http://logs.openstack.org/38/659338/2/gate/tempest-full-py3/96a402f/ : SUCCESS in 1h 13m 26s\n- nova-cells-v1 http://logs.openstack.org/38/659338/2/gate/nova-cells-v1/255a2f6/ : SUCCESS in 1h 02m 58s\n- nova-live-migration http://logs.openstack.org/38/659338/2/gate/nova-live-migration/cc64691/ : SUCCESS in 45m 46s\n- nova-multiattach http://logs.openstack.org/38/659338/2/gate/nova-multiattach/4a0ec6f/ : SUCCESS in 53m 34s\n- nova-next http://logs.openstack.org/38/659338/2/gate/nova-next/08c6793/ : SUCCESS in 1h 23m 24s\n- tempest-slow http://logs.openstack.org/38/659338/2/gate/tempest-slow/f003bdb/ : SUCCESS in 1h 51m 10s\n- nova-tox-functional http://logs.openstack.org/38/659338/2/gate/nova-tox-functional/cbd6bb4/ : SUCCESS in 16m 07s\n- nova-tox-functional-py35 http://logs.openstack.org/38/659338/2/gate/nova-tox-functional-py35/14c3055/ : SUCCESS in 24m 03s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/659338/2/gate/openstack-tox-lower-constraints/7e3d961/ : SUCCESS in 29m 05s","accounts_in_message":[],"_revision_number":2},{"id":"eb331da84dfcd21879de36f227d781c7613d6172","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-24 19:11:03.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"dac3239e92fc1865cacc17bbfbd2316072a9d26e","revisions":{"5c4df173baa5c332cca63dbd69ae1ba37f39e0be":{"kind":"REWORK","_number":1,"created":"2019-05-15 16:39:44.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/38/659338/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/38/659338/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/38/659338/1"}}},"commit":{"parents":[{"commit":"8e130e2be7c2eb604d53fd6053254c6b54d47524","subject":"Use migration_status during volume migrating and retyping","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8e130e2be7c2eb604d53fd6053254c6b54d47524"}]}],"author":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-05-15 16:27:17.000000000","tz":-240},"committer":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-05-15 16:27:17.000000000","tz":-240},"subject":"Delete allocations even if _confirm_resize raises (part 2)","message":"Delete allocations even if _confirm_resize raises (part 2)\n\nThe backport 37ac54a42ec91821d62864d63c486c002608491b to fix\nbug 1821594 did not account for how the _delete_allocation_after_move\nmethod before Stein is tightly coupled to the migration status\nbeing set to \"confirmed\" which is what the _confirm_resize method\ndoes after self.driver.confirm_migration returns.\n\nHowever, if self.driver.confirm_migration raises an exception\nwe still want to cleanup the allocations held on the source node\nand for that we call _delete_allocation_after_move. But because\nof that tightly coupling before Stein, we need to temporarily\nmutate the migration status to \"confirmed\" to get the cleanup\nmethod to do what we want.\n\nThis isn\u0027t a problem starting in Stein because change\nI0851e2d54a1fdc82fe3291fb7e286e790f121e92 removed that\ntight coupling on the migration status, so this is a stable\nbranch only change.\n\nNote that we don\u0027t call self.reportclient.delete_allocation_for_instance\ndirectly since before Stein we still need to account for a\nmigration that does not move the source node allocations to the\nmigration record, and that logic is in _delete_allocation_after_move.\n\nA simple unit test assertion is added here but the functional\ntest added in change I9d6478f492351b58aa87b8f56e907ee633d6d1c6\nwill assert the bug is fixed properly before Stein.\n\nChange-Id: I933687891abef4878de09481937d576ce5899511\nCloses-Bug: #1821594\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5c4df173baa5c332cca63dbd69ae1ba37f39e0be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5c4df173baa5c332cca63dbd69ae1ba37f39e0be"}]},"branch":"refs/heads/stable/rocky"},"dac3239e92fc1865cacc17bbfbd2316072a9d26e":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2019-05-22 20:46:02.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/38/659338/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/38/659338/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/38/659338/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/38/659338/2"}}},"commit":{"parents":[{"commit":"8e130e2be7c2eb604d53fd6053254c6b54d47524","subject":"Use migration_status during volume migrating and retyping","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8e130e2be7c2eb604d53fd6053254c6b54d47524"}]}],"author":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-05-15 16:27:17.000000000","tz":-240},"committer":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-05-22 20:46:00.000000000","tz":0},"subject":"[stable-only] Delete allocations even if _confirm_resize raises (part 2)","message":"[stable-only] Delete allocations even if _confirm_resize raises (part 2)\n\nThe backport 37ac54a42ec91821d62864d63c486c002608491b to fix\nbug 1821594 did not account for how the _delete_allocation_after_move\nmethod before Stein is tightly coupled to the migration status\nbeing set to \"confirmed\" which is what the _confirm_resize method\ndoes after self.driver.confirm_migration returns.\n\nHowever, if self.driver.confirm_migration raises an exception\nwe still want to cleanup the allocations held on the source node\nand for that we call _delete_allocation_after_move. But because\nof that tight coupling before Stein, we need to temporarily\nmutate the migration status to \"confirmed\" to get the cleanup\nmethod to do what we want.\n\nThis isn\u0027t a problem starting in Stein because change\nI0851e2d54a1fdc82fe3291fb7e286e790f121e92 removed that\ntight coupling on the migration status, so this is a stable\nbranch only change.\n\nNote that we don\u0027t call self.reportclient.delete_allocation_for_instance\ndirectly since before Stein we still need to account for a\nmigration that does not move the source node allocations to the\nmigration record, and that logic is in _delete_allocation_after_move.\n\nA simple unit test assertion is added here but the functional\ntest added in change I9d6478f492351b58aa87b8f56e907ee633d6d1c6\nwill assert the bug is fixed properly before Stein.\n\nChange-Id: I933687891abef4878de09481937d576ce5899511\nCloses-Bug: #1821594\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dac3239e92fc1865cacc17bbfbd2316072a9d26e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/dac3239e92fc1865cacc17bbfbd2316072a9d26e"}]},"branch":"refs/heads/stable/rocky"}},"requirements":[],"submit_records":[],"submit_requirements":[]}