commit 9fe465427310f8215890d26bf169617653605e23
Author: Stephen Finucane <email address hidden>
Date: Mon Jan 31 12:56:57 2022 +0000
api: Reject duplicate port IDs in server create
Specifying a duplicate port ID is currently "allowed" but results in an
integrity error when nova attempts to create a duplicate
'VirtualInterface' entry. Start rejecting these requests by checking for
duplicate IDs and rejecting offending requests. This is arguably an API
change because there isn't a HTTP 5xx error (server create is an async
operation), however, users shouldn't have to opt in to non-broken
behavior and the underlying instance was never actually created
previously, meaning automation that relied on this "feature" was always
going to fail in a later step. We're also silently failing to do what
the user asked (per flow chart at [1]).
Reviewed: https:/ /review. opendev. org/c/openstack /nova/+ /827070 /opendev. org/openstack/ nova/commit/ 9fe465427310f82 15890d26bf16961 7653605e23
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 9fe465427310f82 15890d26bf16961 7653605e23
Author: Stephen Finucane <email address hidden>
Date: Mon Jan 31 12:56:57 2022 +0000
api: Reject duplicate port IDs in server create
Specifying a duplicate port ID is currently "allowed" but results in an erface' entry. Start rejecting these requests by checking for
integrity error when nova attempts to create a duplicate
'VirtualInt
duplicate IDs and rejecting offending requests. This is arguably an API
change because there isn't a HTTP 5xx error (server create is an async
operation), however, users shouldn't have to opt in to non-broken
behavior and the underlying instance was never actually created
previously, meaning automation that relied on this "feature" was always
going to fail in a later step. We're also silently failing to do what
the user asked (per flow chart at [1]).
[1] https:/ /docs.openstack .org/nova/ latest/ contributor/ microversions. html#when- do-i-need- a-new-microvers ion
Change-Id: Ie90fb83662dd06 e7188f042fc6340 596f93c5ef9
Signed-off-by: Stephen Finucane <email address hidden>
Closes-Bug: #1821088