-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
migrate to localStorageCapacityIsolation kubelet option from feature gate #14728
Comments
Thanks for the heads up @BenTheElder, we'll look into this! |
Apparently it is only used as a workaround for (unsupported) btrfs if si.StorageDriver == "btrfs" {
klog.Info("auto-setting LocalStorageCapacityIsolation to false because using btrfs storage driver")
cc.KubernetesConfig.FeatureGates = addFeatureGate(cc.KubernetesConfig.FeatureGates, "LocalStorageCapacityIsolation=false")
} |
I don't know if btrfs is supported or not (the feature gate auto setting was added as part of minikube 1.25.0), but with the latest 1.27.0 release, and the switch to Kubernetes 1.25.0, the feature gate setting does not work any more, and using brtfs is broken. I tried various variations around: Something else I noticed in the logs is this:
Seems something is blocking my command line setting, but I don't have any clue about where it comes from. Conclusion is that for me, on Fedora 36, btrfs does not work anymore with minikube. As a work around, I switched docker to |
Might be related #15099. |
The LocalStorageCapacityIsolation feature will be GA in v1.25.0 and thus there will be no feature gate, but for rootless users like this I've requested a kubelet config option that will be available in v1.25.0 thanks to
@jinxu97
. You can setlocalStorageCapacityIsolation: false
in the kubelet config instead of disabling the feature gate when Kubernetes v1.25.0 is out.I've identified this project as using the feature gate, so this issue is a friendly heads up about the coming removal and replacement 😅
see: kubernetes/kubernetes#111513, kubernetes/enhancements#361, coming in Kubernetes v1.25.0
The text was updated successfully, but these errors were encountered: