I ran into a problem with sites using the default value (marked as NULL in the database), patch attached to fix it. I also added a watchdog so you can see if cron ran or not.

CommentFileSizeAuthor
cron_interval_problem.patch1.04 KBattiks

Comments

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented
Status: Needs review » Needs work

Upon visual inspection, the patch makes perfect sense, will test later.

I'm not a big fan of the watchdog entry though: for installs managing a large number of sites, this will quickly start to fill up watchdog. I don't have an Aegir console in front of me, but doesn't a site's info show when cron was last run?

Also, apologies for the delay in replying, this one fell of my radar...

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented

Marked #1600426: Cron is run every minute on all sites not yet defined in the hosting_advanced_cron table as duplicate.

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented
Status: Needs work » Fixed

Fixed (commit, for now without the watchdog part). Thanks for the patch!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.