-
Notifications
You must be signed in to change notification settings - Fork 38.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
Disable kubelet local endpoints on windows #78704
Disable kubelet local endpoints on windows #78704
Conversation
Other than the error message consistency nit, LGTM |
/sig windows |
8964413
to
80ed28b
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dashpole, yujuhong 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 |
/lgtm |
Thanks for all the help on this! |
/retest |
/retest |
What type of PR is this?
/kind bug
/sig node
/priority critical-urgent
What this PR does / why we need it:
This PR removes the local endpoint support for windows, and allows the kubelet to start up without the endpoint on unsupported OS's. The LocalEndpoint support for windows added in previous releases did not work and did not have adequate testing.
Disabling this broken functionality was discussed and approved at sig-windows on June 4th, 2019. Notes
Which issue(s) this PR fixes:
Fixes #78628
Does this PR introduce a user-facing change?:
/assign @yujuhong @PatrickLang