[redis] chart update to radar-redis and redis-operator #5921
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
pvannierop
requested changes
Oct 14, 2025
1ab757f to
da744bf
Compare
pvannierop
requested changes
Oct 15, 2025
pvannierop
requested changes
Oct 16, 2025
There was a problem hiding this comment.
Almost there! Finally, update the .github/workflows/scheduled-snyk-docker.yaml with the new external charts or docker images. When the docker image is mentioned in the external chart, add it to the INCLUDE_CHARTS. When, the docker image is not stated, as often happens when deploying a workload with an operator, hard-code the docker image in the INCLUDE_IMAGES section. Also, do cleanup of obsolete Redis references in these sections.
redis has been split up into a radar-redis chart for redis-cluster and an external radar-operator
a6dbb7f to
f1b3689
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A new CVE was discovered in REDIS CVE-2025-49844. This will be fixed in Redis release 8.2.2. Because we currently use legacy bitnami docker images that do not get security patches, we decided to move away from the current helm chart and use the OT-CONTAINER-KIT chart that uses the official Redis docker image.
This PR will implement a radar-redis chart with default config passed to the redis-cluster CRD chart of OT-CONTAINER-KIT. In addition, the OT-CONTAINER-KIT operator chart named redis-operator with the responsibility of deploying the redis-cluster CRD.
With this PR, Radar-kubernetes has to be updated as well in another PR.
Checklist
Chart.yamlaccording to semver.