From: dongwenjuan <dong.wenjuan@zte.com.cn>
Date: Mon, 8 Jan 2018 06:42:03 +0000 (+0800)
Subject: fix the CI error
X-Git-Tag: opnfv-6.0.0~23^2
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?p=doctor.git;a=commitdiff_plain;h=46991f69c81249b60e40b71964dde3223a59ddd8

fix the CI error

Change-Id: I0b60e6f1e15cbe556c7555186352d28490b98acc
Signed-off-by: dongwenjuan <dong.wenjuan@zte.com.cn>
---

diff --git a/doctor_tests/user.py b/doctor_tests/user.py
index 5598a2d5..a6f58d9a 100644
--- a/doctor_tests/user.py
+++ b/doctor_tests/user.py
@@ -13,6 +13,7 @@ from oslo_config import cfg
 from doctor_tests.identity_auth import get_session
 from doctor_tests.os_clients import keystone_client
 from doctor_tests.os_clients import nova_client
+from keystoneclient import exceptions as ks_exceptions
 
 
 OPTS = [
@@ -136,12 +137,15 @@ class User(object):
         roles_for_user = self.roles_for_admin \
             if is_admin else self.roles_for_user
 
-        if not self.keystone.roles.check(role, user=user, project=project):
+        try:
+            self.keystone.roles.check(role, user=user, project=project)
+            self.log.info('Already grant a role:%s to user: %s on'
+                          ' project: %s'
+                          % (role_name, user_name,
+                             self.conf.doctor_project))
+        except ks_exceptions.NotFound:
             self.keystone.roles.grant(role, user=user, project=project)
             roles_for_user[role_name] = role
-        else:
-            self.log.info('Already grant a role:%s to user: %s on project: %s'
-                          % (role_name, user_name, self.conf.doctor_project))
 
     def delete(self):
         """delete the test user, project and role"""