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

Feature: Rewrite check kernel #533

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LordHepipud
Copy link
Collaborator

The current handling for compiling Icinga for Windows checks causes various issues in certain situations. These include:

  • Wrongly detected units and therefor wrong conversion of values
  • Negative thresholds sometimes not being applied properly
  • Slow execution of check compiling
  • Wrong report of warning/critical value comparison, determining if the thresholds are properly configured

To resolve this, the entire check kernel backend is re-written and redesigned with less overhead and proper evaluation.

Fixes #131
Fixes #435
Fixes #503
Fixes #528

@cla-bot cla-bot bot added the cla/signed label Jun 9, 2022
@LordHepipud LordHepipud added this to the v1.10.0 milestone Jun 9, 2022
@LordHepipud LordHepipud self-assigned this Jun 9, 2022
@LordHepipud LordHepipud modified the milestones: v1.10.0, v1.11.0 Aug 23, 2022
@drapiti
Copy link

drapiti commented Nov 29, 2022

We have noticed on version 1.10 of the framework most likely a side effect of the slightly high consumption in certain moments and in any case much better than previous versions, however many checks timeout and go into an unknown state if there are few resources or the server is under load. They do recover however the monitoring and performance metrics have notable holes. Hopefully this can be addressed with the kernel rewrite. Another small step is required for the framework to be reliable on all server workloads. We have over 5k windows servers now with the framework so we were able to witness this behaviour on a consistent basis.
Cheers.

@LordHepipud LordHepipud modified the milestones: v1.11.0, v1.12.0 Jul 24, 2023
@LordHepipud LordHepipud modified the milestones: v1.12.0, v2.0.0 Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment