style: ran pycodestyle plus added minor style fixes

Author: fstagni

src/DIRAC/Resources/Computing/ARC6ComputingElement.py

@@ -27,7 +27,7 @@ def __init__(self, ceUniqueID):
         self.computingInfoEndpoint = "org.nordugrid.ldapglue2"
 
     def _reset(self):
-        super(ARC6ComputingElement, self)._reset()
+        super()._reset()
         # ComputingInfoEndpoint to get information about queues
         # https://www.nordugrid.org/arc/arc6/users/client_tools.html?#arcinfo
         # Expected values are: [

src/DIRAC/Resources/Computing/ARCComputingElement.py

@@ -40,8 +40,7 @@
 import sys
 
 import arc  # Has to work if this module is called #pylint: disable=import-error
-from DIRAC import S_OK, S_ERROR, gConfig
-from DIRAC.ConfigurationSystem.Client.Helpers.Resources import getCESiteMapping
+from DIRAC import S_OK, S_ERROR
 from DIRAC.Core.Utilities.Subprocess import shellCall
 from DIRAC.Core.Utilities.File import makeGuid
 from DIRAC.Core.Utilities.List import breakListIntoChunks
@@ -113,14 +112,14 @@ def _getARCJob(self, jobID):
             j.JobStatusURL = arc.URL(str(statURL))
             j.JobStatusInterfaceName = "org.nordugrid.ldapng"
 
-            mangURL = "gsiftp://%s:2811/jobs/" % (self.ceHost)
+            mangURL = f"gsiftp://{self.ceHost}:2811/jobs/"
             j.JobManagementURL = arc.URL(str(mangURL))
             j.JobManagementInterfaceName = "org.nordugrid.gridftpjob"
 
             j.ServiceInformationURL = j.JobManagementURL
             j.ServiceInformationInterfaceName = "org.nordugrid.ldapng"
         else:
-            commonURL = "https://%s:8443/arex" % self.ceHost
+            commonURL = f"https://{self.ceHost}:8443/arex"
             j.JobStatusURL = arc.URL(str(commonURL))
             j.JobStatusInterfaceName = "org.ogf.glue.emies.activitymanagement"
 
@@ -191,7 +190,7 @@ def _writeXRSL(self, executableFile, inputs=None, outputs=None, executables=None
             if not isinstance(outputs, list):
                 outputs = [outputs]
             for outputFile in outputs:
-                xrslOutputs += '(%s "")' % (outputFile)
+                xrslOutputs += f'({outputFile} "")'
 
         xrsl = """
 &(executable="{executable}")
@@ -250,7 +249,7 @@ def _reset(self):
         if logLevel:
             arc.Logger_getRootLogger().removeDestinations()
             if logLevel not in self._arcLevels:
-                self.log.warn("ARCLogLevel input is not known:", "%s not in %s" % (logLevel, self._arcLevels))
+                self.log.warn("ARCLogLevel input is not known:", f"{logLevel} not in {self._arcLevels}")
             else:
                 logstdout = arc.LogStream(sys.stdout)
                 logstdout.setFormat(arc.ShortFormat)
@@ -272,7 +271,7 @@ def submitJob(self, executableFile, proxy, numberOfJobs=1, inputs=None, outputs=
             return result
         self.usercfg.ProxyPath(os.environ["X509_USER_PROXY"])
 
-        self.log.verbose("Executable file path: %s" % executableFile)
+        self.log.verbose(f"Executable file path: {executableFile}")
         if not os.access(executableFile, 5):
             os.chmod(executableFile, stat.S_IRWXU | stat.S_IRGRP | stat.S_IXGRP | stat.S_IROTH + stat.S_IXOTH)
 
@@ -299,8 +298,8 @@ def submitJob(self, executableFile, proxy, numberOfJobs=1, inputs=None, outputs=
             jobdescs = arc.JobDescriptionList()
             # Get the job into the ARC way
             xrslString, diracStamp = self._writeXRSL(executableFile, inputs, outputs, executables)
-            self.log.debug("XRSL string submitted : %s" % xrslString)
-            self.log.debug("DIRAC stamp for job : %s" % diracStamp)
+            self.log.debug(f"XRSL string submitted : {xrslString}")
+            self.log.debug(f"DIRAC stamp for job : {diracStamp}")
             # The arc bindings don't accept unicode objects in Python 2 so xrslString must be explicitly cast
             result = arc.JobDescription.Parse(str(xrslString), jobdescs)
             if not result:

src/DIRAC/Resources/Computing/AREXComputingElement.py

@@ -4,9 +4,6 @@
     language is used in both cases. So, we retain the xrslExtraString and xrslMPExtraString strings.
 """
 
-
-__RCSID__ = "$Id$"
-
 import os
 import json
 import requests
@@ -59,7 +56,7 @@ def _reset(self):
         Note : This is not run from __init__ as the design of DIRAC means that ceParameters is
         filled with CEDefaults only at the time this class is initialised for the given CE
         """
-        super(AREXComputingElement, self)._reset()
+        super()._reset()
         self.log.debug("Testing if the REST interface is available", "for %s" % self.ceName)
 
         # Get options from the ceParameters dictionary
@@ -431,10 +428,10 @@ def getCEStatus(self):
         result["SubmittedJobs"] = 0
 
         magic = self.queue + "_" + vo.lower()
-        for i in range(len(queueInfo)):
-            if queueInfo[i]["ID"].endswith(magic):
-                result["RunningJobs"] = queueInfo[i]["RunningJobs"]
-                result["WaitingJobs"] = queueInfo[i]["WaitingJobs"]
+        for _, qi in enumerate(queueInfo):
+            if qi["ID"].endswith(magic):
+                result["RunningJobs"] = qi["RunningJobs"]
+                result["WaitingJobs"] = qi["WaitingJobs"]
                 break  # Pick the first (should be only ...) matching queue + VO
 
         return result