Skip to content

Debounce transient Kubernetes cluster-health failures#6782

Open
mterwill wants to merge 1 commit into
tilt-dev:masterfrom
mterwill:codex/verbose-health-checks
Open

Debounce transient Kubernetes cluster-health failures#6782
mterwill wants to merge 1 commit into
tilt-dev:masterfrom
mterwill:codex/verbose-health-checks

Conversation

@mterwill

@mterwill mterwill commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

#6742 changed tilt ci so that any cluster error immediately fails the CI session. The cluster error it relies on, however, is produced by a single health-check poll and cleared by the next successful poll.

Kubernetes evaluates health over a series of probes rather than treating one failed check as terminal. Its probe model uses a failureThreshold and only considers the overall check failed after successive failures. Tilt should apply similar tolerance before turning a point-in-time health observation into an irreversible CI result.

We see this most often on busy clusters with many concurrent CI jobs, where one brief control-plane interruption can cascade into failures across every active Tilt session.

This change requires health checks to fail continuously for one minute—five successive failed polls at the current 15-second interval—before reporting a cluster error. A successful check resets the timer immediately. Sustained failures still terminate CI, but one failed poll no longer amplifies a brief interruption into many permanent job failures.

@mterwill
mterwill force-pushed the codex/verbose-health-checks branch from c1822d0 to 0d475ed Compare June 17, 2026 22:12
@nicks

nicks commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

this pr is marked as draft -- is this something you're still working on?

@mterwill
mterwill force-pushed the codex/verbose-health-checks branch 3 times, most recently from 577bbc1 to ae720fa Compare July 24, 2026 14:24
@mterwill mterwill changed the title Enable verbose Kubernetes health checks Debounce transient Kubernetes cluster-health failures Jul 24, 2026
Signed-off-by: Matt Terwilliger <matt.terwilliger@reddit.com>
@mterwill
mterwill force-pushed the codex/verbose-health-checks branch from ae720fa to c9e4602 Compare July 24, 2026 14:43
@mterwill
mterwill marked this pull request as ready for review July 24, 2026 14:53
@mterwill

Copy link
Copy Markdown
Contributor Author

Thanks @nicks! The original branch was meant to add observability into a failure we were seeing internally. I’ve investigated further and reworked it around what I believe is the underlying issue.

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