openstack/charm-keystone
Code Issues Proposed changes

Fix error in functional test 910

Browse Source 
The current code for test 910 references self.keystone_sentry
this was replaced by an array in commit
4d2ab6668f.

There was probably a race between the commits and functional
tests was not run on a updated version of the tree prior to
commit 49f993988a.

After the addition of running keystone clustered in commit
4d2ab6668f the test in 910 has
transient failures. Add call to set_api_version after removal
and addition of percona-cluster relation as it will retry and
verify authentication accross all nodes making sure that
configuration has settled before performing final test.

Change-Id: Ib5505adb0ace7c86384bb922008b9a8d73f60f24
This commit is contained in:
Frode Nordahl 2017-01-04 18:24:54 +01:00
parent 49f993988a
commit 23f6363cbd
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tests/basic_deployment.py
View File

@ -804,12 +804,13 @@ class KeystoneBasicDeployment(OpenStackAmuletDeployment):
u.log.debug('Adding keystone percona-cluster relation')
self.d.sentry.wait(timeout=timeout)
self.d.relate('keystone:shared-db', 'percona-cluster:shared-db')
self.set_api_version(3)
self._auto_wait_for_status(
message="Unit is ready",
timeout=timeout,
include_only=['keystone'])
re_auth = u.authenticate_keystone_admin(
self.keystone_sentry,
self.keystone_sentries[0],
user='admin',
password='openstack',
api_version=3,