Skip to content

Conversation

@igor-karpukhin
Copy link
Contributor

Description

CLO will try to autodetect cluster name for the CloudWatch log group prefix

/cc @jcantrill
/assign @jcantrill @vimalk78

Links

@igor-karpukhin igor-karpukhin force-pushed the feature/LOG-1108-cw-cluster-name branch from 929b018 to d4ece78 Compare February 22, 2021 20:00
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like the validations are still missing from the CRD. make check maybe?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry. Fixed.

Copy link
Contributor

@jcantrill jcantrill left a comment

Choose a reason for hiding this comment

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

/approve
/hold

@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. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 23, 2021
@jcantrill
Copy link
Contributor

/retest

@igor-karpukhin igor-karpukhin force-pushed the feature/LOG-1108-cw-cluster-name branch from d4ece78 to abb843e Compare February 24, 2021 14:40
@jcantrill
Copy link
Contributor

/hold cancel
/lgtm

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Feb 24, 2021
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: igor-karpukhin, jcantrill

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

The pull request process is described here

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-merge-robot openshift-merge-robot merged commit a6480b7 into openshift:tech-preview Feb 24, 2021
pmoogi-redhat pushed a commit to pmoogi-redhat/cluster-logging-operator that referenced this pull request Apr 26, 2022
…-cw-cluster-name

LOG-1108: ClusterName for CloudWatch feature
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants