From 1244c6530615d86de356534acb3365b43e643b63 Mon Sep 17 00:00:00 2001 From: Michael Fritch Date: Wed, 4 Mar 2020 13:21:58 -0700 Subject: [PATCH] cephadm: add type checking for `check_host` and `prepare_host` Signed-off-by: Michael Fritch --- src/cephadm/cephadm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/cephadm/cephadm b/src/cephadm/cephadm index a674566cef7..ae747e9e02a 100755 --- a/src/cephadm/cephadm +++ b/src/cephadm/cephadm @@ -3120,6 +3120,7 @@ def check_time_sync(enabler=None): return True def command_check_host(): + # type: () -> None # caller already checked for docker/podman logger.info('podman|docker (%s) is present' % container_path) @@ -3148,6 +3149,7 @@ def command_check_host(): ################################## def command_prepare_host(): + # type: () -> None logger.info('Verifying podman|docker is present...') pkg = None if not container_path: @@ -3177,7 +3179,7 @@ def command_prepare_host(): f.write(args.expect_hostname + '\n') logger.info('Repeating the final host check...') - return command_check_host() + command_check_host() ##################################