Closed Bug 645168 Opened 14 years ago Closed 13 years ago

Enable branch coverage for tegras across all (willing) branches

Categories

(Release Engineering :: General, defect)

ARM
Solaris
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: lsblakk)

References

Details

Attachments

(3 files, 3 obsolete files)

Currently we're only testing m-c and tm.

We saw a build system branch merge kill Android startup.
If we had stable tegra tests tracking that branch, they would have been able to see the bustage on their branch.
(Mobile developers are asking for tree closure when things like this happen, which bumps stability and coverage' priority even more than it already has been.)
Places branch also needs coverage.
Patch incoming for try, places, cedar with the talos suites from bug 648800 comment 6 .

Afterwards I'll keep this bug open to enable unittests, other branches, other talos suites.
Attached patch try,cedar,places talos suites (deleted) — Splinter Review
Attachment #525108 - Flags: review?(lsblakk)
Comment on attachment 525108 [details] [diff] [review]
try,cedar,places talos suites

i'll go take a look at try_parser and update trychooser accordingly if needed
Attachment #525108 - Flags: review?(lsblakk) → review+
FWIW, when I give up using the cedar twig, this change will affect the next person who happens to pick up cedar (as opposed to birch or maple for example).
That's correct.

I'm enabling cedar, try, and places as those are the ones requiring immediate attention.  The plan is to enable all tests everywhere needed, as long as we have enough tegras as to not starve coverage.

I can disable cedar if/when it's no longer needed... when will that be?
(In reply to comment #10)
> That's correct.
> 
> I'm enabling cedar, try, and places as those are the ones requiring immediate
> attention.  The plan is to enable all tests everywhere needed, as long as we
> have enough tegras as to not starve coverage.

OK.

> I can disable cedar if/when it's no longer needed... when will that be?

Not clear yet.  I'll try to ping releng when I decide to abandon it.
Aurora and Beta need coverage.
(In reply to comment #12)
> Aurora and Beta need coverage.

Lukas fixed Aurora in bug 648066.
Depends on: 652604
OS: Android → Solaris
(In reply to comment #11)
> (In reply to comment #10)
...
> > I can disable cedar if/when it's no longer needed... when will that be?
> Not clear yet.  I'll try to ping releng when I decide to abandon it.

Ehsan, you still doing cedar? I saw chatter in newsgroups about you stopping using it for landing other people's patches, but I note you still have it in the bookings page.
Assignee: aki → lsblakk
Attached patch enable tegra talos runs on all project_branches (obsolete) (deleted) — Splinter Review
Attachment #529864 - Flags: review?(aki)
Attached patch enable tegra talos runs on all project_branches (obsolete) (deleted) — Splinter Review
put the tracemonkey v8 suite back in
Attachment #529864 - Attachment is obsolete: true
Attachment #529864 - Flags: review?(aki)
Attachment #529869 - Flags: review?(aki)
Summary: more branch coverage for tegras → Enable branch coverage for tegras across all (willing) branches
Currently the remote-t* suites are enabled on:

mozilla-central
mozilla-beta
mozilla-aurora
mozilla-2.1
places
try
cedar

So we need to establish if the tegra pool can handle having this on by default in the project_branches loop (see attached patch) and then deploy that if we can support that load.
Attachment #529869 - Attachment is obsolete: true
Attachment #529869 - Flags: review?(aki)
Attachment #529871 - Flags: review?(aki)
Attachment #529871 - Attachment is obsolete: true
Attachment #529871 - Flags: review?(aki)
Attachment #529872 - Flags: review?(aki)
Comment on attachment 529872 [details] [diff] [review]
enable tegra talos runs on all project_branches -real-real-real

If/when we want to turn tegra talos on for all branches, this looks good.
Attachment #529872 - Flags: review?(aki) → review+
Philor's asking for tegra unit tests on tracemonkey.
Attachment #536509 - Flags: review?(lsblakk)
Attachment #536509 - Flags: review?(lsblakk) → review+
Bug 663201 is probably more useful than this open-ended bug: it's a specific request with a specific deliverable.

Resolving this; added a patch for mozilla-inbound; we can add more test coverage when requested.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: