From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Tue, 9 Apr 2013 01:45:43 +0000 (-0400)
Subject: fix bugs
X-Git-Tag: 1.0~208
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=a70405382d89ab7643c426b4e48501e8ade019c7;p=plstackapi.git

fix bugs
---

diff --git a/plstackapi/openstack/driver.py b/plstackapi/openstack/driver.py
index cba7d95..f6817cf 100644
--- a/plstackapi/openstack/driver.py
+++ b/plstackapi/openstack/driver.py
@@ -22,8 +22,8 @@ class OpenStackDriver:
             role = roles[0] 
         return role
 
-    def delete_role(self, name):
-        roles = self.shell.keystone.roles.findall(name=name)
+    def delete_role(self, filter):
+        roles = self.shell.keystone.roles.findall(**filter)
         for role in roles:
             self.shell.keystone.roles.delete(role)
         return 1
diff --git a/plstackapi/planetstack/api/roles.py b/plstackapi/planetstack/api/roles.py
index 7d2d8ce..bb964f1 100644
--- a/plstackapi/planetstack/api/roles.py
+++ b/plstackapi/planetstack/api/roles.py
@@ -14,7 +14,7 @@ def add_role(auth, name):
 def delete_role(auth, role_id):
     driver = OpenStackDriver(client = auth_check(auth))   
     role = Role.objects.filter(role_id=role_id)
-    driver.delete_role(name) 
+    driver.delete_role({'role_id': role_id}) 
     role.delete()
     return 1