Closed Bug 1528260 Opened 6 years ago Closed 6 years ago

Clarify and implement metric expiry

Categories

(Toolkit :: Telemetry, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: mdroettboom, Assigned: mdroettboom)

References

Details

(Whiteboard: [telemetry:mobilesdk:m7])

Attachments

(3 files)

This current version of the glean spec says:

Expiry

Every metric definition will need to specify if and when it expires. This means that one of the following properties must be set:

  • disabled: (bool, optional) A boolean value specifying whether or not a metric is disabled. Metrics with this property set to True will silently ignore any collection performed on them.
  • expires_after_build_date: (date/string, optional) If provided, an ISO 8601 date that represents the day from which any collection using the metric should be silently ignored. The date is checked at build time. If the specified date is passed, this metric is seen as expired.

First, there seems to be a contradiction here -- if one of these must be specified, there is no way to specify that we expect a metric to be permanent. It might be a good idea (as :liuche suggested in the data review of 1499761) that we require that to be explicit.

Secondly, these features need to be implemented, and this bug can serve to track that.

Flags: needinfo?(liuche)

I'm also fine with every probe having expiry by default :) This means the framework forces users to set an expiry date/version.
However, that means that there should be documentation about what approval process is needed to make a probe expiry "never". This should be limited to things like baseline telemetry, and should be a rare case for other telemetry.

Blocks: 1491345
Whiteboard: [telemetry:mobilesdk:m?] → [telemetry:mobilesdk:m7]
Priority: -- → P3
Flags: needinfo?(liuche)
Assignee: nobody → mdroettboom
Attached file Proposal
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.

Attachment

General

Created:
Updated:
Size: