Repository navigation
inspector: support attaching to a running process in probe mode - #67201
Open
joyeecheung wants to merge 1 commit into
Open
joyeecheung wants to merge 1 commit into
joyeecheung wants to merge 1 commit into
Annotations
1 error
|
Validate first commit message
# c2881e9056a0258e69600e6c5048c206a6cccdf0
ok 1 assisted-by-is-trailer: no Assisted-by metadata
ok 2 co-authored-by-is-trailer: no Co-authored-by metadata
ok 3 fixes-url: skipping fixes-url # SKIP
ok 4 line-after-title: blank line after title
not ok 5 line-length: Line should be <= 72 columns.
---
{
found: 73,
compare: '<=',
wanted: 72,
at: {
line: 17,
column: 72,
body: [
'',
'Probe mode currently only launches a new process from the entry point',
'script. This change lets it attach to an already-running process, so a',
'long-lived application can be inspected without restarting it.',
'',
'Running `node inspect --probe` without specifying an entry point script',
'instructs Node.js to attach to a running Node.js process and probe.',
'The way attaching is done is specified through command line arguments:',
'',
"- `--pid <pid>` activates the target's inspector (if it's not already",
' activated) and connects to it.',
'- `--host <host>` and `--port <port>` specify an address where an',
' inspector is listening. The host defaults to 127.0.0.1 and the',
' port defaults to 9229.',
'',
'When the session ends, probe mode resumes and detaches instead of killing',
'a process it does not own. The report now includes a top-level `target`',
'field describing what was probed, and each probe contains a `locations`',
'field about where it bound.',
'',
'An attach session ends when a probe reaches `--max-hit`',
'(reported as `completed`), when the target finishes on its',
'own (`completed`, or `miss` with the pending probes, and the target is',
'then allowed to exit), or when `--timeout` fires (reported as `timeout`,',
'the probing process exit with 0 and reports the hits recorded so far).',
'',
'`--timeout` now is also applied to the startup phase in both modes, so a',
'target that never responds before the breakpoints are bound is reported as',
'an error instead of hanging.',
'',
'Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>'
]
}
}
...
not ok 6 line-length: Line should be <= 72 columns.
---
{
found: 74,
compare: '<=',
wanted: 72,
at: {
line: 29,
column: 72,
body: [
'',
'Probe mode currently only launches a new process from the entry point',
'script. This change lets it attach to an already-running process, so a',
'long-lived application can be inspected without restarting it.',
'',
'Running `node inspect --probe` without specifying an entry point script',
'instructs Node.js to attach to a running Node.js process and probe.',
'The way attaching is done is specified through command line arguments:',
'',
"- `--pid <pid>` activates the target's inspector (if it's not already",
' activated) and connects to it.',
'- `--host <host>` and `--port <port>` specify an address where an',
' inspector is listening. The host defaults to 127.0.0.1 and the',
' port defaults to 9229.',
'',
'When the session ends, probe mode resumes and detaches instead of killing',
'a process it does not own. The report now includes a top-level `target`',
'field describing what was probed, and each probe contains a `locations`',
'field about where it bound.',
'',
'An attach session ends when a probe reaches `--max-hit`',
'(reported as `completed`), when the target finishes on its',
'own (`completed`, or `miss` with the pending probes, and the target is',
'then allowed to exit), or when `--timeout` fires (reported as `timeout`,',
'the probing process exit with 0 and reports the hits recor
|
background
wait
wait-all
cancel
parallel
Loading