Closed Bug 737581 Opened 12 years ago Closed 12 years ago

run peptest on m-c

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mcote, Assigned: mozilla)

Details

Attachments

(3 files, 1 obsolete file)

We're collecting peptest results from try (currently in a temporary location at http://mrcote.info/peptest/), and we need to add mozilla-central for "real" results.

Given that peptest is a performance analyzer with quite a variance in results, we've changed our idea from a pass/fail scenario to something more like talos. We want the test to run on every push but go green regardless of results. We will collect results and do analyses over time, reporting big jumps to dev.tree-management or elsewhere. But while we're working on that, we want to collect results so we have data to work with.

Eventually we'd like the tests themselves to report to a central server, but for now we have a pulse listener set up that is gathering the results.
Attached patch add peptest to m-c (obsolete) — Splinter Review
As requested, this will turn on peptest for m-c.
I'm not sure if we want it on project branches, inbound, etc.; if this will go orange or red and will be a tree-closing metric we should probably turn it on everywhere applicable.  If not, this may be fine; we may want to hide it or not depending on signal/noise.
Yeah inbound is probably a good idea too. I'll have to see if we want other branches as well.
Assignee: nobody → aki
Attachment #607690 - Attachment is obsolete: true
Attachment #608442 - Flags: review?(armenzg)
Comment on attachment 608442 [details] [diff] [review]
add peptest to m-c, m-i

If ./tests-masters.sh does not complain about the # of builders fill free to land it.
Otherwise, you might want to wait a day or two for me to finish bug 737632.
Attachment #608442 - Flags: review?(armenzg) → review+
Comment on attachment 608442 [details] [diff] [review]
add peptest to m-c, m-i

http://hg.mozilla.org/build/buildbot-configs/rev/f3d8f3976eec

Passes test-masters; running setup-master.py -t and watching preprod to be sure.
Attachment #608442 - Flags: checked-in+
Also, can you force this test to go green, regardless of results? This is, as I understand, how talos works, since the results will be analyzed externally. I can try to add an option to peptest to mangle logs so it doesn't appear to fail, but it's probably easier to do it on buildbot's side...
Handling in mozharness.
Rather than putting the perma-green hack at the end, I'm still going to detect harness/infrastructure failures.  But peptest test failures should go green with this patch.
Attachment #608473 - Flags: review?(mcote)
Comment on attachment 608473 [details] [diff] [review]
peptest perma-green

Perfect, thanks.
Attachment #608473 - Flags: review?(mcote) → review+
Live in production as of 7AM PDT.
Attachment #608743 - Flags: review?(mcote)
Attachment #608743 - Flags: review?(mcote) → review+
Comment on attachment 608743 [details] [diff] [review]
hg cp win7_config.py w764_config.py

http://hg.mozilla.org/build/mozharness/rev/5f44ba08f4be
Attachment #608743 - Flags: checked-in+
Looks to be fixed.
Status: NEW → RESOLVED
Closed: 12 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: