LZA-163: Change bucket name references to use tf resource value and a…

…dd arn for iam role in replication rule
UKHomeOffice · Mar 12, 2024 · ba23f13 · ba23f13
1 parent 18cc3f5
commit ba23f13
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/modules/aws/cost_usage_reports/main.tf b/modules/aws/cost_usage_reports/main.tf
@@ -14,7 +14,7 @@ resource "aws_cur_report_definition" "cur_report_definitions" {
   format                     = var.format
   compression                = var.compression
   additional_schema_elements = var.additional_schema_elements
-  s3_bucket                  = var.bucket_name
+  s3_bucket                  = aws_s3_bucket.s3_buckets.id
   s3_region                  = var.bucket_region
   additional_artifacts       = var.additional_artifacts
   s3_prefix                  = var.s3_prefix
@@ -181,7 +181,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "cur_bucket_lifecycle_rule" {
 resource "aws_s3_bucket_replication_configuration" "cur_bucket_replication_rule" {
   depends_on = [aws_s3_bucket_versioning.versioning_rules]
   bucket     = var.bucket_name
-  role       = aws_iam_role.cur_role
+  role       = aws_iam_role.cur_role.arn
   rule {
     id = var.replication_rule