Skip to content

[v1] Add CRUD support for encryption in volume v3 types#2714

Merged
mandre merged 3 commits intov1from
bp-v1-a365a68-8d2245e-2267522
Aug 2, 2023
Merged

[v1] Add CRUD support for encryption in volume v3 types#2714
mandre merged 3 commits intov1from
bp-v1-a365a68-8d2245e-2267522

Conversation

@gophercloud-backport-bot
Copy link
Copy Markdown

Backport: #2642

Add Create/Delete/Update/Get support for encryption of volume types.

Fixes #2641

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

  1. Create an encryption for a volume type:

  2. Update an encryption for a volume type:

  3. Delete an encryption for a volume type:

  4. Show an encryption for a volume type:

  5. Show an encryption spec for a volume type:

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 79.085% (+0.04%) from 79.05% when pulling 7192499 on bp-v1-a365a68-8d2245e-2267522 into 283c721 on v1.

@mandre
Copy link
Copy Markdown
Contributor

mandre commented Aug 2, 2023

For the failed cinder tests, only TestVolumesMultiAttach test failed, which is fixed by #2713. It should be safe to merge as is, rather than rebasing and re-running all the tests again. I'm just re-running the xena job because one of the test failed with a timeout.

@mandre mandre added the semver:minor Backwards-compatible change label Aug 2, 2023
Copy link
Copy Markdown
Contributor

@mandre mandre left a comment

Choose a reason for hiding this comment

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

Only the TestVolumesMultiAttach failed in the xena job this time, as expected.

@mandre mandre merged commit b94e2f7 into v1 Aug 2, 2023
@mandre mandre deleted the bp-v1-a365a68-8d2245e-2267522 branch August 2, 2023 04:57
@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.

3 participants