Skip to content

Conversation

@SinfulPhantom
Copy link

Was working on integrating OpenTelemetry into my project at work. We are currently working with OpenCensus, so I ported it to OpenTelemetry.

I figured I'd open a PR with an example of what worked for me with using OpenTelemetry with Azure on a back-end application.

import requests
from opentelemetry import trace
from opentelemetry.ext import http_requests
from opentelemetry.ext.azure_monitor import AzureMonitorSpanExporter
Copy link
Member

Choose a reason for hiding this comment

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

We are deprecating opentelemetry.ext.azure_monitor in favor of opentelemetry-azure-monitor-exporter

#272

@mauriciovasquezbernal
Copy link
Member

I think the problem is not just to switch the package name, the whole folder ext/opentelemetry-ext-azure-monitor will be removed in #272. Probably this repo is not the right place for this example, maybe https://github.com/microsoft/opentelemetry-exporters-python/tree/master/azure_monitor?
@lzchen what do you think?

@c24t
Copy link
Member

c24t commented Nov 12, 2019

Thanks for the PR @SinfulPhantom. The example looks good, but now that the exporter is being moved out this isn't the right repo for it.

@c24t c24t closed this Nov 12, 2019
@SinfulPhantom SinfulPhantom deleted the azureRequestsExample branch November 12, 2019 20:01
@SinfulPhantom
Copy link
Author

Moved PR over to:
microsoft/opentelemetry-azure-monitor-python#11

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.

4 participants