search

flux-framework / flux-pmix

flux shell plugin to bootstrap openmpi v5+
GNU Lesser General Public License v3.0
2 stars 4 forks source link

support flux run -o pmi=pmix #81

Closed garlick closed 1 year ago

garlick commented 1 year ago

Load the pmix plugin by default, but keep it disabled unless it is listed in -o pmi=LIST.

Tests will fail until flux-framework/flux-core#5069 and flux-framework/flux-core#5067 are merged.

Well, actually the tests won't pass until #80 goes in as well.

garlick commented 1 year ago

All the prereq prs went in so I just pushed a rebase on current master.

codecov[bot] commented 1 year ago

Codecov Report

Merging #81 (28a3d10) into main (d4f8f67) will increase coverage by 0.07%. The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   78.94%   79.01%   +0.07%     
==========================================
  Files          12       12              
  Lines        1363     1382      +19     
==========================================
+ Hits         1076     1092      +16     
- Misses        287      290       +3
Impacted Files Coverage Δ
src/shell/plugins/main.c 80.72% <85.71%> (+0.38%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more