Merge pull request #430 from KhiopsML/429-learningtesttool-fails-on-d…

…ebian10

429 learningtesttool fails on debian10

test/LearningTestTool/py/kht_test.py

@@ -330,39 +330,42 @@ def evaluate_tool_on_test_dir(
         os.environ[kht.KHIOPS_CRASH_TEST_MODE] = "true"
 
         # Ajout de variables d'environements propres a OpenMPI, elles remplacent les parametres
-        # on peut ansi lancer indiferemment mpich ou openmpi
+        # on peut ainsi lancer indiferemment mpich ou openmpi
         if platform.system() == "Linux":
             # Supprime les traces en cas d'erreur fatale de khiops. Option --quiet
             os.environ["OMPI_MCA_orte_execute_quiet"] = "true"
 
             # permet de lancer plus de processus qu'il n'y a de coeurs. Option --oversubscribe
             os.environ["OMPI_MCA_rmaps_base_oversubscribe"] = "true"
 
-            # permet de lancer en tant que root. Option --allow-run-as-root
-            os.environ["OMPI_ALLOW_RUN_AS_ROOT"] = "1"
-            os.environ["OMPI_ALLOW_RUN_AS_ROOT_CONFIRM"] = "1"
-
-        # Ajout de variables d'environements propres a OpenMPI, elles remplacent les parametres
-        if platform.system() == "Linux":
-            # Supprime les traces en cas d'erreur fatale de khiops. Option --quiet
-            os.environ["OMPI_MCA_orte_execute_quiet"] = "true"
-            # permet de lancer plus de processus qu'il n'y a de coeurs. Option --oversubscribe
-            os.environ["OMPI_MCA_rmaps_base_oversubscribe"] = "true"
-            # permet de lancer en tant que root. Option --allow-run-as-root
-            os.environ["OMPI_ALLOW_RUN_AS_ROOT"] = "1"
-            os.environ["OMPI_ALLOW_RUN_AS_ROOT_CONFIRM"] = "1"
-
         # Construction des parametres
         khiops_params = []
         if tool_process_number > 1:
             khiops_params.append(mpi_exe_name)
+
             # Option -l, specifique a mpich, valide au moins pour Windows:
             #    "Label standard out and standard error (stdout and stderr) with the rank of the process"
             if platform.system() == "Windows":
                 khiops_params.append("-l")
             if platform.system() == "Darwin":
                 khiops_params.append("-host")
                 khiops_params.append("localhost")
+
+            # Option --allow-run-as-root, specifique a OpenMPI,
+            # permet de lancer OpenMPI en tant que root de maniere portable sur
+            # tous les OS supportes.
+            # Cette option remplace le positionnement des variables
+            # d'environnement OMPI_ALLOW_RUN_AS_ROOT et
+            # OMPI_ALLOW_RUN_AS_ROOT_CONFIRM à '1'.
+            # Sous Debian 10, OpenMPI ne prend pas en compte ces deux variables
+            # d'environnement.
+            if platform.system() == "Linux":
+                # En iterant sur TOOL_MPI_SUFFIXES, on s'assure que "_openmpi"
+                # fait toujours partie des back-ends MPI supportes
+                for suffix in kht.TOOL_MPI_SUFFIXES:
+                    if tool_exe_path.endswith(suffix) and suffix == "_openmpi":
+                        khiops_params.append("--allow-run-as-root")
+                        break
             khiops_params.append("-n")
             khiops_params.append(str(tool_process_number))
         khiops_params.append(tool_exe_path)