Skip to content
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

Open
skather2811 opened this issue Oct 12, 2023 · 6 comments
Open

Sound configuration on basis of severity #1886

skather2811 opened this issue Oct 12, 2023 · 6 comments

Comments

@skather2811
Copy link

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

@skather2811
Copy link
Author

@satterly

@sanpachino
Copy link

This is also something I require.
Any updates on this feature? Or is it already possible to this and I am not aware?

@hyberdk
Copy link
Contributor

hyberdk commented Dec 12, 2023

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..

@skather2811
Copy link
Author

@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

@skather2811
Copy link
Author

This Map will also be useful if in future if we define our own severity and want sound only on that severity alert

@hyberdk
Copy link
Contributor

hyberdk commented Dec 12, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants