Closed Bug 1557046 Opened 6 years ago Closed 6 years ago

Document why events pings can have timestamps outside of their duration

Categories

(Data Platform and Tools :: Glean: SDK, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mdroettboom, Assigned: mdroettboom)

Details

(Whiteboard: [telemetry:glean-rs:m6])

A follow-up to this bug -- we should explain why timestamps in an events ping can be greater than the duration of the ping.

Priority: -- → P3
Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m6]
Assignee: nobody → mdroettboom

This really relates to the format of the ping contents, which isn't really documented anywhere outside of the glean schema in mozilla-pipeline-schemas. I think we should have some narrative documentation of the ping contents -- but where should that go? We could make a new chapter in the docs for this, or we could document it with each metric type. I kind of like the latter -- let's document everything about a particular metric on one page -- but I know there are those who prefer to keep the SDK docs just for SDK users.

Thoughts?

Flags: needinfo?(alessio.placitelli)

(In reply to Michael Droettboom [:mdroettboom] from comment #1)

This really relates to the format of the ping contents, which isn't really documented anywhere outside of the glean schema in mozilla-pipeline-schemas.

Any specific content you're thinking about? I was specifically only considering event timestamps and the ping start_time, end_time.

I think we should have some narrative documentation of the ping contents -- but where should that go? We could make a new chapter in the docs for this, or we could document it with each metric type. I kind of like the latter -- let's document everything about a particular metric on one page -- but I know there are those who prefer to keep the SDK docs just for SDK users.

Thoughts?

If the narrative is about the content mentioned above, maybe it could simply live here: https://mozilla.github.io/glean/book/user/pings/events.html - what do you think?

Flags: needinfo?(alessio.placitelli)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.