Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GRPC entrypoint to download a blob #2044

Merged
merged 1 commit into from
May 28, 2024

Conversation

ndr-ds
Copy link
Contributor

@ndr-ds ndr-ds commented May 17, 2024

Motivation

We need to be able to download blobs directly from the Proxy as part of the user blobs work. Fixes #2028

Proposal

This PR does a few things:

  • Adds a storage connection to the Proxy
  • Adds an entrypoint for downloading a blob given a blob ID

Test Plan

WIP

Copy link
Contributor Author

ndr-ds commented May 17, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @andresilva91 and the rest of your teammates on Graphite Graphite

@ndr-ds ndr-ds mentioned this pull request May 17, 2024
@ndr-ds ndr-ds force-pushed the 05-15-grpc_entrypoint_to_download_a_blob branch from 2a775f9 to 8448c6e Compare May 17, 2024 23:53
@ndr-ds ndr-ds force-pushed the 05-08-publishblob branch 2 times, most recently from a5ab0fe to 1b2def3 Compare May 22, 2024 22:23
Base automatically changed from 05-08-publishblob to main May 23, 2024 04:11
@ndr-ds ndr-ds force-pushed the 05-15-grpc_entrypoint_to_download_a_blob branch from 8448c6e to 81193df Compare May 28, 2024 00:51
@ndr-ds ndr-ds changed the base branch from main to 05-23-add_publishblob_unittest May 28, 2024 00:51
@ndr-ds ndr-ds mentioned this pull request May 28, 2024
@ndr-ds ndr-ds force-pushed the 05-15-grpc_entrypoint_to_download_a_blob branch 3 times, most recently from c53bd85 to 88a86ba Compare May 28, 2024 01:29
@ndr-ds ndr-ds marked this pull request as ready for review May 28, 2024 11:30
@graphite-app graphite-app bot requested review from Twey, MathieuDutSik, jvff, christos-h, afck and ma2bd and removed request for MathieuDutSik, jvff and christos-h May 28, 2024 11:30
Copy link

graphite-app bot commented May 28, 2024

Graphite Automations

"Assign reviewers" took an action on this PR • (05/28/24)

6 reviewers were added to this PR based on Andre da Silva's automation.

Base automatically changed from 05-23-add_publishblob_unittest to main May 28, 2024 13:02
@ndr-ds ndr-ds force-pushed the 05-15-grpc_entrypoint_to_download_a_blob branch 2 times, most recently from 6aed766 to b6c9738 Compare May 28, 2024 14:06
Copy link
Contributor

@ma2bd ma2bd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@afck afck self-requested a review May 28, 2024 15:00
@ndr-ds ndr-ds force-pushed the 05-15-grpc_entrypoint_to_download_a_blob branch from b6c9738 to 8f3cd5f Compare May 28, 2024 16:17
@ndr-ds ndr-ds merged commit 7eaa917 into main May 28, 2024
5 checks passed
@ndr-ds ndr-ds deleted the 05-15-grpc_entrypoint_to_download_a_blob branch May 28, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a GRPC entrypoint to download a blob
3 participants