Skip to content

Commit

Permalink
[test] increase timeout (grpc#36308)
Browse files Browse the repository at this point in the history
This fixes rare failure under MSAN. This does not increate the test run time:

MSAN build:
```
//test/cpp/interop:backend_metrics_lb_policy_test@poller=poll            PASSED in 59.2s
  Stats over 5000 runs: max = 59.2s, min = 4.9s, avg = 6.5s, dev = 2.8s
```

Opt/no MSAN:
```
//test/cpp/interop:backend_metrics_lb_policy_test@poller=poll            PASSED in 26.7s
  Stats over 5000 runs: max = 26.7s, min = 4.9s, avg = 7.7s, dev = 2.8s
```

Closes grpc#36308

COPYBARA_INTEGRATE_REVIEW=grpc#36308 from eugeneo:333396468-msan-backend_metrics_lb_policy_test 4ad9443
PiperOrigin-RevId: 623250444
  • Loading branch information
eugeneo authored and copybara-github committed Apr 9, 2024
1 parent 4feb98e commit 493d873
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/cpp/interop/backend_metrics_lb_policy_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ TEST(BackendMetricsLbPolicyTest, TestOobMetricsReceipt) {
// This report is sent on start, available immediately
auto report = tracker.WaitForOobLoadReport(
[](auto report) { return report.cpu_utilization() == 0.5; },
absl::Milliseconds(1500), 3);
absl::Seconds(5) * grpc_test_slowdown_factor(), 3);
ASSERT_TRUE(report.has_value());
EXPECT_EQ(report->cpu_utilization(), 0.5);
for (size_t i = 0; i < 3; i++) {
Expand Down

0 comments on commit 493d873

Please sign in to comment.