From: Peter Palfrader <peter@palfrader.org>
Date: Mon, 16 Sep 2019 17:03:42 +0000 (+0200)
Subject: Remove setting env vars in the static_mirror part of puppet-cron
X-Git-Url: https://git.adam-barratt.org.uk/?a=commitdiff_plain;ds=sidebyside;h=cff4dc159fd0c6d4af80edd333f3f6ca4f2f0088;p=mirror%2Fdsa-puppet.git

Remove setting env vars in the static_mirror part of puppet-cron

This was re-setting MAILTO=root, which is already set at the top level,
and it was also setting PATH to only a subset of what it was before
(dropped the sbins).
---

diff --git a/modules/staticsync/manifests/static_mirror.pp b/modules/staticsync/manifests/static_mirror.pp
index bcec3110e..3f6a8d899 100644
--- a/modules/staticsync/manifests/static_mirror.pp
+++ b/modules/staticsync/manifests/static_mirror.pp
@@ -31,8 +31,6 @@ class staticsync::static_mirror (
   concat::fragment { 'puppet-crontab--static-mirror':
     target => '/etc/cron.d/puppet-crontab',
     content  => @("EOF"),
-        MAILTO=root
-        PATH=/usr/local/bin:/usr/bin:/bin
         @reboot ${staticsync::user} sleep 60; chronic static-mirror-run-all
         | EOF
   }