xds: force-enable retries in xds_end2end_test to make things work internally (#28564)

markdroth · web-flow · commit e59dcd5c876b · 2022-01-14T09:34:39.000-08:00
diff --git a/test/cpp/end2end/xds/xds_end2end_test.cc b/test/cpp/end2end/xds/xds_end2end_test.cc
@@ -913,6 +913,8 @@ class XdsEnd2endTest : public ::testing::TestWithParam<TestType> {
       const char* xds_authority = "",
       grpc_channel_args* xds_channel_args = nullptr) {
     ChannelArguments args;
+    // TODO(roth): Remove this once we enable retries by default internally.
+    args.SetInt(GRPC_ARG_ENABLE_RETRIES, 1);
     if (failover_timeout > 0) {
       args.SetInt(GRPC_ARG_PRIORITY_FAILOVER_TIMEOUT_MS, failover_timeout);
     }

Original file line number	Diff line number	Diff line change
`@@ -913,6 +913,8 @@ class XdsEnd2endTest : public ::testing::TestWithParam<TestType> {`
`913`	`913`	`const char* xds_authority = "",`
`914`	`914`	`grpc_channel_args* xds_channel_args = nullptr) {`
`915`	`915`	`ChannelArguments args;`
	`916`	`+ // TODO(roth): Remove this once we enable retries by default internally.`
	`917`	`+ args.SetInt(GRPC_ARG_ENABLE_RETRIES, 1);`
`916`	`918`	`if (failover_timeout > 0) {`
`917`	`919`	`args.SetInt(GRPC_ARG_PRIORITY_FAILOVER_TIMEOUT_MS, failover_timeout);`
`918`	`920`	`}`