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.

Sign up for GitHub

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

Jump to bottom

Investigate flaky test - parallel/test-crypto-keygen #41206

Closed
mhdawson opened this issue Dec 16, 2021 · 2 comments
Closed

Investigate flaky test - parallel/test-crypto-keygen #41206

mhdawson opened this issue Dec 16, 2021 · 2 comments
Labels
crypto Issues and PRs related to the crypto subsystem. flaky-test Issues and PRs related to the tests with unstable failures on the CI.

Comments

@mhdawson
Copy link
Member

Test

parallel/test-crypto-keygen

Platform

Windows

Console output

Error Message
fail (1)
Stacktrace
timeout

Build links

From reliability report - 5 in latest report
#40980, #41046, #39239, #40706, #41146
https://ci.nodejs.org/job/node-test-pull-request/41415/
https://ci.nodejs.org/job/node-test-pull-request/41485/

Additional information

No response
@mhdawson mhdawson added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Dec 16, 2021
mhdawson added a commit to mhdawson/io.js that referenced this issue Dec 16, 2021
@mhdawson
test: mark test-crypto-keygen slow on windows
e48d755 
Refs: nodejs#41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>
@mhdawson mhdawson mentioned this issue Dec 16, 2021
Closed
@Mesteery Mesteery added the crypto Issues and PRs related to the crypto subsystem. label Dec 16, 2021
mhdawson added a commit that referenced this issue Dec 21, 2021
@mhdawson
test: mark test-crypto-keygen slow on windows
eee1a6f 
Refs: #41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: #41207
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
targos pushed a commit that referenced this issue Jan 14, 2022
@mhdawson @targos
test: mark test-crypto-keygen slow on windows
367ab2a 
Refs: #41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: #41207
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
danielleadams pushed a commit that referenced this issue Jan 31, 2022
@mhdawson @danielleadams
test: mark test-crypto-keygen slow on windows
9da2224 
Refs: #41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: #41207
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Linkgoron pushed a commit to Linkgoron/node that referenced this issue Jan 31, 2022
@mhdawson @Linkgoron
test: mark test-crypto-keygen slow on windows
d006c7d 
Refs: nodejs#41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: nodejs#41207
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
danielleadams pushed a commit that referenced this issue Feb 1, 2022
@mhdawson @danielleadams
test: mark test-crypto-keygen slow on windows
2e133d5 
Refs: #41206

This test seems to be failing regularly on
windows with a timeout. Mark it slow to reduce
CI noise until it gets investigated.

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: #41207
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott Trott mentioned this issue Jun 29, 2023
Closed
@Trott
Copy link
Member

Trott commented Jun 29, 2023

Closing per #48575

@Trott Trott closed this as completed Jun 29, 2023
This was referenced Aug 16, 2023
Closed
Closed
@joyeecheung joyeecheung reopened this Aug 18, 2023
nodejs-github-bot pushed a commit that referenced this issue Aug 31, 2023
@joyeecheung @nodejs-github-bot
test: split test-crypto-keygen.js
7224940 
To avoid timing out on ARM machines in the CI.

PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
nodejs-github-bot pushed a commit that referenced this issue Aug 31, 2023
@joyeecheung @nodejs-github-bot
test: split JWK async elliptic curve keygen tests
b9a6ab5 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
nodejs-github-bot pushed a commit that referenced this issue Aug 31, 2023
@joyeecheung @nodejs-github-bot
test: reduce length in crypto keygen tests
b781eaf 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
UlisesGascon pushed a commit that referenced this issue Sep 10, 2023
@joyeecheung @UlisesGascon
test: split test-crypto-keygen.js
e04a260 
To avoid timing out on ARM machines in the CI.

PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
UlisesGascon pushed a commit that referenced this issue Sep 10, 2023
@joyeecheung @UlisesGascon
test: split JWK async elliptic curve keygen tests
4faa30c 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
UlisesGascon pushed a commit that referenced this issue Sep 10, 2023
@joyeecheung @UlisesGascon
test: reduce length in crypto keygen tests
6ed47bd 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
alexfernandez pushed a commit to alexfernandez/node that referenced this issue Nov 1, 2023
@joyeecheung @alexfernandez
test: split test-crypto-keygen.js
875216e 
To avoid timing out on ARM machines in the CI.

PR-URL: nodejs#49221
Refs: nodejs#49202
Refs: nodejs#41206
Reviewed-By: Luigi Pinca <[email protected]>
alexfernandez pushed a commit to alexfernandez/node that referenced this issue Nov 1, 2023
@joyeecheung @alexfernandez
test: split JWK async elliptic curve keygen tests
c532ec8 
PR-URL: nodejs#49221
Refs: nodejs#49202
Refs: nodejs#41206
Reviewed-By: Luigi Pinca <[email protected]>
alexfernandez pushed a commit to alexfernandez/node that referenced this issue Nov 1, 2023
@joyeecheung @alexfernandez
test: reduce length in crypto keygen tests
3daee4d 
PR-URL: nodejs#49221
Refs: nodejs#49202
Refs: nodejs#41206
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this issue Nov 27, 2023
@joyeecheung @targos
test: split test-crypto-keygen.js
344592d 
To avoid timing out on ARM machines in the CI.

PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this issue Nov 27, 2023
@joyeecheung @targos
test: split JWK async elliptic curve keygen tests
958e114 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this issue Nov 27, 2023
@joyeecheung @targos
test: reduce length in crypto keygen tests
67cb6f9 
PR-URL: #49221
Refs: #49202
Refs: #41206
Reviewed-By: Luigi Pinca <[email protected]>
@marco-ippolito
Copy link
Member

marco-ippolito commented Dec 16, 2023 •
edited
Loading

It seems it's no longer flaky, since 2023-07-10
Query https://github.com/nodejs/reliability/issues?q=is%3Aissue+is%3Aopen+test-crypto-keygen

lpinca added a commit to lpinca/node that referenced this issue Jan 25, 2024
@lpinca
test: remove test-crypto-keygen flaky designation
4af2580 
Refs: nodejs#41206 (comment)
@lpinca lpinca mentioned this issue Jan 25, 2024
Merged
lpinca added a commit to lpinca/node that referenced this issue Feb 1, 2024
@lpinca
test: remove test-crypto-keygen flaky designation
53ee8e3 
Refs: nodejs#41206 (comment)
nodejs-github-bot pushed a commit that referenced this issue Feb 3, 2024
@lpinca
test: remove test-crypto-keygen flaky designation
415aa77 
Refs: #41206 (comment)
PR-URL: #51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
rdw-msft pushed a commit to rdw-msft/node that referenced this issue Feb 9, 2024
@lpinca @rdw-msft
test: remove test-crypto-keygen flaky designation
24d9ba1 
Refs: nodejs#41206 (comment)
PR-URL: nodejs#51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
targos pushed a commit that referenced this issue Feb 15, 2024
@lpinca @targos
test: remove test-crypto-keygen flaky designation
a8ac337 
Refs: #41206 (comment)
PR-URL: #51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
marco-ippolito pushed a commit to marco-ippolito/node that referenced this issue Feb 19, 2024
@lpinca @marco-ippolito
test: remove test-crypto-keygen flaky designation
bb70e07 
Refs: nodejs#41206 (comment)
PR-URL: nodejs#51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
richardlau pushed a commit that referenced this issue Mar 25, 2024
@lpinca @richardlau
test: remove test-crypto-keygen flaky designation
2a95d52 
Refs: #41206 (comment)
PR-URL: #51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
richardlau pushed a commit that referenced this issue Mar 25, 2024
@lpinca @richardlau
test: remove test-crypto-keygen flaky designation
d0d5bd0 
Refs: #41206 (comment)
PR-URL: #51567
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: split test-crypto-keygen.js
6ebbd6c 
To avoid timing out on ARM machines in the CI.

PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: split JWK async elliptic curve keygen tests
2aa755e 
PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: reduce length in crypto keygen tests
dc50f7f 
PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: split test-crypto-keygen.js
580db80 
To avoid timing out on ARM machines in the CI.

PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: split JWK async elliptic curve keygen tests
2447b77 
PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
@sercher
test: reduce length in crypto keygen tests
2bfb541 
PR-URL: nodejs/node#49221
Refs: nodejs/node#49202
Refs: nodejs/node#41206
Reviewed-By: Luigi Pinca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
crypto Issues and PRs related to the crypto subsystem. flaky-test Issues and PRs related to the tests with unstable failures on the CI.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Trott @joyeecheung @mhdawson @marco-ippolito @Mesteery