feat(storage): benchmark Tensorlake Cloud Volumes (conflict-free rebase of #229) - #257
Merged
Merged
Conversation
Resolve conflicts: adopt master's per-bench script paths and vault secret loading, keeping the Tensorlake matrix entries, scripts, and dependencies. Co-Authored-By: Noah Kiser <noah@computesdk.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor License AgreementAll contributors are covered by a CLA. |
Co-Authored-By: Noah Kiser <noah@computesdk.com>
Resolve package.json conflict: master routes bench scripts through packages/benchsdk-runner/dist/bin.js, so the tensorlake storage and snapshot-fork scripts adopt that form. Co-Authored-By: Noah Kiser <noah@computesdk.com>
Contributor
Storage Benchmark Results1MB Files
View full run · SVGs available as build artifacts |
Contributor
Snapshot/Fork Benchmark Resultssmall dataset
|
Contributor
Browser Benchmark Results
View full run · SVG available as build artifact |
Contributor
Browser Throughput Benchmark Results
View full run · SVG available as build artifact |
Contributor
Sandbox Benchmark ResultsSequential
Staggered
Burst
View full run · SVGs available as build artifacts |
Contributor
Sandbox Dax Benchmark Results
|
Contributor
AI Gateway Benchmark Results
View full run · SVG available as a build artifact |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Supersedes #229, which had merge conflicts and lives on a fork with
maintainer_can_modify: false(so it couldn't be updated in place). This branch is diptanu's two commits merged with currentmaster, conflicts resolved.Adds Tensorlake Cloud Volumes to the storage and snapshot/fork matrices via the first-party
@storagesdk/adapters/tensorlakeadapter — no Tensorlake-specific code in this repo; snapshots/forks are native metadata calls.Conflict resolution
masterdiverged in three ways since #229 branched:benchmarks/src/run.ts --mode storage, and then again (in Split @benchsdk/client (REST) and @benchsdk/runner (framework); rework benchmark authoring to config + task #253) behind the@benchsdk/runnerbin, so the new scripts follow master's current form:env: X: ${{ secrets.X }}blocks in favor ofload-vault-secrets.sh '<regex>', so feat(storage): benchmark Tensorlake Cloud Volumes #229'sTENSORLAKE_*env entries becameTENSORLAKE_API_KEY|TENSORLAKE_API_URL|TENSORLAKE_ORGANIZATION_ID|TENSORLAKE_PROJECT_ID|TENSORLAKE_FILESYSTEMadditions to each workflow's allowlist regex instead. Those five entries are now in the Namespace vault (confirmed by the smoke run below).benchmarks/scripts/provider-vars.jsonis generated fromrequiredEnvVars; regenerated sostorageandsnapshot-forklist theTENSORLAKE_*vars.pnpm-lock.yamlwas regenerated from master's lockfile rather than merged. The StorageSDK release #229 was waiting on is out, so the dependency bumps (@storagesdk/adapters@^0.10.0,@storagesdk/core@^0.5.0,tensorlake@0.5.89) resolve against the registry.benchmarks/storage/providers.tsmerged cleanly (unchanged from #229).Validation
pnpm install --frozen-lockfile,pnpm typecheckLive 1 MiB storage smoke test on this branch against the vault credentials — 3/3 succeeded (run,
dry_run=true, whole 7-provider matrix green):#229's quoted percentiles were measured on an arm64 macOS runner and don't reflect what CI will record.
Open item
The Superagent supply-chain gate is red on 6 heuristic findings across the three dependency bumps. I reviewed each against the package contents and believe all are false positives (details in the two review threads: the
threat-process-download-exechit is acurl | shstring inside an error message;threat-network-outbound-shady-linksis Backblaze's own domain). Needs a human sign-off; I have not touched the scanner config.Link to Devin session: https://app.devin.ai/sessions/6ad3ed1fd503463482618cd1497d11c4