Closed Bug 1550420 Opened 5 years ago Closed 5 years ago

38.17 - 52.26% raptor-tp6m-facebook-geckoview-cold / raptor-tp6m-facebook-geckoview-cold fcp / raptor-tp6m-instagram-geckoview-cold fcp regression on push 3bf1f379d77e758092f84df765e8c8726b0f3b98 (Wed May 8 2019)

Categories

(Testing :: General, defect)

Version 3
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: Bebe, Unassigned)

References

(Regression)

Details

(Keywords: perf, regression)

Raptor has detected a Firefox performance regression from push:

https://hg.mozilla.org/integration/autoland/pushloghtml?changeset=3bf1f379d77e758092f84df765e8c8726b0f3b98

As author of one of the patches included in that push, we need your help to address this regression.

Regressions:

52% raptor-tp6m-facebook-geckoview-cold fcp android-hw-p2-8-0-android-aarch64 opt 853.61 -> 1,299.75
51% raptor-tp6m-instagram-geckoview-cold fcp android-hw-p2-8-0-android-aarch64 opt 312.94 -> 473.75
43% raptor-tp6m-facebook-geckoview-cold android-hw-p2-8-0-android-aarch64 opt 1,102.69 -> 1,576.37
38% raptor-tp6m-instagram-geckoview-cold android-hw-p2-8-0-android-aarch64 opt 544.23 -> 751.95

You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=20864

On the page above you can see an alert for each affected platform as well as a link to a graph showing the history of scores for this test. There is also a link to a Treeherder page showing the Raptor jobs in a pushlog format.

To learn more about the regressing test(s) or reproducing them, please see: https://wiki.mozilla.org/Performance_sheriffing/Raptor

*** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! ***

Our wiki page outlines the common responses and expectations: https://wiki.mozilla.org/Performance_sheriffing/Talos/RegressionBugsHandling

Blocks: 1534654
Flags: needinfo?(yzenevich)
Regressed by: 1549626

Hi Florin,

I'm having hard time seeing how my patch was at fault for this regression. The patch adds a change to a devtools panel functionality that needs to be explicitly triggered, and the devtools/panel open.

Flags: needinfo?(yzenevich) → needinfo?(fstrugariu)

(In reply to Yura Zenevich [:yzen] from comment #1)

Hi Florin,

I'm having hard time seeing how my patch was at fault for this regression. The patch adds a change to a devtools panel functionality that needs to be explicitly triggered, and the devtools/panel open.

I agree with Yura. The autoland pattern hints a pretty likely backout. I suggest we investigate closer on what happened around 3bf1f379d77e and 8fea66166287, by retriggering/backfilling more.

E.g. The improvements landed via 8fea66166287, which backed out bug 1547813. I think that's a likely suspect.

Between May 7 and 8 we added new Android phone to our device pool, which temporarily messed up our perf tests.
Thus, these aren't real perf regressions.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Flags: needinfo?(fstrugariu)
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.