From 57c7e04f04bc343defa2aa9b119963903f937cad Mon Sep 17 00:00:00 2001 From: Hornwitser Date: Tue, 21 Feb 2023 19:55:02 +0100 Subject: [PATCH] Use Summary metric for query log duration --- packages/master/src/ControlConnection.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/master/src/ControlConnection.js b/packages/master/src/ControlConnection.js index 2a7fbd3e..5fe52b8f 100644 --- a/packages/master/src/ControlConnection.js +++ b/packages/master/src/ControlConnection.js @@ -20,9 +20,9 @@ const routes = require("./routes"); const strcmp = new Intl.Collator(undefined, { numerice: "true", sensitivity: "base" }).compare; -const lastQueryLogTime = new libPrometheus.Gauge( - "clusterio_master_last_query_log_duration_seconds", - "Time in seconds the last log query took to execute." +const queryLogTime = new libPrometheus.Summary( + "clusterio_master_query_log_duration_seconds", + "Time in seconds log queries took to execute." ); /** @@ -683,7 +683,7 @@ class ControlConnection extends BaseConnection { } async queryLogRequestHandler(message) { - let setDuration = lastQueryLogTime.startTimer(); + let observeDuration = queryLogTime.startTimer(); let { all, master, slave_ids, instance_ids } = message.data; let log; @@ -693,7 +693,7 @@ class ControlConnection extends BaseConnection { log = await this._master.queryClusterLog(message.data); } - setDuration(); + observeDuration(); return { log }; }