Tracelogger: allow traceLogger to record events without writing to disk

RESOLVED FIXED in Firefox 63

Status

()

P3
normal
RESOLVED FIXED
7 months ago
5 months ago

People

(Reporter: denispal, Assigned: denispal)

Tracking

(Blocks: 1 bug)

Trunk
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
It is not currently possible to record events by the traceLogger without also writing the events to disk.  We'll need to decouple these two actions as a first step to hooking up the traceLogger to the gecko profiler since we need to keep this data in memory and not flush to disk.
(Assignee)

Updated

7 months ago
Assignee: nobody → dpalmeiro
(Assignee)

Comment 1

7 months ago
When enabling the tracelogger, it will automatically spew the data to disk in the location specified by TLDIR or /tmp, if undefined.  However, there needs to be a way to enable the tracelogger without spewing so that we can write this data as part of the gecko profiler JSON output, and ultimately visualized with perf.html.
Comment on attachment 8999314 [details]
Bug 1482560: Do not flush tracelogger to disk unless specifically requested to do so

Guessing that a review is wanted here.
Attachment #8999314 - Flags: review?(sphink)
Priority: -- → P3
(Assignee)

Comment 3

7 months ago
(In reply to Jason Orendorff [:jorendorff] from comment #2)
> Comment on attachment 8999314 [details]
> Bug 1482560: Do not flush tracelogger to disk unless specifically requested
> to do so
> 
> Guessing that a review is wanted here.

Yes, I was planning to chain this up to a couple other reviews (bug 1480104 & another one I will open about connecting the TL to the gecko profiler) so the context would be more clear so I left the reviewers list empty for now.  However, this piece is independent enough that a review now is also welcome!
(Assignee)

Updated

7 months ago
Blocks: 1480104
Comment on attachment 8999314 [details]
Bug 1482560: Do not flush tracelogger to disk unless specifically requested to do so

Steve Fink [:sfink] [:s:] has approved the revision.
Attachment #8999314 - Flags: review+
(Assignee)

Comment 5

7 months ago
Thanks for the review, sfink!
Keywords: checkin-needed
Lando says this can't land because it depends on D3683.
Flags: needinfo?(dpalmeiro)
Keywords: checkin-needed
(Assignee)

Comment 7

7 months ago
Thanks, sorry I fixed it now.  It should be the other way around, so I guess D3683 should maybe be a child revision instead of parent..
Flags: needinfo?(dpalmeiro)

Comment 8

7 months ago
Pushed by rvandermeulen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c8443454c325
Do not flush tracelogger to disk unless specifically requested to do so r=sfink

Comment 9

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c8443454c325
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox63: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment on attachment 8999314 [details]
Bug 1482560: Do not flush tracelogger to disk unless specifically requested to do so

I think this had a stale r? (probably because phabricator stopped mirroring the review state?)
Attachment #8999314 - Flags: review?(sphink) → review+
(Assignee)

Updated

5 months ago
Blocks: 1501377
You need to log in before you can comment on or make changes to this bug.