Merge pull request #227 from silinternational/feature/id-sync-sentry

add SENTRY_DSN to ID Sync task definition
silinternational · Oct 6, 2023 · d4b6ee0 · d4b6ee0
2 parents f11a9a5 + 20b94c9
commit d4b6ee0
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 0 deletions.
diff --git a/terraform/070-id-sync/README.md b/terraform/070-id-sync/README.md
@@ -47,6 +47,7 @@ store.
 - `create_dns_record` - Controls creation of a DNS CNAME record for the ECS service. Default: `true`
 - `enable_new_user_notification` - Enable email notification to HR Contact upon creation of a new user, if set to 'true'. Default: `false`
 - `enable_sync` - Set to false to disable the sync process.
+- `sentry_dsn` - Sentry DSN for error logging and alerting. Obtain from Sentry dashboard: Settings - Projects - (project) - Client Keys
 
 ## Outputs
 

diff --git a/terraform/070-id-sync/main.tf b/terraform/070-id-sync/main.tf
@@ -83,6 +83,7 @@ locals {
     sync_safety_cutoff           = var.sync_safety_cutoff
     allow_empty_email            = var.allow_empty_email
     enable_new_user_notification = var.enable_new_user_notification
+    sentry_dsn                   = var.sentry_dsn
   })
 }
 

diff --git a/terraform/070-id-sync/task-definition.json b/terraform/070-id-sync/task-definition.json
@@ -97,6 +97,10 @@
         "name": "ENABLE_NEW_USER_NOTIFICATION",
         "value": "${enable_new_user_notification}"
       },
+      {
+        "name": "SENTRY_DSN",
+        "value": "${sentry_dsn}"
+      },
       ${id_store_config}
     ],
     "links": null,

diff --git a/terraform/070-id-sync/vars.tf b/terraform/070-id-sync/vars.tf
@@ -147,3 +147,9 @@ variable "create_dns_record" {
   type        = bool
   default     = true
 }
+
+variable "sentry_dsn" {
+  description = "Sentry DSN for error logging and alerting"
+  type        = string
+  default     = null
+}