Download & Extend

wget calls cron.php up to 20 times in a row

Project:Drupal core
Version:4.7.x-dev
Component:install system
Category:bug report
Priority:normal
Assigned:alex_b
Status:closed (fixed)

Issue Summary

If cron.php does not return because of a script timeout, wget calls by default the site up to 20 times total until it gets a valid response.

When cron times out, the 20 times default-retry mostly aggravates things. A typical symptom of those retries are watchdog messages from cron in an unexpected ~ 1 minute interval.

This tiny patch adds the -t flag to the suggested wget call in INSTALL.txt to make sure that wget only calls cron.php once in a script execution.

-t number
--tries=number
     Set number of retries to number.  Specify 0 or inf for infinite
     retrying.  The default is to retry 20 times, with the exception of
     fatal errors like ``connection refused'' or ``not found'' (404),
     which are not retried.
AttachmentSizeStatusTest resultOperations
wget_try_only_once.patch336 bytesIgnored: Check issue status.NoneNone

Comments

#1

patch in the correct format.

cite in box above is from wget man pages.

AttachmentSizeStatusTest resultOperations
wget_try_only_once_0.patch698 bytesIgnored: Check issue status.NoneNone

#2

This seems like a reasonable change to me.

#3

Status:needs review» reviewed & tested by the community

yup.

#4

Status:reviewed & tested by the community» fixed

Committed this patch to CVS HEAD. Thanks.

#5

Version:6.x-dev» 5.x-dev
Status:fixed» patch (to be ported)

shouldn't this be backported ?

#6

Status:patch (to be ported)» needs review

Ok, here we go ;)

AttachmentSizeStatusTest resultOperations
wget_try_only_once_5.patch698 bytesIgnored: Check issue status.NoneNone

#7

Status:needs review» fixed

I've committed this to DRUPAL-5 as well. Thanks!

#8

Version:5.x-dev» 4.7.x-dev
Status:fixed» reviewed & tested by the community

Actually, the -t 1 option is most important for 4.7.x where there is no semaphore that makes sure that cron runs only once at a time.

AttachmentSizeStatusTest resultOperations
wget_try_only_once_47.patch692 bytesIgnored: Check issue status.NoneNone

#9

Status:reviewed & tested by the community» fixed

#10

Status:fixed» closed (fixed)