Skip to content

get must-gather image from imagestream#22528

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
sanchezl:oc_admin_must-gather_image
Apr 22, 2019
Merged

get must-gather image from imagestream#22528
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
sanchezl:oc_admin_must-gather_image

Conversation

@sanchezl
Copy link
Copy Markdown
Contributor

@sanchezl sanchezl commented Apr 10, 2019

User the must-gather ImageStream in the openshift namespace to get the image for the must-gather tool.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 10, 2019
@sanchezl sanchezl force-pushed the oc_admin_must-gather_image branch 2 times, most recently from 11277f2 to 6eeade6 Compare April 11, 2019 05:24
@sanchezl sanchezl mentioned this pull request Apr 11, 2019
8 tasks
@sanchezl sanchezl force-pushed the oc_admin_must-gather_image branch from 6eeade6 to 9a7df80 Compare April 11, 2019 05:57
@sanchezl sanchezl force-pushed the oc_admin_must-gather_image branch from 9a7df80 to 774c412 Compare April 12, 2019 02:50
Copy link
Copy Markdown
Contributor

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

The 2nd commit lgtm:
/lgtm
/approve
/hold
for #22430 to merge first and have comments addressed there

@openshift-ci-robot openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Apr 12, 2019
@sanchezl sanchezl force-pushed the oc_admin_must-gather_image branch from 774c412 to 6d1dc62 Compare April 12, 2019 19:57
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Apr 12, 2019
@sanchezl
Copy link
Copy Markdown
Contributor Author

/test unit

@sanchezl
Copy link
Copy Markdown
Contributor Author

/retest

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Apr 15, 2019

#22430 merged.

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 15, 2019
"k8s.io/kubernetes/pkg/kubectl/util/templates"

"github.com/openshift/client-go/image/clientset/versioned/typed/image/v1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit, no space

@deads2k
Copy link
Copy Markdown
Contributor

deads2k commented Apr 16, 2019

@sanchezl needs an e2e test before we can merge this

@deads2k
Copy link
Copy Markdown
Contributor

deads2k commented Apr 22, 2019

now, let's see if this works

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2019
@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, sanchezl, soltysh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit b9c84a0 into openshift:master Apr 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants