Skip to content

Bump cudf-polars test suite timeouts - #23399

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:release/26.08from
TomAugspurger:tom/cudf-polars-timeout-adjustments-3
Jul 22, 2026
Merged

Bump cudf-polars test suite timeouts#23399
rapids-bot[bot] merged 1 commit into
NVIDIA:release/26.08from
TomAugspurger:tom/cudf-polars-timeout-adjustments-3

Conversation

@TomAugspurger

Copy link
Copy Markdown
Contributor

Description

We've observed some cudf-polars runs taking >1 hour to complete, while runs on different, but similar, hardware take <15 minutes.

We've resolved most of the outright hangs now. Under the assumption that it's preferable to wait for a job to complete rather than restarting, we'll bump the timeout here.

We've observed some cudf-polars runs taking >1 hour to complete, while
runs on different, but similar, hardware take <15 minutes.

We've resolved most of the outright hangs now. Under the assumption
that it's preferable to wait for a job to complete rather than
restarting, we'll bump the timeout here.
@copy-pr-bot

copy-pr-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@TomAugspurger

Copy link
Copy Markdown
Contributor Author

/ok to test 90cff6a

@TomAugspurger TomAugspurger added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 22, 2026
@TomAugspurger
TomAugspurger marked this pull request as ready for review July 22, 2026 16:54
@TomAugspurger
TomAugspurger requested a review from a team as a code owner July 22, 2026 16:54
@TomAugspurger
TomAugspurger requested a review from msarahan July 22, 2026 16:54
@coderabbitai

coderabbitai Bot commented Jul 22, 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: fa33e325-5d95-4937-a00f-04314046f7e1

📥 Commits

Reviewing files that changed from the base of the PR and between 2fb0ed7 and 90cff6a.

📒 Files selected for processing (2)
  • ci/run_cudf_polars_polars_tests.sh
  • ci/run_cudf_polars_pytests.sh

📝 Walkthrough

Summary by CodeRabbit

  • Tests
    • Increased allowed execution time for GPU and Python test runs to improve reliability for longer-running test suites.

Walkthrough

Changes

Test timeout extension

Layer / File(s) Summary
Increase Python-enabled test timeouts
ci/run_cudf_polars_polars_tests.sh, ci/run_cudf_polars_pytests.sh
The Python-enabled timeout increases from 3600 to 4800 seconds for GPU engine and pytest test runs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • rapidsai/cudf#23332: Introduced the timeout_with_stack.py --enable-python 3600 wrappers updated by this PR.

Suggested labels: cudf-polars, Python

Suggested reviewers: wence-, bdice

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: increasing cudf-polars test timeouts.
Description check ✅ Passed The description matches the changeset and explains why the timeout increase is needed.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@vyasr

vyasr commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot
rapids-bot Bot merged commit f367be2 into NVIDIA:release/26.08 Jul 22, 2026
138 checks passed
@TomAugspurger
TomAugspurger deleted the tom/cudf-polars-timeout-adjustments-3 branch July 22, 2026 17:00
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