Closed o-orand closed 2 years ago
This PR provided some bug fixes related to #211.
worker-secret.yml
worker.certsPath
values.yml
worker.workerCertsPath
Which branch are you merging into?
master
concourse/concourse:latest
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)
This section is intended for the core maintainers only, to track review progress. Please do not fill out this section.
dev
Why do we need this PR?
This PR provided some bug fixes related to #211.
Changes proposed in this pull request
worker-secret.yml
removal of worker.additionalCertificates should have been done with 43483576525c8fab2e5af1f3ff41a039b122d433 (included in #244). But it still present inworker-secret.yml
.worker.certsPath
defined invalues.yml
insteadworker.workerCertsPath
(undocumented and not use anywhere else). https://github.com/concourse/concourse-chart/blob/6b13c2077bcdede6ef1615d22eef9d36307e70e7/values.yaml#L2304-L2307Contributor Checklist
Which branch are you merging into?
master
is for changes related to the current release of theconcourse/concourse:latest
image and should be good to publish immediatelyReviewer Checklist