From 296eba2443f4a81248f8f7db75b7a3e8eb13fbae Mon Sep 17 00:00:00 2001 From: Riccardo Pittau Date: Wed, 24 Mar 2021 16:00:20 +0100 Subject: [PATCH] Remove explicit default from store_true The store_true action for argparse already default to False [0] [0] https://docs.python.org/3/library/argparse.html#action Change-Id: I2afa7068f9adc9536b4d96931fd518c065c56652 --- bifrost/cli.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bifrost/cli.py b/bifrost/cli.py index 443933855..f712142b7 100644 --- a/bifrost/cli.py +++ b/bifrost/cli.py @@ -227,7 +227,7 @@ def parse_args(): install.set_defaults(func=cmd_install) install.add_argument('--testenv', action='store_true', help='running in a virtual environment') - install.add_argument('--develop', action='store_true', default=False, + install.add_argument('--develop', action='store_true', help='install packages in development mode') install.add_argument('--dhcp-pool', metavar='START-END', help='DHCP pool to use') @@ -247,11 +247,10 @@ def parse_args(): help='a comma separated list of enabled bare metal ' 'hardware types') install.add_argument('--cleaning-disk-erase', - action='store_true', default=False, + action='store_true', help='enable full disk cleaning between ' 'deployments (can take a lot of time)') install.add_argument('--enable-prometheus-exporter', action='store_true', - default=False, help='Enable Ironic Prometheus Exporter') install.add_argument('--uefi', action='store_true', help='use UEFI by default')