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

[elasticsearch] _helpers.tpl - elasticsearch.endpoints to use elasticsearch.uname #670

Merged
merged 3 commits into from
Jul 13, 2020
Merged

[elasticsearch] _helpers.tpl - elasticsearch.endpoints to use elasticsearch.uname #670

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
de6ac86
[elasticsearch] the elasticsearch.endpoints template now uses the ela…
xario Jun 17, 2020
2a528c0
[elasticsearch] added testcase to cover the case where 'fullnameOverr…
xario Jun 19, 2020
4bcc1c8
[elasticsearch] testcases formated using black
xario Jul 2, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion elasticsearch/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this

{{- define "elasticsearch.endpoints" -}}
{{- $replicas := int (toString (.Values.replicas)) }}
{{- $uname := printf "%s-%s" .Values.clusterName .Values.nodeGroup }}
{{- $uname := (include "elasticsearch.uname" .) }}
{{- range $i, $e := untilStep 0 $replicas 1 -}}
{{ $uname }}-{{ $i }},
{{- end -}}
Expand Down
14 changes: 14 additions & 0 deletions elasticsearch/tests/elasticsearch_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1298,3 +1298,17 @@ def test_full_name_override():

assert "customfullName" in r["statefulset"]
assert "customfullName" in r["service"]


def test_initial_master_nodes_when_using_full_name_override():
config = """
fullnameOverride: "customfullName"
"""
r = helm_template(config)
env = r["statefulset"]["customfullName"]["spec"]["template"]["spec"]["containers"][
0
]["env"]
assert {
"name": "cluster.initial_master_nodes",
"value": "customfullName-0," + "customfullName-1," + "customfullName-2,",
} in env