Skip to content

Commit

Permalink
boostrap: no need to install tortoiseorm instrumentation if pydantic …
Browse files Browse the repository at this point in the history
…is found

Not every user of pydantic is using tortoise orm so checking for
tortoise orm should be enough.
  • Loading branch information
xrmx committed Apr 11, 2024
1 parent 804a909 commit d0f7e7a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,12 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## Unreleased

- `opentelemetry-sdk-extension-aws` Register AWS resource detectors under the
`opentelemetry_resource_detector` entry point
([#2382](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2382))
- `opentelemetry-instrumentation` On bootstrap don't install tortoise orm instrumentation if pydantic is found
([#2409](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2409))

### Breaking changes

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ dependencies = [
[project.optional-dependencies]
instruments = [
"tortoise-orm >= 0.17.0",
"pydantic >= 1.10.2"
]

[project.entry-points.opentelemetry_instrumentor]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -176,10 +176,6 @@
"library": "tortoise-orm >= 0.17.0",
"instrumentation": "opentelemetry-instrumentation-tortoiseorm==0.46b0.dev",
},
{
"library": "pydantic >= 1.10.2",
"instrumentation": "opentelemetry-instrumentation-tortoiseorm==0.46b0.dev",
},
{
"library": "urllib3 >= 1.0.0, < 3.0.0",
"instrumentation": "opentelemetry-instrumentation-urllib3==0.46b0.dev",
Expand Down

0 comments on commit d0f7e7a

Please sign in to comment.