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

PoC v1parser: support built-in config files #4893

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Apr 4, 2025

Which issue(s) this PR fixes:

This is just a PoC.

What this PR does / why we need it:

Currently @include directive supports to reuse configuration files, but no way to
apply specific configuration files by default (without specifying to).

For example, if this feature was merged, a specific configuration could be applied in any case for fluent-package.
By putting some conf files under /etc/fluent/conf.d/*.conf.

Docs Changes:

T.B.D.

Release Note:

@kenhys kenhys changed the title v1parser: support built-in config files PoC v1parser: support built-in config files Apr 4, 2025
@kenhys
Copy link
Contributor Author

kenhys commented Apr 4, 2025

This is just a PoC.

In the previous versions, if you want to manage multiple
configuration files, you must use @include directive.

In this commit, add an option to specify the directory
(e.g. /etc/fluent/conf.d) which stores additional configuration files.
If there are such files under specified directory, they are loaded by
default without @include directive.

If you want to disable this feature, set empty string
for config_include_dir "".

Signed-off-by: Kentaro Hayashi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant