Investigate flaky test-repl-sigint-nested-eval #41123
Labels
flaky-test
Issues and PRs related to the tests with unstable failures on the CI.
linux
Issues and PRs related to the Linux platform.
macos
Issues and PRs related to the macOS platform / OSX.
repl
Issues and PRs related to the REPL subsystem.
Comments
lpinca
added
flaky-test
mhdawson
added a commit
to mhdawson/io.js
that referenced
this issue
Dec 23, 2021
Refs: nodejs#41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com>
mhdawson
added a commit
that referenced
this issue
Jan 4, 2022
Refs: #41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #41302 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos
pushed a commit
that referenced
this issue
Jan 14, 2022
Refs: #41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #41302 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Jan 31, 2022
Refs: #41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #41302 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Linkgoron
pushed a commit
to Linkgoron/node
that referenced
this issue
Jan 31, 2022
Refs: nodejs#41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: nodejs#41302 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Feb 1, 2022
Refs: #41123 - This is one of the remaining high occurance flaky tests from the reliability reports and recent failures in the CI. Mark as flaky until issue is resolved under referenced issue. Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #41302 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
|
Ran into this again: https://ci.nodejs.org/job/node-test-commit-linux/nodes=ubuntu1804-64/45162/console |
Trott
added a commit
to Trott/io.js
that referenced
this issue
Nov 7, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Or at least that's what I think is going on. Regardless, objectively, using setTimeout() to slightly delay the invocation causes the test to not fail anymore using: tools/test.py --repeat=1000 test/parallel/test-repl-sigint-nested-eval Fixes: nodejs#41123
Trott
added a commit
to Trott/io.js
that referenced
this issue
Nov 7, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Or at least that's what I think is going on. Regardless, objectively, using setTimeout() to slightly delay the invocation causes the test to not fail anymore using: tools/test.py --repeat=1000 test/parallel/test-repl-sigint-nested-eval Fixes: nodejs#41123
Trott
added a commit
to Trott/io.js
that referenced
this issue
Nov 9, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Or at least that's what I think is going on. Regardless, objectively, using setTimeout() to slightly delay the invocation causes the test to not fail anymore using: tools/test.py --repeat=1000 test/parallel/test-repl-sigint-nested-eval Fixes: nodejs#41123
Trott
added a commit
to Trott/io.js
that referenced
this issue
Nov 9, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: nodejs#41123
nodejs-github-bot
pushed a commit
that referenced
this issue
Nov 13, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
ruyadorno
pushed a commit
that referenced
this issue
Nov 21, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Dec 30, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Dec 30, 2022
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Jan 3, 2023
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
danielleadams
pushed a commit
that referenced
this issue
Jan 4, 2023
There is a race condition where process.kill can be sent before the target is ready to receive the signal. Fixes: #41123 PR-URL: #45354 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Test
test-repl-sigint-nested-eval
Platform
macos, linux
Console output
Build links
Additional information
No response
The text was updated successfully, but these errors were encountered: