Skip to content

fix(deps): widen OpenTelemetry compatibility range - #7288

Open
kiwoongyoon wants to merge 2 commits into
crewAIInc:mainfrom
kiwoongyoon:gitcontribute/20260905-162306-580447-crewAIInc-crewAI-issue-7287
Open

fix(deps): widen OpenTelemetry compatibility range#7288
kiwoongyoon wants to merge 2 commits into
crewAIInc:mainfrom
kiwoongyoon:gitcontribute/20260905-162306-580447-crewAIInc-crewAI-issue-7287

Conversation

@kiwoongyoon

Copy link
Copy Markdown

Title: fix(deps): widen OpenTelemetry compatibility range

Related issue

Fixes #7287

Summary

  • Replace the OpenTelemetry 1.42 minor pins with >=1.42,<2 in both crewai and crewai-core.
  • Keep the three OpenTelemetry packages aligned while allowing releases whose proto dependency supports protobuf 7.x.
  • Synchronize the requirement metadata in uv.lock without unrelated upgrades.

Test evidence

  • Focused manifest assertion confirms both distributions expose the three widened ranges.
  • uv pip compile with the new ranges and protobuf>=7.34,<8 resolves OpenTelemetry/proto 1.44.0 with protobuf 7.35.1.
  • UV_CACHE_DIR=/tmp/gitcontribute-7287-uv-cache-copy rtk uv lock --check --offline --python /usr/bin/python3.12
  • rtk git diff --check

Risks or notes for maintainers

  • The exact protobuf 7.36.1 reproduction could not be fetched in the network-disabled builder; the focused offline check used cached protobuf 7.35.1.
  • The range permits future pre-2.0 OpenTelemetry minors. All three direct requirements remain identical, and upstream dependencies keep the selected OpenTelemetry components mutually compatible.
  • Duplicate-work check found no related PR for [BUG] opentelemetry ~=1.42.0 pins transitively cap protobuf<7.0, blocking co-installation #7287 in the provided snapshot.
  • Please apply the required llm-generated label when opening this PR.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 57d6377c-b7e1-46fb-bf9b-01c93e1c0ce6

📥 Commits

Reviewing files that changed from the base of the PR and between 143e902 and c8f55ac.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • lib/crewai-core/pyproject.toml
  • lib/crewai/pyproject.toml

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

Changes

OpenTelemetry dependency constraints

Layer / File(s) Summary
Widen OpenTelemetry version ranges
lib/crewai-core/pyproject.toml, lib/crewai/pyproject.toml
The three OpenTelemetry dependencies now allow versions from 1.42 up to, but excluding, 2.0, instead of only 1.42.x.

Suggested reviewers: greysonlalonde, lucasgomide

Merge Risk: ⚪ Minimal · up to e211a

This change widens aligned OpenTelemetry dependency ranges to permit compatible newer 1.x releases, including protobuf 7.x-compatible resolution. The package constraints and lockfile metadata are consistent, with no remaining merge-readiness risk identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the primary change: widening the OpenTelemetry dependency compatibility range.
Description check ✅ Passed The description includes the related issue, change summary, verification evidence, and maintainer context. The headings differ from the template, but all required information is present.
Linked Issues check ✅ Passed The changes satisfy issue #7287 by widening all three OpenTelemetry requirements in both distributions from ~=1.42.0 to >=1.42,<2. This permits newer compatible releases and addresses the protobuf 7.x…
Out of Scope Changes check ✅ Passed The reviewable changes are limited to the OpenTelemetry dependency constraints in crewai and crewai-core. They directly support issue #7287 and show no unrelated code changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kiwoongyoon
kiwoongyoon marked this pull request as ready for review September 5, 2026 20:58
@kiwoongyoon
kiwoongyoon force-pushed the gitcontribute/20260905-162306-580447-crewAIInc-crewAI-issue-7287 branch 6 times, most recently from b8fabcc to e211a2e Compare September 7, 2026 01:59
Signed-off-by: kiwoong <rldnddbs@naver.com>
@kiwoongyoon
kiwoongyoon force-pushed the gitcontribute/20260905-162306-580447-crewAIInc-crewAI-issue-7287 branch from e211a2e to 372e0ac Compare September 7, 2026 06:45
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.

[BUG] opentelemetry ~=1.42.0 pins transitively cap protobuf<7.0, blocking co-installation

2 participants