Closed Bug 998539 Opened 11 years ago Closed 11 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
u429623
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
754 bytes, patch
u429623
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
557 bytes, patch
u429623
: 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
u429623
: 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
u429623
: 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
u429623
: 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
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: 11 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: