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

feat: Use gp3 as default as it saves 20% and is more performant #1134

Merged
merged 5 commits into from
Jan 28, 2021

Conversation

jayzalowitz
Copy link
Contributor

@jayzalowitz jayzalowitz commented Dec 3, 2020

PR o'clock

Provider has caught up hashicorp/terraform-provider-aws#16652, thanks @ewbankkit

Description

AWS now has a cheaper and more performant drive type:
https://aws.amazon.com/about-aws/whats-new/2020/12/introducing-new-amazon-ebs-general-purpose-volumes-gp3/
Re: #1130

Checklist

@jayzalowitz jayzalowitz changed the title Add gp3 as default as it saves 20% and is more performant fix: Add gp3 as default as it saves 20% and is more performant Dec 3, 2020
@jayzalowitz
Copy link
Contributor Author

@antonbabenko Any suggestion on getting that pull request linter working?

@barryib barryib changed the title fix: Add gp3 as default as it saves 20% and is more performant feat: Use gp3 as default as it saves 20% and is more performant Dec 22, 2020
@barryib
Copy link
Member

barryib commented Dec 23, 2020

This is a breking change. I will include this in the next major release (It's on his way ^^).

@barryib barryib merged commit 76537d1 into terraform-aws-modules:master Jan 28, 2021
barryib added a commit that referenced this pull request Jan 29, 2021
@barryib
Copy link
Member

barryib commented Jan 29, 2021

I'm going to revert this change, since it breaks launch configuration usage. See #1205 for more info.

cc @jayzalowitz

@marcusramberg
Copy link

Just hit this bug, any chance of a new release with that revert?

ArchiFleKs pushed a commit to ArchiFleKs/terraform-aws-eks that referenced this pull request Apr 16, 2021
…rraform-aws-modules#1134)

BREAKING CHANGES: The default root volume type is now `gp3` as it saves 20% and is more performant
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants