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

Set FOREMAN_PROCESS_NAME env var for spawned procs #340

Merged
merged 3 commits into from
May 3, 2013
Merged

Set FOREMAN_PROCESS_NAME env var for spawned procs #340

merged 3 commits into from
May 3, 2013

Conversation

ldmosquera
Copy link
Contributor

This way, processes can identify themselves to metrics logging systems.

This is related to issue #162; this env var would be exactly like that PS env var that @RandomEtc mentioned, and in fact I use it for the exact same purpose as him (for naming Graphite metrics).

This way, processes can identify themselves to metrics logging systems.
@lucasefe
Copy link

+1

@ddollar
Copy link
Owner

ddollar commented Apr 14, 2013

If we can change this to be named PS instead, and have the value be in the format web.1 I'd be happy to merge this.

@ldmosquera
Copy link
Contributor Author

Changed to just PS; the value was already in that format.
Added a unit test to prove it :)

ddollar added a commit that referenced this pull request May 3, 2013
Set FOREMAN_PROCESS_NAME env var for spawned procs
@ddollar ddollar merged commit 619bd03 into ddollar:master May 3, 2013
@ddollar
Copy link
Owner

ddollar commented May 3, 2013

Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants