Skip to content

Give bash test completion waits a CI-safe budget - #977

Queued
aft-alfonso[bot] wants to merge 2 commits into
mainfrom
land/bash-progress-test-wait
Queued

aft-alfonso[bot] wants to merge 2 commits into
mainfrom
land/bash-progress-test-wait

Conversation

@aft-alfonso

@aft-alfonso aft-alfonso Bot commented Oct 26, 2026

Copy link
Copy Markdown

Test-only. bash_streams_progress_and_returns_final_response failed on main's Windows integration shard 1/3 (run 40296340355): echo hello through PowerShell had not completed within the test's 5 s wait on a loaded runner. The test checks what a finished task reports, not shell start latency. Its completion wait and the two other completion waits in bash_test.rs now share a 30 s budget and print the last status when they expire, so a task that never finishes still fails with a diagnosis.

ualtinok and others added 2 commits October 26, 2026 10:27
bash_streams_progress_and_returns_final_response failed on main's Windows
integration shard (run 40296340355): 'echo hello' through PowerShell had not
finished within the test's 5 s wait on a loaded runner. The test checks what a
finished task reports, not shell start latency, so its completion wait (and the
two others in the file) now share a 30 s budget and report the last status
when they expire.

Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
Co-authored-by: alfonso-aft <289616620+alfonso-aft@users.noreply.github.com>
@ualtinok
ualtinok added this pull request to the merge queue Oct 26, 2026
Any commits made after this event will not be merged.
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