Skip to content

Relax the importlib-metadata pin#5191

Open
dhirschfeld wants to merge 4 commits intoopen-telemetry:mainfrom
dhirschfeld:patch-1
Open

Relax the importlib-metadata pin#5191
dhirschfeld wants to merge 4 commits intoopen-telemetry:mainfrom
dhirschfeld:patch-1

Conversation

@dhirschfeld
Copy link
Copy Markdown

@dhirschfeld dhirschfeld commented May 8, 2026

Description

This pin was causing me to have some package resolution errors with pixi. They're easily worked around by applying my own pins locally, but the better solution is for opentelemetry-api to just support the latest importlib-metadata.

Type of change

Update dependencies (relax existing pin)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

CI tests still pass

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added N/A
  • Documentation has been updated N/A

@dhirschfeld dhirschfeld requested a review from a team as a code owner May 8, 2026 13:41
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 8, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

@dhirschfeld
Copy link
Copy Markdown
Author

8.8.0 dropped Python 3.9 support:

image

https://importlib-metadata.readthedocs.io/en/latest/history.html


That should be fine since Python 3.9 has also bee dropped here:

@dhirschfeld
Copy link
Copy Markdown
Author

I see this is also being discussed in:

@herin049
Copy link
Copy Markdown
Contributor

herin049 commented May 8, 2026

Make sure to update uv.lock

@github-project-automation github-project-automation Bot moved this to Approved PRs in Python PR digest May 8, 2026
@dhirschfeld
Copy link
Copy Markdown
Author

Make sure to update uv.lock

Ah right 👍

I'm happy for anyone else to push that commit so this can be merged, otherwise I can get to that, hopefully tonight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved PRs

Development

Successfully merging this pull request may close these issues.

2 participants