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

Proposed by Celso Providelo on 2015-09-17
Status: Merged
Approved by: Celso Providelo on 2015-09-17
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) 2015-09-17 Approve on 2015-09-17
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.
Thomi Richards (thomir) wrote :

LGTM

review: Approve

Preview Diff

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

Subscribers

People subscribed via source and target branches

to all changes: