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

[ci] Optimize 'APK's .NET' test job overhead. #7904

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Mar 21, 2023

Optimize overhead for APK's .NET ci job:

  • Use new start-stop-emulator.yaml template to start the emulator before the tests begin. If the emulator isn't working, then each test step tries to run, waits 10 minutes, and then errors out (see screenshot below). Instead, let's just error out completely at the beginning if the emulator isn't available.
  • Bump emulator timeout to 15 minutes (from 10 minutes). Using public Mac bots can be slow, maybe giving them a little extra time will result in more successful builds.
  • Call Xamarin.Android.sln -t:PrepareJavaInterop with --no-restore to prevent it from running restore on the entire solution.
  • Don't waste time installing unneeded dependencies:
    • Legacy dotnet
    • Mono
    • NUnit.Console
    • Extra Android SDK platforms

image

@jpobst jpobst force-pushed the optimize-apk-tests branch 2 times, most recently from 0acafaf to 483ff2c Compare March 21, 2023 16:27
@jpobst jpobst marked this pull request as ready for review March 22, 2023 22:54
@jonpryor jonpryor merged commit 7da9e23 into main Mar 23, 2023
@jonpryor jonpryor deleted the optimize-apk-tests branch March 23, 2023 16:52
grendello added a commit to grendello/xamarin-android that referenced this pull request Mar 24, 2023
* main:
  [Xamarin.Android.Build.Tasks] Fix native code generation when marshal methods are disabled (dotnet#7899)
  [ci] Optimize 'APK's .NET' test job overhead. (dotnet#7904)
  [Mono.Android] delay JNINativeWrapper.get_runtime_types() (dotnet#7913)
  Bump external/Java.Interop from `73ebad2` to `53bfb4a` (dotnet#7914)
grendello added a commit to grendello/xamarin-android that referenced this pull request Apr 6, 2023
* main: (94 commits)
  [ci] Remove remaining Classic test jobs. (dotnet#7924)
  Add Nightly Tests for Humanizer
  [readme] Add aka.ms links for d17.5. (dotnet#7935)
  [tests] Remove `net472` multitargeting (dotnet#7932)
  [monodroid] Fix `ld` build error on Nightly Builds. (dotnet#7925)
  Bump to xamarin/Java.Interop/main@0355acf (dotnet#7931)
  [tests] Use msftconnecttest.com in QuoteInvalidQuoteUrlsShouldWork (dotnet#7919)
  [ci] Don't set demands for megapipeline (dotnet#7929)
  [Mono.Android] Bind API-UpsideDownCake Developer Preview 1 (dotnet#7796)
  Bump to dotnet/installer@d109cba3ff 8.0.100-preview.4.23176.5 (dotnet#7921)
  [Xamarin.Android.Build.Tasks] Fix Android Version Code for Release builds (dotnet#7795)
  Bump to dotnet/installer@27d6769dfb 8.0.100-preview.4.23172.16 (dotnet#7910)
  [Xamarin.Android.Build.Tasks] Fix native code generation when marshal methods are disabled (dotnet#7899)
  [ci] Optimize 'APK's .NET' test job overhead. (dotnet#7904)
  [Mono.Android] delay JNINativeWrapper.get_runtime_types() (dotnet#7913)
  Bump external/Java.Interop from `73ebad2` to `53bfb4a` (dotnet#7914)
  [build] remove .NET 6 support (dotnet#7900)
  [profiled-aot] update `dotnet.aotprofile` for .NET 8 (dotnet#7908)
  [tests] Add backup ssl sites in case of 429 response. (dotnet#7909)
  $(AndroidPackVersionSuffix)=preview.4; net8 is 34.0.0-preview.4 (dotnet#7912)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants