From 8899bc4162df0c9f235f6c3bf84c601ef06bafb9 Mon Sep 17 00:00:00 2001 From: Guojian Shao Date: Fri, 26 Jun 2015 12:07:58 +0800 Subject: 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 --- openstackclient/network/v2/network.py | 19 ++++++------------- 1 file changed, 6 insertions(+), 13 deletions(-) (limited to 'openstackclient/network') 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='', help="Owner's project (name or ID)") - parser.add_argument( - '--domain', - metavar='', - 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} -- cgit v1.2.1