Skip to content
This repository was archived by the owner on Jul 20, 2024. It is now read-only.

Add ASG ARN output variable #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fabioluciano
Copy link

No description provided.

@fabioluciano
Copy link
Author

?

@@ -18,3 +18,8 @@ output "iam_role_name" {
description = "Name of the IAM role for the NAT instance"
value = aws_iam_role.this.name
}

output "autoscaling_group_arm" {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arn not arm

@@ -18,3 +18,8 @@ output "iam_role_name" {
description = "Name of the IAM role for the NAT instance"
value = aws_iam_role.this.name
}

output "autoscaling_group_arm" {
description = "ASG ARN"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"The ARN for this autoscaling group" might be a better description here based on the popular eks module https://github.com/terraform-aws-modules/terraform-aws-eks/blob/771465be280450fc96d889ef9e15f191bb512849/modules/self-managed-node-group/outputs.tf#L30

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
2 participants