-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
Sound configuration on basis of severity #1886
Comments
This is also something I require. |
I think this is more of a web-ui thingy to be honest. to my knowledge the only thing that the backend does is to say which file to play.. and yes from the looks of it, there a no difference between severity levels. that being said, I think if you need this you probably need to develop it and do a PR.. The UI needs an overhaul in my mind, but I am not a UI programmer, so its sorta out of scope for me.. |
@hyberdk isn't it useful say you are monitioring lot of things and you want an sound when something critical or major happened (alert with severity major or critical) . Ideally minor alerts are harmless and doesn't require immediate attention so don't want sound over these . It would be greate if we can define severity sound configuration in alerta setting.py (severitySoundMap = {'critical', 'major'}) so when alerta reads from config (initial load) it will know that it will need to raise sound only for those severity |
This Map will also be useful if in future if we define our own severity and want sound only on that severity alert |
hey, im not saying that the feature is not useful, Im sure it is and I am sure that plenty of people want this. but there are some "issues" with the way the feature is implemented currently for one, you need to distribute the file to all PCs that are connected (from what I can tell) and that will just become more important going forward if you add multiple files. What I am trying to say is that the most people that I know of are actually working on the backend, no so much on the UI. The UI could need some TLC and since no one is doing that, it is unlikely they will take on new features also. We actually ended up not using the UI, but building a plugin that sends all triggered/updated alerts into a Clickhouse database, so we can reference all alerts from there. This means we can build the dashboard the way we like and we have a record of all alerts that we can annotate on top of any graphs we currently have (if we want).. Like I said Im not saying its not a valid feature, Im sure it is.. but the way I see it a great deal of the work has to be done in the UI and that is not being updated as much as the backend. |
Can we add sound on basis of severity
say a json/list which will hold these params
or we can enhance more where each severity can have different sound
The text was updated successfully, but these errors were encountered: