charts icon indicating copy to clipboard operation
charts copied to clipboard

[bitnami/redis] Redict Support for Redis Chart

Open gmemstr opened this issue 2 years ago • 16 comments

Name and Version

bitnami/redis 19.1.0

What is the problem this feature will solve?

Redis is no longer free software and many are looking to adopt forks, including Redict. Redict has taken the stance that Redis is more or less complete software, and will continue maintenence relatively unchanged. This means that going forward, there is a FOSS option for for Redis chart to use. Redict has changed the binary name to redict-*, which will require tweaks from the chart.

What is the feature you are proposing to solve the problem?

Test and support either the upstream Redict image (https://codeberg.org/redict/-/packages/container/redict/7.3.0-bookworm) or build a custom image for the Helm chart.

What alternatives have you considered?

No response

gmemstr avatar Apr 13 '24 09:04 gmemstr

Hi!

Thank you so much for the feature request. I will forward this to the team for evaluation.

javsalgar avatar Apr 15 '24 08:04 javsalgar

Any updates on this? It would be very useful to have a full redis compatible dropin replacement, and redict seems to fit that bill best.

mickenordin avatar Apr 26 '24 12:04 mickenordin

We're working on adding Valkey to the catalog

carrodher avatar Apr 27 '24 17:04 carrodher

Redict recently added compatibility images, which should be drop-in replacements for Redis images in this chart https://codeberg.org/redict/redict-containers/commit/91b0b245ce721a4cc5765417b646d97452a24d33 so I consider this complete to some degree :)

gmemstr avatar May 03 '24 16:05 gmemstr

@carrodher Are we planning to add separate helm charts for valkey deployment as an alternative to redis helm chart ?

AmitKatyal-Sophos avatar May 06 '24 12:05 AmitKatyal-Sophos

Hi!

Yes, creating a helm chart for Valkey is in our roadmap

javsalgar avatar May 07 '24 07:05 javsalgar

@javsalgar Thanks for the update ! if we are already using redis helm charts with master-replica with sentinel configuration, the migration to valkey charts will be seamless ?

AmitKatyal-Sophos avatar May 07 '24 07:05 AmitKatyal-Sophos

That's a good question for the Valkey community, we highly recommend that you refer to the forums and user guides provided by the project responsible for the application itself since there you can find proper guidance about migration from Redis to Valkey.

carrodher avatar May 07 '24 09:05 carrodher

It should be possible to just use the redict compatibility images with the redis-charts now:

https://redict.io/docs/install/containers/#compatibility

https://artifacthub.io/packages/helm/bitnami/redis#use-a-different-redis-version

mickenordin avatar May 07 '24 10:05 mickenordin

@mickenordin we have redict images for redis server only or the same container has redis-sentinel binary as well ?. As per documentation, only scratch container has redis-sentinel binary. How about other variants (debian, alpine...etc)

AmitKatyal-Sophos avatar May 07 '24 11:05 AmitKatyal-Sophos

@mickenordin we have redict images for redis server only or the same container has redis-sentinel binary as well ?. As per documentation, only scratch container has redis-sentinel binary. How about other variants (debian, alpine...etc)

All of them have all the binaries. So if that is not clear, the docs should probably be updated. Can you point me to the doc that gave you the impression it was not there? Edit: I see it now. Docs are being updated to be more clear now: https://codeberg.org/redict/redict.io/pulls/28

mickenordin avatar May 07 '24 11:05 mickenordin

@mickenordin Any specific reason that redict container images are not uploaded to the docker hub ?

AmitKatyal-Sophos avatar May 08 '24 09:05 AmitKatyal-Sophos

@mickenordin Any specific reason that redict container images are not uploaded to the docker hub ?

Yes, we discussed that and since people have to change their charts/configuration anyway, we prefer to run our own infrastructure, since it is something we know how to do and it will not add any additional inconvenience to people to just update the image to the full path registry.redict.io/redict.

mickenordin avatar May 08 '24 09:05 mickenordin

@mickenordin Any specific reason that redict container images are not uploaded to the docker hub ?

Yes, we discussed that and since people have to change their charts/configuration anyway, we prefer to run our own infrastructure, since it is something we know how to do and it will not add any additional inconvenience to people to just update the image to the full path registry.redict.io/redict.

we have a use-case of caching the image in our own hosted artifactory to avoid hitting public registry rate limiting, and unfortunately artifactory is only integrated with docker hub.

AmitKatyal-Sophos avatar May 08 '24 09:05 AmitKatyal-Sophos

I think you can add any compliant docker registry to artefactory: https://medium.com/@andrenit/docker-and-artifactory-as-a-registry-mirror-1365891b39c5

mickenordin avatar May 08 '24 09:05 mickenordin

We just released the Valkey container images, see the images in DockerHub

as well as the source code in GitHub (bitnami/valkey and bitnami/valkey-sentinel)

Any feedback is more than welcome, you can report any issue or suggestion at https://github.com/bitnami/containers/issues/new/choose

Now it's time for the Helm chart which is being developed at https://github.com/bitnami/charts/pull/25643

carrodher avatar May 08 '24 14:05 carrodher