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

add --timeout flag to oc rsh #10745

Conversation

juanvallejo
Copy link
Contributor

@juanvallejo juanvallejo commented Aug 31, 2016

oc rsh currently has a timeout of 10 seconds hardcoded for pod
requests made to the server. This patch adds a flag --timeout to make
this time user-configurable

cc @fabianofranz

@juanvallejo
Copy link
Contributor Author

[test]

@fabianofranz
Copy link
Member

Needs to generate docs and completions. @liggitt PTAL

@juanvallejo juanvallejo force-pushed the jvallejo_make-timeout-configurable-oc-rsh branch 2 times, most recently from 09e2bb1 to bc6bb75 Compare August 31, 2016 21:12
@juanvallejo
Copy link
Contributor Author

re[test]

@juanvallejo juanvallejo force-pushed the jvallejo_make-timeout-configurable-oc-rsh branch from bc6bb75 to 8d554f4 Compare September 7, 2016 14:05
@0xmichalis
Copy link
Contributor

LGTM

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 16, 2016
@juanvallejo
Copy link
Contributor Author

@fabianofranz PTAL

`oc rsh` currently has a timeout of 10 seconds hardcoded for pod
requests made to the server. This patch adds a flag `--timeout` to make
this time user-configurable
@juanvallejo juanvallejo force-pushed the jvallejo_make-timeout-configurable-oc-rsh branch from 8d554f4 to 9ee734e Compare September 21, 2016 20:25
@fabianofranz
Copy link
Member

[merge]

@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 21, 2016
@openshift-bot
Copy link
Contributor

openshift-bot commented Sep 23, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9286/) (Image: devenv-rhel7_5076)

@juanvallejo
Copy link
Contributor Author

integration test flaked on #9203
conformance test flaked on #9548
re[test]

@juanvallejo
Copy link
Contributor Author

conformance test flaked on #9548 re[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 9ee734e

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9286/)

@fabianofranz
Copy link
Member

re[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 9ee734e

@openshift-bot openshift-bot merged commit 468414b into openshift:master Sep 26, 2016
@juanvallejo juanvallejo deleted the jvallejo_make-timeout-configurable-oc-rsh branch September 26, 2016 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants