Skip to content

Update flux-core version to 0.87.0 and flux-pmix version to 0.7.1 - #1042

Merged
jan-janssen merged 1 commit into
mainfrom
flux
Aug 7, 2026
Merged

Update flux-core version to 0.87.0 and flux-pmix version to 0.7.1#1042
jan-janssen merged 1 commit into
mainfrom
flux

Conversation

@jan-janssen

@jan-janssen jan-janssen commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores

    • Updated the project’s runtime and development environments to newer supporting software versions.
    • Refreshed continuous integration configurations to keep automated validation aligned across supported MPI environments.
  • Bug Fixes

    • Improved compatibility and reliability of environment setup and automated testing workflows.
    • No changes to public interfaces or end-user functionality.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The pull request updates flux-core and flux-pmix pins in Binder, integration, and unittest environment configurations.

Changes

Flux dependency updates

Layer / File(s) Summary
Update Flux dependency pins
.ci_support/environment-integration.yml, binder/environment.yml, .github/workflows/pipeline.yml
The configurations update flux-core from 0.81.0 to 0.87.0 and flux-pmix from 0.6.0 to 0.7.1 where used.

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

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the dependency version updates made across the pull request.
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
  • Commit unit tests in branch flux

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/pipeline.yml:
- Line 248: Update the flux-pmix dependency in
.github/workflows/pipeline.yml:248-248,
.ci_support/environment-integration.yml:13-13, and binder/environment.yml:9-9 to
use the same verified conda-forge package version that is available before
merging; replace the unavailable 0.7.1 pin consistently at all three sites.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 421d7e4a-ad65-488d-8f08-c61e56afe550

📥 Commits

Reviewing files that changed from the base of the PR and between 3c6b530 and b41b7e7.

📒 Files selected for processing (3)
  • .ci_support/environment-integration.yml
  • .github/workflows/pipeline.yml
  • binder/environment.yml

timeout-minutes: 5
run: |
echo -e '- coverage\n- flux-core =0.81.0\n- flux-pmix=0.6.0' >> .ci_support/environment-openmpi.yml
echo -e '- coverage\n- flux-core =0.87.0\n- flux-pmix=0.7.1' >> .ci_support/environment-openmpi.yml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

for spec in "flux-core 0.87.0" "flux-pmix 0.7.1"; do
  read -r package version <<< "$spec"
  curl -fsSL "https://api.anaconda.org/package/conda-forge/${package}" |
    jq -e --arg version "$version" '
      [.files[]?
       | select(
           .version == $version
           and .platform == "linux-64"
           and (((.labels // []) | index("main")) != null)
         )]
      | length > 0
    ' >/dev/null
done

rg -n 'flux-(core|pmix)[ =]+0\.(87\.0|7\.1)' \
  .ci_support/environment-integration.yml \
  binder/environment.yml \
  .github/workflows/pipeline.yml

Repository: pyiron/executorlib

Length of output: 156


🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

printf '%s\n' '--- flux-core package info ---'
curl -fsSL "https://api.anaconda.org/package/conda-forge/flux-core" | jq '{name, latest_version, files_count, main_files: [.files[]? | select(.version == "0.87.0" and .platform == "linux-64" and (((.labels // []) | index("main")) != null))]}'

printf '%s\n' '--- flux-pmix package info ---'
curl -fsSL "https://api.anaconda.org/package/conda-forge/flux-pmix" | jq '{name, latest_version, files_count, main_files: [.files[]? | select(.version == "0.7.1" and .platform == "linux-64" and (((.labels // []) | index("main")) != null))]}'

printf '%s\n' '--- exact version matches for flux-core 0.87.0 on linux-64 with main label ---'
curl -fsSL "https://api.anaconda.org/package/conda-forge/flux-core" | jq '[.files[]? | select(.version == "0.87.0" and .platform == "linux-64" and (((.labels // []) | index("main")) != null)) | {version, platform, labels, package_filename}] | length'

printf '%s\n' '--- exact version matches for flux-pmix 0.7.1 on linux-64 with main label ---'
curl -fsSL "https://api.anaconda.org/package/conda-forge/flux-pmix" | jq '[.files[]? | select(.version == "0.7.1" and .platform == "linux-64" and (((.labels // []) | index("main")) != null)) | {version, platform, labels, package_filename}] | length'

printf '%s\n' '--- repository occurrences ---'
rg -n 'flux-(core|pmix)[ =]+"?0\.(87\.0|7\.1)"?' \
  .ci_support/environment-integration.yml \
  binder/environment.yml \
  .github/workflows/pipeline.yml || true

Repository: pyiron/executorlib

Length of output: 1085


Wait for the flux-pmix=0.7.1 conda-forge package before merging.

The pinned version is not available on the default conda-forge channel, so CI and Binder environment creation can fail. Use the same verified package availability for .github/workflows/pipeline.yml, .ci_support/environment-integration.yml, and binder/environment.yml.

📍 Affects 3 files
  • .github/workflows/pipeline.yml#L248-L248 (this comment)
  • .ci_support/environment-integration.yml#L13-L13
  • binder/environment.yml#L9-L9
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/pipeline.yml at line 248, Update the flux-pmix dependency
in .github/workflows/pipeline.yml:248-248,
.ci_support/environment-integration.yml:13-13, and binder/environment.yml:9-9 to
use the same verified conda-forge package version that is available before
merging; replace the unavailable 0.7.1 pin consistently at all three sites.

Source: MCP tools

@codecov

codecov Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.22%. Comparing base (faf71ea) to head (b41b7e7).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1042   +/-   ##
=======================================
  Coverage   94.22%   94.22%           
=======================================
  Files          39       39           
  Lines        2113     2113           
=======================================
  Hits         1991     1991           
  Misses        122      122           

☔ View full report in Codecov by Harness.
📢 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 c60bb07 into main Aug 7, 2026
109 of 131 checks passed
@jan-janssen
jan-janssen deleted the flux branch August 7, 2026 03: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.

1 participant