Closed Bug 787241 Opened 12 years ago Closed 12 years ago

enable android armv6 on mozilla-release (dep + releases) when fx 16 merges in

Categories

(Release Engineering :: General, defect, P3)

ARM
Android
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: kmoir)

References

Details

(Whiteboard: [merge day])

Attachments

(4 files, 1 obsolete file)

This will touch mozilla/config.py, mozilla/release-fennec-mozilla-release.py, mozilla/staging_release-fennec-mozilla-release.py
Priority: -- → P3
Depends on: 775232
Blocks: 782990
Assignee: nobody → kmoir
Attached patch patchSplinter Review
Attachment #668168 - Flags: feedback?
Attachment #668168 - Flags: feedback? → feedback?(bhearsum)
I haven't done this before, not sure if I captured all the changed required.
Comment on attachment 668168 [details] [diff] [review]
patch

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

Yup, this looks good to me. Thanks for updating the staging config too!
Attachment #668168 - Flags: feedback?(bhearsum) → feedback+
I'm going to ride along this fix here, because I saw the issue while I was reviewing :). We set MOZ_SYMBOLS_EXTRA_BUILDID unnecessarily for a whole bunch of platforms. Android and Android armv6 don't collide in terms of namespace on the symbol server:
  inflating: fennec-17.0a2-Android-20121004042009-arm-android-armv6-mozilla-aurora-symbols.txt  
  inflating: fennec-17.0a2-Android-20121004042009-arm-mozilla-aurora-symbols.txt  

Further, mozilla-central is the authoritative "18.0" right now. It doesn't need anything extra set, only project branches. Same thing goes for aurora, beta, and release -- they're authoritative for their respective versions, so don't need to set anything.

Also, I found a bug in the noion setup that probably caused us to lose symbol information for either armv7 or noion (depending on who uploaded first) on project branches.
Attachment #668395 - Flags: review?(rail)
Attachment #668395 - Flags: review?(rail)
Ted reminded me that the reason armv6 is uploading to a different place right is because this is set, let's not break that :).
Attachment #668395 - Attachment is obsolete: true
Attachment #668404 - Flags: review?(rail)
Attachment #668404 - Flags: review?(rail) → review+
Attachment #668404 - Flags: checked-in+
Attachment #668168 - Flags: checked-in+
in production
Attachment #668516 - Flags: review?(bhearsum)
Attachment #668524 - Flags: review?(bhearsum)
Attachment #668524 - Flags: review?(bhearsum) → review+
in production
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment on attachment 668516 [details] [diff] [review]
patch for mozilla-tests

patch was superseded (correctly in the other patch)
Attachment #668516 - Flags: review?(bhearsum) → review-
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: