Closed
Bug 585108
Opened 14 years ago
Closed 14 years ago
We don't need unit test builders for "mozilla-central" and "mozilla-2.0" releases except for win32
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: armenzg, Assigned: armenzg)
References
Details
Attachments
(2 files, 1 obsolete file)
2.05 KB,
patch
|
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
4.26 KB,
patch
|
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
For Beta 3 I noticed that we are still running unit tests on the builders when we should not. This brings confusion to whoever is doing the releases if they don't notice that they should be watching the runs on the minis rather than on the builders. We did this switch on bug 570816.
Attachment #463593 -
Flags: review?(rail)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → armenzg
Status: NEW → ASSIGNED
Priority: -- → P2
Comment 1•14 years ago
|
||
Comment on attachment 463593 [details] [diff] [review] disable unit tests on releases for "mozilla-central" and "mozilla-2.0" Looks good. Could you also update mozilla2-staging release configs?
Attachment #463593 -
Flags: review?(rail) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 463593 [details] [diff] [review] disable unit tests on releases for "mozilla-central" and "mozilla-2.0" http://hg.mozilla.org/build/buildbot-configs/rev/901d79d2447b Added changes for staging. pm reconfigured.
Attachment #463593 -
Flags: checked-in+
Assignee | ||
Comment 3•14 years ago
|
||
As I just noticed, we should still run the unit tests on the builders. I will provide another patch once I have it working.
Summary: We don't need unit test builders for "mozilla-central" and "mozilla-2.0" releases → We don't need unit test builders for "mozilla-central" and "mozilla-2.0" releases except for win32
Assignee | ||
Comment 4•14 years ago
|
||
I switched to use [] instead of ().
Attachment #464550 -
Flags: review?(rail)
Assignee | ||
Comment 5•14 years ago
|
||
correct patch.
Attachment #464550 -
Attachment is obsolete: true
Attachment #464555 -
Flags: review?(rail)
Attachment #464550 -
Flags: review?(rail)
Comment 6•14 years ago
|
||
Comment on attachment 464555 [details] [diff] [review] we still run unit tests on the builders for win32 A nit. Use ('win32',) instead of ['win32'] to be consistent with the previous type (tuple).
Attachment #464555 -
Flags: review?(rail) → review+
Assignee | ||
Comment 7•14 years ago
|
||
Comment on attachment 464555 [details] [diff] [review] we still run unit tests on the builders for win32 http://hg.mozilla.org/build/buildbot-configs/rev/fbb1ffc3f583 Nit taken. production-master reconfigured.
Attachment #464555 -
Flags: checked-in+
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Comment 8•14 years ago
|
||
So this is currently still turned on for 4.0b4 as per http://hg.mozilla.org/build/buildbot-configs/file/55218bd97eef/mozilla2/release-firefox-mozilla-2.0.py -- not sure if there's a new bug tracking turning it off again or not.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 9•14 years ago
|
||
(In reply to comment #8) > So this is currently still turned on for 4.0b4 as per > http://hg.mozilla.org/build/buildbot-configs/file/55218bd97eef/mozilla2/release-firefox-mozilla-2.0.py > -- not sure if there's a new bug tracking turning it off again or not. That file shows only win32 in unittestPlatforms, which is what this bug wants: unittestPlatforms = ('win32',)
Assignee | ||
Comment 10•14 years ago
|
||
This is correct as Ben indicates.
Updated•14 years ago
|
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 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
•