Update Redis from 7.0 to 7.2, bump redis and psql charts
What does this MR do?
- Update Redis chart from 17.14.6 to 18.19.4. This results in shipping Redis 7.2 instead of 7.0.
- Update PostgreSQL chart from 12.5.2 to 12.12.10.
The PSQL bump is a requirement to bump Reids, because both charts have a shared dependency (bitnami/common) that otherwise breaks the Redis chart.
Error seen without bumping psql:
Error: UPGRADE FAILED: template: gitlab/charts/redis/templates/master/serviceaccount.yaml:13:14: executing "gitlab/charts/redis/templates/master/serviceaccount.yaml" at <include "common.labels.standard" (dict "customLabels" .Values.commonLabels "context" $)>: error calling include: template: gitlab/charts/postgresql/charts/common/templates/_labels.tpl:6:27: executing "common.labels.standard" at <include "common.names.name" .>: error calling include: template: gitlab/charts/postgresql/charts/common/templates/_names.tpl:6:18: executing "common.names.name" at <.Chart.Name>: nil pointer evaluating interface {}.Name
Relates upstream issue: https://github.com/bitnami/charts/issues/18933
Related issues
Closes Upgrade Redis to v7.2 (gitlab-org/distribution/team-tasks#1694 - closed)
Author checklist
For general guidance, please follow our Contributing guide.
Required
For anything in this list which will not be completed, please provide a reason in the MR discussion.
-
Merge Request Title and Description are up to date, accurate, and descriptive. -
MR targeting the appropriate branch. -
MR has a green pipeline. -
Documentation created/updated. -
Tests added/updated, and test plan for scenarios not covered by automated tests. -
Equivalent MR/issue for omnibus-gitlab opened.
Reviewers checklist
-
MR has a green pipeline on https://gitlab.com/gitlab-org/charts/gitlab. -
Consider downstream impact to the Operator, as per evaluating impact from changes to GitLab Chart.
Test Plan
-
Do a upgrade test from chart 8.9 (has redis 6.2) -
Do a upgrade test from chart 8.10 (has redis 7.0 from !3977 (merged))