Skip to content

build: proxy worker build & test setup #1664

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
Apr 11, 2025
Merged

build: proxy worker build & test setup #1664

merged 16 commits into from
Apr 11, 2025

Conversation

hallvictoria
Copy link
Contributor

@hallvictoria hallvictoria commented Apr 9, 2025 •
edited
Loading

Description

Pipeline Changes

  • Adds Python 3.13 to emulator & E2E test pipelines

Build Changes

  • Adds 3.13 to build
  • Updates mac/linux/windows build scripts to either copy protos from proxy_worker or azure_functions_worker, depending on python version
  • For <3.13, build includes azure_functions_worker ONLY. For 3.13+, build includes proxy_worker ONLY
  • Updates pyproject.toml to include grpc/protobuf versions for 3.13+
  • Adds library workers as dependencies for 3.13+

Test Changes

  • If python 3.13, imports from proxy_worker. Otherwise, imports from afw
  • Skips shared memory tests for 3.13

Fixes #

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • If applicable, the PR references the bug/issue that it fixes in the description.
  • New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

EvanR-Dev and others added 5 commits March 31, 2025 12:02
@EvanR-Dev @hallvictoria
build: recognize collection_model_binding_data for batch inputs (#1655)
1ae733c 
* add cmbd

* Add

* Add

* Rm newline

* Add tests

* Fix cmbd

* Fix test

* Lint

* Rm

* Rm

* Add back newline

* rm ws

* Rm list

* Rm cmbd from cache

* Avoid caching

* Keep cmbd check

* Add comment

* Lint

---------

Co-authored-by: Evan Roman <[email protected]>
Co-authored-by: hallvictoria <[email protected]>
@hallvictoria
build: update Python Worker Version to 4.36.1 (#1660)
9c34b22 
Co-authored-by: AzureFunctionsPython <[email protected]>
@hallvictoria
initial changes
004e125
@hallvictoria
merge conflicts
46bb600
@hallvictoria
Update Python SDK Version to 1.23.0 (#1663)
91494ba 
Co-authored-by: AzureFunctionsPython <[email protected]>
@hallvictoria hallvictoria changed the title Hallvictoria/proxy changes build: proxy worker build & test setup Apr 9, 2025
@hallvictoria hallvictoria marked this pull request as ready for review April 11, 2025 15:59
@hallvictoria hallvictoria requested a review from vrdmr as a code owner April 11, 2025 15:59
@hallvictoria hallvictoria merged commit be60b01 into gaaguiar/proxy_worker Apr 11, 2025
7 checks passed
@hallvictoria hallvictoria deleted the hallvictoria/proxy-changes branch April 11, 2025 16:27
gavin-aguiar added a commit that referenced this pull request Apr 25, 2025
@gavin-aguiar @hallvictoria @EvanR-Dev
feat: Adding support for python 3.13 (#1666)
c3899d8 
* Proxy Worker: Initial Commit

* Updated worker config to include 3.13

* Updated test_setup

* Updated worker.py

* Updated dispatcher

* Updated syspath in worker.py

* Updated path in worker.py

* Updated worker.py

* Removed reload in dispatcher

* Updating v1 library worker name

* Added dispatcher logs

* Added dispatcher try/catch logs

* Updated sys path

* Dispatcher and dependency manager updates

* Updated dispatcher and pyproject

* Testing updates and refactoring

* Bug fixes and refactoring

* Added more unit tests

* Added tests and fixed test setup

* Updated test_setup

* Updated test setup to add grpc dir copy

* build: proxy worker build & test setup (#1664)

* build: recognize collection_model_binding_data for batch inputs (#1655)

* add cmbd

* Add

* Add

* Rm newline

* Add tests

* Fix cmbd

* Fix test

* Lint

* Rm

* Rm

* Add back newline

* rm ws

* Rm list

* Rm cmbd from cache

* Avoid caching

* Keep cmbd check

* Add comment

* Lint

---------

Co-authored-by: Evan Roman <[email protected]>
Co-authored-by: hallvictoria <[email protected]>

* build: update Python Worker Version to 4.36.1 (#1660)

Co-authored-by: AzureFunctionsPython <[email protected]>

* initial changes

* Update Python SDK Version to 1.23.0 (#1663)

Co-authored-by: AzureFunctionsPython <[email protected]>

* merges from ADO

* merge fixes

* merge fixes

* merge fixes

* merge fixes

* don't run 313 unit tests yet

* changes for builds

---------

Co-authored-by: Evan <[email protected]>
Co-authored-by: Evan Roman <[email protected]>
Co-authored-by: AzureFunctionsPython <[email protected]>
Co-authored-by: AzureFunctionsPython <[email protected]>

* Merging changes

* linting fixes

* Addressed comments

* Updated unit test and added missing protos files

* fix e2e test reference

* lint, mypy, add 3.13 to unittests

* correct version check

* syntax

* syntax

* fix unit tests, mypy

* oops

* format

* lint

* fix unittest dir for proxy

* set env variable

* update pyproject to use real deps

* bump to a2

* bump v2 to a3

* Import v2 by default for LC

* Refactoring and minor fixes

---------

Co-authored-by: hallvictoria <[email protected]>
Co-authored-by: Evan <[email protected]>
Co-authored-by: Evan Roman <[email protected]>
Co-authored-by: AzureFunctionsPython <[email protected]>
Co-authored-by: AzureFunctionsPython <[email protected]>
Co-authored-by: hallvictoria <[email protected]>
Co-authored-by: Victoria Hall <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gavin-aguiar gavin-aguiar gavin-aguiar approved these changes

@vrdmr vrdmr Awaiting requested review from vrdmr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hallvictoria @gavin-aguiar @EvanR-Dev