Commit 26a61f87 authored by Michael Hanselmann's avatar Michael Hanselmann
Browse files

QA: Remove dry run mode

It didn't work as planned because some commands depend on the return
value or output of some operations.

Reviewed-by: iustinp
parent e310b019
......@@ -237,9 +237,6 @@ def main():
"""
parser = optparse.OptionParser(usage="%prog [options] <config-file>")
parser.add_option('--dry-run', dest='dry_run',
action="store_true",
help="Show what would be done")
parser.add_option('--yes-do-it', dest='yes_do_it',
action="store_true",
help="Really execute the tests")
......
......@@ -111,13 +111,7 @@ def GetSSHCommand(node, cmd, strict=True):
args.append('-oClearAllForwardings=yes')
args.append('-oForwardAgent=yes')
args.append(node)
if qa_config.options.dry_run:
prefix = 'exit 0; '
else:
prefix = ''
args.append(prefix + cmd)
args.append(cmd)
print 'SSH:', utils.ShellQuoteArgs(args)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment