Skip to content

Commit 6e1ff4b

Browse files
committed
fix: simplify and test template everywhere
Signed-off-by: WrenIX <dev.github@wrenix.eu>
1 parent 892b4c4 commit 6e1ff4b

File tree

4 files changed

+12
-17
lines changed

4 files changed

+12
-17
lines changed

.github/workflows/lint-test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ jobs:
102102
- name: Extra Manifests using a map Enabled
103103
helm_args: --namespace nextcloud --skip-clean-up --helm-extra-set-args "--create-namespace --values charts/nextcloud/test-values/extra-manifests-map.yaml"
104104
test: true
105-
105+
106106
steps:
107107
- name: Checkout
108108
uses: actions/checkout@v6

charts/nextcloud/templates/extra-manifests.yaml

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,20 +4,11 @@
44
If a map, each key is the name of the manifest.
55
If an array, each item is a manifest, which can be a string (YAML block) or a YAML object.
66
*/ -}}
7-
{{- if .Values.extraManifests }}
8-
{{- if kindIs "map" .Values.extraManifests }}
9-
{{- range $manifest := values .Values.extraManifests }}
10-
---
11-
{{ toYaml $manifest }}
12-
{{- end }}
13-
{{- else if kindIs "slice" .Values.extraManifests }}
14-
{{- range $manifest := .Values.extraManifests }}
7+
{{- range $manifest := values .Values.extraManifests }}
158
---
169
{{- if kindIs "string" $manifest }}
1710
{{ tpl $manifest $ }}
1811
{{- else }}
19-
{{ toYaml $manifest }}
12+
{{ tpl (toYaml $manifest) $ }}
13+
{{- end }}
2014
{{- end }}
21-
{{- end }}
22-
{{- end }}
23-
{{- end }}
Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
11
extraManifests:
2-
my-config-map:
2+
my-config-map: |
33
apiVersion: v1
44
kind: ConfigMap
55
metadata:
66
name: custom-config
7+
labels:
8+
{{- include "nextcloud.labels" ( dict "component" "app" "rootContext" $ ) | nindent 4 }}
79
data:
810
customKey: customValue
911
my-secret:
1012
apiVersion: v1
1113
kind: Secret
1214
metadata:
13-
name: custom-secret
15+
name: '{{ template "nextcloud.fullname" . }}-custom-secret'
1416
data:
15-
secretKey: c2VjcmV0VmFsdWU=
17+
secretKey: c2VjcmV0VmFsdWU=

charts/nextcloud/test-values/extra-manifests.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,13 @@ extraManifests:
44
kind: ConfigMap
55
metadata:
66
name: custom-config
7+
labels:
8+
{{- include "nextcloud.labels" ( dict "component" "app" "rootContext" $ ) | nindent 4 }}
79
data:
810
customKey: customValue
911
- apiVersion: v1
1012
kind: Secret
1113
metadata:
12-
name: custom-secret
14+
name: '{{ template "nextcloud.fullname" . }}-custom-secret'
1315
data:
1416
secretKey: c2VjcmV0VmFsdWU=

0 commit comments

Comments
 (0)