Project

General

Profile

Feature #22482

Avatar?id=14398&size=50x50

Feature #22184: Make Alerts Great Again!

Create new "Alerts UI" tab

Added by Kris Moore about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Expected
Assignee:
Marcelo Araujo
Category:
Middleware
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

As part of the related parent feature we want to make "consul-alerts" a ui option. I've added a consulkv plugin which will enable get / set of consul kv via middlewared. Next we need a new UI tab under "system" which will list the various services that consul-alerts support with checkbox(s) to enable / disable them.

Details on the KV for each service is herw:
https://github.com/AcalephStorage/consul-alerts/blob/master/README.md

I've already added a generic health alert that monitors alerts issued from the UI. It should start reporting to the various enabled services once you create the gui.

Associated revisions

Revision 819497de (diff)
Added by Marcelo Araujo about 4 years ago

feat(consul): Add consul.io as option where users can use it to monitor their FreeNAS. Ticket: #22482

Revision 4127cdb7 (diff)
Added by Marcelo Araujo about 4 years ago

feat(consul): GUI and middlewared implementation for consul.io. Ticket: #22482

Revision 5a21d6f2 (diff)
Added by Warren Block about 4 years ago

Add Alert Services section. Ticket: #22482 Ticket: #23321

History

#1 Updated by Marcelo Araujo about 4 years ago

  • Status changed from Unscreened to Screened

#2 Updated by Marcelo Araujo about 4 years ago

  • Status changed from Screened to Needs Developer Review
  • Assignee changed from Marcelo Araujo to Kris Moore

Ready for review.

#3 Avatar?id=14398&size=24x24 Updated by Kris Moore about 4 years ago

The functionality looks good. Only thing I'd ask is you rename the tab from "Consul Alerts" to just "Alerts" since most users don't know or even will see that its using consul in the background.

#4 Avatar?id=14398&size=24x24 Updated by Kris Moore about 4 years ago

  • Assignee changed from Kris Moore to Marcelo Araujo

#5 Updated by Marcelo Araujo about 4 years ago

  • Status changed from Needs Developer Review to Resolved

#6 Avatar?id=14398&size=24x24 Updated by Kris Moore about 4 years ago

  • Target version changed from 9.10.3 to 11.0

#8 Updated by Vaibhav Chauhan about 4 years ago

  • Target version changed from 11.0 to 11.0-RC

Also available in: Atom PDF