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

Doc change to support custom userdata option for docker-machine AWS driver #37

Merged
merged 1 commit into from
Oct 4, 2016

Conversation

mrburrito
Copy link
Contributor

Migrating docs changes from machine PR docker/machine#3779

Signed-off-by: Gordon Shankman gshankman@gmail.com

Signed-off-by: Gordon Shankman <gshankman@gmail.com>
@londoncalling
Copy link
Contributor

LGTM!

@nathanleclaire
Copy link
Contributor

Hey @londoncalling, it seems this should actually be on the vnext-machine branch.

@nathanleclaire
Copy link
Contributor

It will need to be reverted as this feature does not exist in machine release yet.

@londoncalling
Copy link
Contributor

Hey Nathan, I've been gone for a week, I'm back in tomorrow - will have a
look and change it. Thanks !

On Monday, October 24, 2016, Nathan LeClaire notifications@github.com
wrote:

Hey @londoncalling https://github.com/londoncalling, it seems this
should actually be on the vnext-machine branch.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#37 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALHuA1fNhwlIEigzI2Mc8RTIavjTgqLoks5q3Rt5gaJpZM4KOKH_
.

@londoncalling
Copy link
Contributor

👍

On Monday, October 24, 2016, Nathan LeClaire notifications@github.com
wrote:

It will need to be reverted as this feature does not exist in machine
release yet.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#37 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALHuAx8v9SNbQge3euIpKQ7SP7FBxXdTks5q3UOAgaJpZM4KOKH_
.

@nathanleclaire
Copy link
Contributor

Thanks!

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.

3 participants