hackforla / devops-security

Management of Hack for L.A.'s AWS IAM resources
https://github.com/orgs/hackforla/projects/73/views/4
MIT License
0 stars 12 forks source link

adding mvp terraform config for aws iam resources #1

Closed chelseybeck closed 9 months ago

chelseybeck commented 10 months ago

Resolves this issue

This is a minimum configuration for using Terraform to create IAM resources in AWS. Apologies for the long PR, but w/ this initial setup it just made sense to do it all at once. 😸

Overview:

View the branch


This is now a working solution. If an apply ran right now, it would create a new group with readonly permissions, create a user 'gwenstacy', and assign the user to the group. I'd like to get this merged to main as a basis for adding resources.

Screenshot 2024-02-01 at 10 15 07 PM Screenshot 2024-02-01 at 10 15 34 PM
chelseybeck commented 10 months ago

@nyarly I'd like your feedback if you have a chance to review...we will discuss more in depth at next ops meeting

nyarly commented 10 months ago

Overall, a really nice skeleton. I'd love to see policies filled in and this started to be applied ASAP.

As a separate issue, modernizing existing config should get recorded, so that the manual references to policies or users that already exist can be removed eventually. Not difficult, just a tedious process of finding users and terraform importing them.

chelseybeck commented 9 months ago

Overall, a really nice skeleton. I'd love to see policies filled in and this started to be applied ASAP.

As a separate issue, modernizing existing config should get recorded, so that the manual references to policies or users that already exist can be removed eventually. Not difficult, just a tedious process of finding users and terraform importing them.

Thanks, @nyarly :smile:

Completely agree on all points. I was thinking that importing existing IAM resources would make good issues for members to work on after getting Terraform set up and working on a few first issues