-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Handle TooManyExecutions when more than 10 executions are running #261
Comments
@tetienne, @vlebourl what do you think:
Any other suggestions? |
I am currently pursuing option 1 via iMicknl/python-overkiz-api#48. This is the easiest to implement, however it has the following limitations:
|
I have 14 covers. What can i do to close/open them all at the same time? i getting exactly the same error :( |
@kaczkalolo do you encounter this problem for a while, or is this a new issue? There was another report yesterday (#378). For now, I would advice to add a delay in your automations in Home Assistant... This will be way more reliable, since this is an issue on the Somfy TaHoma side. |
I had this for a while but never really had time to debug but finally found
it. I don't use automation for that- only group.
|
Hi, I have the same problem but for me there are no 10 long running operations necessary. At 06:00 rollers at the front-side of the house are openend. This goes always good without errors and always exactly at 06:00. At 07:00 my rollers at the side of the house are planned to open. This goes almost every time wrong. I get this error:
The rollers that are opened at 06:00 is done by Node-RED by openening a group: `cover:
The rollers that are opened at 07:00 (which has the problems) is done by Node-RED by openening a group: `cover:
` The rollers are openend a minute or 2 after the scheduled time. And also they are NOT openend at the same time. So you see roller 1 is opening, after that number 2 and then number 3. |
Hi, I started to see this behavior for 13 shutters a few days back.
I can't even login to https://www.tahomalink.com/ as I get the same error:
in a pop up message box. I'm using Google Assistant to open close all the shutters at once. Not sure how to introduce delay on my side. |
Hi, I don't think this error is linked to the first one. This one happens when you try to open too many connections to the API in a short period of time. If I'm not mistaken, HA keeps its connection open to reuse it for successive actions. Waiting 10 minutes before retrying to connect is the only fix I know for that one. |
Hi @vlebourl. Thank you for clarification and sorry to mixed up issues. |
Is there a way to have an event if this kind of error happens (like in #573)? This exception catched here: ha-tahoma/custom_components/tahoma/executor.py Lines 65 to 75 in 47c92c6
There is only a log message but can't handle this error from outside (eg: from an automation). |
@laszlojakab Can you try the master branch? I've added event in case of failure. More information here. |
@tetienne Sorry not exactly the same error as in the subject of the issue. I get this error:
No event for this kind of error in master branch. As I mentioned this error logged by line 74 in executor.py |
@laszlojakab I was able to reproduce your error spamming a script where I open and close many times a cover. It seems an event is well triggered when a command is not correctly send:
Or
You can see these logs if you enabled the debug log. |
@tetienne perhaps his case will broadcast |
I’m sure if the events are linked. But, we can forward more events if needed indeed. |
Besides the event generated, is there any intention to have a proper fix to this issue ? |
@rvrignaud That’s not something we can fix easily, if feasible. It means we will have to handle a kind of queue, or something similar. In your case, why not use scenario created on the tahomalink.com side? |
Because I'm using Google Assistant voice queries that are triggering native devices command (for 13 devices for now, soon much more) and not a specific routine. Like "OKG close all the shutters." |
@tetienne I wonder why the backoff is not working. Perhaps we need to retry for a longer time. I am not a huge fan of building a queue in the Data Update Coordinator... |
@iMicknl Probably yes, we can add some delay to check if it helps. |
If you need any help in testing new behavior, I can consistently reproduce this one. Happy to help |
Any updates on that issue? Can we get an event (like the errors in |
I'm having the same issue when trying to control 15 blinds simultaneously when using the Overkiz integration. Does anybody know what would be the required delay between the calls for Overkiz to accept the requests? (I mean if i split the calls in 10 / 5) calls for example. |
Same story here trying to control 14 Somfy blinds at once with Overkiz. All I could do was split them in 7 and 7... |
Hi again. The problem still exists. I know the repo is deprecated but I don't want to open a new issue in core for the same error. As I see Could you please modify the core version of the integartion to raise |
@laszlojakab would be good to raise this on core repo! Happy to see if we can implement it. |
Added to core. |
Describe the bug
Originally reported on Home Assistant Community
Not all executions will succeed, when more than 10 executions are running at the same time.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
All executions running in a sequence, with a pause.
Environment (please complete the following information):
Additional context
The text was updated successfully, but these errors were encountered: