Skip to content

feat(probes): add health probes to all data plane components#226

Merged
fernando-villalba merged 1 commit intomainfrom
probes
Feb 12, 2026
Merged

feat(probes): add health probes to all data plane components#226
fernando-villalba merged 1 commit intomainfrom
probes

Conversation

@fernando-villalba
Copy link
Collaborator

Components (multigateway, multipooler, multiorch) had no liveness or readiness probes, causing Kubernetes to treat them as always healthy even when dependencies like the topo initialization wasn't done.

  • Add HTTP liveness (/live) and readiness (/ready) probes to multigateway in multigateway.go on port 15100
  • Add HTTP probes to multipooler in containers.go on port 15200
  • Add HTTP probes to multiorch in containers.go on port 15300
  • Set InitialDelaySeconds to 60 for both probe types to allow time for topo server and other dependencies to become available
  • Add IgnoreProbeDefaults() helper to compare.go for ignoring API server-applied probe defaults (Scheme, TimeoutSeconds, etc.)
  • Update all unit and integration test expectations to include the new probe configurations

Prevents premature container restarts and provides accurate readiness signaling during cluster bootstrap.

Components (multigateway, multipooler, multiorch) had no liveness or
readiness probes, causing Kubernetes to treat them as always healthy
even when dependencies like the topo initialization wasn't done.

- Add HTTP liveness (/live) and readiness (/ready) probes to
  multigateway in multigateway.go on port 15100
- Add HTTP probes to multipooler in containers.go on port 15200
- Add HTTP probes to multiorch in containers.go on port 15300
- Set InitialDelaySeconds to 60 for both probe types to allow
  time for topo server and other dependencies to become available
- Add IgnoreProbeDefaults() helper to compare.go for ignoring
  API server-applied probe defaults (Scheme, TimeoutSeconds, etc.)
- Update all unit and integration test expectations to include
  the new probe configurations

Prevents premature container restarts and provides accurate
readiness signaling during cluster bootstrap.
@github-actions
Copy link

🔬 Go Test Coverage Report

Summary

Coverage Type Result
Threshold 0%
Previous Test Coverage Unknown%
New Test Coverage Unknown%

Status

⚠️ SKIPPED (Build Failed?)

Detail

Show New Coverage
No coverage report generated (Build likely failed).

@fernando-villalba fernando-villalba merged commit 52a7f10 into main Feb 12, 2026
1 of 3 checks passed
@fernando-villalba fernando-villalba deleted the probes branch February 12, 2026 20:42
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.

1 participant