Skip to content

Commit

Permalink
Merge pull request #6 from MaterializeInc/add-eks-ami-type
Browse files Browse the repository at this point in the history
Add eks AIM type var
  • Loading branch information
bobbyiliev authored Nov 21, 2024
2 parents bf65664 + e9825f6 commit b7b31df
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 0 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ The module has been tested with:
| <a name="input_mz_iam_role_name"></a> [mz\_iam\_role\_name](#input\_mz\_iam\_role\_name) | Name of the IAM role for Materialize S3 access (will be prefixed with environment name) | `string` | `"materialize-s3-role"` | no |
| <a name="input_mz_iam_service_account_name"></a> [mz\_iam\_service\_account\_name](#input\_mz\_iam\_service\_account\_name) | Name of the IAM user for Materialize service authentication (will be prefixed with environment name) | `string` | `"materialize-user"` | no |
| <a name="input_namespace"></a> [namespace](#input\_namespace) | Namespace for Materialize resources | `string` | `"materialize-environment"` | no |
| <a name="input_node_group_ami_type"></a> [node\_group\_ami\_type](#input\_node\_group\_ami\_type) | AMI type for the node group | `string` | `"AL2023_x86_64_STANDARD"` | no |
| <a name="input_node_group_capacity_type"></a> [node\_group\_capacity\_type](#input\_node\_group\_capacity\_type) | Capacity type for worker nodes (ON\_DEMAND or SPOT) | `string` | `"ON_DEMAND"` | no |
| <a name="input_node_group_desired_size"></a> [node\_group\_desired\_size](#input\_node\_group\_desired\_size) | Desired number of worker nodes | `number` | `2` | no |
| <a name="input_node_group_instance_types"></a> [node\_group\_instance\_types](#input\_node\_group\_instance\_types) | Instance types for worker nodes.<br/><br/>Recommended Configuration for Running Materialize with disk:<br/>- Tested instance types: `m6g`, `m7g` families (ARM-based Graviton instances)<br/>- AMI: AWS Bottlerocket (optimized for container workloads)<br/>- Note: Ensure instance store volumes are available and attached to the nodes for optimal performance with disk-based workloads. | `list(string)` | <pre>[<br/> "m6g.medium"<br/>]</pre> | no |
Expand Down
1 change: 1 addition & 0 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ module "eks" {
node_group_min_size = var.node_group_min_size
node_group_max_size = var.node_group_max_size
node_group_instance_types = var.node_group_instance_types
node_group_ami_type = var.node_group_ami_type
tags = var.tags
cluster_enabled_log_types = var.cluster_enabled_log_types
node_group_capacity_type = var.node_group_capacity_type
Expand Down
1 change: 1 addition & 0 deletions modules/eks/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ module "eks" {

instance_types = var.node_group_instance_types
capacity_type = var.node_group_capacity_type
ami_type = var.node_group_ami_type

name = "${var.environment}-mz"

Expand Down
6 changes: 6 additions & 0 deletions modules/eks/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,12 @@ variable "node_group_instance_types" {
type = list(string)
}

variable "node_group_ami_type" {
description = "AMI type for the node group"
type = string
default = "AL2023_x86_64_STANDARD"
}

variable "cluster_enabled_log_types" {
description = "List of desired control plane logging to enable"
type = list(string)
Expand Down
6 changes: 6 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,12 @@ variable "node_group_capacity_type" {
default = "ON_DEMAND"
}

variable "node_group_ami_type" {
description = "AMI type for the node group"
type = string
default = "AL2023_x86_64_STANDARD"
}

variable "cluster_enabled_log_types" {
description = "List of desired control plane logging to enable"
type = list(string)
Expand Down

0 comments on commit b7b31df

Please sign in to comment.