Closed Bug 737458 Opened 12 years ago Closed 12 years ago

add tp5row and cleanup config.py

Categories

(Release Engineering :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jmaher, Assigned: armenzg)

References

Details

Attachments

(6 files, 2 obsolete files)

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.
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) — 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
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.
Attachment #608708 - Flags: review?(armenzg)
Attachment #608708 - Flags: review?(armenzg) → review+
https://hg.mozilla.org/mozilla-central/rev/dc5c600d5fe3
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
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 backSplinter 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: 12 years ago12 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.