Closed Bug 1484304 Opened 6 years ago Closed 6 years ago

8.75 - 9.88% build times / sccache hit rate (windows2012-32) regression on push 5fcbe08fb321a5076f56c380b554c63068efbb00 (Thu Aug 16 2018)

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jmaher, Unassigned)

References

Details

(Keywords: regression)

We have detected a build metrics regression from push:

https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=5fcbe08fb321a5076f56c380b554c63068efbb00

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

Regressions:

 10%  build times windows2012-32 pgo taskcluster-c4.4xlarge     4,406.64 -> 4,842.06
  9%  sccache hit rate windows2012-32 pgo taskcluster-c4.4xlarge0.90 -> 0.98
  9%  build times windows2012-32 pgo taskcluster-c5.4xlarge     3,727.22 -> 4,053.25


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

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 jobs in a pushlog format.

To learn more about the regressing test(s), please see: https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Automated_Performance_Testing_and_Sheriffing/Build_Metrics
:glandium, I see in bug 1483779 that you switched pgo on for clang-cl builds, is this an expected regression in build times?
Flags: needinfo?(mh+mozilla)
Product: Testing → Firefox Build System
Yes, obviously.

When will sccache hit rate tracking know that going up is not a regression?
Flags: needinfo?(mh+mozilla)
:ted.m is one of the primary contacts for the build metrics, he could fix that.
Flags: needinfo?(ted)
(In reply to Joel Maher ( :jmaher ) (UTC+2) from comment #3)
> :ted.m is one of the primary contacts for the build metrics, he could fix
> that.

I'm currently working on bug 1411304, which should address this issue. Canceling the needinfo?.
Flags: needinfo?(ted)
:igoldan, Is this bug resolved  now?
Flags: needinfo?(igoldan)
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #5)
> :igoldan, Is this bug resolved  now?

In terms of build times, not in the slightest of ways. It never returned to the prior baseline, but it did increase constantly on Windows 32bit.
Flags: needinfo?(igoldan)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.