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.

Sign up for GitHub

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

Jump to bottom

Fix local cluster script #18046

Merged
merged 1 commit into from
Dec 8, 2015
Merged

Fix local cluster script #18046

merged 1 commit into from
Dec 8, 2015

Conversation

janetkuo
Copy link
Member

@janetkuo janetkuo commented Dec 2, 2015

After #15645 is merged, pod with ClusterDNSFirst Policy (default) cannot be created when clusterDNS is not known. This causes pods in local cluster failed to run by default.

NAME             READY     STATUS              RESTARTS   AGE
frontend-2vvak   0/1       ContainerCreating   0          6m

Events:
  FirstSeen LastSeen    Count   From            SubobjectPath   Type        Reason          Message
  ─────────   ────────    ───── ────            ───────────── ────────    ──────          ───────
  6m        6m      1   {scheduler }                Normal      Scheduled       Successfully assigned frontend-2vvak to 127.0.0.1
  6m        1s      35  {kubelet 127.0.0.1}         Warning     MissingClusterDNS   kubelet does not have ClusterDNS IP configured and cannot create Pod using "ClusterFirst" policy
  6m        1s      35  {kubelet 127.0.0.1}         Warning     FailedSync      Error syncing pod, skipping: kubelet does not have ClusterDNS IP configured and cannot create Pod using "ClusterFirst" policy. pod:"frontend-2vvak_default"

Set cluster DNS of kubelet in the script to fix it.

cc @ArtfulCoder @kubernetes/goog-cluster @kubernetes/goog-node

@k8s-github-robot
Copy link

Labelling this PR as size/XS

@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 2, 2015
@k8s-bot
Copy link

k8s-bot commented Dec 2, 2015

GCE e2e test build/test passed for commit 36067cf.

@ncdc
Copy link
Member

ncdc commented Dec 2, 2015

cc @kubernetes/rh-cluster-infra

@janetkuo janetkuo assigned ArtfulCoder and unassigned zmerlynn Dec 3, 2015
@ArtfulCoder ArtfulCoder added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 8, 2015
@k8s-github-robot
Copy link

@k8s-bot test this

Tests are more than 48 hours old. Re-running tests.

@k8s-bot
Copy link

k8s-bot commented Dec 8, 2015

GCE e2e test build/test passed for commit 36067cf.

@janetkuo
Copy link
Member Author

janetkuo commented Dec 8, 2015

@k8s-bot unit test this

@k8s-github-robot
Copy link

Automatic merge from submit-queue

k8s-github-robot pushed a commit that referenced this pull request Dec 8, 2015
Merge pull request #18046 from janetkuo/fix-local-cluster
32ac5c5 
Auto commit by PR queue bot
@k8s-github-robot k8s-github-robot merged commit 32ac5c5 into kubernetes:master Dec 8, 2015
@dcbw dcbw mentioned this pull request Jan 12, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ArtfulCoder ArtfulCoder

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@janetkuo @k8s-github-robot @k8s-bot @ncdc @k8s-teamcity-mesosphere @ArtfulCoder @zmerlynn @googlebot @wojtek-t