)]}'
{"id":"openstack%2Fnova~820381","triplet_id":"openstack%2Fnova~master~Idb55c81639139efbd44b9f8c938276827898ae44","project":"openstack/nova","branch":"master","topic":"bug/1952915","attention_set":{},"removed_from_attention_set":{"4690":{"account":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"last_update":"2022-02-07 18:45:25.000000000","reason":"Change was abandoned"},"33608":{"account":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"last_update":"2021-12-07 13:02:42.000000000","reason":"gtrevisan replied on the change"},"9708":{"account":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"last_update":"2021-12-13 10:50:34.000000000","reason":"Removed by gibi by clicking the attention icon"},"7166":{"account":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"last_update":"2022-02-07 18:45:25.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Idb55c81639139efbd44b9f8c938276827898ae44","subject":"[WIP] Set resize claim resource when changing host","status":"ABANDONED","created":"2021-12-03 13:28:52.000000000","updated":"2022-02-07 18:45:25.000000000","total_comment_count":11,"unresolved_comment_count":4,"work_in_progress":true,"has_review_started":true,"meta_rev_id":"24daad88ad003365d7f9988acdc9a8aa805c8402","_number":820381,"virtual_id_number":820381,"owner":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-12-06 23:26:49.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-12-07 10:35:37.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_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":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":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{"CC":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_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":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":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"reviewer_updates":[{"updated":"2021-12-03 13:46:35.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-12-03 14:09:48.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-12-03 15:38:00.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-12-03 16:03: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":"CC"},{"updated":"2021-12-06 07:44:08.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-12-06 20:39:37.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"CC"},{"updated":"2021-12-06 23:27:17.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-12-07 08:24:12.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"CC"},{"updated":"2021-12-07 10:35:37.000000000","updated_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"reviewer":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"state":"REVIEWER"}],"messages":[{"id":"4dcbb3c84ba15b481e63a0dc0f8cac72bebaa3cb","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-03 13:28:52.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e17e392149e58451c8988abad3482d19ae7344d1","tag":"autogenerated:gerrit:setWorkInProgress","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-03 13:31:08.000000000","message":"Set Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"a0d5bd79c05912c7d47735fef15f253251be1d3c","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-12-03 13:46:35.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/820381/1/iscsi : FAILURE in 16m 38s","accounts_in_message":[],"_revision_number":1},{"id":"4fbb3def70e52f64c1d1270dcab091ab8d845772","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-03 14:09:48.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/dae6d1a9177a4263a3d6ccbc84ca2763 : SUCCESS in 16m 48s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/795a441ddd994e2bb696007f01a03460 : SUCCESS in 37m 47s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"46515c50db9fc61a208dc4abaa2f0c4615f48a02","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-03 15:38:00.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/c8b05ba5e9b7482aa2d29f7886696e17 : SUCCESS in 1h 11m 20s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/68cf7b81ff6e49a4970a205302318ca9 : SUCCESS in 1h 34m 05s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/67d31db6982a44eabfde539143e12433 : SUCCESS in 45m 46s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/240380a6673e4c079c38f94f3447a162 : SUCCESS in 16m 09s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/05670894b7d64cb7b3f91986927e5aab : SUCCESS in 12m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fbe3c34a0d3d42369f214ca864fc14f7 : FAILURE in 10m 24s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/f6a944c285e1464c8c250d32b628c67c : SUCCESS in 13m 06s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/7cfd31b0c2964e949792f75339f4da98 : SUCCESS in 12m 40s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7772b36eb15e4c159be6b6d26a87d0a9 : SUCCESS in 10m 29s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/ec91cc0224e84b3f8fc81ae79df8c1fd : SUCCESS in 1h 13m 31s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/c04dc6995d574cc6835f6960b82ceef4 : SUCCESS in 1h 38m 31s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/4a68a45919d1458ba54b1b6dad6d6fef : SUCCESS in 50m 08s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/c51c9dc114044776a52782a2a928980d : SUCCESS in 53m 46s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/2b2d484c574a49618dacadae7e26d5b5 : SUCCESS in 1h 27m 06s\n- nova-next https://zuul.opendev.org/t/openstack/build/2ce6c4f37bb842fe88fd706af8c4ee81 : SUCCESS in 1h 58m 24s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/059a1820b24a42e0945b9d06618538e3 : SUCCESS in 6m 32s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/b2b523ac6f0c40c48516c69e7f38205b : SUCCESS in 20m 16s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/303bbac2e29a44628ce4351b6921036d : SUCCESS in 20m 56s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/41150e975a9a4a28ae7ae4d9b86a6c4c : SUCCESS in 1h 12m 13s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/74c6452bfe63433995a0eedded1ffa3e : SUCCESS in 50m 57s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/f0ec83a86b9e404ba0614369ad25ae90 : SUCCESS in 36m 19s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/7e72b48ef3ca48859916a37941de7d31 : SUCCESS in 39m 11s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"d654663339085bf966883fb29cac6d8f9122e3a9","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-12-03 16:03:00.000000000","message":"Patch Set 1:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/81/820381/1/check/tempest-dsvm-full-focal-py3/aa94547/ : SUCCESS in 2h 22m 00s","accounts_in_message":[],"_revision_number":1},{"id":"bb41a3a1f25a092387daf8fdffdf8c9c11e5ef17","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-03 17:29:53.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"56b30f0b1bde8caaa1c5834009bed22c7d38b8c0","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-12-03 17:46:40.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/820381/2/iscsi : FAILURE in 15m 41s","accounts_in_message":[],"_revision_number":2},{"id":"e3f6db06b77af6fa7868de5f3427ee583623aa24","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-03 18:02:10.000000000","message":"Patch Set 2:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/eddbe8f73fcf494c997522619e46d109 : SUCCESS in 20m 56s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/e168f4baa95d403f943e6f919780e89f : SUCCESS in 28m 46s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"108705213492b460b47686a9de965031f66dfa65","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-03 19:20:05.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/339d63fddb6e45359fe35925e2dea80f : SUCCESS in 1h 40m 34s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/3c792addfab543ba91d45f00ec6e4dc6 : SUCCESS in 1h 35m 04s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/54ee52899c7444d8b5de5352e2c5445d : SUCCESS in 1h 02m 04s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/40ba6ffb28494a6e992f114efc89ab83 : SUCCESS in 15m 30s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2bf6f1d3e03c43f291aa000a7dcd1f10 : SUCCESS in 13m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7ec38db1594641698772d54a6cec9b1d : FAILURE in 10m 38s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/721803f87c4c4ceb903ded7f8b896580 : SUCCESS in 14m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/21d73cd7488d4f32a67c7c7d47649eb1 : SUCCESS in 14m 36s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1915b5302a9a4ff6830f3e47181c05cc : SUCCESS in 9m 11s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/0166796c70d54efe9e1263de654147b5 : SUCCESS in 1h 31m 46s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/900a3f29252148baa196b58223baded8 : SUCCESS in 1h 15m 23s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/0f83a8fa84d2462190d17322bf09cd69 : SUCCESS in 49m 57s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/1f279476a7fe465fa24170e593ef06d7 : SUCCESS in 53m 09s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/bf7d0a17c93d4043b7c3b94711b6b1b8 : SUCCESS in 1h 32m 49s\n- nova-next https://zuul.opendev.org/t/openstack/build/aacda1f9d15a4ee08873aba517d832e4 : SUCCESS in 1h 18m 00s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/cadf600545b54852ba585171233c7789 : SUCCESS in 5m 35s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/cec67ae89af54ed0ad054b31e909632d : SUCCESS in 19m 24s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/d658f31ef73445bebb5cc29aa63d306b : SUCCESS in 19m 41s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/2a748d50857f4fa486f1758ebeb01a82 : SUCCESS in 1h 05m 06s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/0d0e9e74dee94a478f7cd33a42ed14b5 : SUCCESS in 53m 38s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/5e77cce65faf440394a9e8d9b9ab2f49 : SUCCESS in 29m 25s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/960d915641124026842416f0bb6ddf7a : SUCCESS in 38m 51s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"cf41fc7f8f1dd91b6d7af089a5093f970d7c4f9b","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-12-03 20:25:39.000000000","message":"Patch Set 2:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/81/820381/2/check/tempest-dsvm-full-focal-py3/c765a42/ : SUCCESS in 2h 19m 08s","accounts_in_message":[],"_revision_number":2},{"id":"a3229eda9381fa232364544def4664244734638f","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-12-06 07:44:08.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/820381/820381/2/check-not-vote/ext-nova-zuul/7190451 : FAILURE in 0s","accounts_in_message":[],"_revision_number":2},{"id":"6845e5ca95c293acdf9c6767bd2d8d482310e519","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-06 18:59:53.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"2a43516d0b361647cae75c54e473f3fc41177215","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-06 19:28:47.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/1776a3cf8a9242f4877b989da8f37e31 : SUCCESS in 17m 30s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/539d11f308f04d9cbe4209449cf8dd18 : SUCCESS in 24m 27s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"4bb4a19a03a91ee84d6c2a2224c5579347897bd5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-06 20:39:32.000000000","message":"Patch Set 3: Verified-1\n\n(5 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/ca300a3c610f41089167d32e7c51436d : SUCCESS in 1h 19m 32s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/2a3f8fb1c0ef476185b64bb90981c54b : SUCCESS in 1h 28m 29s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/71a1b668c26647c3a3d056bee048a511 : SUCCESS in 52m 03s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/218c65a465d14e2cae93616307b59d8a : SUCCESS in 17m 02s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/caf95af8dc7d4ce08f54f596ce80090b : SUCCESS in 14m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b71ea36fcc894fc48d0cbee713b94cde : FAILURE in 13m 05s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/e3a2a19cbba44273bf0e5c19b5dc2d84 : SUCCESS in 12m 44s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/453a8767635143f1a708839c44c3b2a1 : SUCCESS in 14m 08s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3122f1925684441897b119f52d2f8b39 : SUCCESS in 9m 49s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/d6a0422af7fc438db414a95044d053d0 : SUCCESS in 1h 26m 37s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/e6bbdc320564494a8953f10cfbccea34 : SUCCESS in 1h 29m 12s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/d207ed8b437c46539f9eb046dd1713ec : SUCCESS in 55m 03s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/ab393f96cc5f4266afaa93feeb8c5e1d : SUCCESS in 1h 00m 26s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/116301aab38d4638ad653ef982685d45 : SUCCESS in 1h 25m 53s\n- nova-next https://zuul.opendev.org/t/openstack/build/e6896d8b6e4b4ed7acfef3db63bd04c9 : SUCCESS in 1h 31m 18s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/e4eb6da9d5034c6fa1c21f15b482b90e : SUCCESS in 5m 52s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/77658a81464c4b39a7644acecd32c0e9 : SUCCESS in 22m 38s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3c69685f6ac44a819542896f2003dc70 : SUCCESS in 22m 26s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/f75dcd77de9c4bdab9ef8679597b0fd0 : SUCCESS in 1h 29m 22s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/7b745dcafcb344ca9c6fa2b541662dec : SUCCESS in 57m 41s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/151702b69b694340af692fd97012c469 : SUCCESS in 27m 48s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/925f2b501e8b4c26882ab622176db6b9 : SUCCESS in 57m 35s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"a188c2d64e71444564ca4bbc650fcceb8f1d487f","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-12-06 20:39:37.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"ea8b06b8ea8ab7c368862cb7b41b333431f43a93","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-06 21:08:59.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"e7f0bf11ccf5a0ce1ab7b7a9449584099f8898ff","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-06 21:38:50.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"973f0e0b96399ec577d3f4c1f3e9ea3b7b4d4f28","tag":"autogenerated:gerrit:setPsDescription","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-06 21:40:24.000000000","message":"Description of patch set 4 set to \"Fix test code formatting\"","accounts_in_message":[],"_revision_number":4},{"id":"79934338405908e36074f0a8eb514d5f2308282b","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-06 22:24:34.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/9e5f16564b314b76a0474ddc919555a7 : SUCCESS in 18m 31s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/0a73541b775a4b27992629e26a39e6e1 : SUCCESS in 30m 47s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"ec0f07ce6557fe617822798fa6a0ca757511395e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-12-06 23:26:49.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/95bfbf69933349efb6efd9bce142081b : SUCCESS in 1h 18m 31s\n- tempest-integrated-compute-centos-8-stream https://zuul.opendev.org/t/openstack/build/473c173c5a13483693c17bc1e9f2b60d : SUCCESS in 1h 21m 55s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/e7a39e9529d446eba21497018010d995 : SUCCESS in 43m 45s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/492b505d07e746889a525771ba36a819 : SUCCESS in 17m 43s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3575ff85621343e6979f8c7c9efe09b4 : SUCCESS in 13m 14s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f22a4b64dc6d4a728dce169ae9e545b6 : SUCCESS in 12m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/2aff12c655354e8da081a4af33ceb7a9 : SUCCESS in 13m 58s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/461aa7217eaf4d93af4f683b042629ad : SUCCESS in 14m 52s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/267fe96ab6e74a52a56e88495b9c35a6 : SUCCESS in 9m 45s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/fa2aae05abd2462397a17103c12843d1 : SUCCESS in 1h 05m 25s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/06d09831f8ee40e0a1bb03468bbe1cd5 : SUCCESS in 1h 25m 45s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/69856b3601914fc0b70c3fc1cef7d7c3 : SUCCESS in 46m 22s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/22e586787d6f4c8094dc6e22c7c3983e : SUCCESS in 50m 44s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/c8ce76a82dc241eca3caddf7654b3ef3 : SUCCESS in 1h 37m 00s\n- nova-next https://zuul.opendev.org/t/openstack/build/47ca677aeed84ba3ae56ca155ce4bc79 : SUCCESS in 1h 27m 14s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/a26709bedbc7416387a3268150c4729c : SUCCESS in 5m 28s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/8d84806db1c94c74859291b6a658caac : SUCCESS in 18m 10s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/232e01fb72824cb9899139b0223ab56e : SUCCESS in 24m 58s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/336f1f8c60c74bbabe1ab2dc8c77a033 : SUCCESS in 1h 44m 29s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/1c4c2c80d35341feaab8f58296b00e03 : SUCCESS in 1h 00m 07s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/6619275b9c3e4a36b32916d1c5545969 : SUCCESS in 39m 55s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/e618c943353740598473f587202eb3cd : SUCCESS in 42m 39s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"667ba69ef10eed48980d8e993ab3c2e173bd9e14","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-12-06 23:27:17.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 https://elab-os-logsrv.delllabs.net/81/820381/4/check/EMC_PowerFlex_NOVA/836ccae/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":"e3038de0cc4fcb32fc141b755f354d4d822920d3","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-12-07 07:28:16.000000000","message":"Patch Set 4:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/820381/820381/4/check-not-vote/ext-nova-zuul/d77062e : FAILURE in 1m 45s","accounts_in_message":[],"_revision_number":4},{"id":"122f30f371867edb0992d6260871f4cb65c047aa","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-12-07 08:24:12.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"e80e36b2538df0e29801b9300def612b66fc2d4e","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-12-07 08:37:46.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"7d3b279007a4ee48d4763c510275fe239f9908fc","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2021-12-07 10:35:37.000000000","message":"Patch Set 4: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"1b9dc5768661be70c4c756477988fab565165cdf","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-12-07 10:36:52.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"816ff60658f66066b6f6913672386662a9d0541d","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-07 13:02:42.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"ca32bbf4352cab519ed868f09558e54e5ace29cd","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2021-12-07 13:19:53.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"24daad88ad003365d7f9988acdc9a8aa805c8402","tag":"autogenerated:gerrit:abandon","author":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"date":"2022-02-07 18:45:25.000000000","message":"Abandoned\n\nAbandoning in favor of https://review.opendev.org/c/openstack/nova/+/820549","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"1c1fde108737c8718974ee24f531414d7e6d26a1","revisions":{"278bb32df8a2bf674d553618b689216932ff15c8":{"kind":"REWORK","_number":1,"created":"2021-12-03 13:28:52.000000000","uploader":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"ref":"refs/changes/81/820381/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/820381/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/820381/1"}}},"commit":{"parents":[{"commit":"7670303aabe16d1d7c25e411d7bd413aee7fdcf3","subject":"Merge \"Remove broken tempest-full-py3-opensuse15 job\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7670303aabe16d1d7c25e411d7bd413aee7fdcf3"}]}],"author":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 13:13:21.000000000","tz":-180},"committer":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 13:27:31.000000000","tz":-180},"subject":"[WIP] Set resize claim resource when changing host","message":"[WIP] Set resize claim resource when changing host\n\nWhen migrating/resizing (with migrate) an instance, the\nupdate_available_resource [1] task may execute while the instance being\nmigrated is in an inconsistent state. This may cause the destination\nhost resource inventory to be incorrectly updated, and a subsequent\nrevert attempt to fail when trying to drop the resource claims.\n\nAn observed effect of this is that the revert operation may fail to\nunpin the CPUs from the destination host, leaving the instance in\n\"ERROR\" state, with a \"fault\" message similar to the following:\n\n\"CPU set to unpin [8] must be a subset of pinned CPU set []\"\n\nThis issue occurs when the periodic task executes between two events\nduring the resize operation:\n    - The instance host being updated and saved as the destination host\n      [2]\n    - The instance configuration (such as flavor and NUMA topology)\n      being updated and saved to reflect its selected configuration for\n      the destination host [3]\n\nDuring this interval, the task will consider that the instance belongs\nthe destination host, but since it is still with its source\nconfiguration, it will allocate the resources based on the source host.\nMore specifically, regarding CPU pinning, the task will attempt to pin\nthe CPU numbers that were pinned for the source host on the destination.\n\nThe resize/migration itself will work as expected, since it will update\nthe configuration accordingly afterwards. However, if the selected CPU\nnumbers for the destination differ from the original ones on the source,\nthe PCPU inventory in the resource_tracker will be temporarily\nincorrect. If the revert operation tries to drop the claims before the\ninventory is corrected, then it will fail to unpin the CPUs, because\nthey will not be considered as pinned.\n\nAdding a simple fix to the specific issue of CPU pinning. It consists of\nmoving the \u0027instance.apply_migration_context()\u0027 call from\n_finish_resize() [4] to the same update where the instance host is saved\nas the destination host, on _resize_instance() [2]. This way, if the\nperiodic task runs before [3], the instance will already reference the\ncorrect resource when the inventory is updated.\n\nLimitation: this behavior was not tested with cross-cell resize/migrate,\nand should not address this issue for it, in case it also occurs, since\nthe code flow for cross-cell is different.\n\nTest Plan:\n\nPASS: verify that the periodic task updates the destination\nhost NUMA topology accordingly, when executed between both mentioned\nevents\n\nPASS: verify that the revert operation executes successfully,\nwhen the periodic task runs between the two events during the\nresize/migration\n\n[1]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L10045\n[2]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5653\n[3]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5795\n[4]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5780\n\nPartial-Bug: #1952915\nSigned-off-by: Gabriel Silva Trevisan \u003cgabriel.silvatrevisan@windriver.com\u003e\nCo-Authored-By: Marcelo Mistutoshi Uesono \u003cmarcelomitsutoshi.uesono@windriver.com\u003e\nCo-Authored-By: Ricardo Soares Sarto \u003cricardo.soaressarto@windriver.com\u003e\nChange-Id: Idb55c81639139efbd44b9f8c938276827898ae44\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/278bb32df8a2bf674d553618b689216932ff15c8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/278bb32df8a2bf674d553618b689216932ff15c8"}]},"branch":"refs/heads/master"},"1a144821489a3ebae4bef4a96d68612cde9b5d35":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2021-12-03 17:29:53.000000000","uploader":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"ref":"refs/changes/81/820381/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/820381/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/820381/2"}}},"commit":{"parents":[{"commit":"7670303aabe16d1d7c25e411d7bd413aee7fdcf3","subject":"Merge \"Remove broken tempest-full-py3-opensuse15 job\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7670303aabe16d1d7c25e411d7bd413aee7fdcf3"}]}],"author":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 13:13:21.000000000","tz":-180},"committer":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 17:12:49.000000000","tz":-180},"subject":"[WIP] Set resize claim resource when changing host","message":"[WIP] Set resize claim resource when changing host\n\nWhen single-cell migrating/resizing (with migrate) an instance, the\nupdate_available_resource [1] task may execute while the instance being\nmigrated is in an inconsistent state. This may cause the destination\nhost resource inventory to be incorrectly updated, and a subsequent\nrevert attempt to fail when trying to drop the resource claims.\n\nAn observed effect of this is that the revert operation may fail to\nunpin the CPUs from the destination host, leaving the instance in\n\"ERROR\" state, with a \"fault\" message similar to the following:\n\n\"CPU set to unpin [8] must be a subset of pinned CPU set []\"\n\nThis issue occurs when the periodic task executes between two events\nduring the resize operation:\n    - The instance host being updated and saved as the destination host\n      [2]\n    - The instance configuration (such as flavor and NUMA topology)\n      being updated and saved to reflect its selected configuration for\n      the destination host [3]\n\nDuring this interval, the task will consider that the instance belongs\nthe destination host, but since it is still with its source\nconfiguration, it will allocate the resources based on the source host.\nMore specifically, regarding CPU pinning, the task will attempt to pin\nthe CPU numbers that were pinned for the source host on the destination.\n\nThe resize/migration itself will work as expected, since it will update\nthe configuration accordingly afterwards. However, if the selected CPU\nnumbers for the destination differ from the original ones on the source,\nthe PCPU inventory in the resource_tracker will be temporarily\nincorrect. If the revert operation tries to drop the claims before the\ninventory is corrected, then it will fail to unpin the CPUs, because\nthey will not be considered as pinned.\n\nAdding a simple fix to the specific issue of CPU pinning. It consists of\nmoving the \u0027instance.apply_migration_context()\u0027 call from\n_finish_resize() [4] to the same update where the instance host is saved\nas the destination host, on _resize_instance() [2]. This way, if the\nperiodic task runs before [3], the instance will already reference the\ncorrect resource when the inventory is updated.\n\nNote: This change should only affect single-cell resize/migrate behavior.\nCross-cell resize/migrate behavior should not be affected by it, since\nthe code flow differs.\n\nTest Plan:\n\nPASS: verify that the periodic task updates the destination\nhost NUMA topology accordingly, when executed between both mentioned\nevents\n\nPASS: verify that the revert operation executes successfully,\nwhen the periodic task runs between the two events during the\nresize/migration\n\n[1]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L10045\n[2]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5653\n[3]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5795\n[4]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5780\n\nPartial-Bug: #1952915\nSigned-off-by: Gabriel Silva Trevisan \u003cgabriel.silvatrevisan@windriver.com\u003e\nCo-Authored-By: Marcelo Mistutoshi Uesono \u003cmarcelomitsutoshi.uesono@windriver.com\u003e\nCo-Authored-By: Ricardo Soares Sarto \u003cricardo.soaressarto@windriver.com\u003e\nChange-Id: Idb55c81639139efbd44b9f8c938276827898ae44\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a144821489a3ebae4bef4a96d68612cde9b5d35"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a144821489a3ebae4bef4a96d68612cde9b5d35"}]},"branch":"refs/heads/master"},"0471e0c4dd4c293a54a79b2a8077c92e12e0756c":{"kind":"REWORK","_number":3,"created":"2021-12-06 18:59:53.000000000","uploader":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"ref":"refs/changes/81/820381/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/820381/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/820381/3"}}},"commit":{"parents":[{"commit":"7670303aabe16d1d7c25e411d7bd413aee7fdcf3","subject":"Merge \"Remove broken tempest-full-py3-opensuse15 job\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7670303aabe16d1d7c25e411d7bd413aee7fdcf3"}]}],"author":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 13:13:21.000000000","tz":-180},"committer":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-06 18:59:16.000000000","tz":-180},"subject":"[WIP] Set resize claim resource when changing host","message":"[WIP] Set resize claim resource when changing host\n\nWhen single-cell migrating/resizing (with migrate) an instance, the\nupdate_available_resource [1] task may execute while the instance being\nmigrated is in an inconsistent state. This may cause the destination\nhost resource inventory to be incorrectly updated, and a subsequent\nrevert attempt to fail when trying to drop the resource claims.\n\nAn observed effect of this is that the revert operation may fail to\nunpin the CPUs from the destination host, leaving the instance in\n\"ERROR\" state, with a \"fault\" message similar to the following:\n\n\"CPU set to unpin [8] must be a subset of pinned CPU set []\"\n\nThis issue occurs when the periodic task executes between two events\nduring the resize operation:\n    - The instance host being updated and saved as the destination host\n      [2]\n    - The instance configuration (such as flavor and NUMA topology)\n      being updated and saved to reflect its selected configuration for\n      the destination host [3]\n\nDuring this interval, the task will consider that the instance belongs\nthe destination host, but since it is still with its source\nconfiguration, it will allocate the resources based on the source host.\nMore specifically, regarding CPU pinning, the task will attempt to pin\nthe CPU numbers that were pinned for the source host on the destination.\n\nThe resize/migration itself will work as expected, since it will update\nthe configuration accordingly afterwards. However, if the selected CPU\nnumbers for the destination differ from the original ones on the source,\nthe PCPU inventory in the resource_tracker will be temporarily\nincorrect. If the revert operation tries to drop the claims before the\ninventory is corrected, then it will fail to unpin the CPUs, because\nthey will not be considered as pinned.\n\nAdding a simple fix to the specific issue of CPU pinning. It consists of\nmoving the \u0027instance.apply_migration_context()\u0027 call from\n_finish_resize() [4] to the same update where the instance host is saved\nas the destination host, on _resize_instance() [2]. This way, if the\nperiodic task runs before [3], the instance will already reference the\ncorrect resource when the inventory is updated.\n\nNote: This change should only affect single-cell resize/migrate behavior.\nCross-cell resize/migrate behavior should not be affected by it, since\nthe code flow differs.\n\nTest Plan:\n\nPASS: verify that the periodic task updates the destination\nhost NUMA topology accordingly, when executed between both mentioned\nevents\n\nPASS: verify that the revert operation executes successfully,\nwhen the periodic task runs between the two events during the\nresize/migration\n\n[1]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L10045\n[2]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5653\n[3]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5795\n[4]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5780\n\nPartial-Bug: #1952915\nSigned-off-by: Gabriel Silva Trevisan \u003cgabriel.silvatrevisan@windriver.com\u003e\nCo-Authored-By: Marcelo Mistutoshi Uesono \u003cmarcelomitsutoshi.uesono@windriver.com\u003e\nCo-Authored-By: Ricardo Soares Sarto \u003cricardo.soaressarto@windriver.com\u003e\nChange-Id: Idb55c81639139efbd44b9f8c938276827898ae44\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0471e0c4dd4c293a54a79b2a8077c92e12e0756c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0471e0c4dd4c293a54a79b2a8077c92e12e0756c"}]},"branch":"refs/heads/master"},"1c1fde108737c8718974ee24f531414d7e6d26a1":{"kind":"REWORK","_number":4,"created":"2021-12-06 21:38:50.000000000","uploader":{"_account_id":33608,"name":"Gabriel Silva Trevisan","display_name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","username":"gtrevisan"},"ref":"refs/changes/81/820381/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/81/820381/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/81/820381/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/81/820381/4"}}},"commit":{"parents":[{"commit":"7670303aabe16d1d7c25e411d7bd413aee7fdcf3","subject":"Merge \"Remove broken tempest-full-py3-opensuse15 job\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7670303aabe16d1d7c25e411d7bd413aee7fdcf3"}]}],"author":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-03 13:13:21.000000000","tz":-180},"committer":{"name":"Gabriel Silva Trevisan","email":"gabriel.silvatrevisan@windriver.com","date":"2021-12-06 21:35:17.000000000","tz":-180},"subject":"[WIP] Set resize claim resource when changing host","message":"[WIP] Set resize claim resource when changing host\n\nWhen single-cell migrating/resizing (with migrate) an instance, the\nupdate_available_resource [1] task may execute while the instance being\nmigrated is in an inconsistent state. This may cause the destination\nhost resource inventory to be incorrectly updated, and a subsequent\nrevert attempt to fail when trying to drop the resource claims.\n\nAn observed effect of this is that the revert operation may fail to\nunpin the CPUs from the destination host, leaving the instance in\n\"ERROR\" state, with a \"fault\" message similar to the following:\n\n\"CPU set to unpin [8] must be a subset of pinned CPU set []\"\n\nThis issue occurs when the periodic task executes between two events\nduring the resize operation:\n    - The instance host being updated and saved as the destination host\n      [2]\n    - The instance configuration (such as flavor and NUMA topology)\n      being updated and saved to reflect its selected configuration for\n      the destination host [3]\n\nDuring this interval, the task will consider that the instance belongs\nthe destination host, but since it is still with its source\nconfiguration, it will allocate the resources based on the source host.\nMore specifically, regarding CPU pinning, the task will attempt to pin\nthe CPU numbers that were pinned for the source host on the destination.\n\nThe resize/migration itself will work as expected, since it will update\nthe configuration accordingly afterwards. However, if the selected CPU\nnumbers for the destination differ from the original ones on the source,\nthe PCPU inventory in the resource_tracker will be temporarily\nincorrect. If the revert operation tries to drop the claims before the\ninventory is corrected, then it will fail to unpin the CPUs, because\nthey will not be considered as pinned.\n\nAdding a simple fix to the specific issue of CPU pinning. It consists of\nmoving the \u0027instance.apply_migration_context()\u0027 call from\n_finish_resize() [4] to the same update where the instance host is saved\nas the destination host, on _resize_instance() [2]. This way, if the\nperiodic task runs before [3], the instance will already reference the\ncorrect resource when the inventory is updated.\n\nNote: This change should only affect single-cell resize/migrate behavior.\nCross-cell resize/migrate behavior should not be affected by it, since\nthe code flow differs.\n\nTest Plan:\n\nPASS: verify that the periodic task updates the destination\nhost NUMA topology accordingly, when executed between both mentioned\nevents\n\nPASS: verify that the revert operation executes successfully,\nwhen the periodic task runs between the two events during the\nresize/migration\n\n[1]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L10045\n[2]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5653\n[3]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5795\n[4]\nhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5780\n\nPartial-Bug: #1952915\nSigned-off-by: Gabriel Silva Trevisan \u003cgabriel.silvatrevisan@windriver.com\u003e\nCo-Authored-By: Marcelo Mistutoshi Uesono \u003cmarcelomitsutoshi.uesono@windriver.com\u003e\nCo-Authored-By: Ricardo Soares Sarto \u003cricardo.soaressarto@windriver.com\u003e\nChange-Id: Idb55c81639139efbd44b9f8c938276827898ae44\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1c1fde108737c8718974ee24f531414d7e6d26a1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1c1fde108737c8718974ee24f531414d7e6d26a1"}]},"branch":"refs/heads/master","description":"Fix test code formatting"}},"requirements":[],"submit_records":[],"submit_requirements":[]}