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
suggest exposable resources oc expose #10843
suggest exposable resources oc expose #10843
Conversation
Looks good, but I think it's still valid to print the accepted extensions. E.g.
|
bda0645
to
3bba62b
Compare
Done! |
[test] |
Related Trello Card: https://trello.com/c/04lpfTQR/451-8-cli-improved-flows-and-ux-in-the-cli-evg-ux-p3 This patch updates `oc expose` output (with no resources provided) to a `kcmdutil.UsageError` so that a suggestion to use `oc expose -h` is displayed.
3bba62b
to
095dc9c
Compare
conformance test flaked on #9548 re[test] |
[merge] |
conformance test flaked on #9548 re[test] |
conformance test flaked on #9548 re[test] |
Evaluated for origin test up to 095dc9c |
re[merge] |
Evaluated for origin merge up to 095dc9c |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9302/) |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9318/) (Image: devenv-rhel7_5079) |
Related Trello Card:
https://trello.com/c/04lpfTQR/451-8-cli-improved-flows-and-ux-in-the-cli-evg-ux-p3
This patch updates
oc expose
output (with no resources provided)to a
kcmdutil.UsageError
so that a suggestion to useoc expose -h
is displayed.
Before
$ oc expose
After
cc @openshift/cli-review