Skip to content

Commit b07e9f1

Browse files
committed
fix(tests): Helm 4 compatability
1 parent e510521 commit b07e9f1

3 files changed

Lines changed: 7 additions & 0 deletions

File tree

docs/modules/opensearch/examples/getting_started/opensearch-dashboards-values.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ config:
2121
opensearch_security:
2222
cookie:
2323
secure: true
24+
# See https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch-dashboards/templates/deployment.yaml#L122
25+
opensearchHosts: ""
2426
extraEnvs:
2527
- name: OPENSEARCH_HOSTS
2628
valueFrom:

docs/modules/opensearch/examples/getting_started/opensearch-dashboards-values.yaml.j2

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ config:
2121
opensearch_security:
2222
cookie:
2323
secure: true
24+
# See https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch-dashboards/templates/deployment.yaml#L122
25+
opensearchHosts: ""
2426
extraEnvs:
2527
- name: OPENSEARCH_HOSTS
2628
valueFrom:

tests/templates/kuttl/opensearch-dashboards/20_opensearch-dashboards-values.yaml.j2

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ serviceAccount:
88
# Use the ServiceAccount of OpenSearch because its permissions are already configured to work on
99
# OpenShift.
1010
name: opensearch-serviceaccount
11+
# OPENSEARCH_HOSTS cannot be supplied if opensearchHosts is set (so unset explicitly).
12+
# See https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch-dashboards/templates/deployment.yaml#L122
13+
opensearchHosts: ""
1114
extraEnvs:
1215
- name: OPENSEARCH_HOSTS
1316
valueFrom:

0 commit comments

Comments
 (0)