Skip to content

fix(helm): align Hindsight probes and worker caps #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
allyblockcast merged 2 commits into from
Jul 8, 2026
Merged

fix(helm): align Hindsight probes and worker caps #18

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9079cf5
fix(helm): reduce hindsight DB pressure during liveness
Jul 4, 2026
f205156
fix(helm): restore DB-aware readiness probes, fix shared slot starvation
Jul 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 13 additions & 2 deletions helm/hindsight/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,9 @@ api:
timeoutSeconds: 5
failureThreshold: 3

# readinessProbe hits /health (checks DB connectivity) rather than /version:
# unlike liveness, readiness must detect a DB outage so k8s pulls the pod
# from rotation instead of routing traffic to a pod that can't serve it.
readinessProbe:
httpGet:
path: /health
Expand Down Expand Up @@ -147,6 +150,9 @@ worker:
timeoutSeconds: 5
failureThreshold: 3

# readinessProbe hits /health (checks DB connectivity) rather than /metrics:
# unlike liveness, readiness must detect a DB outage so k8s pulls the pod
# from rotation instead of routing traffic to a pod that can't serve it.
readinessProbe:
httpGet:
path: /health
Expand All @@ -163,9 +169,14 @@ worker:
# Number of tasks to claim per poll cycle
HINDSIGHT_API_WORKER_BATCH_SIZE: "1"
# Bound per-worker task and DB write concurrency for the default embedded DB.
# Keep one shared slot available for reservation-less task types like retain.
HINDSIGHT_API_WORKER_MAX_SLOTS: "2"
# consolidation and retain each get a reserved slot; the remaining slot is
# shared by reservation-less task types (file_convert_retain,
# refresh_mental_model, graph_maintenance, import_documents). Keep
# MAX_SLOTS one greater than the sum of reservations below or the shared
# pool collapses to zero and those task types silently stop running.
HINDSIGHT_API_WORKER_MAX_SLOTS: "3"
HINDSIGHT_API_WORKER_CONSOLIDATION_MAX_SLOTS: "1"
HINDSIGHT_API_WORKER_RETAIN_MAX_SLOTS: "1"
HINDSIGHT_API_RETAIN_MAX_CONCURRENT: "1"
# Max retries before marking a task as failed
HINDSIGHT_API_WORKER_MAX_RETRIES: "3"
Expand Down