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

[Test Failure] Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent #96484

Closed
rzikm opened this issue Jan 4, 2024 · 5 comments
Closed
Labels
area-System.Net.Http blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab
Milestone

Comments

@rzikm
Copy link
Member

rzikm commented Jan 4, 2024

Build Information

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=513214
Build error leg or test failing: System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent
Pull request:

Error Message

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

{
  "ErrorMessage": "Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent [FAIL]",
  "BuildRetry": false,
  "ExcludeConsoleLog": true
}

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=513214
Error message validated: Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent [FAIL]
Result validation: ❌ Known issue did not match with the provided build.
Validation performed at: 1/4/2024 2:22:38 PM UTC

Report

Build Definition Test Pull Request
517705 dotnet/runtime System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent
517177 dotnet/runtime System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent
516782 dotnet/runtime System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent
516007 dotnet/runtime System.Net.Http.Functional.Tests.TelemetryTest_Http30.EventSource_ConnectionPoolAtMaxConnections_LogsRequestLeftQueue
514581 dotnet/runtime System.Net.Http.Functional.Tests.SocketsHttpHandler_PostScenarioTest.PostNonRewindableContentUsingAuth_PreAuthenticate_Success
514565 dotnet/runtime System.Net.Http.Functional.Tests.HttpClientHandler_RemoteServerTest.GetAsync_CredentialIsCredentialCacheUriRedirect_StatusCodeOK
514559 dotnet/runtime System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 7 7
@rzikm rzikm added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab labels Jan 4, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 4, 2024
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jan 4, 2024
@vcsjones vcsjones added area-System.Net.Http and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 4, 2024
@ghost
Copy link

ghost commented Jan 4, 2024

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

Issue Details

Build Information

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=513214
Build error leg or test failing: System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent
Pull request:

Error Message

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

{
  "ErrorMessage": "Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent [FAIL]",
  "BuildRetry": false,
  "ExcludeConsoleLog": false
}

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=513214
Error message validated: Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent [FAIL]
Result validation: ❌ Known issue did not match with the provided build.
Validation performed at: 1/4/2024 2:22:38 PM UTC

Report

Build Definition Test Pull Request
514581 dotnet/runtime System.Net.Http.Functional.Tests.SocketsHttpHandler_PostScenarioTest.PostNonRewindableContentUsingAuth_PreAuthenticate_Success
514565 dotnet/runtime System.Net.Http.Functional.Tests.HttpClientHandler_RemoteServerTest.GetAsync_CredentialIsCredentialCacheUriRedirect_StatusCodeOK
514559 dotnet/runtime System.Net.Http.Functional.Tests.SyncHttpHandler_HttpClientHandler_Cancellation_Test.Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
3 3 3
Author: rzikm
Assignees: -
Labels:

area-System.Net.Http, blocking-clean-ci, untriaged, Known Build Error

Milestone: -

@ManickaP ManickaP added this to the 9.0.0 milestone Jan 5, 2024
@ManickaP ManickaP removed the untriaged New issue has not been triaged by the area owner label Jan 5, 2024
@CarnaViire
Copy link
Member

CarnaViire commented Jan 5, 2024

Triage: Outerloop test, fails a bit randomly from time to time, on different platforms.

6 occurrences in the past 30 days (2023-12-06 to 2024-01-05)

Table
Date Test Class Platform
2023-12-08 SyncHttpHandler_HttpClientHandler_Cancellation_Test Windows.11.Amd64.Client.Open
2023-12-11 SocketsHttpHandler_HttpClientHandler_Cancellation_Test_Http2 Ubuntu.2204.Arm64.Open
2023-12-16 SocketsHttpHandler_HttpClientHandler_Cancellation_Test_Http2 Alpine.318.Arm64.Open
2023-12-20 SocketsHttpHandler_HttpClientHandler_Cancellation_Test_Http2 Ubuntu.2204.Arm64.Open
2023-12-29 SyncHttpHandler_HttpClientHandler_Cancellation_Test Alpine.318.Arm64.Open
2024-01-03 SocketsHttpHandler_HttpClientHandler_Cancellation_Test_Http2 Windows.11.Amd64.Client.Open

First occurrence on 2023-09-16.

We need to investigate in 9.0.

@CarnaViire
Copy link
Member

CarnaViire commented Jan 5, 2024

@AlitzelMendez since you've been looking at KnownIssues behavior in #96481, can you please also take a look at this one?

In the report table of linked build failures, for some reason, some completely unrelated tests are linked. The expected test should be Expect100Continue_WaitsExpectedPeriodOfTimeBeforeSendingContent only, but the linked failed tests are EventSource_ConnectionPoolAtMaxConnections_LogsRequestLeftQueue, PostNonRewindableContentUsingAuth_PreAuthenticate_Success, GetAsync_CredentialIsCredentialCacheUriRedirect_StatusCodeOK... 🤔

How does this happen? What is the correct way to aggregate all the failures for a specific test name? If it's not "ErrorMessage": "<test_name> [FAIL]", then what should it be?

@AlitzelMendez
Copy link
Member

it's because it finds the error on the helix console log, meaning that probably other test is failing but belongs to the same job so it finds the machine on the helix log: https://github.com/dotnet/arcade/blob/main/Documentation/Projects/Build%20Analysis/KnownIssueJsonStepByStep.md#how-to-create-a-known-issue-step-by-step

you can change: "ExcludeConsoleLog": false to "ExcludeConsoleLog": true, and this would fix the scenario

@rzikm
Copy link
Member Author

rzikm commented Jan 9, 2024

Duplicate of #58628

@rzikm rzikm marked this as a duplicate of #58628 Jan 9, 2024
@rzikm rzikm closed this as not planned Won't fix, can't repro, duplicate, stale Jan 9, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Net.Http blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab
Projects
None yet
Development

No branches or pull requests

5 participants