Skip to content

Commit

Permalink
Pin requests < 2.32.0 (#21977)
Browse files Browse the repository at this point in the history
While we investigate breaking docker tests:

https://buildkite.com/dagster/dagster-dagster/builds/83562
  • Loading branch information
jmsanders authored and nikomancy committed May 22, 2024
1 parent e62e928 commit 1c06956
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,7 @@ grpcio<1.48.1; python_version < '3.10'
sqlalchemy<2.0.0

# Added pendulum pin in later versions
pendulum<3
pendulum<3

# 2.32.0 breaks our docker tests https://buildkite.com/dagster/dagster-dagster/builds/83562
requests<2.32.0
1 change: 1 addition & 0 deletions python_modules/dagster/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,7 @@ def get_version() -> str:
"morefs[asynclocal]",
"fsspec<2024.5.0", # morefs incompatibly
"rapidfuzz",
"requests<2.32.0", # 2.32.0 breaks our docker tests https://buildkite.com/dagster/dagster-dagster/builds/83562
],
"mypy": ["mypy==1.8.0"],
"pyright": [
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-celery-docker/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ def get_version() -> str:
f"dagster-celery{pin}",
f"dagster-graphql{pin}",
"docker",
"requests<2.32.0", # 2.32.0 breaks our docker tests https://buildkite.com/dagster/dagster-dagster/builds/83562
],
zip_safe=False,
)
7 changes: 6 additions & 1 deletion python_modules/libraries/dagster-docker/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,11 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_docker_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "docker", "docker-image-py"],
install_requires=[
f"dagster{pin}",
"docker",
"docker-image-py",
"requests<2.32.0", # 2.32.0 breaks our docker tests https://buildkite.com/dagster/dagster-dagster/builds/83562
],
zip_safe=False,
)

0 comments on commit 1c06956

Please sign in to comment.