Skip to content

Commit

Permalink
feat: ✨ display release name and image full path in installation notes
Browse files Browse the repository at this point in the history
  • Loading branch information
mloiseleur committed Jul 23, 2024
1 parent 2b03ee8 commit b77d53d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion traefik/templates/NOTES.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@


Traefik Proxy {{ .Values.image.tag | default .Chart.AppVersion }} has been deployed successfully on {{ template "traefik.namespace" . }} namespace !
{{ .Release.Name }} with {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }} has been deployed successfully on {{ template "traefik.namespace" . }} namespace !

{{- if .Values.persistence }}
{{- if and .Values.persistence.enabled (empty .Values.deployment.initContainer)}}
Expand Down
12 changes: 6 additions & 6 deletions traefik/tests/notes_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ tests:
value: |2
Traefik Proxy 2.3.0 has been deployed successfully on NAMESPACE namespace !
RELEASE-NAME with docker.io/traefik:2.3.0 has been deployed successfully on NAMESPACE namespace !
- it: should use helm managed namespace in release notes output
set:
namespaceOverride: "traefik-ns-override"
Expand All @@ -25,7 +25,7 @@ tests:
value: |2
Traefik Proxy 2.3.0 has been deployed successfully on traefik-ns-override namespace !
RELEASE-NAME with docker.io/traefik:2.3.0 has been deployed successfully on traefik-ns-override namespace !
- it: should use tag version in release notes output if set
set:
image:
Expand All @@ -35,7 +35,7 @@ tests:
value: |2
Traefik Proxy v3.0 has been deployed successfully on NAMESPACE namespace !
RELEASE-NAME with docker.io/traefik:v3.0 has been deployed successfully on NAMESPACE namespace !
- it: should not display a warning when persistence and initContainer are enabled
set:
persistence:
Expand All @@ -53,7 +53,7 @@ tests:
value: |2
Traefik Proxy 2.3.0 has been deployed successfully on NAMESPACE namespace !
RELEASE-NAME with docker.io/traefik:2.3.0 has been deployed successfully on NAMESPACE namespace !
- it: should display warning when enabling persistence without initContainer
set:
persistence:
Expand All @@ -63,7 +63,7 @@ tests:
value: |
Traefik Proxy 2.3.0 has been deployed successfully on NAMESPACE namespace !
RELEASE-NAME with docker.io/traefik:2.3.0 has been deployed successfully on NAMESPACE namespace !
🚨 When enabling persistence for certificates, permissions on acme.json can be
lost when Traefik restarts. You can ensure correct permissions with an
Expand All @@ -83,6 +83,6 @@ tests:
value: |
Traefik Proxy 2.3.0 has been deployed successfully on NAMESPACE namespace !
RELEASE-NAME with docker.io/traefik:2.3.0 has been deployed successfully on NAMESPACE namespace !
🚨 Resources populated with this chart don't match with labelSelector `not_here=CRD` applied on kubernetesCRD provider 🚨
🚨 Resources populated with this chart don't match with labelSelector `not_here=Ingress` applied on kubernetesIngress provider 🚨

0 comments on commit b77d53d

Please sign in to comment.