Merge lp:~cprov/charms/trusty/tanuki-spec-manager/cron-logfile-perms into lp:~tanuki/charms/trusty/tanuki-spec-manager/trunk

Proposed by Celso Providelo
Status: Merged
Approved by: Celso Providelo
Approved revision: 27
Merged at revision: 27
Proposed branch: lp:~cprov/charms/trusty/tanuki-spec-manager/cron-logfile-perms
Merge into: lp:~tanuki/charms/trusty/tanuki-spec-manager/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
hooks/actions.py (+1/-1)
To merge this branch: bzr merge lp:~cprov/charms/trusty/tanuki-spec-manager/cron-logfile-perms
Reviewer Review Type Date Requested Status
Thomi Richards (community) Approve
Review via email: mp+271424@code.launchpad.net

Commit message

Fixing cron logfile perms.

Description of the change

Fixing cron logfile perms.

To post a comment you must log in.
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/actions.py'
--- hooks/actions.py 2015-09-17 04:17:20 +0000
+++ hooks/actions.py 2015-09-17 05:21:50 +0000
@@ -331,7 +331,7 @@
331 log_path = os.path.join(LOG_DIR, 'store-poller.log')331 log_path = os.path.join(LOG_DIR, 'store-poller.log')
332 # Create the file now, so the nagios check doesn't fail right after332 # Create the file now, so the nagios check doesn't fail right after
333 # we've completed the deployment.333 # we've completed the deployment.
334 host.write_file(log_path,"", "www-data", "www-data")334 host.write_file(log_path,"", "www-data", "www-data", perms=0o660)
335 max_age_seconds = int(config['store_poller_nagios_file_age_max_minutes']) * 60335 max_age_seconds = int(config['store_poller_nagios_file_age_max_minutes']) * 60
336 nrpe_compat.add_check(336 nrpe_compat.add_check(
337 shortname="store_poller_check",337 shortname="store_poller_check",

Subscribers

People subscribed via source and target branches

to all changes: