-
Notifications
You must be signed in to change notification settings - Fork 1
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
Task timed out after 5.01 seconds #48
Comments
The problem in this case is at MyQ API level. I've been waiting for a fix from PyMyQ. One's been proposed in this PR, but the maintainer of HomeAsistant has suggested that the fix is incorrect and that the problem may be with the MyQ infrastructure. He wants to wait a few days to see if MyQ fixes the problem. In @hjdhjd's words:
If there is a fix in the library, I'll update the code here to use it. In the meantime, I updated the README to provide a little more help when a test fails. I didn't intend for the HelpIntent to require authentication. It's on my list to change. Here's the log that shows the (unresolved) problem:
Leaving this open until the API problem gets resolved. |
Thanks for the prompt response. We'll wait and see.... |
For what it's worth...I've seen these issues largely be resolved as of the end of the weekend and I, myself, haven't had an issue since Saturday morning. @rjcohn You might want to take a look at my code for how I've implemented an optional region designation as a workaround for these infrastructure challenges when they occur, but as I said elsewhere: I wouldn't permanently select a region and trust the load balancer to do the right thing (unless it's broken). The regions (east and west) are geographic but that doesn't necessarily correlate to what one person should pick (e.g. an east-coast person shouldn't necessarily pick east). The logic I will be implementing in the future (I want a little distance from this event before making additional changes) is essentially:
That should increase the robustness of this API for everyone down the line. I'll be releasing an update to do that in the next week or so, but I prefer to reduce the number of variables changing at once given the myQ infrastructure hiccups of the past few days. |
I've done some more testing (including updating to the current pymyq). The tests in test_lambda.py all succeed locally, while the AWS Lambda tests (and the actual skill) fail. The failure is in the initial auth call. I've tried running exactly a test_lambda.py test in AWS, and it fails. Then I tried running example.py, and it behaves the same -- succeeds locally, fails in AWS. |
I run the HelpIntent test and the TestAllStates Test, and they both fail with the message "Task timed out after 5.01 seconds"
You state to "make sure you have set up the environment variables correctly" - I followed all previous steps under "Instructions", so I don't know what else to inspect.
The text was updated successfully, but these errors were encountered: