Conversation
|
Did you test the kiwigrid sidecar, this includes a major release. Our CI test doesn't cover it AFAIK |
|
I have not tested it locally, but the CI deploys the chart and waits for all containers to be healthy: https://github.com/cortexproject/cortex-helm-chart/blob/master/.github/workflows/lint-test.yml#L44 I reviewed the release notes and the major version bump was due to deprecating two uncommon CPU architectures from the multi-arch image. |
Yeah but it's not enabled by default and we don't check it in the CI See: https://github.com/cortexproject/cortex-helm-chart/blob/master/ci/test-deployment-values.yaml |
|
I will share a PR tomorrow to enable it in the CI and override the health port which now conflicts by default. |
|
Opened #597 to test and fix the sidecar. |
Signed-off-by: nschad <nschad@users.noreply.github.com>
ce705a3 to
2310970
Compare
|
Replaced by #598 |
Automated release PR for version 3.1.1