Closed
Bug 737581
Opened 12 years ago
Closed 12 years ago
run peptest on m-c
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcote, Assigned: mozilla)
Details
Attachments
(3 files, 1 obsolete file)
1.05 KB,
patch
|
armenzg
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
1.20 KB,
patch
|
mcote
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
157 bytes,
patch
|
mcote
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•12 years ago
|
||
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.
Reporter | ||
Comment 2•12 years ago
|
||
Yeah inbound is probably a good idea too. I'll have to see if we want other branches as well.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → aki
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #607690 -
Attachment is obsolete: true
Attachment #608442 -
Flags: review?(armenzg)
Comment 4•12 years ago
|
||
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+
Assignee | ||
Comment 5•12 years ago
|
||
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+
Reporter | ||
Comment 6•12 years ago
|
||
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...
Assignee | ||
Comment 7•12 years ago
|
||
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)
Reporter | ||
Comment 8•12 years ago
|
||
Comment on attachment 608473 [details] [diff] [review] peptest perma-green Perfect, thanks.
Attachment #608473 -
Flags: review?(mcote) → review+
Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 608473 [details] [diff] [review] peptest perma-green http://hg.mozilla.org/build/mozharness/rev/93c2b774fd38
Attachment #608473 -
Flags: checked-in+
Comment 10•12 years ago
|
||
Live in production as of 7AM PDT.
Assignee | ||
Comment 11•12 years ago
|
||
Attachment #608743 -
Flags: review?(mcote)
Reporter | ||
Updated•12 years ago
|
Attachment #608743 -
Flags: review?(mcote) → review+
Assignee | ||
Comment 12•12 years ago
|
||
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+
Assignee | ||
Comment 13•12 years ago
|
||
Looks to be fixed.
Status: NEW → RESOLVED
Closed: 12 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
•