Skip to content

Optimize agentic trace ingestion without changing payloads - #672

Draft
cquil11 wants to merge 1 commit into
masterfrom
agent/optimize-agentic-ingest
Draft

Optimize agentic trace ingestion without changing payloads#672
cquil11 wants to merge 1 commit into
masterfrom
agent/optimize-agentic-ingest

Conversation

@cquil11

@cquil11 cquil11 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Index complete agentic_<suffix>/conc_<N>/aiperf_artifacts uploads before considering their duplicate multinode_server_logs tarballs.
  • Skip nested extraction only for an exact per-concurrency match with all three trace inputs present; incomplete and legacy layouts retain the existing archive fallback.
  • Parse oversized server-metrics JSON once and share the resulting profiling/warmup metric maps between aggregate-stat and chart-series generation.
  • Preserve the existing bounded-vs-streaming metric selection behavior and all malformed-input fallbacks.

Why

Run 30781275770 expanded approximately 92.8 GiB of trace data a second time from nested server-log archives, consuming roughly 56 minutes and causing smaller runners to run out of disk. Each oversized server-metrics document was then independently decompressed and tokenized three times: aggregate metrics, chart profiling metrics, and chart warmup metrics.

The producer copies the same completed logs directory into both artifact families. As an external verification, the direct and nested copies from run 30781275770, concurrency 1, had identical SHA-256 hashes for profile_export.jsonl, server_metrics_export.csv, and server_metrics_export.json.

Functional-safety checks

  • Added byte-for-byte JSON equivalence coverage for the uploaded aggregate_stats, chart_series, and request_timeline payloads.
  • Exercised both the bounded and forced-streaming paths.
  • Verified malformed-input behavior remains isolated by payload.
  • Verified incomplete direct artifacts still extract and use the nested archive.
  • Raw profile, CSV, and server-metrics compression/upload paths are unchanged.

Validation

  • bun run test:unit — 3,574 tests passed across all workspaces
  • bun run fmt
  • bun run lint
  • bun run typecheck
  • git diff --check

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview Aug 4, 2026 10:12pm

Request Review

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.

1 participant