charts icon indicating copy to clipboard operation
charts copied to clipboard

[chart/redis-ha]support configuration for redis container command parameter

Open GDegrove opened this issue 2 years ago • 2 comments

Is your feature request related to a problem? Please describe.

I'm trying to deploy redis-ha using redis-stack-server as the image.

From what I can see, the current helm chart only start the container with redis-server. However, when using the redis-stack-server you want to start the contianer with /entrypoint.sh (don't know why the startup script is name entrypoint though...) or using the redis-stack-server.

Describe the solution you'd like

Add a configuration for the helm chart to contain a way to override/configure the command. Instead of using the hard-coded edis-server, we may want to let user chose. This would allow a developer to easily change the configuration of the helm chart without spending too much time on making sure their image is compatible with it.

Describe alternatives you've considered

  • make a change in the upstream image
  • download module with an init container

GDegrove avatar Jun 05 '23 13:06 GDegrove

seems that would be resolved by #238

GDegrove avatar Jun 05 '23 13:06 GDegrove

Yes. #238 would do this, it was neglected. I'll re-review to see if this is definitely fit for purpose and try to get this merged in.

DandyDeveloper avatar Feb 05 '24 17:02 DandyDeveloper