sensu / sensu-dashboard

A dashboard for Sensu, for displaying & managing events & clients.
http://sensuapp.org
MIT License
96 stars 44 forks source link

occurrences does not work #165

Open khuongdp opened 10 years ago

khuongdp commented 10 years ago

Hi

It does not look like the sensu-dashboard is using the occurrences defined in the checks. Is this a bug or by design?

We have some checks that sometimes fail because of network issue. That is why we are using occurrences in our checks. It will give false negative if the sensu-dashboard is not using the occurrences value.

amdprophet commented 10 years ago

This is by design but I'm open to the idea of changing that. Thoughts @portertech?

nstielau commented 10 years ago

Removing checks that are below the occurrences level makes sense to me. On May 6, 2014 3:23 PM, "Justin Kolberg" notifications@github.com wrote:

This is by design but I'm open to the idea of changing that. Thoughts @portertech https://github.com/portertech?

— Reply to this email directly or view it on GitHubhttps://github.com/sensu/sensu-dashboard/issues/165#issuecomment-42367546 .

khuongdp commented 10 years ago

+1 for nstielau idea

portertech commented 10 years ago

I like to see all events, even if they are due to a "false positive", as they are still an indication of failure of a service or of how its being monitored.

portertech commented 10 years ago

We really need linkable filters, and expose this functionality as a built-in filter.

portertech commented 10 years ago

Could be enabled by default, so an opt-out.

amdprophet commented 10 years ago

Abide by occurrences by default & add a filter / option to show everything?