-
Notifications
You must be signed in to change notification settings - Fork 11
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
CI Reliability 2022-08-29 #361
Comments
joyeecheung
added a commit
to nodejs/node
that referenced
this issue
Aug 31, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
joyeecheung
added a commit
to joyeecheung/node
that referenced
this issue
Sep 1, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
joyeecheung
added a commit
to joyeecheung/node
that referenced
this issue
Sep 1, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
RafaelGSS
pushed a commit
to nodejs/node
that referenced
this issue
Sep 5, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
joyeecheung
added a commit
to joyeecheung/node
that referenced
this issue
Sep 6, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
RafaelGSS
pushed a commit
to nodejs/node
that referenced
this issue
Sep 6, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
RafaelGSS
pushed a commit
to nodejs/node
that referenced
this issue
Sep 7, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
Fyko
pushed a commit
to Fyko/node
that referenced
this issue
Sep 15, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
targos
pushed a commit
to nodejs/node
that referenced
this issue
Sep 16, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
juanarbol
pushed a commit
to nodejs/node
that referenced
this issue
Oct 10, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
juanarbol
pushed a commit
to nodejs/node
that referenced
this issue
Oct 11, 2022
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: #44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
guangwong
pushed a commit
to noslate-project/node
that referenced
this issue
Jan 3, 2023
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs/node#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
guangwong
pushed a commit
to noslate-project/node
that referenced
this issue
Jan 3, 2023
Previously the tests required that Node.js finish the initialization of the watchdog thread and fires the timeout within 100ms, which can be difficult on certain systems under load. This patch relaxes the requirement to 2000ms. If there is a bug and the timeout can actually be escaped, raising the timeout to 2000ms would not make a difference anyway. PR-URL: nodejs/node#44433 Refs: nodejs/reliability#333 Refs: nodejs/reliability#361 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
None yet
0 participants
Failures in node-test-pull-request/46203 to node-test-pull-request/46298 that failed more than 2 PRs
(Generated with
ncu-ci walk pr --stats=true --markdown /home/runner/work/reliability/reliability/results.md
)JSTest Failure
sequential/test-child-process-execsync
Example
parallel/test-child-process-exec-timeout
Example
parallel/test-vm-timeout-escape-promise-2
Example
sequential/test-debugger-backtrace
Example
sequential/test-debugger-custom-port
Example
sequential/test-debugger-exec
Example
sequential/test-debugger-launch
Example
sequential/test-debugger-random-port-with-inspect-port
Example
sequential/test-debugger-watchers
Example
sequential/test-heap-prof
Example
parallel/test-child-process-spawnsync-timeout
Example
sequential/test-debugger-run-after-quit-restart
Example
node-api/test_threadsafe_function/test
Example
parallel/test-debugger-extract-function-name
Example
sequential/test-debugger-auto-resume
Example
sequential/test-debugger-break
Example
sequential/test-debugger-clear-breakpoints
Example
sequential/test-debugger-exec-scope
Example
sequential/test-debugger-heap-profiler
Example
sequential/test-debugger-list
Example
sequential/test-debugger-object-type-remote-object
Example
sequential/test-debugger-preserve-breaks
Example
sequential/test-debugger-profile
Example
sequential/test-debugger-restart-message
Example
sequential/test-debugger-use-strict
Example
report/test-report-fatalerror-oomerror
Example
sequential/test-debugger-random-port
Example
sequential/test-debugger-sb-before-load
Example
sequential/test-worker-prof
Example
Jenkins Failure
Build Failure
ERROR: Step �Publish JUnit test result report� failed: No test report files were found. Configuration error?
Example
Progress
sequential/test-child-process-execsync
(10)parallel/test-child-process-exec-timeout
(6)parallel/test-vm-timeout-escape-promise-2
(5)sequential/test-debugger-backtrace
(5)sequential/test-debugger-custom-port
(5)sequential/test-debugger-exec
(5)sequential/test-debugger-launch
(5)sequential/test-debugger-random-port-with-inspect-port
(5)sequential/test-debugger-watchers
(5)sequential/test-heap-prof
(5)parallel/test-child-process-spawnsync-timeout
(4)sequential/test-debugger-run-after-quit-restart
(4)node-api/test_threadsafe_function/test
(3)parallel/test-debugger-extract-function-name
(3)sequential/test-debugger-auto-resume
(3)sequential/test-debugger-break
(3)sequential/test-debugger-clear-breakpoints
(3)sequential/test-debugger-exec-scope
(3)sequential/test-debugger-heap-profiler
(3)sequential/test-debugger-list
(3)sequential/test-debugger-object-type-remote-object
(3)sequential/test-debugger-preserve-breaks
(3)sequential/test-debugger-profile
(3)sequential/test-debugger-restart-message
(3)sequential/test-debugger-use-strict
(3)report/test-report-fatalerror-oomerror
(2)sequential/test-debugger-random-port
(2)sequential/test-debugger-sb-before-load
(2)sequential/test-worker-prof
(2)ERROR: Step �Publish JUnit test result report� failed: No test report files were found. Configuration error?
(6)The text was updated successfully, but these errors were encountered: