Skip to content

diagnostics: minor fixes#17772

Merged
openshift-merge-robot merged 3 commits intoopenshift:masterfrom
sosiouxme:20171213-diagnostics-fixups
Dec 15, 2017
Merged

diagnostics: minor fixes#17772
openshift-merge-robot merged 3 commits intoopenshift:masterfrom
sosiouxme:20171213-diagnostics-fixups

Conversation

@sosiouxme
Copy link
Member

Just a few things I noticed needed fixing.

If --cluster-context is specified and the context is present, use it as
the cluster-admin. The logic was backward and this gave an error before.
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 14, 2017
@stevekuznetsov stevekuznetsov removed their request for review December 14, 2017 00:49
@sosiouxme
Copy link
Member Author

@pravisankar
Copy link

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 14, 2017
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pravisankar, sosiouxme

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

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 14, 2017
@pravisankar
Copy link

/retest

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 17762, 17790, 17772, 17653, 17655).

@openshift-merge-robot openshift-merge-robot merged commit 1ccf7d2 into openshift:master Dec 15, 2017
@sosiouxme sosiouxme deleted the 20171213-diagnostics-fixups branch December 20, 2017 14:38
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/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants