dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.47k stars 4.76k forks source link

CI timeout in System.Data.Tests.DataSetTest2.Bug537229_BinFormatSerializer_Test #103350

Open amanasifkhalid opened 5 months ago

amanasifkhalid commented 5 months ago

Build Information

Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=704655 Build error leg or test failing: System.Data.Common.Tests.WorkItemExecution Pull request: https://github.com/dotnet/runtime/pull/103319

Error Message

Fill the error message using step by step known issues guidance.

{
  "ErrorMessage": [
    "[Long Running Test] 'System.Data.Tests.DataSetTest2.Bug537229_BinFormatSerializer_Test'",
    "Command timed out, and was killed"
  ],
  "BuildRetry": false,
  "ExcludeConsoleLog": false
}

Known issue validation

Build: :mag_right: https://dev.azure.com/dnceng-public/public/_build/results?buildId=704655 Error message validated: [[Long Running Test] 'System.Data.Tests.DataSetTest2.Bug537229_BinFormatSerializer_Test' Command timed out, and was killed] Result validation: :white_check_mark: Known issue matched with the provided build. Validation performed at: 6/12/2024 4:02:55 PM UTC

Report

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 0
dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to this area: @roji, @ajcvickers See info in area-owners.md if you want to be subscribed.

SamMonoRT commented 5 months ago

@eiriktsarpalis - is this something you can take a quick look at?

eiriktsarpalis commented 5 months ago

Seems to be an old test using BinaryFormatter. Perhaps related to BF being phased out?

cc @bartonjs @adamsitnik

bartonjs commented 5 months ago

It seems to be using BinaryFormatter directly, not via System.Resources.Extensions, so Adam's changes wouldn't be affecting it. And mine haven't gone in yet. So, unless the test is psychic and knows BinaryFormatter will be turned off in the next day or two, it seems coincidental.

That said, when I merge in (today or tomorrow) this test will be disabled. No tests will be able to use BinaryFormatter until we get a package published, just because of how our build system works and how we chose to solve the BF-legacy problem, so it will at least stop showing up as an intermittent CI failure.

SamMonoRT commented 5 months ago

Assigning to @bartonjs for tracking.

jeffschwMSFT commented 4 months ago

removing blocking-clean-ci as it has not failed in 30 days.

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 0
SamMonoRT commented 2 months ago

@bartonjs - this is still marked for 9.0.0 -- please take a look and adjust milestone if necessary, or can we close the issue?