Bug 196797 - rc.subr fails to detect some python daemons
Summary: rc.subr fails to detect some python daemons
Status: Closed Not A Bug
Alias: None
Product: Base System
Classification: Unclassified
Component: conf (show other bugs)
Version: 10.0-STABLE
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-16 06:21 UTC by Jason Wolfe
Modified: 2015-01-22 19:03 UTC (History)
4 users (show)

See Also:


Attachments
rc.subr patch to detect new python proc format (473 bytes, patch)
2015-01-16 06:21 UTC, Jason Wolfe
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Wolfe 2015-01-16 06:21:56 UTC
Created attachment 151723 [details]
rc.subr patch to detect new python proc format

Salt for example causes a seemingly process odd format, possibly because of the way it invokes python:

python2.7: /usr/local/bin/python2.7 /usr/local/bin/salt-minion -c /usr/local/etc/salt -d (python2.7)

This format isn't picked up by rc.subr, so the daemon can't be manipulated.  A simple patch is attached.
Comment 1 Jason Wolfe 2015-01-16 16:59:18 UTC
A coworker of mine found that this behavior is caused by py-setproctitle, which salt wraps the load of in a 'try'.  At any rate, this means the proc title is the intended/actual that will be used, so this issue may start to affect others.

py27-salt-2014.7.0_1
py27-setproctitle-1.1.8
Comment 2 Jason Wolfe 2015-01-17 01:51:04 UTC
I think this is an idiosyncrasy caused by a unique setproctitle invocation, so it's likely better addressed in the python code to append to the proc name, instead of prepend 'python2.7: ' in this case.
Comment 3 Jason Wolfe 2015-01-22 19:03:37 UTC
Closing this. py-setproctitle is modifying the process name.  This is the default format used by setproctitle, but we likely don't want to chose the iterations in rc.subr.