]> 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)
committerSebastian Wagner <sebastian.wagner@suse.com>
Tue, 23 Feb 2021 09:58:23 +0000 (10:58 +0100)
Signed-off-by: Michael Fritch <mfritch@suse.com>
(cherry picked from commit 4e4d4c5e4b6118c1629083b0bfca0fb00b500271)

src/pybind/mgr/cephadm/module.py
src/pybind/mgr/tox.ini

index 9ac1297bad65581e0c9eaa84fd766eb7b8a88448..418ad9f7818b196e316d9f9194255a0f5817c513 100644 (file)
@@ -915,8 +915,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:
@@ -1184,8 +1184,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):
@@ -1295,8 +1295,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 995e8f92d0a60771c70b6d6de20ee8d7ecd371ab..0cfc1d0cc03800f94ef52b48c60d52baf2a309ec 100644 (file)
@@ -16,7 +16,6 @@ ignore =
     F811,
     F841,
     W503,
-    W504,
 exclude =
     .tox,
     .vagrant,