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

[#178898815] alertmanager deployment: wait for ecs service to be stable before proceeding #483

Merged

Conversation

risicle
Copy link
Member

@risicle risicle commented Jul 23, 2021

The hope is that a broken deployment will be caught even if the current release doesn't cause an ECS deployment.

This does seem to have the desired effect in practise when the deployment is deliberately broken.

@risicle risicle requested a review from a team as a code owner July 23, 2021 09:33
@risicle
Copy link
Member Author

risicle commented Jul 23, 2021

Sidenote: working on this reminded me how much I'd like to get #473 merged because its annoying and nerve-wracking having every push you make potentially triggering multiple builds in unexpected orders.

@risicle risicle changed the title alertmanager deployment: wait for ecs service to be stable before proceeding [#178898815] alertmanager deployment: wait for ecs service to be stable before proceeding Jul 23, 2021
…ceeding

the hope is that a broken deployment will be caught even if the
current release doesn't cause an ecs deployment.
@risicle risicle force-pushed the ris-alertmanager-wait-ecs-services-stable-178898815 branch from 1684fb4 to cb020f5 Compare July 26, 2021 15:23
@risicle risicle merged commit 813706a into master Aug 2, 2021
@risicle risicle deleted the ris-alertmanager-wait-ecs-services-stable-178898815 branch August 2, 2021 10:14
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.

None yet

2 participants