[reports-service] Implement Blob storage
ClosedPublic
Actions

Authored by bartek on Aug 28 2023, 4:16 AM.

Details

Reviewers

michal
• patryk
varun
• jon
tomek

Commits

rCOMM925742213d52: [reports-service] Implement Blob storage

Summary

Implemented storing large reports in blob service and retrieving them.

Depends on D8974, D8940

Test Plan

The plan requires D8971 to easily upload and view reports

Commented out the size check to store all reports in blob service.
Uploaded multiple reports. Verified that they are stored in blob service.
Downloaded one report directly from blob service, using blob hash. When opened the file, it contained the report JSON.
Viewed the report using report HTTP endpoint

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Aug 28 2023, 4:16 AM

bartek held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptAug 28 2023, 4:16 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

bartek published this revision for review.Aug 28 2023, 4:24 AM

Harbormaster completed remote builds in B22167: Diff 30433.Aug 28 2023, 4:34 AM

Adding @tomek because of new dependency while @ashoat is unavailable

But also sha2, seems much more popular than sha256. We use in commtest already

In D8975#265478, @michal wrote:

Adding @tomek because of new dependency while @ashoat is unavailable

But also sha2, seems much more popular than sha256. We use in commtest already