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

CoreFX tests sometimes time out during PR validation builds #1487

Closed
dagood opened this issue Feb 18, 2020 · 6 comments
Closed

CoreFX tests sometimes time out during PR validation builds #1487

dagood opened this issue Feb 18, 2020 · 6 comments
Assignees
Labels
area-testing Improvements in CI and testing

Comments

@dagood dagood added the area-testing Improvements in CI and testing label Feb 18, 2020
@adaggarwal
Copy link
Member

I have also observed the same in CI. However, I don't see that whenever I try it locally.

@dseefeld dseefeld self-assigned this Feb 28, 2020
@adaggarwal
Copy link
Member

I saw this in one of the timeout cases

Resolving SDK 'Microsoft.NET.Sdk.IL'...

[SourceBuiltSdkResolver] Looking for SDK Microsoft.NET.Sdk.IL. Detected config(s) in env:
'ARCADE_CONFIGURATION' for 'Microsoft.DotNet.Build.Tasks.Configuration/1.0.0-beta.20113.5' at '/src/Tools/source-built/Microsoft.DotNet.Build.Tasks.Configuration/'
'IL' for 'Microsoft.NET.Sdk.IL/1.0.0-source-built' at '/src/Tools/source-built/Microsoft.NET.Sdk.IL/'
'ARCADE_COREFX_TESTING' for 'Microsoft.DotNet.CoreFxTesting/1.0.0-beta.20113.5' at '/src/Tools/source-built/Microsoft.DotNet.CoreFxTesting/'
'ARCADE' for 'Microsoft.DotNet.Arcade.Sdk/1.0.0-beta.20113.5' at '/src/Tools/source-built/Microsoft.DotNet.Arcade.Sdk/'

[SourceBuiltSdkResolver] Overriding Microsoft.NET.Sdk.IL with 'IL'

Error when reading the file: System.IO.EndOfStreamException: Unable to read beyond the end of the stream.
   at System.IO.__Error.EndOfFile()
   at System.IO.BinaryReader.ReadString()
   at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.ReadBuildEventArgsFields() in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 628
   at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.ReadBuildMessageEventArgs() in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 463
   at Microsoft.Build.Logging.StructuredLogger.BuildEventArgsReader.Read() in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BuildEventArgsReader.cs:line 95
   at Microsoft.Build.Logging.StructuredLogger.BinLogReader.Replay(Stream stream) in C:\MSBuildStructuredLog\src\StructuredLogger\BinaryLogger\BinLogReader.cs:line 78```

@dagood
Copy link
Member Author

dagood commented Mar 9, 2020

I believe that error is caused by an incomplete binlog, it's not indicating any build problem. So from that info it would seem the CI hangs (or dies?) after our SDK resolver replaces the IL SDK.

@adaggarwal
Copy link
Member

I believe that error is caused by an incomplete binlog, it's not indicating any build problem. So from that info it would seem the CI hangs (or dies?) after our SDK resolver replaces the IL SDK.

Yep. And each timeout occurs at this point, which is interesting.

@dseefeld
Copy link
Contributor

dseefeld commented Mar 12, 2020

That's the point where it runs the corefx tests, I think. On a successful build, after that line, it runs for about 4 minutes:

2020-02-27T18:22:22.0647587Z       SOURCE_BUILT_SDK_DIR_IL=/src/Tools/source-built/Microsoft.NET.Sdk.IL/
2020-02-27T18:26:52.3198683Z   [18:26:52.31] Building 'corefx'...done

On a failing build, it takes about 2 hours and then times out:

2020-02-16T06:48:22.1658504Z       SOURCE_BUILT_SDK_DIR_IL=/src/Tools/source-built/Microsoft.NET.Sdk.IL/
2020-02-16T08:41:48.2414019Z ##[error]The task has timed out.

@MichaelSimons
Copy link
Member

[Triage] CoreFX tests are no longer integrated into source-build. The unified build work for .NET 8.0 and 9.0 will make it possible to run any repo's unit tests against source-build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing Improvements in CI and testing
Projects
None yet
Development

No branches or pull requests

4 participants