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

Added kibana_index_replicas property to kibana.yml. #3878

Closed
wants to merge 3 commits into from

Conversation

dgrabows
Copy link
Contributor

Also added supporting code and documentation to provide a default value (1) for the property, make the value accessible to the browser app, and to use the setting when creating a new Kibana index in Elasticsearch.

There is probably an opportunity to add some test cases for these changes. I'm not very familiar with javascript testing frameworks, and I didn't see examples I could base my tests off of for the files being changed. Suggestions for how to automate testing of these changes are welcome, with pointers to examples of testing similar code greatly appreciated. I did manually test these changes with values of 0 and 1 for kibana_index_replicas in kibana.yml.

This is intended to resolve #3877.

dgrabows added 3 commits May 19, 2015 01:31
…he number of replicas for the index used to store Kibana dashboards, settings, etc. in Elasticsearch. Resolves elastic#3877.
…railing spaces stripped by my editor that obscured the meaningful change.
@dgrabows dgrabows changed the title Added kibana_index_replicas property to kibana.yml. Resolves #3877. Added kibana_index_replicas property to kibana.yml. May 19, 2015
@rashidkpc
Copy link
Contributor

Instead of going this route we're simply going to remove the explicit setting of replicas: 1, which will allow you to accomplish this with an elasticsearch template. There's really no good reason for Kibana to set the number of replicas

FWIW, we only set it to 1 on creation, it can be turned down immediately after creation.

@rashidkpc
Copy link
Contributor

Closed per discussion on #3877

@rashidkpc rashidkpc closed this May 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove hardcoded index.number_of_replicas: 1
2 participants