hosting_task_count_running() currently always returns 0, resulting in more tasks running the number specified by the hosting_queue_tasks_items variable.

The below patch resolves the issue.

CommentFileSizeAuthor
#4 hostmaster-fix-task-count-running-2738587.patch960 bytestsplash
#2 hostmaster-fix-task-count-running-2738587.patch961 bytestsplash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tom_spiers created an issue. See original summary.

tsplash’s picture

tsplash CreditAttribution: tsplash as a volunteer commented
FileSize
hostmaster-fix-task-count-running-2738587.patch961 bytes
tsplash’s picture

tsplash CreditAttribution: tsplash as a volunteer commented
tsplash’s picture

tsplash CreditAttribution: tsplash as a volunteer commented
FileSize
hostmaster-fix-task-count-running-2738587.patch960 bytes
tsplash’s picture

tsplash CreditAttribution: tsplash as a volunteer commented
tsplash’s picture

tsplash CreditAttribution: tsplash as a volunteer commented
Issue summary: View changes
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic at Ergon Logic Enterprises for Aegir Cooperative commented
Status: Active » Needs review

Indeed. It looks like the D7 conversion missed that SQL. Code looks fine. Can anyone test?

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Fixed

Thanks, verified with a few runs of:
drush @hostmaster php-eval "echo hosting_task_count_running()"

Status: Fixed » Closed (fixed)

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