-
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
Failures in SocketsHttpHandler_HttpClientHandler_MaxResponseHeadersLength_Http3 #73930
Comments
Tagging subscribers to this area: @dotnet/ncl Issue DetailsHit in https://github.com/dotnet/runtime/pull/72145/checks?check_run_id=7832255512
Cc @MihaZupan
|
Tagging subscribers to this area: @dotnet/ncl Issue DetailsHit in https://github.com/dotnet/runtime/pull/72145/checks?check_run_id=7832255512
Cc @MihaZupan
|
The error is H3_REQUEST_CANCELLED (0x010c), @MihaZupan could this be recently broken by #73907? |
I don't see any failures in Kusto -- perhaps they didn't propagate yet? let failedTests = (testNameSubstring : string, methodName : string, messageSubstr: string, includePR : bool, includePassedOnRerun : bool) {
cluster('engsrvprod.kusto.windows.net').database('engineeringdata').AzureDevOpsTests
| where TestName contains testNameSubstring
| where includePassedOnRerun or (Outcome == 'Failed')
| extend startOfTestName = indexof_regex(TestName, @"[^.]+$")
| extend Method = substring(TestName, startOfTestName)
| extend Type = substring(TestName, 0, startOfTestName - 1)
| project-away startOfTestName
| where (methodName == '') or (Method == methodName)
| where Message contains messageSubstr
| distinct JobId, WorkItemId, Message, StackTrace, Method, Type, Arguments, Outcome
| join kind=inner (cluster('engsrvprod.kusto.windows.net').database('engineeringdata').Jobs
| where ((Branch == 'refs/heads/main') or (Branch == 'refs/heads/master') or (includePR and (Source startswith "pr/")))
| where Type startswith "test/functional/cli/"
and not(Properties contains "runtime-staging")
| where Branch <> 'refs/pull/73374/merge'
| summarize arg_max(Finished, Properties, Type, Branch, Source, Started, QueueName) by JobId
| project-rename JobType = Type) on JobId
| extend PropertiesJson = parse_json(Properties)
| extend OS = replace_regex(tostring(PropertiesJson.operatingSystem), @'\((.*)\).*|([^\(].*)', @'\1\2')
| extend Runtime = iif(PropertiesJson.runtimeFlavor == "mono", "Mono", iif(PropertiesJson.DefinitionName contains "coreclr", "CoreCLR", ""))
| extend TargetBranch = extractjson("$.['System.PullRequest.TargetBranch']", Properties)
| extend Architecture = PropertiesJson.architecture
| extend Scenario = iif(isempty(PropertiesJson.scenario), "--", PropertiesJson.scenario)
//| extend DefinitionName = PropertiesJson.DefinitionName
| project-away PropertiesJson
};
failedTests(
'', //testNameSubstring
'SocketsHttpHandler_HttpClientHandler_MaxResponseHeadersLength_Http3', //methodName
'', //messageSubstr
true, //includePR
true); //includePassedOnRerun |
100% failure in the two pull requests I was involved in today: https://github.com/dotnet/runtime/pull/72145/checks?check_run_id=7832255512 |
Hit issues on the PR (#73937 (comment)). Disabling the test for H3 until we resolve that: #73945 |
Triage: Let's try to re-enable the test in main to see if we get still hits. |
I have local repro and the test still fails. I see no point of trying it in |
Full log: https://helixre107v0xdeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-pull-72145-merge-c270b69ee20f4e269e/System.Net.Http.Functional.Tests/3/console.142a186b.log?helixlogtype=result
Hit in https://github.com/dotnet/runtime/pull/72145/checks?check_run_id=7832255512
Cc @MihaZupan
The text was updated successfully, but these errors were encountered: