717823 - Add-on sync AMO install ping should not skew metrics

Status: RESOLVED WONTFIX

(Firefox :: Sync, defect)

Description

[Gregory Szorc [:gps]]

+++ This bug was initially created as a clone of Bug #708134 +++

This is part 2 of bug 708134 to address the AMO install ping Sync will incur. See comments in that bug for details.

Comment 1

[Gregory Szorc [:gps]]

Per fligtar from bug 708134:

---

The metadata ping and install ping are pretty much the same, and the way we tell them apart for metrics purposes is that the metadata ping will have the default theme's GUID in it, while an install ping will only have the add-on that was just installed. Explicitly differentiating them would be helpful, especially for other apps like mobile which doesn't have a default theme so we can't tell the difference at all.

My main concern regarding install sources was with the API download URL, which should indicate it was downloaded via sync and not the user. A parameter on the metadata install lookup would be interesting but isn't necessary -- we don't currently use that ping for anything metrics-wise and simply filter them out of the daily metadata background checks.

Comment 2

[Gregory Szorc [:gps]]

Justin, et al: What's the priority of this?

Comment 3

[Gregory Szorc [:gps]]

I didn't meant to set that.

Comment 4

[Justin Scott [:fligtar]]

As I understand it, paragraph 2 of comment #1 was already implemented, so this bug is about paragraph 1? If so, the priority is low.

Comment 5

[Gregory Szorc [:gps]]

(In reply to Justin Scott [:fligtar] from comment #4)
> As I understand it, paragraph 2 of comment #1 was already implemented, so
> this bug is about paragraph 1? If so, the priority is low.

Correct, the download URL was changed to include source=sync.

Comment 6

[Richard Newman [:rnewman]]

Will anyone scream if we WONTFIX this?

Comment 7

[Daniel Einspanjer [:dre] [:deinspanjer]]

I'm fine with that on our end.

Comment 8

[Justin Scott [:fligtar]]

I'm no longer working on AMO/add-ons, so clouserw can speak to this.

Comment 9

[Richard Newman [:rnewman]]

Closing, then. Wil, reopen if this is still a concern.