-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add env var to control sending telemetry to appinsights in functions worker #1621
Conversation
If both |
Good question. We do not safeguard against double instrumentation as this is an unsupported scenario. We advise customers to either turn the appsetting on OR use |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The unit tests are failing because of |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Follow up to #1509
We have decided to separate out the context propagation and enabling azure monitor functionality. To do this, we will be using a separate env var
PYTHON_APPLICATIONINSIGHTS_ENABLE_TELEMETRY
, which will be for strictly sending telemetry to appinsights and as well enabling context propagation. Future configuration of the telemetry SDK will most likely have the word_APPLICATIONINSIGHTS_
inside it's name.We treating Opentelemetry context propagation as a separate feature. This will be controlled as an opt-in like before with
PYTHON_ENABLE_OPENTELEMETRY
env var.