You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by whenceforth <gi...@git.apache.org> on 2014/04/18 08:19:52 UTC

[GitHub] spark pull request: SPARK-1532 Provide option in ec2/spark_ec2.py ...

GitHub user whenceforth opened a pull request:

    https://github.com/apache/spark/pull/445

    SPARK-1532 Provide option in ec2/spark_ec2.py to set more restrictive Source in the security groups.

    SPARK-1532 add option to ec2/spark_ec2.py to permit more restrictive IP address to EC2 instance.
    
    The new option is --ips-allowed and it defaults to 0.0.0.0/0 to
    match the existing behavior of ec2/spark_ec2.py

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/whenceforth/spark access_mask_option

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/445.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #445
    
----
commit 8112708ea5167fa4cd7ea0abf343907d6bd3fd46
Author: Art Peel <ar...@safeshepherd.com>
Date:   2014-04-14T20:42:48Z

    Provide option to set more restrictive Source in the security groups.
    
    The new option is --ips-allowed and it defaults to 0.0.0.0/0 to
    match the existing behavior of the script.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] spark pull request: SPARK-1532 Provide option in ec2/spark_ec2.py ...

Posted by whenceforth <gi...@git.apache.org>.
Github user whenceforth closed the pull request at:

    https://github.com/apache/spark/pull/445


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] spark pull request: SPARK-1532 Provide option in ec2/spark_ec2.py ...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/445#issuecomment-40789732
  
    Can one of the admins verify this patch?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] spark pull request: SPARK-1532 Provide option in ec2/spark_ec2.py ...

Posted by whenceforth <gi...@git.apache.org>.
Github user whenceforth commented on the pull request:

    https://github.com/apache/spark/pull/445#issuecomment-40861728
  
    Travis CI failed due to a timeout compiling scala code.  My change is only in the python script for setting up a spark cluster on EC2 so I find it hard to believe they are related.  I'm closing this and will generate another to trigger a new build in hopes that the timeout will not occur a second time.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---