[chart/redis-ha][REQUEST] - Extend redis.config and sentinel.config about charts

[DandyDeveloper]

I've added some comments in your PR to confirm some things. Get back to me when you can!

[fveerden]

When will this be merged, as it is now impossible to add more than one SAVE instruction in the redis.config

[DandyDeveloper]

@fveerden Sorry, I completely missed @SarjakSShah response to my original suggestions.

Right now, I'm torn between what options we offer here without bringing more unnecessary complexity into the chart.

I'd like to give users the option of:

1. Use the default configmap
2. Provide your own existing configmap to replace the default

@SarjakSShah is effectively suggesting we instead add an append option, which I'm not against, but I'm worried will bring me more work to resolve in the future when people expect existing config options to be replaced / duplicates start appearing and I have to offer option 2 anyway to resolve those kinds of issues.

[fveerden]

Well, the nice thing about redis.config.* settings is that you can easily override a setting in a secondary values.yaml. We now use it to override the maxmemory setting for the production environment. Like this:

redis:
  config:
    maxmemory: "12gb"

The drawback of redis.customConfig seems that is overrules the redis.config.* settings and you have to copy all config if you want to override just one.
Or is there another way to have a default redis config and override the maxmemory if necessary?

[DandyDeveloper]

Closing for housekeeping as there's not a lot of movement on this right now. I'm happy to discuss this further if the conversation still needs to happen.