Skip to content

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