Skip to content

Add resourceNamespaceUID variable replacement to ConsoleExternalLogLink#466

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
spadgett:log-link-namespace-uid
Oct 11, 2019
Merged

Add resourceNamespaceUID variable replacement to ConsoleExternalLogLink#466
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
spadgett:log-link-namespace-uid

Conversation

@spadgett
Copy link
Copy Markdown
Member

Console PR: openshift/console#2961

This is needed to add a Kibana link to console as they require the project UID.

/assign @bparees @benjaminapetersen
cc @jcantrill

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 11, 2019
@benjaminapetersen
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 11, 2019
@spadgett
Copy link
Copy Markdown
Member Author

This doesn't change the properties on the CRD, only updates the comment to document a new variable replacement for hrefTemplate

@bparees
Copy link
Copy Markdown
Contributor

bparees commented Oct 11, 2019

/approve

@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: benjaminapetersen, bparees, spadgett

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-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 11, 2019
@openshift-merge-robot openshift-merge-robot merged commit b3cee43 into openshift:master Oct 11, 2019
@spadgett spadgett deleted the log-link-namespace-uid branch October 11, 2019 19:59
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/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