-
Notifications
You must be signed in to change notification settings - Fork 199
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
Gpu Builder Instance Type prop fix #834
Conversation
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.
@Howlla Thanks for the fix, couple of minor changes
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.
BlockSize is needed for nodegroup
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.
@Howlla Could you please make sure you tar
the blueprints and test it on patterns. Since there is no e2e for builder classes, we need to test it before we can merge.
Sure
…On Mon, Sep 4, 2023 at 4:24 PM Ela AWS ***@***.***> wrote:
***@***.**** commented on this pull request.
@Howlla <https://github.com/Howlla> Could you please make sure you tar
the blueprints and test it on patterns. Since there is no e2e for builder
classes, we need to test it before we can merge.
—
Reply to this email directly, view it on GitHub
<#834 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHEWA6GONI2GNZQO5NOAQXDXYZBJPANCNFSM6AAAAAA4DYAW5M>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@elamaran11 Tested in the gpu monitoring pattern here aws-samples/cdk-eks-blueprints-patterns#141 |
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.
LGTM
Issue #, if available:
Description of changes: Concatenate instanceClass and instanceSize for instanceType prop
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.