1232493 - Version bump to 46 on mozilla-central broke several loop telemetry tests

Status: RESOLVED FIXED

(Hello (Loop) :: General, defect, P1)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://treeherder.mozilla.org/logviewer.html#?job_id=2893688&repo=mozilla-central

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I bumped several histogram expirations up to 47 in https://hg.mozilla.org/mozilla-central/rev/ae37fdb042c0

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Looks there's more trouble on the horizon for Monday's merge when Gecko is bumped to 47:
https://treeherder.mozilla.org/logviewer.html#?job_id=15710756&repo=try

Comment 3

[Ian Bicking (:ianbicking)]

We'd like to extend these measurements further:

- LOOP_TWO_WAY_MEDIA_CONN_LENGTH_1
- LOOP_SHARING_ROOM_URL
- LOOP_ROOM_CREATE
- LOOP_ROOM_DELETE
- LOOP_ROOM_SESSION_WITHCHAT

I'm not sure what the process is for requesting that.


We'd like to deprecate these measurements:

- LOOP_ROOM_CONTEXT_ADD
- LOOP_ROOM_CONTEXT_CLICK
- LOOP_SHARING_STATE_CHANGE_1

Because of the release process for Hello, ideally we'd bump these measurements to 48 so we can remove them from the Hello code and import into mozilla-central before disabling the measurements entirely.

Comment 4

[Mark Banner (:standard8)]

Attachment: Extend expiry time for Loop's telemetry hooks.

Per comment 3, we'd like to extend the expiry times for Loop's telemetry measurements. As the merges are on Monday, we're suggesting a bump of one version for the measurements we're going to depreacte. I'll get a bug or two filed on those in a few minutes.

Pinging both Benjamin and Vladan for review - we only need one of you, but hoping to get one before Monday.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8710988 [details] [diff] [review]
Extend expiry time for Loop's telemetry hooks.

Looks good on Try.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

FYI, the version bump has hit m-c now, so this will start permafailing shortly.

Comment 7

[Carsten Book [:Tomcat]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #6)
> FYI, the version bump has hit m-c now, so this will start permafailing
> shortly.

Benjamin, vladan can we get this review soon ?

Comment 8

[Pulsebot]

https://hg.mozilla.org/mozilla-central/rev/bd8bb6298d90

Comment 9

[Mark Banner (:standard8)]

I filed bug 1242524 on removing the obsolete histograms.