X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=plstackapi%2Fopenstack%2Fmanager.py;h=b912934ee5fc6dd75100aa5baa250fa0f8374180;hb=d84caefae90d7329ed5b71872782dceb08ba2edf;hp=86ad0147230233695bc06de51dc0c8f350c4198e;hpb=85d1883963609902c5915f752762355e0c5ac186;p=plstackapi.git

diff --git a/plstackapi/openstack/manager.py b/plstackapi/openstack/manager.py
index 86ad014..b912934 100644
--- a/plstackapi/openstack/manager.py
+++ b/plstackapi/openstack/manager.py
@@ -79,12 +79,12 @@ class OpenStackManager:
             keystone_user = self.driver.create_user(**user_fields)
             user.user_id = keystone_user.id
         if user.site:
+            self.driver.add_user_role(user.user_id, user.site.tenant_id, 'user')
             if user.is_admin:
                 self.driver.add_user_role(user.user_id, user.site.tenant_id, 'admin')
             else:
                 # may have admin role so attempt to remove it
-                self.driver.remove_user_role(user.user_id, user.site.tenant_id, 'admin')
-                self.driver.add_user_role(user.user_id, user.site.tenant_id, 'user')
+                self.driver.delete_user_role(user.user_id, user.site.tenant_id, 'admin')
   
     @require_enabled
     def delete_user(self, user):