Skip to content

Draft: fix: use kubernetes.io/ingress.class when present

Hordur Freyr Yngvason requested to merge fix-ingress-class-annotation-on-old-controllers into master

It is possible to have an up-to-date cluster, but an outdated ingress controller. This means that the API networking.k8s.io/v1/Ingress is available, but no valid ingressClassName exists in the cluster.

This proposes to

  1. make ingress.className optional
  2. use only the kubernetes.io/ingress.class annotation when it is present (skipping ingressClassName)
  3. otherwise, set either the annotation or ingressClassName, depending on the cluster's capabilities, using ingress.className | default "nginx" | quote

See #206 (closed)

Edited by Hordur Freyr Yngvason

Merge request reports