Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support selection of private ip #21

Merged
merged 11 commits into from
Jan 27, 2014

Conversation

Atalanta
Copy link
Contributor

When operating in a VPC environment, in which the security group does not allow access via SSH to the external IP address of an instance, it is necessary to be able to enforce the use of the private IP address.

This patch makes a config setting available to the EC2 driver which defaults to 'public' but which may be set to 'private'.

@sethvargo
Copy link
Contributor

@Atalanta (and @LordCope), this will not merge cleanly now that the hostname has moved to its own method and includes support for DNS names. Would you like to take a shot at rebasing this logic?

@LordCope
Copy link

LordCope commented Jan 4, 2014

Yep. Will take a look next week when I'm back in the office.

@sethvargo
Copy link
Contributor

@LordCope pingles 😄

@Atalanta
Copy link
Contributor Author

Refactored and added tests.

@@ -19,7 +19,7 @@
require 'benchmark'
require 'json'
require 'fog'

require 'pry'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry - this can probably come out...

@fnichol
Copy link
Contributor

fnichol commented Jan 27, 2014

After pairing on this, we're ready to merge

fnichol added a commit that referenced this pull request Jan 27, 2014
@fnichol fnichol merged commit de67adb into test-kitchen:master Jan 27, 2014
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.

4 participants