optional logging path

Author: AJ Steers

Diff for: catalog/aws/singer-taps/main.tf

@@ -79,11 +79,13 @@ module "ecs_tap_sync_task" {
     {
       TAP_CONFIG_DIR                                    = "${var.data_lake_metadata_path}/tap-snapshot-${local.unique_hash}",
       TAP_STATE_FILE                                    = "${coalesce(var.data_lake_storage_path, var.data_lake_metadata_path)}/${var.state_file_naming_scheme}",
-      TAP_LOG_DIR                                       = "${var.data_lake_logging_path}/tap-${local.taps_specs[count.index].name}/",
       PIPELINE_VERSION_NUMBER                           = var.pipeline_version_number
       "${local.tap_env_prefix[count.index]}CONFIG_FILE" = "False" # Config will be passed via env vars
       "${local.target_env_prefix}CONFIG_FILE"           = "False" # Config will be passed via env vars
     },
+    var.data_lake_logging_path == null ? {} : {
+      TAP_LOG_DIR = "${var.data_lake_logging_path}/tap-${local.taps_specs[count.index].name}/"
+    },
     {
       for k, v in local.taps_specs[count.index].settings :
       "${local.tap_env_prefix[count.index]}${k}" => v

Diff for: catalog/aws/singer-taps/variables.tf

@@ -107,6 +107,7 @@ The remote folder for storing tap execution logs and log artifacts.
 Currently only S3 paths (s3://...) are supported.
 EOF
   type        = string
+  default     = null
 }
 variable "data_lake_metadata_path" {
   description = <<EOF