[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 12/25] tests/vm: switch from optsparse to argparse
From: |
Alex Bennée |
Subject: |
[PATCH v2 12/25] tests/vm: switch from optsparse to argparse |
Date: |
Wed, 24 Jun 2020 15:04:33 +0100 |
optparse has been deprecated since version 3.2 and argparse is the
blessed replacement. Take the opportunity to enhance our help output
showing defaults when called.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Robert Foley <robert.foley@linaro.org>
---
tests/vm/basevm.py | 84 +++++++++++++++++++++++-----------------------
1 file changed, 42 insertions(+), 42 deletions(-)
diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
index 04d083409a5..93859362606 100644
--- a/tests/vm/basevm.py
+++ b/tests/vm/basevm.py
@@ -23,7 +23,7 @@ from qemu.accel import kvm_available
from qemu.machine import QEMUMachine
import subprocess
import hashlib
-import optparse
+import argparse
import atexit
import tempfile
import shutil
@@ -556,47 +556,47 @@ def parse_args(vmcls):
else:
return 1
- parser = optparse.OptionParser(
- description="VM test utility. Exit codes: "
- "0 = success, "
- "1 = command line error, "
- "2 = environment initialization failed, "
- "3 = test command failed")
- parser.add_option("--debug", "-D", action="store_true",
- help="enable debug output")
- parser.add_option("--image", "-i", default="%s.img" % vmcls.name,
- help="image file name")
- parser.add_option("--force", "-f", action="store_true",
- help="force build image even if image exists")
- parser.add_option("--jobs", type=int, default=get_default_jobs(),
- help="number of virtual CPUs")
- parser.add_option("--verbose", "-V", action="store_true",
- help="Pass V=1 to builds within the guest")
- parser.add_option("--build-image", "-b", action="store_true",
- help="build image")
- parser.add_option("--build-qemu",
- help="build QEMU from source in guest")
- parser.add_option("--build-target",
- help="QEMU build target", default="check")
- parser.add_option("--build-path", default=None,
- help="Path of build directory, "\
- "for using build tree QEMU binary. ")
- parser.add_option("--interactive", "-I", action="store_true",
- help="Interactively run command")
- parser.add_option("--snapshot", "-s", action="store_true",
- help="run tests with a snapshot")
- parser.add_option("--genisoimage", default="genisoimage",
- help="iso imaging tool")
- parser.add_option("--config", "-c", default=None,
- help="Provide config yaml for configuration. "\
- "See config_example.yaml for example.")
- parser.add_option("--efi-aarch64",
- default="/usr/share/qemu-efi-aarch64/QEMU_EFI.fd",
- help="Path to efi image for aarch64 VMs.")
- parser.add_option("--log-console", action="store_true",
- help="Log console to file.")
- parser.disable_interspersed_args()
- return parser.parse_args()
+ parser = argparse.ArgumentParser(
+ formatter_class=argparse.ArgumentDefaultsHelpFormatter,
+ description="Utility for provisioning VMs and running builds",
+ epilog="""Remaining arguments are passed to the command.
+ Exit codes: 0 = success, 1 = command line error,
+ 2 = environment initialization failed,
+ 3 = test command failed""")
+ parser.add_argument("--debug", "-D", action="store_true",
+ help="enable debug output")
+ parser.add_argument("--image", "-i", default="%s.img" % vmcls.name,
+ help="image file name")
+ parser.add_argument("--force", "-f", action="store_true",
+ help="force build image even if image exists")
+ parser.add_argument("--jobs", type=int, default=get_default_jobs(),
+ help="number of virtual CPUs")
+ parser.add_argument("--verbose", "-V", action="store_true",
+ help="Pass V=1 to builds within the guest")
+ parser.add_argument("--build-image", "-b", action="store_true",
+ help="build image")
+ parser.add_argument("--build-qemu",
+ help="build QEMU from source in guest")
+ parser.add_argument("--build-target",
+ help="QEMU build target", default="check")
+ parser.add_argument("--build-path", default=None,
+ help="Path of build directory, "\
+ "for using build tree QEMU binary. ")
+ parser.add_argument("--interactive", "-I", action="store_true",
+ help="Interactively run command")
+ parser.add_argument("--snapshot", "-s", action="store_true",
+ help="run tests with a snapshot")
+ parser.add_argument("--genisoimage", default="genisoimage",
+ help="iso imaging tool")
+ parser.add_argument("--config", "-c", default=None,
+ help="Provide config yaml for configuration. "\
+ "See config_example.yaml for example.")
+ parser.add_argument("--efi-aarch64",
+ default="/usr/share/qemu-efi-aarch64/QEMU_EFI.fd",
+ help="Path to efi image for aarch64 VMs.")
+ parser.add_argument("--log-console", action="store_true",
+ help="Log console to file.")
+ return parser.parse_known_args()
def main(vmcls, config=None):
try:
--
2.20.1
- Re: [PATCH v2 02/25] crypto/linux_keyring: fix 'secret_keyring' configure test, (continued)
- [PATCH v2 03/25] tests/vm: pass args through to BaseVM's __init__, Alex Bennée, 2020/06/24
- [PATCH v2 04/25] tests/vm: Add configuration to basevm.py, Alex Bennée, 2020/06/24
- [PATCH v2 06/25] tests/vm: Add common Ubuntu python module, Alex Bennée, 2020/06/24
- [PATCH v2 11/25] tests/vm: Add workaround to consume console, Alex Bennée, 2020/06/24
- [PATCH v2 05/25] tests/vm: Added configuration file support, Alex Bennée, 2020/06/24
- [PATCH v2 07/25] tests/vm: Added a new script for ubuntu.aarch64., Alex Bennée, 2020/06/24
- [PATCH v2 08/25] tests/vm: Added a new script for centos.aarch64., Alex Bennée, 2020/06/24
- [PATCH v2 10/25] python/qemu: Add ConsoleSocket for optional use in QEMUMachine, Alex Bennée, 2020/06/24
- [PATCH v2 09/25] tests/vm: change scripts to use self._config, Alex Bennée, 2020/06/24
- [PATCH v2 12/25] tests/vm: switch from optsparse to argparse,
Alex Bennée <=
- [PATCH v2 16/25] .gitignore: un-ignore .gitlab-ci.d, Alex Bennée, 2020/06/24
- [PATCH v2 14/25] tests/docker: check for an parameters not empty string, Alex Bennée, 2020/06/24
- [PATCH v2 15/25] tests/docker: change tag naming scheme of our images, Alex Bennée, 2020/06/24
- [PATCH v2 21/25] tests/docker: add --registry support to tooling, Alex Bennée, 2020/06/24
- [PATCH v2 13/25] tests/vm: allow us to take advantage of MTTCG, Alex Bennée, 2020/06/24
- [PATCH v2 24/25] gitlab: add avocado asset caching, Alex Bennée, 2020/06/24
- [PATCH v2 23/25] gitlab: enable check-tcg for linux-user tests, Alex Bennée, 2020/06/24
- [PATCH v2 20/25] gitlab: build containers with buildkit and metadata, Alex Bennée, 2020/06/24
- [PATCH v2 19/25] gitlab: convert jobs to use custom built containers, Alex Bennée, 2020/06/24