Update OpenTelemetry dependencies#1291
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8465a5f86a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| TARGETARCH="${1:-amd64}" | ||
| DUCKDB_VERSION="${2:-v1.4.3.0}" | ||
| DUCKDB_VERSION="${2:-v1.5.3.0}" |
There was a problem hiding this comment.
When the Docker image is built without an explicit second argument, this default is used by Dockerfile line 55 and curl -f requests duckdb/duckdb-odbc/releases/download/v1.5.3.0/.... I checked the DuckDB ODBC releases page and the available 1.x tags currently stop at v1.4.2.0; v1.5.3.0 is the main DuckDB engine version, not an ODBC-driver release, so the request returns 404 and the SQLPage Docker build fails before installing the driver.
Useful? React with 👍 / 👎.
1 participant
Updates the OpenTelemetry and tracing integration crates to the current Rust 1.95-compatible releases.
This also updates the log formatter to use the public tracing-opentelemetry context API after OtelData became private, and refreshes the lockfile. The resolver removes unused OTLP gRPC-side transitive crates such as tonic, tonic-prost, http-body 1.x, http-body-util, and sync_wrapper.
Validation run locally: