feat: Add supprot for creating placement group for managed node group #2959
Conversation
james-masson
force-pushed
the
feat/placement-groups
branch
from
75e2d88
to
d4235c9
Compare
|
the clustered placement group for EFA is a requirement - what is the reason for needing other placement group (i.e. - spread) functionality? |
|
We run low-latency workloads, and need placement groups to co-locate all the members of the nodegroup. @bryantbiggs Your EFA PR allowed management of placement groups directly from the module, rather than just being a pre-supplied thing. However the rest of your change heavily tied placement groups to EFA only - when it could be a more generic feature. This PR allows anyone to use placement groups with any policy they like, without enabling the rest of your EFA support. The functionality of your EFA change is intact. |
bryantbiggs
changed the title
## [20.7.0](v20.6.0...v20.7.0) (2024-03-09) ### Features * Add supprot for creating placement group for managed node group ([#2959](#2959)) ([3031631](3031631))
|
This PR is included in version 20.7.0 🎉 |
|
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. |
Description
Adds the ability to have a placement group created for your managed node group, with configurable strategy.
Motivation and Context
The EFA feature added placement groups in a very specific way, this makes placement groups available for use outside of EFA.
Breaking Changes
None
How Has This Been Tested?
examples/*to demonstrate and validate my change(s)examples/*projectspre-commit run -aon my pull request