From: Rishabh Dave <ridave@redhat.com>
Date: Sat, 24 Jun 2023 13:46:39 +0000 (+0530)
Subject: qa/cephfs: don't import entire module needlessly
X-Git-Tag: v17.2.8~395^2~15
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2b3445150bf4596cb467d1842c1dc6068cf73946;p=ceph.git

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 <ridave@redhat.com>
(cherry picked from commit 437f2c75f5f5a98b45d8fceb3e209b7945cd6e71)
---

diff --git a/qa/tasks/cephfs/filesystem.py b/qa/tasks/cephfs/filesystem.py
index 269a818a41cd..47d353def9f6 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):
         """