Skip to content

Commit

Permalink
Add pyopenssl pin to dagster-snowflake to work around breaking upstre…
Browse files Browse the repository at this point in the history
…am package change (#26172)

Summary:
Workaround for breaking pin change in the snowflake-connector-python
package - see linked issue

Test Plan:
BK (assets_smoke_test should now pass)
  • Loading branch information
gibsondan authored Nov 27, 2024
1 parent 5c07c70 commit 0306bc8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
5 changes: 4 additions & 1 deletion examples/temp_pins.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,7 @@
responses==0.23.1

# pydantic 2.9.0 release briefly broke dagster
pydantic<2.9.0
pydantic<2.9.0

# https://github.com/snowflakedb/snowflake-connector-python/issues/2109
pyOpenSSL>=22.1.0
3 changes: 3 additions & 0 deletions python_modules/libraries/dagster-snowflake/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@ def get_version() -> str:
install_requires=[
f"dagster{pin}",
"snowflake-connector-python>=3.4.0",
# Workaround for incorrect pin in the snowflake-connector-python package
# See https://github.com/snowflakedb/snowflake-connector-python/issues/2109
"pyOpenSSL>=22.1.0",
],
extras_require={
"snowflake.sqlalchemy": [
Expand Down

0 comments on commit 0306bc8

Please sign in to comment.