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

Host based elements are displaying the check result rather than the host state. #220

Closed
sol1-matt opened this issue Aug 22, 2023 · 1 comment · Fixed by #221
Closed

Host based elements are displaying the check result rather than the host state. #220

sol1-matt opened this issue Aug 22, 2023 · 1 comment · Fixed by #221
Assignees
Milestone

Comments

@sol1-matt
Copy link
Contributor

Describe the bug
Host based elements are displaying the check result rather than the host state.
The check result is from results[0].attr.last_check_result.state and is the result the check returns. This can be OK, WARNING, CRITICAL, UNKNOWN or PENDING.

A hosts status is from results[0].attr.state and is OK, CRITICAL, UNKNOWN or PENDING.

To Reproduce
Steps to reproduce the behavior:

  1. Create test dashboard
  2. Add element with Element Type of Host
  3. Select a host from the drop down that is in a OK state but the check is returning a warning state, eg: host with http check returning 404.
    The element returns as warning which is incorrect.

Expected behavior
The element state should match Icinga's state

Additional context
Meerkat can predict what the host state should be based on the last_check_result.state value if the event stream doesn't have the hosts state and only has the last check results state.

  • Meerkat Version = 3.1.0
@sol1-matt sol1-matt added this to the v3.1 milestone Aug 22, 2023
63 added a commit that referenced this issue Aug 22, 2023
@63
Copy link
Collaborator

63 commented Aug 23, 2023

@sol1-matt sol1-matt linked a pull request Aug 24, 2023 that will close this issue
@sol1-matt sol1-matt mentioned this issue Aug 24, 2023
sol1-matt added a commit that referenced this issue Aug 24, 2023
Fix: #219 Call correct parameter's for host requests
Fix: #220 Make dashboard elements for host based objects return correct mapping for Up/Down based on check state.
Fix: Fix problem with frontends where loss of connect to Meerkat caused the frontend to spam requests at a unreasonable rate.
Fix: version flag (really build string) in Meerkat to report correct build string
---------

Co-authored-by: 63 <55000067+63@users.noreply.github.com>
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

Successfully merging a pull request may close this issue.

2 participants