-
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
[CI] Test failure from System.Net.Http.Functional.Tests.SocketsHttpHandlerTest_Cookies_Http3.GetAsync_DefaultCoookieContainer_NoCookieSent
#101567
Comments
Tagging subscribers to this area: @dotnet/ncl |
Should be a dupe of #91757 |
Ah ok. I think the main difference is #91757 is using ErrorPattern but the pattern doesn't look like a regex, so it wouldn't have matched the ErrorMessage string in this issue. |
Actually nevermind, it seems like that should have matched the error message, not sure why it wouldn't have. |
Oh, it's a different test it seems 🙈 |
Oh is the test taken into account somehow? I thought it was just based on the message/pattern. |
Hmmm you're right. The column with the test name confused me. Then it should have been matched -- as the build in question is present in the table... 🤔 |
The build analysis in my PR #101501 did not recognize this as a known issue (at least in attempt 3, though I saw the same failure in the first two). I had rerun it again. |
I mean in attempt 3 that was the only failure that the build analysis pointed out as unknown, so that's a bit puzzling. |
@CarnaViire, can we close this issue because you merged #101569? |
@JulieLeeMSFT I'd close this one as it should be all matched to #91757 -- in case it wouldn't happen, can you please tag someone from the "Known Issues" team so they can help troubleshoot? As for #101569 -- we need first to verify that the timeouts went away or at least subsided (after the change propagates). And even then, there's going to be quite an untrivial investigation. So at least #91757 would be open as a tracking issue for the investigation. |
Duplicate of #91757 |
could you please provide more details on the expectation of this known issue? like sharing a build that you were expecting this issue would match I compared with the other known issue that you shared but they have a different error message, this one has an ErrorMessage and the other issue has an ErrorPattern which is slightly different so it is naturally that the match different builds, but if you were expecting a particular build please share it with me so I can take a look |
Here's some info about the failure that wasn't matched to #91757:
Even though #91757 is using ErrorPattern, treating it as a regex the expectation was that it would still match the error message in the console log: |
I'm not sure how to access the build analysis from previous build attempts. If there's a way, the 3rd attempt would show the failure as some that is not a known failure. |
As @kouvel also mentioned above, the regex in #91757 is |
Build Information
Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=655031
Build error leg or test failing: Build / Libraries Test Run release coreclr windows x64 Debug / Send to Helix
Pull request: #101501
Error Message
Fill the error message using step by step known issues guidance.
Known issue validation
Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=655031
Error message validated:
[System.Net.Quic.QuicException : The connection timed out from inactivity.
]Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 4/25/2024 8:21:21 PM UTC
Report
Summary
The text was updated successfully, but these errors were encountered: