Skip to content

Commit

Permalink
Merge pull request #190 from davidcaste/fix/deployment_app_version_label
Browse files Browse the repository at this point in the history
Use image.tag for app.kubernetes.io/version label if defined
  • Loading branch information
hagaibarel authored Jan 20, 2025
2 parents 59fa883 + 5e32886 commit b7df8ac
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions charts/coredns/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: coredns
version: 1.37.1
version: 1.37.2
appVersion: 1.11.4
home: https://coredns.io
icon: https://coredns.io/images/CoreDNS_Colour_Horizontal.png
Expand All @@ -19,5 +19,5 @@ maintainers:
type: application
annotations:
artifacthub.io/changes: |
- kind: changed
description: Fixed typo in readme
- kind: fixed
description: Use image.tag for app.kubernetes.io/version label if defined
2 changes: 1 addition & 1 deletion charts/coredns/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
name: {{ default (include "coredns.fullname" .) .Values.deployment.name }}
namespace: {{ .Release.Namespace }}
labels: {{- include "coredns.labels" . | nindent 4 }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
app.kubernetes.io/version: {{ .Values.image.tag | default .Chart.AppVersion | quote }}
{{- if .Values.customLabels }}
{{ toYaml .Values.customLabels | indent 4 }}
{{- end }}
Expand Down

0 comments on commit b7df8ac

Please sign in to comment.