-
Notifications
You must be signed in to change notification settings - Fork 251
Extension for initializing OpenCensus tracer into Azure Functions #1010
Extension for initializing OpenCensus tracer into Azure Functions #1010
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
This PR would also benefit from #886 |
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
contrib/opencensus-ext-azure/opencensus/ext/azure/integration/__init__.py
Outdated
Show resolved
Hide resolved
contrib/opencensus-ext-azure/opencensus/ext/azure/integration/azure_functions.py
Outdated
Show resolved
Hide resolved
contrib/opencensus-ext-azure/opencensus/ext/azure/integration/azure_functions.py
Outdated
Show resolved
Hide resolved
contrib/opencensus-ext-azure/opencensus/ext/azure/integration/azure_functions.py
Outdated
Show resolved
Hide resolved
It would be great if you could add that example to the examples folder and as well as some getting started directions (including linking to getting started with azure functions, as well as setting |
ac20d85
to
37edb38
Compare
contrib/opencensus-ext-azure/opencensus/ext/azure/extension/azure_functions.py
Outdated
Show resolved
Hide resolved
contrib/opencensus-ext-azure/opencensus/ext/azure/extension/azure_functions.py
Outdated
Show resolved
Hide resolved
Rename file OK Update implementation to AppLevel extension Add license info Fix syntax Raise more concrete exceptions Address PR comment Add example Add Python VSCode link in example Add requirements.txt section prettify Fix docstring Address PR issues Don't use specific ingestion endpoint Add new endline Update CHANGELOG.md
cb1d5e2
to
8e89c36
Compare
Background
Currently, in Azure Functions, customers have a hard time initializing OpenCensus. The customer needs to write an initializing section before implementing the
def main()
function.With this PR, Azure Functions exposes an extension interface to install the OpenCensus extension. Related changes in Azure Functions Python Worker and SDK are listed as follow:
Usage
PYTHON_ENABLE_WORKER_EXTENSIONS = 1
in Azure Functions setting or local dev environment variable.Example code:
/cc: @lzchen