Commit eb6f292
feat: add capacity type node groups to allow for spot nodes (#152)
<!--
~ Copyright 2023 StreamNative, Inc.
~
~ Licensed under the Apache License, Version 2.0 (the "License");
~ you may not use this file except in compliance with the License.
~ You may obtain a copy of the License at
~
~ http://www.apache.org/licenses/LICENSE-2.0
~
~ Unless required by applicable law or agreed to in writing, software
~ distributed under the License is distributed on an "AS IS" BASIS,
~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
implied.
~ See the License for the specific language governing permissions and
~ limitations under the License.
-->
<!--
### Contribution Checklist
- Fill out the template below to describe the changes contributed by the
pull request. That will give reviewers the context they need to do the
review.
- Each pull request should address only one issue, not mix up code from
multiple issues.
- Each commit in the pull request has a meaningful commit message
- Once all items of the checklist are addressed, remove the above text
and this checklist, leaving only the filled out template below.
**(The sections below can be removed for hotfixes of typos)**
-->
### Motivation
This PR will allow us to specify the capacity type, which is typically
"on-demand". There are some instances where we would like to make node
groups spot for cost savings on lower level environments.
### Modifications
I've exposed the `node_pool_capacity_type` which allows modification to
the capacity types of all node groups created via the
[eks-managed-node-group](https://github.com/terraform-aws-modules/terraform-aws-eks/tree/50cb230c8a1793f5ef4cc52c4c789b656b141168/modules/eks-managed-node-group#eks-managed-node-group-module)
submodule.
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
*(Please pick either of the following options)*
This change is a trivial rework / code cleanup without any test
coverage.
*(or)*
This change is already covered by existing tests, such as *(please
describe tests)*.
*(or)*
This change added tests and can be verified as follows:
*(example:)*
- *Added integration tests for end-to-end deployment with large payloads
(10MB)*
- *Extended integration test for recovery after broker failure*
### Documentation
Check the box below.
Need to update docs?
- [ ] `doc-required`
(If you need help on updating docs, create a doc issue)
- [ ] `no-need-doc`
(Please explain why)
- [ ] `doc`
(If this PR contains doc changes)
---------
Signed-off-by: Max Xu <[email protected]>
Co-authored-by: Max Xu <[email protected]>1 parent 0998bb4 commit eb6f292
3 files changed
+117
-107
lines changed
0 commit comments