Skip to content

bump: opentelemetry-sdk opentelemetry-exporter-otlp-proto-http 1.33.1 #1829

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

maxday
Copy link
Member

@maxday maxday commented May 24, 2025

Description

Dependabot is failing (#1819) to bump opentelemetry-sdk and opentelemetry-exporter-otlp-proto-http from 1.33.0 to 1.33.1 and failing with this error:

The conflict is caused by:
    The user requested opentelemetry-semantic-conventions==0.54b0
    opentelemetry-sdk 1.33.1 depends on opentelemetry-semantic-conventions==0.54b1

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

Error: CustomMakeBuilder:MakeBuild - Make Failed: ERROR: Cannot install -r /tmp/samcli/scratch/otel_sdk/requirements.txt (line 1) and opentelemetry-semantic-conventions==0.54b0 because these package versions have conflicting dependencies.
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Notice:  A new release of pip is available: 23.0.1 -> 25.0.1
Notice:  To update, run: pip install --upgrade pip
make: *** [build-OTelLayer] Error 1

source: https://github.com/open-telemetry/opentelemetry-lambda/actions/runs/15115839506/job/42486168032?pr=1819

This PR fixes the conflict (the same way as it was done before, for instance here: bc9970f)

Testing

I ran ./run.sh -b locally and it's now passing

@serkan-ozal
Copy link
Contributor

Thanks for taking care of this. I was planning to do before release, but couldn't find time to check it out.

@serkan-ozal serkan-ozal merged commit 344b49b into open-telemetry:main May 24, 2025
17 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.

2 participants