Skip to content
This repository was archived by the owner on May 17, 2021. It is now read-only.

Commit 52b3b57

Browse files
barryibBARRY Thierno Ibrahima (Canal Plus Prestataire)
authored and
BARRY Thierno Ibrahima (Canal Plus Prestataire)
committed
fix: Use the amazon ImageOwnerAlias for worker ami owner instead of owner id (terraform-aws-modules#1038)
1 parent 6b17e30 commit 52b3b57

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -213,8 +213,8 @@ MIT Licensed. See [LICENSE](https://github.com/terraform-aws-modules/terraform-a
213213
| worker\_additional\_security\_group\_ids | A list of additional security group ids to attach to worker instances | `list(string)` | `[]` | no |
214214
| worker\_ami\_name\_filter | Name filter for AWS EKS worker AMI. If not provided, the latest official AMI for the specified 'cluster\_version' is used. | `string` | `""` | no |
215215
| worker\_ami\_name\_filter\_windows | Name filter for AWS EKS Windows worker AMI. If not provided, the latest official AMI for the specified 'cluster\_version' is used. | `string` | `""` | no |
216-
| worker\_ami\_owner\_id | The ID of the owner for the AMI to use for the AWS EKS workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft'). | `string` | `"602401143452"` | no |
217-
| worker\_ami\_owner\_id\_windows | The ID of the owner for the AMI to use for the AWS EKS Windows workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft'). | `string` | `"801119661308"` | no |
216+
| worker\_ami\_owner\_id | The ID of the owner for the AMI to use for the AWS EKS workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft'). | `string` | `"amazon"` | no |
217+
| worker\_ami\_owner\_id\_windows | The ID of the owner for the AMI to use for the AWS EKS Windows workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft'). | `string` | `"amazon"` | no |
218218
| worker\_create\_cluster\_primary\_security\_group\_rules | Whether to create security group rules to allow communication between pods on workers and pods using the primary cluster security group. | `bool` | `false` | no |
219219
| worker\_create\_initial\_lifecycle\_hooks | Whether to create initial lifecycle hooks provided in worker groups. | `bool` | `false` | no |
220220
| worker\_create\_security\_group | Whether to create a security group for the workers or attach the workers to `worker_security_group_id`. | `bool` | `true` | no |

variables.tf

+2-2
Original file line numberDiff line numberDiff line change
@@ -128,13 +128,13 @@ variable "worker_ami_name_filter_windows" {
128128
variable "worker_ami_owner_id" {
129129
description = "The ID of the owner for the AMI to use for the AWS EKS workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft')."
130130
type = string
131-
default = "602401143452" // The ID of the owner of the official AWS EKS AMIs.
131+
default = "amazon"
132132
}
133133

134134
variable "worker_ami_owner_id_windows" {
135135
description = "The ID of the owner for the AMI to use for the AWS EKS Windows workers. Valid values are an AWS account ID, 'self' (the current account), or an AWS owner alias (e.g. 'amazon', 'aws-marketplace', 'microsoft')."
136136
type = string
137-
default = "801119661308" // The ID of the owner of the official AWS EKS Windows AMIs.
137+
default = "amazon"
138138
}
139139

140140
variable "worker_additional_security_group_ids" {

0 commit comments

Comments
 (0)