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

[kubelet] Allow opting out of automatic cloud provider detection in kubelet. By default kubelet will auto-detect cloud providers #28258

Merged
merged 1 commit into from
Jun 30, 2016

Conversation

vishh
Copy link
Contributor

@vishh vishh commented Jun 30, 2016

fixes #28231

Signed-off-by: Vishnu Kannan <vishnuk@google.com>
Signed-off-by: Vishnu kannan <vishnuk@google.com>
@vishh vishh added the release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action. label Jun 30, 2016
@vishh
Copy link
Contributor Author

vishh commented Jun 30, 2016

@dchen1107 Kindly advice if this PR is worth a cherry-pick

@k8s-github-robot k8s-github-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 30, 2016
@k8s-bot
Copy link

k8s-bot commented Jun 30, 2016

GCE e2e build/test passed for commit ea789e8.

@enoodle
Copy link

enoodle commented Jun 30, 2016

Thanks @vishh 👍

@dchen1107 dchen1107 assigned dchen1107 and unassigned yujuhong Jun 30, 2016
@dchen1107
Copy link
Member

This is a behavior regression from 1.2 release, and do break the private cloud user's the cloud configuration without any workaround. We should cherry-pick the fix. Thanks!

@dchen1107 dchen1107 added this to the v1.3 milestone Jun 30, 2016
@dchen1107 dchen1107 added the priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. label Jun 30, 2016
@dchen1107
Copy link
Member

LGTM

@dchen1107 dchen1107 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 30, 2016
@dchen1107
Copy link
Member

cc/ @erictune

@erictune erictune added cherrypick-candidate cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. labels Jun 30, 2016
@erictune erictune mentioned this pull request Jun 30, 2016
@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-bot
Copy link

k8s-bot commented Jun 30, 2016

GCE e2e build/test passed for commit ea789e8.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit ab37fbf into kubernetes:master Jun 30, 2016
eparis pushed a commit to eparis/kubernetes that referenced this pull request Jun 30, 2016
Automatic merge from submit-queue

[kubelet] Allow opting out of automatic cloud provider detection in kubelet. By default kubelet will auto-detect cloud providers

fixes kubernetes#28231
(cherry picked from commit ab37fbf)
@k8s-cherrypick-bot
Copy link

Commit found in the "release-1.3" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked.

@erictune
Copy link
Member

erictune commented Jul 2, 2016

@vishh Does this require action by the user when upgrading to 1.3.0? If so, please edit your first comment to have a release-note block, like in #28132. If not, please change to release-note-none label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In 1.3, kubelet cloud-provider setting is not respected when --cloud-provider=""
9 participants