Dzahn has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/370607 )
Change subject: phabricator: ensure /srv/dumps exists ...................................................................... phabricator: ensure /srv/dumps exists This directory wasn't puppetized and must have been manually created on iridium or something. It exists there but not on phab1001, and i noticed because there was cron spam to root from /srv/phab/tools/public_task_dump.py which said: IOError: [Errno 2] No such file or directory: '/srv/dumps/phabricator_public.dump' when trying to write a dump. Bug: T163938 Change-Id: I3247552f49d4e1e6d67ae78312d1f47e47ff0c96 --- M modules/phabricator/manifests/tools.pp 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: jenkins-bot: Verified Dzahn: Looks good to me, approved diff --git a/modules/phabricator/manifests/tools.pp b/modules/phabricator/manifests/tools.pp index 8bfb4f6..7733d21 100644 --- a/modules/phabricator/manifests/tools.pp +++ b/modules/phabricator/manifests/tools.pp @@ -32,6 +32,13 @@ mode => '0660', } + file { '/srv/dumps': + ensure => 'directory', + owner => 'root', + group => 'root', + mode => '0755', + } + $dump_script = "${directory}/public_task_dump.py" file { $dump_script: -- To view, visit https://gerrit.wikimedia.org/r/370607 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3247552f49d4e1e6d67ae78312d1f47e47ff0c96 Gerrit-PatchSet: 6 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: 20after4 <mmod...@wikimedia.org> Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits