Skip to content

oc status commit message on builds destroys layout #13841

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

Closed
smarterclayton opened this issue Apr 20, 2017 · 1 comment
Closed

oc status commit message on builds destroys layout #13841

smarterclayton opened this issue Apr 20, 2017 · 1 comment
Assignees
Labels
area/usability component/build component/cli help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. priority/P2

Comments

@smarterclayton
Copy link
Contributor

oc status
In project OpenShift CI (ci) on server https://api.ci.openshift.org:443

bc/oauth-proxy docker builds Dockerfile on docker.io/library/busybox:latest
  -> istag/oauth-proxy:latest
  build #9 succeeded 5 seconds ago

bc/oauth-proxy-binary docker builds https://github.com/openshift/oauth-proxy.git on docker.io/openshift/origin-release:golang-1.7
  -> istag/oauth-proxy:binaries
  build #9 succeeded 21 seconds ago - 423edf1: Add -client-secret-file and -cookie-secret-file (Clayton Coleman <[email protected]>)
  build #3 failed 5 days ago - 2ce1a72: Add an OpenShift auth provider (Clayton Coleman <[email protected]>)

bc/prometheus docker builds Dockerfile on docker.io/library/busybox:latest
  -> istag/prometheus:latest
  build #11 succeeded 3 days ago
  build #4 failed 5 days ago

bc/prometheus-binary docker builds https://github.com/openshift/prometheus.git on docker.io/openshift/origin-release:golang-1.7
  -> istag/prometheus:binaries
  build #9 succeeded 3 days ago - 9fbeefc: UPSTREAM: Allow bearer token and basic auth to be discovered

In order to automate simple discovery in a tenanted system, add support
for `__bearer_token__', `__basic_auth_username__`, and
`__basic_auth_password__` to be set during relabel and applied to the
client config. (Clayton Coleman <[email protected]>)
  build #4 failed 5 days ago - bd1182d: Merge pull request #2415 from prometheus/beorn7/release (Björn Rabenstein <[email protected]>)

View details with 'oc describe <resource>/<name>' or list everything with 'oc get all'.

The full commit message is probably not useful, or at minimum should be aligned. I'd be inclined to truncate on the first completed line.

@fabianofranz fabianofranz added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Apr 24, 2017
@matthyx
Copy link
Contributor

matthyx commented Jul 25, 2017

This comes from describeSourceRevision in:
pkg/cmd/cli/describe/projectstatus.go

I will prepare a PR which truncates rev.Git.Message on the first line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/usability component/build component/cli help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. priority/P2
Projects
None yet
Development

No branches or pull requests

4 participants