1548129 - Disable metrics on a disabled custom ping at build time

Status: RESOLVED WONTFIX

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

Description

[Michael Droettboom [:mdroettboom]]

Once we have a disabled and expires flag on pings as part of 1549767, we could pretty easy disable any metrics that are sent only on those pings at build time as well, which would save the work of recording those metrics at run time, only to have them never sent in a ping.

Comment 1

[Jan-Erik Rediger [:janerik]]

The mentioned bug is the wrong one; I couldn't find a relevant one, so I filed a new one: https://bugzilla.mozilla.org/show_bug.cgi?id=1587838

Comment 2

[Jan-Erik Rediger [:janerik]]

Unassigning and moving back to triage, because we don't have the necessary functionality yet.

Comment 3

[Michael Droettboom [:mdroettboom]]

The concept of "disabled" or "expired" pings might have other implications and open up other bugs. For example, what if we collect more than 500 events on a disabled ping. Would they ever be cleared, or just fill up disk indefinitely?

I would be in favor of explicitly disallowing "disabled" or "expired" pings for all time, and closing this bug -- we don't have them now anyway.

Comment 4

[Alessio Placitelli [:Dexter]]

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

I would be in favor of explicitly disallowing "disabled" or "expired" pings for all time, and closing this bug -- we don't have them now anyway.

This sounds like a good idea.