)]}'
{"id":"openstack%2Fnova~783084","triplet_id":"openstack%2Fnova~master~Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d","project":"openstack/nova","branch":"master","hashtags":[],"change_id":"Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d","subject":"Test SRIOV port move operations with PCI conflicts","status":"MERGED","created":"2021-03-25 19:23:34.000000000","updated":"2021-05-01 04:11:18.000000000","submitted":"2021-05-01 04:07:30.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":36,"unresolved_comment_count":9,"has_review_started":true,"submission_id":"783084","meta_rev_id":"7cf50e8b028dd7cb1c3f90d40e81ba6a796ba8e3","_number":783084,"virtual_id_number":783084,"owner":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-05-01 04:07:12.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}],"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":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":2,"date":"2021-04-27 16:51:28.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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-04-30 15:13:53.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}],"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"all":[{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-04-30 15:13:53.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-25 19:26:56.000000000","updated_by":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2021-03-25 19:48:13.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-03-27 15:49:16.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-04-15 10:44:32.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":"2021-04-23 16:48:22.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"CC"},{"updated":"2021-04-30 15:13:53.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"REVIEWER"}],"messages":[{"id":"bfaa915db2796e148b7a4dd08e28f2cd506c76a9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-25 19:23:34.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"961ae55c2f1bae76c7f70dbdd1d23d068bd8e2a5","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-25 19:26:10.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"895a1cbe8677f6b69f363ae627b978a6d9a22f74","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-25 19:27:59.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"fc82c99deb1c2ca3ae08abf896c272cab7aec776","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-25 19:28:57.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"9b35479114a557af52e3f4a81243566ec2533f14","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 19:48:13.000000000","message":"Patch Set 1: Verified-1\n\n(1 comment)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/c8e05fe2a15a4c59a1281039c080e137 : SUCCESS in 23m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4d63b3296a8746f0a228c32a38c17581 : SUCCESS in 19m 59s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5543563732a04e54bc7e071a15ecdc12 : FAILURE in 9m 13s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0acb7af67d924930821ec96768350dab : SUCCESS in 19m 07s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/1f288567f0e44151818fc8261e4f0c02 : SUCCESS in 13m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/e0f790c819ad499eb80aaebb6ea140f7 : SUCCESS in 19m 53s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3ee305670b814191a555ba5c64e621c1 : SUCCESS in 8m 11s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/57aaae1a20774d828b513ed72d3141cd : SUCCESS in 16m 21s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/94d17d99b7a54907b96edf9801b472da : SUCCESS in 16m 04s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"8b0a275daa6bfa2afed368dc3922e3f82f3fd7c6","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-25 19:50:18.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/702ab1fbf4174bcda7e9540c8780796e : SUCCESS in 18m 58s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/e5d4bfbaec4e43bb8fb4c3e22f815b2c : SUCCESS in 22m 13s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"870219cb15ce22af379c3ae9673e2e96af093090","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-25 19:59:45.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\n-1 since this does not actually replicate the failure.\n\nyou need to assert that today the migration will succeed but that the pci_slot is not update in the port bidning profile and that the value does not match the vaule claimed in the db.\n\nto do that you will need create a compute host wtih 2 VFs.\nboot a vm and shelve it then boot another vms and assert it gets teh saem pci device as the shelve one then unshleve the vm.\n\nin reality this woudl fail with the libvirt error in the bug in the test fixture i expecct it will succeed but if you check the profil you will see that both vms have the same pci device.\n\nyou could also add a spy to the xml generation and you could assert that the unshelved vm is infact using the same deivce in its xml but that is not strectly required once you assert that the data in the port used in each vm refers to the same pci adddress.\n\nin addtion to that you can assert that the seconned vm has actully claimed that vf in pci_device table and that the adress of the device claimed for the first vm does not match the value in the port","accounts_in_message":[],"_revision_number":1},{"id":"13f77adf8dd030d61bf7391fad713195792ce358","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-26 16:20:21.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"76069a00feed73a96425878d526a7622f582ba3f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-26 16:45:30.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"32e0045b4e3855698a6001279d7d8b093a09ee9c","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-26 17:20:36.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/347445e2cf324b47ba0930b8d6cc543b : SUCCESS in 22m 33s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/9a051eacea3540858b48e8596c61fec7 : SUCCESS in 30m 03s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"9353fedfcdf4c46f4c7475196ca4b8ac0073e441","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-26 17:22:59.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/42fb07cfd659466babc2b1ca03926c2a : SUCCESS in 36m 05s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ac9d8daa62054ed4953a79b6504d3378 : SUCCESS in 17m 41s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f6ad5c79de224ac59a0d82cec1d6270a : SUCCESS in 7m 48s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/9494758fc8cf41ed95cfeea76b17d42b : SUCCESS in 14m 51s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/74673fda8fec47928ddceac7822262a2 : SUCCESS in 30m 45s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/2a000be3db2d4088a413b708c29ae97b : SUCCESS in 32m 49s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6cad42a511ec4a9d87da217b0a98ecbe : SUCCESS in 10m 21s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/7c67bb359fab4fb1925303c19c6e26b1 : FAILURE in 20m 18s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3a66956b57504bd0ae42f1eff6b9c2d0 : FAILURE in 28m 58s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"a497f2983b8784e2d2b5bbfbe099ad038cfcc9c9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-03-27 15:26:57.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"788134bf332797df64f1f756b43803922f5d7183","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-27 15:48:37.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/67387a4cd4064499a59a76828d77f3bc : SUCCESS in 18m 03s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4bf316368de74f94bad0331a8b70ab0c : SUCCESS in 14m 25s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7e08c8cd20864f8a8b6bba0c1792c603 : SUCCESS in 8m 25s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/fe657b1160ec44dda7ee9fba115ceffd : SUCCESS in 17m 39s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c50a5def59ac4e6d9dc2a3b9ff976d29 : SUCCESS in 10m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/32b00d21d43b4d9fadac0515a3904feb : SUCCESS in 12m 33s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2a936fb97a684cf0aea069a4fc65eb03 : SUCCESS in 10m 56s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/d41246361d0b40188fdd946f14c7cbf7 : SUCCESS in 19m 44s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/846583e177964e7dab31cac0931adb29 : SUCCESS in 17m 44s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"c34c389d1d43f2136a40ca698a95143fb821c5bf","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-03-27 15:49:16.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_PowerFlex_NOVA http://publiclogs.emc.com/84/783084/4/check/EMC_PowerFlex_NOVA/1c21e53/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":4},{"id":"5cdbed7500f7743fa3e6f1bd3b8fbea0557e4b01","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-27 15:56:38.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/973ff9214e484fd1b5980a1429a669e0 : SUCCESS in 19m 46s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/5efc010f446743b3882fd0b15c95f3b4 : SUCCESS in 22m 55s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"256ebe144f51edbc2db41d1f38410f3962eca051","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-03-27 15:57:14.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_PowerFlex_NOVA http://publiclogs.emc.com/84/783084/4/check/EMC_PowerFlex_NOVA/5f58e6a/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":4},{"id":"ac0df203cd75e47030879a39d6108dc0e6560241","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-03-31 23:57:30.000000000","message":"Patch Set 4: Code-Review-1\n\n(7 comments)\n\nthis is a pretty soft -1 mainly i would like you to use uniqe mac and ip adress in the fixture\neverythin else is just minor nits.\n\nthe test logic looks good to me.\ni was debating about resize but the code path we care about is shared between cold migrate and resize so i dont think that would real add value and it would just make the tests longer.","accounts_in_message":[],"_revision_number":4},{"id":"f3bccaba66c830c0b41e49485509d8213a81b93c","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-04-01 16:23:31.000000000","message":"Patch Set 4:\n\n(6 comments)","accounts_in_message":[],"_revision_number":4},{"id":"5e0dd08155510e155d2109ce134ffc82b4bd2944","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-04-01 20:27:49.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"e2054ac3ef36c6bd4efa3ac7bf6e468f4cea89b0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-01 20:54:22.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/95fed3d29c884edd9285e650f3529b66 : SUCCESS in 17m 44s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/20a0688f840140a5a24d49adb6415344 : SUCCESS in 14m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/be583b5243f040b2a6fae25dbd0f773a : SUCCESS in 9m 52s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/39b30f6291f94da9a1ebd0dffa60de7e : SUCCESS in 12m 30s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/f2378f54f5e1400f90840eacd336cc13 : SUCCESS in 14m 22s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/12251a0398ed49c8892de2c34e006218 : SUCCESS in 13m 23s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/66e295666a5441bdb2fe78fbc14205c5 : SUCCESS in 12m 41s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/540c5dd8ea1b458b9dd260fac17f86b4 : SUCCESS in 21m 10s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/e197623f8acb420b8b9f6d1c7f343758 : SUCCESS in 21m 48s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"4e7984e7fa75357f8b6101047488c0ced38e00f4","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-04-01 20:54:54.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_PowerFlex_NOVA http://publiclogs.emc.com/84/783084/5/check/EMC_PowerFlex_NOVA/3b18759/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":5},{"id":"e21bdc56583d1363b43679fa42cc5d042e023d3f","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-01 20:59:12.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/70d9a7877b964b488c40fc2c54f3813b : SUCCESS in 20m 04s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/14cdfd0881254f958dc87f263d45b3ab : SUCCESS in 21m 15s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"c62f2d2ac1d3b56bbbbac0fcf13d5f4a581ed2f1","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-04-01 21:00:00.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_PowerFlex_NOVA http://publiclogs.emc.com/84/783084/5/check/EMC_PowerFlex_NOVA/4e1a036/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":5},{"id":"2a2926c0c7e1cf5c71bbe729b1bdbb6496e80576","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-04-15 10:44:32.000000000","message":"Patch Set 5: Code-Review-1\n\n(8 comments)\n\nMaybe doesn\u0027t justify the -1, but I\u0027d like to see the names of these tests updated to reflect our desired end state rather than the bug which should hopefully be fixed soon. I did try to think of a way that we could start enforcing this in the fake PCI manager instead, which would cause a whole load of _existing_ tests to start failing rather than necessarily having to add more new ones, but it looks like that requires a lot more mocking. (Later: It also assumes we have coverage for these operations atm, which we don\u0027t appear to actually)","accounts_in_message":[],"_revision_number":5},{"id":"a319b45c39692998c6b2eb5a93bf6edc16ef1b14","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-04-15 15:21:43.000000000","message":"Patch Set 5:\n\n(9 comments)","accounts_in_message":[],"_revision_number":5},{"id":"a79363ae642469b855793f8077dff8be96bd0710","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-04-16 11:27:16.000000000","message":"Patch Set 5:\n\nDid you forget to push the changes?","accounts_in_message":[],"_revision_number":5},{"id":"7d3cd07d9476fc5344d43928e4f894304c95798f","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-04-16 11:38:10.000000000","message":"Patch Set 5:\n\n\u003e Patch Set 5:\n\u003e \n\u003e Did you forget to push the changes?\n\nNo, I\u0027m still working on the online data migration you asked for in the patch on top. I can push the addressed feedback without the data migration?","accounts_in_message":[],"_revision_number":5},{"id":"6863689f16e53aaed50807bcab070504c7913b8b","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-04-16 11:40:45.000000000","message":"Patch Set 5:\n\n\u003e Patch Set 5:\n\u003e \n\u003e \u003e Patch Set 5:\n\u003e \u003e \n\u003e \u003e Did you forget to push the changes?\n\u003e \n\u003e No, I\u0027m still working on the online data migration you asked for in the patch on top. I can push the addressed feedback without the data migration?\n\nNope, I can wait. I just saw \"done\" comments and expected to see a revision shortly thereafter 😄","accounts_in_message":[],"_revision_number":5},{"id":"5bcc245fd98e1fe2a0fb0e13917e3c93674af698","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-04-23 14:07:42.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"aeec0ff87ef02355360abfdcfa13af5b436cf285","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-23 14:44:30.000000000","message":"Patch Set 6:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/d07996df506f48c794f06a2ba0e3a599 : SUCCESS in 21m 37s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/e4bd4064997140f0a9fba78e5a305fe9 : SUCCESS in 33m 13s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"af4a1473d9b0435844c4e924a5c4eaf36fe1c4b5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-23 16:32:01.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/dd7d15dc5d5f4570933569824383dabf : SUCCESS in 17m 01s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/00b89eb75b4b4278a4200385f4c0b28e : SUCCESS in 18m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c69764fbe3a34afd9caea9d6e66391fc : SUCCESS in 10m 29s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bc20e75b10234d27a118eb41824d831b : SUCCESS in 12m 42s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/0581e2b6967b4253a666952347be182d : SUCCESS in 14m 35s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/ecb81a1e453046b8bf64bb2f4b0d7f57 : SUCCESS in 15m 22s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c96199a49ada4653bfc69d56c6620836 : SUCCESS in 10m 01s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/9db264c1c0c24f3aa20fd269ab88f5b9 : SUCCESS in 18m 22s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/66b91af911cd4b939c4f2bf210a0c00c : SUCCESS in 18m 20s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"1ebaae85e60b6c2a8bd78f13279e6ab8be75fe6d","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-04-23 16:48:22.000000000","message":"Patch Set 6:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/783084/6/iscsi : SUCCESS in 2h 38m 58s","accounts_in_message":[],"_revision_number":6},{"id":"8702b60b8b31a5e9ab7deffc30c1fd0d18de4585","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-04-27 16:51:28.000000000","message":"Patch Set 6: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"791a2ee96e29fdf64abd8b4186a85cb7c2ad44a9","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-04-30 15:13:53.000000000","message":"Patch Set 6: Code-Review+2 Workflow+1\n\n(1 comment)\n\nNice tests!","accounts_in_message":[],"_revision_number":6},{"id":"5b6a7c0a5d71ba3e0bd35286db628c956a556a8b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-30 15:14:49.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"ba98b676d9210c35ccfd4d5cd5551220161fa9e6","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-30 20:17:18.000000000","message":"Patch Set 6:\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"b704655525e6120bf708611bb28ebdb61b3fb384","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-01 04:07:12.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4b077795d1ad43398bdc91eeb7fe53f9 : SUCCESS in 13m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9d95cded17784f158499bd472090dc51 : SUCCESS in 8m 18s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f4daf138fbb3438699a6fc95e7035c15 : SUCCESS in 13m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/9c157a839675441f9bfd696feccbd68e : SUCCESS in 12m 56s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2775b3fb035147e2a0a0dc633cd7f3d8 : SUCCESS in 10m 22s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/63193c96d382413483df7d3e4cb845d3 : SUCCESS in 18m 27s","accounts_in_message":[],"_revision_number":6},{"id":"43d19397dbddb026c6297f6b2efb11ea3ca0cea1","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-01 04:07:30.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":6},{"id":"7cf50e8b028dd7cb1c3f90d40e81ba6a796ba8e3","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-01 04:11:18.000000000","message":"Patch Set 6:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5e34c7f713b149c4bfeccb3986e41b90 : SUCCESS in 1m 06s","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"bea06123dbba82851fd17a41cc93ab4a519f8bfe","revisions":{"326f1b771d1b1a512f6bebdbaf9350d670946a25":{"kind":"REWORK","_number":1,"created":"2021-03-25 19:23:34.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/1"}}},"commit":{"parents":[{"commit":"572ae578bb228cd2dd62608c2e601af49f95fbbb","subject":"Merge \"Bump the Compute RPC API to version 6.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/572ae578bb228cd2dd62608c2e601af49f95fbbb"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"subject":"Regression test for unshelve with SRIOV port","message":"Regression test for unshelve with SRIOV port\n\nThis patch adds a functional test for unshelve with SRIOV. While bug\n1851545 is related, the error is not the same, indicating the need for\nmore investigation.\n\nRelated bug: 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/326f1b771d1b1a512f6bebdbaf9350d670946a25"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/326f1b771d1b1a512f6bebdbaf9350d670946a25"}]},"branch":"refs/heads/master"},"2dc0c23ecc185ea2fd977575a02b3e139d1e3ae5":{"kind":"REWORK","_number":2,"created":"2021-03-26 16:20:21.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/2"}}},"commit":{"parents":[{"commit":"572ae578bb228cd2dd62608c2e601af49f95fbbb","subject":"Merge \"Bump the Compute RPC API to version 6.0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/572ae578bb228cd2dd62608c2e601af49f95fbbb"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-26 16:18:45.000000000","tz":-240},"subject":"WIP: Test SRIOV port move operations with PCI conflicts","message":"WIP: Test SRIOV port move operations with PCI conflicts\n\nThis patch tests cold migration and unshelve in a situation where the\nexisting port binding\u0027s pci_slot would cause a conflict on the\ndestination compute node.\n\nRelated bug: 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2dc0c23ecc185ea2fd977575a02b3e139d1e3ae5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2dc0c23ecc185ea2fd977575a02b3e139d1e3ae5"}]},"branch":"refs/heads/master"},"941e0aaa448c6f9198edb6a0384ab5c6756389f1":{"kind":"REWORK","_number":3,"created":"2021-03-26 16:45:30.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/3"}}},"commit":{"parents":[{"commit":"5dcb86bbf4d338b88ae3ee1f35b749d605b4d25e","subject":"WIP: Neutron fixture: don\u0027t clobber profile and vif_details if empty","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5dcb86bbf4d338b88ae3ee1f35b749d605b4d25e"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-26 16:45:22.000000000","tz":-240},"subject":"WIP: Test SRIOV port move operations with PCI conflicts","message":"WIP: Test SRIOV port move operations with PCI conflicts\n\nThis patch tests cold migration and unshelve in a situation where the\nexisting port binding\u0027s pci_slot would cause a conflict on the\ndestination compute node.\n\nRelated bug: 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/941e0aaa448c6f9198edb6a0384ab5c6756389f1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/941e0aaa448c6f9198edb6a0384ab5c6756389f1"}]},"branch":"refs/heads/master"},"638c2e72c3ddb7208dc97c7acfe020ba1adc507e":{"kind":"REWORK","_number":4,"created":"2021-03-27 15:26:57.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/4"}}},"commit":{"parents":[{"commit":"3f088d7bb8055b6ec511cacfc583af73ba8dfda3","subject":"Neutron fixture: don\u0027t clobber profile and vif_details if empty","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/3f088d7bb8055b6ec511cacfc583af73ba8dfda3"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-27 15:25:27.000000000","tz":-240},"subject":"Test SRIOV port move operations with PCI conflicts","message":"Test SRIOV port move operations with PCI conflicts\n\nThis patch tests cold migration, unshelve and evacuate in a situation\nwhere the existing port binding\u0027s pci_slot would cause a conflict on\nthe destination compute node. While cold migration and evcacuation\nwork correctly, in the unshelve case the pci_slot is not updated,\nresulting in two instances attempting to consume the same PCI device.\nThis \"passed\" in the functional tests, but with a real libvirt this\nwould obviously explode.\n\nRelated bug: 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/638c2e72c3ddb7208dc97c7acfe020ba1adc507e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/638c2e72c3ddb7208dc97c7acfe020ba1adc507e"}]},"branch":"refs/heads/master"},"7be925fa574267901d2f95e31d791157908ef4f9":{"kind":"REWORK","_number":5,"created":"2021-04-01 20:27:49.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/5"}}},"commit":{"parents":[{"commit":"62868aaac70d908ade6ff80c8ced54afa0435a76","subject":"Neutron fixture: don\u0027t clobber profile and vif_details if empty","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/62868aaac70d908ade6ff80c8ced54afa0435a76"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-04-01 16:22:58.000000000","tz":-240},"subject":"Test SRIOV port move operations with PCI conflicts","message":"Test SRIOV port move operations with PCI conflicts\n\nThis patch tests cold migration, unshelve and evacuate in a situation\nwhere the existing port binding\u0027s pci_slot would cause a conflict on\nthe destination compute node. While cold migration and evcacuation\nwork correctly, in the unshelve case the pci_slot is not updated,\nresulting in two instances attempting to consume the same PCI device.\nThis \"passed\" in the functional tests, but with a real libvirt this\nwould obviously explode.\n\nRelated bug: 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7be925fa574267901d2f95e31d791157908ef4f9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7be925fa574267901d2f95e31d791157908ef4f9"}]},"branch":"refs/heads/master"},"bea06123dbba82851fd17a41cc93ab4a519f8bfe":{"kind":"REWORK","_number":6,"created":"2021-04-23 14:07:42.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/84/783084/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/84/783084/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/84/783084/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/84/783084/6"}}},"commit":{"parents":[{"commit":"5eab13030bc2708c8900f7ac1bdbc8a111f5f823","subject":"Merge \"api: Reject requests to force up computes when `done` evacuation records exist\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/5eab13030bc2708c8900f7ac1bdbc8a111f5f823"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-03-25 19:21:59.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-04-15 15:23:38.000000000","tz":-240},"subject":"Test SRIOV port move operations with PCI conflicts","message":"Test SRIOV port move operations with PCI conflicts\n\nThis patch tests cold migration, unshelve and evacuate in a situation\nwhere the existing port binding\u0027s pci_slot would cause a conflict on\nthe destination compute node. While cold migration and evacuation\nwork correctly, in the unshelve case the pci_slot is not updated,\nresulting in two instances attempting to consume the same PCI device.\nThis \"passed\" in the functional tests, but with a real libvirt this\nwould obviously explode.\n\nRelated: bug 1851545\n\nChange-Id: Ib81532dc1e6dd85822e38eb1785ffb7162d2a84d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bea06123dbba82851fd17a41cc93ab4a519f8bfe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bea06123dbba82851fd17a41cc93ab4a519f8bfe"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
