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

Create PID dir is not existent #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PierreF
Copy link

@PierreF PierreF commented Jun 22, 2017

For a daemon, the init.d script use a PID to track the process. Usually PID are stored in /run or /var/run.

In case of daemon running as non-root, the PID dir must be writable by the daemon user. The solution was to create a folder /var/run/daemon-name/ owned the the daemon user. The folder was created during installation (e.g. in the package .rpm or .deb).

But in most current Linux distribution (at least Ubuntu, CentOS and Fedora), /var/run and /run are tmpfs which is not persisted across reboot, so the /var/run/daemon-name/ need to be re-created.

This PR add the creation of the PID dir and changing ownership to user.

It will fix jmxtrans issue jmxtrans/jmxtrans#578

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.

1 participant