Merge pull request #10: modified task definition memory size

Chores: added ECS redeployment trigger
IrezD · Jan 7, 2024 · 30417b3 · 30417b3
2 parents 54d342d + a39288b
commit 30417b3
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/main.py b/main.py
@@ -13,7 +13,7 @@ class Item(BaseModel):
 
 @app.get("/")
 def read_root():
-    return {"Message": "Hello world! Merry Christmas and a happy new year to everyone!!!"}
+    return {"Message": "Hello world! Welcome to FastAPI!!!"}
 
 
 @app.get("/items/{item_id}")

diff --git a/terraform/asg.tf b/terraform/asg.tf
@@ -36,6 +36,13 @@ resource "aws_autoscaling_group" "ASG_config" {
     propagate_at_launch = true
   }
 
+  tag {
+    key                 = "Name"
+    value               = "${title(var.env)}_Enivronment"
+    propagate_at_launch = true
+  }
+
+
   timeouts {
     delete = "15m"
   }

diff --git a/terraform/ecs.tf b/terraform/ecs.tf
@@ -40,14 +40,15 @@ TASK_DEFINITION
 
 }
 
-# ------ ECS Service -------- **
+# ------ ECS Service --------- **
 
 resource "aws_ecs_service" "fastapi-service" {
   name                 = var.ecs_service_name
   cluster              = aws_ecs_cluster.fastapi-cluster.id
-  task_definition      = aws_ecs_task_definition.task_definition.arn
+  task_definition      = aws_ecs_task_definition.task_definition.arn_without_revision
   desired_count        = 1
   launch_type          = "EC2"
+
   force_new_deployment = true
 
   load_balancer {