{"action":"created","issue":{"url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/207","labels_url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/207/labels{/name}","comments_url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/207/comments","events_url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/207/events","html_url":"https://github.com/puppet-community/puppet-mcollective/issues/207","id":68429919,"number":207,"title":"factsource crontab PATH conflicting with other cronjobs","user":{"login":"neoice","id":71924,"avatar_url":"https://avatars.githubusercontent.com/u/71924?v=3","gravatar_id":"","url":"https://api.github.com/users/neoice","html_url":"https://github.com/neoice","followers_url":"https://api.github.com/users/neoice/followers","following_url":"https://api.github.com/users/neoice/following{/other_user}","gists_url":"https://api.github.com/users/neoice/gists{/gist_id}","starred_url":"https://api.github.com/users/neoice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/neoice/subscriptions","organizations_url":"https://api.github.com/users/neoice/orgs","repos_url":"https://api.github.com/users/neoice/repos","events_url":"https://api.github.com/users/neoice/events{/privacy}","received_events_url":"https://api.github.com/users/neoice/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-14T16:22:09Z","updated_at":"2015-04-14T16:29:22Z","closed_at":null,"body":"https://github.com/puppet-community/puppet-mcollective/blob/master/manifests/server/config/factsource/yaml.pp#L20\r\n\r\napparently, ENV variables set in `crontab` are **global** for all cronjobs. I actually run puppet via cron (https://github.com/theforeman/puppet-puppet/blob/master/manifests/agent/service.pp#L38)\r\n\r\nnow every Puppet run bloats my PATH variable a little more until it looks like this:\r\n\r\n```X-Cron-Env: <PATH=/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin:/opt/puppet/bin...```"},"comment":{"url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/comments/92956528","html_url":"https://github.com/puppet-community/puppet-mcollective/issues/207#issuecomment-92956528","issue_url":"https://api.github.com/repos/puppet-community/puppet-mcollective/issues/207","id":92956528,"user":{"login":"neoice","id":71924,"avatar_url":"https://avatars.githubusercontent.com/u/71924?v=3","gravatar_id":"","url":"https://api.github.com/users/neoice","html_url":"https://github.com/neoice","followers_url":"https://api.github.com/users/neoice/followers","following_url":"https://api.github.com/users/neoice/following{/other_user}","gists_url":"https://api.github.com/users/neoice/gists{/gist_id}","starred_url":"https://api.github.com/users/neoice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/neoice/subscriptions","organizations_url":"https://api.github.com/users/neoice/orgs","repos_url":"https://api.github.com/users/neoice/repos","events_url":"https://api.github.com/users/neoice/events{/privacy}","received_events_url":"https://api.github.com/users/neoice/received_events","type":"User","site_admin":false},"created_at":"2015-04-14T16:29:22Z","updated_at":"2015-04-14T16:29:22Z","body":"I've always wanted native cron.d support in Puppet, now I have another technical reason for *why* I think cron.d is superior to `crontab -e`. I'm personally going to look into using https://forge.puppetlabs.com/torrancew/cron for my in-house modules that use `cron` declarations. I can't have 1 job declaration polluting another job declaration, it destroys the composability of Puppet modules."}}