Skip to content

Commit

Permalink
support ingressClassName on ingress
Browse files Browse the repository at this point in the history
Signed-off-by: christopher dykstra <chrdyks@gmail.com>
  • Loading branch information
hypnopotamus authored and Mirco Hacker committed Sep 24, 2021
1 parent 9399400 commit 4bfb56a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions charts/mailhog/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ Parameter | Description | Default
`service.nodePort.smtp` | If `service.type` is `NodePort` and this is non-empty, sets the smtp node port of the service | `""`
`securityContext` | Pod security context | `{ runAsUser: 1000, fsGroup: 1000, runAsNonRoot: true }`
`ingress.enabled` | If `true`, an ingress is created | `false`
`ingress.ingressClassName` | If set the created Ingress resource will have this class name. kubernetes.io/ingress.class is [deprecated](https://kubernetes.io/docs/concepts/services-networking/ingress/#deprecated-annotation) | `nil`
`ingress.annotations` | Annotations for the ingress | `{}`
`ingress.labels` | Labels for the ingress | `{}`
`ingress.hosts` | A list of ingress hosts | `{ host: mailhog.example.com, paths: [{ path: "/", pathType: Prefix }] }`
Expand Down
3 changes: 3 additions & 0 deletions charts/mailhog/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ metadata:
{{- toYaml . | nindent 4 }}
{{- end }}
spec:
{{- if .Values.ingress.ingressClassName }}
ingressClassName: {{ .Values.ingress.ingressClassName }}
{{- end }}
{{- if .Values.ingress.tls }}
tls:
{{- range .Values.ingress.tls }}
Expand Down
1 change: 1 addition & 0 deletions charts/mailhog/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ containerSecurityContext:

ingress:
enabled: false
# ingressClassName: nginx
annotations: {}
# kubernetes.io/ingress.class: nginx
# kubernetes.io/tls-acme: "true"
Expand Down

0 comments on commit 4bfb56a

Please sign in to comment.