]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cephadm: reformat deploy_daemon arguments to make it readable
authorJohn Mulligan <jmulligan@redhat.com>
Tue, 6 Jun 2023 20:00:08 +0000 (16:00 -0400)
committerJohn Mulligan <jmulligan@redhat.com>
Wed, 9 Aug 2023 17:48:07 +0000 (13:48 -0400)
Reformat the deploy_daemon function's arguments to use the "black"
one-line-per-argument style, which is significantly more readable IMO.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
src/cephadm/cephadm.py

index 184316a63ca6468042e8f8bcde487175290e89e9..eaa52d3ff951fc91490f51fde2d83fa5bcaa32ce 100755 (executable)
@@ -3648,13 +3648,20 @@ def extract_uid_gid(ctx, img='', file_path='/var/lib/ceph'):
     raise RuntimeError('uid/gid not found')
 
 
-def deploy_daemon(ctx: CephadmContext, fsid: str, daemon_type: str,
-                  daemon_id: Union[int, str], c: Optional['CephContainer'],
-                  uid: int, gid: int, config: Optional[str] = None,
-                  keyring: Optional[str] = None, osd_fsid: Optional[str] = None,
-                  deployment_type: DeploymentType = DeploymentType.DEFAULT,
-                  ports: Optional[List[int]] = None) -> None:
-
+def deploy_daemon(
+    ctx: CephadmContext,
+    fsid: str,
+    daemon_type: str,
+    daemon_id: Union[int, str],
+    c: Optional['CephContainer'],
+    uid: int,
+    gid: int,
+    config: Optional[str] = None,
+    keyring: Optional[str] = None,
+    osd_fsid: Optional[str] = None,
+    deployment_type: DeploymentType = DeploymentType.DEFAULT,
+    ports: Optional[List[int]] = None,
+) -> None:
     ports = ports or []
     # only check port in use if fresh deployment since service
     # we are redeploying/reconfiguring will already be using the port