Skip to content

Commit

Permalink
Merge pull request #6786 from jcaianirh/filter-update-alerts
Browse files Browse the repository at this point in the history
Bug 1883918: Proposed fix for duplicate UpdateAvailable notifications in the notification drawer
  • Loading branch information
openshift-merge-robot committed Sep 30, 2020
2 parents 73de5e7 + fd78a81 commit 039e371
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion frontend/public/components/notification-drawer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,12 @@ export const refreshNotificationPollers = () => {
const getAlerts = (alertsResults: PrometheusRulesResponse): Alert[] =>
alertsResults
? getAlertsAndRules(alertsResults.data)
.alerts.filter((a) => a.state === 'firing' && getAlertName(a) !== 'Watchdog')
.alerts.filter(
(a) =>
a.state === 'firing' &&
getAlertName(a) !== 'Watchdog' &&
getAlertName(a) !== 'UpdateAvailable',
)
.sort((a, b) => +new Date(getAlertTime(b)) - +new Date(getAlertTime(a)))
: [];

Expand Down

0 comments on commit 039e371

Please sign in to comment.