Skip to content

Conversation

spadgett
Copy link
Member

Frequently the cause of the failure is in the events, not the deployment logs. Add a link so that users are more likely to check there.

openshift_web_console

Related to https://bugzilla.redhat.com/show_bug.cgi?id=1365525

Frequently the cause of the failure is in the events, not the deployment
logs. Add a link so that users are more likely to check there.
@jwforres
Copy link
Member

LGTM for 1.5

On a related note, I feel like on an RC we should be showing events for the pods for that RC, or for a DC we should be showing events for the RCs and Pods underneath it. We already started doing something similar on the monitoring view, if you expand a deployment it highlights both the RCs events and the pod events.

@jwforres jwforres added the lgtm Indicates that a PR is ready to be merged. label Nov 14, 2016
@jwforres
Copy link
Member

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to d1586a6

@openshift-bot
Copy link

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link

Evaluated for origin web console test up to d1586a6

@openshift-bot
Copy link

Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/697/) (Base Commit: d0a5912)

@openshift-bot
Copy link

openshift-bot commented Nov 17, 2016

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/703/) (Base Commit: 9b21516)

@openshift-bot openshift-bot merged commit f839c85 into openshift:master Nov 17, 2016
@spadgett spadgett deleted the failed-deployment-view-events-link branch January 9, 2017 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants