Separate MOAR Hello from non-Hello calls in Telemetry

RESOLVED FIXED in Firefox 46

Status

()

Product:
Core
Component:
WebRTC
Importance:
P1
normal
Rank:
7
Status:
RESOLVED FIXED
Reported:
3 years ago
Modified:
3 years ago

People

(Reporter: abr, Assigned: jesup)

Tracking

(Depends on: 1 bug)

Version:
Trunk
Target:
mozilla46
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 affected, firefox46 fixed)

Details

Attachments

(1 attachment)

MozReview Request: Bug 1198345 - Split moar Hello Telemetry values from general WebRTC.
40 bytes, text/x-review-board-request
jesup
: review+
Details
(Reporter)

Description

•
3 years ago 
+++ This bug was initially created as a clone of Bug #1188376 +++

Since Hello is much more controlled (especially in pre-release versions, but also release), we should get 'cleaner' stats unaffected by people trying services that may not have TURN servers, people running automated tests, etc.

This is a follow-on bug to separate LOOP_ICE_LATE_TRICKLE_ARRIVAL_TIME and LOOP_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME from WEBRTC_ICE_LATE_TRICKLE_ARRIVAL_TIME and WEBRTC_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME.

In particular, this will help troubleshoot some ICE oddities that we're seeing with the Hello service. Uplifting this split to beta, like we did with bug 1188376, would be extremely useful.
 (Assignee)

Updated

•
3 years ago
Assignee: nobody → rjesup
backlog: --- → webrtc/webaudio+
Rank: 10
Rank: 10 → 7
 (Assignee)

Comment 1

•
3 years ago 
jib - would this be easy to look at?
Flags: needinfo?(jib)

Comment 2

•
3 years ago 
Created attachment 8697468 [details]
MozReview Request: Bug 1198345 - Split moar Hello Telemetry values from general WebRTC.

Bug 1198345 - Split moar Hello Telemetry values from general WebRTC.
Attachment #8697468 - Flags: review?(rjesup)
Flags: needinfo?(jib)
 (Assignee)

Comment 3

•
3 years ago 
Comment on attachment 8697468 [details]
MozReview Request: Bug 1198345 - Split moar Hello Telemetry values from general WebRTC.

https://reviewboard.mozilla.org/r/27615/#review24877
Attachment #8697468 - Flags: review?(rjesup) → review+
Keywords: checkin-needed

Comment 5

•
3 years ago
bugherder 
https://hg.mozilla.org/mozilla-central/rev/6782cf1a3da0
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.