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

Commit

Permalink
add workaround for coordinating node
Browse files Browse the repository at this point in the history
  • Loading branch information
jmlrt committed May 14, 2021
1 parent a217b3b commit e1bf227
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 2 deletions.
24 changes: 22 additions & 2 deletions elasticsearch/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -213,8 +213,27 @@ while they share the same `clusterName` value.

For each Helm release, the nodes types can then be defined using `roles` value.

An example of Elasticsearch cluster using 2 different Helm releases for master
and data nodes can be found in [examples/multi][].
An example of Elasticsearch cluster using 2 different Helm releases for master,
data and coordinating nodes can be found in [examples/multi][].

#### Coordinating nodes

Every node is implicitly a coordinating node. This means that a node that has an
explicit empty list of roles will only act as a coordinating node.

When deploying coordinating-only node with Elasticsearch chart, it is required
to define the empty list of roles in both `roles` value and `node.roles`
settings:

```yaml
roles: []

esConfig:
elasticsearch.yml: |
node.roles: []
```
More details in [#1186 (comment)][]
#### Clustering and Node Discovery
Expand Down Expand Up @@ -381,6 +400,7 @@ about our development and testing process.

[7.x]: https://github.com/elastic/helm-charts/releases
[#63]: https://github.com/elastic/helm-charts/issues/63
[#1186 (comment)]: https://github.com/elastic/helm-charts/pull/1186#discussion_r631166442
[7.9.2]: https://github.com/elastic/helm-charts/blob/7.9.2/elasticsearch/README.md
[BREAKING_CHANGES.md]: https://github.com/elastic/helm-charts/blob/master/BREAKING_CHANGES.md
[CHANGELOG.md]: https://github.com/elastic/helm-charts/blob/master/CHANGELOG.md
Expand Down
6 changes: 6 additions & 0 deletions elasticsearch/examples/multi/client.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@ roles: []

persistence:
enabled: false

# For client nodes, we also need to add an empty node.roles in elasticsearch.yml
# This is due to https://github.com/elastic/helm-charts/pull/1186#discussion_r631225687
esConfig:
elasticsearch.yml: |
node.roles: []

0 comments on commit e1bf227

Please sign in to comment.