Skip to content

Allow installing SDK with API v1.5#1708

Merged
brettmc merged 1 commit into
open-telemetry:mainfrom
IonBazan:patch-1
Sep 5, 2025
Merged

Allow installing SDK with API v1.5#1708
brettmc merged 1 commit into
open-telemetry:mainfrom
IonBazan:patch-1

Conversation

@IonBazan

@IonBazan IonBazan commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

Currently open-telemetry/sdk locks open-telemetry/api at 1.4.0 and does not allow upgrades to 1.5.0 due to incorrect version constraint. Fixes #1706

  Problem 1
    - Root composer.json requires open-telemetry/sdk ^1.7 -> satisfiable by open-telemetry/sdk[1.7.0].
    - open-telemetry/sdk 1.7.0 requires open-telemetry/api ~1.4.0 -> found open-telemetry/api[1.4.0] but the package is fixed to 1.5.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
  Problem 2
    - open-telemetry/exporter-otlp is locked to version 1.3.2 and an update of this package was not requested.
    - open-telemetry/exporter-otlp 1.3.2 requires open-telemetry/sdk ^1.0 -> satisfiable by open-telemetry/sdk[1.7.0].
    - open-telemetry/sdk 1.7.0 requires open-telemetry/api ~1.4.0 -> found open-telemetry/api[1.4.0] but the package is fixed to 1.5.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.

the only way to update at the moment is to downgrade the API package:

$ composer u open-telemetry/sdk -W
Loading composer repositories with package information
Updating dependencies
Lock file operations: 0 installs, 2 updates, 0 removals
  - Downgrading open-telemetry/api (1.5.0 => 1.4.0)
  - Upgrading open-telemetry/sdk (1.5.0 => 1.7.0)
Writing lock file

@IonBazan
IonBazan requested a review from a team as a code owner September 5, 2025 06:57
@welcome

welcome Bot commented Sep 5, 2025

Copy link
Copy Markdown

Thanks for opening your first pull request! If you haven't yet signed our Contributor License Agreement (CLA), then please do so that we can accept your contribution. A link should appear shortly in this PR if you have not already signed one.

@codecov

codecov Bot commented Sep 5, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.40%. Comparing base (0bde621) to head (a3d5a51).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1708      +/-   ##
============================================
- Coverage     68.51%   68.40%   -0.11%     
  Complexity     2878     2878              
============================================
  Files           430      430              
  Lines          8759     8759              
============================================
- Hits           6001     5992       -9     
- Misses         2758     2767       +9
Flag Coverage Δ
8.1 68.12% <ø> (-0.02%) ⬇️
8.2 68.31% <ø> (-0.02%) ⬇️
8.3 68.36% <ø> (-0.11%) ⬇️
8.4 68.31% <ø> (+0.01%) ⬆️
8.5 68.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0bde621...a3d5a51. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brettmc
brettmc merged commit 2f3e214 into open-telemetry:main Sep 5, 2025
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brettmc brettmc brettmc approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

open-telemestry/sdk locked on open-telemetry/api 1.4

2 participants

@IonBazan @brettmc