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

Remove cloudformation depedency #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maxknee
Copy link

@maxknee maxknee commented Oct 5, 2019

Moving to use ASG resource instead of cloudformation
#11

Signed-off-by: Max Knee [email protected]

@maxknee maxknee requested a review from a team as a code owner October 5, 2019 15:25
@itsdalmo
Copy link
Contributor

itsdalmo commented Oct 5, 2019

Hey @maxknee - the reason we use CloudFormation for the autoscaling groups is because CF has a built-in support for e.g. rolling updates, and resources can signal when they are ready (and not just passing EC2 healthchecks) to avoid downtime.

Any particular reason you want/need to get rid of the CF here?

create_before_destory` and having `wait_for_elb_capacity` achieves
the `rollingUpdates` without the cloudformation.

Signed-off-by: Max Knee <[email protected]>
@maxknee
Copy link
Author

maxknee commented Oct 5, 2019

https://www.terraform.io/docs/providers/aws/r/autoscaling_group.html#wait_for_capacity_timeout
This should achieve the same thing without a cloudformation stack

@itsdalmo
Copy link
Contributor

itsdalmo commented Oct 7, 2019

Waiting for capacity will either wait for an EC2 or ELB health check, but for "worker" instances the instance might not be attached to an ELB, and the EC2 health check might pass before the instances is ready to accept work. For example with ECS clusters, the instance itself is not attached to any load balancers (the containers are) and you don't want to signal that you are ready before the ECS agent has started.

Unfortunately I think the only way to support this (and similar) use case is by using Cloudformation and cfn-signal?

@maxknee
Copy link
Author

maxknee commented Oct 7, 2019

I am thinking of another way for it to access traffic to signal workers are healthy in the respects of concourse. ASGs do rolling updates with launch configs, IIRC

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.

2 participants