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

Default config should be valid, but no-op. #10904

Open
ninaspitfire opened this issue Jun 1, 2017 · 2 comments
Open

Default config should be valid, but no-op. #10904

ninaspitfire opened this issue Jun 1, 2017 · 2 comments
Labels

Comments

@ninaspitfire
Copy link
Contributor

When a user passes the pipeline configuration via ENV (see #30), that config is merged with the default config resulting in unexpected behaviour. The default config should do nothing, successfully.

@ninaspitfire ninaspitfire self-assigned this Jun 1, 2017
@slavniyteo
Copy link

Will it be done?
Now we run elk in rancher and pass pipeline through env CONFIG_STRING. And should pass PATH_CONFIG=/dev/null to avoid config merge. It is weird.

@jordansissel
Copy link
Contributor

Maybe it's really a bug that there's any default configuration in the docker container.

Anyway, it is a feature of Logstash that the config_string (-e flag to logstash) merges with the configuration on disk.

However, in Logstash starting with 6.0 (unreleased), this merging behavior is gone:

Once 6.0 is released (though 6.0-beta2 is available),

@robbavey robbavey transferred this issue from elastic/logstash-docker Jun 26, 2019
@ninaspitfire ninaspitfire removed their assignment Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants