Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-02-27 21:00:50 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1391114 - Make mach artifact toolchain validate toolchain downloads. r=gps

Browse Source 
Tooltool manifests contain digests that have been used to validate
tooltool downloads. Toolchain artifacts don't benefit from that, and as
a result, an incomplete download can be considered as finished, and
unpack fail after that, without retrying, even with --retry.

Fortunately, the chain of trust artifacts do contains digests for
taskcluster artifacts, as long as the jobs that created the artifacts
have chain of trust enabled.

As of now, the goal is not cryptographic validation of the download, but
to ensure that we got the complete file, and to trigger a new download
if we haven't.

--HG--
extra : rebase_source : cdf4b4ec0c99db1f671db799f3941804f2bcbaf9
This commit is contained in:
Mike Hommey 2017-08-17 09:36:27 +09:00
parent 597905c94c
commit 003f12df93
2 changed files with 31 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
python/mozbuild/mozbuild/action/tooltool.py
View File

@ -141,7 +141,7 @@ class FileRecord(object):
raise MissingFileException(filename=self.filename)
def validate(self):
if self.validate_size():
if self.size is None or self.validate_size():
if self.validate_digest():
return True
return False

37
python/mozbuild/mozbuild/mach_commands.py
View File

@ -1836,6 +1836,32 @@ class PackageFrontend(MachCommandBase):
return True
return super(DownloadRecord, self).validate()
class ArtifactRecord(DownloadRecord):
def __init__(self, task_id, artifact_name):
cot = cache._download_manager.session.get(
get_artifact_url(task_id, 'public/chainOfTrust.json.asc'))
digest = algorithm = None
if cot.status_code == 200:
# The file is GPG-signed, but we don't care about validating
# that. Instead of parsing the PGP signature, we just take
# the one line we're interested in, which starts with a `{`.
data = {}
for l in cot.content.splitlines():
if l.startswith('{'):
try:
data = json.loads(l)
break
except Exception:
pass
for algorithm, digest in (data.get('artifacts', {})
.get(artifact_name, {}).items()):
pass
name = os.path.basename(artifact_name)
super(ArtifactRecord, self).__init__(
get_artifact_url(task_id, artifact_name), name,
None, digest, algorithm, unpack=True)
records = OrderedDict()
downloaded = []
@ -1904,10 +1930,8 @@ class PackageFrontend(MachCommandBase):
'named `{build}`')
return 1
name = os.path.basename(artifact_name)
records[name] = DownloadRecord(
get_artifact_url(task_id, artifact_name),
name, None, None, None, unpack=True)
record = ArtifactRecord(task_id, artifact_name)
records[record.filename] = record
# Handle the list of files of the form path@task-id on the command
# line. Each of those give a path to an artifact to download.
@ -1917,9 +1941,8 @@ class PackageFrontend(MachCommandBase):
'Expected a list of files of the form path@task-id')
return 1
name, task_id = f.rsplit('@', 1)
records[os.path.basename(name)] = DownloadRecord(
get_artifact_url(task_id, name), os.path.basename(name),
None, None, None, unpack=True)
record = ArtifactRecord(task_id, name)
records[record.filename] = record
for record in records.itervalues():
self.log(logging.INFO, 'artifact', {'name': record.basename},