)]}'
{"id":"openstack%2Fnova~702261","triplet_id":"openstack%2Fnova~master~Ied8f96b4e67f79498519931cb6b35dad5288bbb8","project":"openstack/nova","branch":"master","topic":"bp/support-move-ops-with-qos-ports-ussuri","hashtags":[],"change_id":"Ied8f96b4e67f79498519931cb6b35dad5288bbb8","subject":"Remove extra instance.save() calls related to qos SRIOV ports","status":"MERGED","created":"2020-01-13 17:05:59.000000000","updated":"2020-02-03 19:58:45.000000000","submitted":"2020-02-03 19:55:34.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":14,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"702261-1580759735062-a46f390a","meta_rev_id":"e4e9de804c87b009d0911f398bbc9f8e4bbf222b","_number":702261,"virtual_id_number":702261,"owner":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"date":"2020-02-03 14:14:20.000000000","_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"value":0,"date":"2020-02-03 12:16:30.000000000","_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"date":"2020-02-03 13:49:04.000000000","_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"value":0,"date":"2020-02-03 18:14:00.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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"date":"2020-02-03 12:25:45.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"date":"2020-02-03 10:51:56.000000000","_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-02-03 19:55:34.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"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":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-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":2,"date":"2020-02-03 17:42:53.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":2,"date":"2020-02-03 14:51:35.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"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":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-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":1,"date":"2020-02-03 17:42:53.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"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":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-01-17 17:46:23.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":"2020-01-21 17:12:48.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":"REVIEWER"},{"updated":"2020-02-03 10:51:56.000000000","updated_by":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"reviewer":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"state":"REVIEWER"},{"updated":"2020-02-03 12:16:30.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":"2020-02-03 12:25:45.000000000","updated_by":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"reviewer":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"state":"REVIEWER"},{"updated":"2020-02-03 13:49:04.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-02-03 14:14:20.000000000","updated_by":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-02-03 14:51:35.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":"2020-02-03 17:42:53.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2020-02-03 18:14:00.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":"2020-02-03 19:55:34.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"f70a7c764d6eafc64aad77c39caa9effc748dd3c","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-13 17:05:59.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"53e0deaa6aea2e1dff1591b21ba9dbb9540fe88a","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-13 17:10:01.000000000","message":"Patch Set 1: Workflow-1\n\nNeed to patch up the unit tests and most probably this can replace a sizeable portion of I7b37f84257e1f4a07929cf1b282f013ff7bd68e8","accounts_in_message":[],"_revision_number":1},{"id":"6825b77d5ef01a6f595144a90fbe583fa8dce971","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-01-13 17:30:39.000000000","message":"Patch Set 1:\n\nBuild SUCCESSFUL (check pipeline).\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/61/702261/1/check/pmem-tempest-plugin-filtered/bc3252c/ : SUCCESS in 22m 17s","accounts_in_message":[],"_revision_number":1},{"id":"a74d84b4e3de8634b3e4f95a85cffbf0e1e4e82b","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-01-13 18:20:38.000000000","message":"Patch Set 1:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/61/702261/1/check-vote/ext-nova-zuul/bb7cb90 : FAILURE in 1h 12m 23s","accounts_in_message":[],"_revision_number":1},{"id":"88c978e2800b29f6558ad4de7995bb71a166ed27","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-13 19:04:06.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- grenade-py3 https://zuul.opendev.org/t/openstack/build/fab1ffa9714247e18fccb625c77da55c : SUCCESS in 58m 33s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/60804acef12c46678dd705ea59a48b78 : SUCCESS in 1h 31m 09s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/367fb8f0a32746db8c5834ea6b3bb052 : FAILURE in 15m 32s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/fcc0564b6a044a6b98bc0b0d23dca62e : FAILURE in 15m 41s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/69398e9ed435429b9928f55df9e39e58 : SUCCESS in 10m 26s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/77585f8617f24dd692b7d781803de9f1 : FAILURE in 12m 40s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/2b42ce36496a40fd87c6babfb35733d5 : FAILURE in 12m 56s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2d47714ca2a24183835e4e8703a50295 : SUCCESS in 16m 01s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/497b27c74f944f57aaa9687de95d1157 : SUCCESS in 58m 05s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/a1e99b12abd84bfea9bce54caa3b75d6 : SUCCESS in 59m 38s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/b8930a58e68849cfb6c65b38cd027b9e : SUCCESS in 1h 23m 49s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/dfaf64b677ec4caab1988a7c9ffa2391 : SUCCESS in 51m 19s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/675a97a81de14ce892cd4639d43287cb : SUCCESS in 1h 22m 11s\n- nova-next https://zuul.opendev.org/t/openstack/build/3fda85ef21f847549aa463a0dae04144 : SUCCESS in 1h 50m 24s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/a5f314e8a7a6477db664211890f8f6a2 : SUCCESS in 17m 00s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9d7153c140784d8084feecc8bd4a0344 : SUCCESS in 1h 07m 48s","accounts_in_message":[],"_revision_number":1},{"id":"19e3b5cc960b19d1b616892cc576f41065685f84","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-01-13 19:25:00.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/702261/1 : SUCCESS in 2h 15m 58s","accounts_in_message":[],"_revision_number":1},{"id":"7c477d050f88dc2aa577e442aefe54a91aeb2721","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-14 16:51:24.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"53747b3aeb39c9b50927473a3c77fd5f302d7700","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-01-14 17:16:06.000000000","message":"Patch Set 2:\n\nBuild SUCCESSFUL (check pipeline).\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/61/702261/2/check/pmem-tempest-plugin-filtered/107b2c2/ : SUCCESS in 22m 45s","accounts_in_message":[],"_revision_number":2},{"id":"3abf375ac81c03b6f2f56170a17d1c6802736883","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-01-14 19:23:38.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/702261/2 : SUCCESS in 2h 30m 36s","accounts_in_message":[],"_revision_number":2},{"id":"79d7da54563db28ae7c5fadf0338d2a0416f8ee3","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2020-01-14 21:22:28.000000000","message":"Patch Set 2:\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-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/2/check/tempest-dsvm-full-xenial/ed7b5d6/ : FAILURE in 1h 27m 43s\n- tempest-dsvm-full-xenial-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/2/check/tempest-dsvm-full-xenial-py3/d3d4586/ : FAILURE in 1h 23m 20s (non-voting)\n- grenade-dsvm-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/2/check/grenade-dsvm-xenial/c4cd25a/ : FAILURE in 2h 55m 48s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f506e816bdda9cae506c593525fee89159d3f176","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-15 02:57:39.000000000","message":"Patch Set 2: 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- grenade-py3 https://zuul.opendev.org/t/openstack/build/81e7d87c67244bd9b3351ad838b5bf21 : FAILURE in 1h 17m 42s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/f0131fe3c54c4b909ea837e408c1ce77 : SUCCESS in 1h 34m 00s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/fdceb749a714416da53e797fbe16a134 : SUCCESS in 19m 20s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/bfbd0d539cbb4381b209f0585a064c60 : SUCCESS in 17m 17s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/852e6ff912874d8a86c3fa66c95302d2 : SUCCESS in 12m 33s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/836ccbae408d4098bf33f29273198e8c : SUCCESS in 15m 46s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/01d7cbf6984b43ce8032ef593d0db640 : SUCCESS in 15m 23s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/356d0393934e4d5fbe00158e399ee2a5 : SUCCESS in 9m 15s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/4242ebdd65054904930814c56629dca3 : SUCCESS in 1h 04m 34s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/8f22ea6dc6a545c1b720b9461d9cd9d9 : SUCCESS in 1h 04m 10s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/bc902d16d8834e11a257c9a9df576e42 : SUCCESS in 1h 22m 43s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/e468ad0fd21a40de8361796ca2b49031 : SUCCESS in 55m 51s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/a2c509f7d3c24e9cb4aa2017e0569abc : SUCCESS in 1h 29m 36s\n- nova-next https://zuul.opendev.org/t/openstack/build/be904e09f9a84db682fce33cb7d9c287 : SUCCESS in 1h 37m 50s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/5a5bd81e43c64df795916a6008d04aa9 : SUCCESS in 19m 17s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/01a06c9e77ed4e9c8f1931d622113503 : SUCCESS in 1h 00m 33s","accounts_in_message":[],"_revision_number":2},{"id":"c67063d8e2b267886f7d9e10d9dd48a96b16ce58","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-15 11:40:52.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"7d30d7bcb594f84ae7b484ec91e8fa7911cd67ad","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-01-15 12:05:37.000000000","message":"Patch Set 2:\n\nBuild SUCCESSFUL (check pipeline).\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/61/702261/2/check/pmem-tempest-plugin-filtered/764428c/ : SUCCESS in 23m 03s","accounts_in_message":[],"_revision_number":2},{"id":"017a6d63fdaaf99e11288a545bc249b69eff6bc2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-15 14:23:55.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/d24cc3eba1fc46b3bde301c808cc4ecb : SUCCESS in 1h 00m 11s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/1ba1f754959f4700b4864ffefbf20c35 : SUCCESS in 1h 28m 21s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/62f199aa2b914a8eab830b2fc8fc0fa2 : SUCCESS in 15m 23s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/e9fa068f2df1408c9e915ffa7456f1d7 : SUCCESS in 18m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2261a784647749f88d531bef0f259f79 : SUCCESS in 10m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/074bd82e43ef418aaecbbdeef93c7e3d : SUCCESS in 13m 24s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/7698054388a44c09bf15e6cd686b0002 : SUCCESS in 16m 33s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/94b0f2d12d4146d1bfeface361ed401b : SUCCESS in 9m 38s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/a1b8cb0df3d44b1eb62dea63802d3d12 : SUCCESS in 56m 46s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/ee825bc216f84cd3a4dd5208e7a6b60b : SUCCESS in 1h 10m 16s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/6dc926ae001644efbda39da464dc480f : SUCCESS in 1h 33m 05s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/48344f863b8f483ab1ec1e74bcc52454 : SUCCESS in 1h 02m 53s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/6207f5b627324206aab79f4bf2cfd76d : SUCCESS in 1h 35m 04s\n- nova-next https://zuul.opendev.org/t/openstack/build/f5da86ee800d46bc912bd83e19941524 : SUCCESS in 1h 32m 22s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/99b75bcb4304469782fed55c6b1056c5 : SUCCESS in 17m 01s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/fd49a2801cc14f24bd5d21c090e3565d : SUCCESS in 1h 02m 25s","accounts_in_message":[],"_revision_number":2},{"id":"a4bd111de149da6def89e80be28545f826d01b43","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2020-01-15 14:24:40.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_VxFlexOS_NOVA http://publiclogs.emc.com/61/702261/2/check/EMC_VxFlexOS_NOVA/26b9fe7/EMC_VxFlexOS_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC VxFlexOS CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VxFlexOS_CI","accounts_in_message":[],"_revision_number":2},{"id":"f319bc125fbeabbe905dbfe73cc4b9b95b2d157d","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-01-15 14:53:38.000000000","message":"Patch Set 2:\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/702261/2 : FAILURE in 20m 36s","accounts_in_message":[],"_revision_number":2},{"id":"661477c5cb48244dc88955533928f8bb37ab7318","author":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"date":"2020-01-15 15:03:14.000000000","message":"Patch Set 2:\n\n* nova-quobyteci-dsvm-volume http://78.46.57.153:8081/refs-changes-61-702261-2 : FAILURE \n\nSee https://wiki.openstack.org/wiki/ThirdPartySystems/Quobyte_CI for rechecking and info.","accounts_in_message":[],"_revision_number":2},{"id":"53e90d51fce640ab3eb45b8f0f1b4b3c0f67fcff","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-01-15 15:50:54.000000000","message":"Patch Set 2: Code-Review+2\n\nThe tests show that this works. It\u0027d still be good to have dansmith look at this, but I\u0027m happy with this","accounts_in_message":[],"_revision_number":2},{"id":"ff4770453ac8291aaf35e65a8d116979d836fd2e","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-01-17 07:00:37.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/61/702261/2/check-vote/ext-nova-zuul/e40b0c2 : FAILURE in 34m 41s","accounts_in_message":[],"_revision_number":2},{"id":"40260634a2825627b4b57ef5bf70279614499ff3","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2020-01-17 16:02:35.000000000","message":"Patch Set 2: Code-Review+2\n\nLooks like good simplification to me. But I\u0027d really like Dan to look.","accounts_in_message":[],"_revision_number":2},{"id":"3b59ff24ac22925c7abaa661ccffe888b4c07ab8","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-01-17 17:28:58.000000000","message":"Patch Set 2: Code-Review-1\n\n(4 comments)\n\n-1 for visibility at least, although this also has whitespace damage which should be an easy fix and re-approve, even if my questions have legit answers. Otherwise, it *seems* like this might have an issue if applied to some of the computes and not others, leading to live migration incompatibilities.","accounts_in_message":[],"_revision_number":2},{"id":"56d8cca7635bba573a44499d4ca430f3d3f97598","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2020-01-17 17:46:23.000000000","message":"Patch Set 2: -Code-Review","accounts_in_message":[],"_revision_number":2},{"id":"56f8835792081c12424771b44887e8fcec94fc21","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-20 13:39:04.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"a4d986acae7d2f1f19e9ce1b7636eb356cd99ece","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-20 13:46:18.000000000","message":"Patch Set 3:\n\n(4 comments)","accounts_in_message":[],"_revision_number":3},{"id":"2d9c32ee42c043e5d994e1c235c5c0a03e19f5a8","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-01-20 13:56:47.000000000","message":"Patch Set 3:\n\nBuild failed.\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/61/702261/3/check/pmem-tempest-plugin-filtered/f32a54b/ : FAILURE in 16m 24s","accounts_in_message":[],"_revision_number":3},{"id":"da7582744d01c8c9a4d3610cc4178bd7f49d3c93","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-20 15:37:51.000000000","message":"Patch Set 3: 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- grenade-py3 https://zuul.opendev.org/t/openstack/build/126c9443254147d6b15d39d5662ff9c3 : SUCCESS in 56m 26s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/e062397af58745acbf7c06517bc70969 : SUCCESS in 1h 18m 31s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/035afac61cc14e799cda0e406245fbad : SUCCESS in 22m 01s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7a64b2381f0049c0bd07ee266b24f837 : SUCCESS in 17m 17s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f2f96b01b32c4a3f8adb49fb5aabee8a : SUCCESS in 11m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c879af8337d94b6cb8eb6b5d178abe09 : SUCCESS in 13m 35s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/45e59b5905e343fab83bb37a4b2df257 : SUCCESS in 14m 00s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/33fdd9f0d77d4af389a3c369f46d378a : SUCCESS in 10m 01s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/a7e972537acc4c248f74a40342b4778a : SUCCESS in 59m 34s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/8403745da9db44b5ab5cfb0f7977eebe : SUCCESS in 1h 13m 48s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/d4dfb6cf426c4e69b0a35ea9862a16db : FAILURE in 1h 47m 59s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/a4ce0c9ce49f4309b9865f095e263079 : SUCCESS in 48m 15s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/fb6a097b45ca410ba248c20ce6f1134c : SUCCESS in 1h 29m 37s\n- nova-next https://zuul.opendev.org/t/openstack/build/5383b245a4d94cea8120284cb4d67a26 : SUCCESS in 1h 35m 27s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/4724d5580e924ffd88d16d146338a0a5 : SUCCESS in 19m 26s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e708ef609e674e878707b57915fe65a9 : SUCCESS in 1h 01m 21s","accounts_in_message":[],"_revision_number":3},{"id":"0227d8f9ad3ea5330b4c8af41c517ff408b9ea57","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2020-01-20 16:25:14.000000000","message":"Patch Set 3:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/3/check/tempest-dsvm-full-xenial/95bc90e/ : FAILURE in 1h 23m 23s\n- tempest-dsvm-full-xenial-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/3/check/tempest-dsvm-full-xenial-py3/949da3e/ : FAILURE in 1h 32m 29s (non-voting)\n- grenade-dsvm-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/3/check/grenade-dsvm-xenial/a0742d9/ : FAILURE in 2h 43m 48s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"57195d60855f7b80ed22411472f7e5a71255a33f","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-01-20 16:42:20.000000000","message":"Patch Set 3:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/702261/3 : SUCCESS in 3h 01m 30s","accounts_in_message":[],"_revision_number":3},{"id":"92cea35a8f1a03e5a4d278f5a1191a08561ee3cb","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-01-20 17:38:07.000000000","message":"Patch Set 3:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/61/702261/3/check-vote/ext-nova-zuul/90f1f87 : FAILURE in 32m 20s","accounts_in_message":[],"_revision_number":3},{"id":"5a35940d2a3fb87b7daac0e540b153a045991bbe","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-21 13:41:19.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"84f1ac6ea4aada1eb421c25a184c9d0b56d9571b","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-01-21 14:10:22.000000000","message":"Patch Set 4:\n\nBuild SUCCESSFUL (check pipeline).\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/61/702261/4/check/pmem-tempest-plugin-filtered/abf9805/ : SUCCESS in 27m 27s","accounts_in_message":[],"_revision_number":4},{"id":"ef3cad53f6fe0a772f85242368a02aaab1a1887f","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-21 15:19:20.000000000","message":"Patch Set 4:\n\nI\u0027ve needed to fix a merge conflict with one of the nova-net removal patches","accounts_in_message":[],"_revision_number":4},{"id":"1fa7f16debb5711cd0a413a5c1a2e1bcb011267d","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-01-21 16:16:54.000000000","message":"Patch Set 4:\n\nBuild succeeded.\n\n- nova http://cloudbase-ci.com/nova/702261/4 : SUCCESS in 2h 34m 15s","accounts_in_message":[],"_revision_number":4},{"id":"d371f5e9b943a7d5ef032f19efb2d440148d64ec","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2020-01-21 16:26:01.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-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/4/check/tempest-dsvm-full-xenial/feaa75b/ : FAILURE in 1h 23m 34s\n- tempest-dsvm-full-xenial-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/4/check/tempest-dsvm-full-xenial-py3/e114653/ : FAILURE in 1h 27m 38s (non-voting)\n- grenade-dsvm-xenial https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/4/check/grenade-dsvm-xenial/039d6d6/ : FAILURE in 2h 41m 31s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"9c57461b643670258edd23892f97ac3442428a9a","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-01-21 17:12:48.000000000","message":"Patch Set 4:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/61/702261/4/check-vote/ext-nova-zuul/c2dc532 : FAILURE in 31m 23s","accounts_in_message":[],"_revision_number":4},{"id":"1519c70fa433d5ca80007dbba72d1a6104868ec0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-21 17:48:31.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/d08425ec3c264193a4e2b4deea5523af : SUCCESS in 1h 00m 38s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/c4e1770d6d0e4f889e03093bec7b5a54 : SUCCESS in 1h 17m 40s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/527544175e2a45e3b597a222937fdbb8 : SUCCESS in 18m 04s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/a65c1a1304634bfdae2cd73a3d00e146 : SUCCESS in 15m 16s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/beca6bd772a14cb1925352532287e7bc : SUCCESS in 13m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/7fd102bd86d44f3499b1d2b58a2840b8 : SUCCESS in 14m 09s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/3e9df4c2946c4956a13682945d1a4926 : SUCCESS in 15m 17s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9aabd176dc864eee8b13c629e2d534f7 : SUCCESS in 12m 21s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/39444f9e0c074b7b8ed0288dd4cafccc : SUCCESS in 1h 16m 55s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/abe9aac08d0c430287b14f3f3a2afeb5 : SUCCESS in 1h 09m 17s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/5ac2f44671064f29bffdf647192806ea : SUCCESS in 1h 16m 10s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/94586c721f404cca981daa97266d0efa : SUCCESS in 1h 19m 32s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/54afdda815d54ec2a5b076775528f6c1 : SUCCESS in 1h 25m 54s\n- nova-next https://zuul.opendev.org/t/openstack/build/c5df624d5d6446cb85903a015e487f62 : SUCCESS in 1h 34m 24s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/d5652e7b557543f8bf093b6708eb380b : SUCCESS in 21m 18s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/7a5fd7f937fc4fb4a45213d8cecdec66 : SUCCESS in 1h 29m 49s","accounts_in_message":[],"_revision_number":4},{"id":"00028c6856c8a884890b3e56beb4b976a6c5da16","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2020-01-21 17:49:06.000000000","message":"Patch Set 4:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_VxFlexOS_NOVA http://publiclogs.emc.com/61/702261/4/check/EMC_VxFlexOS_NOVA/e510c08/EMC_VxFlexOS_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC VxFlexOS CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VxFlexOS_CI","accounts_in_message":[],"_revision_number":4},{"id":"533bd630af449d79f9bd5bc8940507ea8f3f6867","author":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"date":"2020-01-21 18:35:35.000000000","message":"Patch Set 4:\n\n* nova-quobyteci-dsvm-volume http://78.46.57.153:8081/refs-changes-61-702261-4 : FAILURE \n\nSee https://wiki.openstack.org/wiki/ThirdPartySystems/Quobyte_CI for rechecking and info.","accounts_in_message":[],"_revision_number":4},{"id":"f6d1e7aa947698c8334106d0c833549c836c471f","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-01-30 17:24:56.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"cf22c7dc7140d468699842de6eadeb110bda8e75","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-30 17:26:46.000000000","message":"Patch Set 4: Workflow-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"34a7a36eba116d20ad9540457d4ff64ce8aedbbf","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-31 09:45:42.000000000","message":"Patch Set 4: -Workflow\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"5b4c1a8166b3a06c581959d06bae82919649beb9","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-01-31 15:04:58.000000000","message":"Patch Set 4:\n\n(1 comment)\n\nI\u0027m still concerned about this \"spooky action at a distance\" despite the fact that there are other things broken which make it hard to know what all has changed.\n\nOne option, by the way, is to directly \"refresh\" the pci_requests by grabbing them specifically. Either by reaching into the instance with instance._load_pci_requests() or by doing what that function does and updating that field.\n\nI feel like my original argument(s) about this being a big hammer still stand, and that other bugs don\u0027t mitigate my concerns about that, nor do they really justify the use of the big hammer. I don\u0027t want to stand in the way of what I\u0027m sure is _a_ solution to an actual problem, I just feel like it\u0027s fragile (if the destination side code started to save() it would stop working) and heavyweight in that it refreshes everything.","accounts_in_message":[],"_revision_number":4},{"id":"e05540f4d5112ab8562dbbf3703937f7c9dd8103","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-01-31 17:15:07.000000000","message":"Patch Set 4: Workflow-1\n\nLet\u0027s make things not break if a future dev introduces an instance.save() in the live migration code path.","accounts_in_message":[],"_revision_number":4},{"id":"e6041a8f828bda5906cb5285bfa2773906ba9aa3","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-02-03 10:49:15.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"9e748aa0a5f8928f220786486893ac55ac1244e0","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2020-02-03 10:51:56.000000000","message":"Patch Set 5:\n\nBuild failed.\n\n- pmem-tempest-plugin-filtered finger://d0fe4b8af32b/b8f0b3d5eafb40d6aadc1096c840da5b : RETRY_LIMIT in 7s","accounts_in_message":[],"_revision_number":5},{"id":"ced6f78584554d11d528571e53cae09e39b2fc84","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2020-02-03 11:46:21.000000000","message":"Patch Set 4:\n\n(2 comments)\n\n\u003e (1 comment)\n \u003e \n \u003e I\u0027m still concerned about this \"spooky action at a distance\"\n \u003e despite the fact that there are other things broken which make it\n \u003e hard to know what all has changed.\n\nThe cause of the \"spooky action at a distance\" is that we pass around a dirty instance object that is save far from the place of the dirtyfication. But I\u0027m pretty sure that we have that all over nova. \n\n \u003e \n \u003e One option, by the way, is to directly \"refresh\" the pci_requests\n \u003e by grabbing them specifically. Either by reaching into the instance\n \u003e with instance._load_pci_requests() or by doing what that function\n \u003e does and updating that field.\n\nI implemented this now. \n\n \u003e \n \u003e I feel like my original argument(s) about this being a big hammer\n \u003e still stand, and that other bugs don\u0027t mitigate my concerns about\n \u003e that, nor do they really justify the use of the big hammer. I don\u0027t\n \u003e want to stand in the way of what I\u0027m sure is _a_ solution to an\n \u003e actual problem, I just feel like it\u0027s fragile (if the destination\n \u003e side code started to save() it would stop working) and heavyweight\n \u003e in that it refreshes everything.\n\nI made the hammer specific.","accounts_in_message":[],"_revision_number":4},{"id":"df832c969a3f5138f93ace4c5993475416eb091e","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2020-02-03 12:16:30.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/702261/5 : FAILURE in 17m 19s","accounts_in_message":[],"_revision_number":5},{"id":"404b8fa144630de6a4c49ba61539ab4c8cb0196f","author":{"_account_id":23498,"name":"IBM zVM CI","email":"zvmosci@us.ibm.com","username":"zvmosci"},"date":"2020-02-03 12:25:45.000000000","message":"Patch Set 5:\n\nTesting completed on the zVM Driver CI system check-nova pipeline and failed.  To recheck only the zVM driver plugins, submit a comment with only  zvm: recheck in the comment.. Contact information: zvmosci@us.ibm.com. For information see https://wiki.openstack.org/wiki/ZVMDriver.\n\n- check-nova-master http://extbasicopstackcilog01.podc.sl.edst.ibm.com/test_logs/jenkins-check-nova-master-40651 : FAILURE in 1h 03m 22s","accounts_in_message":[],"_revision_number":5},{"id":"583740185df69a6bd899765b5b67434e35962ac7","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2020-02-03 12:47:53.000000000","message":"Patch Set 5:\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-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/5/check/tempest-dsvm-full-xenial-py3/158b1e9/ : FAILURE in 31s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"41a659bc2f386d5ed53a76ad2a230c3e2fccd8d2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 13:48:34.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/b36a0522420e483e83a8d77167889c01 : SUCCESS in 1h 11m 38s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/b57add247d554da6900b84f133276041 : SUCCESS in 1h 22m 19s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2e86d1efb68e411794b161f0d880081c : SUCCESS in 25m 21s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/9e6d8ef4aa034475a82569338d63a22b : SUCCESS in 15m 35s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/96de8dc27b3048798da8de04fcdcbc47 : SUCCESS in 9m 44s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d306fdfad25b43d399354d7c9259bb9e : SUCCESS in 25m 58s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/8e92d77d37f74368afc51852ea65b485 : SUCCESS in 13m 00s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/8d11384205854402bb16de04e8f1fc08 : SUCCESS in 17m 35s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/98f856588a8944749399355b58d4c7b6 : SUCCESS in 13m 10s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/cf47a6f2c1c444ad92baf4b00c74a498 : SUCCESS in 1h 02m 03s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/35c30420cc3947708c3220a44aaa13fb : SUCCESS in 1h 05m 17s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/5d560c542bbb4ac395422bcc9990d9f1 : SUCCESS in 1h 21m 34s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/510b5c622f9d47b2806173b1309d99b7 : SUCCESS in 54m 15s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/ac8cc842664841fc8e325975b45dd44d : SUCCESS in 1h 49m 36s\n- nova-next https://zuul.opendev.org/t/openstack/build/0c25582e35bd43d1866ca78c030d0100 : SUCCESS in 1h 59m 15s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/bf57c7679af1441ca94a38cb2a062342 : SUCCESS in 17m 34s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/24c20e71179a48169b2173bf14ed8663 : SUCCESS in 1h 08m 45s","accounts_in_message":[],"_revision_number":5},{"id":"ad2afba9913b60ef2532d41c3180af73d160b43a","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2020-02-03 13:49:04.000000000","message":"Patch Set 5:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_VxFlexOS_NOVA http://publiclogs.emc.com/61/702261/5/check/EMC_VxFlexOS_NOVA/29ec380/EMC_VxFlexOS_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC VxFlexOS CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_VxFlexOS_CI","accounts_in_message":[],"_revision_number":5},{"id":"1ea5139a19bdb1af2d7b57c5d79de33167dacf39","author":{"_account_id":14384,"name":"Quobyte CI","email":"openstack-ci-external@quobyte.com","username":"quobyteci","tags":["SERVICE_USER"]},"date":"2020-02-03 14:14:20.000000000","message":"Patch Set 5:\n\n* nova-quobyteci-dsvm-volume http://78.46.57.153:8081/refs-changes-61-702261-5 : FAILURE \n\nSee https://wiki.openstack.org/wiki/ThirdPartySystems/Quobyte_CI for rechecking and info.","accounts_in_message":[],"_revision_number":5},{"id":"eade5bdb31c41219f16a58a0d8b9124a940661de","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-02-03 14:51:35.000000000","message":"Patch Set 5: Code-Review+2\n\n\u003e The cause of the \"spooky action at a distance\" is that we pass\n \u003e around a dirty instance object that is save far from the place of\n \u003e the dirtyfication. But I\u0027m pretty sure that we have that all over\n \u003e nova.\n\nWe do, and that\u0027s what the object is designed to allow. I think the thing that makes this different and obscure is that we pass the dirty object around and then hit a revert path where we would have been using a low-level time-sensitive DB refresh to facilitate a high level \"set the values back because we\u0027re aborting the whole operation\" activity. The high-level revert is almost like a migration back.\n\nAnyway, looks good to me now, thanks for your patience with my pedantry.","accounts_in_message":[],"_revision_number":5},{"id":"a96c7edfc8e18c519a586f7b5976e68c4df3c30e","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-02-03 17:42:53.000000000","message":"Patch Set 5: Code-Review+2 Workflow+1\n\nI\u0027m mostly going on the fact that dansmith has been watching this closely and it looks fine","accounts_in_message":[],"_revision_number":5},{"id":"f1acf75722726bcbe311c3094b1622114f7dbd69","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 17:43:44.000000000","message":"Patch Set 5: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":5},{"id":"9d73536c570ee5d48888b4c270091656534233f6","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2020-02-03 18:14:00.000000000","message":"Patch Set 5:\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-bionic https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/5/check/tempest-dsvm-full-bionic/803a4dc/ : SUCCESS in 2h 02m 30s\n- tempest-dsvm-full-bionic-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/5/check/tempest-dsvm-full-bionic-py3/73704c5/ : SUCCESS in 2h 06m 48s (non-voting)\n- grenade-dsvm-bionic https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/61/702261/5/check/grenade-dsvm-bionic/4cb9a8a/ : FAILURE in 2h 53m 51s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"5128f2e7480a4e05def2672f8a151ef2f335f35d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 19:55:34.000000000","message":"Patch Set 5: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/bdd70de044bd42c5bc7bad8272f80b5f : SUCCESS in 1h 10m 43s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/602ba6753b0848e78d2cc31bc8fa021a : SUCCESS in 1h 15m 47s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7506737c7f274635928adb2b152d450d : SUCCESS in 16m 29s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c061e8ffdf104a3097129a4438b93276 : SUCCESS in 10m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/95e678286371450880aa66de674993c1 : SUCCESS in 20m 18s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/71879b52924047d981b7905e0515889d : SUCCESS in 12m 59s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f514343fd11245d9948c161d9013539e : SUCCESS in 9m 38s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/b8267acbedfb484dbeaeca4b297efb4f : SUCCESS in 1h 13m 39s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/ee17d1545a3e42ea9a3c4235e281148b : SUCCESS in 51m 19s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/6767df7e552d4b05a15dd1cc3c137810 : SUCCESS in 18m 35s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/f2e18a83f67547f78f764d59075684b7 : SUCCESS in 1h 29m 42s\n- nova-next https://zuul.opendev.org/t/openstack/build/9940dc06fe564d1da91546984ce809fa : SUCCESS in 2h 02m 27s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/27a6733b59234854a4c4a88adbdddff6 : SUCCESS in 1h 16m 11s","accounts_in_message":[],"_revision_number":5},{"id":"1d361167d615466d72fc6aa4577f20fe4ced34c3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 19:55:35.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":5},{"id":"8845d96dab66f4d543ebefb6e60049921225f3e5","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-03 19:58:45.000000000","message":"Patch Set 5:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/08f016fbd5a8405e9af35182548d092c : SUCCESS in 2m 15s","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"56f29b3e4ab0d5bad7ab3de5437e25729528c8e6","revisions":{"23e37478983a610e11c8603772ba0e6225d4a3f2":{"kind":"REWORK","_number":1,"created":"2020-01-13 17:05:59.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/61/702261/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/61/702261/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/61/702261/1"}}},"commit":{"parents":[{"commit":"b9105454876d4f2152db5a0308d8f95e329e8246","subject":"Enable live migration with qos ports","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b9105454876d4f2152db5a0308d8f95e329e8246"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-13 16:37:11.000000000","tz":60},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-13 17:00:05.000000000","tz":60},"subject":"Remove extra instance.save() calls related to qos SRIOV ports","message":"Remove extra instance.save() calls related to qos SRIOV ports\n\nDuring creating or moving of an instance with qos SRIOV port the PCI\ndevice claim on the destination compute needs to be restricted to select\nPCI VFs from the same PF where the bandwidth for the qos port is\nallocated from. This is achieved by updating the spec part of the\nInstancePCIRequest with the device name of the PF by calling\nupdate_pci_request_spec_with_allocated_interface_name(). Until now\nsuch update of the instance object was directly persisted by the call.\n\nDuring code review it was came up that the instance.save() in the util\nis not appropriate as the caller has a lot more context to decide when\nto persist the changes.\n\nThe original eager instance.save was introduced when support added to\nthe server create flow. Now I realized that the need for such save was\ndue to a mistake in the original ResourceTracker.instance_claim() call\nthat loads the InstancePCIRequest from the DB instead of using the\nrequests through the passed in instance object. By removing the extra DB\ncall the need for eagerly persisting the PCI spec update is also\nremoved. It turned out that both the server create code path and every\nserver move code paths eventually persist the instance object either\nduring at the end of the claim process or in case of live migration in\nthe post_live_migration_at_destination compute manager call. This means\nthat the code now can be simplified. Especially the live migration cases.\n\nIn the abort case we needed to roll back the eagerly persisted PCI change\nbut now such change only persisted at the end of the migration so\nthe rollback code can be removed.\n\nAlso in case of rescheduling during live migrate if the rescheduling\nfailed the PCI change needed to be rolled back to the source host by a\nspecific code. But now those change are never persisted until the\nmigration finishes so this rollback code can be removed too.\n\nChange-Id: Ied8f96b4e67f79498519931cb6b35dad5288bbb8\nblueprint: support-move-ops-with-qos-ports-ussuri\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/23e37478983a610e11c8603772ba0e6225d4a3f2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/23e37478983a610e11c8603772ba0e6225d4a3f2"}]},"branch":"refs/heads/master"},"4c1b5d133cd22b912f6e39d2691c813bd167d523":{"kind":"REWORK","_number":2,"created":"2020-01-14 16:51:24.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/61/702261/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/61/702261/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/61/702261/2"}}},"commit":{"parents":[{"commit":"6299157509ce2500cdcd45cdda8ec92766cc4d48","subject":"Func test for failed and aborted live migration","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6299157509ce2500cdcd45cdda8ec92766cc4d48"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-08 08:52:10.000000000","tz":60},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-14 16:45:18.000000000","tz":60},"subject":"Remove extra instance.save() calls related to qos SRIOV ports","message":"Remove extra instance.save() calls related to qos SRIOV ports\n\nDuring creating or moving of an instance with qos SRIOV port the PCI\ndevice claim on the destination compute needs to be restricted to select\nPCI VFs from the same PF where the bandwidth for the qos port is\nallocated from. This is achieved by updating the spec part of the\nInstancePCIRequest with the device name of the PF by calling\nupdate_pci_request_spec_with_allocated_interface_name(). Until now\nsuch update of the instance object was directly persisted by the call.\n\nDuring code review it was came up that the instance.save() in the util\nis not appropriate as the caller has a lot more context to decide when\nto persist the changes.\n\nThe original eager instance.save was introduced when support added to\nthe server create flow. Now I realized that the need for such save was\ndue to a mistake in the original ResourceTracker.instance_claim() call\nthat loads the InstancePCIRequest from the DB instead of using the\nrequests through the passed in instance object. By removing the extra DB\ncall the need for eagerly persisting the PCI spec update is also\nremoved. It turned out that both the server create code path and every\nserver move code paths eventually persist the instance object either\nduring at the end of the claim process or in case of live migration in\nthe post_live_migration_at_destination compute manager call. This means\nthat the code now can be simplified. Especially the live migration cases.\n\nIn the live migrate abort case we don\u0027t need to roll back the eagerly\npersisted PCI change as now such change is only persisted at the end\nof the migration but still we need to refresh the instance object\nduring the rollback as the instance object is stale, containing dest\nhost related PCI information.\n\nAlso in case of rescheduling during live migrate if the rescheduling\nfailed the PCI change needed to be rolled back to the source host by a\nspecific code. But now those change are never persisted until the\nmigration finishes so this rollback code can be removed too.\n\nChange-Id: Ied8f96b4e67f79498519931cb6b35dad5288bbb8\nblueprint: support-move-ops-with-qos-ports-ussuri\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4c1b5d133cd22b912f6e39d2691c813bd167d523"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4c1b5d133cd22b912f6e39d2691c813bd167d523"}]},"branch":"refs/heads/master"},"263ec5a3f32778605eaa862846c908277276cb5f":{"kind":"REWORK","_number":3,"created":"2020-01-20 13:39:04.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/61/702261/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/61/702261/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/61/702261/3"}}},"commit":{"parents":[{"commit":"6299157509ce2500cdcd45cdda8ec92766cc4d48","subject":"Func test for failed and aborted live migration","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6299157509ce2500cdcd45cdda8ec92766cc4d48"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-08 08:52:10.000000000","tz":60},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-20 13:36:13.000000000","tz":60},"subject":"Remove extra instance.save() calls related to qos SRIOV ports","message":"Remove extra instance.save() calls related to qos SRIOV ports\n\nDuring creating or moving of an instance with qos SRIOV port the PCI\ndevice claim on the destination compute needs to be restricted to select\nPCI VFs from the same PF where the bandwidth for the qos port is\nallocated from. This is achieved by updating the spec part of the\nInstancePCIRequest with the device name of the PF by calling\nupdate_pci_request_spec_with_allocated_interface_name(). Until now\nsuch update of the instance object was directly persisted by the call.\n\nDuring code review it was came up that the instance.save() in the util\nis not appropriate as the caller has a lot more context to decide when\nto persist the changes.\n\nThe original eager instance.save was introduced when support added to\nthe server create flow. Now I realized that the need for such save was\ndue to a mistake in the original ResourceTracker.instance_claim() call\nthat loads the InstancePCIRequest from the DB instead of using the\nrequests through the passed in instance object. By removing the extra DB\ncall the need for eagerly persisting the PCI spec update is also\nremoved. It turned out that both the server create code path and every\nserver move code paths eventually persist the instance object either\nduring at the end of the claim process or in case of live migration in\nthe post_live_migration_at_destination compute manager call. This means\nthat the code now can be simplified. Especially the live migration cases.\n\nIn the live migrate abort case we don\u0027t need to roll back the eagerly\npersisted PCI change as now such change is only persisted at the end\nof the migration but still we need to refresh the instance object\nduring the rollback as the instance object is stale, containing dest\nhost related PCI information.\n\nAlso in case of rescheduling during live migrate if the rescheduling\nfailed the PCI change needed to be rolled back to the source host by a\nspecific code. But now those change are never persisted until the\nmigration finishes so this rollback code can be removed too.\n\nChange-Id: Ied8f96b4e67f79498519931cb6b35dad5288bbb8\nblueprint: support-move-ops-with-qos-ports-ussuri\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263ec5a3f32778605eaa862846c908277276cb5f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/263ec5a3f32778605eaa862846c908277276cb5f"}]},"branch":"refs/heads/master"},"20747a7303ae691ee902c696fb3a76b0f355b5d4":{"kind":"REWORK","_number":4,"created":"2020-01-21 13:41:19.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/61/702261/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/61/702261/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/61/702261/4"}}},"commit":{"parents":[{"commit":"4eafc9d5b18445d0ceb7ae738625ae47d68438fa","subject":"Func test for failed and aborted live migration","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4eafc9d5b18445d0ceb7ae738625ae47d68438fa"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-08 08:52:10.000000000","tz":60},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-21 13:33:23.000000000","tz":60},"subject":"Remove extra instance.save() calls related to qos SRIOV ports","message":"Remove extra instance.save() calls related to qos SRIOV ports\n\nDuring creating or moving of an instance with qos SRIOV port the PCI\ndevice claim on the destination compute needs to be restricted to select\nPCI VFs from the same PF where the bandwidth for the qos port is\nallocated from. This is achieved by updating the spec part of the\nInstancePCIRequest with the device name of the PF by calling\nupdate_pci_request_spec_with_allocated_interface_name(). Until now\nsuch update of the instance object was directly persisted by the call.\n\nDuring code review it was came up that the instance.save() in the util\nis not appropriate as the caller has a lot more context to decide when\nto persist the changes.\n\nThe original eager instance.save was introduced when support added to\nthe server create flow. Now I realized that the need for such save was\ndue to a mistake in the original ResourceTracker.instance_claim() call\nthat loads the InstancePCIRequest from the DB instead of using the\nrequests through the passed in instance object. By removing the extra DB\ncall the need for eagerly persisting the PCI spec update is also\nremoved. It turned out that both the server create code path and every\nserver move code paths eventually persist the instance object either\nduring at the end of the claim process or in case of live migration in\nthe post_live_migration_at_destination compute manager call. This means\nthat the code now can be simplified. Especially the live migration cases.\n\nIn the live migrate abort case we don\u0027t need to roll back the eagerly\npersisted PCI change as now such change is only persisted at the end\nof the migration but still we need to refresh the instance object\nduring the rollback as the instance object is stale, containing dest\nhost related PCI information.\n\nAlso in case of rescheduling during live migrate if the rescheduling\nfailed the PCI change needed to be rolled back to the source host by a\nspecific code. But now those change are never persisted until the\nmigration finishes so this rollback code can be removed too.\n\nChange-Id: Ied8f96b4e67f79498519931cb6b35dad5288bbb8\nblueprint: support-move-ops-with-qos-ports-ussuri\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/20747a7303ae691ee902c696fb3a76b0f355b5d4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/20747a7303ae691ee902c696fb3a76b0f355b5d4"}]},"branch":"refs/heads/master"},"56f29b3e4ab0d5bad7ab3de5437e25729528c8e6":{"kind":"REWORK","_number":5,"created":"2020-02-03 10:49:15.000000000","uploader":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"ref":"refs/changes/61/702261/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/61/702261/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/61/702261/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/61/702261/5"}}},"commit":{"parents":[{"commit":"4eafc9d5b18445d0ceb7ae738625ae47d68438fa","subject":"Func test for failed and aborted live migration","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4eafc9d5b18445d0ceb7ae738625ae47d68438fa"}]}],"author":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-01-08 08:52:10.000000000","tz":60},"committer":{"name":"Balazs Gibizer","email":"balazs.gibizer@est.tech","date":"2020-02-03 10:41:38.000000000","tz":60},"subject":"Remove extra instance.save() calls related to qos SRIOV ports","message":"Remove extra instance.save() calls related to qos SRIOV ports\n\nDuring creating or moving of an instance with qos SRIOV port the PCI\ndevice claim on the destination compute needs to be restricted to select\nPCI VFs from the same PF where the bandwidth for the qos port is\nallocated from. This is achieved by updating the spec part of the\nInstancePCIRequest with the device name of the PF by calling\nupdate_pci_request_spec_with_allocated_interface_name(). Until now\nsuch update of the instance object was directly persisted by the call.\n\nDuring code review it was came up that the instance.save() in the util\nis not appropriate as the caller has a lot more context to decide when\nto persist the changes.\n\nThe original eager instance.save was introduced when support added to\nthe server create flow. Now I realized that the need for such save was\ndue to a mistake in the original ResourceTracker.instance_claim() call\nthat loads the InstancePCIRequest from the DB instead of using the\nrequests through the passed in instance object. By removing the extra DB\ncall the need for eagerly persisting the PCI spec update is also\nremoved. It turned out that both the server create code path and every\nserver move code paths eventually persist the instance object either\nduring at the end of the claim process or in case of live migration in\nthe post_live_migration_at_destination compute manager call. This means\nthat the code now can be simplified. Especially the live migration cases.\n\nIn the live migrate abort case we don\u0027t need to roll back the eagerly\npersisted PCI change as now such change is only persisted at the end\nof the migration but still we need to refresh pci_requests field of\nthe instance object during the rollback as that field might be stale,\ncontaining dest host related PCI information.\n\nAlso in case of rescheduling during live migrate if the rescheduling\nfailed the PCI change needed to be rolled back to the source host by a\nspecific code. But now those change are never persisted until the\nmigration finishes so this rollback code can be removed too.\n\nChange-Id: Ied8f96b4e67f79498519931cb6b35dad5288bbb8\nblueprint: support-move-ops-with-qos-ports-ussuri\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/56f29b3e4ab0d5bad7ab3de5437e25729528c8e6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/56f29b3e4ab0d5bad7ab3de5437e25729528c8e6"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}