Closed Bug 998539 Opened 10 years ago Closed 10 years ago

b2g 1.4 branch support

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

Attachments

(15 files, 2 obsolete files)

6.72 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
754 bytes, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
557 bytes, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
1.64 KB, patch
RyanVM
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
764 bytes, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
2.64 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
3.87 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
13.07 KB, patch
Details | Diff | Splinter Review
39.16 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
9.46 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
9.85 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
2.13 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
1.00 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
979 bytes, patch
mozilla
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
5.36 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
      No description provided.
Depends on: 998540
Attachment #8409248 - Flags: review?(hwine)
Attachment #8409249 - Flags: review?(hwine)
Attached patch repo-sync-toolsSplinter Review
Attachment #8409250 - Flags: review?(hwine)
Attached patch b2g30_tbplSplinter Review
Attachment #8409255 - Flags: review?(ryanvm)
Attachment #8409259 - Flags: review?(rail)
Attached patch buildbotcustomSplinter Review
Attachment #8409262 - Flags: review?(rail)
Comment on attachment 8409248 [details] [diff] [review]
mozharness - beagle, gecko.git, l10n

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

lgtm - just looking for an answer to the question I posed, not a blocker

::: configs/vcs_sync/gecko-git.py
@@ +165,5 @@
> +            "target_dest": "gecko-git/.git",
> +            "vcs": "git",
> +            "test_push": True,
> +        }, {
> +            "target_dest": "github-gecko-git",

Is this target still relevant? I thought gecko.git was only going to git.m.o
Attachment #8409248 - Flags: review?(hwine) → review+
(In reply to Hal Wine [:hwine] (use needinfo) from comment #7)
> Comment on attachment 8409248 [details] [diff] [review]
> mozharness - beagle, gecko.git, l10n
> 
> Review of attachment 8409248 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> lgtm - just looking for an answer to the question I posed, not a blocker
> 
> ::: configs/vcs_sync/gecko-git.py
> @@ +165,5 @@
> > +            "target_dest": "gecko-git/.git",
> > +            "vcs": "git",
> > +            "test_push": True,
> > +        }, {
> > +            "target_dest": "github-gecko-git",
> 
> Is this target still relevant? I thought gecko.git was only going to git.m.o

It's relevant because gecko.git in mozharness (non-legacy) needs to go somewhere, and that somewhere is not git.m.o until the legacy->new cutover.
Comment on attachment 8409249 [details] [diff] [review]
repo-sync-configs

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

lgtm
Attachment #8409249 - Flags: review?(hwine) → review+
Comment on attachment 8409250 [details] [diff] [review]
repo-sync-tools

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

lgtm
Attachment #8409250 - Flags: review?(hwine) → review+
Attachment #8409259 - Flags: review?(rail) → review+
Attachment #8409262 - Flags: review?(rail) → review+
Comment on attachment 8409255 [details] [diff] [review]
b2g30_tbpl

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

Please put these above the b2g28 entries rather than below.
Attachment #8409255 - Flags: review?(ryanvm) → review+
Keywords: leave-open
Comment on attachment 8409248 [details] [diff] [review]
mozharness - beagle, gecko.git, l10n

https://hg.mozilla.org/build/mozharness/rev/74a27ec8b633
Attachment #8409248 - Flags: checked-in+
Attached patch b2g30-graphsSplinter Review
Attachment #8409809 - Flags: review?(hwine)
Attached patch Build builders diff (obsolete) — Splinter Review
Comment on attachment 8409809 [details] [diff] [review]
b2g30-graphs

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

using these for database updates lgtm as well
Attachment #8409809 - Flags: review?(hwine) → review+
Attached patch b2g30_configs (obsolete) — Splinter Review
The tests configs are straightforward.

I refactored the mozilla/b2g_config.py file to simplify merge day; a lot of the loops can be removed if we more explicitly define our platforms on older b2gXX branches.

This ends up adding a bunch of device builds to project branches, which aren't necessarily needed.  However, project branches are a very small percentage of our load, and I'd argue that a) detecting bustage before it merges and b) simplicity of configs are both wins.
Attachment #8409813 - Flags: review?(rail)
With fix; new buildbot-configs patches next
Attachment #8409811 - Attachment is obsolete: true
Attached patch b2g30_configsSplinter Review
Add b2g30 to the ACTIVE_B2G_BRANCHES =\
Attachment #8409813 - Attachment is obsolete: true
Attachment #8409813 - Flags: review?(rail)
Attachment #8409821 - Flags: review?(rail)
Attachment #8409821 - Flags: review?(rail) → review+
Attached patch kill_auroraSplinter Review
This removes aurora support (for next Monday).
I commented out the entries and pre-bumped to B2G 2.0.0, Gecko 32, so [in theory] the next person can just uncomment.
Attachment #8409842 - Flags: review?(rail)
Attached patch b2g30_intreeSplinter Review
Dependent on gaia 1.4 l10n
Attachment #8409903 - Flags: review?(hwine)
Comment on attachment 8409903 [details] [diff] [review]
b2g30_intree

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

lgtm
Attachment #8409903 - Flags: review?(hwine) → review+
Attachment #8409842 - Flags: review?(rail) → review+
Depends on: 1000304
TBPL part in production :-)
Attached patch fix_b2g30Splinter Review
For gaia 1.4 and getting the desktop builds building.
Attachment #8412031 - Flags: review?(rail)
Attachment #8412031 - Flags: review?(rail) → review+
Attached patch tinderbox_treeSplinter Review
boo
Attachment #8412211 - Flags: review?(hwine)
Attachment #8412211 - Flags: review?(hwine) → review+
buildbot-config patch is in production: http://hg.mozilla.org/build/buildbot-configs/rev/797a2c9ec59b :)
Attached patch fix_keep_updatedSplinter Review
Attachment #8413836 - Flags: review?(hwine)
Always one more thing
Attachment #8413850 - Flags: review?(rail)
Attachment #8413850 - Flags: review?(rail) → review+
I think we're done here.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
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: