test: stop logging warnings in run_cmd if exception is thrown

If the exception ends up being fatal, then it will get printed in full
anyway, and it contains the entire log message anyway, and we just end
up with duplicate messages. If its not fatal (e.g. caught), then its
arguably up to the caller to decide whether it should be logged.

Signed-off-by: Patrick Roy <[email protected]>

Author: roypat

tests/framework/utils.py

@@ -415,8 +415,6 @@ def run_cmd(cmd, check=False, shell=True, cwd=None, timeout=None) -> CommandRetu
 
     # If a non-zero return code was thrown, raise an exception
     if check and proc.returncode != 0:
-        CMDLOG.warning("Command failed: %s\n", output_message)
-
         raise ChildProcessError(output_message)
 
     CMDLOG.debug(output_message)