Skip to content

Add configurable worker liveness probe with HTTP option#87

Merged
RobertKeyser merged 3 commits intomainfrom
rkeyser/worker-health-checks
Apr 10, 2026
Merged

Add configurable worker liveness probe with HTTP option#87
RobertKeyser merged 3 commits intomainfrom
rkeyser/worker-health-checks

Conversation

@RobertKeyser
Copy link
Copy Markdown
Contributor

@RobertKeyser RobertKeyser commented Apr 10, 2026

Description of Changes

Summary
Adds useHttpProbe toggle to switch worker liveness probes from the default celery inspect ping exec command to an HTTP GET against the in-process health check server (port 9000). The exec probe spawns a full Celery CLI process on every check, which may lead to issues when scaling workers.

Exposes initialDelaySeconds, periodSeconds, timeoutSeconds, and failureThreshold as configurable values under the workerConfiguration key. These were previously hardcoded or only partially configurable via fides.startupTimeSeconds / fides.healthCheckTimeoutSeconds.
Fully backwards compatible — defaults match current behavior.

Pre-merge Checklist

  • All CI Pipelines Succeeded
  • Documentation Updated
  • Increment Applicable Chart Versions
  • Relevant Follow-Up Issues Created
  • Update the Fides chart CHANGELOG.md
  • Update the Fides-minimal chart CHANGELOG.md

@github-actions github-actions bot added documentation Improvements or additions to documentation helm/fides The Fides Helm Chart labels Apr 10, 2026
@RobertKeyser RobertKeyser changed the title Rkeyser/worker health checks Add configurable worker liveness probe with HTTP option Apr 10, 2026
@RobertKeyser RobertKeyser marked this pull request as ready for review April 10, 2026 13:11
@RobertKeyser RobertKeyser requested a review from a team as a code owner April 10, 2026 13:11
@RobertKeyser RobertKeyser self-assigned this Apr 10, 2026
@RobertKeyser RobertKeyser merged commit e8f1251 into main Apr 10, 2026
2 checks passed
@RobertKeyser RobertKeyser deleted the rkeyser/worker-health-checks branch April 10, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation helm/fides The Fides Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants