Skip to content

doc/08-advanced-topics.md: Reference CheckResult state#10421

Merged
yhabteab merged 1 commit intomasterfrom
doc-08-advanced-topics-checkresult-state-ref
May 12, 2025
Merged

doc/08-advanced-topics.md: Reference CheckResult state#10421
yhabteab merged 1 commit intomasterfrom
doc-08-advanced-topics-checkresult-state-ref

Conversation

@oxzi
Copy link
Copy Markdown
Member

@oxzi oxzi commented Apr 25, 2025

Reference the Check Result State Mapping table for the CheckResult state field. This table covers both Service and Host states while the prior documentation string only covered Services.

This change is useful since there are different kinds of states for Hosts when using the Icinga 2 API. For one, there is a "normalized" version of 0 for UP and 1 for DOWN. Then there is the exit code version for 0/1 for UP and 2/3 for DOWN. Unfortunately, often this depends on the context and sometimes even intermingles.

To make it obvious which kind of state one can expect for a CheckResult object, I have linked to the already existing documentation section.

Reference the Check Result State Mapping table for the CheckResult state
field. This table covers both Service and Host states while the prior
documentation string only covered Services.

This change is useful since there are different kinds of states for
Hosts when using the Icinga 2 API. For one, there is a "normalized"
version of 0 for UP and 1 for DOWN. Then there is the exit code version
for 0/1 for UP and 2/3 for DOWN. Unfortunately, often this depends on
the context and sometimes even intermingles.

To make it obvious which kind of state one can expect for a CheckResult
object, I have linked to the already existing documentation section.
@oxzi oxzi added the area/documentation End-user or developer help label Apr 25, 2025
@cla-bot cla-bot bot added the cla/signed label Apr 25, 2025
@yhabteab yhabteab added this to the 2.15.0 milestone May 12, 2025
@yhabteab yhabteab merged commit a53792d into master May 12, 2025
26 checks passed
@yhabteab yhabteab deleted the doc-08-advanced-topics-checkresult-state-ref branch May 12, 2025 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation End-user or developer help cla/signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants