From e84d88fdc98b8b6e73b82e04588a3aa01efd78ba Mon Sep 17 00:00:00 2001
From: Ryan Yin <xiaoyin_c@qq.com>
Date: Mon, 9 Jun 2025 03:12:22 +0800
Subject: [PATCH] fix: loki-operational - trail comma

---
 .../loki-mixin/loki-operational.json          | 34 +++++++++----------
 1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/hosts/idols-aquamarine/grafana/dashboards/loki-mixin/loki-operational.json b/hosts/idols-aquamarine/grafana/dashboards/loki-mixin/loki-operational.json
index 33bc92f6..0a74a5b7 100644
--- a/hosts/idols-aquamarine/grafana/dashboards/loki-mixin/loki-operational.json
+++ b/hosts/idols-aquamarine/grafana/dashboards/loki-mixin/loki-operational.json
@@ -779,7 +779,7 @@
       "steppedLine": false,
       "targets": [
         {
-          "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", }[$__rate_interval])) by (route) > 0",
+          "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\"}[$__rate_interval])) by (route) > 0",
           "interval": "",
           "legendFormat": "{{route}}",
           "refId": "A"
@@ -1300,7 +1300,7 @@
       "steppedLine": false,
       "targets": [
         {
-          "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", }[$__rate_interval])) by (route) > 0",
+          "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\"}[$__rate_interval])) by (route) > 0",
           "interval": "",
           "legendFormat": "{{route}}",
           "refId": "A"
@@ -2031,7 +2031,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\", } | logfmt | level=\"error\"[$__auto]))",
+              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\"} | logfmt | level=\"error\"[$__auto]))",
               "refId": "A"
             }
           ],
@@ -2094,7 +2094,7 @@
           "panels": [],
           "targets": [
             {
-              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\", } |= \"level=error\"",
+              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\"} |= \"level=error\"",
               "refId": "A"
             }
           ],
@@ -2145,7 +2145,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", }[$__rate_interval])) by (route) > 0",
+              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\"}[$__rate_interval])) by (route) > 0",
               "interval": "",
               "intervalFactor": 1,
               "legendFormat": "{{route}}",
@@ -2724,7 +2724,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\", } | logfmt | level=\"error\"[$__auto]))",
+              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\"} | logfmt | level=\"error\"[$__auto]))",
               "refId": "A"
             }
           ],
@@ -2787,7 +2787,7 @@
           "panels": [],
           "targets": [
             {
-              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\", } |= \"level=error\"",
+              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\"} |= \"level=error\"",
               "refId": "A"
             }
           ],
@@ -2838,7 +2838,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", }[$__rate_interval])) by (route) > 0",
+              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\"}[$__rate_interval])) by (route) > 0",
               "interval": "",
               "intervalFactor": 1,
               "legendFormat": "{{route}}",
@@ -2946,7 +2946,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "topk(10,sum by (tenant) (loki_ingester_memory_streams{cluster=\"$cluster\",}))",
+              "expr": "topk(10,sum by (tenant) (loki_ingester_memory_streams{cluster=\"$cluster\"}))",
               "interval": "",
               "legendFormat": "{{ tenant }}",
               "refId": "A"
@@ -3037,7 +3037,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "topk(10, sum by (tenant) (rate(loki_ingester_streams_created_total{cluster=\"$cluster\",}[$__rate_interval]) > 0))",
+              "expr": "topk(10, sum by (tenant) (rate(loki_ingester_streams_created_total{cluster=\"$cluster\"}[$__rate_interval]) > 0))",
               "interval": "",
               "legendFormat": "{{ tenant }}",
               "refId": "A"
@@ -3147,13 +3147,13 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate(loki_ingester_chunks_flushed_total{cluster=\"$cluster\",}[$__rate_interval]))",
+              "expr": "sum(rate(loki_ingester_chunks_flushed_total{cluster=\"$cluster\"}[$__rate_interval]))",
               "interval": "",
               "legendFormat": "Chunks",
               "refId": "A"
             },
             {
-              "expr": "sum(increase(loki_chunk_store_deduped_chunks_total{cluster=\"$cluster\", }[$__rate_interval]))/sum(increase(loki_ingester_chunks_flushed_total{cluster=\"$cluster\", }[$__rate_interval])) < 1",
+              "expr": "sum(increase(loki_chunk_store_deduped_chunks_total{cluster=\"$cluster\"}[$__rate_interval]))/sum(increase(loki_ingester_chunks_flushed_total{cluster=\"$cluster\"}[$__rate_interval])) < 1",
               "interval": "",
               "legendFormat": "De-Dupe Ratio",
               "refId": "B"
@@ -3231,7 +3231,7 @@
           "reverseYBuckets": false,
           "targets": [
             {
-              "expr": "sum(rate(loki_ingester_chunk_size_bytes_bucket{cluster=\"$cluster\",}[$__rate_interval])) by (le)",
+              "expr": "sum(rate(loki_ingester_chunk_size_bytes_bucket{cluster=\"$cluster\"}[$__rate_interval])) by (le)",
               "format": "heatmap",
               "instant": false,
               "interval": "",
@@ -3386,7 +3386,7 @@
           "reverseYBuckets": false,
           "targets": [
             {
-              "expr": "sum by (le) (rate(loki_ingester_chunk_utilization_bucket{cluster=\"$cluster\", }[$__rate_interval]))",
+              "expr": "sum by (le) (rate(loki_ingester_chunk_utilization_bucket{cluster=\"$cluster\"}[$__rate_interval]))",
               "format": "heatmap",
               "instant": false,
               "interval": "",
@@ -3669,7 +3669,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\", } | logfmt |  level=\"error\"[$__auto]))",
+              "expr": "sum(rate({cluster=\"$cluster\", namespace=\"$namespace\"} | logfmt |  level=\"error\"[$__auto]))",
               "refId": "A"
             }
           ],
@@ -3732,7 +3732,7 @@
           "panels": [],
           "targets": [
             {
-              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\", } |= \"level=error\"",
+              "expr": "{cluster=\"$cluster\", namespace=\"$namespace\"} |= \"level=error\"",
               "refId": "A"
             }
           ],
@@ -3783,7 +3783,7 @@
           "steppedLine": false,
           "targets": [
             {
-              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", }[$__rate_interval])) by (route) > 0",
+              "expr": "sum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\", status_code!~\"5[0-9]{2}\"}[$__rate_interval])) by (route)\n/\nsum(rate(loki_request_duration_seconds_count{cluster=\"$cluster\", namespace=\"$namespace\"}[$__rate_interval])) by (route) > 0",
               "interval": "",
               "intervalFactor": 1,
               "legendFormat": "{{route}}",