Closed
Bug 794458
Opened 12 years ago
Closed 12 years ago
Make unpack / unpack tests steps less verbose
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
(Blocks 1 open bug)
Details
(Whiteboard: [sheriff-want])
Attachments
(2 files)
3.54 KB,
patch
|
bhearsum
:
review+
emorley
:
checked-in+
|
Details | Diff | Splinter Review |
1.59 KB,
patch
|
catlee
:
review+
emorley
:
checked-in+
|
Details | Diff | Splinter Review |
Background to bug 765224: Many of the buildbot logs are extremely verbose even in the case of success, causing larger log sizes, thereby meaning slower TBPL parsing (and of late, timeouts) - as well as making it harder for someone to look at the log and ascertain what actually failed without trying a bunch of searches.
One place I think we can cut back is in unpack / unpack tests, ie:
{
========= Started unpack (results: 0, elapsed: 11 secs) (at 2012-09-26 06:28:19.787943) =========
...
Archive: firefox-18.0a1.en-US.win32.zip
inflating: firefox/AccessibleMarshal.dll
inflating: firefox/application.ini
inflating: firefox/blocklist.xml
inflating: firefox/breakpadinjector.dll
inflating: firefox/chrome.manifest
extracting: firefox/components/binary.manifest
inflating: firefox/components/browsercomps.dll
inflating: firefox/crashreporter-override.ini
inflating: firefox/crashreporter.exe
inflating: firefox/crashreporter.ini
inflating: firefox/D3DCompiler_43.dll
inflating: firefox/d3dx9_43.dll
inflating: firefox/defaults/pref/channel-prefs.js
inflating: firefox/dependentlibs.list
inflating: firefox/dictionaries/en-US.aff
...
}
and
{
========= Started unpack tests (results: 0, elapsed: 54 secs) (at 2012-09-26 06:28:46.095814) =========
...
Archive: firefox-18.0a1.en-US.win32.tests.zip
inflating: bin/certutil.exe
inflating: bin/components/httpd.js
inflating: bin/components/httpd.manifest
inflating: bin/components/test_necko.xpt
inflating: bin/crashinject.exe
inflating: bin/crashinjectdll.dll
inflating: bin/fix_stack_using_bpsyms.py
inflating: bin/pk12util.exe
inflating: bin/plugins/npsecondtest.dll
inflating: bin/plugins/nptest.dll
inflating: bin/screenshot.exe
inflating: bin/ssltunnel.exe
inflating: bin/vmwarerecordinghelper.dll
inflating: bin/xpcshell.exe
inflating: certs/bug483440-attack2b.ca
inflating: certs/bug483440-attack7.ca
inflating: certs/bug483440-pk10oflo.ca
inflating: certs/cert8.db
}
Assignee | ||
Updated•12 years ago
|
Summary: Make cleanup steps less verbose (don't use rm -v) → Make unpack / unpack tests steps less verbose
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #664946 -
Flags: review?(bhearsum)
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #664950 -
Flags: review?(catlee)
Updated•12 years ago
|
Attachment #664946 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 664946 [details] [diff] [review]
buildbotcustom part
Cheers :-)
http://hg.mozilla.org/build/buildbotcustom/rev/97efa2a81346
Attachment #664946 -
Flags: checked-in+
Updated•12 years ago
|
Attachment #664950 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 664950 [details] [diff] [review]
tools installdmg.sh part
Thank you :-)
http://hg.mozilla.org/build/tools/rev/ab1e1ce9595f
Attachment #664950 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
Did you check that none of these steps approach the 20 minute timeout where buildbot will kill the job without any output ?
Assignee | ||
Comment 6•12 years ago
|
||
Yes, they're all under a minute.
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
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
•