chore(deps): update dependency opentelemetry-python to v1.40.0#1003
Conversation
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
🧪 CI InsightsHere's what we observed from your CI run for e037272. 🟢 All jobs passed!But CI Insights is watching 👀 |
There was a problem hiding this comment.
Pull request overview
Updates the CLI’s OpenTelemetry dependencies to v1.40.0 and adjusts span upload error reporting to include the HTTP response body (since OTel’s own logging no longer includes it in v1.40.0).
Changes:
- Bump
opentelemetry-exporter-otlp-proto-httpandopentelemetry-sdkto1.40.0(and related transitive OTel packages in the lockfile). - Replace log-capture-based error extraction with an
OTLPSpanExportersubclass that retains the last HTTP response so upload failures can surface status code + response body. - Minor test/comment tweak to align with the new error source.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| uv.lock | Locks OTel packages to 1.40.0 (plus related transitive updates). |
| pyproject.toml | Updates declared OTel dependency pins to 1.40.0. |
| mergify_cli/tests/ci/junit_processing/test_upload.py | Removes an OTel-specific JSON rendering comment now that errors come from HTTP response text. |
| mergify_cli/ci/junit_processing/upload.py | Captures the HTTP response body on export failure and includes it in UploadError. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Also capture the HTTP response body in upload errors instead of relying on OTel's log output, which no longer includes it since v1.40.0. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Change-Id: I3e00b5ad3d0e8509f94a17770453924010e902a2 Claude-Session-Id: d5d86f29-f099-42b6-a8d5-3f3e2d81cc49
bb31100 to
e037272
Compare
|
@Mergifyio queue |
1 similar comment
|
@Mergifyio queue |
Merge Queue StatusRule:
This pull request spent 8 seconds in the queue, with no time running CI. Required conditions to merge
|
|
@Mergifyio refresh |
✅ Pull request refreshed |
Also capture the HTTP response body in upload errors instead of relying
on OTel's log output, which no longer includes it since v1.40.0.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com