Closed Bug 1058888 Opened 8 years ago Closed 8 years ago

enable android 2.3 release/test builders on esr31 branch

Categories

(Release Engineering :: Release Automation: Other, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kmoir, Unassigned)

References

Details

Attachments

(2 files)

buildbot configs to support the work in bug 1040319
Depends on: 1059047
We can use the the build slave in bug 1059047 and the test slave in bug 1058734 for the dev-master testing.
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.
I put my patches for this in bug 1040319 last week so I think we can close this.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1040319
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 → ---
Summary: enable android 2.3 release builders on esr31 branch → enable android 2.3 release/test builders on esr31 branch
Attached patch bug1058888.patchSplinter Review
Attachment #8488160 - Flags: review?(pmoore)
Attached file bug1058888builder.diff
builder diff
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+
Comment on attachment 8488160 [details] [diff] [review]
bug1058888.patch

and fixed whitespace
Attachment #8488160 - Flags: checked-in+
In production
Verified that tests ran
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.