Updates for pre-commit

RedHatQE · Nov 21, 2022 · c347e53 · c347e53
1 parent c972650
commit c347e53
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 11 deletions.
diff --git a/wrapanapi/entities/volume.py b/wrapanapi/entities/volume.py
@@ -6,8 +6,6 @@
 from abc import ABCMeta
 from abc import abstractmethod
 
-import six
-
 from wrapanapi.entities.base import Entity
 from wrapanapi.entities.base import EntityMixin
 from wrapanapi.exceptions import MultipleItemsError

diff --git a/wrapanapi/systems/msazure.py b/wrapanapi/systems/msazure.py
@@ -820,7 +820,7 @@ def list_resources_from_hours_old(self, resource_group=None, hours_old=0):
 
     def remove_resource_group_of_old_resources(self, resource_group=None, hours_old=0):
         """
-        Used for clean_up jobs to remove group containing resources older than hours_old numeric value.
+        Used for clean_up jobs to remove group containing resources older than hours_old.
         Age is calculated as difference of current time and resource creation timestamp.
         """
 

diff --git a/wrapanapi/systems/openstack.py b/wrapanapi/systems/openstack.py
@@ -379,13 +379,11 @@ def mark_as_template(self, template_name=None, **kwargs):
             wait_for(lambda: self._api.images.get(uuid).status == "ACTIVE", num_sec=900, delay=5)
             self.delete()
             wait_for(lambda: not self.exists, num_sec=180, delay=5)
-        except Exception as e:
-            self.logger.error(
-                "Could not mark %s as a OpenStack template! (%s)", original_name, str(e)
-            )
+        except Exception:
+            self.logger.exception("Could not mark %s as a OpenStack template!", original_name)
             try:
                 self.rename(original_name)  # Clean up after ourselves
-            except Exception as e:
+            except Exception:
                 self.logger.exception(
                     "Failed to rename %s back to original name (%s)", image_name, original_name
                 )

diff --git a/wrapanapi/systems/scvmm.py b/wrapanapi/systems/scvmm.py
@@ -263,9 +263,10 @@ def creation_time(self):
         ).astimezone(pytz.UTC)
 
     def _do_vm(self, action, params=""):
-        cmd = 'Get-SCVirtualMachine -ID "{}" -VMMServer $scvmm_server | {}-SCVirtualMachine {}'.format(
-            self._id, action, params
-        ).strip()
+        cmd = (
+            'Get-SCVirtualMachine -ID "{}" -VMMServer $scvmm_server '
+            "| {}-SCVirtualMachine {}".format(self._id, action, params).strip()
+        )
         self.logger.info(cmd)
         self._run_script(cmd)
         return True