Skip to content

neutron: add new router attributes #3447

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kayrus
Copy link
Contributor

@kayrus kayrus commented Jul 1, 2025

Add new extension fields. Not sure if to put them as separate extensions, since other extension fields like distributed or availability_zone_hints were already implemented directly inside the routers package.

Likewise in #3424, if user doesn't specify these parameters, they won't be sent to API. Therefore from my perspective it is safe to keep them here, otherwise the consumer code complexity will be increased.

@github-actions github-actions bot added edit:networking This PR updates networking code semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Jul 1, 2025
@coveralls
Copy link

Coverage Status

coverage: 63.702%. remained the same
when pulling 4e7b6fa on kayrus:new-router-attributes
into f98c2f9 on gophercloud:main.

This was referenced Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v2 This PR will be backported to v2 edit:networking This PR updates networking code semver:minor Backwards-compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants