Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

[IMP] Add EC2 instance tenancy#4356

Open
lasley wants to merge 1 commit intodocker-archive-public:masterfrom
lasley:feature/ec2-host-type
Open

[IMP] Add EC2 instance tenancy#4356
lasley wants to merge 1 commit intodocker-archive-public:masterfrom
lasley:feature/ec2-host-type

Conversation

@lasley
Copy link
Copy Markdown

@lasley lasley commented Jan 4, 2018

  • Add option to control EC2 instance placement tenancy (default, dedicated, host)

I admittedly haven't compiled or tested this yet, but the theory seems sound. Is this something you would accept once tested? If so, is there anything I'm missing from a quick glance?

@GordonTheTurtle
Copy link
Copy Markdown

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "feature/ec2-host-type" git@github.com:lasley/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@lasley lasley changed the title [IMP] Add instance tenancy [IMP] Add EC2 instance tenancy Jan 4, 2018
@lasley lasley force-pushed the feature/ec2-host-type branch from 0a2a1d3 to 8002b84 Compare January 4, 2018 22:29
@lasley lasley force-pushed the feature/ec2-host-type branch from 8002b84 to 22a769a Compare January 4, 2018 22:30
@lasley lasley force-pushed the feature/ec2-host-type branch from 22a769a to 3b28cdf Compare January 4, 2018 22:33
* Add option to control EC2 instance placement tenancy (default, dedicated, host)

Signed-off-by: Dave Lasley <dave@laslabs.com>
@lasley lasley force-pushed the feature/ec2-host-type branch from 3b28cdf to c488220 Compare January 4, 2018 22:51
@dgageot
Copy link
Copy Markdown
Contributor

dgageot commented Feb 22, 2018

@lasley You need to amend the commit so that the code is formatted according to the project rules. See https://github.com/docker/machine/blob/master/CONTRIBUTING.md#tests-and-validation

@dgageot dgageot self-assigned this Feb 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants