Closed Bug 508674 Opened 12 years ago Closed 11 years ago

Simplify branch config

Categories

(Release Engineering :: General, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: catlee, Assigned: catlee)

References

Details

Attachments

(2 files, 1 obsolete file)

There's an awful lot of repeated information in config.py  (e.g. almost every line starting with BRANCHES['mozilla-...']['platforms']['...']).  We should be able to clean this up significantly.

It might also be nice to have each branch config live in its own file.
Cleanup work sounds like future work to me. :)
Component: Release Engineering → Release Engineering: Future
Assignee: nobody → catlee
Component: Release Engineering: Future → Release Engineering
Attachment #424280 - Flags: review?(nrthomas)
Attachment #424280 - Flags: review?(bhearsum)
Comment on attachment 424280 [details] [diff] [review]
simplify staging configs

This looks good. What do you think of setting these things as global defaults:
parts of the env: OBJDIR, SYMBOL_SERVER_{HOST,USER,PATH}, TINDERBOX_OUTPUT, MOZ_CRASHREPORTER_NO_REPORT
platform_objdir
upload_symbols
profiled_build

Some of these are the same everywhere, and some of them are the same for all but one platform.
Attachment #424280 - Flags: review?(bhearsum) → review+
Comment on attachment 424280 [details] [diff] [review]
simplify staging configs

I'd agree with what Ben said, except for maybe profiledbuild. There's a lot of binary things we could set a default for in global or platform vars, and then disable/enable for particular branches as needed. Not sure if that's more readable than the verbosity we have now, since you have to go look up the default value.

Overall this should make it vastly easier to minimize the differences between staging and production configs too.

>+    for branch in branches:
>+        pprint.pprint(BRANCHES[branch])

Dumping the full/part config like this will be great for inspection, but please print out the branch name prior to the pprint.
Attachment #424280 - Flags: review?(nrthomas) → review+
Comment on attachment 424280 [details] [diff] [review]
simplify staging configs

changeset:   2032:f07d6cd870f1
Attachment #424280 - Flags: checked-in+
Blocks: 542647
Attached patch simplify production configs (obsolete) — Splinter Review
same idea as with staging configs.

I've also updated some of the settings and comments in staging so the diff between production and staging is cleaner.
Attachment #425482 - Flags: review?(nrthomas)
Attachment #425482 - Flags: review?(bhearsum)
Comment on attachment 425482 [details] [diff] [review]
simplify production configs

I compared this against the existing production configs and there are some differences:
* build space on some 1.9.1/tracemonkey builders
* mac env vars (this version fixes some inconsistencies)
* mozconfig path for places linux64 builder is fixed


I think all of these changes are fine.
Attachment #425482 - Flags: review?(bhearsum) → review+
Comment on attachment 425482 [details] [diff] [review]
simplify production configs

>diff --git a/mozilla2/config.py b/mozilla2/config.py
>+    'major_version': '1.9.2',

Is this even used any more ? I only see it config.py's. Followup if you prefer.

> BRANCHES = {
>     'mozilla-central': {},
>-    'mozilla-1.9.1': {},
>+    'mozilla-1.9.1': {'platforms': {
>+            'linux': {},
>+            'linux64': {},
>+            'win32': {},
>+            'macosx': {},
>+            'linux-debug': {},
>+            'macosx-debug': {},
>+            'win32-debug': {}
>+        }},

Now that WinCE got disabled this is the standard set of 7 platforms. Want to leave it this way in case WinCE comes back again ? Same for the project branches.
Attachment #425482 - Flags: review?(nrthomas) → review+
same as before, except lists of platforms have been removed where they match the global set, and major_version has been removed
Attachment #425482 - Attachment is obsolete: true
Attachment #425979 - Flags: review?(nrthomas)
Attachment #425979 - Flags: review?(bhearsum)
Attachment #425979 - Flags: review?(bhearsum) → review+
Comment on attachment 425979 [details] [diff] [review]
simplify production configs

Thanks!
Attachment #425979 - Flags: review?(nrthomas) → review+
Comment on attachment 425979 [details] [diff] [review]
simplify production configs

changeset:   2080:75e82e1b4356
Attachment #425979 - Flags: checked-in+
I'd call this done.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Blocks: 555734
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.