Closed Bug 1503495 Opened 7 years ago Closed 7 years ago

63.91 - 78.34% build times (windows-mingw32) regression on push c06d1f31c0914b09091a7e2d531c782607504d0e (Thu Oct 18 2018)

Categories

(Firefox Build System :: General: Unsupported Platforms, defect, P5)

60 Branch
x86
Windows
defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: igoldan, Unassigned)

References

Details

(Keywords: backlog-deferred, regression)

We have detected a build metrics regression from push: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=d30c058c4b94021659054523f759a94aee726d3c&tochange=c06d1f31c0914b09091a7e2d531c782607504d0e As author of one of the patches included in that push, we need your help to address this regression. Regressions: 78% build times windows-mingw32 all 32 clang debug taskcluster-c5.4xlarge 536.77 -> 957.27 64% build times windows-mingw32 all 32 clang debug taskcluster-m5.4xlarge 563.69 -> 923.92 You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=17244 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
Component: General → General: Unsupported Platforms
Product: Testing → Firefox Build System
Flags: needinfo?(tom)
Sorry for being this late. Treeherder/Perfherder just didn't catch this on time.
If I've identified the wrong bug, do mention that. I had to look for it in this big merge patch [1] and it seemed the most related to the issue we have. [1] https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=8d820a4fe6e63e80cb05e5c27351b7a3fdf57577&tochange=f2d7d033c14fe7a9e94c6c5ddec4e5dcf165be7c
Resolving this as invalid - the build time increase is expected. Before we weren't producing debug symbols; now we are. Unfortunately that doubled the time; but 20 minutes isn't prohibitive. Thanks for noticing though!
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(tom)
Resolution: --- → INVALID
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.