564174 - Split the SPARC compilation up between both build slaves

Status: VERIFIED FIXED

(Tamarin Graveyard :: Build Config, defect)

Description

[Brent Baker]

We could trim 6 minutes off of the SPARC compilation time by splitting the compile phase up between both of the solaris-sparc build slaves.

This will require a little more work than just juggling build config (will require local build-check and upload scripts).

Second SPARC machine is slower, but moving a couple of compilation steps to it will be a net gain in reducing the SPARC compilation phase

Build Step              Current asteamsol4  asteamsol3  Difference
Source_Clean            54      54          63
Source_Clone            126     126         118
Source_Update           1       1           1
BB_SLAVEUpdate          0       0           0
Compile_builtin.abc     31      31          38
Build_Release           188                 244
Build_Release-wordcode  178                 236
Build_Debug             208     208        
Build_ReleaseDebugger   200     200        
Build_DebugDebugger     225     225        
Build_Check             0       0           0
Upload_ASTEAM           5       5           5
Upload_MOZILLA          54      54
                        21.17   15.07       11.75       6.10

Comment 2

[Brent Baker]

This /might/ not be worth the work to split up. We used to have a time limit of 20 minutes for the compile phase and this is just over that mark. There are about another 3 build slaves that are in the 19-20 minute range so even doing this work will not be a huge win... not to say that shaving off a minute or so is not a win, just not huge like recent build time savings

Comment 3

[James Sudduth]

Should I go ahead and fix this in light of comment #2?

Comment 4

[Edwin Smith]

its a question of IF vs WHEN.

IF: of course. we need to be striving to lower build times.  (20 minutes now, 10 later, and so on).

WHEN: Comment #2 implies there could be something more important to do before this.  If so, sure, I have no opinion. If you press me for an opinion: unless there is a blocking issue, I'd just go for this now so we dont have to worry about it later.

Comment 5

[Brent Baker]

Closing this bug since SPARC build/testing was optimized in bug #623609

Fixed via changeset 5802:c82ece99d8e0 and 5814:e270f9eeca30