domalab / ha-unraid

Unraid integration for Home Assistant
Apache License 2.0
11 stars 0 forks source link

UPS info command failed #2

Open tjorim opened 1 week ago

tjorim commented 1 week ago

Getting this error and sensor state is Unknown. I think we should mark this entity as unavailable instead.

This error originated from a custom integration.

Logger: custom_components.unraid.unraid
Source: custom_components/unraid/unraid.py:274
integration: UNRAID (documentation, issues)
First occurred: 10:00:38 (2 occurrences)
Last logged: 10:01:27

UPS info command failed
domalab commented 1 week ago

So you don't have UPS attached to your UNRAID server?

I can look at implementing an option to check for UPS. If connected then create sensor, if not connected do nothing.

tjorim commented 1 week ago

Indeed I don't. I disabled the sensor entity but still getting the error. I expected the update function to not be called when it's disabled.

domalab commented 6 days ago

Thanks for letting me know. I'll look into it

domalab commented 5 days ago

@tjorim I added an option to specify if you have a UPS connected or not to your server. I have tested it on my server and it's working. You also have the option now to later enable the UPS option if you do get one.