Skip to content
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

[server] pin anyio support to <4.0.0 #1222

Merged
merged 1 commit into from Aug 31, 2023
Merged

Conversation

bfineran
Copy link
Member

this is for compatibility with the maximum currently allowed version of starlette. anyio just pushed a 4.0.0 that is incompatible with earlier versions of starlette.

Note that this is now fixed in later versions (deepsparse has not yet tested against newer versions of fastapi and starlette). See: encode/starlette#1936

from internal bug report:

Our Deepsparse GHA is failing with starlette error: AttributeError: module 'anyio' has no attribute 'start_blocking_portal'

this is for compatibility with the maximum currently allowed version of `starlette`. `anyio` just pushed a 4.0.0 that is incompatible with earlier versions of `starlette`. 

Note that this is now fixed in later versions (`deepsparse` has not yet tested against newer versions of `fastapi` and `starlette`). See: encode/starlette#1936


from internal bug report:
```
Our Deepsparse GHA is failing with starlette error: AttributeError: module 'anyio' has no attribute 'start_blocking_portal'
```
@bfineran bfineran self-assigned this Aug 31, 2023
Copy link
Member

@mgoin mgoin left a comment

Choose a reason for hiding this comment

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

Looks good from GHA testing for that error. There are still other asserts for another issue

@bfineran
Copy link
Member Author

GHA failure looks unrelated and on the sparsezoo side - pushing this patch

@bfineran bfineran merged commit 136b9f8 into main Aug 31, 2023
6 of 7 checks passed
@bfineran bfineran deleted the anyio-pin-version-4.0.0 branch August 31, 2023 18:54
@bfineran
Copy link
Member Author

Confirmed with @tdg5 the issue may be due to an internal migration. Issue is now tracked internally with fix underway

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.

None yet

2 participants