Hello Yuqi Du, Yifan Zhang, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19478
to look at the new patch set (#2).
Change subject: [clock] output more info in SystemNtp::DumpDiagnostics()
......................................................................
[clock] output more info in SystemNtp::DumpDiagnostics()
This patch adds more details to the output of the DumpDiagnostics()
method for the system clock synchronized with NTP. In particular,
the output now contains relevant metrics for the hybrid clock, the
output of the 'ntpstat' utility (if present), the output of
'chronyc activity' (if the 'chronyc' CLI is present), and a snapshot
of the newly introduced 'clock_ntp_status' metric.
As already mentioned, this patch also adds a new 'clock_ntp_status'
metric that provides information on ntp_gettime()/ntp_adjtime() call
result. It's a string containing current timestamp (usec),
max error (usec), and the result status of the call.
The corresponding HybridClockTest.TestNtpDiagnostics scenario has been
updated correspondingly.
Change-Id: Ibb74e81ba1e4bc4c60acc6bd49bb458085e28207
---
M src/kudu/clock/hybrid_clock-test.cc
M src/kudu/clock/hybrid_clock.cc
M src/kudu/clock/system_ntp.cc
M src/kudu/clock/system_ntp.h
4 files changed, 99 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/78/19478/2
--
To view, visit http://gerrit.cloudera.org:8080/19478
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibb74e81ba1e4bc4c60acc6bd49bb458085e28207
Gerrit-Change-Number: 19478
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yuqi Du <[email protected]>
