Skip to content

ci[cartesian]: increase time limit for cartesian / dace tests on gh200#2335

Merged
romanc merged 1 commit into
GridTools:mainfrom
romanc:romanc/cartesian-dace-timelimit
Oct 24, 2025
Merged

ci[cartesian]: increase time limit for cartesian / dace tests on gh200#2335
romanc merged 1 commit into
GridTools:mainfrom
romanc:romanc/cartesian-dace-timelimit

Conversation

@romanc
Copy link
Copy Markdown
Contributor

@romanc romanc commented Oct 23, 2025

Description

Cartesian dace tests are regularly timing out with the default time limit on the gh200 box. It looks like recent increased development (including better test coverage) in gt4py.cartesian are pushing the limit of the standard 5min slurm time limit. We suggest to increase the time limit to 10 minutes.

An example of running into the time limit can be found here. It's a run associated with PR #2314. Multiple re-runs have shown the same behavior.

Requirements

  • All fixes and/or new features come with corresponding tests.
    N/A
  • Important design decisions have been documented in the appropriate ADR inside the docs/development/ADRs/ folder.
    N/A

/cc @FlorianDeconinck @twicki as discussed

@romanc romanc requested a review from egparedes October 23, 2025 13:02
Cartesian dace tests are regularly timing out with the default timelimit
on the gh200 box. It looks like recent increased development (including
better test coverage) in gt4py.cartesian are pushing the limit of the
standard 5min slurm time limit. We suggest to increase the timelimit to
10 minutes.
@romanc romanc force-pushed the romanc/cartesian-dace-timelimit branch from 85daa1e to f650bc1 Compare October 23, 2025 13:03
@romanc
Copy link
Copy Markdown
Contributor Author

romanc commented Oct 23, 2025

cscs-ci run

@romanc
Copy link
Copy Markdown
Contributor Author

romanc commented Oct 23, 2025

Sidenote: PR #2337 will remove the cuda backend, which should free some time on the cscs machines that we'd like to use for the dace:gpu tests instead.

@romanc
Copy link
Copy Markdown
Contributor Author

romanc commented Oct 23, 2025

cscs-ci run

Copy link
Copy Markdown
Contributor

@egparedes egparedes left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@egparedes egparedes left a comment

Choose a reason for hiding this comment

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

LGTM

@romanc romanc merged commit 0eeeda0 into GridTools:main Oct 24, 2025
31 checks passed
@romanc romanc deleted the romanc/cartesian-dace-timelimit branch October 24, 2025 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants