Skip to content

system_config: support built-in config files #4893

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

Open
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:

fluent/fluentd-docs-gitbook#577

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.

@kenhys kenhys force-pushed the builtin-confd branch 6 times, most recently from 238d9dc to 49f544d Compare April 10, 2025 05:30
@kenhys kenhys changed the title PoC v1parser: support built-in config files PoC system_config: support built-in config files Apr 10, 2025
@kenhys kenhys force-pushed the builtin-confd branch 9 times, most recently from b1329d3 to aa12263 Compare April 14, 2025 06:19
@kenhys kenhys changed the title PoC system_config: support built-in config files system_config: support built-in config files Apr 14, 2025
@kenhys kenhys marked this pull request as ready for review April 14, 2025 06:20
@kenhys kenhys force-pushed the builtin-confd branch 2 times, most recently from a891439 to 0bd54d9 Compare April 15, 2025 01:55
Currently @include directive supports to reuse configuration files,
but no way to apply specific configuration files by default (without
specifying to).

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

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]>
kenhys added a commit to kenhys/fluentd-docs-gitbook that referenced this pull request Apr 15, 2025
@kenhys kenhys requested review from Watson1978 and daipom April 15, 2025 04:31
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