Skip to content

Conversation

bjorn-einar-bjartnes-4ss
Copy link
Contributor

@bjorn-einar-bjartnes-4ss bjorn-einar-bjartnes-4ss commented May 14, 2025

This PR is related to user story ESS-3438

Description

Update to support Python from 3 .11 to 3.13

@bjorn-einar-bjartnes-4ss
Copy link
Contributor Author

bjorn-einar-bjartnes-4ss commented May 14, 2025

Could be this
census-instrumentation/opencensus-python#1248
census-instrumentation/opencensus-python#1249

I tried just copying the changes from the PR into the included package, and that worked just fine. I would prefer to avoid doing any kind of patching of the package, so ideally we wait for this fix to be released. Or we move to https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/monitor/azure-monitor-opentelemetry-exporter or https://learn.microsoft.com/nb-no/azure/azure-monitor/app/opentelemetry-enable?tabs=python

The exporter is intended for users who require advanced configuration or have more complicated telemetry needs that require all of distributed tracing, logging and metrics. If you have simpler configuration requirements, we recommend using the [Azure Monitor OpenTelemetry Distro](https://learn.microsoft.com/azure/azure-monitor/app/opentelemetry-enable?tabs=python) instead for a simpler one-line setup.

There is even a migration guide https://learn.microsoft.com/nb-no/azure/azure-monitor/app/opentelemetry-python-opencensus-migrate

@bjorn-einar-bjartnes-4ss bjorn-einar-bjartnes-4ss marked this pull request as draft May 14, 2025 09:09
@bjorn-einar-bjartnes-4ss
Copy link
Contributor Author

Will move to Open Telemetry before upgrading to 3.13 #139

@bjorn-einar-bjartnes-4ss bjorn-einar-bjartnes-4ss marked this pull request as ready for review May 16, 2025 14:02
@bjorn-einar-bjartnes-4ss bjorn-einar-bjartnes-4ss merged commit 74d3a4d into master May 16, 2025
9 checks passed
@bjorn-einar-bjartnes-4ss bjorn-einar-bjartnes-4ss deleted the ESS-3438-3_11to_3_13 branch May 16, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant