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)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: armenzg, Assigned: armenzg)

References

Details

Attachments

(2 files, 1 obsolete file)

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: nobody → armenzg
Status: NEW → ASSIGNED
Priority: -- → P2
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+
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+
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
I switched to use [] instead of ().
Attachment #464550 - Flags: review?(rail)
correct patch.
Attachment #464550 - Attachment is obsolete: true
Attachment #464555 - Flags: review?(rail)
Attachment #464550 - Flags: review?(rail)
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+
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+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Blocks: 570816
No longer depends on: 570816
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 → ---
(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',)
This is correct as Ben indicates.
Status: REOPENED → RESOLVED
Closed: 14 years ago14 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: