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

Pin OpenAI Python client to <1.66.0 #47643

Merged
merged 1 commit into from
Mar 11, 2025
Merged

Conversation

kaxil
Copy link
Member

@kaxil kaxil commented Mar 11, 2025

OpenAI provider Tests are failing on main with:

___ ERROR collecting providers/openai/tests/unit/openai/hooks/test_openai.py ___
ImportError while importing test module '/opt/airflow/providers/openai/tests/unit/openai/hooks/test_openai.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/local/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
providers/openai/tests/unit/openai/hooks/test_openai.py:25: in <module>
    from openai.types.beta import (
E   ImportError: cannot import name 'VectorStore' from 'openai.types.beta' (/usr/local/lib/python3.9/site-packages/openai/types/beta/__init__.py

This is due to openai/openai-python#2175

Created GH issue to unpin in future: #47642


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

OpenAI provider Tests are failing with:

```
___ ERROR collecting providers/openai/tests/unit/openai/hooks/test_openai.py ___
ImportError while importing test module '/opt/airflow/providers/openai/tests/unit/openai/hooks/test_openai.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/local/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
providers/openai/tests/unit/openai/hooks/test_openai.py:25: in <module>
    from openai.types.beta import (
E   ImportError: cannot import name 'VectorStore' from 'openai.types.beta' (/usr/local/lib/python3.9/site-packages/openai/types/beta/__init__.py
```

This is due to openai/openai-python#2175
@kaxil kaxil changed the title Pin OpenAI Python client to <1.66.0 Pin OpenAI Python client to <1.66.0 Mar 11, 2025
@potiuk
Copy link
Member

potiuk commented Mar 11, 2025

Merging for openai errors - others are still to be fixed.

@potiuk potiuk merged commit b364cf8 into apache:main Mar 11, 2025
141 of 149 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants