Feat: allow var.name to be set to empty (#11)

Co-authored-by: cloudpossebot <[email protected]>

README.md

@@ -149,6 +149,31 @@ module "helm_release" {
 }
 ```
 
+If `var.service_account_name` is set, then `var.name` can be set to "" in order to achieve a shorter name for the IAM
+Role created for the ServiceAccount:
+
+```hcl
+module "helm_release" {
+  source  = "cloudposse/helm-release/aws"
+  # Cloud Posse recommends pinning every module to a specific version
+  # version = "x.x.x"
+
+  name = ""
+
+  create_namespace     = true
+  kubernetes_namespace = "echo"
+
+  service_account_name      = "echo"
+  service_account_namespace = "echo"
+
+  iam_role_enabled = true
+  ...
+}
+```
+
+This will result in an IAM role with a name such as: `eg-uw2-dev-echo@echo` instead of `eg-uw2-dev-echo-echo@echo`.
+Additionally, if `var.name` is empty, the name used for the Helm Release will be that of `var.chart`.
+
 
 
 
@@ -440,14 +465,16 @@ Check out [our other projects][github], [follow us on twitter][twitter], [apply
 ### Contributors
 
 <!-- markdownlint-disable -->
-|  [![Erik Osterman][osterman_avatar]][osterman_homepage]<br/>[Erik Osterman][osterman_homepage] | [![RB][nitrocode_avatar]][nitrocode_homepage]<br/>[RB][nitrocode_homepage] |
-|---|---|
+|  [![Erik Osterman][osterman_avatar]][osterman_homepage]<br/>[Erik Osterman][osterman_homepage] | [![RB][nitrocode_avatar]][nitrocode_homepage]<br/>[RB][nitrocode_homepage] | [![Yonathan Koren][korenyoni_avatar]][korenyoni_homepage]<br/>[Yonathan Koren][korenyoni_homepage] |
+|---|---|---|
 <!-- markdownlint-restore -->
 
   [osterman_homepage]: https://github.com/osterman
   [osterman_avatar]: https://img.cloudposse.com/150x150/https://github.com/osterman.png
   [nitrocode_homepage]: https://github.com/nitrocode
   [nitrocode_avatar]: https://img.cloudposse.com/150x150/https://github.com/nitrocode.png
+  [korenyoni_homepage]: https://github.com/korenyoni
+  [korenyoni_avatar]: https://img.cloudposse.com/150x150/https://github.com/korenyoni.png
 
 [![README Footer][readme_footer_img]][readme_footer_link]
 [![Beacon][beacon]][website]

README.yaml

@@ -118,6 +118,31 @@ usage: |-
   }
   ```
 
+  If `var.service_account_name` is set, then `var.name` can be set to "" in order to achieve a shorter name for the IAM
+  Role created for the ServiceAccount:
+
+  ```hcl
+  module "helm_release" {
+    source  = "cloudposse/helm-release/aws"
+    # Cloud Posse recommends pinning every module to a specific version
+    # version = "x.x.x"
+
+    name = ""
+
+    create_namespace     = true
+    kubernetes_namespace = "echo"
+
+    service_account_name      = "echo"
+    service_account_namespace = "echo"
+
+    iam_role_enabled = true
+    ...
+  }
+  ```
+
+  This will result in an IAM role with a name such as: `eg-uw2-dev-echo@echo` instead of `eg-uw2-dev-echo-echo@echo`.
+  Additionally, if `var.name` is empty, the name used for the Helm Release will be that of `var.chart`.
+
 # Example usage
 examples: |-
   Here is an example of using this module:
@@ -138,3 +163,5 @@ contributors:
     github: "osterman"
   - name: "RB"
     github: "nitrocode"
+  - name: "Yonathan Koren"
+    github: "korenyoni"

main.tf

@@ -35,7 +35,9 @@ module "eks_iam_role" {
 resource "helm_release" "this" {
   count = local.enabled ? 1 : 0
 
-  name = module.this.name
+  # Allows var.name to be empty, which is allowed to be the case for this module since var.name is optional in eks-iam-role.
+  # For more information, see: https://github.com/cloudposse/terraform-aws-eks-iam-role
+  name = coalesce(module.this.name, var.chart)
 
   chart       = var.chart
   description = var.description