Skip to content

Conversation

@satyadevai
Copy link
Collaborator

@satyadevai satyadevai commented Dec 1, 2025

Closes #2491


Note

Increase tox step timeout from 10 to 15 minutes in Python CI and cron workflows.

  • CI:
    • Increase timeout-minutes from 10 to 15 for tox runs in:
      • .github/workflows/python-CI.yaml
      • .github/workflows/python-cron.yaml

Written by Cursor Bugbot for commit bab6695. This will update automatically on new commits. Configure here.

@satyadevai satyadevai marked this pull request as ready for review December 1, 2025 16:57
@satyadevai satyadevai requested a review from a team as a code owner December 1, 2025 16:57
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 1, 2025
- run: uvx --with tox-uv tox run -e ${{ matrix.testenv }}
working-directory: python
timeout-minutes: 10
timeout-minutes: 15
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are the tests actually taking more than 10 minutes now consistently?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @nate,

Yes, the tests that involve installing the LiteLLM package are consistently taking more than 10 minutes. This is because grpcio==1.67.1 does not have prebuilt wheels for Python 3.14, so the CI has to build it from source. That build step alone often exceeds the 10-minute limit, which is why the pipelines are timing out. After increasing the timeout to 15 minutes, these jobs are able to complete.

The actual tests themselves are not taking that long, the delay is specifically due to the grpcio build process on Python 3.14.

@satyadevai satyadevai changed the title fix: Ci Failures for litellm chore: Ci Failures for litellm Dec 2, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 4, 2025
@caroger caroger merged commit 7c9c553 into Arize-ai:main Dec 4, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[bug]CI Failures for litellm

3 participants