fix: follow-up fixes

fix(linstor): prevent use of e before assignment in nested try-except
fix(linstor): add default value for log_failed_call to prevent unassigned value
fix(linstor): convert dict_keys and dict_values to list before indexed access
fix(linstor): use ternary for default value management in log_failed_call
fix(linstor): use util.get_master_ref to get the master ref

Co-authored-by: Damien Thenot <damien.thenot@vates.tech>
Signed-off-by: Mathieu Labourier <mathieu.labourier@vates.tech>

Author: Millefeuille42

drivers/linstorvhdutil.py

@@ -31,21 +31,18 @@
 
 
 def call_remote_method(session, host_ref, method, device_path, args):
-    host_rec = session.xenapi.host.get_record(host_ref)
-    host_uuid = host_rec['uuid']
-
     try:
         response = session.xenapi.host.call_plugin(
             host_ref, MANAGER_PLUGIN, method, args
         )
     except Exception as e:
         util.SMlog('call-plugin on {} ({} with {}) exception: {}'.format(
-            host_uuid, method, args, e
+            host_ref, method, args, e
         ))
         raise util.SMException(str(e))
 
     util.SMlog('call-plugin on {} ({} with {}) returned: {}'.format(
-        host_uuid, method, args, response
+        host_ref, method, args, response
     ))
 
     return response
@@ -83,9 +80,16 @@ def log_successful_call(target_host, device_path, vdi_uuid, remote_method, respo
         priority=util.LOG_DEBUG
     )
 
-def log_failed_call(target_host, next_target, device_path, vdi_uuid, remote_method, e):
+def log_failed_call(target_host, next_target, device_path, vdi_uuid, remote_method, e=None):
     util.SMlog(
-        'Failed to call method on {} for device {} ({}): {}. Trying accessing on {}... (cause: {})'.format(target_host, device_path, vdi_uuid, remote_method, next_target, e),
+        'Failed to call method on {} for device {} ({}): {}. Trying accessing on {}... (cause: {})'.format(
+            target_host,
+            device_path,
+            vdi_uuid,
+            remote_method,
+            next_target,
+            e if e else "no error provided"
+        ),
         priority=util.LOG_DEBUG
     )
 
@@ -107,7 +111,7 @@ def wrapper(*args, **kwargs):
             remote_args = {str(key): str(value) for key, value in remote_args.items()}
 
             try:
-                host_ref_attached = util.get_hosts_attached_on(self._session, [vdi_uuid])[0]
+                host_ref_attached = list(util.get_hosts_attached_on(self._session, [vdi_uuid]))[0]
                 if host_ref_attached:
                     response = call_remote_method(
                         self._session, host_ref_attached, remote_method, device_path, remote_args
@@ -118,7 +122,7 @@ def wrapper(*args, **kwargs):
                 log_failed_call('attached node', 'master', device_path, vdi_uuid, remote_method, e)
 
             try:
-                master_ref = self._session.xenapi.pool.get_all_records().values()[0]['master']
+                master_ref = util.get_master_ref(self._session)
                 response = call_remote_method(self._session, master_ref, remote_method, device_path, remote_args)
                 log_successful_call('master', device_path, vdi_uuid, remote_method, response)
                 return response_parser(self, vdi_uuid, response)
@@ -136,7 +140,7 @@ def wrapper(*args, **kwargs):
                 except Exception as remote_e:
                     self._raise_openers_exception(device_path, remote_e)
             else:
-                log_failed_call('primary', 'another node', device_path, vdi_uuid, remote_method, e)
+                log_failed_call('primary', 'another node', device_path, vdi_uuid, remote_method)
 
                 try:
                     host = self._get_readonly_host(vdi_uuid, device_path, nodes)

drivers/linstorvolumemanager.py

@@ -1058,7 +1058,7 @@ def update_volume_uuid(self, volume_uuid, new_volume_uuid, force=False):
 
             # 5. Ok!
             volume_properties[self.PROP_NOT_EXISTS] = self.STATE_EXISTS
-        except Exception as e:
+        except Exception as err:
             try:
                 # Clear the new volume properties in case of failure.
                 assert volume_properties.namespace == \
@@ -1070,7 +1070,7 @@ def update_volume_uuid(self, volume_uuid, new_volume_uuid, force=False):
                     .format(e)
                 )
             raise LinstorVolumeManagerError(
-                'Failed to copy volume properties: {}'.format(e)
+                'Failed to copy volume properties: {}'.format(err)
             )
 
         try: