1528787 - Glean: Review baseline fields to move to client_info or similar

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P1)

Description

[Georg Fritzsche [:gfritzsche]]

We have multiple fields documented as part of the baseline ping that, per the north star metrics sheet, we are looking to send in all pings.

This is: locale, device_manufacturer, device_model, architecture, os, os_version).

This looks like a "mini environment" to me and might overlap a bit in semantics with the ping_info section.

Maybe these should be part of ping_info? Or part of a client_info field?

Comment 1

[Georg Fritzsche [:gfritzsche]]

Frank, do you have input here?

Comment 2

[Frank Bertsch [:frank]]

Sorry for the delay!

I would love to separate ping_info from client_info. That would make it clearer that some of these are about clients and some of these are about the specific ping we're sending. Generally we're currently treating ping_info as a sort of catch-all metadata, which doesn't seem ideal.

My thoughts on an initial client_info:

{
  "client_id": <UUID>,
  "app_version": <string>,
  "app_build_id": <string>,
  "first_run_date": <datetime>,
  "install_date": <datetime>,
  "manufacturer": <string>,
  "model": <string>,
  "arch": <string>,
  "os": <string>,
  "os_version": <string>
}

Comment 3

[Alessio Placitelli [:Dexter]]

All right, a created a lightweight proposal doc for helping to keep track of comments on this: https://docs.google.com/document/d/1dGiUmR63Kd1oqoL7eQzEana0gadPffvq00LEith3uCY/edit#

Comment 4

[Alessio Placitelli [:Dexter]]

Attachment: PR for the glean implementation

Comment 5

[Alessio Placitelli [:Dexter]]

Attachment: PR for the schema changes

Comment 6

[Alessio Placitelli [:Dexter]]

Everything was merged. Closing this as fixed.