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

Comments

tom_spiers created an issue. See original summary.

tsplash’s picture

tsplash commented
StatusFileSize
new hostmaster-fix-task-count-running-2738587.patch961 bytes
tsplash’s picture

tsplash commented
tsplash’s picture

tsplash commented
StatusFileSize
new hostmaster-fix-task-count-running-2738587.patch960 bytes
tsplash’s picture

tsplash commented
tsplash’s picture

tsplash commented
Issue summary: View changes
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
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 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.