Skip to content

Commit

Permalink
Unpin grpcio for Ray Client (#37368)
Browse files Browse the repository at this point in the history
It seems like the bug grpc/grpc#31885 that caused the problems with Ray Client tests has been fixed in grpcio 1.56, so we are removing the pin so people can upgrade to fix https://nvd.nist.gov/vuln/detail/CVE-2023-32731

Pinning to just the latest version would be too restrictive so we remove the pin (since the Ray client works with other versions as well except for some corner cases).
  • Loading branch information
pcmoritz authored Jul 13, 2023
1 parent da39640 commit 61401da
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
1 change: 0 additions & 1 deletion python/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ requests
pandas
tensorboardX<=2.6.0,>=1.9 # >=2.6.1 uses protobuf>=4, and conflicts with other packages.
gymnasium==0.26.3
grpcio<=1.50.0,>=1.42.0 # ray client
aiohttp>=3.7
starlette
typer
Expand Down
4 changes: 1 addition & 3 deletions python/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -263,9 +263,7 @@ def get_packages(self):
"virtualenv >=20.0.24, < 20.21.1", # For pip runtime env.
],
"client": [
# The Ray client needs a specific range of gRPC to work:
# Tracking issue: https://github.com/grpc/grpc/issues/31885
"grpcio >= 1.42.0, <= 1.50.0",
"grpcio",
],
"serve": ["uvicorn", "requests", "starlette", "fastapi", "aiorwlock"],
"tune": ["pandas", "tensorboardX>=1.9", "requests", pyarrow_dep],
Expand Down

0 comments on commit 61401da

Please sign in to comment.