circleci-demo-aws-ecs-ecr icon indicating copy to clipboard operation
circleci-demo-aws-ecs-ecr copied to clipboard

Refactoring to purely use Terraform

Open ozooxo opened this issue 5 years ago • 3 comments

The original demo repository have CloudFormation templates inside of Terraform files, which is quite confusing to understand and hard to modify.

I refactored it so everything is in Terraform now. My build passed: https://circleci.com/workflow-run/7d65d933-ae77-4e46-bc92-c6eec804aa90

Not sure if it is the direction the original author(s) want to approach, but just for sharing.

P.S I made some necessary changes to make it work with my own AWS account. Those changes need to be reverted: https://github.com/ozooxo/circleci-demo-aws-ecs-ecr/compare/dd7e438df12a1ec23a95236dee8269798fc37f96...ozooxo:872f77ea09caebdf52148ab8f8d74c6ed5f677e4

ozooxo avatar May 02 '19 11:05 ozooxo

@ozooxo Thank you for sharing this, I'll be reviewing it :)

lokst avatar May 08 '19 08:05 lokst

Is this going to be implemented? I would like to use this and personally think it's a much better way to go.

jdbohrman avatar May 26 '19 22:05 jdbohrman

@jdbohrman Unfortunately it might be some time before I will be able to review this. Perhaps you could use the PR branch as a base though? The use of the CloudFormation template (via terraform) has been convenient for the purposes of this demo (it's easy to delete resources via CloudFormation stacks when necessary) so I would evaluate whether to merge this into the default branch or another branch.

lokst avatar May 27 '19 10:05 lokst