From f8c005e7cb8d17d6d744ea20e01b35a55bb356a3 Mon Sep 17 00:00:00 2001 From: Rishabh Dave Date: Sat, 24 Jun 2023 19:16:39 +0530 Subject: [PATCH] qa/cephfs: don't import entire module needlessly Importing entire module ceph_manager.py is pointless since only ceph_manager.CephManager is required in qa/tasks/cephfs/filesystem.py. Signed-off-by: Rishabh Dave (cherry picked from commit 437f2c75f5f5a98b45d8fceb3e209b7945cd6e71) --- qa/tasks/cephfs/filesystem.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/qa/tasks/cephfs/filesystem.py b/qa/tasks/cephfs/filesystem.py index 8457345fc5d2c..902a3e5443922 100644 --- a/qa/tasks/cephfs/filesystem.py +++ b/qa/tasks/cephfs/filesystem.py @@ -18,7 +18,7 @@ from teuthology.nuke import clear_firewall from teuthology.parallel import parallel from teuthology import contextutil from tasks.ceph_manager import write_conf -from tasks import ceph_manager +from tasks.ceph_manager import CephManager log = logging.getLogger(__name__) @@ -225,7 +225,8 @@ class CephCluster(object): def __init__(self, ctx) -> None: self._ctx = ctx - self.mon_manager = ceph_manager.CephManager(self.admin_remote, ctx=ctx, logger=log.getChild('ceph_manager')) + self.mon_manager = CephManager(self.admin_remote, ctx=ctx, + logger=log.getChild('ceph_manager')) def get_config(self, key, service_type=None): """ -- 2.39.5