-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
configuring terraform&aws for bltomlin #103
Conversation
ready for a second review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
take a look at:
https://github.com/hackforla/devops-security/pull/103/files
the tfstate file is still listed in what would be changed. also, please remove aws-user.tf and the deletions (in red) from aws-users.tf. The listed url should have only green in one file.
Terraform plan in terraform Plan: 3 to add, 0 to change, 0 to destroy.Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create
Terraform will perform the following actions:
# module.iam_user_bltomlin.aws_iam_user.user will be created
+ resource "aws_iam_user" "user" {
+ arn = (known after apply)
+ force_destroy = false
+ id = (known after apply)
+ name = "bltomlin"
+ path = "/"
+ tags = {
+ "Access Level" = "1"
+ "Project" = "devops-security"
}
+ tags_all = {
+ "Access Level" = "1"
+ "Project" = "devops-security"
}
+ unique_id = (known after apply)
}
# module.iam_user_bltomlin.aws_iam_user_group_membership.user_group_membership will be created
+ resource "aws_iam_user_group_membership" "user_group_membership" {
+ groups = [
+ "read-only-group",
]
+ id = (known after apply)
+ user = "bltomlin"
}
# module.iam_user_bltomlin.aws_iam_user_login_profile.user_login will be created
+ resource "aws_iam_user_login_profile" "user_login" {
+ encrypted_password = (known after apply)
+ id = (known after apply)
+ key_fingerprint = (known after apply)
+ password = (known after apply)
+ password_length = 20
+ password_reset_required = true
+ user = "bltomlin"
}
Plan: 3 to add, 0 to change, 0 to destroy. ✅ Plan applied in Apply Terraform changes on merge #23 |
Fixes #103
What changes did you make?
Why did you make the changes (we will use this info to test)?