From f4ec65346a2c183bbc3651e0e90bfd09c7c1429d Mon Sep 17 00:00:00 2001
From: JoshuaLicense <JoshuaLicense@users.noreply.github.com>
Date: Thu, 18 Jul 2024 15:05:55 +0100
Subject: [PATCH] fix(terraform): add `jobRoleArn` to Batch containers (#181)

* fix(terraform): add `jobRoleArn` to Batch containers

* fix(terraform): add CW prefix for Batch logs
---
 infra/terraform/modules/service/batch.tf | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/infra/terraform/modules/service/batch.tf b/infra/terraform/modules/service/batch.tf
index 6cd4c3b195..59a3b3c7fc 100644
--- a/infra/terraform/modules/service/batch.tf
+++ b/infra/terraform/modules/service/batch.tf
@@ -59,6 +59,7 @@ locals {
       ],
 
       executionRoleArn = module.ecs_service["api"].task_exec_iam_role_arn
+      jobRoleArn       = module.ecs_service["api"].tasks_iam_role_arn
 
       logConfiguration = {
         logDriver = "awslogs"
@@ -110,6 +111,6 @@ module "batch" {
 }
 
 resource "aws_cloudwatch_log_group" "this" {
-  name              = "/aws/batch"
+  name              = "/aws/batch/vol-app-${var.environment}"
   retention_in_days = 1
 }