-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integration crashes when iDRAC is unreachable during HA boot #17
Comments
Just to add, i've noticed as well that if a device becomes unreachable at some point, the particular device errors and then no longer updates. |
Hi @Breina, I installed your updated version and at least for me it seems like the error unfortunately is not fixed. When Home Assistant is started and the server is not reachable, the integration errors out: Log
When I then plug in the server, the sensors remain unavailable (I did wait for over an hour). If I restart Home Assistant, the integration is able to connect to iDRAC and the sensors show the data. I do not know (didn't test) if @cbrherms issue was fixed. I also removed and re-added the integration to see if it makes any difference, but the behavior did not change. |
Just to make sure we're on the same page, the integration needs to be set up when the server is active. It will then store stuff so that it can survive a restart if a server is unreachable. Subsequent restarts should be fine from then. On my phone right now, will check your logs when I get home. |
Been a while since I looked at this but I branched and made a few janky changes that fixed my issue with the dropouts. I'd have to go back through what I changed locally though and whether I pushed that all up to the dev branch I was storing it on. @Breina Admittedly can't remember how much testing i'd done with an idrac port being fully unreachable during homeasisstant startup (which i think is what @PocketMiner82 is saying), but i think this just needed the connection error catching and raising with ConfigEntryNotReady so that it'd try setting the platform up again once it became available cbrherms@d65bc3b ? While playing around with it I also added in a bunch of exception catching and also made use of setting the entity availability attribute when errors were raised which may be something to look in to? |
Those are some good clues, thanks for the info! |
I set up the integration when the server was reachable, yes. Edit: I just noticed that I wrote "setup" in the title, will update it 😅 |
Released this as 1.4.0. Thanks a lot PocketMiner! :) |
The integration fails setup when the server is fully powered off - I use a smart plug to cut the power to the server when it is unused. When Home Assistant starts when the server is powered down, the integration fails to setup and never displays the server status, even if the server gets powered on afterwards.
Log
Moreover, if I try to reload the integration when it failed to setup, this error gets thrown:
Log
So it is impossible to get the server iDRAC status when powering on the server after home assistant without completely restarting home assistant.
The text was updated successfully, but these errors were encountered: