Skip to content

Commit

Permalink
chore: Prevent displaying constant diffs for nextcloud-token in ArgoCD (
Browse files Browse the repository at this point in the history
#554)

* chore: Prevent displaying constant diffs for nextcloud-token in ArgoCD

When metrics are not enabled, the use of randAlphaNum was causing constant diffs for nextcloud-token in ArgoCD. To mitigate this issue, now nextcloud-token won't be configured when metrics are not enabled.

Signed-off-by: kahirokunn <[email protected]>

* Update charts/nextcloud/templates/secrets.yaml

Co-authored-by: JesseBot <[email protected]>
Signed-off-by: kahirokunn <[email protected]>

* Update charts/nextcloud/Chart.yaml

Co-authored-by: JesseBot <[email protected]>
Signed-off-by: kahirokunn <[email protected]>

---------

Signed-off-by: kahirokunn <[email protected]>
Signed-off-by: JesseBot <[email protected]>
Co-authored-by: JesseBot <[email protected]>
  • Loading branch information
kahirokunn and jessebot authored May 1, 2024
1 parent 30c69c1 commit 1ae7421
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/nextcloud/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: nextcloud
version: 4.6.7
version: 4.6.8
appVersion: 29.0.0
description: A file sharing server that puts the control and security of your own data back into your hands.
keywords:
Expand Down
6 changes: 3 additions & 3 deletions charts/nextcloud/templates/secrets.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ data:
{{- else }}
nextcloud-password: {{ randAlphaNum 10 | b64enc | quote }}
{{- end }}
{{- with .Values.metrics.token }}
nextcloud-token: {{ . | b64enc | quote }}
{{- else }}
{{- if and .Values.metrics.enabled .Values.metrics.token }}
nextcloud-token: {{ .Values.metrics.token | b64enc | quote }}
{{- else if and .Values.metrics.enabled (not .Values.metrics.token) }}
nextcloud-token: {{ randAlphaNum 10 | b64enc | quote }}
{{- end }}
{{- if .Values.nextcloud.mail.enabled }}
Expand Down

0 comments on commit 1ae7421

Please sign in to comment.