Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Update target group destroyer to wait for listener #4497

Merged
merged 3 commits into from
Feb 22, 2023

Conversation

paladin-devops
Copy link
Contributor

This PR updates the ECS platform destroyer to wait for there to be zero AWS ALB listeners targeting the target group to attempt to delete the target group, for a maximum of 5 minutes.

This does not fully address #4461, because my destroy attempts are still failing on deleting the security group, but it's a start! And deleting that will be fixed in a subsequent PR.

This commit updates the ECS platform destroyer to wait for there to be zero AWS ALB listeners targeting the target group to attempt to delete the target group, for a maximum of 5 minutes.
@paladin-devops paladin-devops marked this pull request as ready for review February 7, 2023 22:52
@paladin-devops paladin-devops requested a review from a team February 7, 2023 22:52
@paladin-devops
Copy link
Contributor Author

When #4496 is addressed, it might be nicer to update the ECS default releaser to modify the listener to no longer route to the target group; but this will do for now.

builtin/aws/ecs/platform.go Outdated Show resolved Hide resolved
builtin/aws/ecs/platform.go Outdated Show resolved Hide resolved
builtin/aws/ecs/platform.go Outdated Show resolved Hide resolved
There should be only one target group returned by the ARN lookup, so this is now enforced by returning an error if that is not the case. This allows for the elimination of the most exterior loop in the deletion of the target group.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants