openstack/keystoneauth
Code Issues Proposed changes

Fix interface argument to get_all_version_data

Browse Source 
The new get_all_version_data call lists 'public' as the default value
for interface, but had None in the arguments. 'public' was the intent,
and is what the similar call on the base auth plugin does.

Change-Id: I5db2b66d0c130ed80b13ee0c41e0e37c6f7234a7
This commit is contained in:
Monty Taylor 2018-05-10 19:00:01 -05:00
parent c0d5157157
commit 0bebdaf0f9
No known key found for this signature in database
GPG Key ID: 7BAE94BC7141A594
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
keystoneauth1/session.py
View File

@ -1048,7 +1048,7 @@ class Session(object):
auth = self._auth_required(auth, 'determine endpoint URL')
return auth.get_api_major_version(self, **kwargs)
def get_all_version_data(self, auth=None, interface=None,
def get_all_version_data(self, auth=None, interface='public',
region_name=None, **kwargs):
"""Get version data for all services in the catalog.

6
releasenotes/notes/fix-get-all-version-data-a01ee58524755b9b.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
The docstring for ``keystoneauth1.session.Session.get_all_version_data``
correctly listed ``'public'`` as the default value, but the argument list
had ``None``. The default has been fixed to match the documented value.