From: Vallari Agrawal <val.agl002@gmail.com>
Date: Mon, 20 Jun 2022 18:33:11 +0000 (+0530)
Subject: orch/run: set mutable default arguments as None
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=35435807167d2851eaa646facd7513ef9f9ff10f;p=teuthology.git

orch/run: set mutable default arguments as None

Signed-off-by: Vallari Agrawal <val.agl002@gmail.com>
---

diff --git a/teuthology/orchestra/run.py b/teuthology/orchestra/run.py
index 0c661c7b8..a69730624 100644
--- a/teuthology/orchestra/run.py
+++ b/teuthology/orchestra/run.py
@@ -40,7 +40,7 @@ class RemoteProcess(object):
     deadlock_warning = "Using PIPE for %s without wait=False would deadlock"
 
     def __init__(self, client, args, check_status=True, hostname=None,
-                 label=None, timeout=None, wait=True, logger=None, cwd=None, scan_tests_errors=[]):
+                 label=None, timeout=None, wait=True, logger=None, cwd=None, scan_tests_errors=None):
         """
         Create the object. Does not initiate command execution.
 
@@ -265,7 +265,7 @@ class ErrorScanner(object):
                 return True
         return False
 
-    def scan(self, test_names=[]):
+    def scan(self, test_names=None):
         logfile = self.__logfile__
         if not logfile or not test_names:
             return None, None
@@ -473,7 +473,7 @@ def run(
     quiet=False,
     timeout=None,
     cwd=None,
-    scan_tests_errors=[],
+    scan_tests_errors=None,
     # omit_sudo is used by vstart_runner.py
     omit_sudo=False
 ):