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

Improve daemon reloading #131

Merged
merged 14 commits into from
Sep 2, 2023
Merged

Improve daemon reloading #131

merged 14 commits into from
Sep 2, 2023

Conversation

Callum027
Copy link
Member

@Callum027 Callum027 commented Sep 2, 2023

  • Perform update job scheduling and configuration file watch registering as soon as the new configuration is loaded
  • Only update job schedule and configuration file watching when there are changes
  • Add a lock to the daemon to ensure that only one job thread is performing a run at a given time
  • Add error handling to the config watcher thread to ensure it stays running even when an unexpected error occurs

@Callum027 Callum027 added bug Something isn't working cli Issue or pull request related to the Buildarr CLI labels Sep 2, 2023
@Callum027 Callum027 added this to the v0.6.0 milestone Sep 2, 2023
@Callum027 Callum027 self-assigned this Sep 2, 2023
@Callum027 Callum027 merged commit 93e130c into main Sep 2, 2023
1 check passed
@Callum027 Callum027 deleted the 123-job-error-handling branch September 2, 2023 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cli Issue or pull request related to the Buildarr CLI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Buildarr daemon gets "stuck" whenever an exception occurs in a scheduled job thread
1 participant