concourse / concourse-chart

Helm chart to install Concourse

Apache License 2.0

146 stars 176 forks source link

Support defining which ingressClass to use. #273

Closed pipeworks-asmith closed 3 years ago

pipeworks-asmith commented 3 years ago

Existing Issue

Fixes #264 .

Changes proposed in this pull request

Set web.ingress.ingressClassName to choose IngressClass

Contributor Checklist

[x] Variables are documented in the README.md
[x] Which branch are you merging into?
- master is for changes related to the current release of the concourse/concourse:latest image and should be good to publish immediately
- dev is for changes related to the next release of Concourse (aka unpublished code on master in concourse/concourse)

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)

pipeworks-asmith commented 3 years ago

~~Hmm I signed the commit with my personal key not my professional one. If that's a problem I can re-commit.~~ It was bothering me so I fixed it :)