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

update hardcoded "oc" cmd suggestion in cmd output #10112

Merged

Conversation

juanvallejo
Copy link
Contributor

@juanvallejo juanvallejo commented Jul 29, 2016

oc status command output contains suggestions for oc commands with oc
hardcoded rather than using the current root command (e.g. openshift cli status or atomic-enterprise cli status)

Related BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1256274#c1 and https://bugzilla.redhat.com/show_bug.cgi?id=1256274#c3

@juanvallejo
Copy link
Contributor Author

[test]

@juanvallejo
Copy link
Contributor Author

@fabianofranz Could you take a look please?

@juanvallejo
Copy link
Contributor Author

flake re[test]

@juanvallejo
Copy link
Contributor Author

re[test]

`oc` status output contains suggestions for "oc" commands with "oc"
hardcoded, rather than being dynamic to the current root cmd being used
@juanvallejo juanvallejo force-pushed the jvallejo_update-hardcoded-oc-cmd branch from 76123ac to bcf8d9b Compare July 29, 2016 21:40
@juanvallejo
Copy link
Contributor Author

flake re[test]

@stevekuznetsov
Copy link
Contributor

re[test]

@juanvallejo
Copy link
Contributor Author

@openshift-bot, the last build failed from the following flakes:

re[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to bcf8d9b

@openshift-bot
Copy link
Contributor

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

@fabianofranz
Copy link
Member

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 2, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to bcf8d9b

@openshift-bot openshift-bot merged commit 2dee2ff into openshift:master Aug 2, 2016
@juanvallejo juanvallejo deleted the jvallejo_update-hardcoded-oc-cmd branch August 2, 2016 13:27
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