-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Update documentation about alpha provisioning. #2557
Conversation
In 1.6 the alpha annotation does not have any meaning. Reflect it in portable configuration documentation.
@jsafrane If this is for 1.6, please change the base branch from |
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.
One small grammar typo
At some point, the alpha annotation will cease to have meaning, but the unset `storageClass` field on the PVC | ||
will have the desired effect. | ||
In Kubernetes 1.6 the alpha annotation ceases to have meaning, but a default storage class should be | ||
configured there to provisions all PVCs, including those with the alpha annotation. |
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.
type: "provisions" => "provision"
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.
Fixed, thanks
There is currently discussion if we're going to remove alpha dynamic provisioning in 1.6 or not. I am closing it for now and I will open a new one (against the right branch) when the decision is done and we actually need this PR. |
@jsafrane Ok, sounds good. Thx |
In 1.6 the alpha annotation does not have any meaning. Reflect it in portable
configuration documentation.
I explicitly did not update any example in this repository that still uses alpha provisioning, as it is the best portable solution for now. It should be updated few releases after 1.6 where all clusters will have a default storage class and alpha annotation won't be needed there.
This change is