Merge pull request #270 from ewencp/really-force-ssh-pseudo-tty-master

Force pseudo-tty allocation in spark-ec2 script.

ssh commands need the -t argument repeated twice if there is no local
tty, e.g. if the process running spark-ec2 uses nohup and the parent
process exits.

Without this change, if you run the script this way (e.g. using nohup from a 
cron job), it will fail setting up the nodes because some of the ssh commands 
complain about missing ttys and then fail.

(This version is for the master branch. I've filed a separate request for the 
0.8 since changes to the script caused the patches to be different.)


Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/964a3b69
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/964a3b69
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/964a3b69

Branch: refs/heads/master
Commit: 964a3b6971716823f473f672611951d1e489a552
Parents: 883e034 d17c142
Author: Patrick Wendell <[email protected]>
Authored: Mon Dec 16 15:23:51 2013 -0800
Committer: Patrick Wendell <[email protected]>
Committed: Mon Dec 16 15:23:51 2013 -0800

----------------------------------------------------------------------
 ec2/spark_ec2.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


Reply via email to