From: Vallari Agrawal <vallari.agrawal@ibm.com>
Date: Mon, 20 Jan 2025 11:43:44 +0000 (+0530)
Subject: qa/tasks/nvmeof.py: make seperate calls in do_checks()
X-Git-Tag: v20.0.0~263^2~5
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=5a58114498bcc155dfdbd61a54fa77097b5d5a45;p=ceph.git

qa/tasks/nvmeof.py: make seperate calls in do_checks()

When running 'nvme list-subsys <device>' command
in do_checks(), instead of combining command for
all devices with '&&', make seperate calls.

Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
---

diff --git a/qa/tasks/nvmeof.py b/qa/tasks/nvmeof.py
index e5c60f195bd..33cb51b8c01 100644
--- a/qa/tasks/nvmeof.py
+++ b/qa/tasks/nvmeof.py
@@ -356,12 +356,14 @@ class NvmeofThrasher(Thrasher, Greenlet):
             run.Raw('&&'), 'ceph', 'nvme-gw', 'show', 'mypool', 'mygroup0',
             run.Raw('&&'), 'sudo', 'nvme', 'list',
         ]
+        self.checker_host.run(args=check_cmd).wait()
+
         for dev in self.devices:
-            check_cmd += [
-                run.Raw('&&'), 'sudo', 'nvme', 'list-subsys', dev,
+            device_check_cmd = [
+                'sudo', 'nvme', 'list-subsys', dev,
                 run.Raw('|'), 'grep', 'live optimized'
-            ] 
-        self.checker_host.run(args=check_cmd).wait()        
+            ]
+            self.checker_host.run(args=device_check_cmd)    
 
     def switch_task(self):
         """