Skip to content
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

Storage Classes - contradicting behavior statements #47297

Closed
dmytro-storozhuk opened this issue Jul 29, 2024 · 11 comments
Closed

Storage Classes - contradicting behavior statements #47297

dmytro-storozhuk opened this issue Jul 29, 2024 · 11 comments
Labels
language/en Issues or PRs related to English language needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/storage Categorizes an issue or PR as relevant to SIG Storage.

Comments

@dmytro-storozhuk
Copy link

The default Storage class behaviour in https://kubernetes.io/docs/concepts/storage/storage-classes/#default-storageclass says:

"If you set the storageclass.kubernetes.io/is-default-class annotation to true on more than one StorageClass in your cluster, and you then create a PersistentVolumeClaim with no storageClassName set, Kubernetes uses the most recently created default StorageClass."

whereas, in the addmission controllers page (https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#defaultstorageclass) for controller that does this amendment:

"This admission controller does not do anything when no default storage class is configured. When more than one storage class is marked as default, it rejects any creation of PersistentVolumeClaim with an error and an administrator must revisit their StorageClass objects and mark only one as default. This admission controller ignores any PersistentVolumeClaim updates; it acts only on creation."

So, what is the correct one?

Thanks

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Jul 29, 2024
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

SIG Docs takes a lead on issue triage for this website, but any Kubernetes member can accept issues by applying the triage/accepted label.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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-sigs/prow repository.

@dmytro-storozhuk
Copy link
Author

and one more place https://kubernetes.io/docs/concepts/storage/dynamic-provisioning/#defaulting-behavior

"Note that if you set the storageclass.kubernetes.io/is-default-class annotation to true on more than one StorageClass in your cluster, and you then create a PersistentVolumeClaim with no storageClassName set, Kubernetes uses the most recently created default StorageClass."

@T-Lakshmi
Copy link
Contributor

/sig storage

@k8s-ci-robot k8s-ci-robot added the sig/storage Categorizes an issue or PR as relevant to SIG Storage. label Jul 29, 2024
@dmytro-storozhuk
Copy link
Author

The test on local minikube cluster (v.1.30.0) showed that PVC provisioned (BOUND state) with the latest "default" StorageClass assigned. So, the description of DefaultStorageClass is misleading.

@sftim
Copy link
Contributor

sftim commented Jul 29, 2024

Is this a duplicate of issue #42288?

@sftim
Copy link
Contributor

sftim commented Jul 29, 2024

/language en

@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Jul 29, 2024
@dmytro-storozhuk
Copy link
Author

@sftim yes, it is a duplicate. The documentation hasn't been updated for a year

@sftim
Copy link
Contributor

sftim commented Jul 29, 2024

/close not-planned

There is an existing issue; help is welcome.

@k8s-ci-robot
Copy link
Contributor

@sftim: Closing this issue, marking it as "Not Planned".

In response to this:

/close not-planned

There is an existing issue; help is welcome.

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-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 29, 2024
@iheartNathan
Copy link
Contributor

/assign

@iheartNathan
Copy link
Contributor

/unassign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
language/en Issues or PRs related to English language needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/storage Categorizes an issue or PR as relevant to SIG Storage.
Projects
None yet
Development

No branches or pull requests

5 participants