Closed Bug 956999 Opened 11 years ago Closed 11 years ago

Please unhide the linux64-valgrind job on TBPL


(Tree Management Graveyard :: Visibility Requests, defect)

Not set


(Not tracked)



(Reporter: n.nethercote, Assigned: emorley)



As soon as bug 956951 (which will deploy relevant TBPL changes) is completed, the linux64-valgrind job will meet the requirements at:

Please may it be unhidden!


> 1) Has an active owner

Yes, me.

> 2) Breakage is expected to be followed by tree closure or backout
> 3) Runs on mozilla-central and all trees that merge into it
> 4) Scheduled on every push
> 5) Easily run on try server

Yes, yes, yes, yes. 

> 6) Outputs failures in a TBPL-starrable format

Yes. When Valgrind finds errors, the "ERROR SUMMARY" line gets highlighted. The
specific errors don't get shown in the summary, so you have to open the log
file to see them.  But the ASan builds are the same, and edmorley indicated
this was acceptable in bug 952605.

(This is just waiting on bug 956951 to put the relevant TBPL changes into

> 7) Low intermittent failure rate
> 8) Must avoid patterns known to cause non deterministic failures

Yes and yes.

> 9) Supports the disabling of individual tests

N/A. However, individual error reports can be suppressed if necessary, which is

> 10) Has sufficient documentation

Yes: has
an entry that points to

> 11) Easy for a dev to run locally

|mach valgrind-test|

> 12) Crashes should produce valid crash stacks

The crash reporter is enabled. I don't think anything else needs doing?
Looks good to me - thank you for driving this!

Valgrind unhidden on {mozilla-central, mozilla-inbound, fx-team, b2g-inbound, try} :-)
Assignee: nobody → emorley
Closed: 11 years ago
Resolution: --- → FIXED
No longer depends on: valgrind-on-tbpl
Product: Webtools → Tree Management
Component: TBPL → Visibility Requests
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.