Skip to content

Commit

Permalink
Fix: add password env to new sql image for pr envs
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisnruud committed Aug 29, 2024
1 parent 45cd22c commit 9640c87
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 16 deletions.
2 changes: 1 addition & 1 deletion charts/dsb-spring-boot/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ 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: 5.1.5
version: 5.1.6
2 changes: 2 additions & 0 deletions charts/dsb-spring-boot/templates/databaseDeployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ spec:
value: {{ .Values.database_container.password | quote }}
- name: MSSQL_SA_PASSWORD
value: {{ .Values.database_container.password | quote }}
- name: PASSWORD
value: {{ .Values.database_container.password | quote }}
ports:
- containerPort: 1433
{{ end }}
32 changes: 17 additions & 15 deletions charts/dsb-spring-boot/tests/__snapshot__/rendering_test.yaml.snap
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ Full manifest should match snapshot:
template:
metadata:
annotations:
checksum: chart-version=5.1.5_config-hash=96c18f22071cbc3581bd33b6a894673c3fd6e73587031d8d11a712a4cd55876c
checksum: chart-version=5.1.6_config-hash=6ae86511fd94e460018a98bc3e1dba6ea42cd33c938df86e4b62cf1b3571babe
labels:
app: RELEASE-NAME-db-app
spec:
Expand All @@ -199,6 +199,8 @@ Full manifest should match snapshot:
value: password123
- name: MSSQL_SA_PASSWORD
value: password123
- name: PASSWORD
value: password123
image: test-db:v1
name: RELEASE-NAME-database-container
ports:
Expand Down Expand Up @@ -245,7 +247,7 @@ Full manifest should match snapshot:
metadata:
annotations:
apparmor.security.beta.kubernetes.io/pod: runtime/default
checksum: chart-version=5.1.5_config-hash=e0678d2458f6034658a06150389218f43c8e4e3256756525d0a4529a2ce8f279
checksum: chart-version=5.1.6_config-hash=e0678d2458f6034658a06150389218f43c8e4e3256756525d0a4529a2ce8f279
co.elastic.logs/json.add_error_key: "true"
co.elastic.logs/json.keys_under_root: "true"
co.elastic.logs/json.overwrite_keys: "true"
Expand All @@ -258,9 +260,9 @@ Full manifest should match snapshot:
app.kubernetes.io/part-of: Verification stuff
app.kubernetes.io/version: greatest
azure.workload.identity/use: "true"
helm.sh/chart: dsb-spring-boot-5.1.5
helm.sh/chart: dsb-spring-boot-5.1.6
helm.sh/chart-name: dsb-spring-boot
helm.sh/chart-version: 5.1.5
helm.sh/chart-version: 5.1.6
spec:
affinity:
podAntiAffinity:
Expand Down Expand Up @@ -453,9 +455,9 @@ Full manifest should match snapshot:
app.kubernetes.io/managed-by: Helm
app.kubernetes.io/name: Rendering test
app.kubernetes.io/part-of: Verification stuff
helm.sh/chart: dsb-spring-boot-5.1.5
helm.sh/chart: dsb-spring-boot-5.1.6
helm.sh/chart-name: dsb-spring-boot
helm.sh/chart-version: 5.1.5
helm.sh/chart-version: 5.1.6
name: RELEASE-NAME
namespace: NAMESPACE
spec:
Expand Down Expand Up @@ -504,10 +506,10 @@ Full manifest should match snapshot:
app.kubernetes.io/part-of: Verification stuff
app.kubernetes.io/version: greatest
chart-name: dsb-spring-boot
chart-version: 5.1.5
helm.sh/chart: dsb-spring-boot-5.1.5
chart-version: 5.1.6
helm.sh/chart: dsb-spring-boot-5.1.6
helm.sh/chart-name: dsb-spring-boot
helm.sh/chart-version: 5.1.5
helm.sh/chart-version: 5.1.6
management.port: "81"
spring-boot: "true"
name: RELEASE-NAME
Expand Down Expand Up @@ -569,7 +571,7 @@ Minimal manifest should match snapshot:
metadata:
annotations:
apparmor.security.beta.kubernetes.io/pod: runtime/default
checksum: chart-version=5.1.5_config-hash=a852fe4f1f87db4cae8924a3cc9190d1a6d20810cb3b0739daf5977ba95e553b
checksum: chart-version=5.1.6_config-hash=a852fe4f1f87db4cae8924a3cc9190d1a6d20810cb3b0739daf5977ba95e553b
co.elastic.logs/json.add_error_key: "true"
co.elastic.logs/json.keys_under_root: "true"
co.elastic.logs/json.overwrite_keys: "true"
Expand All @@ -582,9 +584,9 @@ Minimal manifest should match snapshot:
app.kubernetes.io/part-of: RELEASE-NAME
app.kubernetes.io/version: latest
azure.workload.identity/use: "false"
helm.sh/chart: dsb-spring-boot-5.1.5
helm.sh/chart: dsb-spring-boot-5.1.6
helm.sh/chart-name: dsb-spring-boot
helm.sh/chart-version: 5.1.5
helm.sh/chart-version: 5.1.6
spec:
affinity:
podAntiAffinity:
Expand Down Expand Up @@ -680,10 +682,10 @@ Minimal manifest should match snapshot:
app.kubernetes.io/part-of: RELEASE-NAME
app.kubernetes.io/version: latest
chart-name: dsb-spring-boot
chart-version: 5.1.5
helm.sh/chart: dsb-spring-boot-5.1.5
chart-version: 5.1.6
helm.sh/chart: dsb-spring-boot-5.1.6
helm.sh/chart-name: dsb-spring-boot
helm.sh/chart-version: 5.1.5
helm.sh/chart-version: 5.1.6
management.port: "8180"
spring-boot: "true"
name: RELEASE-NAME
Expand Down

0 comments on commit 9640c87

Please sign in to comment.