diff options
| author |  Juanli Shen <aspirinsjl@gmail.com> | 2018-02-23 14:07:01 -0800 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2018-02-23 14:07:01 -0800 |
| commit | 5739d34c469f7ffb5f74cbc8308b182d3fad3664 (patch) | |
| tree | ef7d40c78bb6f5ccd775803eb6c3f6bb37a9546f /test | |
| parent | dc46c7f565196d6684d5cf6ee81e99b413ba5282 (diff) | |
| parent | acdefd764686a926f6c858af4a933f3b2b32f8cf (diff) | |
Merge pull request #14497 from AspirinSJL/fix_rpc_failure_flake
Add warmup before first batch in UpdateBalancers tests
Diffstat (limited to 'test')
| -rw-r--r-- | test/cpp/end2end/grpclb_end2end_test.cc | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/test/cpp/end2end/grpclb_end2end_test.cc b/test/cpp/end2end/grpclb_end2end_test.cc index 25e108e1a1..eb354907f6 100644 --- a/test/cpp/end2end/grpclb_end2end_test.cc +++ b/test/cpp/end2end/grpclb_end2end_test.cc @@ -892,7 +892,10 @@ TEST_F(UpdatesTest, UpdateBalancers) { ScheduleResponseForBalancer( 1, BalancerServiceImpl::BuildResponseForBackends(second_backend, {}), 0); - // Start servers and send 10 RPCs per server. + // Wait until the first backend is ready. + WaitForBackend(0); + + // Send 10 requests. gpr_log(GPR_INFO, "========= BEFORE FIRST BATCH =========="); CheckRpcSendOk(10); gpr_log(GPR_INFO, "========= DONE WITH FIRST BATCH =========="); @@ -954,7 +957,10 @@ TEST_F(UpdatesTest, UpdateBalancersRepeated) { ScheduleResponseForBalancer( 1, BalancerServiceImpl::BuildResponseForBackends(second_backend, {}), 0); - // Start servers and send 10 RPCs per server. + // Wait until the first backend is ready. + WaitForBackend(0); + + // Send 10 requests. gpr_log(GPR_INFO, "========= BEFORE FIRST BATCH =========="); CheckRpcSendOk(10); gpr_log(GPR_INFO, "========= DONE WITH FIRST BATCH =========="); |