From 57137e799b234dbb43952acb6302124eca063deb Mon Sep 17 00:00:00 2001 From: Dhairya Parmar Date: Tue, 25 Feb 2025 16:28:06 +0530 Subject: [PATCH] qa: remove unreacheable/redundant code from test_data_scan.py call to self.fs.journal_tool(["journal", "reset", "--force", "--yes-i-really-really-mean-it"], 0) is already made below it, the same call under if False is unreachable. Signed-off-by: Dhairya Parmar --- qa/tasks/cephfs/test_data_scan.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/qa/tasks/cephfs/test_data_scan.py b/qa/tasks/cephfs/test_data_scan.py index 73d40d4aa7d..bee8c292edb 100644 --- a/qa/tasks/cephfs/test_data_scan.py +++ b/qa/tasks/cephfs/test_data_scan.py @@ -499,11 +499,6 @@ class TestDataScan(CephFSTestCase): self.fs.table_tool([self.fs.name + ":0", "reset", "inode"]) # Run the recovery procedure - if False: - with self.assertRaises(CommandFailedError): - # Normal reset should fail when no objects are present, we'll use --force instead - self.fs.journal_tool(["journal", "reset", "--yes-i-really-really-mean-it"], 0) - self.fs.journal_tool(["journal", "reset", "--force", "--yes-i-really-really-mean-it"], 0) self.fs.data_scan(["init", "--force-init"]) self.fs.data_scan(["scan_extents"], worker_count=workers) -- 2.47.3