Closed Bug 1465735 Opened 6 years ago Closed 6 years ago

Add Mac code coverage build definition

Categories

(Testing :: Code Coverage, enhancement)

enhancement
Not set
normal

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: marco, Assigned: marco)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

Even if we're not running it on every mozilla-central push, we should at least have support for it.
Attached patch Patch (obsolete) (deleted) — Splinter Review
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #8982162 - Flags: review?(jmaher)
Comment on attachment 8982162 [details] [diff] [review] Patch Review of attachment 8982162 [details] [diff] [review]: ----------------------------------------------------------------- 2 small nits here ::: browser/config/mozconfigs/macosx64/code-coverage-opt @@ +1,1 @@ > +. "$topsrcdir/browser/config/mozconfigs/macosx64/code-coverage" why do we have opt and debug? for win/linux it is debug only ::: taskcluster/ci/build/macosx.yml @@ +288,5 @@ > + symbol: B > + tier: 1 > + worker-type: aws-provisioner-v1/gecko-{level}-b-macosx64 > + worker: > + max-run-time: 36000 10 hours? that seems too high, does this finish in 2 hours?
Attachment #8982162 - Flags: review?(jmaher) → review-
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #2) > Comment on attachment 8982162 [details] [diff] [review] > Patch > > Review of attachment 8982162 [details] [diff] [review]: > ----------------------------------------------------------------- > > 2 small nits here > > ::: browser/config/mozconfigs/macosx64/code-coverage-opt > @@ +1,1 @@ > > +. "$topsrcdir/browser/config/mozconfigs/macosx64/code-coverage" > > why do we have opt and debug? for win/linux it is debug only We've added opt on Linux lately for fuzzing, not sure if they will ever need opt on Mac though. > > ::: taskcluster/ci/build/macosx.yml > @@ +288,5 @@ > > + symbol: B > > + tier: 1 > > + worker-type: aws-provisioner-v1/gecko-{level}-b-macosx64 > > + worker: > > + max-run-time: 36000 > > 10 hours? that seems too high, does this finish in 2 hours? I hadn't noticed, I just copied it from the normal mac debug build. It seems like all builds in macosx.yml have 36000 as max-run-time. Maybe a mistake that got copied and pasted over and over? https://bugzilla.mozilla.org/show_bug.cgi?id=1286075#c212
I don't see osx builds on m-c taking >45 minutes; what do try pushes show for osx ccov builds? I would like to keep this within 20% of normal build times if possible.
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #4) > I don't see osx builds on m-c taking >45 minutes; what do try pushes show > for osx ccov builds? I would like to keep this within 20% of normal build > times if possible. In my latest few try pushes, they took between 19 and 58 minutes. So maybe we can set a limit of 5400 seconds?
5400 is great
Attached patch Patch (deleted) — Splinter Review
I've reduced the max run time to 5400 and removed the code-coverage-opt mozconfig.
Attachment #8982162 - Attachment is obsolete: true
Attachment #8982233 - Flags: review?(jmaher)
Attached patch Only run on try (deleted) — Splinter Review
Attachment #8982234 - Flags: review?(jmaher)
Attachment #8982234 - Flags: review?(jmaher) → review+
Comment on attachment 8982233 [details] [diff] [review] Patch Review of attachment 8982233 [details] [diff] [review]: ----------------------------------------------------------------- thanks
Attachment #8982233 - Flags: review?(jmaher) → review+
Pushed by mcastelluccio@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/803d83c6abc3 Add Mac code coverage build definition. r=jmaher https://hg.mozilla.org/integration/mozilla-inbound/rev/7c22a7591cb9 Allow Mac build to only run on try. r=jmaher
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Depends on: 1466077
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: