Closed Bug 820592 Opened 12 years ago Closed 11 years ago

buildbot config changes to support Android x86 builds

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

x86
macOS
task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kmoir, Assigned: kmoir)

References

Details

Attachments

(6 files, 5 obsolete files)

11.34 KB, patch
armenzg
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
2.12 KB, patch
armenzg
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
1.46 KB, patch
armenzg
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
1.80 MB, text/html
Details
11.82 KB, patch
armenzg
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
2.29 KB, patch
bhearsum
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
      No description provided.
Assignee: nobody → kmoir
Summary: buildbot config changes to support Android x86 → buildbot config changes to support Android x86 builds
Attached patch buildbotcustom patch (obsolete) — Splinter Review
Attached patch buildbotcustom patch (obsolete) — Splinter Review
Attachment #691354 - Attachment is obsolete: true
Attachment #691433 - Flags: review?(armenzg)
Attachment #691355 - Flags: review?(armenzg)
Attachment #691433 - Flags: review?(armenzg) → review+
Attachment #691355 - Flags: review?(armenzg) → review+
Comment on attachment 691433 [details] [diff] [review]
buildbotcustom patch

I think that you attached twice the same patch.

On another not you will need a puppet change to update Buildslaves.py on the masters.
Attachment #691433 - Flags: review+ → review-
s/not/note/
Attachment #691433 - Attachment is obsolete: true
Comment on attachment 691458 [details] [diff] [review]
buildbotcustom patch

sorry for the wrong patch the first time :-(
Attachment #691458 - Flags: review?(armenzg)
Comment on attachment 691458 [details] [diff] [review]
buildbotcustom patch

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

No worries!
Attachment #691458 - Flags: review?(armenzg) → review+
Attached patch patch for puppetSplinter Review
Attachment #691516 - Flags: review?(armenzg)
Attachment #691516 - Flags: review?(armenzg) → review+
Attachment #691516 - Flags: checked-in+
Attachment #691458 - Flags: checked-in+
Attachment #691355 - Flags: checked-in+
In production
Not sure if I should be building with a different revision to avoid this compile error.
Brad could you look at the compile log I just attached for the android x86 release build?  Not sure what needs to be changed to avoid the compile error, if we should be building with a a different revision or something else.
as described in bug 822653

as well as add them for other streams
Attachment #693408 - Flags: review?(armenzg)
Blocks: 822653
Attachment #693408 - Flags: review?(armenzg) → review+
Attachment #693408 - Flags: checked-in+
Attachment #693362 - Attachment mime type: text/plain → text/html
The issue in comment 12 is bug 791775 which hasn't made it to release yet, will test in beta.
I was able to build a beta android x86 build, no compile error.
Blocks: 823114
I was able to build a android x86 release build today, no compile error. I'll attach the patches for a release build for android x86, even though we won't run this in production for a while.
Attached patch patch for release configs (obsolete) — Splinter Review
there are also some changes needed for aurora
Coop suggested enabling Android x86 release builds as a preference rather than implemented by default, still thinking about how to implement that.
Attachment #700542 - Attachment is obsolete: true
Attachment #709073 - Flags: review?(armenzg)
Attachment #709073 - Flags: review?(armenzg) → review+
Comment on attachment 709073 [details] [diff] [review]
unbitrotten patch for android x86 release configs

joduinn suggested landing this in since we can build it but not push it to the release directories until a testing story is in place
Attachment #709073 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
In production
Comment on attachment 709073 [details] [diff] [review]
unbitrotten patch for android x86 release configs

Reverted the fix due to bustage during 19b6

android x86 is in the mozconfig list but missing in the platforms list
Attachment #709073 - Flags: checked-in+ → checked-in-
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Added missing platform bits
Attachment #709073 - Attachment is obsolete: true
Attachment #713133 - Flags: review?(bhearsum)
Comment on attachment 713133 [details] [diff] [review]
patch for android x86 release configs

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

We don't want android-x86 in any release configs until it's actually shipping. I think that means we only want the config.py part for now?
Attachment #713133 - Flags: review?(bhearsum) → review-
joduinn asked me to include it in the configs and have it built, but just not push it to mirrors or enable on google play.
Attachment #713133 - Attachment is obsolete: true
Attachment #713921 - Flags: review?(bhearsum)
Attachment #713921 - Flags: review?(bhearsum) → review+
Attachment #713921 - Flags: checked-in+
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: