Last modified: 2013-12-03 20:23:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T59598, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 57598 - Parsoid js/api/server.js should register main server in a pid file
Parsoid js/api/server.js should register main server in a pid file
Status: RESOLVED WONTFIX
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks: 57599
  Show dependency tree / graph
 
Reported: 2013-11-26 16:29 UTC by Antoine "hashar" Musso (WMF)
Modified: 2013-12-03 20:23 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Antoine "hashar" Musso (WMF) 2013-11-26 16:29:18 UTC 
It would be very nice to have the ability to register the pid of js/api/server.js via a pidfile.  start-stop-daemon has -p|--pidfile parameter pointing to a file such as /var/run/parsoid.pid

You would have to trap unexpected nodejs crash of the main process which should delete the pidfile when node is shutdowning.
Comment 1 Gerrit Notification Bot 2013-11-27 21:29:19 UTC 
Change 98004 had a related patch set uploaded by Arlolra:
Register a pidfile for api/server.js

https://gerrit.wikimedia.org/r/98004
Comment 2 Arlo Breault 2013-12-03 20:23:18 UTC 
It was suggested to use --make-pidfile
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links