Skip to content

[release/7.0-rc1] Move all build pools to -Svc equivalents - #74521

Merged
carlossanlop merged 1 commit into
dotnet:release/7.0-rc1from
MattGal:move-builds-to-servicing
Aug 25, 2022
Merged

[release/7.0-rc1] Move all build pools to -Svc equivalents#74521
carlossanlop merged 1 commit into
dotnet:release/7.0-rc1from
MattGal:move-builds-to-servicing

Conversation

@MattGal

@MattGalMattGal commented Aug 24, 2022

Copy link
Copy Markdown
Member

(except eng/common; this needs to be updated in release branch of Arcade once generated)

This should greatly improve 7.0 RC1 PR speeds since the pools for -Svc are currently much less busy than main.

@MattGal
Move all build pools to -Svc equivalents (except eng/common; this nee…
150ddf8
…ds to be updated in release branch of Arcade once generated)
@ghostghost assigned MattGalAug 24, 2022
@MattGal
MattGal requested a review from hoyosjsAugust 24, 2022 20:42
@ghost

ghost commented Aug 24, 2022

Copy link
Copy Markdown

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

Issue Details

(except eng/common; this needs to be updated in release branch of Arcade once generated)

This should greatly improve 7.0 RC1 PR speeds since the pools for -Svc are currently much less busy than main.

Author:MattGal
Assignees:-
Labels:

area-Infrastructure-libraries

Milestone:-

carlossanlop
carlossanlop approved these changes Aug 24, 2022
View reviewed changes

@carlossanlopcarlossanlop left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you, @MatatGal

@MattGalMattGal mentioned this pull request Aug 24, 2022
Closed
31 tasks
@carlossanlopcarlossanlop changed the title Move all build pools to -Svc equivalents (7.0 RC1)[release/7.0-rc1] Move all build pools to -Svc equivalentsAug 25, 2022
@carlossanlopcarlossanlop mentioned this pull request Aug 25, 2022
Merged
akoeplinger
akoeplinger reviewed Aug 25, 2022
View reviewed changes

@akoeplingerakoeplinger left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, I wonder if we should extract the pool name into a variable in main so we could conditionally assign it to the Svc pool for all release/* builds and get this automatically once we branch for 8.0?

@carlossanlop
carlossanlop merged commit 7b1acce into dotnet:release/7.0-rc1Aug 25, 2022
@ghostghost locked as resolved and limited conversation to collaborators Sep 24, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@akoeplingerakoeplingerakoeplinger left review comments
@hoyosjshoyosjshoyosjs approved these changes
+1 more reviewer
@carlossanlopcarlossanlopcarlossanlop approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@MattGalMattGal

Labels

area-Infrastructure-libraries

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MattGal@carlossanlop@akoeplinger@hoyosjs