Closed
Bug 1021873
Opened 11 years ago
Closed 11 years ago
rm_old_symbols step is failing for win64-ff64 nightlies
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlund, Assigned: nthomas)
Details
Attachments
(1 file)
1.30 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
not sure when it started failing or if it always had. looks like it could be a syntax error:
log example: https://pastebin.mozilla.org/5364463
build example: http://buildbot-master84.srv.releng.scl3.mozilla.com:8001/builders/WINNT%206.1%20x86-64%20mozilla-central%20nightly/builds/84
Assignee | ||
Comment 1•11 years ago
|
||
I think we should just remove this code, as it's a leftover from the old way of doing symbols. These days they are at
<objdir>/dist/firefox-32.0a1.en-US.mac64.crashreporter-symbols-full.zip
<objdir>/dist/firefox-32.0a1.en-US.mac64.crashreporter-symbols.zip
and something else handles cleaning up (probably just by using the same filename so they get replaced).
I also double checked the status db. So far this year we did 17745 rm_old_symbols steps, and every single one ended up with FAILURE result. This isn't completely watertight, given the example above
find: missing argument to `-exec'
and the usual
find: `20*': No such file or directory
both end up in the same state, but the fact we had not a single SUCCESS indicates to me we're never finding a file to delete. Besides we always clobber the nightly dir as the 2nd last step.
Updated•11 years ago
|
Attachment #8436577 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 2•11 years ago
|
||
Comment on attachment 8436577 [details] [diff] [review]
b1021873.diff
https://hg.mozilla.org/build/buildbotcustom/rev/94ca5411a67f
Attachment #8436577 -
Flags: checked-in+
Comment 3•11 years ago
|
||
Live with reconfig on 2014-06-11 08:47 PT
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → 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
•