apache / apache/pinot

Support helm chart server separate liveness and readiness probe endpoints

Open Beginner friendly
#11,798 1 comment 0 reactions 0 assignees View on GitHub
enhancement kubernetes stale
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
1d 21h
Merged PRs (30d)
189

Description

### Issue
Latest pinot helm chart doesn't support separate liveness and readiness probe endpoint for server ([link](https://github.com/apache/pinot/blob/master/helm/pinot/values.yaml#L311-L314)) (both use `server.probes.endpoint`) but the service underneath supports it ([link](https://github.com/apache/pinot/pull/9031)) .

### Proposal
Add supports for `server.probes.livenessEndpoint` and `server.probes.readinessEndpoint` to helm chart server sts template, but make it backward compatible by falling back to the existing `server.probes.endpoint`.

Contributor guide

Open the contributing guide

Research direction

Start with helm/pinot/values.yaml around the existing server.probes.endpoint setting, then locate the Helm chart server StatefulSet (STS) template. Add separate livenessEndpoint and readinessEndpoint values while preserving fallback to the existing endpoint. Done means the generated probes use the separate values when set and remain backward compatible when they are absent.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm
Domain
devops
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.