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

Commit 43acb08

Browse files
spcaipers-armnpalm
andauthored
fix(multi-runner): convertdistribution_bucket_name to lowercase (#3219)
refactor: force lowercase for distribution_bucket_name Co-authored-by: Niek Palm <[email protected]>
1 parent 775a548 commit 43acb08

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Diff for: modules/multi-runner/runner-binaries.tf

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ module "runner_binaries" {
44
prefix = "${var.prefix}-${each.value.os_type}-${each.value.architecture}"
55
tags = local.tags
66

7-
distribution_bucket_name = "${var.prefix}-${each.value.os_type}-${each.value.architecture}-dist-${random_string.random.result}"
7+
# force mandatory lower case for s3 bucketname
8+
distribution_bucket_name = lower("${var.prefix}-${each.value.os_type}-${each.value.architecture}-dist-${random_string.random.result}")
89

910
runner_os = each.value.os_type
1011
runner_architecture = each.value.architecture

0 commit comments

Comments
 (0)