Skip to content

chore(deps): update dependency opentelemetry-python to v1.40.0#1003

Merged
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-opentelemetry/I3e00b5ad3d0e8509f94a17770453924010e902a2
Mar 6, 2026
Merged

chore(deps): update dependency opentelemetry-python to v1.40.0#1003
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-opentelemetry/I3e00b5ad3d0e8509f94a17770453924010e902a2

Conversation

@jd
Copy link
Member

@jd jd commented Mar 5, 2026

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

Copilot AI review requested due to automatic review settings March 5, 2026 21:25
@mergify mergify bot had a problem deploying to Mergify Merge Protections March 5, 2026 21:25 Failure
@mergify
Copy link
Contributor

mergify bot commented Mar 5, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success=ci-gate

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify
Copy link
Contributor

mergify bot commented Mar 5, 2026

🧪 CI Insights

Here's what we observed from your CI run for e037272.

🟢 All jobs passed!

But CI Insights is watching 👀

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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-http and opentelemetry-sdk to 1.40.0 (and related transitive OTel packages in the lockfile).
  • Replace log-capture-based error extraction with an OTLPSpanExporter subclass 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
@jd jd force-pushed the devs/jd/worktree-opentelemetry/I3e00b5ad3d0e8509f94a17770453924010e902a2 branch from bb31100 to e037272 Compare March 6, 2026 08:39
@mergify mergify bot deployed to Mergify Merge Protections March 6, 2026 08:40 Active
@mergify mergify bot requested a review from a team March 6, 2026 08:42
@mergify mergify bot requested a review from a team March 6, 2026 09:23
@jd
Copy link
Member Author

jd commented Mar 6, 2026

@Mergifyio queue

1 similar comment
@jd
Copy link
Member Author

jd commented Mar 6, 2026

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Mar 6, 2026

Merge Queue Status

Rule: default


  • Entered queue2026-03-06 10:05 UTC
  • Checks passed · in-place
  • Merged2026-03-06 10:05 UTC · at e03727222ab9ced3640e5c833caa28cdeb7ec138

This pull request spent 8 seconds in the queue, with no time running CI.

Required conditions to merge

@jd
Copy link
Member Author

jd commented Mar 6, 2026

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Mar 6, 2026

refresh

✅ Pull request refreshed

@mergify mergify bot added the queued label Mar 6, 2026
@mergify mergify bot merged commit cb04738 into main Mar 6, 2026
10 checks passed
@mergify mergify bot deleted the devs/jd/worktree-opentelemetry/I3e00b5ad3d0e8509f94a17770453924010e902a2 branch March 6, 2026 10:05
@mergify mergify bot removed the queued label Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants