]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: Line break occurred after a binary operator (W504)
authorMichael Fritch <mfritch@suse.com>
Thu, 28 Jan 2021 20:16:04 +0000 (13:16 -0700)
committerMichael Fritch <mfritch@suse.com>
Fri, 5 Feb 2021 18:20:57 +0000 (11:20 -0700)
Signed-off-by: Michael Fritch <mfritch@suse.com>
src/pybind/mgr/cephadm/module.py
src/pybind/mgr/tox.ini

index cba40a970857cc03dbbd83886bcefe15aba74ab1..e58d10815c3f44b9e2b90258aac3a7a8b7a02c89 100644 (file)
@@ -907,8 +907,8 @@ class CephadmOrchestrator(orchestrator.Orchestrator, MgrModule,
                 return 1, '', ('check-host failed:\n' + '\n'.join(err))
         except OrchestratorError as e:
             self.log.exception(f"check-host failed for '{host}'")
-            return 1, '', ('check-host failed:\n' +
-                           f"Host '{host}' not found. Use 'ceph orch host ls' to see all managed hosts.")
+            return 1, '', ('check-host failed:\n'
+                           f"Host '{host}' not found. Use 'ceph orch host ls' to see all managed hosts.")
         # if we have an outstanding health alert for this host, give the
         # serve thread a kick
         if 'CEPHADM_HOST_CHECK_FAILED' in self.health_checks:
@@ -1176,8 +1176,8 @@ class CephadmOrchestrator(orchestrator.Orchestrator, MgrModule,
         """
         return [
             h for h in self.inventory.all_specs()
-            if self.cache.host_had_daemon_refresh(h.hostname) and
-            h.status.lower() not in ['maintenance', 'offline']
+            if self.cache.host_had_daemon_refresh(h.hostname)
+            and h.status.lower() not in ['maintenance', 'offline']
         ]
 
     def _add_host(self, spec):
@@ -1287,8 +1287,8 @@ class CephadmOrchestrator(orchestrator.Orchestrator, MgrModule,
             return 1, '\n'.join(error_notifications)
 
         if notifications:
-            return 0, (f'It is presumed safe to stop host {hostname}. ' +
-                       'Note the following:\n\n' + '\n'.join(notifications))
+            return 0, (f'It is presumed safe to stop host {hostname}. '
+                       'Note the following:\n\n' + '\n'.join(notifications))
         return 0, f'It is presumed safe to stop host {hostname}'
 
     @trivial_completion
index 742aaa60f9a2a49f9ad1f23677e0a85753f99b33..b365e48048456849aabcc9a5f5ddd2a3e4a8d12e 100644 (file)
@@ -16,7 +16,6 @@ ignore =
     F811,
     F841,
     W503,
-    W504,
 exclude =
     .tox,
     .vagrant,