Skip to content

Conversation

@IonBazan
Copy link
Contributor

@IonBazan IonBazan commented Sep 5, 2025

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
Copy link

welcome bot commented Sep 5, 2025

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
Copy link

codecov bot commented Sep 5, 2025

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants