Merge pull request DIRACGrid#7731 from DIRACGridBot/cherry-pick-2-aa0…

…228832-integration [sweep:integration] fix pylint
fstagni · Jul 15, 2024 · b4ea2d7 · b4ea2d7
2 parents c861882 + 4fdf9db
commit b4ea2d7
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/DIRAC/DataManagementSystem/Agent/RequestOperations/MoveReplica.py b/src/DIRAC/DataManagementSystem/Agent/RequestOperations/MoveReplica.py
@@ -379,7 +379,7 @@ def singleRemoval(self, opFile, targetSE):
         proxyFile = None
         if "Write access not permitted for this credential" in opFile.Error:
             # # not a DataManger? set status to failed and return
-            if "DataManager" in self.shifter:
+            if "DataManager" in self.shifter:  # pylint: disable =unsupported-membership-test
                 # #  you're a data manager - save current proxy and get a new one for LFN and retry
                 saveProxy = os.environ["X509_USER_PROXY"]
                 try:

diff --git a/src/DIRAC/DataManagementSystem/Agent/RequestOperations/PhysicalRemoval.py b/src/DIRAC/DataManagementSystem/Agent/RequestOperations/PhysicalRemoval.py
@@ -154,7 +154,7 @@ def singleRemoval(self, opFile, targetSE):
         proxyFile = None
         if "Write access not permitted for this credential" in opFile.Error:
             # # not a DataManger? set status to failed and return
-            if "DataManager" not in self.shifter:
+            if "DataManager" not in self.shifter:  # pylint: disable =unsupported-membership-test
                 opFile.Status = "Failed"
             elif not opFile.LFN:
                 opFile.Error = "LFN not set"

diff --git a/src/DIRAC/DataManagementSystem/Agent/RequestOperations/RemoveFile.py b/src/DIRAC/DataManagementSystem/Agent/RequestOperations/RemoveFile.py
@@ -199,7 +199,7 @@ def singleRemoval(self, opFile):
         # # try to remove with owner proxy
         proxyFile = None
         if "Write access not permitted for this credential" in opFile.Error:
-            if "DataManager" in self.shifter:
+            if "DataManager" in self.shifter:  # pylint: disable =unsupported-membership-test
                 # #  you're a data manager - get proxy for LFN and retry
                 saveProxy = os.environ["X509_USER_PROXY"]
                 try:

diff --git a/src/DIRAC/DataManagementSystem/Agent/RequestOperations/RemoveReplica.py b/src/DIRAC/DataManagementSystem/Agent/RequestOperations/RemoveReplica.py
@@ -178,7 +178,7 @@ def _removeWithOwnerProxy(self, opFile, targetSE):
         """
         if "Write access not permitted for this credential" in opFile.Error:
             proxyFile = None
-            if "DataManager" in self.shifter:
+            if "DataManager" in self.shifter:  # pylint: disable =unsupported-membership-test
                 # #  you're a data manager - save current proxy and get a new one for LFN and retry
                 saveProxy = os.environ["X509_USER_PROXY"]
                 try: