Skip to content

Bump run_cudf_polars_pytests timeout - #22505

Merged
rapids-bot[bot] merged 4 commits into
NVIDIA:mainfrom
TomAugspurger:tom/cudf-polars-timeout-bump
May 18, 2026
Merged

Bump run_cudf_polars_pytests timeout#22505
rapids-bot[bot] merged 4 commits into
NVIDIA:mainfrom
TomAugspurger:tom/cudf-polars-timeout-bump

Conversation

@TomAugspurger

Copy link
Copy Markdown
Contributor

Description

https://github.com/rapidsai/cudf/actions/runs/25847270404/job/76002739708 is a recent job where the cudf-polars tests exited abruptly. That job started around 05/14/26 14:54:16 and ended around 05/14/26 15:09:16, roughly 15 minutes later.

This PR bumps the timeout we give for running that script from 15 minutes to 25 minutes.

https://github.com/rapidsai/cudf/actions/runs/25847270404/job/76002739708
is a recent job where the cudf-polars tests exited abrupty. That job
started around 05/14/26 14:54:16 and ended around 05/14/26 15:09:16,
roughly 15 minutes later.

This PR bumps the timeout on that job.
@TomAugspurger
TomAugspurger requested a review from a team as a code owner May 14, 2026 15:42
@TomAugspurger
TomAugspurger requested a review from gforsyth May 14, 2026 15:42
@TomAugspurger TomAugspurger added non-breaking Non-breaking change improvement Improvement / enhancement to an existing function labels May 14, 2026
@coderabbitai

coderabbitai Bot commented May 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 51121c72-013d-4e4f-8b2b-21e7947264c7

📥 Commits

Reviewing files that changed from the base of the PR and between 1d451cf and 4ac3cdb.

📒 Files selected for processing (1)
  • ci/test_wheel_cudf_polars.sh

📝 Walkthrough

Summary by CodeRabbit

Release Notes

No user-facing changes in this release. This update includes internal testing infrastructure improvements to enhance reliability.

  • Chores
    • Adjusted test execution timing for improved stability

Walkthrough

This pull request increases the polars test execution timeout in the CI pipeline from 15 minutes to 25 minutes. The change modifies the timeout parameter passed to the polars pytest runner to prevent test timeouts during longer-running test suites.

Changes

CI Configuration Update

Layer / File(s) Summary
Polars test timeout parameter
ci/test_wheel_cudf_polars.sh
The timeout for polars test execution is increased from 15m to 25m in the call to ./ci/run_cudf_polars_pytests.sh to allow test runs more time to complete.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and clearly summarizes the main change: bumping the timeout for the cudf-polars pytests script execution.
Description check ✅ Passed The description is directly related to the changeset, providing context about the timeout issue and explaining why the bump from 15 to 25 minutes is necessary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@jameslamb
jameslamb removed the request for review from gforsyth May 14, 2026 16:04
@TomAugspurger

Copy link
Copy Markdown
Contributor Author

/merge

@TomAugspurger
TomAugspurger changed the base branch from main to branch-25.06 May 15, 2026 16:06
@TomAugspurger
TomAugspurger requested review from a team as code owners May 15, 2026 16:06
@TomAugspurger
TomAugspurger requested review from Matt711, bdice, gforsyth and karthikeyann and removed request for a team, Matt711, bdice, gforsyth and karthikeyann May 15, 2026 16:06
@TomAugspurger
TomAugspurger changed the base branch from branch-25.06 to main May 15, 2026 16:07
@rapids-bot

rapids-bot Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

This PR's base branch has been changed since the last /merge command. Please issue the command again to confirm the merge with the new base branch.

1 similar comment
@rapids-bot

rapids-bot Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

This PR's base branch has been changed since the last /merge command. Please issue the command again to confirm the merge with the new base branch.

@TomAugspurger

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit 9b1f470 into NVIDIA:main May 18, 2026
91 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants