limepotato/akkoma
1
0
Fork 0
Code Pull requests Activity

Don't add summary metrics to prometheus

Browse source 
The exporter doesn’t support them thus we don't lose anything by this,
but it avoids a bunch of warnings each time the server starts up.
This commit is contained in:
Oneric 2024-02-03 17:30:00 +01:00
parent e97d08ee98
commit a6df71eebb
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
lib/pleroma/web/telemetry.ex
View file

@ -101,6 +101,7 @@ defmodule Pleroma.Web.Telemetry do
]
end
# Summary metrics are currently not (yet) supported by the prometheus exporter
defp summary_metrics do
[
# Phoenix Metrics
@ -121,7 +122,12 @@ defmodule Pleroma.Web.Telemetry do
summary("vm.memory.total", unit: {:byte, :kilobyte}),
summary("vm.total_run_queue_lengths.total"),
summary("vm.total_run_queue_lengths.cpu"),
summary("vm.total_run_queue_lengths.io"),
summary("vm.total_run_queue_lengths.io")
]
end
defp common_metrics do
[
last_value("pleroma.local_users.total"),
last_value("pleroma.domains.total"),
last_value("pleroma.local_statuses.total"),
@ -129,8 +135,8 @@ defmodule Pleroma.Web.Telemetry do
]
end
def prometheus_metrics, do: summary_metrics() ++ distribution_metrics()
def live_dashboard_metrics, do: summary_metrics()
def prometheus_metrics, do: common_metrics() ++ distribution_metrics()
def live_dashboard_metrics, do: common_metrics() ++ summary_metrics()
defp periodic_measurements do
[