Skip to content

Commit

Permalink
fix var
Browse files Browse the repository at this point in the history
  • Loading branch information
morga471 committed Jun 16, 2026
1 parent e8c6523 commit ee69df2
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ The module deploys Karpenter needed AWS resources, namely in IAM. It copies the
| Name | Description | Type | Default | Required |
|------|-------------|------|---------|:--------:|
| <a name="input_account_id"></a> [account\_id](#input\_account\_id) | aws account number | `string` | `""` | no |
| <a name="input_ami_type"></a> [ami\_type](#input\_ami\_type) | AMI type to use for the Karpenter node group | `string` | `"AL2023_x86_64"` | no |
| <a name="input_ami_type"></a> [ami\_type](#input\_ami\_type) | AMI type to use for the Karpenter node group | `string` | `"AL2023_x86_64_STANDARD"` | no |
| <a name="input_cluster_endpoint"></a> [cluster\_endpoint](#input\_cluster\_endpoint) | Endpoint for your Kubernetes API server | `string` | n/a | yes |
| <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | EKS cluster name name component used through out the EKS cluster describing its purpose (ex: dice-dev) | `string` | n/a | yes |
| <a name="input_create_spot_service_linked_role"></a> [create\_spot\_service\_linked\_role](#input\_create\_spot\_service\_linked\_role) | Whether to create the service-linked role for EC2 Spot (required for Karpenter spot instances) | `bool` | `false` | no |
Expand Down
3 changes: 1 addition & 2 deletions main.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
locals {
ami_type_family = upper(split("_", var.ami_type)[0])
amd_ami_family = local.ami_type_family == "BOTTLEROCKET" ? "Bottlerocket" : local.ami_type_family
amd_ami_family = local.ami_type_family
amd_ami_alias = lower(format("%v@latest", local.amd_ami_family))

# Calculate the role name with prefix
Expand All @@ -18,7 +18,6 @@ locals {
rule_name_prefix = length(local.rule_name_raw) > local.max_rule_name_length ? substr(local.rule_name_raw, 0, local.max_rule_name_length) : local.rule_name_raw
queue_name = format("%v%v", local.prefixes["eks-queue"], var.cluster_name)
subnet_ids = try(tolist(var.subnets), [])

}

# Replicating from here: https://github.e.it.census.gov/SCT-Engineering/terraform-aws-eks/tree/master/modules/karpenter
Expand Down
2 changes: 1 addition & 1 deletion variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -101,5 +101,5 @@ variable "create_spot_service_linked_role" {
variable "ami_type" {
description = "AMI type to use for the Karpenter node group"
type = string
default = "AL2023_x86_64"
default = "AL2023_x86_64_STANDARD"
}

0 comments on commit ee69df2

Please sign in to comment.