Add ingress class name, app version, and fix templating length

Author: Techwolf12

pretix-helm/Chart.yaml

@@ -2,6 +2,7 @@ name: pretix
 description: Reinventing ticket presales, one ticket at a time.
 type: application
 version: 2024.3.0
+appVersion: 2024.3.0
 apiVersion: v2
 keywords:
   - event-tickets

pretix-helm/README.md

@@ -138,6 +138,7 @@ More options can be overridden from the Redis chart here.
 |------------------------------------|-------------------------------------------------------------------------------|------------------------|
 | ingress.enabled                    | If the ingress is enabled                                                     | false                  |
 | ingress.annotations                | Annotations to be added to the ingress                                        | {}                     |
+| ingress.ingressClassName           | Class name to use for the ingress                                             | ""                     |
 | ingress.hosts[0].host              | The host name to be used for the ingress, make sure it matches the Pretix env | pretix.example.com     |
 | ingress.hosts[0].paths[0].path     | The path under the host                                                       | /                      |
 | ingress.hosts[0].paths[0].pathType | The pathType of the path under the host                                       | ImplementationSpecific |

pretix-helm/templates/pretix-cron.yaml

@@ -1,7 +1,7 @@
 apiVersion: batch/v1
 kind: CronJob
 metadata:
-  name: {{ include "pretix.fullname" . }}-cron
+  name: {{ include "pretix.fullname" . | trunc 58 }}-cron
   labels:
     {{- include "pretix.labels" . | nindent 4 }}
 spec:
@@ -14,7 +14,7 @@ spec:
             volumes:
             - name: pretix-data
               persistentVolumeClaim:
-                claimName: {{ include "pretix.fullname" . }}-data
+                claimName: {{ include "pretix.fullname" .  | trunc 58 }}-data
             containers:
             - name: pretix-cron
               image: {{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}

pretix-helm/templates/pretix-data-pvc.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
-  name: {{ include "pretix.fullname" . }}-data
+  name: {{ include "pretix.fullname" . | trunc 58 }}-data
 spec:
   accessModes:
     - {{ .Values.persistence.accessMode }}

pretix-helm/templates/ingress.yaml pretix-helm/templates/pretix-ingress.yaml

@@ -11,6 +11,9 @@ metadata:
     {{- toYaml . | nindent 4 }}
   {{- end }}
 spec:
+  {{- with .Values.ingress.ingressClassName }}
+  ingressClassName: {{ . }}
+  {{- end }}
   {{- if .Values.ingress.tls }}
   tls:
     {{- range .Values.ingress.tls }}

pretix-helm/templates/pretix-migrate.yaml

@@ -1,7 +1,7 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
-  name: {{ include "pretix.fullname" . }}-migrate
+  name: {{ include "pretix.fullname" .  | trunc 55 }}-migrate
   labels:
     {{- include "pretix.labels" . | nindent 4 }}
   annotations:
@@ -15,7 +15,7 @@ spec:
       volumes:
       - name: pretix-data
         persistentVolumeClaim:
-          claimName: {{ include "pretix.fullname" . }}-data
+          claimName: {{ include "pretix.fullname" .  | trunc 58 }}-data
       containers:
       - name: pretix-migrate
         image: {{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}

pretix-helm/templates/pretix-web-deployment.yaml

@@ -1,7 +1,7 @@
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  name: {{ include "pretix.fullname" . }}-web
+  name: {{ include "pretix.fullname" . | trunc 59 }}-web
   labels:
     app: {{ include "pretix.fullname" . }}
     service: web
@@ -51,4 +51,4 @@ spec:
       volumes:
       - name: pretix-data
         persistentVolumeClaim:
-          claimName: {{ include "pretix.fullname" . }}-data
+          claimName: {{ include "pretix.fullname" .  | trunc 58 }}-data

pretix-helm/templates/pretix-worker-deployment.yaml

@@ -1,7 +1,7 @@
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  name: {{ include "pretix.fullname" . }}-worker
+  name: {{ include "pretix.fullname" . | trunc 56 }}-worker
   labels:
     app: {{ include "pretix.fullname" . }}
     service: worker
@@ -51,4 +51,4 @@ spec:
       volumes:
       - name: pretix-data
         persistentVolumeClaim:
-          claimName: {{ include "pretix.fullname" . }}-data
+          claimName: {{ include "pretix.fullname" .  | trunc 58 }}-data

pretix-helm/values.yaml

@@ -70,6 +70,7 @@ redis:
 ingress:
   enabled: false
   annotations: {}
+  ingressClassName: ""
   hosts:
     - host: pretix.example.com
       paths: