Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Commit

Permalink
fix: Replace defaul AMI for WIndows (#3445)
Browse files Browse the repository at this point in the history
Replace Window AMI by  Windows_Server-2022-English-Full-ECS_Optimized-*

close #3423
  • Loading branch information
npalm authored Aug 22, 2023
1 parent 00e5479 commit a590da5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@ runner_config:
runner_boot_time_in_minutes: 20
ami_filter:
name:
- Windows_Server-2022-English-Core-ContainersLatest-*
- Windows_Server-2022-English-Full-ECS_Optimized-*
state:
- available
2 changes: 1 addition & 1 deletion modules/runners/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ locals {
kms_key_arn = var.kms_key_arn != null ? var.kms_key_arn : ""
s3_location_runner_distribution = var.enable_runner_binaries_syncer ? "s3://${var.s3_runner_binaries.id}/${var.s3_runner_binaries.key}" : ""
default_ami = {
"windows" = { name = ["Windows_Server-2022-English-Core-ContainersLatest-*"] }
"windows" = { name = ["Windows_Server-2022-English-Full-ECS_Optimized-*"] }
"linux" = var.runner_architecture == "arm64" ? { name = ["amzn2-ami-kernel-5.*-hvm-*-arm64-gp2"] } : { name = ["amzn2-ami-kernel-5.*-hvm-*-x86_64-gp2"] }
}

Expand Down

0 comments on commit a590da5

Please sign in to comment.