-
Notifications
You must be signed in to change notification settings - Fork 547
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
cephfs: add support for encryption in ceph-csi-cephfs chart #4504
cephfs: add support for encryption in ceph-csi-cephfs chart #4504
Conversation
a9e66f0
to
eeae5ac
Compare
@Mergifyio rebase |
the chart currently lacks access to configmap and secrets this causes the mounting of encrypted file systems to fail Signed-off-by: NymanRobin <nyman.robin@gmail.com>
✅ Branch has been successfully rebased |
eeae5ac
to
99f5101
Compare
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 5224d58 |
/test ci/centos/k8s-e2e-external-storage/1.28 |
/test ci/centos/k8s-e2e-external-storage/1.27 |
/test ci/centos/mini-e2e-helm/k8s-1.28 |
/test ci/centos/k8s-e2e-external-storage/1.29 |
/test ci/centos/mini-e2e-helm/k8s-1.27 |
/test ci/centos/mini-e2e/k8s-1.28 |
/test ci/centos/upgrade-tests-cephfs |
/test ci/centos/mini-e2e-helm/k8s-1.29 |
/test ci/centos/mini-e2e/k8s-1.27 |
/test ci/centos/upgrade-tests-rbd |
/test ci/centos/mini-e2e/k8s-1.29 |
The chart currently lacks access to configmap and secrets this causes the mounting of encrypted file systems to fail
To enable the chart to support the encrypted file systems add the following permissions:
Gives the cephfs nodeplugin clusterrole get access to configmaps and secrets.
While also adding cephfs provisioner get access to configs
Fixes issue: #4470
This still needs the kernel version 6.6 to be able to be tested, I should be able to get this tested.