terraform fmt and fix README.md

examples/iam-group-w-policy-and-opt-roles/README.md

@@ -33,8 +33,8 @@ No providers.
 
 | Name | Source | Version |
 |------|--------|---------|
-| <a name="module_iam_group_complete"></a> [iam\_group\_complete](#module\_iam\_group\_complete) | ../../modules/iam-group-with-assumable-roles-policy | n/a |
-| <a name="module_iam_group_complete_with_custom_policy"></a> [iam\_group\_complete\_with\_custom\_policy](#module\_iam\_group\_complete\_with\_custom\_policy) | ../../modules/iam-group-with-policies | n/a |
+| <a name="module_iam_group_optional_assumable_roles"></a> [iam\_group\_optional\_assumable\_roles](#module\_iam\_group\_optional\_assumable\_roles) | ../../modules/iam-group-with-assumable-roles-policy | n/a |
+| <a name="module_iam_group_with_custom_policy"></a> [iam\_group\_with\_custom\_policy](#module\_iam\_group\_with\_custom\_policy) | ../../modules/iam-group-with-policies | n/a |
 | <a name="module_iam_user1"></a> [iam\_user1](#module\_iam\_user1) | ../../modules/iam-user | n/a |
 | <a name="module_iam_user2"></a> [iam\_user2](#module\_iam\_user2) | ../../modules/iam-user | n/a |
 
@@ -44,7 +44,9 @@ No resources.
 
 ## Inputs
 
-No inputs.
+| Name | Description | Type | Default | Required |
+|------|-------------|------|---------|:--------:|
+| <a name="input_assumable_roles"></a> [assumable\_roles](#input\_assumable\_roles) | (possibly empty) List of ARNS for roles assumable by this group | `list(string)` | `[]` | no |
 
 ## Outputs
 

examples/iam-group-w-policy-and-opt-roles/variables.tf

@@ -1,5 +1,5 @@
 variable "assumable_roles" {
   type        = list(string)
-  description = "List of ARNS for roles assumable by this group"
+  description = "(possibly empty) List of ARNS for roles assumable by this group"
   default     = []
 }

modules/iam-group-with-assumable-roles-policy/README.md

@@ -28,6 +28,7 @@ No modules.
 | [aws_iam_group_membership.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_group_membership) | resource |
 | [aws_iam_group_policy_attachment.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_group_policy_attachment) | resource |
 | [aws_iam_policy.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_policy) | resource |
+| [aws_iam_group.ref_group](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_group) | data source |
 | [aws_iam_policy_document.assume_role](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) | data source |
 
 ## Inputs
@@ -36,6 +37,7 @@ No modules.
 |------|-------------|------|---------|:--------:|
 | <a name="input_assumable_roles"></a> [assumable\_roles](#input\_assumable\_roles) | List of IAM roles ARNs which can be assumed by the group | `list(string)` | `[]` | no |
 | <a name="input_assumable_roles_policy_name_suffix"></a> [assumable\_roles\_policy\_name\_suffix](#input\_assumable\_roles\_policy\_name\_suffix) | Append this name to the policy name that will be created for assuming the given roles (default: null -- the policy name will be group name) | `string` | `""` | no |
+| <a name="input_create_group"></a> [create\_group](#input\_create\_group) | Whether to create IAM group | `bool` | `true` | no |
 | <a name="input_group_users"></a> [group\_users](#input\_group\_users) | List of IAM users to have in an IAM group which can assume the role | `list(string)` | `[]` | no |
 | <a name="input_name"></a> [name](#input\_name) | Name of IAM policy and IAM group | `string` | n/a | yes |
 | <a name="input_path"></a> [path](#input\_path) | Path of IAM policy and IAM group | `string` | `"/"` | no |

modules/iam-group-with-assumable-roles-policy/outputs.tf

@@ -19,7 +19,7 @@ output "group_name" {
 }
 
 data "aws_iam_group" "ref_group" {
-  count = var.create_group ? 0 : 1
+  count      = var.create_group ? 0 : 1
   group_name = var.name
 }