Merge branch 'bcgov:main' into feature/WFPREV-34_3

bcgov · Oct 1, 2024 · 804b270 · 804b270
2 parents bfa40dc + 4ed2ad1
commit 804b270
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/terraform/ecs.tf b/terraform/ecs.tf
@@ -150,8 +150,8 @@ resource "aws_ecs_task_definition" "wfprev_client" {
       portMappings = [
         {
           protocol      = "tcp"
-          containerPort = var.client_port
-          hostPort      = var.client_port
+          containerPort = var.WFPREV_CLIENT_PORT
+          hostPort      = var.WFPREV_CLIENT_PORT
         }
       ]
       environment = [
@@ -294,7 +294,7 @@ resource "aws_ecs_service" "client" {
   load_balancer {
     target_group_arn = aws_alb_target_group.wfprev_ui.id
     container_name   = var.client_container_name
-    container_port   = var.client_port
+    container_port   = var.WFPREV_CLIENT_PORT
   }
 
   # depends_on = [aws_iam_role_policy_attachment.wfprev_ecs_task_execution_role]

diff --git a/terraform/variables.tf b/terraform/variables.tf
@@ -54,11 +54,6 @@ variable "WFPREV_API_IMAGE" {
   default     = ""
 }
 
-variable "client_port" {
-  description = "Port exposed by the docker image to redirect traffic to"
-  default     = 8081
-}
-
 variable "server_port" {
   description = "Port exposed by the docker image to redirect traffic to"
   default     = 443
@@ -133,7 +128,7 @@ variable "TARGET_ENV" {
 variable "WFPREV_API_PORT" {
   description = "Port exposed by the docker image to redirect traffic to"
   type = number
-  default     = 443
+  default  = 8080
 }
 
 variable "WFPREV_CLIENT_PORT" {