-
Notifications
You must be signed in to change notification settings - Fork 108
feat: V2 library worker #1706
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
base: dev
Are you sure you want to change the base?
feat: V2 library worker #1706
Conversation
"grpcio~=1.70.0", | ||
"grpcio-tools~=1.70.0", | ||
"pytest-sugar", | ||
"opentelemetry-api", # Used for OpenTelemetry unit tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check if azure-monitor-opentelemetry installs opentelemetry-api. If it does then this can be removed
] | ||
|
||
[project.urls] | ||
Documentation = "https://github.com/Azure/azure-functions-python-worker/blob/hallvictoria/library-worker/README.md" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is pointing to your branch
| Runtime | Script Host & Language Extensibility | [File an Issue](https://github.com/Azure/azure-functions-host/issues) | | ||
| VSCode | VSCode Extension for Azure Functions | [File an Issue](https://github.com/microsoft/vscode-azurefunctions/issues) | | ||
| Core Tools | Command Line Interface for Local Development | [File an Issue](https://github.com/Azure/azure-functions-core-tools/issues) | | ||
| Portal | User Interface or Experience Issue | [File an Issue](https://github.com/azure/azure-functions-ux/issues) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the portal item. The repo does not exist
| Item | Description | Link | | ||
|---------------|----------------------------------------------|--------------------------------------------------------------------------------| | ||
| Python Worker | Programming Model, Triggers & Bindings | [File an Issue](https://github.com/Azure/azure-functions-python-worker/issues) | | ||
| Linux | Base Docker Images | [File an Issue](https://github.com/Azure/azure-functions-docker/issues) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed
from .utils.tracing import serialize_exception | ||
from .utils.validators import validate_script_file_name | ||
|
||
metadata_result: Optional[List] = None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add the _ for all variables.
return is_false_like(os.environ[env_key]) | ||
|
||
|
||
def get_app_setting( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have an app setting manager. Can we move this there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or somehow combine both?
@@ -0,0 +1,295 @@ | |||
# type: ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add this as a dependency?
|
||
|
||
@attach_message_to_exception( | ||
expt_type=ImportError, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this needs to be fixed. It does not catch ModuleNotFound errors
@@ -0,0 +1,2 @@ | |||
# Required dependencies listed in pyproject.toml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this file?
jobs: | ||
- template: /eng/templates/jobs/build.yml@self | ||
parameters: | ||
PYTHON_VERSION: '3.11' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you use 3.13?
Description
Adds V2 library worker code.
Sample official build: https://dev.azure.com/azfunc/internal/_build/results?buildId=226215&view=results
Fixes #
Pull Request Checklist
Host-Worker Contract
Worker Execution Logic
If yes, please answer the following:
Python Version Coverage
Programming Model Compatibility (for Python 3.13+)