Closed
Bug 645168
Opened 13 years ago
Closed 13 years ago
Enable branch coverage for tegras across all (willing) branches
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: lsblakk)
References
Details
Attachments
(3 files, 3 obsolete files)
3.91 KB,
patch
|
lsblakk
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
2.94 KB,
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
1.04 KB,
patch
|
lsblakk
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•13 years ago
|
||
(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.)
Comment 4•13 years ago
|
||
Places branch also needs coverage.
Reporter | ||
Comment 5•13 years ago
|
||
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.
Reporter | ||
Comment 6•13 years ago
|
||
Attachment #525108 -
Flags: review?(lsblakk)
Assignee | ||
Comment 7•13 years ago
|
||
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+
Reporter | ||
Comment 8•13 years ago
|
||
Comment on attachment 525108 [details] [diff] [review] try,cedar,places talos suites http://hg.mozilla.org/build/buildbot-configs/rev/97c9140d9c8c
Attachment #525108 -
Flags: checked-in+
Comment 9•13 years ago
|
||
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).
Reporter | ||
Comment 10•13 years ago
|
||
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?
Comment 11•13 years ago
|
||
(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.
Reporter | ||
Comment 12•13 years ago
|
||
Aurora and Beta need coverage.
Reporter | ||
Comment 13•13 years ago
|
||
(In reply to comment #12) > Aurora and Beta need coverage. Lukas fixed Aurora in bug 648066.
Assignee | ||
Updated•13 years ago
|
Blocks: try_enhancements
Comment 14•13 years ago
|
||
(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 | ||
Updated•13 years ago
|
Assignee: aki → lsblakk
Assignee | ||
Comment 15•13 years ago
|
||
Attachment #529864 -
Flags: review?(aki)
Assignee | ||
Comment 16•13 years ago
|
||
put the tracemonkey v8 suite back in
Attachment #529864 -
Attachment is obsolete: true
Attachment #529864 -
Flags: review?(aki)
Attachment #529869 -
Flags: review?(aki)
Assignee | ||
Updated•13 years ago
|
Summary: more branch coverage for tegras → Enable branch coverage for tegras across all (willing) branches
Assignee | ||
Comment 17•13 years ago
|
||
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.
Assignee | ||
Comment 18•13 years ago
|
||
Attachment #529869 -
Attachment is obsolete: true
Attachment #529869 -
Flags: review?(aki)
Attachment #529871 -
Flags: review?(aki)
Assignee | ||
Comment 19•13 years ago
|
||
Attachment #529871 -
Attachment is obsolete: true
Attachment #529871 -
Flags: review?(aki)
Attachment #529872 -
Flags: review?(aki)
Reporter | ||
Comment 20•13 years ago
|
||
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+
Reporter | ||
Comment 21•13 years ago
|
||
Philor's asking for tegra unit tests on tracemonkey.
Attachment #536509 -
Flags: review?(lsblakk)
Assignee | ||
Updated•13 years ago
|
Attachment #536509 -
Flags: review?(lsblakk) → review+
Reporter | ||
Comment 22•13 years ago
|
||
Comment on attachment 536509 [details] [diff] [review] enable tracemonkey tegra unit tests http://hg.mozilla.org/build/buildbot-configs/rev/3b22c458b3eb
Attachment #536509 -
Flags: checked-in+
Reporter | ||
Comment 23•13 years ago
|
||
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
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•