Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8557 )
Change subject: ntp: dump diagnostics when NTP is unsynchronized or error is too high ...................................................................... ntp: dump diagnostics when NTP is unsynchronized or error is too high This adds some code to shell out to various NTP diagnostics tools when clock problems are detected, just before fataling. Hopefully this will make it easier to diagnose improper NTP setups in the wild. Change-Id: Ifeb2206a0475b8c8e183a74aee21315e6e43dc33 Reviewed-on: http://gerrit.cloudera.org:8080/8557 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/clock/hybrid_clock-test.cc M src/kudu/clock/hybrid_clock.cc M src/kudu/clock/hybrid_clock.h M src/kudu/clock/system_ntp.cc M src/kudu/clock/system_ntp.h M src/kudu/clock/time_service.h M src/kudu/security/test/mini_kdc.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/util/path_util.cc M src/kudu/util/path_util.h M src/kudu/util/test_util.cc M src/kudu/util/test_util.h 12 files changed, 154 insertions(+), 42 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8557 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifeb2206a0475b8c8e183a74aee21315e6e43dc33 Gerrit-Change-Number: 8557 Gerrit-PatchSet: 6 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
