Closed Bug 1037107 Opened 10 years ago Closed 10 years ago

create mozilla-b2g32_v2_0 branch

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

Attachments

(11 files)

(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
RyanVM
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
pmoore
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
bhearsum
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
(deleted), patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
      No description provided.
Depends on: 1037148
Attached patch b2g_tag (deleted) — Splinter Review
Attachment #8457683 - Flags: review?(hwine)
Attachment #8457683 - Flags: review?(hwine) → review+
Comment on attachment 8457683 [details] [diff] [review]
b2g_tag

http://hg.mozilla.org/build/mozharness/rev/34c5d8272f03
Attachment #8457683 - Flags: checked-in+
Attached patch b2g32_configs (deleted) — Splinter Review
Attachment #8457700 - Flags: review?(rail)
Attached patch b2g32_custom (deleted) — Splinter Review
Attachment #8457701 - Flags: review?(rail)
Attached patch b2g32_tools (deleted) — Splinter Review
Attachment #8457702 - Flags: review?(rail)
Attached patch b2g32_tbpl (deleted) — Splinter Review
Attachment #8457705 - Flags: review?(ryanvm)
Attached patch b2g32_beagle (deleted) — Splinter Review
Attachment #8457707 - Flags: review?(pmoore)
Attachment #8457705 - Flags: review?(ryanvm) → review+
Comment on attachment 8457705 [details] [diff] [review]
b2g32_tbpl

https://hg.mozilla.org/webtools/tbpl/rev/35fc0b71256f
Attachment #8457705 - Flags: checked-in+
Attachment #8457700 - Flags: review?(rail) → review+
Attachment #8457701 - Flags: review?(rail) → review+
Attachment #8457702 - Flags: review?(rail) → review+
Comment on attachment 8457707 [details] [diff] [review]
b2g32_beagle

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

Nice work Aki.

::: configs/vcs_sync/beagle.py
@@ +254,5 @@
> +            "tag_config": {
> +                "tag_regexes": [
> +                    "^B2G_",
> +                ],
> +            },

This tag_config is the same as the tag_config below (lines 266 to 270). I think this isn't needed, because I believe if it is not provided by the specific target, it is taken from the generic cross-target tag_config section (http://hg.mozilla.org/build/mozharness/file/7e18e11a38be/scripts/vcs-sync/vcs_sync.py#l500).

However, it should not hurt, and it would be consistent with the other branches in this config file - but I believe it may be redundant to define it in both places.
Attachment #8457707 - Flags: review?(pmoore) → review+
(In reply to Pete Moore [:pete][:pmoore] from comment #9)
> ::: configs/vcs_sync/beagle.py
> @@ +254,5 @@
> > +            "tag_config": {
> > +                "tag_regexes": [
> > +                    "^B2G_",
> > +                ],
> > +            },
> 
> This tag_config is the same as the tag_config below (lines 266 to 270). I
> think this isn't needed, because I believe if it is not provided by the
> specific target, it is taken from the generic cross-target tag_config
> section
> (http://hg.mozilla.org/build/mozharness/file/7e18e11a38be/scripts/vcs-sync/
> vcs_sync.py#l500).
> 
> However, it should not hurt, and it would be consistent with the other
> branches in this config file - but I believe it may be redundant to define
> it in both places.

Aha.  Sure, up to you.  Might be a safety net, might be a pain keeping them in sync; I think I agree it's probably the latter.
Attached patch graphs (deleted) — Splinter Review
Looks complete based on https://wiki.mozilla.org/ReleaseEngineering/Merge_Duty/Steps#Add_to_Graphs .
Attachment #8458122 - Flags: review?(hwine)
Comment on attachment 8457707 [details] [diff] [review]
b2g32_beagle

Committed without the duplicate tag_config.
https://hg.mozilla.org/build/mozharness/rev/c5821474653e
Attachment #8457707 - Flags: checked-in+
Comment on attachment 8457701 [details] [diff] [review]
b2g32_custom

https://hg.mozilla.org/build/buildbotcustom/rev/433adf0b3644
Attachment #8457701 - Flags: checked-in+
Comment on attachment 8457700 [details] [diff] [review]
b2g32_configs

https://hg.mozilla.org/build/buildbot-configs/rev/7707449277f1
Attachment #8457700 - Flags: checked-in+
Comment on attachment 8457702 [details] [diff] [review]
b2g32_tools

https://hg.mozilla.org/build/tools/rev/a5d1390ed38a
Attachment #8457702 - Flags: checked-in+
Comment on attachment 8458122 [details] [diff] [review]
graphs

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

lgtm
Attachment #8458122 - Flags: review?(hwine) → review+
Attached patch in-tree-l10n (deleted) — Splinter Review
Attachment #8458443 - Flags: review?(rail)
Attached patch in-tree-channel (deleted) — Splinter Review
Since bug 1022907 isn't fixed yet.
Attachment #8458445 - Flags: review?(bhearsum)
Attached patch bump_bb-c_versions (deleted) — Splinter Review
Attachment #8458447 - Flags: review?(hwine)
Comment on attachment 8458447 [details] [diff] [review]
bump_bb-c_versions

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

lgtm
Attachment #8458447 - Flags: review?(hwine) → review+
Attachment #8458445 - Flags: review?(bhearsum) → review+
Attachment #8458443 - Flags: review?(rail) → review+
Depends on: 1040852
(In reply to Aki Sasaki [:aki] from comment #8)
> https://hg.mozilla.org/webtools/tbpl/rev/35fc0b71256f

In production :)
In production.
Comment on attachment 8458122 [details] [diff] [review]
graphs

http://hg.mozilla.org/graphs/rev/632a9d16cb0d
Attachment #8458122 - Flags: checked-in+
Attached patch disable_dolphin (deleted) — Splinter Review
It looks like we were intentionally only building dolphin on 1.4, like we're only building Tarako on 1.3t.  Re-disabling...
Attachment #8458921 - Flags: review?(rail)
Comment on attachment 8458445 [details] [diff] [review]
in-tree-channel

https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/e5db66301250
Attachment #8458445 - Flags: checked-in+
Comment on attachment 8458443 [details] [diff] [review]
in-tree-l10n

https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/bcd1e9db532a
Attachment #8458443 - Flags: checked-in+
Attachment #8458921 - Flags: review?(rail) → review+
Comment on attachment 8458447 [details] [diff] [review]
bump_bb-c_versions

http://hg.mozilla.org/build/buildbot-configs/rev/ea1dd17b4a35
Attachment #8458447 - Flags: checked-in+
Comment on attachment 8458921 [details] [diff] [review]
disable_dolphin

http://hg.mozilla.org/build/buildbot-configs/rev/3d0e0896b4e2
Attachment #8458921 - Flags: checked-in+
In production.
Er,
https://bugzilla.mozilla.org/show_bug.cgi?id=1017923#c32
dev-b2g:
As of today's uplift, the Gecko side of B2G v2.0 now lives on the
mozilla-b2g32_v2_0 branch, *NOT* mozilla-aurora. Patches that are
blocking-b2g:2.0+ should be landed there now. For more information, see
the page below which always contains the latest information.
https://wiki.mozilla.org/Release_Management/B2G_Landing#v2.0.0

As a reminder, B2G v2.1 development remains on mozilla-central through
the duration of the Gecko 34 development cycle due to end on Sept 1, so
there is no change for where v2.1 patches are to land.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
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: