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

Restart Handling #8

Open
arlaneenalra opened this issue Dec 24, 2021 · 0 comments
Open

Restart Handling #8

arlaneenalra opened this issue Dec 24, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@arlaneenalra
Copy link
Member

arlaneenalra commented Dec 24, 2021

Watches have a built in timeout that causes them to eventually expire and disconnect. This leads to conditions similar to #6. To work around this, we have a restart loop in the code and a Restart=always policy in the systemd unit right now. (Of course, that unit file is not presently part of this code base.) The real fix here is to define configurable restart handling in the service itself.

Here's a proposed approach:

  • Add a configurable reconnect loop that supports max number of restarts, including unlimited.
  • Add the systemd unit file to the code base to be packaged in the dist tarball.

It might make sense to setup deb/rpm packaging for this. (Packaging is covered in #9)

@arlaneenalra arlaneenalra added the enhancement New feature or request label Dec 24, 2021
@arlaneenalra arlaneenalra added this to the producitzation milestone Dec 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant