Skip to content

[release/6.0] bump sourcebuild leg timeout - #63298

Merged
safern merged 3 commits into
release/6.0from
updateSourceBuildTimeout60
Jan 5, 2022
Merged

[release/6.0] bump sourcebuild leg timeout#63298
safern merged 3 commits into
release/6.0from
updateSourceBuildTimeout60

Conversation

@safern

Copy link
Copy Markdown
Member

This leg sometimes hits timeout on the official build as the 60 mins default timeout is too tight to how long it takes to finish.

This is a port of: #61436

@safern
safern requested review from a team and danmoseleyJanuary 3, 2022 17:57
@ghostghost assigned safernJan 3, 2022
@ghost

ghost commented Jan 3, 2022

Copy link
Copy Markdown

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@safern

Copy link
Copy Markdown
MemberAuthor

@agocke could you help me looking at the installer failures, they are completely unrelated and the first time I see them.

@ghost

ghost commented Jan 4, 2022

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

This leg sometimes hits timeout on the official build as the 60 mins default timeout is too tight to how long it takes to finish.

This is a port of: #61436

Author:safern
Assignees:safern
Labels:

area-Infrastructure

Milestone:-

@agocke

Copy link
Copy Markdown
Member

Investigating now

* Up retry for ETXTBSY
Still seeing occasional failure in Alpine
(https://dev.azure.com/dnceng/public/_build/results?buildId=1434759) so
raising the retry time.
* Linearly scale wait time
(cherry picked from commit 6c09b7b)
@agocke

Copy link
Copy Markdown
Member

Looks like this was a race condition that was fixed after the 6.0 snap. There's no direct relation to this PR -- it's just chance. I've backported the commits that include the fixes anyway.

@safern

Copy link
Copy Markdown
MemberAuthor

Thanks, @agocke

@safern

Copy link
Copy Markdown
MemberAuthor

Failures are known networking tests, I'm looking at why they are running as they're supposed to be disabled.

@safern
safern merged commit 51b6749 into release/6.0Jan 5, 2022
@safern
safern deleted the updateSourceBuildTimeout60 branch January 5, 2022 19:08
@ghostghost locked as resolved and limited conversation to collaborators Feb 5, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@safern@agocke@steveisok@jeffhandley@ericstj