Skip to content

Logging cluster labels#25882

Merged
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:logging-cluster-labels-2
Oct 9, 2020
Merged

Logging cluster labels#25882
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:logging-cluster-labels-2

Conversation

@mburke5678
Copy link
Contributor

@mburke5678 mburke5678 commented Sep 29, 2020

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 29, 2020
@openshift-docs-preview-bot

The preview will be available shortly at:

@mburke5678
Copy link
Contributor Author

@igor-karpukhin PTAL

@mburke5678
Copy link
Contributor Author

@sichvoge Do you this level of information is sufficient or do we need more?

@mburke5678
Copy link
Contributor Author

@igor-karpukhin PTAL

@mburke5678 mburke5678 force-pushed the logging-cluster-labels-2 branch from 8e188f8 to ff4e4ac Compare October 7, 2020 19:19
@mburke5678
Copy link
Contributor Author

@QiaolingTang Can you please review this PR for me? Thank you!

Choose a reason for hiding this comment

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

@mburke5678 we can add several labels here, so A label is not accurate.
The labels can be added like:

labels:
  name1: value1
  name2: value2
  ...

Choose a reason for hiding this comment

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

Same here

Choose a reason for hiding this comment

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

Same issue here.

Choose a reason for hiding this comment

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

Same issue here.

@mburke5678 mburke5678 force-pushed the logging-cluster-labels-2 branch from cd3be50 to a07a15c Compare October 9, 2020 16:02
Copy link
Contributor

@kalexand-rh kalexand-rh left a comment

Choose a reason for hiding this comment

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

I have a request and a suggestion. This is looking good. :)

Copy link
Contributor

Choose a reason for hiding this comment

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

I might add the annotation to this line, too, or add expand the description to acknowledge that each pipeline definition contains an optional label. (Please apply any changes that you make to this YAML to the other YAML examples that you've changed, except the last one.)

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
You can add labels to outbound log messages using `key:value` pairs in the pipeline. For example, you could add a label to messages forwarded to others data centers or label the logs by type. Labels added to objects are also forwarded with the log message.
You can add labels to outbound log messages by using `key:value` pairs in the pipeline. For example, you might add a label to messages that are forwarded to others data centers or label the logs by type. Labels that are added to objects are also forwarded with the log message.

@kalexand-rh kalexand-rh added the peer-review-done Signifies that the peer review team has reviewed this PR label Oct 9, 2020
@mburke5678 mburke5678 force-pushed the logging-cluster-labels-2 branch from 29785c6 to 28f3653 Compare October 9, 2020 20:26
@mburke5678 mburke5678 merged commit 008b881 into openshift:master Oct 9, 2020
@mburke5678 mburke5678 deleted the logging-cluster-labels-2 branch October 9, 2020 20:36
@mburke5678
Copy link
Contributor Author

mburke5678 commented Oct 9, 2020

/cherrypick enterprise-4.6

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Oct 9, 2020

@mburke5678: new pull request created: #26308

Details In response to this: >/cherrypick enterprise-4.6 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.6 peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants