Skip to content

Conversation

@jan-janssen
Copy link
Member

@jan-janssen jan-janssen commented Oct 4, 2025

Summary by CodeRabbit

  • Chores
    • Updated CI environments to use Flux core 0.78.0 and aligned PMIx (0.6.0) for OpenMPI setups, improving consistency of MPI-based test runs.
    • Refreshed the interactive environment to Flux core 0.78.0, ensuring compatibility with the latest Flux stack.
    • These dependency updates enhance stability, reproducibility, and alignment across testing and example environments without changing user-facing functionality.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 4, 2025

Walkthrough

Version pins for Flux components were updated: the CI workflow references were changed to use flux-core 0.78.0 (with flux-pmix remaining 0.6.0), and Binder’s environment was updated to flux-core 0.78.0. No other logic or control-flow changes were made.

Changes

Cohort / File(s) Summary
CI workflow environment pins
.github/workflows/pipeline.yml
Updated CI steps to use environment specs with flux-core pinned to 0.78.0. For MPICH: flux-core 0.65.0 → 0.78.0. For OpenMPI: flux-core 0.65.0 → 0.78.0; flux-pmix remains 0.6.0.
Binder environment
binder/environment.yml
Bumped flux-core 0.65.0 → 0.78.0. No other dependency changes.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

I nibbled the pins to shiny new might,
From 0.65 to 0.78—just right!
In Binder and CI, the carrots align,
Flux hums along, dependencies fine.
A hop, a bump, a whisker’s delight—
Ship it swift through the moonlit night! 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and specifically describes the main change in the pull request, which is updating flux-core to version 0.78.0 across CI workflows and environment files. It is concise, accurately reflects the primary change, and would make sense to any teammate reviewing the project history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch core

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7809b34 and f873ba9.

📒 Files selected for processing (2)
  • .github/workflows/pipeline.yml (2 hunks)
  • binder/environment.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (19)
  • GitHub Check: unittest_openmpi (ubuntu-latest, 3.13)
  • GitHub Check: unittest_mpich (ubuntu-22.04-arm, 3.13)
  • GitHub Check: unittest_slurm_mpich
  • GitHub Check: unittest_mpich (ubuntu-latest, 3.11)
  • GitHub Check: unittest_mpich (ubuntu-latest, 3.13)
  • GitHub Check: unittest_mpich (macos-latest, 3.13)
  • GitHub Check: unittest_mpich (ubuntu-24.04-arm, 3.13)
  • GitHub Check: unittest_mpich (ubuntu-latest, 3.12)
  • GitHub Check: unittest_openmpi (ubuntu-latest, 3.12)
  • GitHub Check: notebooks_integration
  • GitHub Check: unittest_openmpi (macos-latest, 3.13)
  • GitHub Check: unittest_openmpi (ubuntu-22.04-arm, 3.13)
  • GitHub Check: unittest_openmpi (ubuntu-latest, 3.11)
  • GitHub Check: unittest_openmpi (ubuntu-24.04-arm, 3.13)
  • GitHub Check: unittest_win
  • GitHub Check: unittest_old
  • GitHub Check: unittest_flux_mpich
  • GitHub Check: benchmark (ubuntu-latest, 3.13, .ci_support/environment-openmpi.yml)
  • GitHub Check: benchmark (ubuntu-latest, 3.13, .ci_support/environment-mpich.yml)

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

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

@codecov
Copy link

codecov bot commented Oct 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.22%. Comparing base (7809b34) to head (f873ba9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #837      +/-   ##
==========================================
+ Coverage   98.16%   98.22%   +0.05%     
==========================================
  Files          34       34              
  Lines        1688     1688              
==========================================
+ Hits         1657     1658       +1     
+ Misses         31       30       -1     

☔ 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.

@jan-janssen jan-janssen merged commit dcdb29a into main Oct 4, 2025
86 of 91 checks passed
@jan-janssen jan-janssen deleted the core branch October 4, 2025 19:20
@coderabbitai coderabbitai bot mentioned this pull request Oct 8, 2025
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.

1 participant