From cd8408ba82afabf4f2fe6f23dd809f1cf2e0675c Mon Sep 17 00:00:00 2001 From: Joseph Kyalo Date: Mon, 27 Oct 2025 22:14:53 +0300 Subject: [PATCH 1/2] add backups_bucket_uniform_level_access variable --- modules/gcp/velero/main.tf | 1 + modules/gcp/velero/variables.tf | 6 ++++++ 2 files changed, 7 insertions(+) diff --git a/modules/gcp/velero/main.tf b/modules/gcp/velero/main.tf index d5fd96e..af03499 100644 --- a/modules/gcp/velero/main.tf +++ b/modules/gcp/velero/main.tf @@ -3,6 +3,7 @@ resource "google_storage_bucket" "backups" { project = var.backup_project name = var.backups_bucket_name location = var.backups_bucket_location + uniform_bucket_level_access = var.backups_bucket_uniform_level_access soft_delete_policy { retention_duration_seconds = try(var.soft_delete_policy, 0) } diff --git a/modules/gcp/velero/variables.tf b/modules/gcp/velero/variables.tf index 3d4117c..6949fe9 100644 --- a/modules/gcp/velero/variables.tf +++ b/modules/gcp/velero/variables.tf @@ -27,3 +27,9 @@ variable "soft_delete_policy" { description = "The soft delete policy to use when deleting the bucket" type = number } + +variable "backups_bucket_uniform_level_access" { + description = "Whether to enable uniform bucket-level access for the backups bucket." + type = bool + default = true +} From 2476a2f1a7dc7d1acf2ea9bb60a6d4fbb7d05518 Mon Sep 17 00:00:00 2001 From: Joseph Kyalo Date: Tue, 28 Oct 2025 23:13:13 +0300 Subject: [PATCH 2/2] align indentations --- modules/gcp/velero/main.tf | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/modules/gcp/velero/main.tf b/modules/gcp/velero/main.tf index af03499..17ce2c9 100644 --- a/modules/gcp/velero/main.tf +++ b/modules/gcp/velero/main.tf @@ -1,11 +1,11 @@ # Create the velero backups bucket resource "google_storage_bucket" "backups" { - project = var.backup_project - name = var.backups_bucket_name - location = var.backups_bucket_location - uniform_bucket_level_access = var.backups_bucket_uniform_level_access + project = var.backup_project + name = var.backups_bucket_name + location = var.backups_bucket_location + uniform_bucket_level_access = var.backups_bucket_uniform_level_access soft_delete_policy { - retention_duration_seconds = try(var.soft_delete_policy, 0) + retention_duration_seconds = try(var.soft_delete_policy, 0) } }