-
-
Notifications
You must be signed in to change notification settings - Fork 148
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 circuit breaker (question) #161
Comments
Hi @stadskle, I have not tested a deployment with ecs-deploy and an active circuit breaker configuration yet. My gut feeling is, that it should not cause problems, as ecs-deploy only fetches the deployment state from ECS and waits until it changed to Best |
Yes, for now only a timeout would be reported if the whole process takes too long. Currently the check of the Deployment entity is not as explicit as it could be. I'll look into this a bit deeper with the goal to report explicitly when the deployment failed, independent if the circuit breaker is activated or not - at the time, when the original deployment failed (before an optional rollback). During my tests I discovered, that this new feature does not cover all cases of failing containers. For example, if you specify an invalid Docker This is a known limitation, it's covered in this issue: aws/containers-roadmap#1206 Best |
The deployment check now utilizes the new With this change, we can utilize the new circuit breaker feature and:
The feature is not released yet, but available in a feature branch for now, if anybody wants to chime it and test this new behavior. Best |
Has anyone tested this tool with the new circuit breaker?
https://aws.amazon.com/blogs/containers/announcing-amazon-ecs-deployment-circuit-breaker/
It is on our plan, but have not manged to do it yet, so just wanted to hear if anyone have tried. I suspect it might require some changes in the deploy script, to ensure that ecs deploy reports failed deploy correctly if ECS aborts it?
The text was updated successfully, but these errors were encountered: