Merge pull request #291 from ministryofjustice/feature/glue-fixes

Bug fix: glue s3 path
ministryofjustice · Nov 10, 2023 · 2e3dd3c · 2e3dd3c
2 parents 7d9ac59 + c2c7fea
commit 2e3dd3c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/main.tf b/main.tf
@@ -349,6 +349,6 @@ resource "aws_glue_crawler" "ssm_resource_sync" {
   schedule      = var.log_schedule
 
   s3_target {
-    path = "s3://${var.existing_bucket_name}"
+    path = var.existing_bucket_name != "" ? "s3://${var.existing_bucket_name}/${var.application_name}/AWSLogs/${var.account_number}/elasticloadbalancing/" : "s3://${module.s3-bucket[0].bucket.id}/${var.application_name}/AWSLogs/${var.account_number}/elasticloadbalancing/"
   }
 }