fix: Handle log4j2 not being yaml (#110) and chart bump. (#123)

* fix: Handle log4j2 not being yaml (#110) and chart bump.

Signed-off-by: Stein Arne Storslett <[email protected]>

* Including tpl changes

Signed-off-by: Aaron Layfield <[email protected]>

* Adding log4j example.

Signed-off-by: Aaron Layfield <[email protected]>

* Adding some documentation AND updated per comment.s

Signed-off-by: Aaron Layfield <[email protected]>

* Use project name and clarify from/to.

Signed-off-by: Stein Arne Storslett <[email protected]>

* Explicitly document that config must be YAML multiline strings.

Signed-off-by: Stein Arne Storslett <[email protected]>

* Cast as string for use with tpl.

Signed-off-by: Stein Arne Storslett <[email protected]>

* Because this would be really annoying.

Signed-off-by: Stein Arne Storslett <[email protected]>

* fix: Handle log4j2 not being yaml (#110) and chart bump to 1.4.0.

Signed-off-by: Stein Arne Storslett <[email protected]>

Co-authored-by: Aaron Layfield <[email protected]>

Author: sastorsl

charts/opensearch/CHANGELOG.md

@@ -13,6 +13,26 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 ### Security
 ---
+## [1.4.0]
+### Changed
+- Changed support for any kind of configuration type. Including `log4j2.properties` file. Added example.
+### BREAKING CHANGE
+- `.Values.config` items must now be interpreted as a string. Existing items must now be updated from YAML to string:
+
+Change from YAML:
+```yaml
+config:
+  opensearch.yml:
+    cluster.name: opensearch-cluster
+```
+
+Change to YAML multiline string:
+```yaml
+config:
+  opensearch.yml: |
+    cluster.name: opensearch-cluster
+```
+---
 ## [1.3.1]
 ### Added
 - Added image definition for `fsgroup-volume` initContainer to `values.yaml`.

charts/opensearch/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 1.3.1
+version: 1.4.0
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

charts/opensearch/ci/ci-rbac-enabled-values.yaml

@@ -28,7 +28,7 @@ global:
 opensearchHome: /usr/share/opensearch
 # such as opensearch.yml and log4j2.properties
 config:
-  opensearch.yml:
+  opensearch.yml: |
     cluster.name: opensearch-cluster
 
     # Bind to all interfaces because we don't know what IP address Docker will assign to us.

charts/opensearch/ci/ci-values.yaml

@@ -28,7 +28,7 @@ global:
 opensearchHome: /usr/share/opensearch
 # such as opensearch.yml and log4j2.properties
 config:
-  opensearch.yml:
+  opensearch.yml: |
     cluster.name: opensearch-cluster
 
     # Bind to all interfaces because we don't know what IP address Docker will assign to us.

charts/opensearch/templates/configmap.yaml

@@ -1,3 +1,4 @@
+{{- $root := . }}
 {{- if .Values.config }}
 apiVersion: v1
 kind: ConfigMap
@@ -8,6 +9,6 @@ metadata:
 data:
 {{- range $configName, $configYaml := .Values.config }}
   {{ $configName }}: |
-    {{- toYaml $configYaml | nindent 4 }}
+    {{- tpl $configYaml $root | nindent 4 }}
 {{- end -}}
 {{- end -}}

charts/opensearch/values.yaml

@@ -28,7 +28,20 @@ global:
 opensearchHome: /usr/share/opensearch
 # such as opensearch.yml and log4j2.properties
 config:
-  opensearch.yml:
+  # Values must be YAML literal style scalar / YAML multiline string.
+  # <filename>: |
+  #   <formatted-value(s)>
+  # log4j2.properties: |
+  #   status = error
+  #
+  #   appender.console.type = Console
+  #   appender.console.name = console
+  #   appender.console.layout.type = PatternLayout
+  #   appender.console.layout.pattern = [%d{ISO8601}][%-5p][%-25c{1.}] [%node_name]%marker %m%n
+  #
+  #   rootLogger.level = info
+  #   rootLogger.appenderRef.console.ref = console
+  opensearch.yml: |
     cluster.name: opensearch-cluster
 
     # Bind to all interfaces because we don't know what IP address Docker will assign to us.