Skip to content

Commit

Permalink
fix(externalrole): adjust outputs.
Browse files Browse the repository at this point in the history
Returning an `arn` rather than a `role` reduces the stuttering when
referring to the `externalrole` module, e.g: `module.metrics_role.arn`
vs `module.metrics_role.role.arn`.
  • Loading branch information
jta committed Aug 8, 2024
1 parent 65b2982 commit 9f6627f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion modules/externalrole/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,5 +70,5 @@ No modules.

| Name | Description |
|------|-------------|
| <a name="output_role"></a> [role](#output\_role) | IAM role to be assummed by Observe |
| <a name="output_arn"></a> [arn](#output\_arn) | ARN for IAM role to be assumed by Observe |
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
6 changes: 3 additions & 3 deletions modules/externalrole/outputs.tf
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
output "role" {
description = "IAM role to be assummed by Observe"
value = aws_iam_role.this
output "arn" {
description = "ARN for IAM role to be assumed by Observe"
value = aws_iam_role.this.arn
}

0 comments on commit 9f6627f

Please sign in to comment.