fix(otel): remove OpenTelemetry upper bounds - #609
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Claude AI reviewNo actionable findings. This PR removes the temporary I verified:
Residual test risk: this only affects dependency resolution, not the pinned versions actually exercised. CI installs continue to resolve whatever the resolver picks, so newer OpenTelemetry releases (>1.42.1) are not directly pinned/tested by this change — a future breaking OTel release could surface only at consumer install/runtime rather than in this repo's test matrix. That is an accepted trade-off of removing the cap and not a defect in the diff. Reviewed commit |
Codex AI reviewNo actionable findings. Residual risk is limited to compatibility with future, not-yet-tested OpenTelemetry releases admitted by the widened constraints. Tests were not run per review constraints. Reviewed commit |
Summary
<=1.42.1upper bounds from package dependenciesWhy
The cap was added to remain compatible with the ADOT Lambda layer. Upstream fixed the incompatibility in AWS OTel Python instrumentation v0.19.0, so retaining the restriction would unnecessarily prevent consumers and development environments from resolving newer OpenTelemetry releases.
Impact
Consumers can use OpenTelemetry versions newer than 1.42.1 while the existing minimum API and SDK version remains 1.20.0.
Validation
hatch run dev-otel:test(105 passed)