search

openshift / origin-aggregated-logging

139 stars 230 forks source link

LOG-1071: CloudWatch fluentd plugin posts all logs it forwards to AWS to its own log #2159

Closed syedriko closed 3 years ago

syedriko commented 3 years ago

Description

The issue is here: https://github.com/aws/aws-sdk-ruby/blob/92338358970bc776baab082b670c7c5031842402/gems/aws-sdk-core/lib/aws-sdk-core/plugins/logging.rb#L53

This line

config.logger.send(config.log_level, format(config, response))

always logs at the current log level, so whatever it logs, always ends up in the fluentd's log. There're more details at https://issues.redhat.com/browse/LOG-1071?focusedCommentId=16502229&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16502229

/cc @igor-karpukhin /assign @jcantrill

Links

jcantrill commented 3 years ago

/approve /lgtm

openshift-ci[bot] commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcantrill, syedriko

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)~~ [jcantrill] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
syedriko commented 3 years ago

/test smoke

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

syedriko commented 3 years ago

/test cluster-logging-operator-e2e

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

jcantrill commented 3 years ago

/retest

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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

jcantrill commented 3 years ago

/retest

openshift-bot commented 3 years ago

/retest-required

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

openshift-bot commented 3 years ago

/retest-required

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