Skip to content
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

Fix/updater update stuck in running status after failure #731

Conversation

schroda
Copy link
Collaborator

@schroda schroda commented Oct 24, 2023

In case the channel failed due to an exception, the update for the source failed completely.
This however was never handled and the pending updates for the source were never set to failed.
Due to this, the global updates running state was always true

@schroda schroda force-pushed the fix/updater_update_stuck_in_running_status_after_failure branch from 7c390ca to 9b7fb81 Compare October 24, 2023 20:06
In case the channel failed due to an exception, the update for the source failed completely.
This however was never handled and the pending updates for the source were never set to failed.
Due to this, the global updates running state was always true
@schroda schroda force-pushed the fix/updater_update_stuck_in_running_status_after_failure branch from 9b7fb81 to b718253 Compare October 29, 2023 17:33
@Syer10 Syer10 merged commit 9d2b098 into Suwayomi:master Oct 30, 2023
@schroda schroda deleted the fix/updater_update_stuck_in_running_status_after_failure branch October 31, 2023 15:39
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