-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use new Hosted build pools #58484
Use new Hosted build pools #58484
Conversation
Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer Issue DetailsWe are transitioning all servicing builds to a new set of build pools
|
Same as #57785 |
@@ -55,11 +55,11 @@ jobs: | |||
testDisplayName: ${{ parameters.runtimeFlavor }}_interpreter_${{ parameters.liveRuntimeBuildConfig }} | |||
|
|||
# To run the tests we just send to helix and wait, use ubuntu hosted pools for faster providing and to not back up our build pools | |||
${{ if startsWith(parameters.pool.queue, 'BuildPool.Ubuntu') }}: | |||
${{ if startsWith(parameters.pool.queue, 'Build.Ubuntu') }}: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I agree with this sentiment. I favor as many machines as can be of the 200 parallelism limit be macOS since ostensibly 1ES can handle these but not MacOS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is not correct anyway, as the new pool
property is demands
and it doesn't start with Build.Ubuntu
.
pool.queue
no longer exists after this change. So we can safely delete this as now are a no-op.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks correct, not sure I want the wait on helix jobs to use hosted but don't feel strongly either.
cc @danmoseley (tell-mode) |
This reverts commit 0e71ebc.
This reverts commit 0e71ebc.
We are transitioning all servicing builds to a new set of build pools