Skip to content

Conversation

@rhamilto
Copy link
Member

@rhamilto rhamilto commented Feb 6, 2020

…rectly

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1760767

Discussed this with @spadgett and determined the safest and best fix is to simply display the image name as is and don't try to do clever display of the data.

Screen Shot 2020-02-06 at 2 39 49 PM

@openshift-ci-robot
Copy link

@rhamilto: This pull request references Bugzilla bug 1760767, which is invalid:

  • expected Bugzilla bug 1760767 to depend on a bug in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Bug 1760767: fix bug where container image name does not display cor…

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 6, 2020
@rhamilto
Copy link
Member Author

rhamilto commented Feb 6, 2020

/bugzilla refresh

@openshift-ci-robot
Copy link

@rhamilto: This pull request references Bugzilla bug 1760767, which is invalid:

  • expected Bugzilla bug 1760767 to depend on a bug in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

/bugzilla refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhamilto
Copy link
Member Author

rhamilto commented Feb 6, 2020

@benjaminapetersen, not sure how to get around the bugzilla dependency requirement given this bug doesn't exist in 4.x.

@openshift-ci-robot
Copy link

@rhamilto: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/openshift-jenkins 1762f7a link /test

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@spadgett
Copy link
Member

spadgett commented Feb 7, 2020

/override ci/openshift-jenkins

Manually adding the valid bug label. The 4.x console is a different code base that doesn't have this bug.

@spadgett spadgett added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Feb 7, 2020
@openshift-ci-robot
Copy link

@spadgett: Overrode contexts on behalf of spadgett: ci/openshift-jenkins

In response to this:

/override ci/openshift-jenkins

Manually adding the valid bug label. The 4.x console is a different code base that doesn't have this bug.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2020
@openshift-merge-robot openshift-merge-robot merged commit 27f7900 into openshift:enterprise-3.11 Feb 7, 2020
@openshift-ci-robot
Copy link

@rhamilto: All pull requests linked via external trackers have merged. Bugzilla bug 1760767 has been moved to the MODIFIED state.

In response to this:

Bug 1760767: fix bug where container image name does not display cor…

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhamilto rhamilto deleted the bz1760767 branch February 7, 2020 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants