From b7addc8e71890dde04dfa155ea063d300e5e6829 Mon Sep 17 00:00:00 2001
From: Rakesh Gupta <rakesh.gupta.dev264@gmail.com>
Date: Mon, 18 May 2026 16:07:58 +0530
Subject: [PATCH] RANGER-5553: Incorrect Tags count reported in
 SummaryTOTALTAGS metric from /service/metrics/json API

---
 .../apache/ranger/metrics/RangerMetricsFetcher.java   | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/security-admin/src/main/java/org/apache/ranger/metrics/RangerMetricsFetcher.java b/security-admin/src/main/java/org/apache/ranger/metrics/RangerMetricsFetcher.java
index c01de60121f..00e8f928734 100644
--- a/security-admin/src/main/java/org/apache/ranger/metrics/RangerMetricsFetcher.java
+++ b/security-admin/src/main/java/org/apache/ranger/metrics/RangerMetricsFetcher.java
@@ -57,6 +57,7 @@ public class RangerMetricsFetcher {
 
     private final HashMap<String, Long> summaryReusable = new HashMap<>();
     private final HashMap<Integer, Long> summaryPolicy  = new HashMap<>();
+    private final Map<Integer, Long> summaryTagPolicy   = new HashMap<>();
 
     public Long getGroupCount() {
         Long totalGroupCount = groupService.getAllGroupCount();
@@ -121,6 +122,7 @@ public Map<String, Long> getPolicyMetrics(Integer policyType) {
 
         Map<String, Long> ret   = new HashMap<>();
         long              total = 0L;
+        long              totalTagPolicies = 0L;
 
         for (Map.Entry<String, Long> entry : svcStore.getPolicyCountByTypeAndServiceType(policyType).entrySet()) {
             ret.put(entry.getKey(), entry.getValue());
@@ -128,12 +130,13 @@ public Map<String, Long> getPolicyMetrics(Integer policyType) {
             total += entry.getValue();
 
             if ("tag".equalsIgnoreCase(entry.getKey())) {
-                summaryReusable.put("TotalTags", entry.getValue());
+                totalTagPolicies += entry.getValue().longValue();
             }
         }
 
         ret.put("Total", total);
         summaryPolicy.put(policyType, total);
+        summaryTagPolicy.put(policyType, totalTagPolicies);
 
         return ret;
     }
@@ -174,6 +177,9 @@ public Map<String, Long> getSummary() {
         //policies
         ret.put("TotalPolicies", summaryPolicy.values().stream().mapToLong(Long::longValue).sum());
 
+        //tag_policies
+        ret.put("TotalTagPolicies", summaryTagPolicy.values().stream().mapToLong(Long::longValue).sum());
+
         //roles
         ret.put("TotalRoles", daoMgr.getXXRole().getAllCount());
 
@@ -192,6 +198,9 @@ public Map<String, Long> getSummary() {
         //x_policy_export_audit
         ret.put("TotalPluginDownloads", daoMgr.getXXPolicyExportAudit().getAllCount());
 
+        //x_tag
+        ret.put("TotalTags", daoMgr.getXXTag().getAllCount());
+
         return ret;
     }