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

Disabled Devices | Still show in Alerts Widget #6213

Closed
dneto82 opened this Issue Mar 17, 2017 · 5 comments

Comments

Projects
None yet
3 participants
@dneto82

dneto82 commented Mar 17, 2017

DO NOT DELETE THIS INFORMATION.

Please read this information carefully.

GitHub issues is for bugs, please do not post issues asking for help or how to do X, Y or Z.
You can use our irc channel ##librenms on freenode to ask questions or our community site.
If you have a feature request please post this on our community site.

Please confirm each of the sections below by putting an x in the box like [x].

  • Is your install up to date? Updating your install
    Please do not submit an issue if your install is not up to date within the last 24 hours or on a stable monthly release.
  • Please include all of the information between the ==================================== section of ./validate.php which you can run from the cli.
  • Please provide ALL info asked for here.
  • Please provide as much detail as possible.
  • Please do NOT post more than 10 lines of debug information here, use a pastebin service or GitHub Gists.

==========================================================
Component | Version
--------- | -------
LibreNMS | 559a221
DB Schema | 177
PHP | 7.0.16
MySQL | 5.5.52-MariaDB
RRDTool | 1.6.0
SNMP | NET-SNMP 5.7.2

Disabled device correctly isn't showed in Summary Widget, WorldMap but still present in Alerts Widget:

image

image

Maybe to just make sure to show only enabled devices in Alert widget.

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Mar 17, 2017

Member

You need to post your rule as the showing of disabled devices in alerts is down to that and nothing else

Member

laf commented Mar 17, 2017

You need to post your rule as the showing of disabled devices in alerts is down to that and nothing else

@dneto82

This comment has been minimized.

Show comment
Hide comment
@dneto82

dneto82 Mar 17, 2017

Hi @laf . Thanks.

This rule is a default / predefined rule from install.

image

On widget, I'm using these filters:

image

I tried to change my rule to include an AND device.status !=0, but without success. Device status is related to Enable/Disabled status right?

image

dneto82 commented Mar 17, 2017

Hi @laf . Thanks.

This rule is a default / predefined rule from install.

image

On widget, I'm using these filters:

image

I tried to change my rule to include an AND device.status !=0, but without success. Device status is related to Enable/Disabled status right?

image

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Mar 22, 2017

Member

I expect this is because the device was down before you disabled it and so the alert is still active as the device is no longer polled due to it being disabled and therefore can't refresh or clear out it's stale alerts.

So the question is really, what should we do in this instance.

Clear out alerts when a device is disabled?
Leave as is?
Hide alerts for disabled devices in the alerts screen?

@librenms/reviewers Thoughts?

Member

laf commented Mar 22, 2017

I expect this is because the device was down before you disabled it and so the alert is still active as the device is no longer polled due to it being disabled and therefore can't refresh or clear out it's stale alerts.

So the question is really, what should we do in this instance.

Clear out alerts when a device is disabled?
Leave as is?
Hide alerts for disabled devices in the alerts screen?

@librenms/reviewers Thoughts?

@laf laf added the Discussion label Mar 22, 2017

@murrant

This comment has been minimized.

Show comment
Hide comment
@murrant

murrant Mar 23, 2017

Member

Hide the alerts, that way if the device is re-enabled, they will still be there.

Member

murrant commented Mar 23, 2017

Hide the alerts, that way if the device is re-enabled, they will still be there.

@lock

This comment has been minimized.

Show comment
Hide comment
@lock

lock bot May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

lock bot commented May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

@lock lock bot locked as resolved and limited conversation to collaborators May 17, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.