diff options
| author | Guojian Shao <guojian@unitedstack.com> | 2015-06-26 12:07:58 +0800 |
|---|---|---|
| committer | Guojian Shao <guojian@unitedstack.com> | 2015-06-26 12:08:22 +0800 |
| commit | 8899bc4162df0c9f235f6c3bf84c601ef06bafb9 (patch) | |
| tree | 5ffefa4997271509764daf2402baeb372db8fb9c /openstackclient/network | |
| parent | ec31a2a12efe54b9b1149e198560a0164f070485 (diff) | |
| download | python-openstackclient-8899bc4162df0c9f235f6c3bf84c601ef06bafb9.tar.gz | |
fix confused domain argument for network create v2
we have used domain scope arguments --project-domain, --user-domain
and --group-domain in identity commands, for example, role add v3,
to prevent resources conflict from same resource name existence.
To keep with the style of identity commands, it's better to rename
--domain to --project-domain.
Closes-Bug: #1468988
Change-Id: Ic6ccb895cf9be4a3d5f0001525e3b80cd340da8b
Diffstat (limited to 'openstackclient/network')
| -rw-r--r-- | openstackclient/network/v2/network.py | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/openstackclient/network/v2/network.py b/openstackclient/network/v2/network.py index 9b246642..1fa05462 100644 --- a/openstackclient/network/v2/network.py +++ b/openstackclient/network/v2/network.py @@ -87,10 +87,7 @@ class CreateNetwork(show.ShowOne): '--project', metavar='<project>', help="Owner's project (name or ID)") - parser.add_argument( - '--domain', - metavar='<domain>', - help="Owner's domain (name or ID)") + identity_common.add_project_domain_option_to_parser(parser) return parser def take_action(self, parsed_args): @@ -112,15 +109,11 @@ class CreateNetwork(show.ShowOne): body['shared'] = parsed_args.shared if parsed_args.project is not None: identity_client = self.app.client_manager.identity - if parsed_args.domain is not None: - domain = identity_common.find_domain(identity_client, - parsed_args.domain) - project_id = utils.find_resource(identity_client.projects, - parsed_args.project, - domain_id=domain.id).id - else: - project_id = utils.find_resource(identity_client.projects, - parsed_args.project).id + project_id = identity_common.find_project( + identity_client, + parsed_args.project, + parsed_args.project_domain, + ).id body['tenant_id'] = project_id return {'network': body} |