Add timezone to Quantum check-ins (#860)

Author: whatyouhide

lib/sentry/integrations/quantum/cron.ex

@@ -70,7 +70,7 @@ defmodule Sentry.Integrations.Quantum.Cron do
         check_in_id: id,
         # This is already a binary.
         monitor_slug: "quantum-#{slugify(job.name)}",
-        monitor_config: [schedule: schedule_opts]
+        monitor_config: [schedule: schedule_opts, timezone: timezone_to_string(job)]
       ]
     else
       nil
@@ -97,6 +97,11 @@ defmodule Sentry.Integrations.Quantum.Cron do
     |> Kernel./(1000)
   end
 
+  defp timezone_to_string(%{timezone: :utc}), do: "Etc/UTC"
+  defp timezone_to_string(%{timezone: nil}), do: "Etc/UTC"
+  defp timezone_to_string(%{timezone: timezone}), do: timezone
+  defp timezone_to_string(_job_without_timezone_key_for_some_reason), do: "Etc/UTC"
+
   defp slugify(job_name) when is_reference(job_name) do
     Logger.error(
       """

test/sentry/integrations/quantum/cron_test.exs

@@ -67,7 +67,8 @@ defmodule Sentry.Integrations.Quantum.CronTest do
                "schedule" => %{
                  "type" => "crontab",
                  "value" => "0 0 * * * *"
-               }
+               },
+               "timezone" => "Etc/UTC"
              }
 
       send(test_pid, {ref, :done})
@@ -108,7 +109,8 @@ defmodule Sentry.Integrations.Quantum.CronTest do
                "schedule" => %{
                  "type" => "crontab",
                  "value" => "0 0 * * * *"
-               }
+               },
+               "timezone" => "Europe/Rome"
              }
 
       send(test_pid, {ref, :done})
@@ -122,7 +124,8 @@ defmodule Sentry.Integrations.Quantum.CronTest do
       job:
         Scheduler.new_job(
           name: :test_job,
-          schedule: Crontab.CronExpression.Parser.parse!("@daily")
+          schedule: Crontab.CronExpression.Parser.parse!("@daily"),
+          timezone: "Europe/Rome"
         ),
       telemetry_span_context: ref
     })
@@ -151,7 +154,8 @@ defmodule Sentry.Integrations.Quantum.CronTest do
                "schedule" => %{
                  "type" => "crontab",
                  "value" => "0 0 * * * *"
-               }
+               },
+               "timezone" => "Etc/UTC"
              }
 
       send(test_pid, {ref, :done})