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

Change Metricbeat default config to reduce disk space #4329

Merged
merged 1 commit into from
May 29, 2017

Conversation

tsg
Copy link
Contributor

@tsg tsg commented May 16, 2017

This implements several changes from #4112:

  • reduce the frequency for the fsstats and filesystem
  • filter out some filesystems by mount point
  • set one shard by default
  • enable best_compression by default

It also removes some of the redundant comments.

Question:

  • This currently modifies the template only for metricbeat. Do we want some of these changes for other beats as well?

@tsg tsg added discuss Issue needs further discussion. in progress Pull request is currently in progress. review labels May 16, 2017
period: 1m
metricsets:
- filesystem
- fsstat
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should really rename this to filbeatsystem_summary ;-)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess you mean filesystem_summary. I agree with you.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol, yes :-)

@@ -0,0 +1,5 @@
#==================== Elasticsearch template setting ==========================
setup.template.settings:
index.number_of_shards: 1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be the damned default in ES to begin with...

@s1monw
Copy link

s1monw commented May 16, 2017

I am just giving a LGTM for the sake of the number of shards setting!

@tsg tsg force-pushed the change_metricbeat_default_config branch from f596a51 to 5421319 Compare May 26, 2017 13:58
This implements several changes from elastic#4112:

* reduce the frequency for the fsstats and filesystem
* filter out some filesystems by mount point
* set one shard by default
* enable best_compression by default
@tsg tsg force-pushed the change_metricbeat_default_config branch from 5421319 to 8f4f51f Compare May 26, 2017 14:04
@tsg tsg removed the in progress Pull request is currently in progress. label May 26, 2017
@monicasarbu monicasarbu merged commit 2f8d7bc into elastic:master May 29, 2017
@monicasarbu monicasarbu added the Metricbeat Metricbeat label May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Issue needs further discussion. Metricbeat Metricbeat review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants