Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[v2] Improve support for network standard-attr-* extensions #3328

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

Merged
merged 4 commits into from
Mar 18, 2025

Conversation

mandre
Copy link
Contributor

@mandre mandre commented Mar 12, 2025

Manual backport of #3318 and #3326

Consistently add support for the neutron Resource revision numbers
extension [1] for more resources that support it. It adds
a RevisionNumber field to all responses, and allows filtering via the
RevisionNumber list option. It was only partially implemented.

Support the If-Match constraints for these resources, and add test for it.

Add a few missing CreatedAt and UpdatedAt fields for networking
resources.

[1] https://docs.openstack.org/api-ref/network/v2/#revisions

mandre added 4 commits March 12, 2025 08:17
Consistently add support for the neutron `Resource revision numbers`
extension [1] for more resources that support it. It adds
a `RevisionNumber` field to all responses, and allows filtering via the
`RevisionNumber` list option. It was only partially implemented.

Also support the `If-Match` constraints, and add test for it.

[1] https://docs.openstack.org/api-ref/network/v2/#revisions
Add a few missing `CreatedAt` and `UpdatedAt` fields for networking
resources.
Since 5819dc0, `getReleaseFromEnv()`
trims the `stable/` prefix, which caused the condition in
`SkipRelease()` to fail if we looked for a release name including the
prefix. Fix `SkipRelease()` to account for release names with a `stable/` prefix.
@github-actions github-actions bot added edit:networking This PR updates networking code v2 This PR targets v2 edit:testinfra This PR updates testing infrastructure code semver:minor Backwards-compatible change labels Mar 12, 2025
@coveralls
Copy link

Coverage Status

coverage: 78.647% (-0.09%) from 78.732%
when pulling 9e7a3d4 on shiftstack:v2-network-standard-attr
into fe8ded1 on gophercloud:v2.

@mandre
Copy link
Contributor Author

mandre commented Mar 12, 2025

man, this TestBGPAgentRUD test is so flaky, this is annoying.

@mandre mandre merged commit b4af4f3 into gophercloud:v2 Mar 18, 2025
17 of 18 checks passed
@mandre mandre deleted the v2-network-standard-attr branch March 18, 2025 10:45
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 edit:testinfra This PR updates testing infrastructure code semver:minor Backwards-compatible change v2 This PR targets v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.