Correctly translate the plm_rsh MCA params

Need to convert to plm_ssh Signed-off-by: Ralph Castain <[email protected]>
openpmix · Nov 17, 2023 · c5db39a · c5db39a
1 parent 361592b
commit c5db39a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/mca/schizo/base/schizo_base_stubs.c b/src/mca/schizo/base/schizo_base_stubs.c
@@ -327,6 +327,10 @@ int prte_schizo_base_parse_prte(int argc, int start, char **argv, char ***target
                     pmix_asprintf(&param, "prtedl_%s", &p1[strlen("dl_")]);
                     free(p1);
                     p1 = param;
+                } else if (0 == strncasecmp(p1, "plm_rsh", strlen("plm_rsh"))) {
+                    pmix_asprintf(&param, "plm_ssh_%s", &p1[strlen("plm_rsh_")]);
+                    free(p1);
+                    p1 = param;
                 }
                 if (NULL == target) {
                     /* push it into our environment */

diff --git a/src/mca/schizo/ompi/schizo_ompi.c b/src/mca/schizo/ompi/schizo_ompi.c
@@ -1807,6 +1807,14 @@ static bool check_prte_overlap(char *var, char *value)
         setenv(tmp, value, false);
         free(tmp);
         return true;
+    } else if (0 == strncmp(var, "plm_rsh_", strlen("plm_rsh_"))) {
+        // need to convert rsh to ssh
+        pmix_asprintf(&tmp, "PRTE_MCA_plm_ssh_%s", &var[strlen("plm_rsh_")]);
+        // set it, but don't overwrite if they already
+        // have a value in our environment
+        setenv(tmp, value, false);
+        free(tmp);
+        return true;
     } else if (0 == strncmp(var, "orte_", strlen("orte_"))) {
         // need to convert "orte" to "prte"
         pmix_asprintf(&tmp, "PRTE_MCA_prte_%s", &var[strlen("orte_")]);