Skip to content

Workaround bug in google-claude-aiplatform evaluation extra (Python 3… - #71273

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:workaround-evaluation-extra-limits
Aug 7, 2026
Merged

Workaround bug in google-claude-aiplatform evaluation extra (Python 3…#71273
potiuk merged 1 commit into
apache:mainfrom
potiuk:workaround-evaluation-extra-limits

Conversation

@potiuk

@potiuk potiuk commented Aug 7, 2026

Copy link
Copy Markdown
Member

….15)

The "evaluation" extra of google-cloud-aiplatform limits litellm to range that has security vulnerabilities when we want to get 3.14 support. Until this is fixed we need to bump litellm to higher version and remove the "evaluation" extra from google-cloud-aiplatform.

Tracked in: googleapis/python-aiplatform#7057


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@potiuk
potiuk requested a review from shahar1 as a code owner August 7, 2026 09:21
@boring-cyborg boring-cyborg Bot added area:providers provider:google Google (including GCP) related issues labels Aug 7, 2026
@potiuk
potiuk requested a review from vatsrahul1001 August 7, 2026 09:22
@potiuk potiuk added the all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs label Aug 7, 2026
@potiuk potiuk closed this Aug 7, 2026
@potiuk potiuk reopened this Aug 7, 2026
@potiuk

potiuk commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Reopened for 3.14 version

@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Aug 7, 2026
@potiuk
potiuk force-pushed the workaround-evaluation-extra-limits branch from a91baa5 to ec9955a Compare August 7, 2026 10:01
….15)

The "evaluation" extra of google-cloud-aiplatform limits litellm to range that
has security vulnerabilities when we want to get 3.14 support. Until this is
fixed we need to bump litellm to higher version and remove the "evaluation"
extra from google-cloud-aiplatform.

Tracked in: googleapis/python-aiplatform#7057
@potiuk
potiuk force-pushed the workaround-evaluation-extra-limits branch from ec9955a to 613b169 Compare August 7, 2026 15:10
@potiuk
potiuk merged commit 2c9d91e into apache:main Aug 7, 2026
284 checks passed
@potiuk
potiuk deleted the workaround-evaluation-extra-limits branch August 7, 2026 17:08
Vamsi-klu added a commit to Vamsi-klu/airflow that referenced this pull request Aug 11, 2026
Reworks the optional Vertex AI evaluation change on top of apache#71273
(2c9d91e), which had removed the google-cloud-aiplatform[evaluation]
extra from the base deps and inlined the evaluation requirements
directly instead, as a workaround for the litellm security range
tracked in googleapis/python-aiplatform#7057.

Rather than reverting that, the inlined block -- including the litellm
python_version markers and the tracking comment -- moves verbatim into a
new "evaluation" entry under [project.optional-dependencies], so base
installs stop pulling litellm and scikit-learn while the 3.14 security
pin is preserved for anyone installing the extra.

providers/google/README.rst and docs/index.rst tables are regenerated to
match, and uv.lock is regenerated with the pinned uv 0.11.29 (uv lock
--check passes; note main's lock was already stale, so the diff also
picks up a few unrelated requires-dist specifiers).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:providers full tests needed We need to run full set of tests for this PR to merge provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants