Skip to content

[v1] (manual clean backport) Add tag field to compute block_device_v2#2782

Merged
mandre merged 4 commits intogophercloud:v1from
EmilienM:backport_tag_volume
Sep 22, 2023
Merged

[v1] (manual clean backport) Add tag field to compute block_device_v2#2782
mandre merged 4 commits intogophercloud:v1from
EmilienM:backport_tag_volume

Conversation

@EmilienM
Copy link
Copy Markdown
Contributor

  • Add tag field to compute block_device_v2
  • Add requests test for new field
  • acceptance/compute: remove flavor ID check
  • Add acceptance tests for tag field to compute block_device_v2

This is a clean backport with no conflict.
I had to do it manually because the workflow has been fixed after these commits so it didn't run.

Matt Pryor and others added 4 commits September 21, 2023 15:48
(cherry picked from commit f218ca0)
If the client uses a microversion > 2.46, the flavor ID doesn't exist.
See https://docs.openstack.org/api-ref/compute/#id30

To avoid complexity when checking the flavor, let's just remove that
test.

(cherry picked from commit 93e5d0f)
@github-actions github-actions bot added the semver:minor Backwards-compatible change label Sep 21, 2023
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 79.122%. remained the same when pulling 4ecebfb on EmilienM:backport_tag_volume into 48897e2 on gophercloud:v1.

@EmilienM
Copy link
Copy Markdown
Contributor Author

For CI to work we need #2783

@mandre mandre merged commit 5bf280d into gophercloud:v1 Sep 22, 2023
@pierreprinetti pierreprinetti added the v1 This PR targets v1 label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:minor Backwards-compatible change v1 This PR targets v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants