From: cloudbehl <cloudbehl@gmail.com>
Date: Wed, 4 Oct 2023 11:05:26 +0000 (+0530)
Subject: mgr/dashboard: Filter active alerts
X-Git-Tag: v19.0.0~320^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a40fd8ca92ad2ad0890cbab0cde1f17417b718ca;p=ceph.git

mgr/dashboard: Filter active alerts

- List alerts which are not suppressed

fixes: https://tracker.ceph.com/issues/62990

Signed-off-by: cloudbehl <cloudbehl@gmail.com>
---

diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/dashboard-v3/dashboard/dashboard-v3.component.html b/src/pybind/mgr/dashboard/frontend/src/app/ceph/dashboard-v3/dashboard/dashboard-v3.component.html
index 8faa23a5b2d0..5da863e29704 100644
--- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/dashboard-v3/dashboard/dashboard-v3.component.html
+++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/dashboard-v3/dashboard/dashboard-v3.component.html
@@ -299,7 +299,9 @@
 <ng-template #alertsCard>
   <ng-container *ngFor="let alert of prometheusAlertService.alerts; let i = index;  trackBy: trackByFn">
     <div [ngClass]="['border-'+alertClass[alert.labels.severity]]"
-         *ngIf="alert.labels.severity === alertType || !alertType">
+         *ngIf="alert.status.state === 'active' &&
+         alert.labels.severity === alertType ||
+         !alertType">
       <div class="card tc_alerts border-0 pt-3">
         <div class="row no-gutters ps-2">
           <div class="col-sm-1 text-center">
diff --git a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/prometheus-alert.service.ts b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/prometheus-alert.service.ts
index be6c27da6e2a..2830fd00c242 100644
--- a/src/pybind/mgr/dashboard/frontend/src/app/shared/services/prometheus-alert.service.ts
+++ b/src/pybind/mgr/dashboard/frontend/src/app/shared/services/prometheus-alert.service.ts
@@ -80,9 +80,9 @@ export class PrometheusAlertService {
         alert.status.state === 'active' && alert.labels.severity === 'warning' ? ++result : result,
       0
     );
-    this.alerts = alerts.reverse().sort((a, b) => {
-      return a.labels.severity.localeCompare(b.labels.severity);
-    });
+    this.alerts = alerts
+      .reverse()
+      .sort((a, b) => a.labels.severity.localeCompare(b.labels.severity));
     this.canAlertsBeNotified = true;
   }