scr icon indicating copy to clipboard operation
scr copied to clipboard

consider pdsh as optional dependency

Open adammoody opened this issue 5 years ago • 2 comments

pdsh is not installed on many systems. In the spirit of simplifying the scr install, it might be nice to make pdsh an optional dependency.

pdsh is used in two places:

  1. to check node health when restarting a job within an allocation
  2. to execute the scavenge logic

If someone does not care about in-place restart or scavenge, then we could make pdsh an optional dependency. We'd need to modify the scripts to avoid executing the pdsh logic in that case.

adammoody avatar Nov 10 '20 23:11 adammoody

I've set the spack package up to make +pdsh an optional parameter for SCR.

It looks like none of the scripts that I've checked so far do anything at all without pdsh. Do you have some replacement mechanism in mind. If not, I would propose that each script check for the existence of PDSH_EXE at launch, and if it is not defined, to print a message and quit.

robertkb avatar Dec 09 '20 17:12 robertkb

Saw a note that pdsh is being phased out by clush (https://github.com/cea-hpc/clustershell).

CamStan avatar Mar 09 '21 01:03 CamStan