From 2904d2ae13cb165ad7c08a62791a6d618584455d Mon Sep 17 00:00:00 2001 From: Rishabh Dave Date: Thu, 21 Mar 2024 18:29:52 +0530 Subject: [PATCH] qa/cephfs: don't use run_cluster_cmd() The method was deleted in favour of run_ceph_cmd() and get_ceph_cmd_stdout() in commit 35a0d84972d691535533ffeb1635e5f94d808575. This commit is not a backport but a new commit to fix the code present on Reef branch but not present in release branches after Reef. At least some of these issues were exposed during QA done for the backport. Signed-off-by: Rishabh Dave --- qa/tasks/cephfs/test_admin.py | 2 +- qa/tasks/cephfs/test_snapshots.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/qa/tasks/cephfs/test_admin.py b/qa/tasks/cephfs/test_admin.py index 731148c4a5ea4..4f3100bbecec2 100644 --- a/qa/tasks/cephfs/test_admin.py +++ b/qa/tasks/cephfs/test_admin.py @@ -1387,7 +1387,7 @@ class TestFsAuthorize(CephFSTestCase): self.run_cap_test_one_by_one(FS_AUTH_CAPS) def run_cap_test_one_by_one(self, fs_auth_caps): - keyring = self.run_cluster_cmd(f'auth get {self.client_name}') + keyring = self.run_ceph_cmd(f'auth get {self.client_name}') for i, c in enumerate(fs_auth_caps): self.assertIn(i, (0, 1)) PATH = c[0] diff --git a/qa/tasks/cephfs/test_snapshots.py b/qa/tasks/cephfs/test_snapshots.py index 29dc06066ad4d..a9639a7ebbcaf 100644 --- a/qa/tasks/cephfs/test_snapshots.py +++ b/qa/tasks/cephfs/test_snapshots.py @@ -553,12 +553,12 @@ class TestMonSnapsAndFsPools(CephFSTestCase): with self.assertRaises(CommandFailedError): test_pool_name = self.fs.get_data_pool_name() base_cmd = f'osd pool mksnap {test_pool_name} snap3' - self.run_cluster_cmd(base_cmd) + self.run_ceph_cmd(base_cmd) with self.assertRaises(CommandFailedError): test_pool_name = self.fs.get_metadata_pool_name() base_cmd = f'osd pool mksnap {test_pool_name} snap4' - self.run_cluster_cmd(base_cmd) + self.run_ceph_cmd(base_cmd) def test_attaching_pools_with_snaps_to_fs_fails(self): """ -- 2.39.5