[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-04-06 Thread voukka
Github user voukka closed the pull request at: https://github.com/apache/spark/pull/4038 --- 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

[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-04-06 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/4038#issuecomment-90395449 Superseded by https://github.com/apache/spark/pull/5244 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: SPARK-6588: Add --private-ips flag to EC2 scri...

2015-04-03 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/5244#issuecomment-89505748 @srowen my 2 cents: I like https://issues.apache.org/jira/browse/SPARK-6220 as a general solution better as it is naturally extensible, but it is a lot less user-friendly

[GitHub] spark pull request: SPARK-6588: Add --private-ips flag to EC2 scri...

2015-04-03 Thread voukka
Github user voukka commented on a diff in the pull request: https://github.com/apache/spark/pull/5244#discussion_r27766535 --- Diff: ec2/spark_ec2.py --- @@ -700,7 +704,8 @@ def get_instances(group_names): # Deploy configuration files and run setup scripts on a newly launched

[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-03-18 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/4038#issuecomment-82812271 I will try to update/resolve conflicts today/tomorrow. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-02-15 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/4038#issuecomment-74464588 @shivaram that's good point to store selected address type. @srowen I think it requires a bit more testing. Hopefully I'll get some time for that today -

[GitHub] spark pull request: [SPARK-5335] Fix deletion of security groups w...

2015-02-12 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/4122#issuecomment-74064263 I just rebased my fix and made changes according to review suggestions. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-5173]support python application running...

2015-01-30 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/3976#issuecomment-72173406 @lianhuiwang thank you for non-local files feature. Could admins review this PR and merge it, please? --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: fix for Spark-5335: Destroying cluster in VPC ...

2015-01-20 Thread voukka
GitHub user voukka opened a pull request: https://github.com/apache/spark/pull/4122 fix for Spark-5335: Destroying cluster in VPC with "--delete-groups" fails to remove security groups Please see https://issues.apache.org/jira/browse/SPARK-5335. The fix it

[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-01-14 Thread voukka
Github user voukka commented on the pull request: https://github.com/apache/spark/pull/4038#issuecomment-70044623 @nchammas yes, http://docs.aws.amazon.com/AmazonVPC/latest/UserGuide/VPC_Scenario2.html should be sufficient to test this bug. After you followed that guide, start new

[GitHub] spark pull request: Bug fix for SPARK-5242: "ec2/spark_ec2.py lauc...

2015-01-13 Thread voukka
GitHub user voukka opened a pull request: https://github.com/apache/spark/pull/4038 Bug fix for SPARK-5242: "ec2/spark_ec2.py lauch" does not work with VPC if no public DNS or IP is available The fix for https://issues.apache.org/jira/browse/SPARK-5242 This pull