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

Add remote-download command to CLI #8375

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

bduffany
Copy link
Member

Useful for ad-hoc testing of the Fetch API (--experimental_remote_downloader in bazel).

@bduffany bduffany force-pushed the cli-remote-download branch 2 times, most recently from 2c38898 to 008c443 Compare February 11, 2025 16:32
@bduffany bduffany force-pushed the cli-remote-download branch 2 times, most recently from 4253242 to 65ce425 Compare February 11, 2025 16:57
@bduffany bduffany force-pushed the cli-remote-download branch from 65ce425 to 72e6882 Compare February 12, 2025 03:38
Copy link
Contributor

@sluongng sluongng left a comment

Choose a reason for hiding this comment

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

lgtm

var (
flags = flag.NewFlagSet("remote-download", flag.ContinueOnError)

target = flags.String("target", login.DefaultApiTarget, "Remote gRPC target. Must support bytestream and remote asset APIs.")
Copy link
Contributor

Choose a reason for hiding this comment

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

I would appreciate getting an API key flag here to target specific tenants.
Or if we can get the existing API key from .git/config by default, that would be nice as well.

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.

3 participants