[GH-ISSUE #59] How to stop integrated worker #44

Closed
opened 2026-02-25 21:31:04 +03:00 by kerem · 1 comment
Owner

Originally created by @mtonnie on GitHub (Aug 8, 2020).
Original GitHub issue: https://github.com/ciur/papermerge/issues/59

When I start the worker from manage.py worker and detach the process I'm not able to stop the worker.

There is no stop command and no possibility to write a pid file for the worker process.
I'm facing this issue when I try to create the start/stop mechanism for Synolog package #31 .

I have made a quick an dirty patch to pass the pidfile attributte, but I guess there are other more smarter solution.

Baybe to implement a worker start / workerr stop feature or to implement the pidfile as an optional command line argument.

001-worker_pidfile.txt

What's the prefered solution?

Originally created by @mtonnie on GitHub (Aug 8, 2020). Original GitHub issue: https://github.com/ciur/papermerge/issues/59 When I start the worker from manage.py worker and detach the process I'm not able to stop the worker. There is no stop command and no possibility to write a pid file for the worker process. I'm facing this issue when I try to create the start/stop mechanism for Synolog package #31 . I have made a quick an dirty patch to pass the pidfile attributte, but I guess there are other more smarter solution. Baybe to implement a worker start / workerr stop feature or to implement the pidfile as an optional command line argument. [001-worker_pidfile.txt](https://github.com/ciur/papermerge/files/5045417/001-worker_pidfile.txt) What's the prefered solution?
kerem 2026-02-25 21:31:04 +03:00
Author
Owner

@ciur commented on GitHub (Aug 8, 2020):

Good one ! Very well formulated issue 👍

start/stop commands should be part of external process management service like SystemD.
I would go with last option - pass optional pid file path as argument - if that works for you, for me it would be fine as well.
I would gladly accept your PR with that change. With only humble remark - argument should be optional.

Thank you ! Danke Danke!

<!-- gh-comment-id:670959932 --> @ciur commented on GitHub (Aug 8, 2020): Good one ! Very well formulated issue :+1: start/stop commands should be part of external process management service like SystemD. I would go with last option - pass optional pid file path as argument - if that works for you, for me it would be fine as well. I would gladly accept your PR with that change. With only humble remark - argument should be optional. Thank you ! Danke Danke!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/papermerge#44
No description provided.