Support IngressClassName in OpenSearch and OpenSearch Dashboards charts.

[mprimeaux]

Is your feature request related to a problem? Please describe.
Kubernetes 1.18 and greater support the IngressClassName spec stanza as part of the Ingress kind.

Describe the solution you'd like
If defined in the chart values file, the Ingress spec stanza should use the IngressClassName value.

Describe alternatives you've considered
While we may be able to also leverage an ingress class name annotation, this is not always accepted.

Additional context

Ingress

{{- if .Values.ingress.enabled -}}
{{- $fullName := include "opensearch.uname" . -}}
{{- $servicePort := .Values.httpPort -}}
{{- $ingressPath := .Values.ingress.path -}}
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: {{ $fullName }}
  labels:
    app: {{ .Chart.Name }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
{{- with .Values.ingress.annotations }}
  annotations:
{{ toYaml . | indent 4 }}
{{- end }}
spec:
+  {{- if .Values.ingress.ingressClassName }}
+  ingressClassName: {{ .Values.ingress.ingressClassName | quote }}
+  {{- end }}
{{- if .Values.ingress.tls }}
  tls:
  {{- range .Values.ingress.tls }}
    - hosts:
      {{- range .hosts }}
        - {{ . }}
      {{- end }}
      secretName: {{ .secretName }}
  {{- end }}
{{- end }}
  rules:
  {{- range .Values.ingress.hosts }}
    - host: {{ . }}
      http:
        paths:
          - path: {{ $ingressPath }}
            backend:
              serviceName: {{ $fullName }}
              servicePort: {{ $servicePort }}
  {{- end }}
{{- end }}

[peterzhuamazon]

@mprimeaux could you please change the title to reflect this issue? Thanks.

[aschi1]

Hi, is there any update for this? IngressClassName is blocking us to migrate to this helm chart (which we would very much like to)

Thank you!

[mprimeaux]

I'll review and update today.

[mprimeaux]

@aschi1 PR #149 has been submitted for community review.

@peterzhuamazon @TheAlgo @DandyDeveloper @smlx FYI and thanks for your continued support. Note that I synchronized the CI values files to match the values.yaml file in the OpenSearch chart as they had gotten out of sync over the past weeks.

[aschi1]

@mprimeaux Thank you!