launchpad-reviewers team mailing list archive

[Guruprasad <mp+456510@xxxxxxxxxxxxxxxxxx>]

Guruprasad has proposed merging ~lgp171188/launchpad:codehosting-charm-fix-cron-job-whitespace-typo into launchpad:master.

Commit message:
charm/codehosting: Fix a typo in the supermirror-pull cron job


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/456510
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:codehosting-charm-fix-cron-job-whitespace-typo into launchpad:master.

diff --git a/charm/launchpad-codehosting/templates/crontab.j2 b/charm/launchpad-codehosting/templates/crontab.j2
index b6ae48d..676d4ec 100644
--- a/charm/launchpad-codehosting/templates/crontab.j2
+++ b/charm/launchpad-codehosting/templates/crontab.j2
@@ -5,7 +5,7 @@ LPCONFIG=launchpad-codehosting
 
 {%- if active %}
 
-* * * * * {% if http_proxy %}http_proxy={{ http_proxy }} https_proxy={{ http_proxy }}{% endif %}{{ code_dir }}/cronscripts/supermirror-pull.py -q --log-file=INFO:{{ logs_dir }}/puller.log
+* * * * * {% if http_proxy %}http_proxy={{ http_proxy }} https_proxy={{ http_proxy }} {% endif %}{{ code_dir }}/cronscripts/supermirror-pull.py -q --log-file=INFO:{{ logs_dir }}/puller.log
 
 # remove from disk, deleted branches
 10 0 * * * {{ code_dir }}/cronscripts/process-job-source.py IReclaimBranchSpaceJobSource -q --log-file=DEBUG:{{ logs_dir }}/process-job-source.IReclaimBranchSpaceJobSource.log