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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants