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.

Comments

alex_b’s picture

StatusFileSize
new698 bytes

patch in the correct format.

cite in box above is from wget man pages.

drewish’s picture

This seems like a reasonable change to me.

moshe weitzman’s picture

Status: Needs review » Reviewed & tested by the community

yup.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed this patch to CVS HEAD. Thanks.

yched’s picture

Version: 6.x-dev » 5.x-dev
Status: Fixed » Patch (to be ported)

shouldn't this be backported ?

alex_b’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new698 bytes

Ok, here we go ;)

dries’s picture

Status: Needs review » Fixed

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

alex_b’s picture

Version: 5.x-dev » 4.7.x-dev
Status: Fixed » Reviewed & tested by the community
StatusFileSize
new692 bytes

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.

killes@www.drop.org’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)