Default throttle value on ESR should be 100%

VERIFIED FIXED in 7

Status

Socorro
Backend
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Scoobidiver (away), Assigned: rhelmer)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
There are only about 200K ADU on ESR and the top-1 crasher has only 59 crashes.
The default throttle value should be 100% like in Fennec.

Comment 1

6 years ago
I think this is WFM. AFAIK, we have taken special care when we added ESR support that those are not throttled, and in fact only crashes matching "Firefox" and the "release" channel are throttled at all, except if they contain comments, when they are unthrottled as well.

Comment 2

6 years ago
Also, throttling is done in the processor, which AFAIK is Backend.
Component: Webapp → Backend
QA Contact: webapp → backend

Comment 3

6 years ago
Rob, Lars, can we verify that Firefox ESR isn't throttled?
(Assignee)

Comment 4

6 years ago
(In reply to Robert Kaiser (:kairo@mozilla.com) from comment #3)
> Rob, Lars, can we verify that Firefox ESR isn't throttled?

Throttling is done by the collector (it flags incoming crashes), configured here:
https://github.com/mozilla/socorro/blob/master/scripts/config/collectorconfig.py.dist#L133

ESR releases have ReleaseChannel set to "esr" right? I think we need to add that to the set of ("nightly", "aurora, "beta") at https://github.com/mozilla/socorro/blob/master/scripts/config/collectorconfig.py.dist#L136 otherwise it'll be caught by the general "Firefox" rule and be throttled to 10%

Comment 5

6 years ago
This was discussed in bug 729658, and behavior is as follows:
Things that are 'Firefox' and not aurora, beta, nightly will be throttled at 10%
Things that are Thunderbird or Fennec will go through at 100%

So the desired behavior change is for Firefox esr releases to be processed at 100%?  Once we have confirmation, we can make that change.

Comment 6

6 years ago
Yes, 'esr' channel should be at 100% - I had the logic wrong in my head, obviously, I thought only the 'release*' channel (and now that I think of it, 'default' possibly as well, though I'm still wondering if that's ideal) would be throttled.

Let's add 'esr' to the set of channels not being throttled.

Oh, and can we make those channel names match the beginning of the ReleaseChannel variable? We really don't want nightly-foo to be throttled, we have that nightly-* pattern for development branches.
The rule that currently catches 'nightly' handles only exact matches. A throttle rule that would catch all of 'nightly-*' could look like this:

    ('ReleaseChannel', lambda x: x.startswith('nightly'), 100)

Comment 8

6 years ago
Can we please get the 'esr' channel throttling fixed to be at 100% ASAP?

Updated

6 years ago
Assignee: nobody → rhelmer
Target Milestone: --- → 7
original line:

    ("ReleaseChannel", lambda x: x in ("nightly", "aurora", "beta"), 100),

new version of that line:    

    ("ReleaseChannel", lambda x: x in ("nightly", "aurora", "beta", "esr"), 100),
(Assignee)

Comment 10

6 years ago
(In reply to K Lars Lohn [:lars] [:klohn] from comment #9)
> original line:
> 
>     ("ReleaseChannel", lambda x: x in ("nightly", "aurora", "beta"), 100),
> 
> new version of that line:    
> 
>     ("ReleaseChannel", lambda x: x in ("nightly", "aurora", "beta", "esr"),
> 100),

Master:
https://github.com/mozilla/socorro/pull/525

Stage:
https://github.com/mozilla/socorro/pull/527
Status: NEW → ASSIGNED

Comment 11

6 years ago
Cool, thanks! Let's file a separate followup bug for doing what comment #7 suggests, please, we'd like to have that as well in the future.

Comment 12

6 years ago
Commits pushed to stage at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/c2e6d6037ae2f566816cc48ef1bf0a34f5124d4d
bug 741710 - default ESR throttle level should be 100%

https://github.com/mozilla/socorro/commit/34fd7a9b2ba026c8c6297576097c27ad07b4d015
Merge pull request #527 from rhelmer/stage

bug 741710 - default ESR throttle level should be 100%

Comment 13

6 years ago
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/809a26bf09717103c310de179a21195f99dc6e4d
bug 741710 - default ESR throttle level should be 100%

https://github.com/mozilla/socorro/commit/a103c2d7141457d5eb85bf1e0b7f24f096c64543
Merge pull request #525 from rhelmer/bug741710-default-esr-throttle-level

bug 741710 - default ESR throttle level should be 100%
I spun off bug 748774 for the nightly change.

This one is fixed and will ship today.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
QA verified on prod default throttle values for ESR releases are 100%
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.