circleci-demo-aws-ecs-ecr
circleci-demo-aws-ecs-ecr copied to clipboard
Refactoring to purely use Terraform
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 Thank you for sharing this, I'll be reviewing it :)
Is this going to be implemented? I would like to use this and personally think it's a much better way to go.
@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.