Skip to content

Commit

Permalink
Don't include resource types in resource names (#3)
Browse files Browse the repository at this point in the history
  • Loading branch information
aknysh authored Mar 27, 2018
1 parent 7a91ef4 commit 1dd369a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@ module "label" {

# https://www.terraform.io/docs/providers/aws/r/iam_group.html
resource "aws_iam_group" "default" {
name = "${module.label.id}${var.delimiter}group"
name = "${module.label.id}"
}

# https://www.terraform.io/docs/providers/aws/r/iam_group_membership.html
resource "aws_iam_group_membership" "default" {
name = "${module.label.id}${var.delimiter}group${var.delimiter}membership"
name = "${module.label.id}"
group = "${aws_iam_group.default.id}"
users = ["${var.user_names}"]
}
Expand All @@ -37,7 +37,7 @@ data "aws_iam_policy_document" "default" {

# https://www.terraform.io/docs/providers/aws/r/iam_group_policy.html
resource "aws_iam_group_policy" "default" {
name = "${module.label.id}${var.delimiter}group${var.delimiter}policy"
name = "${module.label.id}"
group = "${aws_iam_group.default.id}"
policy = "${data.aws_iam_policy_document.default.json}"
}

0 comments on commit 1dd369a

Please sign in to comment.