--- doc/loncapafiles/ntpcheck.piml	2017/10/23 23:57:46	1.31
+++ doc/loncapafiles/ntpcheck.piml	2020/01/21 16:31:27	1.45
@@ -3,7 +3,7 @@
 <!-- ntpcheck.piml -->
 <!-- Matthew Hall -->
 
-<!-- $Id: ntpcheck.piml,v 1.31 2017/10/23 23:57:46 raeburn Exp $ -->
+<!-- $Id: ntpcheck.piml,v 1.45 2020/01/21 16:31:27 raeburn Exp $ -->
 
 <!--
 
@@ -36,8 +36,19 @@ http://www.lon-capa.org/
 <target dist="default"></target>
 <perlscript mode="fg">
 print "Checking to be sure the Network Time Protocol is running properly.\n";
-print "For more information on ntpd please see http://www.ntp.org/\n";
+if (('<DIST />' eq 'fedora26') || ('<DIST />' eq 'fedora27') ||
+    ('<DIST />' eq 'fedora28') || ('<DIST />' eq 'fedora29') ||
+    ('<DIST />' eq 'fedora30') || ('<DIST />' eq 'fedora31') ||
+    ('<DIST />' eq 'sles15')   || ('<DIST />' eq 'centos8')  || 
+    ('<DIST />' eq 'rhes8')    || ('<DIST />' eq 'oracle7')  ||
+    ('<DIST />' eq 'oracle8')  || ('<DIST />' eq 'ubuntu18') ||
+    ('<DIST />' eq 'ubuntu20')) {
+    print "For more information on chronyd please see https://chrony.tuxfamily.org/\n";
+} else {
+    print "For more information on ntpd please see http://www.ntp.org/\n";
+}
 my $NTPD;
+my $name = 'ntp';
 my $checkcmd = 'ps -ef |grep ntp |grep -v grep |grep -v ntpcheck';
 my $startntpcmd;
 my $is_running;
@@ -47,12 +58,34 @@ if (('<DIST />' eq 'fedora15')    || ('<
     ('<DIST />' eq 'fedora21')    || ('<DIST />' eq 'fedora22') ||
     ('<DIST />' eq 'fedora23')    || ('<DIST />' eq 'fedora24') ||
     ('<DIST />' eq 'fedora25')    || ('<DIST />' eq 'fedora26') ||
-    ('<DIST />' eq 'centos7')     || ('<DIST />' eq 'rhes7')    ||
-    ('<DIST />' eq 'scientific7') || ('<DIST />' eq 'sles12')) {
+    ('<DIST />' eq 'fedora27')    || ('<DIST />' eq 'fedora28') ||
+    ('<DIST />' eq 'fedora29')    || ('<DIST />' eq 'fedora30') ||
+    ('<DIST />' eq 'fedora31')    || ('<DIST />' eq 'centos7')  ||
+    ('<DIST />' eq 'rhes7')       || ('<DIST />' eq 'oracle7')  ||
+    ('<DIST />' eq 'scientific7') || ('<DIST />' eq 'sles12')   ||
+    ('<DIST />' eq 'sles15')      || ('<DIST />' eq 'ubuntu18') ||
+    ('<DIST />' eq 'ubuntu20')    || ('<DIST />' eq 'rhes8')    || 
+    ('<DIST />' eq 'centos8')     || ('<DIST />' eq 'oracle8')) { 
     $NTPD = 'ntpd';
-    $startntpcmd = 'systemctl start ntpd.service';
-    if (!-e '/usr/sbin/ntpd') {
-        print "ntp is not installed.\n";
+    my $binname = $NTPD;
+    if (('<DIST />' eq 'ubuntu18') || ('<DIST />' eq 'ubuntu20')) {
+        $NTPD = 'chrony';
+        $name = $NTPD;
+        $binname = 'chronyd';  
+    } elsif (('<DIST />' eq 'fedora26') || ('<DIST />' eq 'fedora27') ||
+             ('<DIST />' eq 'fedora28') || ('<DIST />' eq 'fedora29') ||
+             ('<DIST />' eq 'fedora30') || ('<DIST />' eq 'fedora31') ||
+             ('<DIST />' eq 'sles15')   || ('<DIST />' eq 'rhes8')    || 
+             ('<DIST />' eq 'centos8')  || ('<DIST />' eq 'oracle7')  ||
+             ('<DIST />' eq 'oracle8')) {
+        $NTPD = 'chronyd';
+        $name = $NTPD;
+        $binname = $NTPD;
+    }
+    $checkcmd = "systemctl is-enabled $NTPD.service";
+    $startntpcmd = "systemctl start $NTPD.service";
+    if (!-e "/usr/sbin/$binname") {
+        print "$name is not installed.\n";
         exit;
     }
 } else {
@@ -74,12 +107,16 @@ if (open(PIPE,'-|',$checkcmd)) {
     chomp($is_running);
     close(PIPE);
     if (!$is_running) {
-        print "WARNING: ntp is installed but is not currently running.\nPlease verify the configuration of ntp in /etc/ntp.conf and /etc/ntp/step-tickers\nStart ntp by executing\n $startntpcmd\n";
+        print "WARNING: $name is installed but is not currently running.\n";
+        if ($name eq 'ntp') {
+            print "Please verify the configuration of ntp in /etc/ntp.conf and /etc/ntp/step-tickers\n";
+        }
+        print "Start $name by executing\n $startntpcmd\n";
     } else {
-        print "ntp is installed and running\n";
+        print "$name is installed and running\n";
     }
 } else {
-    print "Could not determine ntp status.\n";
+    print "Could not determine $name status.\n";
 }
 
 # chkconfig barfs to stderr if the target isn't set up right.
@@ -87,12 +124,18 @@ if (('<DIST />' eq 'fedora15')    || ('<
     ('<DIST />' eq 'fedora17')    || ('<DIST />' eq 'fedora18') ||
     ('<DIST />' eq 'fedora19')    || ('<DIST />' eq 'fedora20') ||
     ('<DIST />' eq 'fedora21')    || ('<DIST />' eq 'fedora22') ||
-    ('<DIST />' eq 'fedora23')    || ('<DIST />' eq 'fedora24') || 
-    ('<DIST />' eq 'fedora25')    || ('<DIST />' eq 'fedora26') || 
-    ('<DIST />' eq 'centos7')     || ('<DIST />' eq 'rhes7')    ||
-    ('<DIST />' eq 'scientific7') || ('<DIST />' eq 'sles12')) {
-    if (!-l "/etc/systemd/system/multi-user.target.wants/ntpd.service") {  
-        print "**** WARNING: ntpd is not configured to run at boot.  To correct this run:\nsystemctl enable ntpd.service\n";
+    ('<DIST />' eq 'fedora23')    || ('<DIST />' eq 'fedora24') ||
+    ('<DIST />' eq 'fedora25')    || ('<DIST />' eq 'fedora26') ||
+    ('<DIST />' eq 'fedora27')    || ('<DIST />' eq 'fedora28') ||
+    ('<DIST />' eq 'fedora29')    || ('<DIST />' eq 'fedora30') ||
+    ('<DIST />' eq 'fedora31')    || ('<DIST />' eq 'centos7')  || 
+    ('<DIST />' eq 'rhes7')       || ('<DIST />' eq 'oracle7')  ||
+    ('<DIST />' eq 'scientific7') || ('<DIST />' eq 'sles12')   ||
+    ('<DIST />' eq 'sles15')      || ('<DIST />' eq 'ubuntu18') ||
+    ('<DIST />' eq 'ubuntu20')    || ('<DIST />' eq 'rhes8')    ||
+    ('<DIST />' eq 'centos8')     || ('<DIST />' eq 'oracle8')) {
+    if (!-l "/etc/systemd/system/multi-user.target.wants/$NTPD.service") {  
+        print "**** WARNING: $NTPD is not configured to run at boot.  To correct this run:\nsystemctl enable $NTPD.service\n";
     }
     exit;
 } else {