pyghmi 1.6.4__tar.gz → 1.6.5__tar.gz

{pyghmi-1.6.4 → pyghmi-1.6.5}/ChangeLog

@@ -1,6 +1,11 @@
 CHANGES
 =======
 
+1.6.5
+-----
+
+* Attempt better redfish account deletion
+
 1.6.4
 -----
 

{pyghmi-1.6.4 → pyghmi-1.6.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pyghmi
-Version: 1.6.4
+Version: 1.6.5
 Summary: Python General Hardware Management Initiative (IPMI and others)
 Home-page: http://github.com/openstack/pyghmi/
 Author: Jarrod Johnson

{pyghmi-1.6.4 → pyghmi-1.6.5}/pyghmi/redfish/command.py

@@ -445,7 +445,7 @@ class Command(object):
         return self.oem.get_ikvm_launchdata()
 
     def user_delete(self, uid):
-        self.oem.user_delete(uid)
+        self.oem.user_delete(uid, self)
 
     def set_user_name(self, uid, name):
         """Set user name

{pyghmi-1.6.4 → pyghmi-1.6.5}/pyghmi/redfish/oem/generic.py

@@ -383,13 +383,19 @@ class OEMHandler(object):
             summary['badreadings'].append(unkinf)
         return summary
 
-    def user_delete(self, uid):
+    def user_delete(self, uid, fishclient):
         # Redfish doesn't do so well with Deleting users either...
         # Blanking the username seems to be the convention
         # First, set a bogus password in case the implementation does honor
         # blank user, at least render such an account harmless
-        self.set_user_password(uid, base64.b64encode(os.urandom(15)))
-        self.set_user_name(uid, '')
+        try:
+            accinfo = fishclient._account_url_info_by_id(uid)
+            if not accinfo:
+                raise Exception("No such account found")
+            self._do_web_request(accinfo[0], method='DELETE')
+        except Exception: # fall back to old ipmi-like behavior for such implementations
+            fishclient.set_user_password(uid, base64.b64encode(os.urandom(15)))
+            fishclient.set_user_name(uid, '')
         return True
 
     def set_bootdev(self, bootdev, persist=False, uefiboot=None,

{pyghmi-1.6.4 → pyghmi-1.6.5}/pyghmi/redfish/oem/lenovo/xcc.py

@@ -1662,7 +1662,7 @@ class OEMHandler(generic.OEMHandler):
                     license_errors[rsp['return']])
         return self.get_licenses(fishclient)
 
-    def user_delete(self, uid):
+    def user_delete(self, uid, fishclient=None):
         userinfo = self.wc.grab_json_response('/api/dataset/imm_users')
         uidtonamemap = {}
         for user in userinfo.get('items', [{'users': []}])[0].get('users', []):

{pyghmi-1.6.4 → pyghmi-1.6.5}/pyghmi.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pyghmi
-Version: 1.6.4
+Version: 1.6.5
 Summary: Python General Hardware Management Initiative (IPMI and others)
 Home-page: http://github.com/openstack/pyghmi/
 Author: Jarrod Johnson

pyghmi-1.6.5/pyghmi.egg-info/pbr.json

@@ -0,0 +1 @@
+{"git_version": "f56e000", "is_release": true}

pyghmi-1.6.4/pyghmi.egg-info/pbr.json

@@ -1 +0,0 @@
-{"git_version": "99af22b", "is_release": true}