github-personal/firezone
1
0
Fork 0
mirror of https://github.com/outbackdingo/firezone.git synced 2026-01-27 10:18:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(portal): Only compute diff for metrics with both start and end times (#8147)

Browse Source 
A fix for a nil error from #8146
This commit is contained in:
Jamil
2025-02-16 12:57:03 -08:00
committed by GitHub
parent 36b887e98e
commit 311988c5a2
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
elixir/apps/domain/lib/domain/telemetry/reporter/google_cloud_metrics.ex
View File

@@ -144,15 +144,14 @@ defmodule Domain.Telemetry.Reporter.GoogleCloudMetrics do
defp all_intervals_greater_than_5s?(buffer) do
Enum.all?(buffer, fn {{schema, _name, _tags, _unit}, measurements} ->
{started_at, ended_at, _} = measurements
diff = DateTime.diff(ended_at, started_at, :second)
# Only Distribution and Summary metrics use intervals
case schema do
Metrics.Counter -> true
Metrics.Sum -> true
Metrics.LastValue -> true
Metrics.Distribution -> diff > 5
Metrics.Summary -> diff > 5
Metrics.Distribution -> DateTime.diff(ended_at, started_at, :second) > 5
Metrics.Summary -> DateTime.diff(ended_at, started_at, :second) > 5
end
end)
end