Make unpack / unpack tests steps less verbose

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
6 months ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sheriff-want])

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
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

6 years ago
Summary: Make cleanup steps less verbose (don't use rm -v) → Make unpack / unpack tests steps less verbose
(Assignee)

Comment 1

6 years ago
Created attachment 664946 [details] [diff] [review]
buildbotcustom part
Attachment #664946 - Flags: review?(bhearsum)
(Assignee)

Comment 2

6 years ago
Created attachment 664950 [details] [diff] [review]
tools installdmg.sh part
Attachment #664950 - Flags: review?(catlee)
Attachment #664946 - Flags: review?(bhearsum) → review+
(Assignee)

Comment 3

6 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

6 years ago
Attachment #664950 - Flags: review?(catlee) → review+
(Assignee)

Comment 4

6 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

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
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

6 years ago
Yes, they're all under a minute.
(Assignee)

Updated

6 years ago
Blocks: 811286
Product: mozilla.org → Release Engineering
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.