concourse / concourse-chart

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

Update image version to 7.3.1 #254

Closed qlikcoe closed 3 years ago

qlikcoe commented 3 years ago

This new version has a fix for guardian that was failing due to bad parsing of the kernel version.

Why do we need this PR?

Fixes a bug where guardian would fail to start up when the kernel version contained an unexpected suffix

Contributor Checklist

Reviewer Checklist

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

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

Thanks for the PR and reminder to do this. We actually have a job in CI for this: https://ci.concourse-ci.org/teams/main/pipelines/helm-chart/jobs/concourse-app-bump/

Running it now and pushing out a new version of the chart as well. I'll close this PR now.

taylorsilva commented 3 years ago

https://github.com/concourse/concourse-chart/commit/dd8656fd80129906d0f9b62fe059a24a337eb1df