Closed
Bug 775526
Opened 12 years ago
Closed 12 years ago
support for multiple partial updates in tuxedo client
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
References
Details
Attachments
(1 file)
(deleted),
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
https://github.com/mozilla/build-tools/blob/master/release/tuxedo-add.py
Roughly, we need to:
- Rename the --old-version argument to --partial-version, make it accept it multiple times.
- Change TuxedoEntrySubmitter's oldVersion argument to partialVersions
- Change TuxedoentrySubmitter's self.partial_mar_bouncer_product_name to an array and adjust the logic of whether or not to add a partial to make decisions based on the length of the array rather than the existence of self.oldVersion.
Assignee | ||
Comment 1•12 years ago
|
||
Working on this over in https://github.com/bhearsum/tools/compare/mozilla:master...tuxedo-partials.
Assignee: nobody → bhearsum
Assignee | ||
Comment 2•12 years ago
|
||
Pretty simple stuff here:
* Removed devpreview ini, because we don't need it anymore
* Iterate over partial versions in place of looking at oldVersion
* Decide whether or not we should add MARs based on partialVersions, since that's what we'd be doing in the factory anyways.
Attachment #652453 -
Flags: review?(rail)
Comment 3•12 years ago
|
||
Comment on attachment 652453 [details] [diff] [review]
support multiple partials in tuxedo client
lgtm!
Attachment #652453 -
Flags: review?(rail) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #652453 -
Flags: checked-in+
Assignee | ||
Comment 4•12 years ago
|
||
Worked fine in 15.0b6.
Status: NEW → RESOLVED
Closed: 12 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
•