Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15275 )
Change subject: [clock] record maximum error in microseconds ...................................................................... [clock] record maximum error in microseconds Dedicated NTP servers of AWS/EC2 cloud instances report sub-millisecond root delay and dispersion (see below), so it makes sense to switch true time maximum error histogram into microseconds. In addition, as a follow-up to the recent series of patches in the context of KUDU-3048, I also added an extra metric gauge to track latest maximum time error computed by the built-in NTP client. I also renamed 'builtin_ntp_walltime' into 'builtin_ntp_time' for brevity. This is a follow-up to 7d9d7009 and 6371ffdb5. [root@aws-centos]# chronyc sources 210 Number of sources = 1 MS Name/IP address Stratum Poll Reach LastRx Last sample =============================================================================== ^* 169.254.169.123 3 8 377 106 +1063ns[+1441ns] +/- 466us [root@aws-centos]# chronyc tracking Reference ID : A9FEA97B (169.254.169.123) Stratum : 4 Ref time (UTC) : Sun Feb 23 06:59:58 2020 System time : 0.000001781 seconds slow of NTP time Last offset : +0.000000378 seconds RMS offset : 0.000012111 seconds Frequency : 30.588 ppm fast Residual freq : -0.000 ppm Skew : 0.019 ppm Root delay : 0.000298246 seconds Root dispersion : 0.000397126 seconds Update interval : 260.0 seconds Leap status : Normal [root@aws-centos]# chronyc ntpdata 169.254.169.123 Remote address : 169.254.169.123 (A9FEA97B) Remote port : 123 Local address : 10.65.14.145 (0A410E91) Leap status : Normal Version : 4 Mode : Server Stratum : 3 Poll interval : 8 (256 seconds) Precision : -18 (0.000003815 seconds) Root delay : 0.000198 seconds Root dispersion : 0.000305 seconds Reference ID : A9FEA97A () Reference time : Sun Feb 23 06:55:06 2020 Offset : +0.000014476 seconds Peer delay : 0.000101295 seconds Peer dispersion : 0.000003855 seconds Response time : 0.000015916 seconds Jitter asymmetry: +0.00 NTP tests : 111 111 1111 Interleaved : No Authenticated : No TX timestamping : Kernel RX timestamping : Kernel Total TX : 712 Total RX : 712 Total valid RX : 712 Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c Reviewed-on: http://gerrit.cloudera.org:8080/15275 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/clock/builtin_ntp.cc M src/kudu/clock/builtin_ntp.h M src/kudu/clock/hybrid_clock-test.cc M src/kudu/clock/hybrid_clock.cc 4 files changed, 49 insertions(+), 19 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15275 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c Gerrit-Change-Number: 15275 Gerrit-PatchSet: 5 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
