From 401c5f768f3955acac54330469beb1fd90ed4c97 Mon Sep 17 00:00:00 2001 From: Vinicius Lopes da Silva Date: Thu, 2 Sep 2021 10:04:40 -0400 Subject: [PATCH] Return tuple in method that resizes filesystems All calls of _config_resize_filesystems expect a tuple to be returned, however, this method has two returns and one of them is returning a single value instead of a tuple. This commit fix this behavior. Closes-bug: #1942449 Signed-off-by: Vinicius Lopes da Silva Change-Id: Ic350f678a348eb1ef9ffe49e4ea4391011e5a9eb --- sysinv/sysinv/sysinv/sysinv/conductor/manager.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sysinv/sysinv/sysinv/sysinv/conductor/manager.py b/sysinv/sysinv/sysinv/sysinv/conductor/manager.py index 10dc298e96..cd0ed456da 100644 --- a/sysinv/sysinv/sysinv/sysinv/conductor/manager.py +++ b/sysinv/sysinv/sysinv/sysinv/conductor/manager.py @@ -9438,7 +9438,7 @@ class ConductorManager(service.PeriodicService): try: if standby_host: if not self._drbd_connected(): - return rc + return rc, drbd_fs_resized if not os.path.isfile(CFS_DRBDADM_RECONFIGURED): progress = "drbdadm resize all" @@ -9538,7 +9538,7 @@ class ConductorManager(service.PeriodicService): {"cmd": ex.cmd, "stdout": ex.stdout, "stderr": ex.stderr, "rc": ex.exit_code}) - return (rc, drbd_fs_resized) + return rc, drbd_fs_resized # Retrying a few times and waiting between each retry should provide # enough protection in the unlikely case LVM's own locking mechanism