tests: migrate: bool instead of on/off option in cmd line

Simpler. Make the default to be False as well as
 1) No args should run as simple as possible
 2) True is currently broken

Signed-off-by: Victor Toso <victortoso@redhat.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
This commit is contained in:
Victor Toso 2019-09-25 10:46:35 +02:00
parent 0d351864b1
commit 4241806de7

View File

@ -58,7 +58,8 @@ def get_args():
parser.add_argument('--log_filename', dest='log_filename', default='migrate.log')
parser.add_argument('--image', dest='image', default='')
parser.add_argument('--client', dest='client', default='spicy', choices=['spicy', 'remote-viewer'])
parser.add_argument('--vdagent', choices=['on', 'off'], default='on')
parser.add_argument('--vdagent', dest="vdagent", action='store_true', default=False,
help="Append options for agent's virtserialport")
args = parser.parse_args(sys.argv[1:])
if os.path.exists(args.qemu):
args.qemu_exec = args.qemu
@ -209,7 +210,7 @@ def main():
migrator = Migrator(client=args.client, qemu_exec=args.qemu_exec,
image=args.image, log=log, monitor_files=[args.qmp1, args.qmp2],
migration_port=args.migrate_port, spice_ports=[args.spice_port1,
args.spice_port2], vdagent=(args.vdagent=='on'))
args.spice_port2], vdagent=args.vdagent)
atexit.register(cleanup, migrator)
while True:
migrator.iterate()