Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fluentd config: turn off S3 bucket checks #1999

Merged
merged 1 commit into from
Apr 12, 2019
Merged

Conversation

christianberg
Copy link
Collaborator

Turn off the feature to create the S3 bucket and to check for S3
bucket existence at startup. The logging-agent doesn't have permission
to create S3 buckets, and the check causes the Fluentd worker to fail
and restart if the bucket is not present. Constant restarting leads to
high CPU consumption.

Instead we choose to keep the worker running and try to ship logs to
S3. This will fail if the bucket does not exist, but the buffer size
is limited and we can alert on buffer size and errors and the CPU
consumption stays low.

Turn off the feature to create the S3 bucket and to check for S3
bucket existance at startup. The logging-agent doesn't have permission
to create S3 buckets, and the check causes the Fluentd worker to fail
and restart if the bucket is not present. Constant restarting leads to
high CPU consumption.

Instead we choos to keep the worker running and try to ship logs to
S3. This will fail if the bucket does not exist, but the buffer size
is limited and we can alert on buffer size and errors and the CPU
consumption stays low.

Signed-off-by: Christian Berg <[email protected]>
@arjunrn
Copy link
Contributor

arjunrn commented Apr 12, 2019

👍

1 similar comment
@mikkeloscar
Copy link
Contributor

👍

@mikkeloscar mikkeloscar merged commit 4fc99d1 into dev Apr 12, 2019
@mikkeloscar mikkeloscar deleted the fluentd-bucket-options branch April 12, 2019 10:15
This was referenced Apr 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants