cloudera-ps / prereq-checks

Prerequisites checker for Cloudera Manager and CDP PVC Base installations
GNU General Public License v3.0
57 stars 58 forks source link

If chrony is used, warn that Kudu does not support it #109

Closed myloginid closed 5 years ago

myloginid commented 5 years ago

git diff -

diff --git a/lib/checks.sh b/lib/checks.sh
index e73e9fc..0b4e7d3 100644
--- a/lib/checks.sh
+++ b/lib/checks.sh
@@ -186,6 +186,10 @@ function check_os() (
                 _check_service_is_not_running 'System' 'chronyd'
             else
                 _check_service_is_running 'System' 'chronyd'
+                get_service_state 'chronyd'
+                if [ "${SERVICE_STATE['running']}" = true ]; then
+                    state "System: kudu supports only ntpd. If kudu is not used, this warning can be ignored." 2
+                fi
             fi
         else
             _check_service_is_running 'System' 'ntpd'

Tested on local -

Redirecting to /bin/systemctl status  chronyd.service
● chronyd.service - NTP client/server
   Loaded: loaded (/usr/lib/systemd/system/chronyd.service; enabled; vendor preset: enabled)
   Active: active (running) since Wed 2018-10-24 06:13:55 PDT; 8min ago
     Docs: man:chronyd(8)
 PASS  System: chronyd is running
 PASS  System: chronyd auto-starts on boot
 WARN  System: kudu supports only ntpd. If kudu is not used, this warning can be ignored.
 PASS  System: Only 64bit packages should be installed
myloginid commented 5 years ago

Created PR #110