Closed Bug 737458 Opened 13 years ago Closed 13 years ago

add tp5row and cleanup config.py

Categories

(Release Engineering :: General, defect, P1)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: jmaher, Assigned: armenzg)

References

Details

Attachments

(6 files, 2 obsolete files)

add tprow to mozilla-central
(deleted), patch
jmaher
: review+
armenzg
: checked-in+
Details | Diff | Splinter Review
m-c talos.json update (1.0)
(deleted), patch
armenzg
: review+
armenzg
: checked-in+
Details | Diff | Splinter Review
missed adding tprow
(deleted), patch
jmaher
: review+
armenzg
: checked-in+
Details | Diff | Splinter Review
put hack back
(deleted), patch
jmaher
: review+
armenzg
: checked-in+
Details | Diff | Splinter Review
add tprow and disable tpr_responsiveness
(deleted), patch
jmaher
: review+
armenzg
: checked-in+
Details | Diff | Splinter Review
add tpr_responsiveness temporarily for mozilla-central and larch
(deleted), patch
jmaher
: review+
armenzg
: checked-in-
Details | Diff | Splinter Review
we have rolled m-a->m-b and we can cleanup hacks to not run tests on firefox12 m-b. Also we want to have a definition of tp5row ready and in place for turning on.
Attached patch cleanup config.py and add tp5row (1.0) (obsolete) (deleted) — — Splinter Review
Attachment #607576 - Flags: review?(armenzg)
Comment on attachment 607576 [details] [diff] [review] cleanup config.py and add tp5row (1.0) Review of attachment 607576 [details] [diff] [review]: ----------------------------------------------------------------- Want me to land it?
Attachment #607576 - Flags: review?(armenzg) → review+
Assignee: nobody → armenzg
Attached patch add tprow to pine and cleanup (obsolete) (deleted) — — Splinter Review
Attachment #607576 - Attachment is obsolete: true
Attachment #607665 - Flags: review?(jmaher)
After running ./test-masters.sh AssertionError: talos-r3-fed-068 has 1013 builders; limit is 1012 I will have to work on a patch to reduce the # of our builders.
Attachment #607665 - Flags: review?(jmaher) → review+
Priority: -- → P2
Attached patch add tprow to mozilla-central (deleted) — — Splinter Review
This removes some builders from pine and adds the side by side for mozilla-central. This would give me time to finish up my patch in bug 737632 while not blocking you.
Attachment #608395 - Flags: review?(jmaher)
Comment on attachment 608395 [details] [diff] [review] add tprow to mozilla-central Review of attachment 608395 [details] [diff] [review]: ----------------------------------------------------------------- ::: mozilla-tests/config.py @@ +1073,1 @@ > we should still have tp_tests, right?
Attachment #608395 - Flags: review?(jmaher) → review+
tp is already disabled (note the 0): -BRANCHES['mozilla-central']['tp_tests'] = (0, True, TALOS_TP_OPTS, ALL_PLATFORMS)
Comment on attachment 608395 [details] [diff] [review] add tprow to mozilla-central http://hg.mozilla.org/build/buildbot-configs/rev/bdc9918a2055 I will reconfigure the masters in the morning.
Attachment #608395 - Flags: checked-in+
Live in production as of 7AM PDT.
Attached patch m-c talos.json update (1.0) (deleted) — — Splinter Review
Attachment #608708 - Flags: review?(armenzg)
Attachment #608708 - Flags: review?(armenzg) → review+
https://hg.mozilla.org/mozilla-central/rev/dc5c600d5fe3
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Attached patch missed adding tprow (deleted) — — Splinter Review
I tested this locally. It adds it this time.
Attachment #608729 - Flags: review?(jmaher)
Attachment #608729 - Flags: review?(jmaher) → review+
Masters being reconfigured with: http://hg.mozilla.org/build/buildbot-configs/rev/36ce86337cb2
I think we introduced a regression on Beta with the cleanup: https://tbpl.mozilla.org/php/getParsedLog.php?id=10318534&tree=Mozilla-Beta
Attached patch put hack back (deleted) — — Splinter Review
This puts us back into a known state.
Attachment #608801 - Flags: review?(jmaher)
Comment on attachment 608708 [details] [diff] [review] m-c talos.json update (1.0) http://hg.mozilla.org/mozilla-central/rev/dc5c600d5fe3
Attachment #608708 - Flags: checked-in+
Attachment #607665 - Attachment is obsolete: true
Attachment #608801 - Flags: review?(jmaher) → review+
Depends on: 738888
Depends on: 739505
We are going to switch over.
Status: RESOLVED → REOPENED
Priority: P2 → P1
Resolution: FIXED → ---
Attachment #610590 - Flags: review?(jmaher)
Comment on attachment 610590 [details] [diff] [review] add tprow and disable tpr_responsiveness Review of attachment 610590 [details] [diff] [review]: ----------------------------------------------------------------- thanks!
Attachment #610590 - Flags: review?(jmaher) → review+
Comment on attachment 610590 [details] [diff] [review] add tprow and disable tpr_responsiveness http://hg.mozilla.org/build/buildbot-configs/rev/321a3198d97d
Attachment #610590 - Flags: checked-in+
khuey is landing something tomorrow from larch and would like to make sure he doesn't regress anything. Let's add this temporarily.
Attachment #610625 - Flags: review?(jmaher)
Comment on attachment 610625 [details] [diff] [review] add tpr_responsiveness temporarily for mozilla-central and larch Review of attachment 610625 [details] [diff] [review]: ----------------------------------------------------------------- cool, lets not forget about it!
Attachment #610625 - Flags: review?(jmaher) → review+
Comment on attachment 610625 [details] [diff] [review] add tpr_responsiveness temporarily for mozilla-central and larch When have we ever forgotten anything running for months and months?... pfff... never! :)
Attachment #610625 - Flags: checked-in+
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&jobname=tprow
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment on attachment 610625 [details] [diff] [review] add tpr_responsiveness temporarily for mozilla-central and larch http://hg.mozilla.org/build/buildbot-configs/rev/54518db0c6ce khuey gave the acknowledgement of going back to the normal state of affairs through email.
Attachment #610625 - Flags: checked-in+ → checked-in-
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: