test: reduce Go test timeout to 80m to stay below 90m ADO job limit#8395
Merged
test: reduce Go test timeout to 80m to stay below 90m ADO job limit#8395
Conversation
The Go test timeout (90m) was equal to the ADO job timeout (90m), leaving zero buffer. When tests take the full duration, ADO kills the job before Go can finish, resulting in no test output and silent failures (e.g. windows-23H2-gen2 in build 161489045). Reducing to 80m ensures Go finishes and flushes test results before ADO cancels the job, giving 10 minutes of buffer for cleanup and artifact upload. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
r2k1
requested review from
AbelHu,
Devinwong,
SriHarsha001,
awesomenix,
calvin197,
cameronmeissner,
djsly,
ganeshkumarashok,
junjiezhang1997,
lilypan26,
mxj220,
pdamianov-dev,
phealy,
sulixu,
surajssd,
timmy-wright and
zachary-bailey
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
Adjusts the default Go E2E test timeout used by the Azure DevOps pipeline runner script so the test process terminates and flushes results before the 90-minute job timeout kills the agent, improving debuggability and artifact reliability.
Changes:
- Reduce default
E2E_GO_TEST_TIMEOUTfrom90mto80min the E2E pipeline run script.
r2k1
changed the title
rchincha
approved these changes
Apr 24, 2026
djsly
approved these changes
Apr 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The Go test timeout (
E2E_GO_TEST_TIMEOUT=90m) equals the ADO job timeout (timeoutInMinutes: 90), leaving zero buffer.ADO kills the job before Go can finish, resulting in:
Fix
Reduce default Go test timeout from 90m → 80m, giving 10 minutes of buffer for Go to finish, flush results, and upload artifacts before ADO cancels.
Impact