)]}'
{"id":"openstack%2Fnova~827123","triplet_id":"openstack%2Fnova~stable%2Fussuri~Ie90fb83662dd06e7188f042fc6340596f93c5ef9","project":"openstack/nova","branch":"stable/ussuri","topic":"bug/1821088","attention_set":{},"removed_from_attention_set":{"15334":{"account":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"last_update":"2023-11-13 17:15:59.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Ie90fb83662dd06e7188f042fc6340596f93c5ef9","subject":"api: Reject duplicate port IDs in server create","status":"ABANDONED","created":"2022-01-31 15:09:18.000000000","updated":"2023-11-13 17:15:59.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a11f8cc72ef248151444df93033523a2bb1f3a1c","_number":827123,"virtual_id_number":827123,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2022-01-31 15:56:34.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"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":"","default_value":0,"optional":true},"Workflow":{"all":[{"_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":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":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"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2022-01-31 15:56:34.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2022-01-31 17:28: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"}],"messages":[{"id":"19bf032f895a3a36d3ef1f13b407767dd0ddf083","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2022-01-31 15:09:18.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"53709d11bfededd8c6f90a930f11a0ec36877aab","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-01-31 15:56:34.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/8537b7c86ac740609b0bd8fc1b31c144 : FAILURE in 18m 35s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/0fd2fd0a6522404fa9064348fb62530c : POST_FAILURE in 17m 08s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/f92a22389502402c94914d21e27dd9b2 : SUCCESS in 18m 45s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/21c5b6876065419da612dcc2dc9d7ac7 : SUCCESS in 8m 32s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/9f03394971a24225ae862f315c6fd885 : SUCCESS in 15m 59s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/9c7d74430d724daca16d760ac4a228dd : SUCCESS in 14m 12s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5dd761b6c7844ce9bcdef0cf7427ea74 : SUCCESS in 12m 25s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d2b9458c407e43f2b494ab87872baffb : SUCCESS in 23m 50s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/6b16bae234f34ab39cd6840f3a7d9211 : FAILURE in 11m 09s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/5ab75563c1834051aa7955cea73e9867 : FAILURE in 14m 58s (non-voting)\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/e35fea0dbf3c4a0c8a0f897825a99d25 : POST_FAILURE in 13m 34s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/2e767f1d3df74d5a847b7d38b9d6dd8d : FAILURE in 13m 28s\n- nova-next https://zuul.opendev.org/t/openstack/build/3fe6001707c74a739385762bfeff7559 : POST_FAILURE in 20m 52s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/4bd14e5f3c204ef69dc58eceb21475ff : SUCCESS in 21m 38s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/e4bcba0806cb446e9e0ff38b186d17c8 : FAILURE in 7m 10s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b453bb9866724150a9ccbd8d8c73b8ec : SUCCESS in 17m 08s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/d7c77b0c9a554eb48ac5233d8a51121b : FAILURE in 23m 10s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/83ab304a09cf4fbf9a2ed3f493504284 : FAILURE in 16m 50s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/917de3ba960d42e19b7f91c41e703613 : FAILURE in 17m 56s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a6bd4648849b625b50c810d349bff37e14b1a193","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2022-01-31 17:28:56.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-bionic https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/23/827123/1/check/tempest-dsvm-full-bionic/ff6c91b/ : SUCCESS in 2h 03m 46s\n- tempest-dsvm-full-bionic-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/23/827123/1/check/tempest-dsvm-full-bionic-py3/5acaf68/ : SUCCESS in 1h 52m 33s","accounts_in_message":[],"_revision_number":1},{"id":"9e2d663a6e1ca358381248177a43a79e23176355","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2022-02-02 12:24:18.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"b0ec40f48cd9d82deb2e66b65be03ff868144c45","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-02-02 12:58:52.000000000","message":"Patch Set 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/7a50c652839643dc916296de53364aac : FAILURE in 16m 57s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/aa70129e3a0247b8894f424de43c83eb : POST_FAILURE in 8m 40s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/9410eab88d4048738ef35185b80a32f6 : SUCCESS in 17m 40s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/14030723139843628ae0f36ba604a381 : SUCCESS in 9m 34s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/60a61a5415d3403691efaff1872a7b16 : SUCCESS in 14m 45s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/75336afad31d486a95aa3899ddfff930 : SUCCESS in 13m 33s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2eca3c23228a4ab4bf5da343a629c7f4 : SUCCESS in 9m 36s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/98dcf7589dd94a22a9958d43bcec6c3c : SUCCESS in 21m 17s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/a3a063930b804be79b4c3624e740c2eb : FAILURE in 12m 10s (non-voting)\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/0c78dcd1e6914b1da380d972e020819e : FAILURE in 9m 21s (non-voting)\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/07adbfb7793e466396b6d53adb63fc7c : POST_FAILURE in 19m 05s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/25724ce062ce41ba9081b20d797993ef : FAILURE in 19m 45s\n- nova-next https://zuul.opendev.org/t/openstack/build/849390da32d443fea32e2dfee459d9c5 : POST_FAILURE in 18m 13s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/f557c20c50824de6bd259843b331d671 : SUCCESS in 18m 54s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/631ff3dd751a47a4bb98a2f099195e31 : FAILURE in 5m 56s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b24e2f32b9c14739a9f9964df66a20bd : SUCCESS in 15m 34s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/721cb3c676ba429eb2e5864837678440 : FAILURE in 16m 50s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4b233370582c4ec8affab9085be15a85 : FAILURE in 17m 13s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/a6c3e49f0f3e4e7b92dda93b890a69de : FAILURE in 15m 45s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e5bb59998ad0828a524743c9be8a4fc3e2aadacc","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2022-02-02 15:46:47.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-bionic https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/23/827123/1/check/tempest-dsvm-full-bionic/65f9650/ : SUCCESS in 2h 50m 52s\n- tempest-dsvm-full-bionic-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/23/827123/1/check/tempest-dsvm-full-bionic-py3/0d08d31/ : SUCCESS in 2h 32m 58s","accounts_in_message":[],"_revision_number":1},{"id":"a11f8cc72ef248151444df93033523a2bb1f3a1c","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2023-11-13 17:15:59.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"23e7e6011b9087e25ca14f0da42ebafb0bef653a","revisions":{"23e7e6011b9087e25ca14f0da42ebafb0bef653a":{"kind":"REWORK","_number":1,"created":"2022-01-31 15:09:18.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/23/827123/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/23/827123/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/23/827123/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/23/827123/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/23/827123/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/23/827123/1"}}},"commit":{"parents":[{"commit":"184a3c976faed38907af148a533bc6e9faa410f5","subject":"Ensure MAC addresses characters are in the same case","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/184a3c976faed38907af148a533bc6e9faa410f5"}]}],"author":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2022-01-31 12:56:57.000000000","tz":0},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2022-01-31 15:09:08.000000000","tz":0},"subject":"api: Reject duplicate port IDs in server create","message":"api: Reject duplicate port IDs in server create\n\nSpecifying a duplicate port ID is currently \"allowed\" but results in an\nintegrity error when nova attempts to create a duplicate\n\u0027VirtualInterface\u0027 entry. Start rejecting these requests by checking for\nduplicate IDs and rejecting offending requests. This is arguably an API\nchange because there isn\u0027t a HTTP 5xx error (server create is an async\noperation), however, users shouldn\u0027t have to opt in to non-broken\nbehavior and the underlying instance was never actually created\npreviously, meaning automation that relied on this \"feature\" was always\ngoing to fail in a later step. We\u0027re also silently failing to do what\nthe user asked (per flow chart at [1]).\n\n[1] https://docs.openstack.org/nova/latest/contributor/microversions.html#when-do-i-need-a-new-microversion\n\nChange-Id: Ie90fb83662dd06e7188f042fc6340596f93c5ef9\nSigned-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e\nCloses-Bug: #1821088\n(cherry picked from commit 9fe465427310f8215890d26bf169617653605e23)\n(cherry picked from commit d4c92bc2314c6402c4e3464a5aa61a94293eaf91)\n(cherry picked from commit fcbe6eecd7358dfa22b2dca8eca2afaa11f882f3)\n(cherry picked from commit 3731d1f5afe28b1cb6d36e88c407b65d1653a6df)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/23e7e6011b9087e25ca14f0da42ebafb0bef653a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/23e7e6011b9087e25ca14f0da42ebafb0bef653a"}]},"branch":"refs/heads/stable/ussuri"}},"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"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"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":{}}}]}
