[GitHub] [accumulo-testing] ctubbsii commented on pull request #179: Replace use of `pssh` with `parallel-ssh`

[GitBox <gi...@apache.org>]

ctubbsii commented on pull request #179:
URL: https://github.com/apache/accumulo-testing/pull/179#issuecomment-995017925


   > > Fedora/CentOS/RHEL has not renamed this binary. It still ships as pssh.rpm and installs a binary at /usr/bin/pssh.
   > > An alternative to this would be to set a variable, `$PSSH` equal to `pssh` by default, and then you can set `export PSSH=parallel-ssh` in your own environment to override.
   > 
   > Gotcha, maybe this PR could add a note to the log stmts and/or Readme instead, explaining why this might be failing, if that seems like it would be helpful.
   
   If you change the implementation to provide the binary in a variable, like `$PSSH`, it would be totally appropriate to update the log message to indicate that `pssh` could not be found and that `$PSSH` is not set to an equivalent executable that exists, and to update the README to explain the requirement and how to set the `$PSSH` variable.
   
   Another alternative might be to automatically fall back to `parallel-ssh` if that [binary exists on the `$PATH` when `pssh` doesn't](https://stackoverflow.com/questions/592620/how-can-i-check-if-a-program-exists-from-a-bash-script).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org