Please add additional platforms to the per-push builds on birch/b2g-inbound

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
4 months ago

People

(Reporter: RyanVM, Assigned: hwine)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
We've had two recent instances of bustage resulting from merging birch to m-c where we didn't catch it prior due to it only occurring on platforms we intentionally don't run tests on. While we knew that the risk of this happening was also present, it has gotten worse now that birch is getting more heavily used.

Therefore, please add the following builds/tests to be run per-checkin on birch/b2g-inbound:
* OSX 10.8 debug
* Win8 debug
* Android 2.2 opt

This can be considered our last attempt at a reduced testing schedule before I think we'll be at the point of throwing in the towel and just making it 100% the same as inbound.
Component: Release Engineering: Platform Support → Release Engineering: Automation (General)
QA Contact: coop → catlee
(Reporter)

Comment 1

5 years ago
Can we get this done before b2g-inbound goes live please?

Comment 2

5 years ago
FWIW, I think we share enough code with Android that we should always run all android tests/builds on birch. We can probably cut nearly everything that isn't b2g for gaia pushbot pushes though. (if it's possible to make decisions on that)
(Reporter)

Comment 3

5 years ago
(In reply to Michael Wu [:mwu] from comment #2)
> We can probably cut nearly everything
> that isn't b2g for gaia pushbot pushes though. (if it's possible to make
> decisions on that)

That we already do.
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #1)
> Can we get this done before b2g-inbound goes live please?

Yes, exactly why I added this bug as blocking bug#875989 :-)
(Assignee)

Comment 5

5 years ago
Created attachment 784106 [details] [diff] [review]
don't delete the tests that are now wanted

passes test-masters.sh
Assignee: nobody → hwine
Status: NEW → ASSIGNED
Attachment #784106 - Flags: review?(bugspam.Callek)
Comment on attachment 784106 [details] [diff] [review]
don't delete the tests that are now wanted

Review of attachment 784106 [details] [diff] [review]:
-----------------------------------------------------------------

This does as c#0 asks for, however I wonder if we really want to omit android armv6 and similar if we are actually including android itself. That said, I'll leave others to debate those points.
Attachment #784106 - Flags: review?(bugspam.Callek) → review+
(Reporter)

Comment 7

5 years ago
Per comment #2, the B2G team would like all Android builds running on b2g-inbound going forward.
(Assignee)

Comment 8

5 years ago
Created attachment 784110 [details] [diff] [review]
don't delete the tests that are now wanted

updated per comment 7 to include all android tests

tests in progress, but "should pass"
Attachment #784106 - Attachment is obsolete: true
Attachment #784110 - Flags: review?(bugspam.Callek)
Comment on attachment 784110 [details] [diff] [review]
don't delete the tests that are now wanted

Review of attachment 784110 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozilla/config.py
@@ +2127,5 @@
>          del BRANCHES[branch]['platforms']['linux64-dbg-st-an']
>  
>  # B2G's INBOUND
>  for b in ('b2g-inbound', 'birch',):
> +    # per bug 898439, stop disabling osx, win8, android *

This comment will just confuse us in a month, please drop it.
Attachment #784110 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 10

5 years ago
Comment on attachment 784110 [details] [diff] [review]
don't delete the tests that are now wanted

https://hg.mozilla.org/build/buildbot-configs/rev/9744cde7c313
Attachment #784110 - Flags: checked-in+
(Assignee)

Comment 11

5 years ago
in production, closing
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.