Closed
Bug 1232031
Opened 9 years ago
Closed 9 years ago
Intermittent repack failures due to truncated mar files
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
Details
Attachments
(3 files)
1.48 KB,
patch
|
rail
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
1.64 KB,
patch
|
rail
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
4.66 KB,
patch
|
rail
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
Typically shows up: Traceback (most recent call last): File "/builds/slave/rel-m-esr38-m64_rpk_8-00000000/scripts/scripts/l10n/create-release-repacks.py", line 134, in createRepacks mozillaDir=mozillaDir, mozillaSrcDir=mozillaSrcDir) File "/builds/slave/rel-m-esr38-m64_rpk_8-00000000/scripts/lib/python/build/l10n.py", line 207, in repackLocale cwd=path.join(nativeDistDir, 'previous'), env=env) File "/builds/slave/rel-m-esr38-m64_rpk_8-00000000/scripts/lib/python/util/commands.py", line 51, in run_cmd return subprocess.check_call(cmd, **kwargs) File "/tools/python27/lib/python2.7/subprocess.py", line 511, in check_call raise CalledProcessError(retcode, cmd) CalledProcessError: Command '['perl', '../../../tools/update-packaging/unwrap_full_update.pl', '../../../..//firefox-38.3.0esr.ru.complete.mar']' returned non-zero exit The previous complete mar files are smaller than they should be.
Assignee | ||
Comment 1•9 years ago
|
||
Lets see if a retry solves the problem. It may be that Cloudfront is caching the wrong size and this won't help, but lets hope it does.
Assignee: nobody → nthomas
Attachment #8697672 -
Flags: review?(rail)
Comment 2•9 years ago
|
||
Comment on attachment 8697672 [details] [diff] [review] [tools] Check size after d/l Review of attachment 8697672 [details] [diff] [review]: ----------------------------------------------------------------- ::: lib/python/release/download.py @@ +77,5 @@ > local_f.write(remote_f.read()) > local_f.close() > + > + expected_size = int(remote_f.info()['Content-Length']) > + actual_size = os.stat(destFileName).st_size I'd use os.path.getsize(), which uses os.stat() under the hood.
Attachment #8697672 -
Flags: review?(rail) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8697672 [details] [diff] [review] [tools] Check size after d/l Landed with getsize(): https://hg.mozilla.org/build/tools/rev/b8b1fb9b95eb Now we wait to see if it helps, looking for 'File is truncated' messages and retries.
Attachment #8697672 -
Flags: checked-in+
Assignee | ||
Comment 4•9 years ago
|
||
We didn't hit any job failures with 38.0.5 build2 because the new code used retry 3 times, eg Downloading http://archive.mozilla.org/pub/mozilla.org/firefox/candidates/38.3.0esr-candidates/build2/update/linux-x86_64/lij/firefox-38.3.0esr.complete.mar to firefox-38.3.0esr.lij.complete.mar retry: Calling <function downloadUpdateIgnore404 at 0x2a87d70> with args: ('archive.mozilla.org', 'firefox', '38.3.0esr', 2, 'linux64', 'lij'), kwargs: {}, attempt #2 Downloading http://archive.mozilla.org/pub/mozilla.org/firefox/candidates/38.3.0esr-candidates/build2/update/linux-x86_64/lij/firefox-38.3.0esr.complete.mar to firefox-38.3.0esr.lij.complete.mar Strange that the warning (File is truncated ....') didn't show up given the master code sets logging.INFO. And actually, we could have prevented this much earlier by not using .../mozilla.org/... in the path, and hence avoiding the proxy that serves those requests.
Assignee | ||
Comment 5•9 years ago
|
||
Use log.info when we detect truncated file, and probably avoid them altogether by going direct to S3 by dropping mozilla.org from the request path.
Attachment #8699302 -
Flags: review?(rail)
Updated•9 years ago
|
Attachment #8699302 -
Flags: review?(rail) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8699302 [details] [diff] [review] Tweaks https://hg.mozilla.org/build/tools/rev/80741989e34f
Attachment #8699302 -
Flags: checked-in+
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•9 years ago
|
||
Reopening to fix tests.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8699617 -
Flags: review?(rail)
Updated•9 years ago
|
Attachment #8699617 -
Flags: review?(rail) → review+
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8699617 [details] [diff] [review] Remove /mozilla.org/ from makeReleasesDir(), fix tests https://hg.mozilla.org/build/tools/rev/39da3b2f7be5
Attachment #8699617 -
Flags: checked-in+
Assignee | ||
Comment 10•9 years ago
|
||
In the event of any bustage I suggest backing out 39da3b2f7be5 and 80741989e34f. That would get you to a working but not logging state.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•