Skip to content

Commit

Permalink
Merge pull request #682 from douglasjacobsen/ssh-pdsh
Browse files Browse the repository at this point in the history
Ensure ssh is used when executing pdsh commands
  • Loading branch information
linsword13 authored Oct 4, 2024
2 parents 67233a1 + b73978b commit 6f9efe7
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 6f9efe7

Please sign in to comment.