Skip to content

Separate iam into root-iam and iam #18

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 23, 2018
Merged

Separate iam into root-iam and iam #18

merged 1 commit into from
Jun 23, 2018

Conversation

@aknysh
Copy link
Member

@aknysh aknysh commented Jun 15, 2018

what

  • Separate iam into root-iam and iam

why

  • Simpler cold start process
  • Don't have to use -target with terraform apply

notes

  • this is for @goruha to provision new accounts
  • we'll restructure it again after we introduce subfolders (e.g. root/aim, root/dns)
  • that restructuring will require re-applying the existing accounts or copying Terraform states into the new S3 folders

@aknysh aknysh self-assigned this Jun 15, 2018
@aknysh aknysh requested review from goruha, osterman and sarkis June 15, 2018 14:54
@goruha goruha merged commit f50ecdf into master Jun 23, 2018
@goruha goruha deleted the restructure-iam branch June 23, 2018 06:58
goruha added a commit that referenced this pull request Jun 23, 2018
@goruha
Merge branch 'master' into feature-add-install-tf-state
2a4abd1 
* master:
  Separate `iam` into `root-iam` and `iam` (#18)
  fix usage syntax of data.aws_availability_zones... (#16)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@osterman osterman Awaiting requested review from osterman

@goruha goruha Awaiting requested review from goruha

1 more reviewer

@sarkis sarkis sarkis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@aknysh aknysh

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aknysh @sarkis @goruha