Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

Commit 1ca66c3

Browse files
committed
Add port for refresh worker and remove health check for cubestore
1 parent 714ee22 commit 1ca66c3

File tree

1 file changed

+12
-7
lines changed
  • terraform/modules/cube-cluster

1 file changed

+12
-7
lines changed

terraform/modules/cube-cluster/ecs.tf

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -270,6 +270,10 @@ resource "aws_ecs_task_definition" "cube_refresh_worker" {
270270
name = "CUBEJS_REFRESH_WORKER"
271271
value = "true"
272272
},
273+
{
274+
name = "PORT"
275+
value = "80"
276+
},
273277
{
274278
name = "CUBEJS_CUBESTORE_HOST"
275279
value = local.cubestore_router_http_dns_name
@@ -514,13 +518,14 @@ resource "aws_ecs_task_definition" "cubestore" {
514518
"awslogs-stream-prefix" : "ecs"
515519
}
516520
}
517-
healthCheck = {
518-
command = ["CMD-SHELL", "curl -f http://localhost:3031/readyz || exit 1"]
519-
interval = 30
520-
retries = 3
521-
startPeriod = 60 # Grace period before starting checks (in seconds)
522-
timeout = 5
523-
}
521+
# bring back when I figure out where to get a health check port
522+
# healthCheck = {
523+
# command = ["CMD-SHELL", "curl -f http://localhost:3031/readyz || exit 1"]
524+
# interval = 30
525+
# retries = 3
526+
# startPeriod = 60 # Grace period before starting checks (in seconds)
527+
# timeout = 5
528+
# }
524529
portMappings = [
525530
{
526531
containerPort = local.cubestore_worker_port,

0 commit comments

Comments
 (0)