Skip to content

Restart task queue if it has crashed (daemon only)#1472

Merged
cvium merged 3 commits intoFlexget:developfrom
cvium:fix_task_queue_crash
Nov 2, 2016
Merged

Restart task queue if it has crashed (daemon only)#1472
cvium merged 3 commits intoFlexget:developfrom
cvium:fix_task_queue_crash

Conversation

@cvium
Copy link
Copy Markdown
Contributor

@cvium cvium commented Oct 29, 2016

Motivation for changes:

Fix stuff

Detailed changes:

  • Restart task queue if is_alive() returns False when executing a task in daemon

Addressed issues:

@cvium
Copy link
Copy Markdown
Contributor Author

cvium commented Oct 29, 2016

It doesn't fix the underlying issue that causes the task queue crash, but it at least makes the daemon not entirely useless.

Thoughts @gazpachoking?

@gazpachoking
Copy link
Copy Markdown
Member

This seems pretty reasonable. I think maybe we should add a high level log message though that something has gone wrong when restarting it though.

@cvium cvium merged commit a1d693d into Flexget:develop Nov 2, 2016
@cvium cvium deleted the fix_task_queue_crash branch November 14, 2016 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants