concourse / concourse-chart

Helm chart to install Concourse
Apache License 2.0
143 stars 174 forks source link

Decoupling init-rm initContainer and extra initContainers #329

Closed m-roskrow closed 1 year ago

m-roskrow commented 1 year ago

Why do we need this PR?

Decouples the extraInitContainers and cleanUpWorkDirOnStart variables. Currently it is impossible to have an extraInitContainer without also having the init-rm container

Changes proposed in this pull request

Slightly changed conditionality around the initContainers in the statefulset yaml

Contributor Checklist

merging into master

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)