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

Commit

Permalink
fix(multi-runner): convertdistribution_bucket_name to lowercase (#3219)
Browse files Browse the repository at this point in the history
refactor: force lowercase for distribution_bucket_name

Co-authored-by: Niek Palm <[email protected]>
  • Loading branch information
spcaipers-arm and npalm authored Apr 28, 2023
1 parent 775a548 commit 43acb08
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion modules/multi-runner/runner-binaries.tf
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ module "runner_binaries" {
prefix = "${var.prefix}-${each.value.os_type}-${each.value.architecture}"
tags = local.tags

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

runner_os = each.value.os_type
runner_architecture = each.value.architecture
Expand Down

0 comments on commit 43acb08

Please sign in to comment.