-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
issue-1146 add support for instance fleet in EMR #14758
Conversation
Thank you for your hard work on this, @c4po! 😄 I would love to continue working with you on these EMR pull requests and ensure they get merged in the next few releases. 👍 To simplify things for both of us, could you split this PR into two separate ones? One for the initial instance fleet handling in the |
@bflad I can split this into 2 PRs. But the EMR cluster can only have one type of instance group. If we have instance fleet for master and core node, we must provide instance fleet for task node. so these 2 part of changes need be add together to make it works. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Closes #1146
Release note for CHANGELOG:
Output from acceptance testing: