Skip to content

Upgrade OpenTelemetry dependencies#36

Open
mladjan-gadzic wants to merge 4 commits intomasterfrom
nojira/upgrade-otel-deps
Open

Upgrade OpenTelemetry dependencies#36
mladjan-gadzic wants to merge 4 commits intomasterfrom
nojira/upgrade-otel-deps

Conversation

@mladjan-gadzic
Copy link
Copy Markdown
Contributor

What

  • remove bounds for OTel deps
  • refactor code to comply with OTel breaking changes in the latest version

Why

OTel world introduced breaking changes in the latest version.

Todo (in separate PR):

  • in separate PR bump up version to v0.1.0
  • after above PR is merged, create v0.1.0 tag and publish new version

pyproject.toml Outdated
"opentelemetry-exporter-otlp-proto-grpc",
"opentelemetry-exporter-otlp-proto-http",
"opentelemetry-proto",
"opentelemetry-sdk",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would definitely like to see a lower bound on at least this one please - hopefully not bleeding edge but whatever suffices for everything we use

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like we may need 1.39

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on the second thought, let's be safe and go with 1.39 everywhere. it's lower bound in the end.

"opentelemetry-exporter-otlp-proto-grpc>=1.39.0",
"opentelemetry-exporter-otlp-proto-http>=1.39.0",
"opentelemetry-proto>=1.39.0",
"opentelemetry-sdk>=1.39.0",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This implies a number of the other deps and bounds so I'd only tend to declare the ones we consume directly - but this is looking good now, no need to change.

(See https://github.com/open-telemetry/opentelemetry-python/blob/release/v1.39.x-0.60bx/opentelemetry-sdk/pyproject.toml)

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