Skip to content

Commit

Permalink
Fix openai 1.32 breaking openai tests (#40110)
Browse files Browse the repository at this point in the history
The new openai release adds new required parameter `parallel_tool_calls`
and our mock did not have it. Bumping version and adding the parameter
should solve the problem. It turned out also that uv==0.2.6 introduced
a bug for lowest-direct resolution, so we need to limit it to 0.2.5.

Details in astral-sh/uv#4136

(cherry picked from commit 65dbf86)
  • Loading branch information
potiuk authored and utkarsharma2 committed Jul 2, 2024
1 parent 2fe6483 commit 2b15ecf
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,8 @@ ARG AIRFLOW_VERSION="2.9.0"
ARG PYTHON_BASE_IMAGE="python:3.8-slim-bookworm"

ARG AIRFLOW_PIP_VERSION=24.0
ARG AIRFLOW_UV_VERSION=0.2.9
# until https://github.com/astral-sh/uv/issues/4136 is solved we limit UV to version 0.2.5
ARG AIRFLOW_UV_VERSION=0.2.5
ARG AIRFLOW_USE_UV="false"
ARG UV_HTTP_TIMEOUT="300"
ARG AIRFLOW_IMAGE_REPOSITORY="https://github.com/apache/airflow"
Expand Down
6 changes: 4 additions & 2 deletions Dockerfile.ci
Original file line number Diff line number Diff line change
Expand Up @@ -1224,7 +1224,8 @@ ARG DEFAULT_CONSTRAINTS_BRANCH="constraints-main"
ARG AIRFLOW_CI_BUILD_EPOCH="10"
ARG AIRFLOW_PRE_CACHED_PIP_PACKAGES="true"
ARG AIRFLOW_PIP_VERSION=24.0
ARG AIRFLOW_UV_VERSION=0.2.9
# until https://github.com/astral-sh/uv/issues/4136 is solved we limit UV to version 0.2.5
ARG AIRFLOW_UV_VERSION=0.2.5
ARG AIRFLOW_USE_UV="true"
# Setup PIP
# By default PIP install run without cache to make image smaller
Expand All @@ -1248,7 +1249,8 @@ ARG AIRFLOW_VERSION=""
ARG ADDITIONAL_PIP_INSTALL_FLAGS=""

ARG AIRFLOW_PIP_VERSION=24.0
ARG AIRFLOW_UV_VERSION=0.2.9
# until https://github.com/astral-sh/uv/issues/4136 is solved we limit UV to version 0.2.5
ARG AIRFLOW_UV_VERSION=0.2.5
ARG AIRFLOW_USE_UV="true"

ENV AIRFLOW_REPO=${AIRFLOW_REPO}\
Expand Down

0 comments on commit 2b15ecf

Please sign in to comment.