LZA-163: Updates from testing

UKHomeOffice · Mar 13, 2024 · 2492075 · 2492075
1 parent 110ebf4
commit 2492075
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/modules/aws/cost_usage_reports/main.tf b/modules/aws/cost_usage_reports/main.tf
@@ -28,7 +28,7 @@ resource "aws_cur_report_definition" "cur_report_definitions" {
   s3_bucket                  = aws_s3_bucket.s3_buckets.id
   s3_region                  = var.bucket_region
   additional_artifacts       = var.additional_artifacts
-  s3_prefix                  = var.s3_prefix
+  s3_prefix                  = "cur/${var.billing_account}"
   refresh_closed_reports     = var.refresh_closed_reports
   report_versioning          = var.report_versioning
 }

diff --git a/modules/aws/cost_usage_reports/variables.tf b/modules/aws/cost_usage_reports/variables.tf
@@ -78,16 +78,6 @@ variable "additional_artifacts" {
   }
 }
 
-variable "s3_prefix" {
-  type        = string
-  description = "Report path prefix."
-
-  validation {
-    condition     = length(var.s3_prefix) >= 1 && length(var.s3_prefix) <= 256
-    error_message = "The s3_prefix must be less than 256 characters."
-  }
-}
-
 variable "refresh_closed_reports" {
   type        = string
   description = "Set to true to update your reports after they have been finalized if AWS detects charges related to previous months."