launchpad-reviewers team mailing list archive

[Ines Almeida <mp+462433@xxxxxxxxxxxxxxxxxx>]

Ines Almeida has proposed merging ~ines-almeida/launchpad:update-ftpmaster-generate-content-cron-start into launchpad:master.

Commit message:
charm: update ftpmaster-publisher genrate-content cron script to run earlier
    
This is relevant because currently this job runs during most Launchpad developer's working hours, and can block DB deployments.
Updating the start time of the script to start daily at 9pm instead of 4am


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/462433

This change was manually applied in production last week.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ines-almeida/launchpad:update-ftpmaster-generate-content-cron-start into launchpad:master.

diff --git a/charm/launchpad-ftpmaster-publisher/templates/crontab.j2 b/charm/launchpad-ftpmaster-publisher/templates/crontab.j2
index 87b4237..2cb676c 100644
--- a/charm/launchpad-ftpmaster-publisher/templates/crontab.j2
+++ b/charm/launchpad-ftpmaster-publisher/templates/crontab.j2
@@ -5,7 +5,7 @@ LPCONFIG=launchpad-ftpmaster-publisher
 
 {% if active -%}
 03-58/5 * * * * umask 022; {{ code_dir }}/cronscripts/publish-ftpmaster.py -v -d ubuntu >> {{ logs_dir }}/publish-ftpmaster.log 2>&1
-02 4 * * * {{ code_dir }}/cronscripts/generate-contents-files.py -v --distribution=ubuntu >> {{ logs_dir }}/generate-contents-files.log 2>&1
+02 21 * * * {{ code_dir }}/cronscripts/generate-contents-files.py -v --distribution=ubuntu >> {{ logs_dir }}/generate-contents-files.log 2>&1
 
 # cprov 2008-02-07: parallel run of death-row, at :10 it will run simultaneously with apt-ftparchive.
 10 */6 * * * {{ code_dir }}/scripts/process-death-row.py -d ubuntu -q --log-file=DEBUG:{{ logs_dir }}/process-death-row.log