`test-perf-hooks` failing on v18.x SmartOS CI

[aduh95]

Test

sequential/test-perf-hooks

Platform

SmartOS

Console output

not ok 3959 sequential/test-perf-hooks
  ---
  duration_ms: 352.25600
  severity: fail
  exitcode: 1
  stack: |-
    node:assert:399
        throw err;
        ^
    
    AssertionError [ERR_ASSERTION]: Date.now() - performance.timeOrigin (169.60791015625) - process.uptime() * 1000 (94.956) = 74.65191015625 >= +- 50
        at Object.<anonymous> (/home/iojs/build/workspace/node-test-commit-smartos/nodes/smartos23-x64/test/sequential/test-perf-hooks.js:35:3)
        at Module._compile (node:internal/modules/cjs/loader:1364:14)
        at Module._extensions..js (node:internal/modules/cjs/loader:1422:10)
        at Module.load (node:internal/modules/cjs/loader:1203:32)
        at Module._load (node:internal/modules/cjs/loader:1019:12)
        at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12)
        at node:internal/main/run_main_module:28:49 {
      generatedMessage: false,
      code: 'ERR_ASSERTION',
      actual: false,
      expected: true,
      operator: '=='
    }
    
    Node.js v18.20.7-pre

Build links

• https://ci.nodejs.org/job/node-test-commit-smartos/59179/
• https://ci.nodejs.org/job/node-test-commit-smartos/59141/
• https://ci.nodejs.org/job/node-test-commit-smartos/59166/

Additional information

No response

[aduh95]

/cc @nodejs/platform-smartos

[danmcd]

Please add me ( @danmcd ) to the platform-smartos team in the short term. ANd in the longer term we need to make this "ILLUMOS" because it affects all illumos distros, not just SmartOS.

I know we provide resources, but with the recent departure of @bahamat I want to make sure our end is informed and active.

[jclulow]

NB: I've created nodejs/admin#954 about adding you, @danmcd.

[danmcd]

BTW, due to strange-to-me OAUTH requirements of the ci.nodejs.org, I cannot yet see the CI links posted above. Asked for permissions too from orgs I belong to, but am not an admin in. That's odd, but honestly the orgs in question are illumos-related, so it's good you know that in the long run this is about illumos, not just SmartOS.

[danmcd]

BTW, due to strange-to-me OAUTH requirements of the ci.nodejs.org, I cannot yet see the CI links posted above. Asked for permissions too from orgs I belong to, but am not an admin in. That's odd, but honestly the orgs in question are illumos-related, so it's good you know that in the long run this is about illumos, not just SmartOS.

Yeah, one org I belong to declined you because it wanted private repos. COuld be a rate-limit thing so I'll try again tomorrow.

[danmcd]

I STILL can't access the CI @aduh95 because one of my orgs won't grant permission (private repos). Would appreciate assistance in being able to see the three failures so I can help.

[aduh95]

You can find the output in #57000 (comment); other than that, you should be able to access the CI as long as you have a GitHub account, you shouldn't need to get permissions (at least, not from our side).
I would recommend getting access to the CI, and watch failures in daily builds, that's the best place to spot flaky tests.

[danmcd]

as long as you have a GitHub account, you shouldn't need to get permissions (at least, not from our side).

Yeah, I thought that was all I needed too, but OAuth is whining at me a lot. First about needing org access, then rate-limit blocking me after too many tries. :(

[danmcd]

WHOA NEVER MIND! Somehow it cleared up!