Skip to content

doc: mention encryption is not supported for static PVC#6212

Merged
mergify[bot] merged 1 commit intoceph:develfrom
iPraveenParihar:doc-static-pvc-no-encryption-support
Apr 6, 2026
Merged

doc: mention encryption is not supported for static PVC#6212
mergify[bot] merged 1 commit intoceph:develfrom
iPraveenParihar:doc-static-pvc-no-encryption-support

Conversation

@iPraveenParihar
Copy link
Copy Markdown
Contributor

Describe what this PR does

doc: mention encryption is not supported for static PVC
Closes: #6209

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

@iPraveenParihar iPraveenParihar marked this pull request as ready for review March 31, 2026 06:18
@iPraveenParihar iPraveenParihar added component/docs Issues and PRs related to documentation ci/skip/e2e skip running e2e CI jobs labels Mar 31, 2026
@mergify mergify Bot added the ci/skip/multi-arch-build skip building on multiple architectures label Mar 31, 2026
Comment thread docs/static-pvc.md
> static PVC can be created, deleted, mounted and unmounted but
currently ceph-csi doesn't support other operations like snapshot,clone,
resize, etc for static PVC
resize, encryption etc for static PVC
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i think we also need to add a generic statement so that we dont need to keep this updated for all new features.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Keeping the current one + I have added a statement in both RBD and CephFS section.
Let me know if that makes sense?

Comment thread docs/static-pvc.md
resize, etc for static PVC
resize, encryption etc for static PVC

## RBD static PVC
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same for RBD as well

@iPraveenParihar iPraveenParihar force-pushed the doc-static-pvc-no-encryption-support branch from a2d200c to d01cd40 Compare March 31, 2026 08:14
@iPraveenParihar iPraveenParihar requested a review from Madhu-1 March 31, 2026 08:16
Copy link
Copy Markdown
Member

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

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

Thanks!

@nixpanic nixpanic requested a review from a team April 2, 2026 16:15
@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Apr 6, 2026

@Mergifyio rebase

@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Apr 6, 2026

@Mergifyio queue

Also add a generic warning under both RBD and CephFS sections
that features requiring ceph-csi to manage the volume lifecycle
are not supported for static PVC.

Closes: ceph#6209
Signed-off-by: Praveen M <m.praveen@ibm.com>
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 6, 2026

Deprecation notice: This pull request comes from a fork and was rebased using bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the rebase action will no longer be able to rebase fork pull requests with this configuration. Please switch to the update action/command to ensure compatibility going forward.

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 6, 2026

rebase

✅ Branch has been successfully rebased

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 6, 2026

Deprecation notice: This pull request comes from a fork and was queued with update_method=rebase and update_bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the merge queue will no longer be able to rebase fork pull requests with this configuration. To avoid disruption, switch to update_method=merge in your queue rule.

@ceph-csi-bot ceph-csi-bot force-pushed the doc-static-pvc-no-encryption-support branch from d01cd40 to 9052280 Compare April 6, 2026 07:37
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 6, 2026

Merge Queue Status

  • Entered queue2026-04-06 07:37 UTC · Rule: default
  • Checks passed · in-place
  • Merged2026-04-06 07:42 UTC · at 9052280354fd057b110e668a44d6a2b3f3e0fa31

This pull request spent 4 minutes 55 seconds in the queue, including 4 minutes 38 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Apr 6, 2026
@mergify mergify Bot merged commit 40ffe36 into ceph:devel Apr 6, 2026
19 checks passed
@mergify mergify Bot removed the queued label Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cephfs static pv doesn't support params encrypted encryptionKMSID for fscrypt,err:Failed as pool not found

3 participants