Skip to content

Commit f83eb3c

Browse files
authored
Configuration fixes for vpa and external-secrets (#72)
* add quote step for vpa updateMode * add gitignore * rework apiVersion config for ExternalSecret * bump version
1 parent 2cb742c commit f83eb3c

File tree

5 files changed

+17
-7
lines changed

5 files changed

+17
-7
lines changed

.gitignore

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
*.tgz
2+
/.idea/*
3+
.vscode
4+
.DS_Store
5+
.aider*
6+
.env

standard-app/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v2
22
name: standard-app
33
description: A Helm chart library by Cloudkite
44
type: application
5-
version: 1.2.0
5+
version: 1.2.1
66
maintainters:
77
88
name: cloudkite

standard-app/templates/autoscaling/vpa.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ spec:
1717
kind: {{ if $appConfig.rollout }}Rollout{{ else }}Deployment{{ end }}
1818
name: {{ $appName }}
1919
updatePolicy:
20-
updateMode: {{ $appConfig.vpa.mode | default "Off" }}
20+
updateMode: {{ $appConfig.vpa.mode | default "Off" | quote }}
2121
{{- with $appConfig.vpa.resources }}
2222
resourcePolicy:
2323
containerPolicies:

standard-app/templates/configs/_externalsecret.tpl

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{{- define "standard-app.externalSecret" -}}
2+
{{- $apiVersion := .apiVersion }}
23
{{- $name := .name }}
34
{{- $userLabels := .labels | default dict }}
45
{{- $globalLabels := $.Values.labels | default dict }}
@@ -10,11 +11,7 @@
1011
{{- $secretPath := .secretPath }}
1112
{{- $refreshInterval := .refreshInterval | default "1m" }}
1213

13-
{{ if and (.Capabilities) (.Capabilities.APIVersions.Has "external-secrets.io/v1/ExternalSecret") }}
14-
apiVersion: external-secrets.io/v1
15-
{{ else }}
16-
apiVersion: external-secrets.io/v1beta1
17-
{{ end }}
14+
apiVersion: {{ $apiVersion }}
1815
kind: ExternalSecret
1916
metadata:
2017
name: {{ $name }}

standard-app/templates/configs/externalsecret.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{{- if .Values.externalSecret }}
2+
{{- $apiVersion := ternary "external-secrets.io/v1" "external-secrets.io/v1beta1" ($.Capabilities.APIVersions.Has "external-secrets.io/v1/ExternalSecret") -}}
23
{{- $type := .Values.externalSecret.type }}
34
{{- $storeName := .Values.externalSecret.secretStoreName }}
45
{{- $secretPath := .Values.externalSecret.secretPath }}
@@ -11,6 +12,7 @@
1112
{{- range $name, $config := index $appConfig $containerType }}
1213
{{- if $config.secrets }}
1314
{{- template "standard-app.externalSecret" (dict
15+
"apiVersion" $apiVersion
1416
"name" $name
1517
"labels" (merge (dict "app" $appName "product" $.Release.Name) (default dict $.Values.labels))
1618
"secrets" $config.secrets
@@ -28,6 +30,7 @@
2830

2931
{{- if $appConfig.secrets }}
3032
{{- template "standard-app.externalSecret" (dict
33+
"apiVersion" $apiVersion
3134
"name" $appName
3235
"labels" (merge (dict "app" $appName "product" $.Release.Name) (default dict $.Values.labels))
3336
"secrets" $appConfig.secrets
@@ -47,6 +50,7 @@
4750
{{- range $jobName, $jobConfig := .Values.jobs }}
4851
{{- if $jobConfig.secrets }}
4952
{{- template "standard-app.externalSecret" (dict
53+
"apiVersion" $apiVersion
5054
"name" $jobName
5155
"labels" (merge (dict "app" $jobName "product" $.Release.Name) (default dict $.Values.labels))
5256
"secrets" $jobConfig.secrets
@@ -67,6 +71,7 @@
6771
{{- range $name, $config := index $cronjobConfig $containerType }}
6872
{{- if $config.secrets }}
6973
{{- template "standard-app.externalSecret" (dict
74+
"apiVersion" $apiVersion
7075
"name" $name
7176
"labels" (merge (dict "app" $cronjobName "product" $.Release.Name) (default dict $.Values.labels))
7277
"secrets" $config.secrets
@@ -84,6 +89,7 @@
8489

8590
{{- if $cronjobConfig.secrets }}
8691
{{- template "standard-app.externalSecret" (dict
92+
"apiVersion" $apiVersion
8793
"name" $cronjobName
8894
"labels" (merge (dict "app" $cronjobName "product" $.Release.Name) (default dict $.Values.labels))
8995
"secrets" $cronjobConfig.secrets
@@ -102,6 +108,7 @@
102108
# Global secret
103109
{{- if .Values.secrets }}
104110
{{- template "standard-app.externalSecret" (dict
111+
"apiVersion" $apiVersion
105112
"name" $.Release.Name
106113
"labels" (merge (dict "app" $.Release.Name) (default dict $.Values.labels))
107114
"secrets" .Values.secrets

0 commit comments

Comments
 (0)