Skip to content

Commit

Permalink
fix: Sub-modules output the correct eks worker iam arn when workers u…
Browse files Browse the repository at this point in the history
…tilize custom iam role (terraform-aws-modules#1912)
  • Loading branch information
kongdewen authored and spr-mweber3 committed Apr 26, 2022
1 parent 793fb1b commit 6462f94
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion modules/eks-managed-node-group/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ output "iam_role_name" {

output "iam_role_arn" {
description = "The Amazon Resource Name (ARN) specifying the IAM role"
value = try(aws_iam_role.this[0].arn, "")
value = try(aws_iam_role.this[0].arn, var.iam_role_arn)
}

output "iam_role_unique_id" {
Expand Down
2 changes: 1 addition & 1 deletion modules/fargate-profile/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ output "iam_role_name" {

output "iam_role_arn" {
description = "The Amazon Resource Name (ARN) specifying the IAM role"
value = try(aws_iam_role.this[0].arn, "")
value = try(aws_iam_role.this[0].arn, var.iam_role_arn)
}

output "iam_role_unique_id" {
Expand Down
2 changes: 1 addition & 1 deletion modules/self-managed-node-group/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ output "iam_role_unique_id" {

output "iam_instance_profile_arn" {
description = "ARN assigned by AWS to the instance profile"
value = try(aws_iam_instance_profile.this[0].arn, "")
value = try(aws_iam_instance_profile.this[0].arn, var.iam_instance_profile_arn)
}

output "iam_instance_profile_id" {
Expand Down

0 comments on commit 6462f94

Please sign in to comment.