]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
cephadm: remove redundant `ERROR` during check-host
authorMichael Fritch <mfritch@suse.com>
Wed, 20 Jan 2021 21:28:38 +0000 (14:28 -0700)
committerSebastian Wagner <sebastian.wagner@suse.com>
Tue, 9 Mar 2021 14:29:32 +0000 (15:29 +0100)
```
ERROR: ERROR: No time synchronization is active
```

Signed-off-by: Michael Fritch <mfritch@suse.com>
(cherry picked from commit ace5d41dc580a1ea0d0aba19f07da355b06823ef)

src/cephadm/cephadm

index 3da8ea0b72567c03651c3335f5ecd29818e6d315..161e0fa76093118794d136ca55034e2556a64ccf 100755 (executable)
@@ -5277,21 +5277,21 @@ def command_check_host(ctx: CephadmContext) -> None:
             find_program(command)
             logger.info('%s is present' % command)
         except ValueError:
-            errors.append('ERROR: %s binary does not appear to be installed' % command)
+            errors.append('%s binary does not appear to be installed' % command)
 
     # check for configured+running chronyd or ntp
     if not check_time_sync(ctx):
-        errors.append('ERROR: No time synchronization is active')
+        errors.append('No time synchronization is active')
 
     if "expect_hostname" in ctx and ctx.expect_hostname:
         if get_hostname().lower() != ctx.expect_hostname.lower():
-            errors.append('ERROR: hostname "%s" does not match expected hostname "%s"' % (
+            errors.append('hostname "%s" does not match expected hostname "%s"' % (
                 get_hostname(), ctx.expect_hostname))
         logger.info('Hostname "%s" matches what is expected.',
                     ctx.expect_hostname)
 
     if errors:
-        raise Error('\n'.join(errors))
+        raise Error('\nERROR: '.join(errors))
 
     logger.info('Host looks OK')