chore(pylintrc): get rid of E1120 errors for LINSTOR using signature-mutators

Signed-off-by: Ronan Abhamon <[email protected]>

Author: Wescoeur

drivers/LinstorSR.py

@@ -161,9 +161,7 @@ def attach_thin(session, journaler, linstor, sr_uuid, vdi_uuid):
     # If the virtual COW size is lower than the LINSTOR volume size,
     # there is nothing to do.
     cow_size = linstorcowutil.compute_volume_size(
-        # TODO: Replace pylint comment with this feature when possible:
-        # https://github.com/PyCQA/pylint/pull/2926
-        linstorcowutil.get_size_virt(vdi_uuid)  # pylint: disable = E1120
+        linstorcowutil.get_size_virt(vdi_uuid)
     )
 
     volume_info = linstor.get_volume_info(vdi_uuid)
@@ -203,9 +201,7 @@ def check_vbd_count():
     device_path = linstor.get_device_path(vdi_uuid)
     linstorcowutil = LinstorCowUtil(session, linstor, vdi_type)
     new_volume_size = LinstorVolumeManager.round_up_volume_size(
-        # TODO: Replace pylint comment with this feature when possible:
-        # https://github.com/PyCQA/pylint/pull/2926
-        linstorcowutil.get_size_phys(vdi_uuid)  # pylint: disable = E1120
+        linstorcowutil.get_size_phys(vdi_uuid)
     )
 
     volume_info = linstor.get_volume_info(vdi_uuid)
@@ -1215,10 +1211,7 @@ def _load_vdis_ex(self):
             self.vdis[vdi_uuid] = vdi
 
             if USE_KEY_HASH and VdiType.isCowImage(vdi.vdi_type):
-                # TODO: Replace pylint comment with this feature when possible:
-                # https://github.com/PyCQA/pylint/pull/2926
-                vdi.sm_config_override['key_hash'] = \
-                    vdi.linstorcowutil.get_key_hash(vdi_uuid)  # pylint: disable = E1120
+                vdi.sm_config_override['key_hash'] = vdi.linstorcowutil.get_key_hash(vdi_uuid)
 
             # 4.c. Update CBT status of disks either just added
             # or already in XAPI.

drivers/linstorcowutil.py

@@ -196,7 +196,7 @@ def check(self, vdi_uuid, ignore_missing_footer=False, fast=False):
             'ignoreMissingFooter': ignore_missing_footer,
             'fast': fast
         }
-        return self._check(vdi_uuid, **kwargs)  # pylint: disable = E1123
+        return self._check(vdi_uuid, **kwargs)
 
     @linstorhostcall(CowUtil.check, 'check')
     def _check(self, vdi_uuid, response):
@@ -207,9 +207,7 @@ def get_info(self, vdi_uuid, include_parent=True):
             'includeParent': include_parent,
             'resolveParent': False
         }
-        # TODO: Replace pylint comment with this feature when possible:
-        # https://github.com/PyCQA/pylint/pull/2926
-        return self._get_info(vdi_uuid, self._extract_uuid, **kwargs)  # pylint: disable = E1123
+        return self._get_info(vdi_uuid, self._extract_uuid, **kwargs)
 
     @linstorhostcall(CowUtil.getInfo, 'getInfo')
     def _get_info(self, vdi_uuid, response):
@@ -323,9 +321,7 @@ def inflate(self, journaler, vdi_uuid, vdi_path, new_size, old_size):
         )
         self._linstor.resize_volume(vdi_uuid, new_size)
 
-        # TODO: Replace pylint comment with this feature when possible:
-        # https://github.com/PyCQA/pylint/pull/2926
-        result_size = self.get_drbd_size(vdi_uuid)  # pylint: disable = E1120
+        result_size = self.get_drbd_size(vdi_uuid)
         if result_size < new_size:
             util.SMlog(
                 'WARNING: Cannot inflate volume to {}B, result size: {}B'

tests/pylintrc

@@ -92,6 +92,8 @@ generated-members=REQUEST,acl_users,aq_parent,linstor.*
 # # supports qualified module names, as well as Unix pattern matching.
 ignored-modules=bitarray
 
+# List of decorators that change the signature of a decorated function.
+signature-mutators=linstorcowutil.linstorhostcall
 
 [SIMILARITIES]