[Flakiness] Fix flakiness in retry_transparent_max_concurrent_streams_test (#37561)

Refer b/333896115 for more details

Closes #37561

COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/37561 from yashykt:TransparentRetryFlakyFix 65b3e1283a
PiperOrigin-RevId: 668579926
pull/37597/head
Yash Tibrewal 5 months ago committed by Copybara-Service
parent 9a162e12d9
commit 2d99ca4e0b
  1. 4
      bazel/cc_grpc_library.bzl
  2. 5
      test/core/end2end/tests/retry_transparent_max_concurrent_streams.cc

@ -102,7 +102,7 @@ def cc_grpc_library(
generate_cc(
name = codegen_grpc_target,
srcs = proto_targets,
plugin = Label("//src/compiler:grpc_cpp_plugin"),
plugin = "@com_github_grpc_grpc//src/compiler:grpc_cpp_plugin",
well_known_protos = well_known_protos,
generate_mocks = generate_mocks,
**kwargs
@ -114,6 +114,6 @@ def cc_grpc_library(
hdrs = [":" + codegen_grpc_target],
deps = deps +
extra_deps +
[Label("//:grpc++_codegen_proto")],
["@com_github_grpc_grpc//:grpc++_codegen_proto"],
**kwargs
)

@ -110,7 +110,10 @@ CORE_END2END_TEST(RetryHttp2Test, RetryTransparentMaxConcurrentStreams) {
// Server should get the second call.
auto s2 = RequestCall(201);
Expect(201, true);
Step();
// Give enough time for the handshake to timeout, and a new handshake to start
// if needed. (b/333896115)
// TODO(yashykt): Remove the extra duration on fixing b/362326480.
Step(Duration::Seconds(30));
EXPECT_EQ(s2.method(), "/service/method");
// Make sure the "grpc-previous-rpc-attempts" header was NOT sent, since
// we don't do that for transparent retries.

Loading…
Cancel
Save