search

DandyDeveloper / charts

Various helm charts migrated from [helm/stable] due to deprecation
https://dandydeveloper.github.io/charts
Apache License 2.0
157 stars 145 forks source link

[chart/redis-ha][BUG] Cannot specify resources for split-brain-fix conainter #153

Closed artur-reaboi closed 3 years ago

artur-reaboi commented 3 years ago

Describe the bug Cannot specify resources for the new split-brain-fix container, part of redis-ha-statefulset. This is required to be able to deploy Redis into namespaces with defined resource quotas.

Expected behavior A way to define container resources for split-brain-fix container.

DandyDeveloper commented 3 years ago

@artur-reaboi Adding now.

zerkms commented 3 years ago

Right, sorry I initially didn't add it: it consumes little to no resources and I don't run tightly restricted k8s clusters.

DandyDeveloper commented 3 years ago

@zerkms No worries. Just an fyi incase your chart uses the old snake case syntax.