diff --git a/qa/qa_utils.py b/qa/qa_utils.py index 102832e55b6f05eb069a6beff517084a721d950d..5ed9ebab3272dfde2641de588a745217710c1338 100644 --- a/qa/qa_utils.py +++ b/qa/qa_utils.py @@ -155,8 +155,9 @@ def AssertCommand(cmd, fail=False, node=None, log_cmd=True, max_seconds=None): @param cmd: either a string (the command to execute) or a list (to be converted using L{utils.ShellQuoteArgs} into a string) - @type fail: boolean - @param fail: if the command is expected to fail instead of succeeding + @type fail: boolean or None + @param fail: if the command is expected to fail instead of succeeding, + or None if we don't care @param node: if passed, it should be the node on which the command should be executed, instead of the master node (can be either a dict or a string) @@ -185,7 +186,8 @@ def AssertCommand(cmd, fail=False, node=None, log_cmd=True, max_seconds=None): stdout, stderr = popen.communicate() rcode = popen.returncode duration_seconds = TimedeltaToTotalSeconds(datetime.datetime.now() - start) - _AssertRetCode(rcode, fail, cmdstr, nodename) + if fail is not None: + _AssertRetCode(rcode, fail, cmdstr, nodename) if max_seconds is not None: if duration_seconds > max_seconds: