github-mirrors/terraform-aws-eks
1
0
Fork 0
mirror of https://github.com/ysoftdevs/terraform-aws-eks.git synced 2026-03-26 03:11:06 +01:00
Code Issues Packages Projects Releases Wiki Activity

Adding tags for Log groups and workers IAM role (#448)

Browse Source 
* Adding tags for Cloudwatch log group

* Adding tags for workers IAM role

* Update CHANGELOG.md

* Removing change in CHANGELOG.md

* Fixing formatting issues
This commit is contained in:
Lucas Giacomazzi
2019-08-06 13:01:01 -03:00
committed by Max Williams
parent ac62edc872
commit ebac6c92bf
3 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGELOG.md
View File

@@ -11,6 +11,8 @@ project adheres to [Semantic Versioning](http://semver.org/).
### Added ### 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) - Write your awesome addition here (by @you)
- Added `required_providers` to enforce provider minimum versions (by @dpiddockcmp) - Added `required_providers` to enforce provider minimum versions (by @dpiddockcmp)

1
cluster.tf
View File

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

1
workers.tf
View File

@@ -286,6 +286,7 @@ resource "aws_iam_role" "workers" {
permissions_boundary = var.permissions_boundary permissions_boundary = var.permissions_boundary
path = var.iam_path path = var.iam_path
force_detach_policies = true force_detach_policies = true
tags = var.tags
} }
resource "aws_iam_instance_profile" "workers" { resource "aws_iam_instance_profile" "workers" {