Closed
Bug 973492
Opened 11 years ago
Closed 11 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•11 years ago
|
Attachment #8376975 -
Flags: review?(armenzg)
Updated•11 years ago
|
Attachment #8376975 -
Flags: review?(armenzg) → review+
| Assignee | ||
Comment 1•11 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•11 years ago
|
||
in production.
| Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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
•