Closed
Bug 587949
Opened 14 years ago
Closed 14 years ago
Staging should use production mozconfigs
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rail, Assigned: rail)
References
Details
(Whiteboard: [automation])
Attachments
(1 file, 1 obsolete file)
165.50 KB,
patch
|
catlee
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
At the moment we have 2 separate mozconfig directories which should be synced on every change we make to mozconfigs, what is error-prone. It will be better to use the same mozconfigs for both production and staging, but also have a possibility to use custom mozconfigs.
As a possible solution I'd suggest to remove mozconfigs from mozilla2-stage directory. Production and staging will use mozconfigs from mozilla2 directory. If someone needs custom mozconffigs, he/she can clone buildbot-configs repo to stage-ffxbld or personal repo and use it.
Please review the attached patch (which will be tested in staging).
Attachment #466563 -
Flags: review?(catlee)
Comment 1•14 years ago
|
||
Comment on attachment 466563 [details] [diff] [review]
Don't use mozilla2-staging mozconfigs
I think this is probably fine. Ben, what do you think?
Attachment #466563 -
Flags: review?(catlee)
Attachment #466563 -
Flags: review?(bhearsum)
Attachment #466563 -
Flags: review+
Assignee | ||
Updated•14 years ago
|
Priority: P3 → P2
Comment 2•14 years ago
|
||
Comment on attachment 466563 [details] [diff] [review]
Don't use mozilla2-staging mozconfigs
Looks good to me
Attachment #466563 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Great! I'll test it in staging next week to be sure.
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•14 years ago
|
||
(In reply to comment #3)
> I'll test it in staging next week to be sure.
Staging run went fine except one thing. config.py GLOBAL_VARS overrides the same variable from localconfg.py, so I had to move config_repo_path to the corresponding files ({production,preproduction,staging}_config.py) to prevent the collision.
Attachment #466563 -
Attachment is obsolete: true
Attachment #468255 -
Flags: review?(catlee)
Updated•14 years ago
|
Attachment #468255 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 5•14 years ago
|
||
Comment on attachment 468255 [details] [diff] [review]
Don't use mozilla2-staging mozconfigs
http://hg.mozilla.org/build/buildbot-configs/rev/903bb63bdce8
Attachment #468255 -
Flags: checked-in+
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•