From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Thu, 9 May 2013 21:22:31 +0000 (-0400)
Subject: alway give user 'user' role at their home site
X-Git-Tag: 1.0~61^2~5
X-Git-Url: http://git.onelab.eu/?p=plstackapi.git;a=commitdiff_plain;h=d84caefae90d7329ed5b71872782dceb08ba2edf

alway give user 'user' role at their home site
---

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):