Skip to content
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

Update internal macOS image in xplat-setup.yml #107871

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Conversation

akoeplinger
Copy link
Member

The old label was replaced with macos-latest-internal.

The old label was replaced with macos-latest-internal.
Copy link
Contributor

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

Copy link
Member

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

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

I assume this runs in the regular CI? No need to run a separate azp run to test this?

@akoeplinger
Copy link
Member Author

It runs in official builds only but this is a no-op since they have been silently redirecting the old label to the new one since last week: https://dev.azure.com/mseng/AzureDevOps/_wiki/wikis/AzureDevOps.wiki/40632/macos-13-arm64-migration-to-macos-latest-internal

This is just to prevent issues once they remove the old label and redirection.

@@ -175,7 +175,7 @@ jobs:
# Official build OSX pool
${{ if and(in(parameters.osGroup, 'osx', 'maccatalyst', 'ios', 'iossimulator', 'tvos', 'tvossimulator'), ne(variables['System.TeamProject'], 'public')) }}:
name: "Azure Pipelines"
vmImage: 'macos-13-arm64'
vmImage: 'macos-latest-internal'
Copy link
Member

Choose a reason for hiding this comment

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

These have limited disk. Are we OK with this?

Copy link
Member Author

@akoeplinger akoeplinger Sep 17, 2024

Choose a reason for hiding this comment

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

They actually have enough space and we've been silently running on this pool for the last week. It was just the testing pool that had limited space.

@akoeplinger akoeplinger merged commit 06f0b37 into main Sep 17, 2024
146 of 150 checks passed
@akoeplinger akoeplinger deleted the akoeplinger-patch-1 branch September 17, 2024 08:19
jtschuster pushed a commit to jtschuster/runtime that referenced this pull request Sep 17, 2024
The old label was replaced with macos-latest-internal.
sirntar pushed a commit to sirntar/runtime that referenced this pull request Sep 30, 2024
The old label was replaced with macos-latest-internal.
@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2024
Sign up for free to 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.

4 participants