Skip to content

More flexible ElasticSearch template #10

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dbackeus
Copy link
Contributor

@dbackeus dbackeus commented Nov 4, 2024

When just having a single nodeSet with count for replication, we can't control which pod gets decommissioned when scaling down.

Using named replicas we can achieve better control for these kinds of use-cases.

@dbackeus dbackeus changed the title New approach for ElasticSearch template More flexible approach for ElasticSearch template Nov 4, 2024
@dbackeus dbackeus changed the title More flexible approach for ElasticSearch template More flexible ElasticSearch template Nov 4, 2024
@dbackeus dbackeus force-pushed the alternative-elasticsearch branch 3 times, most recently from 1b02f80 to 1a4f0ef Compare November 5, 2024 13:33
@@ -37,8 +40,9 @@ spec:
elasticsearchRefs:
- name: {{ .Chart.Name }}%{suffix}
nodeSets:
- name: default
count: {{ .Values.resources.%{camelName}.replicas }}

This comment was marked as resolved.

@dbackeus
Copy link
Contributor Author

dbackeus commented Nov 6, 2024

Note to self: We should add a service that will include all nodesets except the tiebreaker and use that for the recommended ELASTICSEARCH_URL

When just having a single nodeSet with count for replication, we can't control which pod gets decommissioned when scaling down.

Using named replicas we can achieve better control for these kinds of use-cases.
@dbackeus dbackeus force-pushed the alternative-elasticsearch branch from 1a4f0ef to 701f328 Compare December 3, 2024 13:52
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.

2 participants