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

[elasticsearch] fix node roles for clients nodes #1693

Merged
merged 1 commit into from
Sep 9, 2022
Merged

Commits on Sep 8, 2022

  1. [elasticsearch] fix node roles for clients nodes

    This commit fix the node.roles variable for client role with
    Elasticsearch version > 8.3.0.
    
    As client nodes are nodes that don't have any other roles,
    setting the client roles is done by configuring `node.roles: []`
    (empty list).
    
    Elasticsearch chart usually define `node.roles` as an environment
    variable, however, for client nodes, setting an empty list as value of
    an environment variable isn't recognized by Elasticsearch so we were
    required to also add it to the `elasticsearch.yaml` config file (more
    details in
    elastic#1186 (comment)).
    
    Starting with Elasticsearch 8.3.0 this is not working anymore and
    Elasticsearch fails to start is a `node.roles` environment variable is
    defined with an empty list as value.
    
    This commit define the `node.roles` environment variable only if the
    `roles` list isn't empty.
    
    Fixes also the tests for the `multi` example.
    
    Relates to elastic#1186
    jmlrt committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    d46992e View commit details
    Browse the repository at this point in the history