Upgrade redis-ratelimiting to 6.2
Background
The helm chart for redis is tied to a redis version. When I tried running it with a Redis 6.0 tag, I got errors due to incompatible config.
Running an old chart is not feasible, as there are critical fixes in later versions that have not been backported. Not to mention the integration pain of a chart upgrade (lots of parameters have changed).
Going from 6.0 in VMs directly to 6.2 on k8s might be doable but seems risky and does not give us proper A/B comparison. I'd like to avoid this if possible.
Options
I believe our options are:
- Fork the bitnami chart and make it compatible with Redis 6.0
- Contribute Redis 6.0 BC to bitnami chart (if they are willing to merge)
- Upgrade Redis to 6.2 before we move to k8s, then run on 6.2 in k8s
Proposal
Given we want to upgrade to 6.2 eventually anyway, bitnami may not accept a BC patch, and I don't want to maintain a fork of the chart long-term, I'm leaning towards 3.
Doing so would require some upgrade testing. We can probably reuse this prior art.
If that looks good, we can proceed with bumping the version in omnibus, and then going through the upgrade process for the ratelimiting Redis.
Decision
We are going with option 3, as this gives us the most flexibility.