Merge pull request #39 from apsureda/folder-include-children

Adding the include_children option

Author: morgante

modules/event-folder-log-entry/README.md

@@ -23,6 +23,7 @@ The
 |------|-------------|:----:|:-----:|:-----:|
 | filter | The filter to apply when exporting logs. | string | n/a | yes |
 | folder\_id | The ID of the folder to look for changes. | string | n/a | yes |
+| include\_children | Determines whether or not to include folder's children in the sink export. If true, logs associated with child projects are also exported; otherwise only logs relating to the provided folder are included. | bool | `"false"` | no |
 | labels | A set of key/value label pairs to assign to any labelable resources. | map(string) | `<map>` | no |
 | name | The name to apply to any nameable resources. | string | n/a | yes |
 | project\_id | The ID of the project to which resources will be applied. | string | n/a | yes |

modules/event-folder-log-entry/main.tf

@@ -27,6 +27,7 @@ module "log_export" {
   filter                 = var.filter
   log_sink_name          = var.name
   parent_resource_id     = var.folder_id
+  include_children       = var.include_children
   parent_resource_type   = "folder"
   unique_writer_identity = "true"
 }

modules/event-folder-log-entry/variables.tf

@@ -39,3 +39,9 @@ variable "folder_id" {
   type        = string
   description = "The ID of the folder to look for changes."
 }
+
+variable "include_children" {
+  description = "Determines whether or not to include folder's children in the sink export. If true, logs associated with child projects are also exported; otherwise only logs relating to the provided folder are included."
+  type        = bool
+  default     = false
+}