Merge pull request #49 from Suprajaa-27/feature/dev

Merge feature/dev into main
Suprajaa-27 · Dec 22, 2023 · f4d5d99 · f4d5d99
2 parents b5b84bd + 36dded1
commit f4d5d99
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/terraform/iam.tf b/terraform/iam.tf
@@ -32,7 +32,7 @@ resource "aws_iam_policy" "cloud_watch_iam_policy" {
         "logs:CreateLogStream",
         "logs:PutLogEvents"
       ],
-      "Resource": "${aws_cloudwatch_log_group.app_log.arn}",
+      "Resource": "${aws_cloudwatch_log_group.app_log.arn}:*",
       "Effect": "Allow"
     }
   ]

diff --git a/terraform/log_group.tf b/terraform/log_group.tf
@@ -1,5 +1,5 @@
 resource "aws_cloudwatch_log_group" "app_log" {
-  name              = "/aws/lambda/${var.lambda_function_name}"
+  name              = "lambda_log"
   retention_in_days = 5
   lifecycle {
     prevent_destroy = false

diff --git a/terraform/main.tf b/terraform/main.tf
@@ -6,6 +6,10 @@ resource "aws_lambda_function" "s3_trigger_lambda_function" {
   runtime       = var.runtime
   role          = aws_iam_role.aws_lambda_iam_role.arn
   filename      = "${path.module}/lambda_function.zip"
+  depends_on = [ 
+    aws_cloudwatch_log_group.app_log,
+    aws_iam_role_policy_attachment.attach_cloud_watch_policy_to_lambda_role
+   ]
 }