Change example so CLI names match object arguments

TerryHowe · Terry Howe · commit e744e6006acd · 2014-09-07T14:54:43.000Z
I would like to extract the arguments automatically from the
example command line arguments.  If the names matched, it would
be much easier.

Change-Id: I0877fe8a0b6f5a61505fdecddd2f72ee8c505d29
diff --git a/examples/authenticate.py b/examples/authenticate.py
@@ -48,15 +48,15 @@ def get_endpoint(self, transport, service, **kwargs):
 
 def make_authenticate(opts):
     return authenticator.create(
-        username=opts.os_username,
-        password=opts.os_password,
-        token=opts.os_token,
-        auth_url=opts.os_auth_url,
-        version=opts.os_identity_api_version,
-        project_name=opts.os_project_name,
-        domain_name=opts.os_domain_name,
-        project_domain_name=opts.os_project_domain_name,
-        user_domain_name=opts.os_user_domain_name,
+        username=opts.username,
+        password=opts.password,
+        token=opts.token,
+        auth_url=opts.auth_url,
+        version=opts.identity_api_version,
+        project_name=opts.project_name,
+        domain_name=opts.domain_name,
+        project_domain_name=opts.project_domain_name,
+        user_domain_name=opts.user_domain_name,
     )
 
 
diff --git a/examples/common.py b/examples/common.py
@@ -118,57 +118,66 @@ def option_parser():
     # Global arguments
     parser.add_argument(
         '--os-auth-url',
+        dest='auth_url',
         metavar='<auth-url>',
         default=env('OS_AUTH_URL'),
         help='Authentication URL (Env: OS_AUTH_URL)',
     )
     parser.add_argument(
         '--os-project-name',
+        dest='project_name',
         metavar='<auth-project-name>',
         default=env('OS_PROJECT_NAME', default=env('OS_TENANT_NAME')),
         help='Project name of the requested project-level'
              'authorization scope (Env: OS_PROJECT_NAME)',
     )
     parser.add_argument(
         '--os-domain-name',
+        dest='domain_name',
         metavar='<auth-domain-name>',
         default=env('OS_DOMAIN_NAME'),
         help='Domain name for scope of '
              'authorization (Env: OS_DOMAIN_NAME)',
     )
     parser.add_argument(
         '--os-project-domain-name',
+        dest='project_domain_name',
         metavar='<auth-project-domain-name>',
         default=env('OS_PROJECT_DOMAIN_NAME'),
         help='Project domain name for scope of '
              'authorization (Env: OS_PROJECT_DOMAIN_NAME)',
     )
     parser.add_argument(
         '--os-user-domain-name',
+        dest='user_domain_name',
         metavar='<auth-user-domain-name>',
         default=env('OS_USER_DOMAIN_NAME'),
         help='User domain name for scope of '
              'authorization (Env: OS_USER_DOMAIN_NAME)',
     )
     parser.add_argument(
         '--os-username',
+        dest='username',
         metavar='<auth-username>',
         default=env('OS_USERNAME'),
         help='Authentication username (Env: OS_USERNAME)',
     )
     parser.add_argument(
         '--os-password',
+        dest='password',
         metavar='<auth-password>',
         default=env('OS_PASSWORD'),
         help='Authentication password (Env: OS_PASSWORD)',
     )
     parser.add_argument(
         '--os-region-name',
+        dest='region_name',
         metavar='<region>',
         default=env('OS_REGION_NAME'),
         help='Service region (Env: OS_REGION_NAME)')
     parser.add_argument(
         '--os-cacert',
+        dest='cacert',
         metavar='<ca-bundle-file>',
         default=env('OS_CACERT'),
         help='CA certificate bundle file (Env: OS_CACERT)',
@@ -186,6 +195,7 @@ def option_parser():
     )
     parser.add_argument(
         '--os-identity-api-version',
+        dest='identity_api_version',
         metavar='<identity-api-version>',
         default=env(
             'OS_IDENTITY_API_VERSION',
@@ -194,6 +204,7 @@ def option_parser():
     )
     parser.add_argument(
         '--os-token',
+        dest='token',
         metavar='<token>',
         default=env('OS_TOKEN'),
         help='Defaults to env[OS_TOKEN]',
diff --git a/examples/session.py b/examples/session.py
@@ -29,19 +29,19 @@
 
 def make_session(opts):
     return session.Session.create(
-        username=opts.os_username,
-        password=opts.os_password,
-        token=opts.os_token,
-        auth_url=opts.os_auth_url,
-        version=opts.os_identity_api_version,
-        project_name=opts.os_project_name,
-        domain_name=opts.os_domain_name,
-        project_domain_name=opts.os_project_domain_name,
-        user_domain_name=opts.os_user_domain_name,
-        cacert=opts.os_cacert,
+        username=opts.username,
+        password=opts.password,
+        token=opts.token,
+        auth_url=opts.auth_url,
+        version=opts.identity_api_version,
+        project_name=opts.project_name,
+        domain_name=opts.domain_name,
+        project_domain_name=opts.project_domain_name,
+        user_domain_name=opts.user_domain_name,
+        cacert=opts.cacert,
         insecure=opts.insecure,
         user_agent='SDKExample',
-        region=opts.os_region_name,
+        region=opts.region_name,
     )
 
 
diff --git a/examples/transport.py b/examples/transport.py
@@ -31,7 +31,7 @@
 
 def make_transport(opts):
     return transport.Transport.create(
-        cacert=opts.os_cacert,
+        cacert=opts.cacert,
         insecure=opts.insecure,
         user_agent=USER_AGENT
     )

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@`
`31`	`31`
`32`	`32`	`def make_transport(opts):`
`33`	`33`	`return transport.Transport.create(`
`34`		`- cacert=opts.os_cacert,`
	`34`	`+ cacert=opts.cacert,`
`35`	`35`	`insecure=opts.insecure,`
`36`	`36`	`user_agent=USER_AGENT`
`37`	`37`	`)`