From d2702e17c9c03e3de71cc6fdd464f57e5830ce62 Mon Sep 17 00:00:00 2001 From: Dolph Mathews Date: Fri, 14 Jun 2013 15:51:16 -0500 Subject: [PATCH] Remove endpoint.name attribute from v3 manager (bug 1191152) This also provides backwards compatible support for any clients that were specifying a name by handling it through **kwargs. Change-Id: I49c542c52e1cdfde5fff1be46d658a805d233320 --- keystoneclient/v3/endpoints.py | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/keystoneclient/v3/endpoints.py b/keystoneclient/v3/endpoints.py index 010572bde..7030d287a 100644 --- a/keystoneclient/v3/endpoints.py +++ b/keystoneclient/v3/endpoints.py @@ -47,22 +47,23 @@ class EndpointManager(base.CrudManager): msg = msg % ', '.join(VALID_INTERFACES) raise Exception(msg) - def create(self, service, url, name=None, interface=None, region=None, - enabled=True): + def create(self, service, url, interface=None, region=None, enabled=True, + **kwargs): self._validate_interface(interface) return super(EndpointManager, self).create( service_id=base.getid(service), interface=interface, url=url, region=region, - enabled=enabled) + enabled=enabled, + **kwargs) def get(self, endpoint): return super(EndpointManager, self).get( endpoint_id=base.getid(endpoint)) - def list(self, service=None, interface=None, region=None, - enabled=None, **kwargs): + def list(self, service=None, interface=None, region=None, enabled=None, + **kwargs): """List endpoints. If ``**kwargs`` are provided, then filter endpoints with @@ -76,8 +77,8 @@ class EndpointManager(base.CrudManager): enabled=enabled, **kwargs) - def update(self, endpoint, service=None, url=None, name=None, - interface=None, region=None, enabled=None): + def update(self, endpoint, service=None, url=None, interface=None, + region=None, enabled=None, **kwargs): self._validate_interface(interface) return super(EndpointManager, self).update( endpoint_id=base.getid(endpoint), @@ -85,7 +86,8 @@ class EndpointManager(base.CrudManager): interface=interface, url=url, region=region, - enabled=enabled) + enabled=enabled, + **kwargs) def delete(self, endpoint): return super(EndpointManager, self).delete(