Ottomata has submitted this change and it was merged.

Change subject: Ensure log file for mobile data job exists
......................................................................


Ensure log file for mobile data job exists

The mobile data job's log file's specification failed to ensure that
the log file is present. Hence, the log file would have been created
by the cron job. That however, would fail due to /var/log's
permissions. Hence, we now ensure that the file is present beforehand,
so puppet gets a chance to apply ownerships to it.

Bug: 66600
Change-Id: I7ea917ce58f6673aeb1589b092dc7a029b02fca5
---
M manifests/misc/statistics.pp
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Ottomata: Verified; Looks good to me, approved



diff --git a/manifests/misc/statistics.pp b/manifests/misc/statistics.pp
index 835475a..5746803 100644
--- a/manifests/misc/statistics.pp
+++ b/manifests/misc/statistics.pp
@@ -675,6 +675,7 @@
         owner   => $user,
         group   => $user,
         mode    => '0660',
+        ensure  => present,
     }
 
     file { $mysql_credentials:

-- 
To view, visit https://gerrit.wikimedia.org/r/139785
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7ea917ce58f6673aeb1589b092dc7a029b02fca5
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: QChris <[email protected]>
Gerrit-Reviewer: Matanya <[email protected]>
Gerrit-Reviewer: Milimetric <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: QChris <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to