more explicit conversion / scaling

1 files changed, 5 insertions, 3 deletions

diff --git a/configd/src/apps/sentinel/outward-check.cpp b/configd/src/apps/sentinel/outward-check.cpp
index f9a39cbec8a..613dd736d38 100644
--- a/configd/src/apps/sentinel/outward-check.cpp
+++ b/configd/src/apps/sentinel/outward-check.cpp
@@ -9,7 +9,7 @@ namespace config::sentinel {
 
 OutwardCheckContext::~OutwardCheckContext() = default;
 
-OutwardCheck::OutwardCheck(const std::string &spec, OutwardCheckContext &context, int ping_timeout)
+OutwardCheck::OutwardCheck(const std::string &spec, OutwardCheckContext &context, int ping_timeout_ms)
   : _spec(spec),
     _context(context)
 {
@@ -18,8 +18,10 @@ OutwardCheck::OutwardCheck(const std::string &spec, OutwardCheckContext &context
     _req->SetMethodName("sentinel.check.connectivity");
     _req->GetParams()->AddString(context.targetHostname.c_str());
     _req->GetParams()->AddInt32(context.targetPortnum);
-    _req->GetParams()->AddInt32(ping_timeout);
-    _target->InvokeAsync(_req, 1.0 + ping_timeout*0.002, this);
+    _req->GetParams()->AddInt32(ping_timeout_ms);
+    double ping_s = ping_timeout_ms * 0.001;
+    double outer_timeout = 1.0 + (2 * ping_s);
+    _target->InvokeAsync(_req, outer_timeout, this);
 }
 
 OutwardCheck::~OutwardCheck() = default;