diff options
| author | TerryHowe <terrylhowe@gmail.com> | 2015-05-08 13:14:15 -0600 |
|---|---|---|
| committer | TerryHowe <terrylhowe@gmail.com> | 2015-07-18 06:28:09 -0600 |
| commit | b3335b3474e559d8afb0b7f86796fbb68a0687a8 (patch) | |
| tree | 7fb6af85a7ae7f77c97af02012471984b4d77f3c /openstackclient/network | |
| parent | fcadb16470832cdde2d090a19ebd20a3ad0d1476 (diff) | |
| download | python-openstackclient-b3335b3474e559d8afb0b7f86796fbb68a0687a8.tar.gz | |
Do not set default versions in parsed args
Setting default versions in parsed args makes it so OCC
cannot tell if the argument was parsed, an environment
variable was set or it is just defaulted. In order to
set api versions from OCC, it will have to be defaulted
after processing OCC.
Closes-Bug: #1453229
Change-Id: I4d065919397b783f3bdd4022c986c0234a7a16e6
Diffstat (limited to 'openstackclient/network')
| -rw-r--r-- | openstackclient/network/client.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/openstackclient/network/client.py b/openstackclient/network/client.py index 0ef68852..5f72782b 100644 --- a/openstackclient/network/client.py +++ b/openstackclient/network/client.py @@ -18,7 +18,7 @@ from openstackclient.common import utils LOG = logging.getLogger(__name__) -DEFAULT_NETWORK_API_VERSION = '2' +DEFAULT_API_VERSION = '2' API_VERSION_OPTION = 'os_network_api_version' API_NAME = "network" API_VERSIONS = { @@ -83,10 +83,8 @@ def build_option_parser(parser): parser.add_argument( '--os-network-api-version', metavar='<network-api-version>', - default=utils.env( - 'OS_NETWORK_API_VERSION', - default=DEFAULT_NETWORK_API_VERSION), + default=utils.env('OS_NETWORK_API_VERSION'), help='Network API version, default=' + - DEFAULT_NETWORK_API_VERSION + + DEFAULT_API_VERSION + ' (Env: OS_NETWORK_API_VERSION)') return parser |
