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

Remove PID file error message #6915

Merged
merged 1 commit into from Aug 28, 2019

Conversation

@davidMcneil
Copy link
Contributor

commented Aug 28, 2019

Remove the error message when a PID file is not found. We use read_pid to check if a service is running. If it is not running the PID file will not be found. This always happens at startup and is not a true error.

Signed-off-by: David McNeil mcneil.david2@gmail.com

Signed-off-by: David McNeil <mcneil.david2@gmail.com>
@chef-expeditor

This comment has been minimized.

Copy link

commented Aug 28, 2019

Hello davidMcneil! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@davidMcneil davidMcneil self-assigned this Aug 28, 2019
@davidMcneil davidMcneil merged commit fb947ac into master Aug 28, 2019
5 checks passed
5 checks passed
DCO This commit has a DCO Signed-off-by
Details
buildkite/habitat-sh-habitat-master-verify Build #3261 passed (28 minutes, 51 seconds)
Details
buildkite/habitat-sh-habitat-master-website Build #382 passed (42 seconds)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
expeditor/config-validation Validated your Expeditor config file
Details
@chef-ci chef-ci deleted the dmcneil/remove-pid-file-error-message branch Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.