Merge "Fix S311 and S603 ruff security violations"

cyborg/accelerator/drivers/spdk/util/pyspdk/py_spdk.py

@@ -14,7 +14,6 @@
 import os
 import psutil
 import re
-import subprocess
 
 from oslo_concurrency import processutils
 from oslo_log import log as logging
@@ -89,10 +88,5 @@ class PySPDK:
         if sub_args is None:
             sub_args = []
         exec_cmd.extend(sub_args)
-        p = subprocess.Popen(
-            exec_cmd,
-            stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE
-        )
-        out, err = p.communicate()
+        out, _err = processutils.execute(*exec_cmd)
         return out

pyproject.toml

@@ -85,8 +85,6 @@ select = ["E4", "E7", "E9", "F", "S", "UP", "W", "C90"]
 ignore = [
     # asserts used for type narrowing only
     "S101",
-    # random not used for crypto
-    "S311",
     # S104 Possible binding to all interfaces
     "S104",
     # S105/S106 Possible hardcoded password
@@ -102,7 +100,6 @@ ignore = [
 
 [tool.ruff.lint.per-file-ignores]
 "cyborg/tests/*" = ["S"]
-"cyborg/accelerator/drivers/spdk/util/pyspdk/*" = ["S603"]
 
 [tool.ruff.lint.mccabe]
 max-complexity = 20