Closed Bug 718777 Opened 12 years ago Closed 12 years ago

Tracking bug for build and release of Firefox/Fennec 11.0b1

Categories

(Release Engineering :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

(Whiteboard: [releases])

Attachments

(8 files, 7 obsolete files)

3.62 KB, patch
mozilla
: review+
Details | Diff | Splinter Review
3.76 KB, patch
mozilla
: review+
Details | Diff | Splinter Review
5.32 KB, patch
rail
: review+
Details | Diff | Splinter Review
24.71 KB, patch
bear
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
5.75 KB, patch
rail
: review+
Details | Diff | Splinter Review
491 bytes, text/plain
mozilla
: review+
Details
13.15 KB, patch
bhearsum
: review+
Details | Diff | Splinter Review
1.38 KB, patch
catlee
: review+
Details | Diff | Splinter Review
Waiting for "go to build".
Assignee: nobody → rail
Priority: -- → P4
Attached patch tools (obsolete) — Splinter Review
Attachment #592787 - Flags: review?(aki)
Attached patch buildbotcustom (obsolete) — Splinter Review
Attachment #592788 - Flags: review?(aki)
Attachment #592787 - Flags: review?(aki) → review+
Attached patch configs (obsolete) — Splinter Review
inline comments  incoming
Attachment #592789 - Flags: review?(aki)
Comment on attachment 592789 [details] [diff] [review]
configs

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

::: mozilla/config.py
@@ +1481,5 @@
> +BRANCHES['mozilla-beta']['platforms']['android']['env']['MOZ_SYMBOLS_EXTRA_BUILDID'] = 'mozilla-beta'
> +BRANCHES['mozilla-beta']['platforms']['android-xul']['env']['MOZ_SYMBOLS_EXTRA_BUILDID'] = 'android-xul-mozilla-beta'
> +BRANCHES['mozilla-beta']['platforms']['android']['enable_dep'] = True
> +BRANCHES['mozilla-beta']['platforms']['android-debug']['enable_dep'] = True
> +BRANCHES['mozilla-beta']['platforms']['android-xul']['enable_dep'] = True

Not sure if we need android-xul enable_dep

::: mozilla/release-fennec-mozilla-beta.py
@@ +105,5 @@
> +    'android': 'mobile/android/config/mozconfigs/android/release',
> +    'android-xul': 'mobile/xul/config/mozconfigs/android/release',
> +    'linux-mobile': 'mobile/xul/config/mozconfigs/linux-desktop/release',
> +    'macosx-mobile': 'mobile/xul/config/mozconfigs/macosx-desktop/release',
> +    'win32-mobile': 'mobile/xul/config/mozconfigs/win32-desktop/release',

In tree mozconfigs don't have release mozconfigs, would be great to review mozilla2 mozconfigs.

::: mozilla/release-firefox-mozilla-beta.py
@@ +85,5 @@
>  # Update-specific configuration
>  releaseConfig['cvsroot']             = ':ext:cltbld@cvs.mozilla.org:/cvsroot'
>  releaseConfig['patcherConfig']       = 'mozBeta-branch-patcher2.cfg'
>  releaseConfig['commitPatcherConfig'] = True
> +releaseConfig['patcherToolsTag']     = 'UPDATE_PACKAGING_R16'

required for signing
Attached patch buildbotcustomSplinter Review
Iterate over signedPlatforms
Attachment #592788 - Attachment is obsolete: true
Attachment #592788 - Flags: review?(aki)
Attachment #592798 - Flags: review?(aki)
Attached patch configs (obsolete) — Splinter Review
* kill desktop builds
* repack android, android-xul
Attachment #592789 - Attachment is obsolete: true
Attachment #592789 - Flags: review?(aki)
Attachment #592802 - Flags: review?(aki)
Comment on attachment 592798 [details] [diff] [review]
buildbotcustom

I think this is right... does it work? :)
Attachment #592798 - Flags: review?(aki) → review+
(In reply to Aki Sasaki [:aki] from comment #7)
> Comment on attachment 592798 [details] [diff] [review]
> buildbotcustom
> 
> I think this is right... does it work? :)

So far it prints proper URLs in the log. I'm going to run a full staging release in a bit.
Attachment #592802 - Flags: review?(aki) → review+
Attached patch toolsSplinter Review
This also fixes l10n repacks
Attachment #592787 - Attachment is obsolete: true
Attachment #592955 - Flags: review?(aki)
Attachment #592955 - Flags: review?(aki) → review+
Attached patch configs (obsolete) — Splinter Review
interdiff: https://gist.github.com/1708396
* fixes fennec tagging
* release templates (symlinks) for android-xul
* -j4
Attachment #592802 - Attachment is obsolete: true
Attachment #592956 - Flags: review?(aki)
Still need a mozharness config for android-xul signing (configs/signing/{,staging_}android_mozilla-beta.py)
Attachment #592956 - Flags: review?(aki) → review+
Attachment #592957 - Flags: review?(rail) → review+
This disables the snippet creation and signs both platforms.

Later, I'm going to fix:

a) surviving broken locales
b) surviving broken snippets
c) downloading the fennec apk rather than the unsigned-unaligned apk (for locales)
d) reporting on everything
e) reenabling snippets for beta2

but this should be sufficient for beta1.
Attachment #592975 - Flags: review?(rail)
I will also backport the controlling-actions-from-config-file patch from my talosrunner branch one of these days =P
Comment on attachment 592975 [details] [diff] [review]
sign en-US and multi for android and android-xul

lgtm
Attachment #592975 - Flags: review?(rail) → review+
Comment on attachment 592975 [details] [diff] [review]
sign en-US and multi for android and android-xul

http://hg.mozilla.org/build/mozharness/rev/62e67a176fec
Attachment #592975 - Flags: checked-in+
I split the "configs" patch into 2 pieces. This one should be landed once aurora migrates to beta. Release can be landed separately.
Attachment #593154 - Flags: review+
Depends on: 722719
this should be the same as attachment 593154 [details] [diff] [review], except with the release configs bumped as well. We needed to adjust the release configs to enable the new platforms on mozilla-beta otherwise bm08-build1 would fail to reconfig.
Attachment #593309 - Flags: review?(bear)
Attachment #593309 - Flags: review?(bear) → review+
Attachment #593309 - Flags: checked-in+
Attachment #593154 - Attachment is obsolete: true
Comment on attachment 592957 [details] [diff] [review]
switch tests on mozilla-beta from linux-android to android, android-xul

Incorporated into another patch.
Attachment #592957 - Attachment is obsolete: true
Attached patch configsSplinter Review
Refreshed patch. Carry on r=aki
Attachment #592956 - Attachment is obsolete: true
Attachment #593314 - Flags: review+
Attached file push fennec script
A script for pushing android files to the releases directory. No l10n repacks this time.
Attachment #593466 - Flags: review?(aki)
Attachment #593466 - Flags: review?(aki) → review+
Attached patch configsSplinter Review
* s/TBD/ee7378639638/
* l10n json for fennec is {}
* disabled multilocale for mozilla-beta branch (need a backout when we're ready)
* disabled l10n repacks for fennec
Attachment #593603 - Flags: review?(bhearsum)
Comment on attachment 593603 [details] [diff] [review]
configs

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

::: mozilla/release-firefox-mozilla-beta.py
@@ +133,5 @@
>  # Misc configuration
>  releaseConfig['enable_repo_setup'] = False
>  releaseConfig['enableAutomaticPushToMirrors'] = True
> +releaseConfig['enableSigningAtBuildTime'] = True
> +releaseConfig['enablePartialMarsAtBuildTime'] = True

You can just drop these since they're the default, right? r=me with that change.
Attachment #593603 - Flags: review?(bhearsum) → review+
Priority: P4 → P1
Attachment #593847 - Flags: review?(catlee)
Attachment #593847 - Flags: review?(catlee) → review+
This is now live in production since 8:20 AM PDT:
http://hg.mozilla.org/build/buildbot-configs/rev/2145be876274
Depends on: 723615
all done here
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: