Let us know how this works out. If my comment was accurate, and this works fine with the included dependency chart, we may look to directly documenting that. Perhaps, even consider removing the explicit recommendation that the in-chart dependency not be used for production
Interesting just found that after migration test instance to sentinel instance is unavailable (but all pods are ready). And in workhorse I have next logs:
@WarheadsSE It seems that it's still not ready for production usage.
Suddenly redis chart in sentinel configuration not performs failover before actual master pod shutdown. So even if you are gracefully stoping actual master pod you still needs to wait downAfterMilliseconds timeout. :(
Hmm i'm not actually sure. It seems that #1810 (closed) is not actual, cause gitlab could work with redis sentinel installed by chart dependency (with new verison of redis subchart).
So there're two problems:
Sentinel 127.0.0.1 master sometimes. It seems that could be solved by using latest version of redis chart.