Merge pull request #108 from uktrade/feat/bump-celery-memory

feat: increase memory on Celery
uktrade · Jul 19, 2024 · 84ec2f0 · 84ec2f0
2 parents 9ba955f + a7d53a8
commit 84ec2f0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/infra/ecs_main_admin.tf b/infra/ecs_main_admin.tf
@@ -1,10 +1,9 @@
 locals {
   admin_container_vars = {
-    container_image  = "${aws_ecr_repository.admin.repository_url}:${data.external.admin_current_tag.result.tag}"
-    container_name   = "${local.admin_container_name}"
-    container_port   = "${local.admin_container_port}"
-    container_cpu    = "${local.admin_container_cpu}"
-    container_memory = "${local.admin_container_memory}"
+    container_image = "${aws_ecr_repository.admin.repository_url}:${data.external.admin_current_tag.result.tag}"
+    container_name  = "${local.admin_container_name}"
+    container_port  = "${local.admin_container_port}"
+    container_cpu   = "${local.admin_container_cpu}"
 
     log_group  = "${aws_cloudwatch_log_group.admin.name}"
     log_region = "${data.aws_region.aws_region.name}"

diff --git a/infra/ecs_main_admin_container_definitions.json b/infra/ecs_main_admin_container_definitions.json
@@ -1008,7 +1008,6 @@
         "awslogs-stream-prefix": "${container_name}"
       }
     },
-    "memoryReservation": ${container_memory},
     "mountPoints": [],
     "name": "${container_name}",
     "portMappings": [{

diff --git a/infra/main.tf b/infra/main.tf
@@ -242,7 +242,7 @@ locals {
   admin_alb_port         = "443"
   admin_api_path         = "/api/v1/databases"
 
-  celery_container_memory = 4096
+  celery_container_memory = 8192
   celery_container_cpu    = 1024
 
   notebook_container_name     = "jupyterhub-notebook"