Closed Bug 719859 Opened 12 years ago Closed 12 years ago

buildbot-custom changes for side by side staging of ignore_first numbers

Categories

(Release Engineering :: General, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jmaher, Assigned: armenzg)

References

Details

Attachments

(2 files, 1 obsolete file)

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.
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)
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+
small updates to change testnames to testname.2
Attachment #590223 - Attachment is obsolete: true
Attachment #590239 - Flags: review?(armenzg)
Attachment #590239 - Flags: review?(armenzg) → review+
I will deploy this once bug 719169 is fixed and jmaher deploys a new talos.zip with the latest .config.
Assignee: nobody → armenzg
Depends on: 719169
Priority: -- → P2
Depends on: 719946
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+
This is live on production as of 11:40 AM PDT.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
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 → ---
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.
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)
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+
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+
The follow up work to this bug is being done in bug 724561.
Blocks: 724561
Status: REOPENED → RESOLVED
Closed: 12 years ago12 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: