Adding tags for Log groups and workers IAM role (terraform-aws-module…

…s#448) * Adding tags for Cloudwatch log group * Adding tags for workers IAM role * Update CHANGELOG.md * Removing change in CHANGELOG.md * Fixing formatting issues
michyweb · Aug 6, 2019 · ebac6c9 · ebac6c9
1 parent ac62edc
commit ebac6c9
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,8 @@ project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Added
 
+ - Added support for log group tag in `./cluster.tf` (@lucas-giaco)
+ - Added support for workers iam role tag in `./workers.tf` (@lucas-giaco)
  - Write your awesome addition here (by @you)
  - Added `required_providers` to enforce provider minimum versions (by @dpiddockcmp)
 

diff --git a/cluster.tf b/cluster.tf
@@ -3,6 +3,7 @@ resource "aws_cloudwatch_log_group" "this" {
   name              = "/aws/eks/${var.cluster_name}/cluster"
   retention_in_days = var.cluster_log_retention_in_days
   kms_key_id        = var.cluster_log_kms_key_id
+  tags              = var.tags
 }
 
 resource "aws_eks_cluster" "this" {

diff --git a/workers.tf b/workers.tf
@@ -286,6 +286,7 @@ resource "aws_iam_role" "workers" {
   permissions_boundary  = var.permissions_boundary
   path                  = var.iam_path
   force_detach_policies = true
+  tags                  = var.tags
 }
 
 resource "aws_iam_instance_profile" "workers" {