v7.x backport: src: add tracing controller#11106
v7.x backport: src: add tracing controller#11106matthewloring wants to merge 1 commit intonodejs:v7.x-stagingfrom matthewloring:v7.x-trace-event
Conversation
|
Should #10959 be included in this or backported separately? |
|
There were some concerns in #9304 we wanted to address before including this in a release, in particular making the log output location configurable (#9304 (comment)). /cc @jasongin @nodejs/diagnostics |
|
@joshgav The use of the @richardlau I think there are a few dependent PRs that can land if this one does. They can probably be opened separately if/once this has gone in. |
ac72c3e to
2f1ce29
Compare
|
@matthewloring can you rebase please |
|
@italoacasas All rebased. |
1fde990 to
9c45758
Compare
|
@joshgav Can this land? |
|
It also looks like the trace event configuration options landed in the |
LGTM @italoacasas do the approvals/LGTMs from the original PR count for backports too? Or do backports need another approval? |
joshgav
left a comment
There was a problem hiding this comment.
Rubber stamp LGTM from reviewing original PR.
|
@joshgav same process. we need LGTM here. |
|
This is not landing in version 7 staging, because of 4b8b7e9. @matthewloring can you update the backport again please, and sorry for the inconvenience. |
This commit adds support for trace-event tracing to Node.js. It provides a mechanism to centralize tracing information generated by V8, Node core, and userspace code. It includes: - A trace writer responsible for serializing traces and cycling the output files so that no individual file becomes to large. - A buffer for aggregating traces to allow for batched flushes. - An agent which initializes the tracing controller and ensures that trace serialization is done on a separate thread. - A set of macros for generating trace events. - Tests and documentation. Author: Raymond Kang <raymondksi@gmail.com> Author: Kelvin Jin <kelvinjin@google.com> Author: Matthew Loring <mattloring@google.com> Author: Jason Ginchereau <jasongin@microsoft.com> PR-URL: #9304 Backport PR-URL: #11106 Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
|
Rebased cleanly (and force-pushed to @matthewloring's branch). Still need another LGTM I think? @nodejs/diagnostics PTAL. |
|
LGTM but I'm not sure I count since I opened the PR. |
This commit adds support for trace-event tracing to Node.js. It provides a mechanism to centralize tracing information generated by V8, Node core, and userspace code. It includes: - A trace writer responsible for serializing traces and cycling the output files so that no individual file becomes to large. - A buffer for aggregating traces to allow for batched flushes. - An agent which initializes the tracing controller and ensures that trace serialization is done on a separate thread. - A set of macros for generating trace events. - Tests and documentation. Author: Raymond Kang <raymondksi@gmail.com> Author: Kelvin Jin <kelvinjin@google.com> Author: Matthew Loring <mattloring@google.com> Author: Jason Ginchereau <jasongin@microsoft.com> PR-URL: nodejs#11106 Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
|
Landed |
|
Adding |
|
selected semver-minor's can land on LTS branches. It people feel that it is worthwhile, then mention @nodejs/lts in the comments and a decision can be made. |
|
This change is not compatible with the versions of V8 in 4.x and 6.x. |
This commit adds support for trace-event tracing to Node.js. It provides
a mechanism to centralize tracing information generated by V8, Node
core, and userspace code. It includes:
output files so that no individual file becomes to large.
trace serialization is done on a separate thread.
Author: Raymond Kang raymondksi@gmail.com
Author: Kelvin Jin kelvinjin@google.com
Author: Matthew Loring mattloring@google.com
Author: Jason Ginchereau jasongin@microsoft.com
PR-URL: #9304
Reviewed-By: Trevor Norris trev.norris@gmail.com
Reviewed-By: Michael Dawson michael_dawson@ca.ibm.com
Reviewed-By: Josh Gavant josh.gavant@outlook.com