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

kubectl run --restart=Never creates pods #25253

Merged
merged 1 commit into from
May 31, 2016

Conversation

soltysh
Copy link
Contributor

@soltysh soltysh commented May 6, 2016

Fixes #24533.

@bgrant0607 @janetkuo ptal
/fyi @thockin

* kubectl run --restart=Never creates pods

Analytics

@k8s-github-robot k8s-github-robot added kind/old-docs size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 6, 2016
@soltysh
Copy link
Contributor Author

soltysh commented May 6, 2016

Silly me forgot about updating e2e tests, should be ok.

@k8s-github-robot k8s-github-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 6, 2016
@bgrant0607 bgrant0607 assigned janetkuo and unassigned thockin May 6, 2016
@soltysh
Copy link
Contributor Author

soltysh commented May 6, 2016

@k8s-bot test this github flake: #24211

@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 7, 2016
@k8s-github-robot k8s-github-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 9, 2016
@soltysh
Copy link
Contributor Author

soltysh commented May 9, 2016

Rebased, @janetkuo ptal.

@janetkuo janetkuo added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-label-needed labels May 9, 2016
@@ -25,7 +25,7 @@ options:
If true, a public, external service is created for the container(s) which are run
- name: generator
usage: |
The name of the API generator to use. Default is 'deployment/v1beta1' if --restart=Always, otherwise the default is 'job/v1'. This will happen only for cluster version at least 1.2, for olders we will fallback to 'run/v1' for --restart=Always, 'run-pod/v1' for others.
The name of the API generator to use. Default is 'deployment/v1beta1' if --restart=Always, 'job/v1' for OnFailure and 'pod/v1' for Never. This will happen only for cluster version at least 1.2, for olders we will fallback to 'run/v1' for --restart=Always, 'run-pod/v1' for others.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, but we don't support "restart=Never creates pods" in 1.2 (we're adding this in 1.3). I'd however love to keep this succinct...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same for .md

@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 10, 2016
@bgrant0607
Copy link
Member

@soltysh @janetkuo I'd like to wrap this up and get it merged.

@k8s-github-robot k8s-github-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 27, 2016
@soltysh
Copy link
Contributor Author

soltysh commented May 27, 2016

@bgrant0607 @janetkuo rebase and updated except for the new tests in e2e. ptal

@k8s-bot
Copy link

k8s-bot commented May 28, 2016

GCE e2e build/test passed for commit d76fa8a.

@janetkuo janetkuo added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 31, 2016
@janetkuo janetkuo added this to the v1.3 milestone May 31, 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 May 31, 2016

GCE e2e build/test passed for commit d76fa8a.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit d957e78 into kubernetes:master May 31, 2016
@soltysh soltysh deleted the issue24533 branch May 31, 2016 21:25
k8s-github-robot pushed a commit that referenced this pull request Aug 21, 2016
Automatic merge from submit-queue

Return container command exit codes in kubectl run/exec

Fixes #26424
Based on #25273.

TODO:
- [x] add e2e tests
- [x] investigate `kubectl run` exit code for `--restart=Never` (compare issue #24533 and PR #25253)
- [x] document exit codes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. 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.

7 participants