-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Conversation
Silly me forgot about updating e2e tests, should be ok. |
Rebased, @janetkuo ptal. |
@@ -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. |
There was a problem hiding this comment.
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...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same for .md
@bgrant0607 @janetkuo rebase and updated except for the new tests in e2e. ptal |
GCE e2e build/test passed for commit d76fa8a. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit d76fa8a. |
Automatic merge from submit-queue |
Fixes #24533.
@bgrant0607 @janetkuo ptal
/fyi @thockin