Skip to content

Commit

Permalink
Fix: bucket name length exceeds 63 characters with non-US region depl…
Browse files Browse the repository at this point in the history
…oyment (#1347)
  • Loading branch information
mariammartins authored Nov 6, 2024
1 parent 0bc707b commit d295755
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion 1-org/modules/cai-monitoring/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ module "cloudfunction_source_bucket" {
version = "~> 6.0"

project_id = var.project_id
name = "bkt-cai-monitoring-${random_id.suffix.hex}-sources-${data.google_project.project.number}-${var.location}"
name = "bkt-cai-monitoring-${random_id.suffix.hex}-sources-${data.google_project.project.number}"
location = var.location
storage_class = "REGIONAL"
force_destroy = true
Expand Down
4 changes: 2 additions & 2 deletions 4-projects/modules/base_env/example_storage_cmek.tf
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,11 @@ resource "random_string" "bucket_name" {

module "gcs_buckets" {
source = "terraform-google-modules/cloud-storage/google//modules/simple_bucket"
version = "~> 6.0"
version = "~> 6.0.0"

project_id = module.base_shared_vpc_project.project_id
location = var.location_gcs
name = "${var.gcs_bucket_prefix}-${module.base_shared_vpc_project.project_id}-${lower(var.location_gcs)}-cmek-encrypted-${random_string.bucket_name.result}"
name = "${var.gcs_bucket_prefix}-${module.base_shared_vpc_project.project_id}-cmek-encrypted-${random_string.bucket_name.result}"
bucket_policy_only = true
custom_placement_config = var.gcs_custom_placement_config

Expand Down

0 comments on commit d295755

Please sign in to comment.