Skip to content

Conversation

@rcholic
Copy link
Contributor

@rcholic rcholic commented Dec 30, 2025

Summary: Trace Index Upload Implementation Complete ✅

1. SDK Implementations

Python SDK :

  • Added _upload_index() method that requests upload URL and uploads compressed index
  • Integrated into upload flow after trace upload succeeds
  • Non-fatal error handling (trace succeeds even if index fails)

TypeScript SDK:

  • Added _uploadIndex(), _requestIndexUploadUrl(), and _uploadIndexToCloud() methods
  • Same flow as Python: upload index after trace upload
  • Graceful error handling with logging

2. Upload Flow

  1. Generate trace index (local .index.json file)
  2. Upload trace file (.jsonl.gz)
  3. Request index upload URL from POST /v1/traces/index_upload ← NEW!
  4. Upload index file (.index.json.gz) ← NEW!
  5. Call /v1/traces/complete
  6. Clean up local files
  7. Tests Added

Python: 4 new tests in test_cloud_tracing.py

Index upload success
No API key (skips upload)
Upload failure (non-fatal)
Missing index file (graceful)

TypeScript: 5 new tests

Index upload success
No API key (skips upload)
Upload failure (non-fatal)
Missing index file (graceful)
Gzip compression validation

@rcholic rcholic changed the base branch from main to video_recording December 30, 2025 02:22
Base automatically changed from video_recording to main December 30, 2025 02:23
@rcholic rcholic merged commit 1d51a87 into main Dec 30, 2025
3 checks passed
@rcholic rcholic deleted the upload_indexing branch December 30, 2025 03:09
rcholic pushed a commit that referenced this pull request Jan 6, 2026
rcholic pushed a commit that referenced this pull request Jan 10, 2026
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.

2 participants