From b188a168af1850ac9c2385c7d7cd432ccb7b8099 Mon Sep 17 00:00:00 2001 From: Kenneth Jenkins <51246568+kenjenkins@users.noreply.github.com> Date: Tue, 25 Mar 2025 14:48:07 -0700 Subject: [PATCH] metrics: fix an apparent metric setup error (#5543) The IdentityManagerLastSessionRefreshErrorView appears to be a duplicate of IdentityManagerLastUserRefreshErrorView. Adjust it to use the matching identityManagerLastSessionRefreshError instead. --- internal/telemetry/metrics/info.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/telemetry/metrics/info.go b/internal/telemetry/metrics/info.go index 902cc404b..9b18d0eba 100644 --- a/internal/telemetry/metrics/info.go +++ b/internal/telemetry/metrics/info.go @@ -240,11 +240,11 @@ var ( Measure: identityManagerLastSessionRefreshSuccess, Aggregation: view.Count(), } - // IdentityManagerLastSessionRefreshErrorView contains user refresh errors counter + // IdentityManagerLastSessionRefreshErrorView contains session refresh errors counter IdentityManagerLastSessionRefreshErrorView = &view.View{ - Name: identityManagerLastUserRefreshError.Name(), - Description: identityManagerLastUserRefreshError.Description(), - Measure: identityManagerLastUserRefreshError, + Name: identityManagerLastSessionRefreshError.Name(), + Description: identityManagerLastSessionRefreshError.Description(), + Measure: identityManagerLastSessionRefreshError, Aggregation: view.Count(), } // IdentityManagerLastSessionRefreshSuccessTimestampView contains successful session refresh counter