From 0bebdaf0f90deef5121234ac98daa58e6f1f0f77 Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Thu, 10 May 2018 19:00:01 -0500 Subject: [PATCH] Fix interface argument to get_all_version_data 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 --- keystoneauth1/session.py | 2 +- .../notes/fix-get-all-version-data-a01ee58524755b9b.yaml | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/fix-get-all-version-data-a01ee58524755b9b.yaml diff --git a/keystoneauth1/session.py b/keystoneauth1/session.py index 35e47e6f..6ab1e144 100644 --- a/keystoneauth1/session.py +++ b/keystoneauth1/session.py @@ -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. diff --git a/releasenotes/notes/fix-get-all-version-data-a01ee58524755b9b.yaml b/releasenotes/notes/fix-get-all-version-data-a01ee58524755b9b.yaml new file mode 100644 index 00000000..7ef980ed --- /dev/null +++ b/releasenotes/notes/fix-get-all-version-data-a01ee58524755b9b.yaml @@ -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.