Skip to content

test(platform): raise FT recovery waits from 90s to 300s - #6864

Merged
gz merged 1 commit into
mainfrom
ft-recovery-timeout
Aug 14, 2026
Merged

test(platform): raise FT recovery waits from 90s to 300s#6864
gz merged 1 commit into
mainfrom
ft-recovery-timeout

Conversation

@gz

@gz gz commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

A force-stop/start cycle can reschedule the pipeline pod onto another Kubernetes node, forcing its storage volume to detach and reattach. Under attach/detach contention the reattach alone can exceed 90s, which intermittently timed out test_ft_input_replay_determinism even though replay determinism held: deployment_error stayed unset through every cycle, and the pod events showed a 2m35s volume-attach gap on the failing cycle.

The determinism assertion runs on every 0.5s poll, so the longer wait adds infrastructure headroom without weakening the regression check. Normal cycles recover in seconds; only outlier cycles wait longer.

A force-stop/start cycle can reschedule the pipeline pod onto another
Kubernetes node, forcing its storage volume to detach and reattach.
Under attach/detach contention the reattach alone can exceed 90s, which
intermittently timed out test_ft_input_replay_determinism even though
replay determinism held (deployment_error stayed unset through every
cycle).

The determinism assertion runs on every 0.5s poll, so the longer wait
adds infrastructure headroom without weakening the regression check.

Signed-off-by: Gerd Zellweger <mail@gerdzellweger.com>
@gz
gz requested a review from mihaibudiu August 14, 2026 17:03

@mihaibudiu mihaibudiu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hope they don't charge for this

@gz
gz added this pull request to the merge queue Aug 14, 2026
Merged via the queue into main with commit 61b52de Aug 14, 2026
1 check passed
@gz
gz deleted the ft-recovery-timeout branch August 14, 2026 18:44
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