search

openshift / origin-aggregated-logging

139 stars 231 forks source link

LOG-1199: Allow log level for record to be set from message #2178

Closed jcantrill closed 2 years ago

jcantrill commented 2 years ago

Description

This PR is the first part to fixing normalization of log level. It

This will additionally require a change in CLO collector config generation to consume the changes

/cc @alanconway @vimalk78

Links

jcantrill commented 2 years ago

/cherrypick release-5.2

openshift-cherrypick-robot commented 2 years ago

@jcantrill: once the present PR merges, I will cherry-pick it on top of release-5.2 in a new PR and assign it to you.

In response to [this](https://github.com/openshift/origin-aggregated-logging/pull/2178#issuecomment-916448576): >/cherrypick release-5.2 Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
jcantrill commented 2 years ago

/hold

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/hold cancel

jcantrill commented 2 years ago

/hold

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/test smoke-5-3

jcantrill commented 2 years ago

/hold cancel

jcantrill commented 2 years ago

/retest

vimalk78 commented 2 years ago

/retest

vimalk78 commented 2 years ago

/cherrypick release-5.2

@jcantrill do we need cherry-pick now? downstream all the components are using same image now

jcantrill commented 2 years ago

@vimalk78 we will not need the cherrypick but we do need to solve the consistent multi-tenant test failure (cc @igor-karpukhin @periklis )

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/test smoke-5-3

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

jcantrill commented 2 years ago

/retest

vparfonov commented 2 years ago

/lgtm

openshift-ci[bot] commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alanconway, jcantrill, vparfonov

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: - ~~[OWNERS](https://github.com/openshift/origin-aggregated-logging/blob/master/OWNERS)~~ [alanconway,jcantrill] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
openshift-bot commented 2 years ago

/retest-required

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

openshift-bot commented 2 years ago

/retest-required

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

openshift-bot commented 2 years ago

/retest-required

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

openshift-bot commented 2 years ago

/retest-required

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

openshift-bot commented 2 years ago

/retest-required

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

openshift-ci[bot] commented 2 years ago

@jcantrill: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/cluster-logging-operator-e2e a2a248c0ea737e16499728bd6f7284f12e110c04 link /test cluster-logging-operator-e2e

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
openshift-bot commented 2 years ago

/retest-required

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

openshift-cherrypick-robot commented 2 years ago

@jcantrill: new pull request created: #2187

In response to [this](https://github.com/openshift/origin-aggregated-logging/pull/2178#issuecomment-916448576): >/cherrypick release-5.2 Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.