Closed
Bug 719859
Opened 13 years ago
Closed 13 years ago
buildbot-custom changes for side by side staging of ignore_first numbers
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: armenzg)
References
Details
Attachments
(2 files, 1 obsolete file)
3.79 KB,
patch
|
armenzg
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
1.63 KB,
patch
|
jmaher
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
as part of the signal_from_noise project we are changing the way we report numbers and will be ignoring the first value instead of the max value. In addition for osx on tp5 we will be collecting the RSS value from inside of pageloader instead of from the python harness.
Reporter | ||
Comment 1•13 years ago
|
||
this is what I am thinking we need to do. After this is successful, there is some other cleanup we can do to the config file as we didn't do it during the last m-c->aurora merge.
Attachment #590223 -
Flags: review?(armenzg)
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 590223 [details] [diff] [review]
buildbot-config changes for ignore_first (1.0)
This looks good.
I see that you had 5 new replacements with --ignoreFirst and a different sample.2.config.
I also see that you prepare v8 in case we enable it on mozilla-inbound on project_branches.
PLATFORMS and --activeTests seems to be the same as the originals.
jmaher, if you need to stage two different talos.zip then we should enable the tests on pine where we could land a talos.zip that should not yet go to mozilla-central. This is in case, you need to run two different talos.zip side by side. Not that it seem you need to in this bug.
Attachment #590223 -
Flags: review?(armenzg) → review+
Reporter | ||
Comment 3•13 years ago
|
||
small updates to change testnames to testname.2
Attachment #590223 -
Attachment is obsolete: true
Attachment #590239 -
Flags: review?(armenzg)
Assignee | ||
Updated•13 years ago
|
Attachment #590239 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 4•13 years ago
|
||
I will deploy this once bug 719169 is fixed and jmaher deploys a new talos.zip with the latest .config.
Assignee | ||
Comment 5•13 years ago
|
||
Comment on attachment 590239 [details] [diff] [review]
buildbot-config changes for ignore_first (1.1)
Landed on "default":
http://hg.mozilla.org/build/buildbot-configs/rev/93012a94d687
Attachment #590239 -
Flags: checked-in+
Assignee | ||
Comment 6•13 years ago
|
||
This is live on production as of 11:40 AM PDT.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•13 years ago
|
||
We're not really done in here, right? (what was my rush? :D)
I see "T" jobs showing up in:
https://tbpl.mozilla.org/?rev=dead9dc86cfa&jobname=talos&noignore=1
I've hidden svg.2.
I will do the rest once they appear. We don't want people to be confused.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 8•13 years ago
|
||
We just learned one new thing.
* If we are adding new talos jobs that require a newer talos.zip then land the talos.zip ahead of time
Some jobs prior to dead9dc86cfa (which points to a newer talos.zip) burned because they were pointing to the older talos.zip.
I will star them properly.
Assignee | ||
Comment 9•13 years ago
|
||
We should land this ASAP so we don't double post for ts_paint and tpaint.
I can see it already happening:
http://graphs.mozilla.org/graph.html#tests=[[83,1,21],[83,1,1],[83,1,12],[83,1,15],[83,1,14],[83,1,13]]&sel=1327350431063.5234,1327357233079&displayrange=7&datatype=running
I'm afraid we could blame the wrong changeset after dead9dc86cfa.
Attachment #590872 -
Flags: review?(jmaher)
Reporter | ||
Comment 10•13 years ago
|
||
Comment on attachment 590872 [details] [diff] [review]
remove suites that we're double posting
Review of attachment 590872 [details] [diff] [review]:
-----------------------------------------------------------------
this looks good. we need to remember to disable only the duplicated tests and/or add back the ts stuff into the new suites.
Attachment #590872 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 11•13 years ago
|
||
Comment on attachment 590872 [details] [diff] [review]
remove suites that we're double posting
Landed and live:
http://hg.mozilla.org/build/buildbot-configs/rev/25ccbf339b4f
Attachment #590872 -
Flags: checked-in+
Assignee | ||
Comment 12•13 years ago
|
||
The follow up work to this bug is being done in bug 724561.
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•