Merge "Fix keystoneclient unit tests"

This commit is contained in:
Zuul
2019-03-05 12:27:08 +00:00
committed by Gerrit Code Review
2 changed files with 44 additions and 1 deletions

View File

@@ -0,0 +1,41 @@
From 7b74cc9adb77f025190557e47f6dadae110e6ab5 Mon Sep 17 00:00:00 2001
From: Colleen Murphy <colleen.murphy@suse.de>
Date: Tue, 5 Mar 2019 10:00:46 +0100
Subject: [PATCH] Fix rate semaphore for keystoneclient
When using keystoneclient sessions, the new parameter is not available
and breaks the keystoneclient unit tests[1]. Only use the semaphore
kwarg when using keystoneauth sessions.
[1] https://review.openstack.org/640953
Change-Id: I0cc7f2514e143ec532d8fb895618f7cf1fea9cc3
---
keystoneauth1/adapter.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/keystoneauth1/adapter.py b/keystoneauth1/adapter.py
index a15fd54..6bba62f 100644
--- a/keystoneauth1/adapter.py
+++ b/keystoneauth1/adapter.py
@@ -217,6 +217,8 @@ class Adapter(object):
kwargs.setdefault('client_name', self.client_name)
if self.client_version:
kwargs.setdefault('client_version', self.client_version)
+ if self._rate_semaphore:
+ kwargs.setdefault('rate_semaphore', self._rate_semaphore)
else:
warnings.warn('Using keystoneclient sessions has been deprecated. '
@@ -232,8 +234,6 @@ class Adapter(object):
if self.raise_exc is not None:
kwargs.setdefault('raise_exc', self.raise_exc)
- kwargs.setdefault('rate_semaphore', self._rate_semaphore)
-
return self.session.request(url, method, **kwargs)
def get_token(self, auth=None):
--
2.21.0

View File

@@ -7,6 +7,8 @@ License: {{ license('Apache-2.0') }}
Group: Development/Languages/Python
URL: https://launchpad.net/keystoneauth
Source0: https://files.pythonhosted.org/packages/source/k/%{sname}/%{sname}-%{version}.tar.gz
# https://review.openstack.org/640957
Patch0: 0001-Fix-rate-semaphore-for-keystoneclient.patch
BuildRequires: openstack-macros
BuildRequires: {{ py2pkg('PyYAML', py_versions=['py2', 'py3']) }}
BuildRequires: {{ py2pkg('betamax', py_versions=['py2', 'py3']) }}
@@ -55,7 +57,7 @@ BuildRequires: {{ py2pkg('openstackdocstheme') }}
Documentation for OpenStack authenticating tools.
%prep
%autosetup -n %{sname}-%{version}
%autosetup -p1 -n %{sname}-%{version}
%py_req_cleanup
sed -i 's/^warning-is-error.*/warning-is-error = 0/g' setup.cfg