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

Added t2 instance types #1446

Closed
wants to merge 2 commits into from
Closed

Added t2 instance types #1446

wants to merge 2 commits into from

Conversation

24601
Copy link

@24601 24601 commented Jul 16, 2014

New t2 instance types require HVM amis, bailout assumption of pvm
causes failures when using t2 instance types.

New t2 instance types require HVM amis, bailout assumption of pvm
causes failures when using t2 instance types.
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

"r3.8xlarge": "hvm",
"t2.micro": "hvm",
"t2.small":"hvm",
"t2.medium":"hvm"
Copy link
Contributor

Choose a reason for hiding this comment

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

Ca you format these correctly with the other ones?

Copy link
Author

Choose a reason for hiding this comment

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

Sure, commit 24601@01fe128 does this, I can submit another PR against that commit if needed.

Align values with others
@aarondav
Copy link
Contributor

Adding the commit to the branch made it already update this PR.

@pwendell
Copy link
Contributor

LGTM - thanks

@asfgit asfgit closed this in 7f87ab9 Jul 18, 2014
asfgit pushed a commit that referenced this pull request Jul 18, 2014
New t2 instance types require HVM amis, bailout assumption of pvm
causes failures when using t2 instance types.

Author: Basit Mustafa <basitmustafa@computes-things-for-basit.local>

Closes #1446 from 24601/master and squashes the following commits:

01fe128 [Basit Mustafa] Makin' it pretty
392a95e [Basit Mustafa] Added t2 instance types

Conflicts:

	ec2/spark_ec2.py
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
New t2 instance types require HVM amis, bailout assumption of pvm
causes failures when using t2 instance types.

Author: Basit Mustafa <basitmustafa@computes-things-for-basit.local>

Closes apache#1446 from 24601/master and squashes the following commits:

01fe128 [Basit Mustafa] Makin' it pretty
392a95e [Basit Mustafa] Added t2 instance types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants