Skip to content

Commit

Permalink
chore: enforce supported Python versions on pip installation (#19740)
Browse files Browse the repository at this point in the history
## Summary & Motivation
Resolves #17933.

## How I Tested These Changes
- Create a virtualenv of Python 3.12, see installation fail.
- Create a virtualenv of Python 3.7, see installation fail.
- Create a virtualenv of Python 3.8, see installation succeed.
- See Buildkite still succeed.
  • Loading branch information
rexledesma authored and PedramNavid committed Mar 28, 2024
1 parent eb15d6f commit eeef1e3
Show file tree
Hide file tree
Showing 57 changed files with 57 additions and 0 deletions.
1 change: 1 addition & 0 deletions python_modules/dagit/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ def get_version():
],
packages=find_packages(exclude=["dagit_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster-webserver{pin}",
],
Expand Down
1 change: 1 addition & 0 deletions python_modules/dagster-graphql/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_graphql_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"graphene>=3,<4",
Expand Down
1 change: 1 addition & 0 deletions python_modules/dagster-pipes/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,5 +35,6 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_pipes_tests*"]),
python_requires=">=3.8,<3.13",
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/dagster-test/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_test_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
"dagster",
"pyspark",
Expand Down
1 change: 1 addition & 0 deletions python_modules/dagster-webserver/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ def get_version():
],
packages=find_packages(exclude=["dagster_webserver_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
# cli
"click>=7.0,<9.0",
Expand Down
1 change: 1 addition & 0 deletions python_modules/dagster/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
# cli
"click>=5.0",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-airbyte/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_airbyte_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"requests",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-airflow/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_airflow_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"docker>=5.0.3,<6.0.0",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-aws/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_aws_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
"boto3",
f"dagster{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-azure/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_azure_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
"azure-core<2.0.0,>=1.7.0",
"azure-identity<2.0.0,>=1.7.0",
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 @@ -30,6 +30,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_celery_docker_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-celery{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-celery-k8s/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_celery_k8s_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-k8s{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-celery/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_celery_tests*"]),
entry_points={"console_scripts": ["dagster-celery = dagster_celery.cli:main"]},
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"celery>=4.3.0",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-census/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_census_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-dask/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_dask_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
"bokeh",
f"dagster{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-databricks/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_databricks_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-pipes{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-datadog/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_datadog_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "datadog"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-datahub/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_datahub_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
"acryl-datahub[datahub-rest, datahub-kafka]",
f"dagster{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-dbt/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_dbt_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
# Follow the version support constraints for dbt Core: https://docs.getdbt.com/docs/dbt-versions/core
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-deltalake-pandas/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_deltalake_pandas_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-deltalake{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-deltalake-polars/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_deltalake_polars_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-deltalake{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-deltalake/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_deltalake_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
"deltalake>=0.15",
f"dagster{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-docker/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_docker_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "docker", "docker-image-py"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-duckdb-pandas/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_duckdb_pandas_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-duckdb{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-duckdb-polars/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_duckdb_polars_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-duckdb{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-duckdb-pyspark/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_duckdb_pyspark_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-duckdb{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-duckdb/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_duckdb_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
"duckdb",
f"dagster{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-embedded-elt/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_embedded_elt_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "sling>=1.0.20"],
zip_safe=False,
extras_require={},
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-fivetran/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_fivetran_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}"],
zip_safe=False,
entry_points={
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-gcp-pandas/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_gcp_pandas_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-gcp{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-gcp-pyspark/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_gcp_pyspark_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-gcp{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-gcp/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_gcp_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster_pandas{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-ge/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_ge_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-pandas{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-github/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_github_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
# Using a Github app requires signing your own JWT :(
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-k8s/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_k8s_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"kubernetes",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-managed-elements/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_managed_elements_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "requests", "click_spinner"],
zip_safe=False,
entry_points={
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-mlflow/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_mlflow_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "mlflow", "pandas"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-msteams/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version():
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_msteams_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"requests>=2,<3",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-mysql/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ def get_version():
]
},
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "mysql-connector-python"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-pagerduty/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_pagerduty_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "pypd"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-pandas/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,5 +46,6 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_pandas_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "pandas"],
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-pandera/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
],
packages=find_packages(exclude=["dagster_pandera_tests*"]),
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "pandas", "pandera>=0.14.2"],
extras_require={
"test": [
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-papertrail/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_papertrail_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-polars/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_polars_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"polars>=0.20.0",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-postgres/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ def get_version() -> str:
]
},
include_package_data=True,
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "psycopg2-binary"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-prometheus/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_prometheus_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}", "prometheus_client"],
zip_safe=False,
)
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-pyspark/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_pyspark_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster_spark{pin}",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-shell/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_shell_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[f"dagster{pin}"],
extras_require={"test": ["psutil"]},
zip_safe=False,
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-slack/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def get_version():
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_slack_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
"slack_sdk",
Expand Down
1 change: 1 addition & 0 deletions python_modules/libraries/dagster-snowflake-pandas/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ def get_version() -> str:
"Operating System :: OS Independent",
],
packages=find_packages(exclude=["dagster_snowflake_pandas_tests*"]),
python_requires=">=3.8,<3.13",
install_requires=[
f"dagster{pin}",
f"dagster-snowflake{pin}",
Expand Down
Loading

0 comments on commit eeef1e3

Please sign in to comment.