Skip to content

Commit 2924d19

Browse files
krwqradical
andauthored
[wasm][aot] Disable (again) Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests failing due to OOM (#61323)
* [wasm] Disable tests failing due to OOM Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests #61339 * [wasm] Use special exitcodes when AOT test build fails, and when it gets oom-killed Co-authored-by: Ankit Jain <[email protected]>
1 parent d39d805 commit 2924d19

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

eng/testing/WasmRunnerAOTTemplate.sh

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,22 @@ function _buildAOTFunc()
3838
dotnet msbuild $binLog -clp:PerformanceSummary -v:q -nologo
3939
if [[ "$(uname -s)" == "Linux" && $buildExitCode -ne 0 ]]; then
4040
echo "\nLast few messages from dmesg:\n"
41-
dmesg | tail -n 20
41+
local lastLines=`dmesg | tail -n 20`
42+
echo $lastLines
43+
44+
if [[ "$lastLines" =~ "oom-kill" ]]; then
45+
return 9200 # OOM
46+
fi
4247
fi
4348

4449
echo
4550
echo
4651

47-
return $buildExitCode
52+
if [[ $buildExitCode -ne 0 ]]; then
53+
return 9100 # aot build failure
54+
fi
55+
56+
return 0
4857
}
4958

5059
# RunCommands defined in tests.mobile.targets

src/libraries/tests.proj

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
</PropertyGroup>
1515

1616
<ItemGroup Condition="'$(TargetOS)' == 'Browser' and '$(BuildAOTTestsOnHelix)' == 'true' and '$(RunDisabledWasmTests)' != 'true' and '$(RunAOTCompilation)' == 'true'">
17+
<!-- Exceeds VM resources in CI on compilation: https://github.com/dotnet/runtime/issues/61339 -->
18+
<ProjectExclusions Include="$(MSBuildThisFileDirectory)Microsoft.Extensions.Logging.Abstractions\tests\Microsoft.Extensions.Logging.Generators.Tests\Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests.csproj" />
1719
</ItemGroup>
1820

1921
<!-- Projects that don't support code coverage measurement. -->

0 commit comments

Comments
 (0)