add sentry to uk forecasts (#619)

* add sentry to uk forecasts * add sentry to uk production * update version of forecasts * add variable * fix typo * remove production changes
openclimatefix · Sep 22, 2024 · 92daea0 · 92daea0
1 parent 942fc1e
commit 92daea0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/terraform/nowcasting/development/main.tf b/terraform/nowcasting/development/main.tf
@@ -95,7 +95,7 @@ module "api" {
   container-env_vars = [
     { "name" : "DB_URL", "value" :  module.database.forecast-database-secret-url},
     { "name" : "ORIGINS", "value" : "*" },
-    { "name" : "SENTRY_DSN", "value" : var.sentry_dsn },
+    { "name" : "SENTRY_DSN", "value" : var.sentry_dsn_api },
     { "name" : "AUTH0_DOMAIN", "value" : var.auth_domain },
     { "name" : "AUTH0_API_AUDIENCE", "value" : var.auth_api_audience },
     { "name" : "AUTH0_RULE_NAMESPACE", "value" : "https://openclimatefix.org"},
@@ -326,6 +326,8 @@ module "national_forecast" {
     bucket_read_policy_arn = module.s3.iam-policy-s3-nwp-read.arn
     datadir                = "data-national"
   }
+
+  sentry_dsn = var.sentry_dsn
 }
 
 # 4.4
@@ -363,7 +365,7 @@ module "forecast_pvnet" {
   pvnet_gsp_sum = "true"
   ecs-task_execution_role_arn = module.ecs.ecs_task_execution_role_arn
   run_extra_models = "true"
-  sentry_dsn = var.sentry_dsn_backend
+  sentry_dsn = var.sentry_dsn
 }
 
 # 4.5
@@ -400,6 +402,7 @@ module "forecast_pvnet_day_ahead" {
   loglevel      = "INFO"
   ecs-task_execution_role_arn = module.ecs.ecs_task_execution_role_arn
   day_ahead_model = "true"
+  sentry_dsn = var.sentry_dsn
 }
 
 # 5.1
@@ -498,7 +501,7 @@ module "pvsite_api" {
     { "name" : "DB_URL", "value" : module.pvsite_database.default_db_connection_url},
     { "name" : "FAKE", "value" : "0" },
     { "name" : "ORIGINS", "value" : "*" },
-    { "name" : "SENTRY_DSN", "value" : var.sentry_dsn },
+    { "name" : "SENTRY_DSN", "value" : var.sentry_dsn_api },
     { "name" : "AUTH0_API_AUDIENCE", "value" : var.auth_api_audience },
     { "name" : "AUTH0_DOMAIN", "value" : var.auth_domain },
     { "name" : "AUTH0_ALGORITHM", "value" : "RS256" },
@@ -553,6 +556,7 @@ module "pvsite_forecast" {
     datadir                = "data-national"
   }
   ecs-task_execution_role_arn = module.ecs.ecs_task_execution_role_arn
+  sentry_dsn = var.sentry_dsn
 }
 
 # 6.5

diff --git a/terraform/nowcasting/development/variables.tf b/terraform/nowcasting/development/variables.tf
@@ -117,7 +117,7 @@ variable "forecast_pvnet_day_ahead_docker_version" {
   default = "not-set"
 }
 
-variable "sentry_dsn_backend" {
+variable "sentry_dsn_api" {
   type = string
   description = "The Sentry DSN for all backend components"
   default = ""