Skip to content

Conversation

@douglaz
Copy link
Contributor

@douglaz douglaz commented Aug 21, 2014

Adds the --authorized-address and --additional-security-group options as explained in the issue.

…d another option to provide an additional existing SG
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@JoshRosen
Copy link
Contributor

Jenkins, this is ok to test.

@SparkQA
Copy link

SparkQA commented Aug 24, 2014

QA tests have started for PR 2088 at commit e3e48ca.

  • This patch merges cleanly.

@JoshRosen
Copy link
Contributor

This looks like a reasonable feature. I'll let Jenkins run just to be safe (there's no tests for this code, but I think the stylechecker still runs), then merge this if everything looks good. Thanks!

@SparkQA
Copy link

SparkQA commented Aug 24, 2014

QA tests have finished for PR 2088 at commit e3e48ca.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class BoundedFloat(float):

@pwendell
Copy link
Contributor

Yeah this is a good idea - thanks @douglaz for contributing this.

@JoshRosen
Copy link
Contributor

LGTM, so I've merged this into master. Thanks!

@asfgit asfgit closed this in cc40a70 Aug 25, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Adds the --authorized-address and --additional-security-group options as explained in the issue.

Author: Allan Douglas R. de Oliveira <[email protected]>

Closes apache#2088 from douglaz/configurable_sg and squashes the following commits:

e3e48ca [Allan Douglas R. de Oliveira] Adds the option to specify the address authorized to access the SG and another option to provide an additional existing SG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants