Closed
Bug 973492
Opened 10 years ago
Closed 10 years ago
allow for multiple basedirs to be passed to PurgeMixin.purge_builds()
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlund, Assigned: jlund)
References
Details
Attachments
(1 file)
1.59 KB,
patch
|
armenzg
:
review+
jlund
:
checked-in+
|
Details | Diff | Splinter Review |
for fx desktop builds we sometimes have multiple basedirs (like for linux) eg: http://mxr.mozilla.org/build/source/buildbotcustom/process/factory.py#504 this just adds that ability in Mozharness's PurgeMixin. Should have no effect for other scripts that only had one basedir. Tested regardless below: push: - https://hg.mozilla.org/users/asasaki_mozilla.com/ash-mozharness/rev/0724ff69ddf2 tbpl (any builds that were triggered twice): - https://tbpl.mozilla.org/?showall=all&tree=Ash&rev=e7cc8d3ae34c or the logs directly: - linux jsreftest: https://tbpl.mozilla.org/php/getParsedLog.php?id=34763357&full=1&branch=ash - linux dromaeojs: https://tbpl.mozilla.org/php/getParsedLog.php?id=34763416&tree=Ash - 10.6 xpcshell: https://tbpl.mozilla.org/php/getParsedLog.php?id=34763274&tree=Ash - b2g emulator build: https://tbpl.mozilla.org/php/getParsedLog.php?id=34764018&tree=Ash - b2g crashtest: https://tbpl.mozilla.org/php/getParsedLog.php?id=34764432&tree=Ash - b2g mochitest: https://tbpl.mozilla.org/php/getParsedLog.php?id=34764414&full=1&branch=ash
Assignee | ||
Updated•10 years ago
|
Attachment #8376975 -
Flags: review?(armenzg)
Updated•10 years ago
|
Attachment #8376975 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 1•10 years ago
|
||
Comment on attachment 8376975 [details] [diff] [review] 858797_fx_desktop_builds_mh_purge-160214.diff Review of attachment 8376975 [details] [diff] [review]: ----------------------------------------------------------------- pushed to default: http://hg.mozilla.org/build/mozharness/rev/a1d43814e22f
Attachment #8376975 -
Flags: checked-in+
Comment 2•10 years ago
|
||
in production.
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•