DandyDeveloper / charts

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

[chart/redis-ha] Make exporter probes optional #228

Closed sonrai-doyle closed 5 months ago

sonrai-doyle commented 2 years ago

What this PR does / why we need it:

This adds the option to remove the livenessProbe and readinessProbe from the exporter container.

Which issue this PR fixes

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

DandyDeveloper commented 2 years ago

@sonrai-doyle May be better here to do a toYaml templating for the probes that way if people want to remove it they just add the following to the values file:

livenessProbe: {}
DandyDeveloper commented 2 years ago

@sonrai-doyle Any way you can update / give input on my comment? Will close soon if I don't hear from you.,

sonrai-doyle commented 2 years ago

Sorry @DandyDeveloper missed your comment while I was away on vacation. Will look at making the suggested change as I can.

mhkarimi1383 commented 1 year ago

+1 Waiting for this, sometimes Redis wants to recover, and we want exporter to be enabled (without probes) since it will get timeouts, etc. and makes entire pod to be unavailable.

sonrai-doyle commented 1 year ago

@DandyDeveloper Sorry for the delay in addressing your comments. Would you be able to re-review?

mhkarimi1383 commented 1 year ago

@sonrai-doyle But that would be better if we can make all of the probes optional (some times we want to disable probes for all of the containers for debugging, etc.)