Skip to content

Commit

Permalink
Merge pull request #33 from replicatedhq/danj/fix-updates
Browse files Browse the repository at this point in the history
bug: chart cannot be updated due to wanting to change immutable fields.
  • Loading branch information
danj-replicated authored Jan 16, 2024
2 parents 419f1f2 + 2971f8f commit ce00790
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 4 deletions.
10 changes: 9 additions & 1 deletion templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,14 @@ kots.io/kotsadm: "true"
app.kubernetes.io/managed-by: {{ .Release.Service }}
{{- end }}

{{/*
Labels for immutable resources
*/}}
{{- define "admin-console.immutableLabels" -}}
kots.io/backup: velero
kots.io/kotsadm: "true"
{{- end }}

{{/*
Selector labels
*/}}
Expand All @@ -61,4 +69,4 @@ Create the name of the service account to use
{{- else }}
{{- default "default" .Values.serviceAccount.name }}
{{- end }}
{{- end }}
{{- end }}
4 changes: 2 additions & 2 deletions templates/minio-statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ spec:
volumeClaimTemplates:
- metadata:
labels:
{{- include "admin-console.labels" . | nindent 8 }}
{{- include "admin-console.immutableLabels" . | nindent 8 }}
name: kotsadm-minio
spec:
accessModes:
Expand All @@ -113,4 +113,4 @@ spec:
requests:
storage: 4Gi
status: {}
{{ end }}
{{ end }}
2 changes: 1 addition & 1 deletion templates/rqlite-statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ spec:
- metadata:
name: kotsadm-rqlite
labels:
{{- include "admin-console.labels" . | nindent 8 }}
{{- include "admin-console.immutableLabels" . | nindent 8 }}
spec:
accessModes:
- "ReadWriteOnce"
Expand Down

0 comments on commit ce00790

Please sign in to comment.