-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Include CSIDriver capability to indicate fsGroup support #92001
Include CSIDriver capability to indicate fsGroup support #92001
Conversation
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
1a0fb89
to
4a08142
Compare
/remove-sig api-machinery |
4a08142
to
abf5a4c
Compare
daee349
to
46b925c
Compare
/lgtm |
/retest Review the full test history for this PR. Silence the bot with an |
2 similar comments
/retest Review the full test history for this PR. Silence the bot with an |
/retest Review the full test history for this PR. Silence the bot with an |
/test pull-kubernetes-verify |
/test pull-kubernetes-e2e-gce |
/retest |
/test pull-kubernetes-e2e-gce |
/retest |
/retest Review the full test history for this PR. Silence the bot with an |
2 similar comments
/retest Review the full test history for this PR. Silence the bot with an |
/retest Review the full test history for this PR. Silence the bot with an |
/retest |
5 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest Review the full test history for this PR. Silence the bot with an |
1 similar comment
/retest Review the full test history for this PR. Silence the bot with an |
/test pull-kubernetes-integration |
/test pull-kubernetes-e2e-gce-ubuntu-containerd |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR is the followup to https://github.com/kubernetes/enhancements/tree/master/keps/sig-storage/1682-csi-driver-skip-permission , and introduces the
SupportsFsGroup
field for CSIDrivers. This allows CSIDrivers to declare if they wish to allow modifying the fsGroup when volumes are mounted.Which issue(s) this PR fixes:
Fixes kubernetes/enhancements#1682
Special notes for your reviewer:
I've currently added the field in the v1 and v1beta1 staging APIs; however, this should be an alpha field on the CSIDriver.Spec object. I noticed another field (InlineVolumeSpec) that includes a comment mentioning it's alpha-level.
What's the correct way to include this?
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: