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.

Sign up for GitHub

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

Jump to bottom

out_file: Avoid performing multi-worker check in configure() #3942

Merged
merged 1 commit into from Nov 2, 2022
Merged

out_file: Avoid performing multi-worker check in configure() #3942

merged 1 commit into from Nov 2, 2022

Commits on Nov 1, 2022

  1. out_file: Avoid performing multi-worker check in configure() 

    The check did not work well with `<worker 0-N>` directive for
the following reasons:

 * When Fluentd starts up, supervisor performs sanity check on
   configuration by creating actual instances of plugins.

 * This check precedes SeverModule, which prepares various assets
   for workers (lockdir, RPC endpoints and shared sockets etc.)

 * Fluent::Plugin::Base dynamically overrides `system_config.workers`
   during start-up, and leaves the value > 1 only when `<worker 0-N>`
   directive is used.

Fix this issue by avoiding faulty sanity check in `configure()`.

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
    Fujimoto Seiji committed Nov 1, 2022
    Configuration menu
    Copy the full SHA
    6263f74 View commit details
    Browse the repository at this point in the history