Move linux code coverage to deep phase, saving 20m in test phase for linux builder

VERIFIED FIXED in Q3 11 - Serrano

Status

defect
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: brbaker, Assigned: brbaker)

Tracking

unspecified
Q3 11 - Serrano
All
Linux
Bug Flags:
in-testsuite -
flashplayer-qrb +
flashplayer-triage +

Details

(Whiteboard: buildbot, code-coverage)

Attachments

(2 attachments)

We do not need to be compiling and running full code coverage on every build that we make. Move the linux code-coverage work to the linux deep phase build slave.

Currently the linux build slave spends almost 3 minutes compiling the code coverage shell in the compile phase, and an additional 17+ minutes generating code coverage results in the test phase.
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
OS: All → Linux
Summary: [buildbot] Move code-coverage compilation and measurement to deep phase → Move linux code coverage to deep phase, saving 20m in test phase for linux builder
Whiteboard: build-speedup → build-speedup, buildbot
Assignee: nobody → brbaker
Status: NEW → ASSIGNED
Flags: flashplayer-qrb? → flashplayer-qrb+
Target Milestone: --- → flash10.2
+ remove code coverage scripts and knowledge from the "linux" build slave
+ add code coverage to the linux-deep build slave, mimic exactly what is happening in the windows-deep code coverage
Attachment #444086 - Flags: superreview?(cpeyer)
Attachment #444086 - Flags: review?(jsudduth)
Move linux code coverage out of phase1 and add it to the deep phase
Attachment #444088 - Flags: superreview?(cpeyer)
Attachment #444088 - Flags: review?(jsudduth)
Attachment #444086 - Flags: review?(jsudduth) → review+
Attachment #444088 - Flags: review?(jsudduth) → review+
Attachment #444086 - Flags: superreview?(cpeyer) → superreview+
Attachment #444088 - Flags: superreview?(cpeyer) → superreview+
Patches pushed
tr-argo -> 4034:51157c8bcb80
tr -> 4665:669e8ded3030, 4666:8839a33b6012
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Whiteboard: build-speedup, buildbot → buildbot
Whiteboard: buildbot → buildbot, code-coverage
You need to log in before you can comment on or make changes to this bug.