Extend SLURM time out

[jan-janssen]

Summary by CodeRabbit

• Chores
  • Continuous integration workflow updated: increased the timeout for the SLURM setup step in the unittest job from 5 to 10 minutes. This helps accommodate slower runners and stabilizes automated test execution in certain environments.
  • No changes to application behavior or public APIs. End users will not see any interface or feature differences.

[coderabbitai]

Walkthrough

The GitHub Actions workflow file was updated to increase the timeout for the koesterlab/setup-slurm-action@v1 step in the unittest_slurm_mpich job from 5 to 10 minutes. No other changes were made.

Changes

Cohort / File(s)	Summary
CI workflow timeout adjustment .github/workflows/pipeline.yml	Increased timeout-minutes for the koesterlab/setup-slurm-action@v1 step in unittest_slurm_mpich job from 5 to 10.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A hop, a pause, then longer still,
Our runners stretch atop the hill.
Ten-minute carrots on the vine,
The CI burrow’s doing fine.
Thump-thump—timeouts realigned,
More time to chomp, then cross the line! 🥕

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

✨ Finishing Touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch slurm_time_out_extension

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

.github/workflows/pipeline.yml (1)

278-279: Fix minor typo and prefer apt-get for non-interactive CI installs.

Rename the step and use apt-get with update to avoid intermittent failures.

-    - name: ubnuntu install
-      run: sudo apt install -y mpich
+    - name: Ubuntu install MPICH
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y --no-install-recommends mpich

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e05297e and 07ecbb8.

📒 Files selected for processing (1)

• .github/workflows/pipeline.yml (1 hunks)

🔇 Additional comments (1)

.github/workflows/pipeline.yml (1)

277-277: SLURM setup timeout increase to 10 minutes looks good.

This should reduce spurious failures from slow SLURM bring-up on shared runners.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.76%. Comparing base (e05297e) to head (07ecbb8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #809   +/-   ##
=======================================
  Coverage   97.76%   97.76%           
=======================================
  Files          32       32           
  Lines        1479     1479           
=======================================
  Hits         1446     1446           
  Misses         33       33           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.