enable android 2.3 release/test builders on esr31 branch

RESOLVED FIXED

Status

Release Engineering
Release Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: kmoir, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
buildbot configs to support the work in bug 1040319
(Reporter)

Updated

3 years ago
Depends on: 1059047
(Reporter)

Comment 1

3 years ago
We can use the the build slave in bug 1059047 and the test slave in bug 1058734 for the dev-master testing.
(Reporter)

Comment 2

3 years ago
I've installed a new dev-master here that I can use for testing. /builds/buildbot/kmoir/build3/  We can integrate my patches into Pete's dev-master once his other patches are landed.
(Reporter)

Comment 3

3 years ago
I put my patches for this in bug 1040319 last week so I think we can close this.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1040319
(Reporter)

Comment 4

3 years ago
There are nightly builder for this branch, however the test jobs on emulators are needed (as confirmed in today's mobile meeting)
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
(Reporter)

Updated

3 years ago
Summary: enable android 2.3 release builders on esr31 branch → enable android 2.3 release/test builders on esr31 branch
(Reporter)

Comment 5

3 years ago
Created attachment 8488160 [details] [diff] [review]
bug1058888.patch
Attachment #8488160 - Flags: review?(pmoore)
(Reporter)

Comment 6

3 years ago
Created attachment 8488163 [details]
bug1058888builder.diff

builder diff
(Reporter)

Comment 7

3 years ago
diff from enabling the android esr31 test jobs on emulators, not tegras.  As a note, right now we just have esr31 nighties, my thinking is that the unittest sendchange from the nightly will start the test jobs.  Talos jobs will not be run.
Sorry Kim I've been on PTO for a few days. I was back this afternoon but did not get around to looking deeply into this (although superficially it looks very good) - I will review first thing in the morning. Thanks for taking care of this!
Pete
Comment on attachment 8488160 [details] [diff] [review]
bug1058888.patch

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

Looks pretty good to me Kim! Thanks for this. The diffs look good, and the code changes too, and I ran mobile_config.py before and after, and the diff there looks sane too.

::: mozilla-tests/mobile_config.py
@@ +1472,4 @@
>          'tegra_android-armv6': {
>              'opt_unittest_suites': [],
>              'debug_unittest_suites': [],
> +        },       

some trailing whitespace
Attachment #8488160 - Flags: review?(pmoore) → review+
(Reporter)

Comment 10

3 years ago
Comment on attachment 8488160 [details] [diff] [review]
bug1058888.patch

and fixed whitespace
Attachment #8488160 - Flags: checked-in+
In production
(Reporter)

Comment 12

3 years ago
Verified that tests ran
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.