-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 Failed: GetAsync_SetCookieContainer_CookieSent failed #12043
Comments
@karelz Could you take a look? |
@wfurt to help |
The test was disabled in dotnet/coreclr#22697. |
|
This test is now having a pretty significant impact on the build. If we can't fix we should look to disable soon. Console Log Summary
Builds
Configurations
Helix LogsSystem.Net.Http.Functional.Tests.SocketsHttpHandlerTest_Cookies_Http2.GetAsync_SetCookieContainer_CookieSentConsole Log SummaryBuilds
Configurations
Helix Logs
|
ping @dotnet/ncl |
Looks like it had a recent spike ... on WinHttpHandler. @alnikola can you please take a look? |
I'm investigating this since yesterday, this is the same issue as here: #33930 |
Unless @alnikola wants to take over or immediately know where the problem is, I'd like to tackle this, since I've already put some time into it. |
@karelz |
Do we have an ETA for a fix or disable? This test is the #1 source of CI reliability issues for runtime for last two days. If the fix is not imminent can we disable ASAP? |
@ManickaP great thanks! |
Ah it just got merged an hour ago hence no rolling builds have completed since the merge. That's why I'm still seeing it as atop issue. Should fade out now. |
Duplicate of #33930 |
Job: (JitStress - arm64)
https://ci.dot.net/job/dotnet_coreclr/job/master/view/arm64/job/jitstress/job/arm64_cross_checked_ubuntu16.04_corefx_tieredcompilation_tst/122/
Failed tests:
System.Net.Http.Functional.Tests.SocketsHttpHandlerTest_Cookies_Http2.GetAsync_SetCookieContainer_CookieSent
Log:
The text was updated successfully, but these errors were encountered: