A commented cron job reminds admins of its existence and potential use. It can be easily enabled automatically with a sed statement in a Docker file or so. This patch deals with archive_purchase_suggestions.pl (which came with 20.05)
Created attachment 178969 [details] [review] Bug 39250: Add archive_purchase_suggestions.pl to cron.daily commented It will be a commented line in the debian template and added also in the crontab.example file. As mentioned on BZ, the comment can easily be removed automatically during Docker builds or startups etc. Test plan: Just read the patch. Nothing changes here actually. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Actually nothing to test. Pushing it a level further.
Created attachment 179068 [details] [review] Bug 39250: Add archive_purchase_suggestions.pl to cron.daily commented It will be a commented line in the debian template and added also in the crontab.example file. As mentioned on BZ, the comment can easily be removed automatically during Docker builds or startups etc. Test plan: Just read the patch. Nothing changes here actually. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Trivial improvement, no regressions, no tests required, no qa script failures.. Passing QA
Pushed for 25.05! Well done everyone, thank you!