You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by pw...@apache.org on 2013/12/17 00:23:34 UTC
[1/2] git commit: Force pseudo-tty allocation in spark-ec2 script.
Updated Branches:
refs/heads/branch-0.8 8f5639077 -> f898238a2
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.
Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/2e2ead40
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/2e2ead40
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/2e2ead40
Branch: refs/heads/branch-0.8
Commit: 2e2ead402003e77e78961583581b28d106f65c12
Parents: 8f56390
Author: Ewen Cheslack-Postava <me...@ewencp.org>
Authored: Mon Dec 16 08:07:20 2013 -0800
Committer: Ewen Cheslack-Postava <me...@ewencp.org>
Committed: Mon Dec 16 08:16:56 2013 -0800
----------------------------------------------------------------------
ec2/spark_ec2.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-spark/blob/2e2ead40/ec2/spark_ec2.py
----------------------------------------------------------------------
diff --git a/ec2/spark_ec2.py b/ec2/spark_ec2.py
index d508ab3..dfee9bf 100755
--- a/ec2/spark_ec2.py
+++ b/ec2/spark_ec2.py
@@ -565,7 +565,7 @@ def ssh(host, opts, command):
while True:
try:
return subprocess.check_call(
- "ssh -t -o StrictHostKeyChecking=no -i %s %s@%s '%s'" %
+ "ssh -t -t -o StrictHostKeyChecking=no -i %s %s@%s '%s'" %
(opts.identity_file, opts.user, host, command), shell=True)
except subprocess.CalledProcessError as e:
if (tries > 2):
[2/2] git commit: Merge pull request #271 from
ewencp/really-force-ssh-pseudo-tty-0.8
Posted by pw...@apache.org.
Merge pull request #271 from ewencp/really-force-ssh-pseudo-tty-0.8
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 0.8 branch. I've filed a separate request for master 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/f898238a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/f898238a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/f898238a
Branch: refs/heads/branch-0.8
Commit: f898238a26d2f16796dda51212baf3f267747783
Parents: 8f56390 2e2ead4
Author: Patrick Wendell <pw...@gmail.com>
Authored: Mon Dec 16 15:23:25 2013 -0800
Committer: Patrick Wendell <pw...@gmail.com>
Committed: Mon Dec 16 15:23:25 2013 -0800
----------------------------------------------------------------------
ec2/spark_ec2.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------