Merge branch 'master' into issue/2/iam_deploy

Author: gozer

README.md

@@ -139,6 +139,7 @@ Here's the gist of using it directly from github.
 | cluster\_name | Cluster name. | `string` | n/a | yes |
 | description | Namespace description | `string` | `"velero-back-up-and-restore"` | no |
 | iam\_deploy | whther or not to deploy iam role | `bool` | `true` | no |
+| iam\_role\_name | Name of the Velero IAM role | `string` | `""` | no |
 | name | namespace name | `string` | `"velero"` | no |
 | namespace\_deploy | whther or not to deploy namespace | `bool` | `false` | no |
 | openid\_connect\_provider\_uri | OpenID Connect Provider for EKS to enable IRSA. | `string` | n/a | yes |

iam.tf

@@ -63,7 +63,8 @@ data aws_iam_policy_document policy {
 
 resource aws_iam_role this {
   count              = var.iam_deploy ? 1 : 0
-  name               = format("%s-%s", var.cluster_name, var.name)
+  name               = var.iam_role_name == "" ? format("%s-%s", var.cluster_name, var.name) : var.iam_role_name
+
   assume_role_policy = data.aws_iam_policy_document.assume_role.json
   tags = merge(var.tags,
     { Attached = var.name },

variables.tf

@@ -65,3 +65,9 @@ variable "bucket" {
   description = "Backup and Restore bucket."
   type        = string
 }
+
+variable "iam_role_name" {
+  description = "Name of the Velero IAM role"
+  type        = string
+  default     = ""
+}