Skip to content

[release/8.0-staging][infra][android] Fix Android helix queue name#107023

Merged
matouskozak merged 1 commit intodotnet:release/8.0-stagingfrom
matouskozak:infra/net8-fix-android-emulator-queue-name
Sep 9, 2024
Merged

[release/8.0-staging][infra][android] Fix Android helix queue name#107023
matouskozak merged 1 commit intodotnet:release/8.0-stagingfrom
matouskozak:infra/net8-fix-android-emulator-queue-name

Conversation

@matouskozak
Copy link
Member

The Android emulator queue name was changed to Ubuntu.2004.Amd64.Android.29.Open in #103393 but there is no such queue on helix, causing the Android x64 jobs to fail: error : Helix API does not contain an entry for Ubuntu.2004.Amd64.Android.29.Open

Changing to Ubuntu.2204.Amd64.Android.29.Open.

@dotnet-policy-service
Copy link
Contributor

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

@matouskozak
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

approved. we can treat this one as tell mode.

cc @carlossanlop

@jeffschwMSFT jeffschwMSFT added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 29, 2024
@rbhanda rbhanda added this to the 8.0.10 milestone Aug 29, 2024
@carlossanlop
Copy link
Contributor

@matouskozak can you please look at the CI failures? There are several without a linked issue, so they must be new/related to this PR.

Friendly reminder that Code Complete for the October Release is September 9. If we want this fix to be included in that release, please merge this PR before that date.

@matouskozak
Copy link
Member Author

@matouskozak can you please look at the CI failures? There are several without a linked issue, so they must be new/related to this PR.

Friendly reminder that Code Complete for the October Release is September 9. If we want this fix to be included in that release, please merge this PR before that date.

The BuildAnalysis checks runtime-extra-platforms on servicing branches? I thought only runtime pipeline is checked for BA to be green.

I'll go over them and link the issues.

@carlossanlop
Copy link
Contributor

I think BA does check runtime-extra-platforms, but in release branches, the BuildAnalysis check will always show as failed to ensure the PR submitters look at their failures and confirm they are all indeed unrelated (we want to minimize risk).

@richlander
Copy link
Member

richlander commented Sep 6, 2024

Sorry that my change broke Android!

@matouskozak
Copy link
Member Author

No problem @richlander.

Looking at the failures related to this PR, this change only affected android-x64 and linux-bionic-x64 jobs of which all of them are green (previously failing). Did a quick check and the "other" failures are also seen on rolling-builds of release/8.0-staging https://dev.azure.com/dnceng-public/public/_build/results?buildId=799223. I think it is good to merge @carlossanlop.

@matouskozak
Copy link
Member Author

/ba-g the runtime-extra-platforms are unrelated as they are occurring on different than android-x64 jobs and are also present on rolling-builds

@matouskozak matouskozak merged commit 88d07e4 into dotnet:release/8.0-staging Sep 9, 2024
@matouskozak matouskozak deleted the infra/net8-fix-android-emulator-queue-name branch October 3, 2024 13:13
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 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.

5 participants