Closed
Bug 842870
Opened 12 years ago
Closed 12 years ago
Tracking bug for 29-oct-2013 migration work
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: joduinn, Assigned: rail)
References
Details
Attachments
(4 files, 1 obsolete file)
|
3.84 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
|
1.93 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
|
1.10 KB,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
|
31.25 KB,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
Instructions: https://wiki.mozilla.org/ReleaseEngineering:MergeDuty
| Reporter | ||
Comment 1•12 years ago
|
||
Found in triage.
Component: Release Engineering → Release Engineering: Releases
QA Contact: bhearsum
Comment 2•12 years ago
|
||
This is not release work - migration bugs track config.py migrations.
Component: Release Engineering: Releases → Release Engineering: Automation (General)
QA Contact: bhearsum → catlee
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
Updated•12 years ago
|
Depends on: asan-tests
Updated•12 years ago
|
| Assignee | ||
Updated•12 years ago
|
| Assignee | ||
Comment 3•12 years ago
|
||
A noop cleanup:
* replace a loop with items_before() loop
* remove duplicate loop for linux64-asan and invalid platforms (linux64-dbg-asan and linux64-dbg-st-an)
zero diff for config.py dumps
Attachment #817871 -
Flags: review?(bhearsum)
Comment 4•12 years ago
|
||
Comment on attachment 817871 [details] [diff] [review]
clean up configs
Review of attachment 817871 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla/config.py
@@ +2193,5 @@
> BRANCHES[branch]['platforms'][platform]['slaves'] = oldslaves
> if 'l10n_slaves' in BRANCHES[branch]['platforms'][platform] and branch != 'mozilla-esr17':
> BRANCHES[branch]['platforms'][platform]['l10n_slaves'] = oldslaves
>
> # MERGE DAY - Once gecko 26 reaches 'mozilla-release' remove this line
Do we really need these comments when using items_before?
Attachment #817871 -
Flags: review?(bhearsum) → review+
| Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 817871 [details] [diff] [review]
clean up configs
(In reply to Ben Hearsum [:bhearsum] from comment #4)
> Do we really need these comments when using items_before?
No. I removed them:
https://hg.mozilla.org/build/buildbot-configs/rev/8181d7ab7631
Attachment #817871 -
Flags: checked-in+
| Assignee | ||
Comment 6•12 years ago
|
||
Attachment #819051 -
Flags: review?(bhearsum)
Updated•12 years ago
|
Attachment #819051 -
Flags: review?(bhearsum) → review+
| Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 819051 [details] [diff] [review]
beta -> release merge
remote: https://hg.mozilla.org/build/buildbot-configs/rev/1b063a6fe1c4
remote: https://hg.mozilla.org/build/buildbot-configs/rev/52d0c3e94af0
I will reconfig shortly.
Attachment #819051 -
Flags: checked-in+
| Assignee | ||
Comment 8•12 years ago
|
||
In production
| Assignee | ||
Comment 9•12 years ago
|
||
Phew... It took almost a day of my life!
Since we have to do the uplift and add v1.2 branch at the same time I created a patch with has ALL THE THINGS. :)
Attachment #821921 -
Flags: review?(aki)
Comment 10•12 years ago
|
||
Comment on attachment 821921 [details] [diff] [review]
beta + aurora + b2g26_v1_2
>diff --git a/mozilla/b2g_config.py b/mozilla/b2g_config.py
>--- a/mozilla/b2g_config.py
>+++ b/mozilla/b2g_config.py
>@@ -804,29 +804,30 @@ PLATFORM_VARS = {
> }
>
>
> # All branches (not in project_branches) that are to be built MUST be listed here, along with their
> # platforms (if different from the default set).
> BRANCHES = {
> 'mozilla-central': {
> },
>- 'mozilla-aurora': {
>- },
> 'mozilla-b2g18': {
> 'gecko_version': 18,
> },
> 'mozilla-b2g18_v1_0_1': {
> 'gecko_version': 18,
> 'b2g_version': (1, 0, 1),
> },
> 'mozilla-b2g18_v1_1_0_hd': {
> 'gecko_version': 18,
> 'b2g_version': (1, 1, 0),
> },
>+ 'mozilla-b2g26_v1_2': {
>+ 'gecko_version': 26,
>+ },
should we set the b2g_version to (1, 2, 0) here?
I don't know if we use b2g_version yet.
Attachment #821921 -
Flags: review?(aki) → review+
Comment 11•12 years ago
|
||
Comment on attachment 821921 [details] [diff] [review]
beta + aurora + b2g26_v1_2
Oops, ValueError: Cannot shorten "b2g_mozilla-b2g26_v1_2_emulator-debug_dep" to maximum length (30). Got to: b2g_m-b2g26_v1_2_emulator-d_dep
Comment 12•12 years ago
|
||
If you don't see the errors, this might not be needed. Fixed up test-masters.sh for me, though.
Attachment #821959 -
Flags: review?(rail)
| Assignee | ||
Updated•12 years ago
|
Attachment #821959 -
Flags: review?(rail) → review+
| Assignee | ||
Comment 13•12 years ago
|
||
Comment on attachment 821959 [details] [diff] [review]
(custom) b2g26_shorten
https://hg.mozilla.org/build/buildbotcustom/rev/d1838fed5416
Attachment #821959 -
Flags: checked-in+
| Assignee | ||
Comment 14•12 years ago
|
||
* addressed comments
* refreshed
* carry over r=aki
Attachment #821921 -
Attachment is obsolete: true
Attachment #823308 -
Flags: review+
| Assignee | ||
Comment 15•12 years ago
|
||
Comment on attachment 823308 [details] [diff] [review]
beta + aurora + b2g26_v1_2 (v2)
https://hg.mozilla.org/build/buildbot-configs/rev/9f7687feb5f8
Attachment #823308 -
Flags: checked-in+
| Assignee | ||
Comment 16•12 years ago
|
||
in production
| Assignee | ||
Updated•12 years ago
|
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•