Skip to content
This repository has been archived by the owner on Feb 5, 2021. It is now read-only.

Feature request: Use file-based output buffers #93

Open
NfNitLoop opened this issue Nov 1, 2018 · 1 comment
Open

Feature request: Use file-based output buffers #93

NfNitLoop opened this issue Nov 1, 2018 · 1 comment

Comments

@NfNitLoop
Copy link

Background

We noticed that some log messages were never delivered to SumoLogic. Those same log messages did end up in StackDriver, despite similar log collection setups. (k8s -> fluentd daemonset -> log sink.)

The core issue was that our sumologic-fluentd pods had a memory leak which caused the pods to be SIGKILLed by kubernetes when they exceeded their memory limits.

But, all messages currently in the memory-backed buffer were lost during the restart, and were never delivered.

Feature Request

It would be nice if the default settings would set up a file-based output buffer so that delivery of messages in the output buffer during a crash could be resumed once the pod is restarted.

For example, the GKE-provided fluentd pod sets up a file-based buffer like this.

This would help not only in the case of a crash, but in any other circumstances where k8s might kill or restart the pod.

@frankreno
Copy link
Contributor

This is a good idea, we do default to memory. Let me take a look at this over the coming weeks as I get some more bandwidth.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants