Skip to content

Commit

Permalink
Ensure ssh is used when executing pdsh commands
Browse files Browse the repository at this point in the history
This commit configured the rcmd module for pdsh to be ssh in existing
pdsh uses.

This gets around an issue where a machine might not have a default rcmd
module, and as a result pdsh fails to execute remote commands.
  • Loading branch information
douglasjacobsen committed Oct 4, 2024
1 parent c376093 commit b73978b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ def gcp_metadata_exec(self, executable_name, executable, app_inst=None):
prefix = ""
suffix = ""
if per_node:
prefix = "pdsh -N -w {hostlist} '"
prefix = "pdsh -R ssh -N -w {hostlist} '"
suffix = "'"
log_name = end_point.split("/")[-1]
pre_cmds.append(
Expand Down
4 changes: 2 additions & 2 deletions var/ramble/repos/builtin/modifiers/tuned-adm/modifier.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ class TunedAdm(BasicModifier):

def set_tuning_profile(self):
return [
"pdsh -w {hostlist} sudo tuned-adm profile {tuned-profile}",
"pdsh -w {hostlist} sudo tuned-adm active > {experiment_run_dir}/tuning_profile",
"pdsh -R ssh -w {hostlist} sudo tuned-adm profile {tuned-profile}",
"pdsh -R ssh -w {hostlist} sudo tuned-adm active > {experiment_run_dir}/tuning_profile",
]

def _prepare_analysis(self, workspace):
Expand Down

0 comments on commit b73978b

Please sign in to comment.