Skip to content

Conversation

dylanratcliffe
Copy link
Member

No description provided.

Copy link

github-actions bot commented Aug 1, 2025

Overmind

Open in Overmind ↗


🔥 Risks

Risk of Health Check Failures Due to Port Misconfiguration ‼️High Open Risk ↗
The proposed change to the health check command from port 1234 to port 8080 may lead to failures if the application and related infrastructure are not updated accordingly. The ECS task definition, ELB target group, and security group are configured for port 1234, with no evidence of changes to support port 8080. This misconfiguration could result in the ECS service marking tasks as unhealthy, causing unnecessary restarts and potential service downtime. To mitigate this risk, ensure that the application is configured to listen on port 8080, update the ELB target group and security group rules to allow traffic on this port, and verify that the load balancer routes traffic correctly to port 8080.


🟣 Expected Changes

+/- ecs-task-definition › facial-recognition-terraform-example
- arn: arn:aws:ecs:eu-west-2:540044833068:task-definition/facial-recognition-terraform-example:7
+ arn: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
- arn_without_revision: arn:aws:ecs:eu-west-2:540044833068:task-definition/facial-recognition-terraform-example
+ arn_without_revision: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
- container_definitions: [{"cpu":1024,"environment":[{"name":"DATABASE_URL","value":"tf-20240827194315707700000013.cnx7xf6hwmba.eu-west-2.rds.amazonaws.com"}],"essential":true,"healthCheck":{"command":["CMD-SHELL","wget -q --spider localhost:1234"],"interval":30,"retries":3,"timeout":5},"image":"harshmanvar/face-detection-tensorjs:slim-amd","memory":2048,"mountPoints":[],"name":"facial-recognition","portMappings":[{"appProtocol":"http","containerPort":1234,"hostPort":1234,"protocol":"tcp"}],"systemControls":[],"volumesFrom":[]}]
+ container_definitions: [{"cpu":1024,"environment":[{"name":"DATABASE_URL","value":"tf-20240827194315707700000013.cnx7xf6hwmba.eu-west-2.rds.amazonaws.com"}],"essential":true,"healthCheck":{"command":["CMD-SHELL","wget -q --spider localhost:8080"],"interval":30,"retries":3,"timeout":5},"image":"harshmanvar/face-detection-tensorjs:slim-amd","memory":2048,"mountPoints":[],"name":"facial-recognition","portMappings":[{"appProtocol":"http","containerPort":1234}],"volumesFrom":[]}]
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [[{"cpu":1024,"environment":[{"name":"DATABASE_URL","value":"tf-20240827194315707700000013.cnx7xf6hwm].
- enable_fault_injection: false
+ enable_fault_injection: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
- execution_role_arn: 
+ execution_role_arn: <nil>
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [<nil>].
- id: facial-recognition-terraform-example
+ id: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
- ipc_mode: 
+ ipc_mode: <nil>
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [<nil>].
- pid_mode: 
+ pid_mode: <nil>
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [<nil>].
- revision: 7
+ revision: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
+ tags: <nil>
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [<nil>].
+ tags_all: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].
- task_role_arn: 
+ task_role_arn: <nil>
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [<nil>].

🟠 Unmapped Changes

~ aws_ecs_service › module.scenarios[0].aws_ecs_service.face
- task_definition: arn:aws:ecs:eu-west-2:540044833068:task-definition/facial-recognition-terraform-example:7
+ task_definition: (known after apply)
# → 🔁 This attribute has changed 1 times in the last 30 days.
#      The previous values were [(known after apply)].

💥 Blast Radius

Items 9

Edges 9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant