Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15168


Change subject: [util] remove timeout parameter for cloud::InstanceDetector
......................................................................

[util] remove timeout parameter for cloud::InstanceDetector

With this patch, the detector relies solely on the timeouts reported
by metadata fetchers and doesn't set the timeout on the auto-detection
process itself.  This change simplifies the semantics of the cloud
instance detection.

In addition, and this patch fixes intermittent failures recently
seen in the following test scenarios:
  * InstanceDetectorTest.Basic
  * BuiltinNtpWithMiniChronydTest.CloudInstanceNtpServer

Change-Id: I6f7cac9273e884abda22b9e9a7d37fa3e307c2d4
---
M src/kudu/clock/ntp-test.cc
M src/kudu/util/cloud/instance_detector-test.cc
M src/kudu/util/cloud/instance_detector.cc
M src/kudu/util/cloud/instance_detector.h
M src/kudu/util/cloud/instance_metadata.cc
M src/kudu/util/cloud/instance_metadata.h
6 files changed, 73 insertions(+), 46 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/15168/1
--
To view, visit http://gerrit.cloudera.org:8080/15168
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6f7cac9273e884abda22b9e9a7d37fa3e307c2d4
Gerrit-Change-Number: 15168
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to