-
Notifications
You must be signed in to change notification settings - Fork 13
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
A problem state may be opened which will never be closed by detection #53
Comments
While I'm thinking about possible suggestions, I might add that the same issue would arise if the resolving alert never reaches AAS for some reason (server errors, network errors, whatever issues). We could always make NAV generate the corresponding alert to trigger dispatch to AAS - but the generic issue remains - a resolving alert may never reach AAS and it will become out of sync |
Looks like we need both an ack-function and an stay-dead-function I'd say. UI-wise... the turn-off button needs be less comfortable to use so it isn't triggered by accident. |
Yes, we need a function for manually closing an alert by a human using the frontend. This would be logged to the Event table, associated with that user, as the event is closed, so you can see the difference between an alert that was resolved by message from the alert source, and one that was closed manually. NAV has this same function. |
We now have a manual way for a human to close an incident. I wonder: could we also have an agent that can close "old" incidents? Maybe if we decommission a source, we could ask that agent to close everything from that source. Which reminds me, some logic only in the api needs to move to the models/queryset. |
Adding a note from an old discussion concering the glue service for NAV: We expect to make the glue service itself able to sync pre-existing NAV problems to Argus - making it able to do it the other way around might be just as interesting - i.e. make it able to see which of its active Argus incidents have already been closed in NAV, and update the Argus Incidents. |
|
Originally posted by @lunkwill42 in #45 (comment)
The text was updated successfully, but these errors were encountered: