[codex] Add OpenTelemetry IPFS retrieval instrumentation#2
Merged
Conversation
Collaborator
Author
|
Validation note: local JS validation passed with:
The GitHub This appears to be an existing workflow access/configuration issue rather than a failure introduced by this branch; recent |
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
Adds passive OpenTelemetry API instrumentation to the JavaScript client IPFS/Zarr open path, mirroring the Python client instrumentation where this repository owns the behavior.
@opentelemetry/apias a runtime dependency without adding exporters or SDK setup.openDatasetFromCidwith spans for dataset opens and Jaxray IPFS store opens.Notes
The JS client delegates HAMT/block traversal and lower-level IPFS/Zarr behavior to
@dclimate/jaxray, which is already at the latest published version (0.6.8). Those lower-level Pythonpy-hamtcounters do not have an owned equivalent in this repo.Validation
npm run buildnpx vitest run tests/open-dataset.test.tsnpm test -- --run