search

home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.73k stars 29.99k forks source link

Ondilo Ico - Unable to retrieve sensor information #105238

Closed StewartM30 closed 6 months ago

StewartM30 commented 9 months ago

The problem

It appears that as of today I'm unable to pull back any of the sensor information from Ondilo. It may be a coincidence but I did upgrade to HA 12.0 yesterday. All entities under the integration just say 'unavailable'

What version of Home Assistant Core has the issue?

core-2023.12.0

What was the last working version of Home Assistant Core?

core-2023.11.0

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Ondilo ICO

Link to integration documentation on our website

https://www.home-assistant.io/integrations/ondilo_ico/

Diagnostics information

home-assistant_ondilo_ico_2023-12-07T16-34-33.894Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.sensor
Source: helpers/entity_platform.py:361
Integration: Sensor (documentation, issues)
First occurred: 13:22:55 (2 occurrences)
Last logged: 16:34:22

Error while setting up ondilo_ico platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 361, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/src/homeassistant/homeassistant/components/ondilo_ico/sensor.py", line 122, in async_setup_entry
    for poolidx, pool in enumerate(coordinator.data):
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: 'NoneType' object is not iterable

Logger: homeassistant.components.ondilo_ico.sensor
Source: helpers/update_coordinator.py:313
Integration: Ondilo ICO (documentation, issues)
First occurred: 11:41:08 (3 occurrences)
Last logged: 16:34:22

Error requesting sensor data: HTTPSConnectionPool(host='interop.ondilo.com', port=443): Max retries exceeded with url: /api/customer/v1/pools (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:1006)')))
Error requesting sensor data: Cannot connect to host interop.ondilo.com:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:1006)')]

Additional information

No response

chheiss commented 4 months ago

Same issue here, still on 2024.4. No sensors available, all gone since yesterday.

sentience commented 4 months ago

Same, this issue is back. @JeromeHXP can you take another look?

andy-ich commented 4 months ago

Same issue here, still on 2024.4. No sensors available, since yesterday.

unbreakmat commented 4 months ago

Sale issue 2024.5

Loic691 commented 4 months ago

Issue is not solved in HA 2025.5.1

Pyradad commented 4 months ago

Working again for me now. Thank you

On Sat, 4 May 2024, 06:28 Loic69, @.***> wrote:

Issue is not solved in HA 2025.5.1

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/105238#issuecomment-2094003450, or unsubscribe https://github.com/notifications/unsubscribe-auth/BHHYV5CHA7QR6SUGUBQUXFLZARPWBAVCNFSM6AAAAABALLIY76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUGAYDGNBVGA . You are receiving this because you commented.Message ID: @.***>

felipe-colinas commented 4 months ago

Stopped working on may 1st at 22:44. Back online this morning. Needs to restart integration.

davidmarli commented 4 months ago

Worked again this morning but now it's unavaible again !!!

Pyradad commented 4 months ago

Yes..spoke too fast...

On Mon, 6 May 2024, 20:47 davidmarli, @.***> wrote:

Worked again this morning but now it's unavaible again !!!

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/105238#issuecomment-2096685696, or unsubscribe https://github.com/notifications/unsubscribe-auth/BHHYV5EENBRDHJBCWBSVCNTZA7F3LAVCNFSM6AAAAABALLIY76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGY4DKNRZGY . You are receiving this because you commented.Message ID: @.***>

davidmarli commented 4 months ago

And it works again now !!!

What's going on ?

Inrego commented 4 months ago

Sounds like an issue on the ICO cloud servers

parislyon69 commented 4 months ago

it works and then it does not, i keeps going in and out during the day. maybe an issue with the number of api requests allowed during an hour? Just guessing I really have no idea besides that it does not seem stable.

sentience commented 4 months ago

I've opened a fresh email query with Ondilo product support. Hopefully they will let us know what's going on.

paazevedo commented 4 months ago

The integration, is working again (at least for me), but this last month's inconsistency made me change to my own costum_integration to control the requests myself. Aligned with @parislyon69 and reading the documentation, the number of requests allowed is not being respected. Sometimes it works other not, this may be due to maintenance or related with the recent changes they did on the API, but in one case or other this should be better communicated to end-users. One way or the other, it's becoming really difficult to use the product in the way we so much need (open one). On top of that, I'm truly worried with this last fmw update, which consumed around 10% of battery in just 2 days! @sentience let us know if there is any feedback coming from them. I'll probably open one myself as well.

Loic691 commented 4 months ago

A PR is done on another thread to souvent this issue. We have to wait the next release...

paazevedo commented 4 months ago

A PR is done on another thread to souvent this issue. We have to wait the next release...

Which one can you share ? I would love to see what the changes are...

Loic691 commented 4 months ago

For the battery with new firmware.and This is the first time I charged the ICO... I switched from 98% (full charge) to 88% in one month. Before the first charge, one year without charging including winter.

I had a problem of last ICO with the battery which after the first charge is going from 100% to 0% in one month... they changed me the ICO sensor !

paazevedo commented 4 months ago

For the battery with new firmware.and This is the first time I charged the ICO... I switched from 98% (full charge) to 88% in one month. Before the first charge, one year without charging...

I had a problem of last ICO with the battery which after the first charge is going from 100% to 0% in one month... they changed me the ICO sensor ! I lost 10% in one day... image

parislyon69 commented 4 months ago

The integration, is working again (at least for me), but this last month's inconsistency made me change to my own costum_integration to control the requests myself. Aligned with @parislyon69 and reading the documentation, the number of requests allowed is not being respected. Sometimes it works other not, this may be due to maintenance or related with the recent changes they did on the API, but in one case or other this should be better communicated to end-users. One way or the other, it's becoming really difficult to use the product in the way we so much need (open one). On top of that, I'm truly worried with this last fmw update, which consumed around 10% of battery in just 2 days! @sentience let us know if there is any feedback coming from them. I'll probably open one myself as well.

parislyon69 commented 4 months ago

@paazevedo in their doc. They mention the following...

Quotas We remind that ICO takes measures every hour.

In order to avoid excessive load of our servers, the requests to the Ondilo Customer API are limited to the following per user quotas :

5 requests per second 30 requests per hour

I hope this helps

paazevedo commented 4 months ago

@paazevedo in their doc. They mention the following...

Quotas We remind that ICO takes measures every hour.

In order to avoid excessive load of our servers, the requests to the Ondilo Customer API are limited to the following per user quotas :

5 requests per second 30 requests per hour

I hope this helps

that's not the problem...I'm aware of their documentation and the amount of requests allowed. I'm speaking about 1 week everything is working flawlessly and for instance this last week, it's not.

Loic691 commented 4 months ago

A PR is done on another thread to souvent this issue. We have to wait the next release...

Which one can you share ? I would love to see what the changes are...

Hi all Here's the other thread ;-) https://github.com/home-assistant/core/issues/116593#issuecomment-2097702280

sentience commented 4 months ago

From Ondilo support in response to my ticket:

Dear user, 

Thank you for contacting us, we are sorry you’re experiencing this issue.

Please note that The Ondilo ICO API  issue has been resolved.

Please verify.

Awaiting your reply,

Kind regards,  The Ondilo Team

I guess their resolution is dependent on the throttling changes that have been merged but not yet released in HA according to #116593.

Pyradad commented 4 months ago

When is the release date?

On Tue, 14 May 2024, 13:23 Kevin Yank, @.***> wrote:

From Ondilo support in response to my ticket:

Dear user,

Thank you for contacting us, we are sorry you’re experiencing this issue.

Please note that The Ondilo ICO API issue has been resolved.

Please verify.

Awaiting your reply,

Kind regards, The Ondilo Team

I guess their resolution is dependent on the throttling changes that have been merged but not yet released in HA according to #116593 https://github.com/home-assistant/core/issues/116593.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/105238#issuecomment-2109951111, or unsubscribe https://github.com/notifications/unsubscribe-auth/BHHYV5A5G6JPCKMARIVSUV3ZCHX4JAVCNFSM6AAAAABALLIY76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBZHE2TCMJRGE . You are receiving this because you commented.Message ID: @.***>