xtreme-vikram-yadav commented 4 years ago

Existing Issue

Fixes #56 .

Changes proposed in this pull request

Update charts to support helm3

Contributor Checklist

[x] Variables are documented in the README.md
[x] Which branch are you merging into?
- 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)

supahgreg commented 4 years ago

Minor, but the helm install near the start of the README should be updated to include a release name (or --generate-name) since v3 no longer auto-generates one.

taylorsilva commented 3 years ago

Can't publish a new major of the chart until concourse/ci#375 is merged because that updates all of our scripts in CI to use helm v3

concourse / concourse-chart

Helm 3 #146

Existing Issue

Changes proposed in this pull request

Contributor Checklist

Reviewer Checklist