x/python-tatuclient
Code Issues Proposed changes

Added extra previllege to list all domains from all tenants

Browse Source 
The user has to use the command "designate --all-tenants domain-list" to list all domains from all tenants.
When the above command is used "X-Auth-All-Projects" value is set to true and it is passed as a header to
designate.This will allow us to list all domains from all tenants.

Change-Id: I4cd4dd5427f5f35cdec95dbdf36c7386b60a2949
Fixes: bug #1418156
This commit is contained in:
Satyanarayana Patibandla 2015-02-23 20:41:22 +05:30 committed by satya.patibandla
parent 4118d0f062
commit debf39a629
3 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
designateclient/shell.py
View File

@ -162,6 +162,9 @@ class DesignateShell(App):
parser.add_argument('--insecure', action='store_true', parser.add_argument('--insecure', action='store_true',
help="Explicitly allow 'insecure' SSL requests") help="Explicitly allow 'insecure' SSL requests")
parser.add_argument('--all-tenants', action='store_true',
help="Allows to list all domains from all tenants")
return parser return parser
def configure_logging(self): def configure_logging(self):
@ -228,6 +231,7 @@ class DesignateShell(App):
token=self.options.os_token, token=self.options.os_token,
insecure=self.options.insecure, insecure=self.options.insecure,
cacert=self.options.os_cacert, cacert=self.options.os_cacert,
all_tenants=self.options.all_tenants,
) )
def run(self, argv): def run(self, argv):

3
designateclient/utils.py
View File

@ -100,7 +100,7 @@ def get_columns(data):
def get_session(auth_url, endpoint, domain_id, domain_name, project_id, def get_session(auth_url, endpoint, domain_id, domain_name, project_id,
project_name, project_domain_name, project_domain_id, username, project_name, project_domain_name, project_domain_id, username,
user_id, password, user_domain_id, user_domain_name, token, user_id, password, user_domain_id, user_domain_name, token,
insecure, cacert): insecure, cacert, all_tenants):
session = ks_session.Session() session = ks_session.Session()
# Build + Attach Authentication Plugin # Build + Attach Authentication Plugin
@ -138,5 +138,6 @@ def get_session(auth_url, endpoint, domain_id, domain_name, project_id,
session.verify = False session.verify = False
else: else:
session.verify = cacert session.verify = cacert
session.all_tenants = all_tenants
return session return session

5
designateclient/v1/__init__.py
View File

@ -32,7 +32,7 @@ class Client(object):
project_domain_id=None, auth_url=None, token=None, project_domain_id=None, auth_url=None, token=None,
endpoint_type='publicURL', region_name=None, endpoint_type='publicURL', region_name=None,
service_type='dns', insecure=False, session=None, service_type='dns', insecure=False, session=None,
cacert=None): cacert=None, all_tenants=False):
""" """
:param endpoint: Endpoint URL :param endpoint: Endpoint URL
:param token: A token instead of username / password :param token: A token instead of username / password
@ -63,6 +63,7 @@ class Client(object):
token=token, token=token,
insecure=insecure, insecure=insecure,
cacert=cacert, cacert=cacert,
all_tenants=all_tenants,
) )
# Since we have to behave nicely like a legacy client/bindings we use # Since we have to behave nicely like a legacy client/bindings we use
@ -97,6 +98,8 @@ class Client(object):
kw['raise_exc'] = False kw['raise_exc'] = False
kw.setdefault('headers', {}) kw.setdefault('headers', {})
kw['headers'].setdefault('Content-Type', 'application/json') kw['headers'].setdefault('Content-Type', 'application/json')
if self.session.session.all_tenants:
kw['headers'].update({'X-Auth-All-Projects': 'true'})
# Trigger the request # Trigger the request
response = func(*args, **kw) response = func(*args, **kw)