Add-on sync AMO install ping should not skew metrics

RESOLVED WONTFIX

Status

Cloud Services
Firefox Sync: Backend
RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: gps, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sync:addons])

(Reporter)

Description

6 years ago
+++ 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.
(Reporter)

Comment 1

6 years ago
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.
(Reporter)

Comment 2

6 years ago
Justin, et al: What's the priority of this?
Target Milestone: --- → mozilla12
(Reporter)

Comment 3

6 years ago
I didn't meant to set that.
Target Milestone: mozilla12 → ---
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.
(Reporter)

Comment 5

6 years ago
(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.
Will anyone scream if we WONTFIX this?
Severity: major → normal
Flags: needinfo?(fligtar)
Priority: P2 → --
Whiteboard: [sync:addons]
I'm fine with that on our end.
I'm no longer working on AMO/add-ons, so clouserw can speak to this.
Flags: needinfo?(fligtar)
Closing, then. Wil, reopen if this is still a concern.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.