Skip to content

Commit

Permalink
16776 FIX chrony: change default warning level
Browse files Browse the repository at this point in the history
SUP-19602

Change-Id: Iecb7f0db069522d68999c15bbc66cd5f267be8dd
  • Loading branch information
DavidGerva committed Aug 23, 2024
1 parent f79c7ec commit 7a48ee1
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 1 deletion.
19 changes: 19 additions & 0 deletions .werks/16776.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
[//]: # (werk v2)
# chrony: change default warning level

key | value
---------- | ---
date | 2024-08-19T06:14:01+00:00
version | 2.4.0b1
class | fix
edition | cre
component | checks
level | 1
compatible | no

Chrony's default `minpoll` and `maxpoll` values are 64 seconds and 1024 seconds.
In standard situations (good internet connection, low error etc)
the interval between polls should stick near the max value (1024 seconds).
(see `https://chrony-project.org/examples.html#_client_using_public_servers`)

Beacuse of that, the default WARN alert is now set to 1025 seconds.
2 changes: 1 addition & 1 deletion cmk/plugins/collection/agent_based/chrony.py
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ def check_chrony(params, section_chrony, section_ntp):
check_function=check_chrony,
check_default_parameters={
"ntp_levels": (10, 200.0, 500.0),
"alert_delay": (1800, 3600), # chronys default maxpoll is 10 (1024s)
"alert_delay": (1025, 3600), # chronys default maxpoll is 10 (1024s)
},
check_ruleset_name="ntp_time",
)
5 changes: 5 additions & 0 deletions cmk/plugins/collection/checkman/chrony
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,11 @@ description:
You can set levels for the maximum stratum and for the time offset
in milliseconds. The configuration is compatible with {ntp}.

The default {WARN} alert is set to 1025 seconds
(default chrony's {maxpoll} values is 1024 seconds,
see https://chrony-project.org/examples.html#_client_using_public_servers)
and thus differs from the default {WARN} alert of the {NTP} check.

When {chrony} is not found running, the check goes to {CRIT}.

discovery:
Expand Down

0 comments on commit 7a48ee1

Please sign in to comment.