adds tags to resource aws_iam_role.cluster (#390)

* adds tagging to iam role

Signed-off-by: camilo santana <camilo.santana@procore.com>

* emulates name_prefix

Signed-off-by: camilo santana <camilo.santana@procore.com>

* incriment version

Signed-off-by: camilo santana <camilo.santana@procore.com>

* tags iam role

Signed-off-by: camilo santana <camilo.santana@procore.com>

* improve resource name

Signed-off-by: camilo santana <camilo.santana@procore.com>

* removes comments

Signed-off-by: camilo santana <camilo.santana@procore.com>

* reverts erroneous increment to release version

Signed-off-by: camilo santana <camilo.santana@procore.com>

* reverts aws_iam_role.cluster name to orig value

Signed-off-by: camilo santana <camilo.santana@procore.com>

* removes random suffix idea

i was using random_id until i could figure out a datestamp or something here. `name_prefix` does a better job.

Signed-off-by: camilo santana <camilo.santana@procore.com>

* terraform fmt on local.tf

Signed-off-by: camilo santana <camilo.santana@procore.com>
This commit is contained in:
Camilo Santana
2019-06-13 01:36:58 -07:00
committed by Max Williams
parent bf5dae00e1
commit 3f0601551f
2 changed files with 2 additions and 1 deletions

View File

@@ -13,9 +13,9 @@ project adheres to [Semantic Versioning](http://semver.org/).
- Added Termination Policy Option to worker ASGs (by @undeadops)
- Update EBS optimized instances type (by @gloutsch)
- Added tagging for iam role created in `./cluster.tf` (@camilosantana)
- Enable log retention for cloudwatch log groups (by @yuriipolishchuk)
### Changed
- Fix toggle for IAM instance profile creation for mixed launch templates (by @jnozo)

View File

@@ -67,6 +67,7 @@ resource "aws_iam_role" "cluster" {
permissions_boundary = "${var.permissions_boundary}"
path = "${var.iam_path}"
force_detach_policies = true
tags = "${var.tags}"
}
resource "aws_iam_role_policy_attachment" "cluster_AmazonEKSClusterPolicy" {