diff options
author | 2018-03-23 11:08:23 -0700 | |
---|---|---|
committer | 2018-03-23 11:08:23 -0700 | |
commit | 9e4d954077db20b259ae2edd3ac61f2c3f5c7d61 (patch) | |
tree | ccdc62b5ba35af88355f8e3a523fa9b70c345ef8 | |
parent | 90af9346de2e4e6f0d45b130ce0de3d08d075c3f (diff) | |
parent | 236bec52579f6cd32924883b885c0f2a1f6d2087 (diff) |
Merge pull request #14780 from muxi/fix-keepalive
Keepalive watchdog timer used a wrong timeout value
-rw-r--r-- | src/core/ext/transport/chttp2/transport/chttp2_transport.cc | 2 | ||||
-rw-r--r-- | test/core/end2end/tests/keepalive_timeout.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/core/ext/transport/chttp2/transport/chttp2_transport.cc b/src/core/ext/transport/chttp2/transport/chttp2_transport.cc index a4d616d778..bd588e3c9b 100644 --- a/src/core/ext/transport/chttp2/transport/chttp2_transport.cc +++ b/src/core/ext/transport/chttp2/transport/chttp2_transport.cc @@ -2630,7 +2630,7 @@ static void start_keepalive_ping_locked(void* arg, grpc_error* error) { grpc_chttp2_transport* t = static_cast<grpc_chttp2_transport*>(arg); GRPC_CHTTP2_REF_TRANSPORT(t, "keepalive watchdog"); grpc_timer_init(&t->keepalive_watchdog_timer, - grpc_core::ExecCtx::Get()->Now() + t->keepalive_time, + grpc_core::ExecCtx::Get()->Now() + t->keepalive_timeout, &t->keepalive_watchdog_fired_locked); } diff --git a/test/core/end2end/tests/keepalive_timeout.cc b/test/core/end2end/tests/keepalive_timeout.cc index f0dd061e33..c0771b8bd2 100644 --- a/test/core/end2end/tests/keepalive_timeout.cc +++ b/test/core/end2end/tests/keepalive_timeout.cc @@ -102,7 +102,7 @@ static void test_keepalive_timeout(grpc_end2end_test_config config) { grpc_arg keepalive_arg_elems[3]; keepalive_arg_elems[0].type = GRPC_ARG_INTEGER; keepalive_arg_elems[0].key = const_cast<char*>(GRPC_ARG_KEEPALIVE_TIME_MS); - keepalive_arg_elems[0].value.integer = 1500; + keepalive_arg_elems[0].value.integer = 3500; keepalive_arg_elems[1].type = GRPC_ARG_INTEGER; keepalive_arg_elems[1].key = const_cast<char*>(GRPC_ARG_KEEPALIVE_TIMEOUT_MS); keepalive_arg_elems[1].value.integer = 0; |