concourse / concourse-chart

Helm chart to install Concourse
Apache License 2.0
145 stars 177 forks source link

Fix ingress API group for >1.20 k8s #205

Closed taylorsilva closed 3 years ago

taylorsilva commented 3 years ago

Existing Issue

Fixes #204

Why do we need this PR?

The ingress resource was moved to a new API group, with a full cutover starting in 1.22. Moved from networking.k8s.io/v1beta1 to networking.k8s.io/v1.

Changes proposed in this pull request

Based on this comment the new API group is only applied when the cluster version is >=1.20, which is when k8s starts reading from the stable api group instead of the beta api group for the ingress controller.

Contributor Checklist

Reviewer Checklist

This section is intended for the core maintainers only, to track review progress. Please do not fill out this section.

  • [ ] Code reviewed
  • [ ] Topgun tests run
  • [ ] Back-port if needed
  • [ ] Is the correct branch targeted? (master or dev)