openstack/python-openstackclient
Code Issues Proposed changes

v3 endpoint set shouldn't always need service option

Browse Source 
Change-Id: I71aab1ee4f467dc963e7afa7fc1c82b4255ea822
Closes-Bug: #1351121
This commit is contained in:
wanghong 2014-07-29 15:33:18 +08:00
parent 75e8490e54
commit a9fb5fa102
1 changed files with 6 additions and 2 deletions

8
openstackclient/identity/v3/endpoint.py

@ -181,16 +181,20 @@ class SetEndpoint(command.Command):
identity_client = self.app.client_manager.identity identity_client = self.app.client_manager.identity
endpoint = utils.find_resource(identity_client.endpoints, endpoint = utils.find_resource(identity_client.endpoints,
parsed_args.endpoint) parsed_args.endpoint)
service = common.find_service(identity_client, parsed_args.service)
if (not parsed_args.interface and not parsed_args.url if (not parsed_args.interface and not parsed_args.url
and not parsed_args.service and not parsed_args.region): and not parsed_args.service and not parsed_args.region):
sys.stdout.write("Endpoint not updated, no arguments present") sys.stdout.write("Endpoint not updated, no arguments present")
return return
service_id = None
if parsed_args.service:
service = common.find_service(identity_client, parsed_args.service)
service_id = service.id
identity_client.endpoints.update( identity_client.endpoints.update(
endpoint.id, endpoint.id,
service=service.id, service=service_id,
url=parsed_args.url, url=parsed_args.url,
interface=parsed_args.interface, interface=parsed_args.interface,
region=parsed_args.region, region=parsed_args.region,