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

Move to alexs aws #10

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

Conversation

kurtwheeler
Copy link
Member

@kurtwheeler kurtwheeler commented Feb 9, 2018

This PR does a few things:

  • Changes the AWS account from Greenelab to Alexslemonade. This mostly happens via running terraform with AWS credentials from a different account, but that change is also explicit because a different route53 zone is specified. (That resource cannot be created via Terraform.)
  • Changes the availability zone. Both availability zones appear to have similar costs, but we accidentally created the route53 zone in the new zone so rather than destroying it and recreating it (which would require reconfirming the domain name ownership) we just changed the AZ we're spinning this up in.
  • Tightens up the IAM policy statements so Cognoma resourcces are limited to the Cognoma VPC.

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.

@kurtwheeler kurtwheeler requested a review from Miserlou February 9, 2018 19:08
@kurtwheeler
Copy link
Member Author

BTW, this PR no longer changes the AZ because SES is not supported in us-east-2.

@Miserlou
Copy link

LGTM - how is our Route53 being managed?

@kurtwheeler
Copy link
Member Author

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 data block, but now I found https://www.terraform.io/docs/providers/aws/r/route53_zone.html. However at this point I think it may not be worth fixing that small thing because we already have cognoma.org as a verified domain within the zone which involved @cgreene's authorization.

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.

3 participants