-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[Android] Free up more disk space on CI builds #84354
Conversation
The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built.
Tagging subscribers to this area: @directhex Issue DetailsThe android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built.
|
/azp run runtime-android |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-android |
091ab81
to
2541c05
Compare
2541c05
to
2a801bc
Compare
Comment was made before the most recent commit for PR 84354 in repo dotnet/runtime |
/azp run runtime-android |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -14,6 +14,8 @@ | |||
</ItemGroup> | |||
|
|||
<ItemGroup> | |||
<ProjectReference Include="..\..\..\TestAssets\StartupHookForFunctionalTest\StartupHookForFunctionalTest.csproj" /> | |||
<ProjectReference Include="..\..\..\TestAssets\StartupHookForFunctionalTest\StartupHookForFunctionalTest.csproj"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem trivial to get working at the moment, should we just disable this test so we can atleast have coverage for all other tests?
/azp run runtime-android |
Azure Pipelines successfully started running 1 pipeline(s). |
/backport to release/7.0-staging |
Started backporting to release/7.0-staging: https://github.com/dotnet/runtime/actions/runs/4623465416 |
/backport to release/6.0-staging |
Started backporting to release/6.0-staging: https://github.com/dotnet/runtime/actions/runs/4623467456 |
@steveisok backporting to release/7.0-staging failed, the patch most likely resulted in conflicts: $ git am --3way --ignore-whitespace --keep-non-patch changes.patch
Applying: [Android] Free up more disk space on CI builds
Applying: Fix startuphook test.
Using index info to reconstruct a base tree...
A src/tests/FunctionalTests/Android/Device_Emulator/StartupHook/Android.Device_Emulator.StartupHook.Test.csproj
Falling back to patching base and 3-way merge...
Auto-merging src/tests/FunctionalTests/Android/Device_Emulator/InvariantCultureOnlyMode/Android.Device_Emulator.InvariantCultureOnlyMode.Test.csproj
CONFLICT (content): Merge conflict in src/tests/FunctionalTests/Android/Device_Emulator/InvariantCultureOnlyMode/Android.Device_Emulator.InvariantCultureOnlyMode.Test.csproj
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 Fix startuphook test.
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128 Please backport manually! |
@steveisok an error occurred while backporting to release/7.0-staging, please check the run log for details! Error: git am failed, most likely due to a merge conflict. |
@steveisok backporting to release/6.0-staging failed, the patch most likely resulted in conflicts: $ git am --3way --ignore-whitespace --keep-non-patch changes.patch
Applying: [Android] Free up more disk space on CI builds
Using index info to reconstruct a base tree...
A eng/testing/tests.android.targets
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): eng/testing/tests.android.targets deleted in HEAD and modified in [Android] Free up more disk space on CI builds. Version [Android] Free up more disk space on CI builds of eng/testing/tests.android.targets left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 [Android] Free up more disk space on CI builds
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128 Please backport manually! |
@steveisok an error occurred while backporting to release/6.0-staging, please check the run log for details! Error: git am failed, most likely due to a merge conflict. |
backport of dotnet#84354 The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built.
backport of dotnet#84354 The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built.
backport of #84354 The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built. Co-authored-by: Steve Pfister <[email protected]>
backport of #84354 The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built. Co-authored-by: Steve Pfister <[email protected]>
The android builds are running out of disk space when building the library test apps. This change tries to recoup some of that space by deleting artifacts after each test was built.
Addresses dotnet/arcade#13036