Merge pull request #4 from mike-rsi/feature/custom-logging-target

Feature/custom logging target

Author: cdaniluk

README.md

@@ -60,8 +60,10 @@ region               = "us-east-1"
 | Name | Description | Type | Default | Required |
 |------|-------------|:----:|:-----:|:-----:|
 | allowed\_account\_ids | Account IDs that are allowed to access the bucket/KMS key | list(string) | `[]` | no |
-| bucket | Name of bucket to create \(do not provide if using `remote\_bucket`\) | string | `""` | no |
+| bucket | Name of bucket to create \(do not provide if using `remote_bucket`\) | string | `""` | no |
 | kms\_key\_id | ARN for KMS key for all encryption operations. | string | `""` | no |
+| logging\_target\_bucket | The name of the bucket that will receive the log objects | string | `var.bucket` | no |
+| logging\_target\_prefix | A key prefix for log objects | string | `"TFStateLogs/"` | no |
 | region | Region bucket will be created in | string | n/a | yes |
 | remote\_bucket | If specified, the remote bucket will be used for the backend. A new bucket will not be created | string | `""` | no |
 | table | Name of Dynamo Table to create | string | `"tf-locktable"` | no |

main.tf

@@ -49,8 +49,8 @@ resource "aws_s3_bucket" "this" {
   }
 
   logging {
-    target_bucket = var.bucket
-    target_prefix = "TFStateLogs/"
+    target_bucket = coalesce(var.logging_target_bucket, var.bucket)
+    target_prefix = var.logging_target_prefix
   }
 }
 

variables.tf

@@ -16,6 +16,18 @@ variable "kms_key_id" {
   type        = string
 }
 
+variable "logging_target_bucket" {
+  default     = null
+  description = "The name of the bucket that will receive the log objects"
+  type        = string
+}
+
+variable "logging_target_prefix" {
+  default     = "TFStateLogs/"
+  description = "A key prefix for log objects"
+  type        = string
+}
+
 variable "region" {
   description = "Region bucket will be created in"
   type        = string