)]}'
{"id":"openstack%2Fnova~791553","triplet_id":"openstack%2Fnova~master~Ia3b9bf837b3aeb8c7bc2b1887ae745a2a82cf595","project":"openstack/nova","branch":"master","hashtags":[],"change_id":"Ia3b9bf837b3aeb8c7bc2b1887ae745a2a82cf595","subject":"WIP: Don\u0027t update the MTU during a live migration","status":"ABANDONED","created":"2021-05-14 21:08:17.000000000","updated":"2023-03-14 13:15:57.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ee46fcabee80e8516d7ad2170b1ec2ea60a002de","_number":791553,"virtual_id_number":791553,"owner":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-05-14 22:28:19.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"all":[{"value":-1,"date":"2021-05-14 21:28:04.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-05-14 21:28:04.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2021-05-14 21:33:57.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-05-14 22:12:56.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-05-14 22:28:19.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-05-14 23:34:32.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"}],"messages":[{"id":"11fc2096045874bdc1ab43ba36953327224afca8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-05-14 21:08:17.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"17f4fe6bc789dd5dba4f3b6800733f37f6481cb4","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2021-05-14 21:28:04.000000000","message":"Patch Set 1: Code-Review-1\n\nwe can do this but it is still incorreect.\n\nif you change the mtu of a neutron network you are required to reatach all ports from the vm or hard reboot the instance.","accounts_in_message":[],"_revision_number":1},{"id":"67e889483152ab0aeba8499fb74472ba2cbdafb3","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-14 21:33:57.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/d07e2b2aff34402f990931f58367f2dd : FAILURE in 17m 53s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/959410aa0bde4cdaa09445013b973341 : FAILURE in 20m 08s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"814acc9e7600d7faf5e5f38b1d7532a9c5dba055","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-05-14 22:12:56.000000000","message":"Patch Set 1:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 finger://zuul-v3-executor.novalocal/ab2d94ca74a846d0a2e9964d206b008d : POST_FAILURE in 58m 41s","accounts_in_message":[],"_revision_number":1},{"id":"bcbd3487e1ce338f5ca04af0be9c76a92c1593ca","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-14 22:28:19.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/b45125edcc1f41b08e40f1099354ecc1 : SUCCESS in 1h 12m 22s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/8ea913c94a9d4c41991e59682e67b2b6 : FAILURE in 15m 54s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/020e0137e9164865b6e40cd8154d6bad : FAILURE in 14m 40s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/edd8cbfa91cc4974a1e472457592ca7a : SUCCESS in 8m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/12087944fedd431299c105ea59794b5d : FAILURE in 12m 58s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/57183f20982748779862c79047f21e31 : FAILURE in 13m 27s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/f2537639f600490e915f1c022537e48b : FAILURE in 13m 23s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e76414c4ff98400c97d91942c7b6f59d : SUCCESS in 9m 06s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/caed9ef803b94122b1cb620b81e8a2c6 : SUCCESS in 1h 06m 31s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/64cdb04a3b754594ac5c5ebb0f2ead3d : SUCCESS in 1h 14m 52s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/8d539b79d70e4177a1309222bf2a17f9 : SUCCESS in 45m 49s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/bedea24c6d8c4eefad74e789fd668b90 : SUCCESS in 45m 46s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/e0893fede48a4e1b895530b4d59dd10a : SUCCESS in 43m 25s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/a4e74376b1514c53a1016947a14e227a : SUCCESS in 1h 17m 58s\n- nova-next https://zuul.opendev.org/t/openstack/build/4b4ef0527ecd4dd49aa59d49769d98a6 : FAILURE in 1h 13m 44s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/8aee4190c5fc4a0e92d3cbfc5681f3af : FAILURE in 16m 43s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/f1f8a0feb0384b699c2988eb756f50da : FAILURE in 16m 39s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/096dc2913174445f95a80652284950d8 : SUCCESS in 1h 04m 23s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/af36e45ed45f4aa5b13e74486b0db77d : SUCCESS in 47m 32s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/03f282a5c63044c0a60e5b880a1c061a : SUCCESS in 44m 59s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/3f89b14387a943458fe1d9e7e2c7664f : SUCCESS in 30m 45s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/dc21cdaae066402dbd36782221a6fc08 : SUCCESS in 33m 01s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"8f63c359842dee822fe436c061d77e4d3b03f0ca","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-05-14 23:34:32.000000000","message":"Patch Set 1:\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/791553/1/iscsi : SUCCESS in 2h 23m 58s","accounts_in_message":[],"_revision_number":1},{"id":"06f5acd05981f426d92499a4cc18992503d46604","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2021-05-19 21:47:59.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e we can do this but it is still incorreect.\n\u003e \n\u003e if you change the mtu of a neutron network you are required to reatach all ports from the vm or hard reboot the instance.\n\nRight, I understand that Nova\u0027s half of the mutable MTU feature isn\u0027t implemented. But until that happens, I\u0027m not sure this patch is incorrect. Say an instance *wasn\u0027t* live migrated. It would still have the old MTU, with all the consequences that that entails, right? So this patch just persists that state across live migrations, with the added advantage that said live migrations don\u0027t explode :)","accounts_in_message":[],"_revision_number":1},{"id":"ee46fcabee80e8516d7ad2170b1ec2ea60a002de","tag":"autogenerated:gerrit:abandon","author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"date":"2023-03-14 13:15:57.000000000","message":"Abandoned\n\nhttps://review.opendev.org/c/openstack/nova/+/852367/1","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"8d33f5dfd757252a78d32253012c6b7589a41abd","revisions":{"8d33f5dfd757252a78d32253012c6b7589a41abd":{"kind":"REWORK","_number":1,"created":"2021-05-14 21:08:17.000000000","uploader":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"ref":"refs/changes/53/791553/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/53/791553/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/53/791553/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/53/791553/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/53/791553/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/53/791553/1"}}},"commit":{"parents":[{"commit":"397bcd939685e06d584efba20d8627a240334740","subject":"WIP: Regression test for changing the MTU during a live migration","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/397bcd939685e06d584efba20d8627a240334740"}]}],"author":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-05-14 21:07:20.000000000","tz":-240},"committer":{"name":"Artom Lifshitz","email":"alifshit@redhat.com","date":"2021-05-14 21:08:08.000000000","tz":-240},"subject":"WIP: Don\u0027t update the MTU during a live migration","message":"WIP: Don\u0027t update the MTU during a live migration\n\nChange-Id: Ia3b9bf837b3aeb8c7bc2b1887ae745a2a82cf595\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8d33f5dfd757252a78d32253012c6b7589a41abd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8d33f5dfd757252a78d32253012c6b7589a41abd"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}