Add a script to run local docker based cluster #18425
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Labelling this PR as size/L |
k8s-github-robot
added
the
size/L
|
GCE e2e build/test failed for commit bd7cd74. |
|
@k8s-bot test this please |
|
This LGTM, but you might want to at least add a TODO to integrate/share code with: |
brendandburns
added
the
lgtm
| @@ -0,0 +1,168 @@ | |||
| #!/bin/bash | |||
|
|
|||
| # Copyright 2014 The Kubernetes Authors All rights reserved. | |||
There was a problem hiding this comment.
nit: 2015 (at least for a few more weeks)
|
GCE e2e test build/test passed for commit bd7cd74. |
|
@k8s-bot unit test this please |
|
Automatic merge from submit-queue |
k8s-github-robot
pushed a commit
that referenced
this pull request
Dec 10, 2015
Auto commit by PR queue bot
fgrzadkowski
added a commit
to fgrzadkowski/kubernetes
that referenced
this pull request
Dec 10, 2015
|
@roberthbailey I addressed your comments in #18512 |
timstclair
pushed a commit
to timstclair/kubernetes
that referenced
this pull request
Dec 10, 2015
openshift-publish-robot
pushed a commit
to openshift/kubernetes
that referenced
this pull request
Mar 23, 2018
Automatic merge from submit-queue. UPSTREAM: 59301: dockershim: don't check pod IP in StopPodSandbox We're about to tear the container down, there's no point. It also suppresses an annoying error message due to kubelet stupidity that causes multiple parallel calls to StopPodSandbox for the same sandbox. docker_sandbox.go:355] failed to read pod IP from plugin/docker: NetworkPlugin cni failed on the status hook for pod "docker-registry-1-deploy_default": Unexpected command output nsenter: cannot open /proc/22646/ns/net: No such file or directory 1) A first StopPodSandbox() request triggered by SyncLoop(PLEG) for a ContainerDied event calls into TearDownPod() and thus the network plugin. Until this completes, networkReady=true for the sandbox. 2) A second StopPodSandbox() request triggered by SyncLoop(REMOVE) calls PodSandboxStatus() and calls into the network plugin to read the IP address because networkReady=true 3) The first request exits the network plugin, sets networReady=false, and calls StopContainer() on the sandbox. This destroys the network namespace. 4) The second request finally gets around to running nsenter but the network namespace is already destroyed. It returns an error which is logged by getIP(). Fixes: openshift/origin#18414 @danwinship Origin-commit: 73e9c8fa185d0f7bb9a0798185f78598cf4bb42b
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This introduces a script to run a local docker based cluster, which is similar to what we already have for a regular cluster. Once we fix remaining issues with docker based setup I'd like to serve it from
get.k8s.io/localso that user can play with kubernetes using just one command.Output from the successful run of this script:
@brendandburns - I assigned this to you as wrote the other script for regular clusters, but feel free to reassign.
@roberthbailey @mikedanese
Ref #12722
This depends on #18351