Skip to content

Conversation

@ninjatec
Copy link

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #367 format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

Checklist

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

  • DCO signed
  • [N ] Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

Copilot AI review requested due to automatic review settings November 12, 2025 14:17
Copilot finished reviewing on behalf of ninjatec November 12, 2025 14:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds configurable liveness and readiness probes for the split brain detection sidecar container in the Redis HA StatefulSet to improve monitoring and health checking capabilities.

Key Changes:

  • Added liveness and readiness probe configuration to splitBrainDetection section in values.yaml
  • Implemented probe definitions in the StatefulSet template for the split-brain-fix container

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
charts/redis-ha/values.yaml Added configuration parameters for liveness and readiness probes under splitBrainDetection section
charts/redis-ha/templates/redis-ha-statefulset.yaml Implemented liveness and readiness probes for the split-brain-fix container using the new configuration values

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DandyDeveloper
Copy link
Owner

@ninjatec This would be better done as a toYaml inside the manifest, so you don't need the enable:

  splitBrainDetection:
    livenessProbe:
        ...

Then:

  livenessProbe: {{ .Values.splitBrainDetection.livenessProbe | toYaml | nindent X }}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants