-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Skip OnRootMismatch tests if the CSI driver supports VOLUME_MOUNT_GROUP. #106395
Skip OnRootMismatch tests if the CSI driver supports VOLUME_MOUNT_GROUP. #106395
Conversation
/kind feature |
/milestone 1.23 |
@verult: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your and have them propose you as an additional delegate for this responsibility. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test pull-kubernetes-integration |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gnufied, verult The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
policy := v1.PodFSGroupChangePolicy(test.podfsGroupChangePolicy) | ||
|
||
if dInfo.Capabilities[storageframework.CapVolumeMountGroup] && | ||
utilfeature.DefaultFeatureGate.Enabled(features.DelegateFSGroupToCSIDriver) && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not think this feature gate check works. I don't think we have an api to detect which feature gates are enabled on a cluster. I think we need to leave it up to the provider to appropriately set the driver capabilities.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
uh yeah. I thought we were solely relying on CapVolumeMountGroup
, didn't notice feature gate check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, in the (common?) case of e2e binary being the same minor version as the cluster this might help. But if a lower version test binary runs against a higher version cluster, this test will not skip even if the driver has the right capability. So I'll remove this check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even if the e2e.test and the Kubernetes version match exactly you cannot use utilfeature
in the client to check for features in the server.
cc @andyzhangx |
What type of PR is this?
/kind feature
What this PR does / why we need it:
CSI drivers that support volume mount group are not required to have the capability to respect the fsgroup change policy of
OnRootMismatch
, i.e. recursively apply fsgroup only when the root directory has a different fsgroup. This PR skips those test and prevents a regression when theDelegateFSGroupToCSIDriver
feature gate is on by default in beta.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
/assign @gnufied @saikat-royc
/sig storage