Skip to content

Conversation

radical
Copy link
Member

@radical radical commented Aug 15, 2025

…2Tests.Http2_RequestWithDataAndContentLength_Success

Issue: #63266

@Copilot Copilot AI review requested due to automatic review settings August 15, 2025 06:21
@radical radical requested review from wtgodbe and a team as code owners August 15, 2025 06:21
@github-actions github-actions bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Aug 15, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a retry mechanism for a flaky HTTP/2 test that has been experiencing intermittent failures. The change updates the test configuration to retry the Http2_RequestWithDataAndContentLength_Success test when it fails, helping to reduce false negatives in the CI pipeline.

Key changes:

  • Added a retry configuration for the failing HTTP/2 test
  • Updated Copilot instructions to document the quarantining process for flaky tests
  • Updated googletest submodule reference

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
eng/test-configuration.json Added retry configuration for the flaky HTTP/2 test
.github/copilot-instructions.md Added documentation about quarantining flaky tests with examples
src/submodules/googletest Updated submodule commit reference

…2Tests.Http2_RequestWithDataAndContentLength_Success
@radical radical merged commit 28bbfc3 into dotnet:main Aug 15, 2025
29 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-rc1 milestone Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants