search

rorr73 / LifeSOSpy_MQTT

MQTT Client to communicate with LifeSOS alarm systems.
MIT License
5 stars 2 forks source link

Shutdown might take over 30 seconds. #8

Closed bratanon closed 5 years ago

bratanon commented 5 years ago

I have noticed a strange behavior when the app is shutting down. It can take over 30 seconds for it to do so.

I have debugged as good as I can, and from what I can tell there is something in the _translator::asyncloop that causes this.

The shutdown signal is catched, and signal_shutdown executed, but then it might take some time until you receive a asyncio.CancelledError in the async_loop. Not sure why this might be that time consuming, but this is causing problem running this in a docker container as docker will send SIGKILL after 10 secounds if the container is not stopped or killed.

I would really appreciate if you could find some time and take a look at this.

rorr73 commented 5 years ago

Thanks for finding this issue and listing your findings.

It appears the Queue does not process the cancellation until a message is next received. I've added a fix to the next release that should ensure cancellation occurs immediately.

Please let me know if there is still an issue after updating.