github-mirrors/terraform-aws-eks
1
0
Fork 0
mirror of https://github.com/ysoftdevs/terraform-aws-eks.git synced 2026-03-31 22:43:07 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: The type of the output cloudwatch_log_group_name should be a string instead of a list of strings (#1061)

Browse Source 
NOTES: The output `cloudwatch_log_group_name` was incorrectly returning the log group name as a list of strings. As a workaround, people were using `module.eks_cluster.cloudwatch_log_group_name[0]` but that was totally inconsistent with output name. Those users can now use `module.eks_cluster.cloudwatch_log_group_name` directly.
This commit is contained in:
Paul Pop
2020-10-28 21:37:21 +00:00
committed by GitHub
parent 751c422330
commit ca7593a6db
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@@ -202,7 +202,7 @@ MIT Licensed. See [LICENSE](https://github.com/terraform-aws-modules/terraform-a
| map\_roles | Additional IAM roles to add to the aws-auth configmap. See examples/basic/variables.tf for example format. | <pre>list(object({<br> rolearn = string<br> username = string<br> groups = list(string)<br> }))</pre> | `[]` | no | | map\_roles | Additional IAM roles to add to the aws-auth configmap. See examples/basic/variables.tf for example format. | <pre>list(object({<br> rolearn = string<br> username = string<br> groups = list(string)<br> }))</pre> | `[]` | no |
| map\_users | Additional IAM users to add to the aws-auth configmap. See examples/basic/variables.tf for example format. | <pre>list(object({<br> userarn = string<br> username = string<br> groups = list(string)<br> }))</pre> | `[]` | no | | map\_users | Additional IAM users to add to the aws-auth configmap. See examples/basic/variables.tf for example format. | <pre>list(object({<br> userarn = string<br> username = string<br> groups = list(string)<br> }))</pre> | `[]` | no |
| node\_groups | Map of map of node groups to create. See `node_groups` module's documentation for more details | `any` | `{}` | no | | node\_groups | Map of map of node groups to create. See `node_groups` module's documentation for more details | `any` | `{}` | no |
| node\_groups\_defaults | Map of values to be applied to all node groups. See `node_groups` module's documentaton for more details | `any` | `{}` | no | | node\_groups\_defaults | Map of values to be applied to all node groups. See `node_groups` module's documentation for more details | `any` | `{}` | no |
| permissions\_boundary | If provided, all IAM roles will be created with this permissions boundary attached. | `string` | `null` | no | | permissions\_boundary | If provided, all IAM roles will be created with this permissions boundary attached. | `string` | `null` | no |
| subnets | A list of subnets to place the EKS cluster and workers within. | `list(string)` | n/a | yes | | subnets | A list of subnets to place the EKS cluster and workers within. | `list(string)` | n/a | yes |
| tags | A map of tags to add to all resources. | `map(string)` | `{}` | no | | tags | A map of tags to add to all resources. | `map(string)` | `{}` | no |

2
outputs.tf
View File

@@ -58,7 +58,7 @@ output "cluster_primary_security_group_id" {
output "cloudwatch_log_group_name" { output "cloudwatch_log_group_name" {
description = "Name of cloudwatch log group created" description = "Name of cloudwatch log group created"
value = aws_cloudwatch_log_group.this[*].name value = element(concat(aws_cloudwatch_log_group.this[*].name, list("")), 0)
} }
output "kubeconfig" { output "kubeconfig" {

2
variables.tf
View File

@@ -318,7 +318,7 @@ variable "create_eks" {
} }
variable "node_groups_defaults" { variable "node_groups_defaults" {
description = "Map of values to be applied to all node groups. See `node_groups` module's documentaton for more details" description = "Map of values to be applied to all node groups. See `node_groups` module's documentation for more details"
type = any type = any
default = {} default = {}
} }