1410161 - Get sync_ping_schema.json into mozilla-pipeline-schemas

Status: RESOLVED INVALID

(Firefox :: Sync, enhancement, P3)

Description

[Thom Chiovoloni [:tcsc] (ex-moco)]

This would probably have caught a number of issues we've encountered with our telemetry, since we'd get alerted on failures to pass the schema.

Unfortunately, it can't actually land as-is, since what we actually accept is much more generous than the schema specified there. If we were to land without updating it, it would complain a great deal, and also throw out most records from iOS and Android, both of which aren't as good at following the ping schema as Desktop (yay).

The schema should be updated to reflect the real schema accepted by the SyncPingConversion.scala code, and then incorporated into https://github.com/mozilla-services/mozilla-pipeline-schemas.

Comment 1

[Julie McCracken (:julie)]

Do you want to take this one, Thom?

Comment 2

[Thom Chiovoloni [:tcsc] (ex-moco)]

Yep, but I'll leave it as P2 since it's not critical.

Comment 3

[Thom Chiovoloni [:tcsc] (ex-moco)]

For the reasons mentioned above (we can't land as-is so really this bug is "write a new schema that accurately reflects what SyncPingConversion.scala accepts"), this is getting pushed to he back burner (and I'm unassigning it from myself).

In the future it should be easier due to the release version of more clients sending acceptable schemas anyway.

Comment 4

[Mark Hammond [:markh] [:mhammond]]

The pipelines have all changed.