-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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(ec2): support for r7i instance type in aws-ec2 #27737
Merged
Merged
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
a72ec34
add r7i option to ec2 instance types
kevntao 26f98b7
Merge branch 'main' into kevntao/ec2-r7i
kevntao a06c984
Merge branch 'main' into kevntao/ec2-r7i
kevntao 13ce290
Merge branch 'main' into kevntao/ec2-r7i
kevntao 3afe9a3
Merge branch 'main' into kevntao/ec2-r7i
kevntao 5219fcd
Merge branch 'main' into kevntao/ec2-r7i
kevntao 29a9cbd
switch r7i to MEMORY7_INTEL_BASE
kevntao 0d7c6f5
Merge branch 'main' into kevntao/ec2-r7i
kevntao c741408
Merge branch 'main' into kevntao/ec2-r7i
kevntao 4231c0c
Merge branch 'main' into kevntao/ec2-r7i
kevntao 6afaba6
Merge branch 'main' into kevntao/ec2-r7i
kevntao f9473ee
Merge branch 'main' into kevntao/ec2-r7i
kevntao 5752fc8
Merge branch 'main' into kevntao/ec2-r7i
kevntao 38d9458
Update packages/aws-cdk-lib/aws-ec2/lib/instance-types.ts
scanlonp 25906bb
Update packages/aws-cdk-lib/aws-ec2/lib/instance-types.ts
scanlonp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets change this to not be the same as r7iz. Or if the better change would be to that doc string, do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i agree, i had difficulty finding the nuanced difference between the
r7i
andr7iz
, unfortunately you're correct, theMEMORY7_INTEL
should have gone to ther7i
instead of ther7iz
, but I had thought ther7iz
was the replacement for ther7i
since it used the same processor generation as thec7i
andm7i
.I'm totally open to suggestions, but happy to take a stab at it for community review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that mistake is on me. Do what you think looks good and we can go from there!