-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Solution Provided] Summarization by Alert State #103
Comments
Hi @renzsickle, |
Sure. It's been some time since I posted this, but I have been able to branch from an older commit and apply the patch file. However, when I try to push my new branch up, I encounter the following error: $ git push origin feature/alert_summary Please make sure you have the correct access rights As I haven't contributed to a project on Github before, this may well be an error in my configuration (I just setup a fresh SSH key) - or I might genuinely not have access (my new key was enough to clone the repository). I've also uploaded the patch below for viewing: It seems to apply cleanly to commit a22631e. Please let me know how you'd like to proceed. |
Hi @renzsickle There is the how-to:
Don't hesitate to ping me if you need further help 👍 |
I've made a few changes for a customer and would like to offer them back to the core project. Could I get some feedback on the below?
In summary:
In detail:
Added the ability to show an additional column with the state of each alert (active, suppressed or inhibited).
Added a summary display mode, which shows a count of alerts in each state (works better with the multistat panel). Added the ability to toggle between full (original) and summary display modes.
Column and label filters work in summary display mode.
Summary display mode easily translates into multistat panels.
Sample configuration for multistat panels:
If these changes are of interest, I have a patchfile ready to go.
The text was updated successfully, but these errors were encountered: