#airship - June 2020

hey everyone
Is there pls any update on porting https://github.com/blinkist/terraform-aws-airship-ecs-service to terraform 0.12?

or we should just stop praying for that? 🙂

heh, I was coming to ask the same question

@Mads Hvelplund IS https://github.com/mhvelplund/terraform-aws-airship-ecs-service/tree/terraform12 your branch? How did it go?

👋 @Clayton Wheeler Just having a look at https://github.com/Genomenon/terraform-aws-airship-ecs-service/tree/terraform12

Would you mind me asking what functionality you have working?

I'd like to understand if it's more important to have the current module fitted into 0.12 or have a cleaner design based on 0.13.

By the latter I assume you mean for_each and depends_on for modules etc?

oh hey, hmm, I got pulled away from that so I don't have it quite in release-worthy shape yet like I'd hoped to, but a decent amount. The basics, definitely. I've only done a little bit of load balancing, and haven't touched autoscaling, cron tasks, placement, or volumes at all. But running normal containers esp. with Fargate, using Secrets Manager and IAM, service discovery, repository credentials, that works.

Wondering what folks are doing to deploy ECS tasks via CI/CD that services etc are managed with Terraform

OK, I made PR #91. Thanks for the nudge on this!

@Clayton Wheeler / @Mads Hvelplund / @Maarten van der Hoef Looks like https://github.com/Genomenon/terraform-aws-airship-ecs-service/blob/master/modules/live_task_lookup/output.tf#L19 is broken in 0.12

Adding an explicit depends_on gets over the bootstrap issue so I’m not sure the live lookup stuff is necessary?

Bah, trying try(data.aws_ecs_container_definition.default.image, local.bootstrap_service_container_image) causes a cyclical dependency