1656568 - Update nimbus events to set event_object="nimbus_experiment"

Status: RESOLVED FIXED

(Firefox :: Nimbus Desktop Client, enhancement, P1)

Description

[Kate Hudson :k88hudson]

Let's audit all the fields for how we send enrollment telemetry for Nimbus experiments, if we want to send something other than normandy, etc.

Comment 1

[Kate Hudson :k88hudson]

Let's take a look at:

• if this should use the shared codepath / helpers that Normandy uses
• if we should use a different category than normandy

Tim, any opinions on how hard it would be to support a different category type for nimbus?

Comment 2

[Tim Smith (inactive) 👨‍🔬 [:tdsmith]]

If the enrolment events have the same format, this should be as easy as replacing = 'normandy' with IN ('normandy', 'nimbus') in mozanalysis, Jetstream, and the aggregates table powering Grafana. If the formats are a little different, we can probably publish a view that smooths over the differences.

So this doesn't seem like a catastrophic amount of work. We'd have to explain to data scientists why they might have to look in a second place for data.

We could add a source: <normandy|nimbus> extra to the event if we don't want to change the name and feel it's important to disambiguate the origin.

Comment 3

[Michael Cooper [:mythmon]]

I expect that overtime Nimbus will have it's own unique needs for telemetry, and trying to align Normandy and Nimbus long term will produce a rather confused schema. I think we should fork the Normandy schema, and keep just the parts relevant to Nimbus. Then we don't have to worry about cross-compatibility. I'll make these code changes soon.

Comment 4

[Michael Cooper [:mythmon]]

Attachment: Bug 1656568 - Use a seperate category for nimbus event telemetry r?k88hudson

Comment 5

[Kate Hudson :k88hudson]

After discussing this with mythmon we're going to go with with updating the event object v.s. changing the category for now. All nimbus-related events should will a category of nimbus_experiment (which also means updating the exposure event)

Comment 6

[Kate Hudson :k88hudson]

Attachment: Bug 1656568 - Use a specific event object for nimbus events

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/24b91deee352
Use a specific event object for nimbus events r=andreio

Comment 8

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/24b91deee352