Skip to content

Increase libraries run timeouts from default 150 to 300 minutes.#114722

Merged
BruceForstall merged 1 commit intodotnet:mainfrom
BruceForstall:IncreaseSpmiCollectTimeouts
Apr 16, 2025
Merged

Increase libraries run timeouts from default 150 to 300 minutes.#114722
BruceForstall merged 1 commit intodotnet:mainfrom
BruceForstall:IncreaseSpmiCollectTimeouts

Conversation

@BruceForstall
Copy link
Contributor

osx-arm64 queues have been timing out due to too many jobs sent to too few machines, by the superpmi-collect pipeline.

osx-arm64 queues have been timing out due to too many jobs sent to too
few machines, by the superpmi-collect pipeline.
Copilot AI review requested due to automatic review settings April 15, 2025 23:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

unifiedArtifactsName: BuildArtifacts_$(osGroup)$(osSubgroup)_$(archType)_Checked
helixArtifactsName: LibrariesTestArtifacts_$(osGroup)$(osSubgroup)_$(archType)_Checked
unifiedBuildConfigOverride: checked
# Default timeout is 150 minutes, which is too low for osx-arm64 queue.
Copy link

Copilot AI Apr 15, 2025

Choose a reason for hiding this comment

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

Both sections set the timeout to 300 minutes for the osx-arm64 queue. Consider extracting the timeout value to a shared variable or parameter to avoid duplication and ensure consistency in future updates.

Copilot uses AI. Check for mistakes.
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@BruceForstall BruceForstall requested a review from a team April 15, 2025 23:43
@BruceForstall
Copy link
Contributor Author

Work around #114715

@BruceForstall
Copy link
Contributor Author

This allowed the superpmi-collect pipeline libraries tests on osx-arm64 to complete. @dotnet/jit-contrib PTAL

@BruceForstall
Copy link
Contributor Author

/ba-g unrelated failures

@BruceForstall BruceForstall merged commit 267465c into dotnet:main Apr 16, 2025
87 of 92 checks passed
@BruceForstall BruceForstall deleted the IncreaseSpmiCollectTimeouts branch April 16, 2025 17:38
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants