Merge pull request #6945 from fstagni/80_fixes56

[8.0] print a warning message when no CFG file is found

Author: fstagni

src/DIRAC/ConfigurationSystem/Client/LocalConfiguration.py

@@ -412,8 +412,8 @@ def __parseCommandLine(self):
 
         groupArgs = []
         step = 0
-        for i in range(len(self.commandArgumentList)):
-            argMarking, description, mandatory, values, default = self.commandArgumentList[i]
+        for i, commandArgument in enumerate(self.commandArgumentList):
+            argMarking, description, mandatory, values, default = commandArgument
 
             # Check whether the required arguments are given in the command line
             if len(self.commandArgList) <= (i + step):
@@ -433,7 +433,7 @@ def __parseCommandLine(self):
                 if values and cArg not in values:
                     gLogger.fatal(
                         "Error when parsing command line arguments: "
-                        '"%s" does not match the allowed values for %s' % (cArg, argMarking)
+                        f'"{cArg}" does not match the allowed values for {argMarking}'
                     )
                     self.showHelp(exitCode=1)
 
@@ -444,30 +444,51 @@ def __parseCommandLine(self):
     def __loadCFGFiles(self):
         """
         Loads possibly several cfg files, in order:
-        1. ~/.dirac.cfg
-        2. cfg files pointed by DIRACSYSCONFIG env variable (comma-separated)
+        1. cfg files pointed by DIRACSYSCONFIG env variable (comma-separated)
+        2. ~/.dirac.cfg
         3. cfg files specified in addCFGFile calls
         4. cfg files that come from the command line
         """
         errorsList = []
+        foundCFGFile = False
+
+        # 1. $DIRACSYSCONFIG
         if "DIRACSYSCONFIG" in os.environ:
             diracSysConfigFiles = os.environ["DIRACSYSCONFIG"].replace(" ", "").split(",")
             for diracSysConfigFile in reversed(diracSysConfigFiles):
                 gLogger.debug(f"Loading file from DIRACSYSCONFIG {diracSysConfigFile}")
+                if os.path.isfile(diracSysConfigFile):
+                    foundCFGFile = True
                 gConfigurationData.loadFile(diracSysConfigFile)
+
+        # 2. ~/.dirac.cfg
+        if os.path.isfile(os.path.expanduser("~/.dirac.cfg")):
+            foundCFGFile = True
         gConfigurationData.loadFile(os.path.expanduser("~/.dirac.cfg"))
+
+        # 3. cfg files specified in addCFGFile calls
         for fileName in self.additionalCFGFiles:
-            gLogger.debug(f"Loading file {fileName}")
+            if os.path.isfile(fileName):
+                foundCFGFile = True
+                gLogger.debug(f"Loading file {fileName}")
             retVal = gConfigurationData.loadFile(fileName)
             if not retVal["OK"]:
                 gLogger.debug(f"Could not load file {fileName}: {retVal['Message']}")
                 errorsList.append(retVal["Message"])
+
+        # 4. cfg files that come from the command line
         for fileName in self.cliAdditionalCFGFiles:
-            gLogger.debug(f"Loading file {fileName}")
+            if os.path.isfile(fileName):
+                foundCFGFile = True
+                gLogger.debug(f"Loading file {fileName}")
             retVal = gConfigurationData.loadFile(fileName)
             if not retVal["OK"]:
                 gLogger.debug(f"Could not load file {fileName}: {retVal['Message']}")
                 errorsList.append(retVal["Message"])
+
+        if not foundCFGFile:
+            gLogger.warn("No CFG file loaded, was that intentional?")
+
         return errorsList
 
     def __addUserDataToConfiguration(self):

src/DIRAC/ConfigurationSystem/private/ConfigurationData.py

@@ -7,9 +7,10 @@
 import _thread
 import time
 import datetime
-import DIRAC
 
 from diraccfg import CFG
+
+import DIRAC
 from DIRAC.Core.Utilities.File import mkDir
 from DIRAC.Core.Utilities import List
 from DIRAC.Core.Utilities.ReturnValues import S_OK, S_ERROR
@@ -251,15 +252,13 @@ def getAutoPublish(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/AutoPublish", self.localCFG)
         if value and value.lower() in ("no", "false", "n"):
             return False
-        else:
-            return True
+        return True
 
     def getAutoSlaveSync(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/AutoSlaveSync", self.localCFG)
         if value and value.lower() in ("no", "false", "n"):
             return False
-        else:
-            return True
+        return True
 
     def getServers(self):
         return list(self.remoteServerList)
@@ -290,8 +289,7 @@ def isMaster(self):
         value = self.extractOptionFromCFG(f"{self.configurationPath}/Master", self.localCFG)
         if value and value.lower() in ("yes", "true", "y"):
             return True
-        else:
-            return False
+        return False
 
     def getServicesPath(self):
         return "/Services"