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
Include list of recent builds in error message #3890
Conversation
[test] |
This is how is looks like if you have one or more builds in the current project: $ oc start-build rails-example
rails-example-12
$ oc build-logs
error: A build name is required. Most recent builds:
* rails-example-12 New 2 seconds ago
* rails-example-11 Failed 5 hours ago
* rails-example-10 Failed 5 hours ago
* rails-example-9 Failed 5 hours ago
* rails-example-8 Failed 5 hours ago
* rails-example-7 Failed 5 hours ago
* rails-example-6 Failed 5 hours ago
* rails-example-5 Failed 5 hours ago
* rails-example-4 Failed 5 hours ago
* rails-example-3 Failed 15 hours ago
see 'oc build-logs -h' for help. And in case of no builds: $ oc build-logs
error: There are no builds in the current project
see 'oc build-logs -h' for help. |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/3843/) |
Evaluated for origin up to 159c06d |
tagging do not merge as this is not a priority for 3.0.1, we'll merge it after 3.0.1 gets out the door. |
lgtm though, nice work @rhcarvalho! |
Looks great, thanks @rhcarvalho |
Evaluated for origin test up to 159c06d |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/2866/) (Image: devenv-fedora_2086) |
Evaluated for origin merge up to 159c06d |
Merged by openshift-bot
Fixes #3359
@bparees @smarterclayton PTAL