-
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
Deprecate cloud-provider/cloud-config in apiserver CLI #120903
Deprecate cloud-provider/cloud-config in apiserver CLI #120903
Conversation
/sig api-machinery |
Signed-off-by: Davanum Srinivas <[email protected]>
/triage accepted |
e598c7a
to
4d2d994
Compare
@liggitt here's proof that kube-apiserver does not need either The situation mentioned in #49402 w.r.t the current plan is for https://github.com/kubernetes-sigs/cloud-pv-admission-labeler to take over from that admission controller. Note that PersistentVolumeLabel admission controller is already marked as deprecated as well. |
/lgtm worth noting for anyone wanting to copy this for other |
LGTM label has been added. Git tree hash: a3e822cc694202b9d5110e51510b8b76bc9d228e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, liggitt 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 |
Changelog suggestion Deprecated the `--cloud-provider` and `--cloud-config` CLI parameters in kube-apiserver.
These parameters will be removed in a future release. |
Updated with your suggestion @sftim thanks! |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Related to #49402 not sure why we missed deprecating it a while ago!
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.: