Merge pull request #12 from ministryofjustice/bugfix/issue-when-repli…

…cation-is-not-required bugfix/fix-issue-when-replication-is-not-required
ministryofjustice · Jul 30, 2021 · e4e1c4a · e4e1c4a
2 parents 8eb6e75 + 198b681
commit e4e1c4a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -84,7 +84,8 @@ module "s3-bucket" {
 | lifecycle_rule         | Lifecycle rules                                                                       | object  | `null`    | no       |
 | log_bucket             | Bucket for server access logging, if applicable                                       | string  | ""        | no       |
 | log_prefix             | Prefix to use for server access logging, if applicable                                | string  | ""        | no       |
-| replication_role_arn   | IAM Role ARN for replication. See below for more information                          | string  |           | yes      |
+| replication_role_arn   | IAM Role ARN for replication. See below for more information (Required if             |         |           |          |
+|                        | 'replication enabled' variable is set to true)                                        | string  | ""        | depends  |
 | tags                   | Tags to apply to resources, where applicable                                          | map     |           | yes      |
 
 

diff --git a/variables.tf b/variables.tf
@@ -56,6 +56,7 @@ variable "log_prefix" {
 variable "replication_role_arn" {
   type        = string
   description = "Role ARN to access S3 and replicate objects"
+  default     = ""
 }
 
 variable "tags" {