Fixed the issue in sysinfo where the dmidecode and fdisk -l commands were executed.

Signed-off-by: xianglongfei <[email protected]>

Author: xianglongfei

avocado/core/sysinfo.py

@@ -11,11 +11,11 @@
 # This code was inspired in the autotest project,
 # client/shared/settings.py
 # Author: John Admanski <[email protected]>
+import configparser
 import filecmp
 import logging
 import os
 import time
-import configparser
 
 from avocado.core import output
 from avocado.core.settings import settings
@@ -84,6 +84,47 @@ def __init__(self, basedir=None, log_packages=None, profiler=None):
             log.debug("sudo_distros config is empty or missing")
 
         def _load_sudo_list(raw_value, key): 
+            # pylint: disable=wrong-spelling-in-docstring
+            """
+            If `raw_value` is a path to an INI file, read `[sysinfo] / key`
+            from it; otherwise, treat `raw_value` itself as a CSV list.
+            """
+            if not raw_value:
+                return ""
+            if os.path.isfile(raw_value):
+                parser = configparser.ConfigParser()
+                parser.read(raw_value)
+                return parser.get("sysinfo", key, fallback="")
+            return raw_value
+
+        # Retrieve the actual sudo commands and distros values from the config files,
+        # falling back to empty string if the keys are missing
+        sudo_commands_value = _load_sudo_list(sudo_commands_conf, "sudo_commands")
+        sudo_distros_value = _load_sudo_list(sudo_distros_conf, "sudo_distros")
+
+        self.sudo_commands = {
+            cmd.strip().lower() for cmd in sudo_commands_value.split(",") if cmd.strip()
+        }
+
+        self.sudo_distros = {
+            dst.strip().lower() for dst in sudo_distros_value.split(",") if dst.strip()
+        }
+
+        # Retrieve the configured paths for sudo commands and distros from the settings dictionary
+        sudo_commands_conf = self.config.get("sysinfo.sudo_commands", "")
+        sudo_distros_conf = self.config.get("sysinfo.sudo_distros", "")
+
+        if sudo_commands_conf:
+            log.info("sudo_commands loaded from config: %s", sudo_commands_conf)
+        else:
+            log.debug("sudo_commands config is empty or missing")
+
+        if sudo_distros_conf:
+            log.info("sudo_distros loaded from config: %s", sudo_distros_conf)
+        else:
+            log.debug("sudo_distros config is empty or missing")
+
+        def _load_sudo_list(raw_value, key):
             # pylint: disable=wrong-spelling-in-docstring
             """
             If `raw_value` is a path to an INI file, read `[sysinfo] / key`

avocado/utils/sysinfo.py

@@ -14,14 +14,14 @@
 #          John Admanski <[email protected]>
 
 import json
+import logging
 import os
+import platform
 import shlex
 import subprocess
 import tempfile
-import platform
-import logging
-
 from abc import ABC, abstractmethod
+
 from avocado.utils import astring, process
 from avocado.utils.process import can_sudo
 
@@ -138,7 +138,7 @@ class Command(Collectible):
 
     def __init__(
         self, cmd, timeout=-1, locale="C", sudo_commands=None, sudo_distros=None
-    ): # pylint: disable=R0913
+    ):  # pylint: disable=R0913
         super().__init__(cmd)
         self._name = self.log_path
         self.cmd = cmd