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

Add display_status property to hosts and services objects #463

Closed
wants to merge 3 commits into from

Conversation

dirtyren
Copy link
Contributor

@dirtyren dirtyren commented May 20, 2024

This is a idea we have been using for quite sometime in order to gain performance when filtering and adding a new concept of status to naemon.

The idea is to order items by priority and as shown in the image bellow.

status_colors

Before I fix any of the required steps to merge, let me see what you guys think about it.

Tks.

@dirtyren dirtyren closed this May 20, 2024
@dirtyren dirtyren reopened this May 20, 2024
@sni
Copy link
Contributor

sni commented May 21, 2024

tbh, this is a hard coded and very opinionated sort order. I don't see a general benefit.
Could you explain how this improves performance?

@dirtyren
Copy link
Contributor Author

dirtyren commented May 24, 2024

For this specific sort, it increases performance when doing in livestatus.
This is just a proposal to order objects based on incidents that should be checked first in a monitoring system, in my opinion, off course. And also the ideia for the new statuses colors I find it useful.
Off course this could be acompleshed by filtering out current statuses.

Tks anyway, we can close this pull request, no problem.

@dirtyren dirtyren closed this May 29, 2024
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 this pull request may close these issues.

None yet

2 participants