bazelbuild / bazel-buildfarm

Bazel remote caching and execution service
https://bazel.build
Apache License 2.0
635 stars 199 forks source link

bazel.canonical_id unsupported in Asset Fetch API #1769

Closed werkt closed 3 weeks ago

werkt commented 3 weeks ago

Bazel currently dispatches downloader requests with bazel.canonical_id qualifier.

Ordinarily this will match with the URL. If the keys are to be stored in CAS, we will likely have to salt the content with this canonical_id for storage to generate a unique name. This can be relatively easily distinguished from previous implementation with empty canonical_id check.

werkt commented 3 weeks ago

Fixed by PR, needed separate lines :(