-
Notifications
You must be signed in to change notification settings - Fork 2
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
Move to alexs aws #10
base: master
Are you sure you want to change the base?
Conversation
…ut it doesn't matter.
…to move-to-alexs-aws
BTW, this PR no longer changes the AZ because SES is not supported in us-east-2. |
LGTM - how is our Route53 being managed? |
Route53 is primarily managed through terraform, however the zone itself is not. I'm not sure why now... I had thought terraform didn't offer a way to create a zone, only reference it via a |
New collaborators: 0A053C53 Rich Jones <[email protected]>
…ause it is a service linked role which makes it not scary that it has a *.
…e ECS with a passion.
…with health checks for ECS instances, and database access.
This PR does a few things:
I tested that the syntax is parsable by terraform and that the resources I have IAM permissions to create are creatable, however I cannot create IAM roles so there are 13 resources which I cannot test the creation of.