search

bb-Ricardo / check_redfish

A monitoring/inventory plugin to check components and health status of systems which support Redfish. It will also create a inventory of all components of a system.
MIT License
113 stars 34 forks source link

Question: Is it possible to ignore not installed CPUs? #47

Closed log1-c closed 3 years ago

log1-c commented 3 years ago

Hi, (me again) I have a question regarding the sys-info check:

Is it possible to ignore a not installed CPU? Reason is, that the check is going to critical state because the 2nd CPU slot is "Enabled/Unknown": image

The server just has a single cpu, and my co-worker and me did not find an option in the iDrac settings to change the behavior (no "error" in IDrac either).

I helped my self and hacked a 

elif dell_sensor.get('HealthState') == "Unknown":
                            this_sensor_status = "OK"

into line 145 of check_redfish/cr_module/system_chassi.py, but I do not like it very much, because we could miss some other real errors like that.

Thanks again for the nice script!

bb-Ricardo commented 3 years ago

Can you test the "next-release" branch? It schould be fixed their.

log1-c commented 3 years ago

It is, message is not shown anymore! Thanks!