Skip to content

[release/7.0][PERF][MAUI] Update Maui build Mac vmimage to macos-12 - #76973

Merged
carlossanlop merged 1 commit into
dotnet:release/7.0from
LoopedBard3:7.0PerfMauiUpdateMacVMIMage
Oct 13, 2022
Merged

[release/7.0][PERF][MAUI] Update Maui build Mac vmimage to macos-12#76973
carlossanlop merged 1 commit into
dotnet:release/7.0from
LoopedBard3:7.0PerfMauiUpdateMacVMIMage

Conversation

@LoopedBard3

@LoopedBard3LoopedBard3 commented Oct 12, 2022

Copy link
Copy Markdown
Member

Backport of: #76945 and only makes changes to perf testing yml. Updates the Maui builds to use vmImage mac-12 instead of mac-11 in order to gain access to XCode 14 when building.

Customer Impact

None, this only impacts the performance infra.

Testing

Tested on the main version of the change: https://dev.azure.com/dnceng/internal/_build/results?buildId=2018712&view=results

Risk

Low
The only impact from this will be for the performance infra.

@LoopedBard3LoopedBard3 self-assigned this Oct 12, 2022
@ghostghost added the area-Infrastructure-coreclr Only use for closed issues label Oct 12, 2022
@ghost

Copy link
Copy Markdown

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

Issue Details

Backport of: #76945 and only makes changes to perf testing yml. Updates the Maui builds to use vmImage mac-12 instead of mac-11 in order to gain access to XCode 14 when building.

Author:LoopedBard3
Assignees:LoopedBard3
Labels:

area-Infrastructure-coreclr

Milestone:-

@LoopedBard3

Copy link
Copy Markdown
MemberAuthor

@danmoseley Is there a preferred process for getting infrastructure changes like this merged into servicing branches? Thanks.

@hoyosjshoyosjs left a comment

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.

Our official and most of our other builds are on macOS-11. Not sure if you care if that might give you some difference.

@carlossanlop

carlossanlop commented Oct 13, 2022

Copy link
Copy Markdown
Contributor

No need for Tactics approval (tell-mode). For this coreclr infra change, I'd like to ask @hoyosjs for a sign-off.

Edit: Ah, he did it before I sent this message 😄 . Merging now.

@carlossanlop
carlossanlop merged commit aacc243 into dotnet:release/7.0Oct 13, 2022
@LoopedBard3
LoopedBard3 deleted the 7.0PerfMauiUpdateMacVMIMage branch October 13, 2022 18:05
@ghostghost locked as resolved and limited conversation to collaborators Nov 12, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@LoopedBard3@carlossanlop@cincuranet@hoyosjs