<p>It seems that the peer check in check_ntp_peers is backwards from what is useful in a production environment.</p>

<p>I have a server with 5 peers, and running the check with the following arguments yields the following results:</p>

<table>
<tr>
<th>Arguments</th>
<th>Results</th>
</tr>
<tr>
<td>-n 4</td>
<td>NTP CRITICAL</td>
</tr>
<tr>
<td>-n 5</td>
<td>NTP OK</td>
</tr>
<tr>
<td>-n 6</td>
<td>NTP OK</td>
</tr>
</table><p>That seems backwards from what I would expect. Ideally the logic should be reversed. I don't care if I have "more than" 5 peers. I care if I have less. Looking at the source seems to confirm that the peer count metrics are all handled exactly the same as the jitter and offset, both of which logically work with a greater than concept.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/nagios-plugins/nagios-plugins/issues/1204'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/_EHT0K2RMtzboqJPrl-P-OQj2eHdmJlOSEZ0c7oXP9Yr4drw_FxbkG-UYpxTcmYz.gif' height='1' width='1'></p>