Merge lp:~abentley/charms/trusty/juju-reports/disable-cloud-health-sync into lp:~juju-qa/charms/trusty/juju-reports/trunk

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 71
Proposed branch: lp:~abentley/charms/trusty/juju-reports/disable-cloud-health-sync
Merge into: lp:~juju-qa/charms/trusty/juju-reports/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
templates/crontab.tmpl (+1/-1)
To merge this branch: bzr merge lp:~abentley/charms/trusty/juju-reports/disable-cloud-health-sync
Reviewer Review Type Date Requested Status
John George (community) Approve
Review via email: mp+286925@code.launchpad.net

Commit message

Disable sync_health_results.

Description of the change

This branch disables the crontab entry for sync_health_results.

sync_health_results is using too much memory, but fixing it is not a high priority at this time, because the current cloud-health report is not what we need.

To post a comment you must log in.
Revision history for this message
John George (jog) wrote :

Thank you.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/crontab.tmpl'
2--- templates/crontab.tmpl 2015-09-25 19:01:03 +0000
3+++ templates/crontab.tmpl 2016-02-23 17:06:54 +0000
4@@ -4,7 +4,7 @@
5 INI=$reports_home/production.ini
6
7 */15 * * * * ubuntu flock -n $lockdir/charm_test.lock python $reports_home/jujureports/jobs/charm_test.py --log-file $logs/charm_test.log
8-*/30 * * * * ubuntu flock -n $lockdir/sync_health_results.lock python $reports_home/jujureports/jobs/sync_health_results.py --log-file $logs/sync_health_results.log
9+# */30 * * * * ubuntu flock -n $lockdir/sync_health_results.lock python $reports_home/jujureports/jobs/sync_health_results.py --log-file $logs/sync_health_results.log
10 */$interval * * * * ubuntu flock -n $lockdir/get_s3_ci_results.lock get-s3-ci-results --email-logs --log-file $logs/get-s3-ci-results.log
11 # The weekday below should match the value scorecard.collection_day
12 # in the file production.ini in lp:juju-reports.

Subscribers

People subscribed via source and target branches