Spamming HA with entities

jeroenterheerdt / HAsmartirrigation

Smart Irrigation custom component for Home Assistant

https://jeroenterheerdt.github.io/HAsmartirrigation/

MIT License

361 stars 49 forks source link

Spamming HA with entities #441

Open jchristianj opened 3 months ago

jchristianj commented 3 months ago

What happened?

Each time the integration is reloaded, a new entitiy for each zone with suffix _n is created.

How to reproduce

Reload integration config_entry-smart_irrigation-c34d6b3034b69cc97400485090710263.json

Relevant log output

No response

Which version are you running?

v2024.7.5

Diagnostics file

[X] I have attached a diagnostics file

Additional information

No response

mistergyle commented 3 months ago

duplicates https://github.com/jeroenterheerdt/HAsmartirrigation/issues/372

jeroenterheerdt commented 3 months ago

Duplicate of #372

jeroenterheerdt commented 3 months ago

uninstall, reinstall should fix this.

jchristianj commented 3 months ago

Unfortunately not. Completely removed the device, removed the integration, removed the HACS integration, rebooted, started from scratch with v2024.7.6, needed to configure everything (zones, modules, sensor group, weather API) new, but it still produces the *_n entities upon each reload.

jeroenterheerdt commented 3 months ago

OK, thanks. I will need to do some more digging then on why this happens. Thanks.

jeroenterheerdt commented 3 months ago

I tried, I tried, and I tried again but cannot reproduce this on any environment I have :(

lukasz-burak commented 2 months ago

I do have the same issue, entities just duplicate every integration refresh.

MonsterEnergy-wtf commented 2 months ago

Me2. 3 zones, 57 Entities. More to come 😁

jeroenterheerdt commented 2 months ago

I see two potential issues that might be causing it. The add this sensor activity gets called twice per zone and somehow the name generator adds the extra N all the time. The weird part is I didn't change anything in this area recently. Are you seeing the original zone and zones with _N added? Or just zone_Ns?

jeroenterheerdt commented 2 months ago

Come to think of it, was it working correctly for you before? If so, what version worked normally?

MonsterEnergy-wtf commented 2 months ago

I saw the regular one without the extension _n and a lot of same with the extension. I endet up by removing the smart irrigation device ( not the integration) and added it again. The settings were kept and now everything seems to be fine. For now.

Not sure if this helps, but I played around with the experimental "Continuous updates for sensors" before I faced this issue. After my testing I realized that the multiple entities were created. Maybe this is an side effect of the experimental function?

jeroenterheerdt commented 2 months ago

OK, maybe thanks for letting me know you found a solution. The others in this thread, can you let me know if the above helps for you as well?

jchristianj commented 2 months ago

Sorry, for me it is still not working and each reload of the integration doubles up my entities. I followed the steps to remove the smart irrigation device (which removes the integration as well) and re-adding it, with a HA restart in between. Unfortunately, all my settings were not kept and I had to reconfigure the integration, starting with the OWM Api Key and re-configuring all zones and modules.

mistergyle commented 2 months ago

Same here

github-actions[bot] commented 9 hours ago

Stale issue message