Skip to content

[v2] neutron: allow omission of subnet_id for IP address - #3452

Merged
kayrus merged 1 commit into
v2from
bp-v2-3a9dda3
Jul 9, 2025
Merged

kayrus merged 1 commit into
v2from
bp-v2-3a9dda3

Conversation

@gophercloud-backport-bot

Copy link
Copy Markdown

Backport: #3448

I could not find any docs about external_fixed_ip struct to support omitted subnet_id, but I created test case in terraform provider and it works as expected, see terraform-provider-openstack/terraform-provider-openstack#1915

As for the port's subnet_id, the omission behavior is documented here:

The IP addresses for the port. If you would like to assign multiple IP addresses for the port, specify multiple entries in this field. Each entry consists of IP address (ip_address) and the subnet ID from which the IP address is assigned (subnet_id).
If you specify both a subnet ID and an IP address, OpenStack Networking tries to allocate the IP address on that subnet to the port.
If you specify only a subnet ID, OpenStack Networking allocates an available IP from that subnet to the port.
If you specify only an IP address, OpenStack Networking tries to allocate the IP address if the address is a valid IP for any of the subnets on the specified network.

@gophercloud-backport-bot
gophercloud-backport-bot Bot requested a review from kayrus July 9, 2025 16:21
@github-actions github-actions Bot added edit:networking This PR updates networking code v2 This PR targets v2 semver:patch No API change labels Jul 9, 2025
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 63.571%. remained the same
when pulling a776073 on bp-v2-3a9dda3
into 01ba8ed on v2.

@kayrus kayrus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kayrus
kayrus merged commit 3238181 into v2 Jul 9, 2025
@kayrus
kayrus deleted the bp-v2-3a9dda3 branch July 9, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:networking This PR updates networking code semver:patch No API change v2 This PR targets v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants