Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

elk: increase default resources #1569

Merged
merged 1 commit into from
Jun 21, 2016
Merged

elk: increase default resources #1569

merged 1 commit into from
Jun 21, 2016

Conversation

ryane
Copy link
Contributor

@ryane ryane commented Jun 17, 2016

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

Increase the resources allocated for elasticsearch and kibana with the default configuration. Same testing process as #1481 but with a cluster that has sufficient resources.

I'm not sure what the correct balance is between providing a default configuration that doesn't require a huge investment in compute resources vs a configuration that might be appropriate for real workloads. Feedback welcome.

@ryane ryane added this to the 1.2 milestone Jun 17, 2016
@kbroughton
Copy link
Contributor

I think the most important thing is to Cull the indexes. Perhaps a small
elasticsearch curator script to delete any logstash indices over three days
old. I would vote for moderate Resources with aggressive pruning.
On Jun 17, 2016 4:51 PM, "Ryan Eschinger" notifications@github.com wrote:

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

Increase the resources allocated for elasticsearch and kibana with the
default configuration. Same testing process as #1481
#1481 but with a cluster that
has sufficient resources.

I'm not sure what the correct balance is between providing a default
configuration that doesn't require a huge investment in compute resources
vs a configuration that might be appropriate for real workloads. Feedback

welcome.

You can view, comment on, or merge this pull request online at:

#1569
Commit Summary

  • elk: increase default resources

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#1569, or mute the thread
https://github.com/notifications/unsubscribe/AKErsrB8zjWKMwkAmXQvGci3WsyI_5viks5qMzL9gaJpZM4I40MO
.

@sehqlr
Copy link
Contributor

sehqlr commented Jun 21, 2016

OK, I finally got all of the tests done. This is a go!

@sehqlr sehqlr merged commit 54a5181 into master Jun 21, 2016
@ryane ryane deleted the fix/elk-sizing branch June 21, 2016 20:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants