Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test-crypto-secure-heap fails on IBM i #36929

Open
richardlau opened this issue Jan 14, 2021 · 6 comments
Open

test-crypto-secure-heap fails on IBM i #36929

richardlau opened this issue Jan 14, 2021 · 6 comments
Assignees
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. ibm i Issues and PRs related to the IBM i platform.

Comments

@richardlau
Copy link
Member

  • Test: parallel/test-crypto-secure-heap
  • Platform: IBM i
  • Console Output:
09:04:30 not ok 525 parallel/test-crypto-secure-heap
09:04:30   ---
09:04:30   duration_ms: 3.340
09:04:30   severity: fail
09:04:30   exitcode: 1
09:04:30   stack: |-
09:04:30     node:assert:119
09:04:30       throw new AssertionError(obj);
09:04:30       ^
09:04:30     
09:04:30     AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
09:04:30     
09:04:30     null !== 0
09:04:30     
09:04:30         at ChildProcess.<anonymous> (/home/iojs/build/workspace/node-test-commit-ibmi/nodes/ibmi72-ppc64/test/parallel/test-crypto-secure-heap.js:59:10)
09:04:30         at ChildProcess.<anonymous> (/home/iojs/build/workspace/node-test-commit-ibmi/nodes/ibmi72-ppc64/test/common/index.js:377:15)
09:04:30         at ChildProcess.emit (node:events:379:20)
09:04:30         at Process.ChildProcess._handle.onexit (node:internal/child_process:285:12) {
09:04:30       generatedMessage: true,
09:04:30       code: 'ERR_ASSERTION',
09:04:30       actual: null,
09:04:30       expected: 0,
09:04:30       operator: 'strictEqual'
09:04:30     }
09:04:30   ...

This test was newly added earlier in the week by 03c0564 and has been intermittently failing on IBM i (the CI for which is only run on the daily CI run) since.

@richardlau richardlau added flaky-test Issues and PRs related to the tests with unstable failures on the CI. ibm i Issues and PRs related to the IBM i platform. labels Jan 14, 2021
@richardlau
Copy link
Member Author

cc @nodejs/platform-ibmi

@ThePrez
Copy link
Contributor

ThePrez commented Jan 14, 2021

Can you please assign this to @dmabupt and me?
This might be due to some differences in the openssl build between the IBM distribution and the CI, since the IBM distribution uses the system-provided openssl that may have IBM patches. I haven't done debug/investigation, just a hunch.

@richardlau richardlau assigned ThePrez and unassigned ThePrez Jan 14, 2021
@richardlau
Copy link
Member Author

Can you please assign this to @dmabupt and me?

I've assigned to you. I can't seem to assign to @dmabupt, probably because he's not in the nodejs org (that will happen as part of #36569).

@jasnell
Copy link
Member

jasnell commented Jan 14, 2021

Would be good to go ahead and open a PR marking it flaky on that platform (if you haven't done so already :-) ...)

@targos
Copy link
Member

targos commented Jan 14, 2021

probably because he's not in the nodejs org

I think it's possible to assign external people after they commented on the thread.

@richardlau
Copy link
Member Author

Would be good to go ahead and open a PR marking it flaky on that platform (if you haven't done so already :-) ...)

Opened #36986.

Trott pushed a commit that referenced this issue Jan 19, 2021
PR-URL: #36986
Refs: #34410
Refs: #36925
Refs: #36929
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
ruyadorno pushed a commit that referenced this issue Jan 22, 2021
PR-URL: #36986
Refs: #34410
Refs: #36925
Refs: #36929
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
ruyadorno pushed a commit that referenced this issue Jan 25, 2021
PR-URL: #36986
Refs: #34410
Refs: #36925
Refs: #36929
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
targos pushed a commit that referenced this issue May 1, 2021
PR-URL: #36986
Refs: #34410
Refs: #36925
Refs: #36929
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. ibm i Issues and PRs related to the IBM i platform.
Projects
None yet
Development

No branches or pull requests

4 participants