Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ECS deployment succeeds despite Load Balancer health "Unknown" #6946

Open
jwach opened this issue Apr 18, 2024 · 1 comment
Open

ECS deployment succeeds despite Load Balancer health "Unknown" #6946

jwach opened this issue Apr 18, 2024 · 1 comment

Comments

@jwach
Copy link

jwach commented Apr 18, 2024

Issue Summary:

Pipeline deployment of the ECS provider succeeds right after the ECS task is up & healthy, but ignores the health status of the LB's Target Group being "Unknown".

Cloud Provider(s):

AWS ECS

Environment:

Spinnaker 1.33.0 (self-hosted on Ubuntu).

Feature Area:

Pipelines.

Description:

The application is configured with the option Consider only cloud provider health when executing tasks disabled, so it should take Load Balancer's Target Group health into account when deploying.
Screenshot 2024-04-18 at 15 18 07

During pipeline deploy, when new sever group is created, Spinnaker proceeds to destroy the old server group right after the ECS task health is Up, despite Load Balancer health still being Unknown.
Screenshot 2024-04-18 at 15 20 07

This leads to traffic being directed to unhealthy instances and alerts from CloudWatch on HealthyHostCount being 0.

ECS task definition does not have its own healthcheck defined.

This used to work correctly before, but I've made a long overdue upgrade from version 1.20 to 1.33.0 and it broke.

Expected behaviour

Old server group is destroyed only after both health checks for the new server group are "Up".

Steps to Reproduce:

  1. Create a deployment pipeline with ECS provider and Highlander strategy.
  2. Execute the pipeline.
@jwach
Copy link
Author

jwach commented Apr 19, 2024

The pipeline JSON also has the health check type set to EC2.
Screenshot 2024-04-19 at 10 31 32

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

No branches or pull requests

1 participant