Repository: spark
Updated Branches:
  refs/heads/branch-1.3 1cde04f21 -> ab1b8edb8


[SPARK-6636] Use public DNS hostname everywhere in spark_ec2.py

The spark_ec2.py script uses public_dns_name everywhere in the script except 
for testing ssh availability, which is done using the public ip address of the 
instances. This breaks the script for users who are deploying the cluster with 
a private-network-only security group. The fix is to use public_dns_name in the 
remaining place.

Author: Matt Aasted <aas...@twitch.tv>

Closes #5302 from aasted/master and squashes the following commits:

60cf6ee [Matt Aasted] [SPARK-6636] Use public DNS hostname everywhere in 
spark_ec2.py

(cherry picked from commit 6f0d55d76f758d217fd18ffa0ccf273d7ab0377b)
Signed-off-by: Josh Rosen <joshro...@databricks.com>


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

Branch: refs/heads/branch-1.3
Commit: ab1b8edb81feb2ad5cd28611f809c57cb61db9fb
Parents: 1cde04f
Author: Matt Aasted <aas...@twitch.tv>
Authored: Mon Apr 6 23:50:48 2015 -0700
Committer: Josh Rosen <joshro...@databricks.com>
Committed: Mon Apr 6 23:52:46 2015 -0700

----------------------------------------------------------------------
 ec2/spark_ec2.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/ab1b8edb/ec2/spark_ec2.py
----------------------------------------------------------------------
diff --git a/ec2/spark_ec2.py b/ec2/spark_ec2.py
index 95082b5..e121dad 100755
--- a/ec2/spark_ec2.py
+++ b/ec2/spark_ec2.py
@@ -715,7 +715,7 @@ def is_cluster_ssh_available(cluster_instances, opts):
     Check if SSH is available on all the instances in a cluster.
     """
     for i in cluster_instances:
-        if not is_ssh_available(host=i.ip_address, opts=opts):
+        if not is_ssh_available(host=i.public_dns_name, opts=opts):
             return False
     else:
         return True


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to