diff options
| author | Jia Dong <jiadong.jia@huawei.com> | 2014-01-17 14:36:35 +0800 |
|---|---|---|
| committer | Jia Dong <jiadong.jia@huawei.com> | 2014-01-24 17:24:12 +0800 |
| commit | 5a03ff2b42683d32f6fe7b6c54ea7e788dd5e83a (patch) | |
| tree | 8007ad2aecb173ea5f7bdeabe3f5f66d85b4d9c0 /ceilometerclient/shell.py | |
| parent | ff9c08d1806ae680f3511aa8e44c5aa0f9bbf022 (diff) | |
| download | python-ceilometerclient-5a03ff2b42683d32f6fe7b6c54ea7e788dd5e83a.tar.gz | |
Using common methods from oslo cliutils
There are some common methods in cliutils we can use in ceilometerclient:
arg, env, unauthenticated, isunauthenticated.
+ Replaces utils.env to add alias env from cliutils.
+ Removes unused methods in utils: env
To use methods from clituils.
partially implements blueprint common-client-library-2
Change-Id: I295cc80635ed15c91e2832acb315e9deede09516
Diffstat (limited to 'ceilometerclient/shell.py')
| -rw-r--r-- | ceilometerclient/shell.py | 28 |
1 files changed, 15 insertions, 13 deletions
diff --git a/ceilometerclient/shell.py b/ceilometerclient/shell.py index 57cb604..aeea239 100644 --- a/ceilometerclient/shell.py +++ b/ceilometerclient/shell.py @@ -27,6 +27,7 @@ import ceilometerclient from ceilometerclient import client as ceiloclient from ceilometerclient.common import utils from ceilometerclient import exc +from ceilometerclient.openstack.common import cliutils from ceilometerclient.openstack.common import strutils @@ -53,7 +54,8 @@ class CeilometerShell(object): version=ceilometerclient.__version__) parser.add_argument('-d', '--debug', - default=bool(utils.env('CEILOMETERCLIENT_DEBUG')), + default=bool(cliutils.env('CEILOMETERCLIENT_DEBUG') + ), action='store_true', help='Defaults to env[CEILOMETERCLIENT_DEBUG]') @@ -84,7 +86,7 @@ class CeilometerShell(object): parser.add_argument('--os-cacert', metavar='<ca-certificate-file>', dest='os_cacert', - default=utils.env('OS_CACERT'), + default=cliutils.env('OS_CACERT'), help='Path of CA TLS certificate(s) used to verify' 'the remote server\'s certificate. Without this ' 'option ceilometer looks for the default system ' @@ -98,63 +100,63 @@ class CeilometerShell(object): help='Number of seconds to wait for a response') parser.add_argument('--os-username', - default=utils.env('OS_USERNAME'), + default=cliutils.env('OS_USERNAME'), help='Defaults to env[OS_USERNAME]') parser.add_argument('--os_username', help=argparse.SUPPRESS) parser.add_argument('--os-password', - default=utils.env('OS_PASSWORD'), + default=cliutils.env('OS_PASSWORD'), help='Defaults to env[OS_PASSWORD]') parser.add_argument('--os_password', help=argparse.SUPPRESS) parser.add_argument('--os-tenant-id', - default=utils.env('OS_TENANT_ID'), + default=cliutils.env('OS_TENANT_ID'), help='Defaults to env[OS_TENANT_ID]') parser.add_argument('--os_tenant_id', help=argparse.SUPPRESS) parser.add_argument('--os-tenant-name', - default=utils.env('OS_TENANT_NAME'), + default=cliutils.env('OS_TENANT_NAME'), help='Defaults to env[OS_TENANT_NAME]') parser.add_argument('--os_tenant_name', help=argparse.SUPPRESS) parser.add_argument('--os-auth-url', - default=utils.env('OS_AUTH_URL'), + default=cliutils.env('OS_AUTH_URL'), help='Defaults to env[OS_AUTH_URL]') parser.add_argument('--os_auth_url', help=argparse.SUPPRESS) parser.add_argument('--os-region-name', - default=utils.env('OS_REGION_NAME'), + default=cliutils.env('OS_REGION_NAME'), help='Defaults to env[OS_REGION_NAME]') parser.add_argument('--os_region_name', help=argparse.SUPPRESS) parser.add_argument('--os-auth-token', - default=utils.env('OS_AUTH_TOKEN'), + default=cliutils.env('OS_AUTH_TOKEN'), help='Defaults to env[OS_AUTH_TOKEN]') parser.add_argument('--os_auth_token', help=argparse.SUPPRESS) parser.add_argument('--ceilometer-url', - default=utils.env('CEILOMETER_URL'), + default=cliutils.env('CEILOMETER_URL'), help='Defaults to env[CEILOMETER_URL]') parser.add_argument('--ceilometer_url', help=argparse.SUPPRESS) parser.add_argument('--ceilometer-api-version', - default=utils.env( + default=cliutils.env( 'CEILOMETER_API_VERSION', default='2'), help='Defaults to env[CEILOMETER_API_VERSION] ' 'or 2') @@ -163,14 +165,14 @@ class CeilometerShell(object): help=argparse.SUPPRESS) parser.add_argument('--os-service-type', - default=utils.env('OS_SERVICE_TYPE'), + default=cliutils.env('OS_SERVICE_TYPE'), help='Defaults to env[OS_SERVICE_TYPE]') parser.add_argument('--os_service_type', help=argparse.SUPPRESS) parser.add_argument('--os-endpoint-type', - default=utils.env('OS_ENDPOINT_TYPE'), + default=cliutils.env('OS_ENDPOINT_TYPE'), help='Defaults to env[OS_ENDPOINT_TYPE]') parser.add_argument('--os_endpoint_type', |
