Page MenuHomePhabricator

Icinga SMART check returns OK when not getting data
Closed, ResolvedPublic

Description

The SMART Icinga check "Device not healthy -SMART-" returns OK and is green even when the actual message is "(No output returned from plugin)".

See screenshot.

device-not-healthy-but-ok.png (116×1 px, 32 KB)

It should not be OK when it gets nothing from plugin.

Should be UNKNOWN instead i would say.

Event Timeline

Dzahn triaged this task as Medium priority.May 3 2018, 8:08 PM
Dzahn updated the task description. (Show Details)
Dzahn added a subscriber: fgiunchedi.

That's the current behavior of the check, i.e. when things are ok exit 0 and no output. We can change it to print "OK" or sth similar, and the values/thresholds perhaps

Change 430898 had a related patch set uploaded (by Filippo Giunchedi; owner: Filippo Giunchedi):
[operations/puppet@production] prometheus_check_metric: print message when status is OK

https://gerrit.wikimedia.org/r/430898

The screenshot above is from a time when a host being reinstalled and every other check on the host was red. Is it possible that it was actually ok during this time ?

The screenshot above is from a time when a host being reinstalled and every other check on the host was red. Is it possible that it was actually ok during this time ?

Yeah in this case it was OK but no output

Change 430898 merged by Dzahn:
[operations/puppet@production] prometheus_check_metric: print message when status is OK

https://gerrit.wikimedia.org/r/430898

Dzahn claimed this task.

Thanks! merged and confirmed working. It shows the text on Icinga.

Vvjjkkii renamed this task from Icinga SMART check returns OK when not getting data to 6ndaaaaaaa.Jul 1 2018, 1:12 AM
Vvjjkkii reopened this task as Open.
Vvjjkkii removed fgiunchedi as the assignee of this task.
Vvjjkkii raised the priority of this task from Medium to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot renamed this task from 6ndaaaaaaa to Icinga SMART check returns OK when not getting data.Jul 2 2018, 2:08 PM
CommunityTechBot closed this task as Resolved.
CommunityTechBot assigned this task to fgiunchedi.
CommunityTechBot lowered the priority of this task from High to Medium.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added subscribers: gerritbot, Aklapper.