Quarantine Re-quarantine Http2ConnectionTests.IgnoreNewStreamsDuringClosedConnection

[github-actions]

Test was unquarantined in #66083 on 2026-04-02 (original issue #60111) but has failed since.

Failing Test(s)

Microsoft.AspNetCore.Server.Kestrel.Core.Tests.Http2ConnectionTests.IgnoreNewStreamsDuringClosedConnection

Error Message

Assert.Equal() Failure: Values differ
Expected: ConnectionReset
Actual:   null

Stacktrace

Details

at Microsoft.AspNetCore.Server.Kestrel.Core.Tests.Http2TestBase.AssertConnectionEndReason(ConnectionEndReason expectedEndReason)
  in /_/src/Servers/Kestrel/test/InMemory.FunctionalTests/Http2/Http2TestBase.cs:line 435
at Microsoft.AspNetCore.Server.Kestrel.Core.Tests.Http2ConnectionTests.IgnoreNewStreamsDuringClosedConnection()
  in /_/src/Servers/Kestrel/test/InMemory.FunctionalTests/Http2/Http2ConnectionTests.cs:line 5453

Logs

Details

[0.009s] TestLifetime Information: Starting test IgnoreNewStreamsDuringClosedConnection
[0.029s] Http2 Trace: sending SETTINGS frame for stream ID 0 with length 24 and flags NONE.
[0.030s] Http2 Trace: sending WINDOW_UPDATE frame for stream ID 0 with length 4 and flags 0x0.
[0.034s] Http2 Trace: received SETTINGS frame for stream ID 0 with length 6 and flags NONE.
[0.035s] Http2 Trace: sending SETTINGS frame for stream ID 0 with length 0 and flags ACK.
[0.037s] Http2 Debug: Connection id "TestConnectionId" is closed. The last processed stream ID was 0.
[0.038s] Http2 Debug: The connection queue processing loop for TestConnectionId completed.
[0.051s] Http2ConnectionTests Error: Test threw an exception.
Xunit.Sdk.EqualException: Assert.Equal() Failure: Values differ
Expected: ConnectionReset
Actual:   null

Build

https://dev.azure.com/dnceng-public/public/_build/results?buildId=1364809

Note

🔒 Integrity filter blocked 12 items

The following items were blocked because they don't meet the GitHub integrity level.

• #65815 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65817 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65827 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65816 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #66130 issue_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #66135 issue_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #40626 search_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #46028 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #49286 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #54846 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #55092 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #55707 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

Generated by Daily Test Quarantine Management · ● 25.1M · ◷

[github-actions]

Investigation

This test was unquarantined on 2026-04-02 via PR #66083 (issue #60111). It failed again in build 1364809.

Failure pattern: Assert.Equal() Failure: Expected: ConnectionReset, Actual: null in AssertConnectionEndReason.

Root cause: The test verifies that IgnoreNewStreamsDuringClosedConnection correctly records the connection end reason as ConnectionReset. The failure indicates the ConnectionEndReason field is null when checked — the connection closed without setting this reason, or the assignment happened after the assertion was evaluated.

The race: The test closes the connection while new streams are being received. In the failing case, the connection is closed so quickly that the ConnectionReset reason is never set — the connection terminates via a different path (e.g., clean close) before the reset reason is recorded.

Recommendation: Review Http2Connection.ProcessRequestsAsync() and specifically the StopProcessingNextRequest() → connection end reason assignment flow to ensure ConnectionReset is always set when the connection is terminated during active stream processing. This may require synchronization between the connection close path and the stream processing path.

Note

🔒 Integrity filter blocked 12 items

The following items were blocked because they don't meet the GitHub integrity level.

• #65815 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65817 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65827 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #65816 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #66130 issue_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #66135 issue_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #40626 search_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #46028 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #49286 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #54846 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #55092 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
• #55707 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

Generated by Daily Test Quarantine Management · ● 25.1M · ◷