-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fluent Bit still buffers to the filesystem if the folder in storage.path
exists even if Emitter_Storage.type
is set to memory
#8259
Comments
@edsiper @leonardo-albertovich
We can't modify it since checking duplication. |
cio_load creates storage stream if directory exists. It prevents to create memory stream by input plugins. This patch is to recreate stream if the type is different. Signed-off-by: Takahiro Yamashita <[email protected]>
I sent a patch #8290 . |
* storage: recreate cio_stream if storage type is different(#8259) cio_load creates storage stream if directory exists. It prevents to create memory stream by input plugins. This patch is to recreate stream if the type is different. Signed-off-by: Takahiro Yamashita <[email protected]> * storage: modify log message Signed-off-by: Takahiro Yamashita <[email protected]> --------- Signed-off-by: Takahiro Yamashita <[email protected]>
fluent#8290) * storage: recreate cio_stream if storage type is different(fluent#8259) cio_load creates storage stream if directory exists. It prevents to create memory stream by input plugins. This patch is to recreate stream if the type is different. Signed-off-by: Takahiro Yamashita <[email protected]> * storage: modify log message Signed-off-by: Takahiro Yamashita <[email protected]> --------- Signed-off-by: Takahiro Yamashita <[email protected]>
fluent#8290) * storage: recreate cio_stream if storage type is different(fluent#8259) cio_load creates storage stream if directory exists. It prevents to create memory stream by input plugins. This patch is to recreate stream if the type is different. Signed-off-by: Takahiro Yamashita <[email protected]> * storage: modify log message Signed-off-by: Takahiro Yamashita <[email protected]> --------- Signed-off-by: Takahiro Yamashita <[email protected]> Signed-off-by: ahspw <[email protected]>
Bug Report
Describe the bug
If a rewrite tag filter has
Emitter_Storage.type
set tomemory
, Fluent Bit will act as ifEmitter_Storage.type
is set tofilesystem
ifstorage.path
is set in the[SERVICE]
section and the filter's folder exists.I can confirm this also happens with inputs with
Storage.type
set tomemory
.To Reproduce
Fluent Bit config
mkdir -p "$(pwd)/data/emitter.2"
data
directory:inotifywait
in Linux orfswatch
in Mac, for example.docker run --name fluentbit --rm -v $(pwd)/data:/fluent-bit/data:rw -v $(pwd)/fluentbit.conf:/config/fluentbit.conf:ro cr.fluentbit.io/fluent/fluent-bit:2.2.0 -c /config/fluentbit.conf
.Expected behavior
No files will be created under the
data/emitter.2
directory.Your Environment
Additional context
This makes having some inputs/rewrite filters use filesystem buffering while others use memory impossible.
The text was updated successfully, but these errors were encountered: