Changeset View
Changeset View
Standalone View
Standalone View
swh/loader/package/npm/loader.py
Show First 20 Lines • Show All 83 Lines • ▼ Show 20 Lines | def build_revision( | ||||
message = i_metadata['version'].encode('ascii') | message = i_metadata['version'].encode('ascii') | ||||
# from extrinsic metadata | # from extrinsic metadata | ||||
# No date available in intrinsic metadata: retrieve it from the API | # No date available in intrinsic metadata: retrieve it from the API | ||||
# metadata, using the version number that the API claims this package | # metadata, using the version number that the API claims this package | ||||
# has. | # has. | ||||
extrinsic_version = a_metadata['version'] | extrinsic_version = a_metadata['version'] | ||||
if 'time' in self.info: | |||||
date = self.info['time'][extrinsic_version] | date = self.info['time'][extrinsic_version] | ||||
else: | |||||
artifact_name = os.path.basename(a_metadata['dist']['tarball']) | |||||
raise ValueError( | |||||
'Origin %s: Cannot determine upload time for artifact %s.' % | |||||
(self.url, artifact_name) | |||||
) | |||||
date = iso8601.parse_date(date) | date = iso8601.parse_date(date) | ||||
date = normalize_timestamp(int(date.timestamp())) | date = normalize_timestamp(int(date.timestamp())) | ||||
return { | return { | ||||
ardumont: I'll raise a value error with that message here.
So that it stands out in sentry if it even… | |||||
'type': 'tar', | 'type': 'tar', | ||||
'message': message, | 'message': message, | ||||
'author': author, | 'author': author, | ||||
'date': date, | 'date': date, | ||||
'committer': author, | 'committer': author, | ||||
'committer_date': date, | 'committer_date': date, | ||||
'parents': [], | 'parents': [], | ||||
'metadata': { | 'metadata': { | ||||
▲ Show 20 Lines • Show All 165 Lines • Show Last 20 Lines |
I'll raise a value error with that message here.
So that it stands out in sentry if it even happens (simpler than to parse logs to detect the log pattern now).