Page MenuHomePhabricator

selenium-daily Jenkins jobs failing with `node: ../src/coroutine.cc:134: void* find_thread_id_key(void*): Assertion `thread_id_key != 0x7777' failed`
Closed, ResolvedPublic

Description

Failing jobs:

The likely cause is 890944 (jjb: Switch incidental uses of nodejs images from 14 to 16). The fix is a partial revert in 896026.

I have updated jobs (revering node.js version from v16 to v14) and after re-running the jobs, the error messages were gone.

Event Timeline

bwang renamed this task from Minerva selenium tests are failing to Page preview and Minerva selenium tests failing.Feb 23 2023, 8:00 PM
bwang added a project: Page-Previews.
bwang updated the task description. (Show Details)
ovasileva triaged this task as Medium priority.Mar 6 2023, 10:52 PM
ovasileva moved this task from Incoming to Current Quarter on the Web-Team-Backlog board.
Jdlrobson renamed this task from Page preview and Minerva selenium tests failing to Page preview and Minerva selenium tests failing (node: ../src/coroutine.cc:134: void* find_thread_id_key(void*): Assertion `thread_id_key != 0x7777' failed).Mar 8 2023, 6:44 PM
zeljkofilipin changed the task status from Open to In Progress.Mar 9 2023, 8:36 AM
zeljkofilipin claimed this task.
zeljkofilipin moved this task from Backlog 🪒 to Deep work 🌊 on the User-zeljkofilipin board.

Change 896026 had a related patch set uploaded (by Zfilipin; author: Zfilipin):

[integration/config@master] WIP Revert "jjb: Switch incidental uses of nodejs images from 14 to 16"

https://gerrit.wikimedia.org/r/896026

I've reverted selenium-daily-beta-Popups back to node14 and it's not longer failing (diff).

zeljkofilipin renamed this task from Page preview and Minerva selenium tests failing (node: ../src/coroutine.cc:134: void* find_thread_id_key(void*): Assertion `thread_id_key != 0x7777' failed) to selenium-daily Jenkins jobs failing with `node: ../src/coroutine.cc:134: void* find_thread_id_key(void*): Assertion `thread_id_key != 0x7777' failed`.Mar 9 2023, 10:49 AM
zeljkofilipin raised the priority of this task from Medium to High.
zeljkofilipin moved this task from Inbox to Selenium on the MediaWiki-Core-Tests board.

selenium-daily-beta-WikibaseLexeme (various error messages)

The WikibaseLexeme daily browser tests have been flaky for a while, I’m afraid (partly due to beta cluster instability, IIRC) – unless you’ve seen the assertion error message there, I don’t think it’s related to this task. (I don’t see that message in the latest two failing builds.)

This is because we use wdio-sync which depends on fibers https://www.npmjs.com/package/fibers-nodejs which is no more supported https://github.com/laverdet/node-fibers#readme

Update [April 13th, 2021]
Fibers is not compatible with nodejs v16.0.0 or later. Unfortunately, v8 commit dacc2fee0f is a breaking change and workarounds are non-trivial.

We need to get rid of wdio-sync which is T256626: Refactor WebdriverIO tests from sync to async mode and is a blocker for the NodeJS 16 upgrade T314470.

As far as CI is concerned, for recent branches we will switch to NodeJS 16 as soon as wdio-sync got phased out. For the old branch I think we can stick the job to NodeJS 14 since I don't think we can easily backport the removal of wdio-sync.

The immediate issue is solved, the selenium-daily jobs are back to NodeJS 14.

Change 896026 merged by jenkins-bot:

[integration/config@master] Partial revert "jjb: Switch incidental uses of nodejs images from 14 to 16"

https://gerrit.wikimedia.org/r/896026

Thanks for the swift action here!

Change 904931 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/config@master] Revert "Docker: Drop node14 images, all now unused"

https://gerrit.wikimedia.org/r/904931

Change 904931 merged by jenkins-bot:

[integration/config@master] Revert "Docker: Drop node14 images, all now unused"

https://gerrit.wikimedia.org/r/904931