clock: handle different service names

Restarting ntpd involves two different service names (thanks
again, distro maintainers, much appreciated)

Signed-off-by: Dan Mick <dan.mick@redhat.com>

diff --git a/teuthology/task/clock.py b/teuthology/task/clock.py
index bbfedffba4da25de6cfab9dcb1d3912848546b4b..0af18495a4e7623f4474de2f8de0b9232bd19035 100644 (file)
--- a/teuthology/task/clock.py
+++ b/teuthology/task/clock.py
@@ -38,9 +38,11 @@ def task(ctx, config):
             if l.startswith('server')
         ]
         os.remove(ntpconf)
+        # CentOS calls it ntpd, Xenial/Trusty are ntp.  Thanks guys.
         args = [
-            'sudo',
-            'service', 'ntp', 'stop',
+            'sudo', 'service', 'ntp', 'stop',
+            run.Raw('||'),
+            'sudo', 'service', 'ntpd', 'stop',
             run.Raw(';'),
             'sudo',
             'ntpdate',
@@ -48,8 +50,9 @@ def task(ctx, config):
         args.extend(servers)
         args.extend([
             run.Raw(';'),
-            'sudo',
-            'service', 'ntp', 'start',
+            'sudo', 'service', 'ntp', 'start',
+            run.Raw('||'),
+            'sudo', 'service', 'ntpd', 'start',
             run.Raw(';'),
             'PATH=/usr/bin:/usr/sbin',
             'ntpdc', '-p',