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

Change all files to be owned by root rather than cron owner #17

Merged
merged 1 commit into from
Jun 20, 2013

Conversation

mattyindustries
Copy link

  • If files are owned by a different user than root then cron
    displays error
    crond[2221]: (root) WRONG FILE OWNER

  - If files are owned by a different user than root then cron
    displays error
      crond[2221]: (root) WRONG FILE OWNER
@mattyindustries
Copy link
Author

My github fu bad. I was trying to create an issue and then attach a pull request to it but looks like you need to do it the other way.

These are the comments from Issue #16 which can be closed, merged or deleted.

If you implement a cron job with an owner other than root then the resulting file in /etc/cron.d gets owned by that user. Cron then complains that the file is owned by the wrong user and doesn't load the job.

Pull request to follow which forces all files to be owned by root and the "user" parameter is only used in the erb template

torrancew pushed a commit that referenced this pull request Jun 20, 2013
Change all files to be owned by root rather than cron owner
@torrancew torrancew merged commit c00d50d into torrancew:develop Jun 20, 2013
@torrancew
Copy link
Owner

I'll cut a new Forge release from this over the weekend, which will bring in the fix for #14 as well.

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.

2 participants