[ci] Fix nightly test environment provisioning - #8113
Merged
Conversation
Commit 475b912 improved Android SDK dependency provisioning steps by combining the install of emulator and non emulator related dependencies. These changes also moved emulator component provisioning to a step that includes preview .NET SDK provisioning. In the nightly build this step was running _after_ we installed our workload packages, and our workload install would be removed as part of SDK re-provisioning. This caused ~all of our nightly tests to fail with: /Users/runner/work/1/s/xamarin-android/bin/Release/dotnet/sdk/8.0.100-preview.6.23276.3/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.ImportWorkloads.targets(38,5): error NETSDK1147: To build this project, the following workloads must be installed: android or System.IO.DirectoryNotFoundException : Unable to locate a Microsoft.Android.Sdk in either '/Users/runner/work/1/s/xamarin-android/bin/Release/dotnet/packs' or '/Users/runner/work/1/s/xamarin-android/bin/Release/lib/packs'. Fix this by updating the nightly build to bring in the handful of recent improvements that were made to our to emulator test jobs. Emulator component provisioning will now happen _before_ .NET SDK and workload installation. The localization test job template has also been removed as it is no longer in use.
Member
Author
Member
Author
|
Probably still some reliability issues to work in individual tests out but this is working as expected. |
jpobst
approved these changes
Jun 7, 2023
grendello
added a commit
to grendello/xamarin-android
that referenced
this pull request
Jun 12, 2023
* main: LEGO: Merge pull request 8119 [build] Bump `$(XABuildToolsVersion)`=34 (dotnet#8118) [Xamarin.Android.Build.Tasks] missing resource error handling (dotnet#8066) [CI] Allow MSBuild test stages to run in the megapipeline (dotnet#8033) [ci] Fix nightly test environment provisioning (dotnet#8113)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Commit 475b912 improved Android SDK dependency provisioning steps by
combining the install of emulator and non emulator related dependencies.
These changes also moved emulator component provisioning to a step that
includes preview .NET SDK provisioning. In the nightly build this step
was running after we installed our workload packages, and our workload
install would be removed as part of SDK re-provisioning. This caused
~all of our nightly tests to fail with:
or
Fix this by updating the nightly build to bring in the handful of recent
improvements that were made to our to emulator test jobs. Emulator
component provisioning will now happen before .NET SDK and workload
installation.
The localization test job template has also been removed as it is no
longer in use.