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

Set short_config option to false by default in Metricbeat #4038

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Apr 18, 2017

So far for each new module the configs were added to the short config by default. But by default we didnt want to have it there. So not every module has to add short_config: false to each module, short_config is now set to False by default. This will not have any affect except for new modules.

  • System module short_config was set to true
  • Redis short_config entry was removed as an example
  • Update filebeat modules as all modules are in short_config at the moment
  • Move config_collector.py to global script directory so all beats can use it

@ruflin ruflin force-pushed the make-shortconfig-false branch 2 times, most recently from e12fe72 to 519e242 Compare April 18, 2017 12:45
So far for each new module the configs were added to the short config by default. But by default we didnt want to have it there. So not every module has to add `short_config: false` to each module, short_config is now set to False by default. This will not have any affect except for new modules.

* System module short_config was set to true
* Redis short_config entry was removed as an example
* Update filebeat modules as all modules are in short_config at the moment
* Move config_collector.py to global script directory so all beats can use it
@ruflin ruflin force-pushed the make-shortconfig-false branch from 519e242 to 38f9e94 Compare April 19, 2017 06:44
@tsg tsg merged commit e288cf6 into elastic:master Apr 19, 2017
@ruflin ruflin deleted the make-shortconfig-false branch April 19, 2017 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants