flux-framework / flux-pmix

flux shell plugin to bootstrap openmpi v5+

GNU Lesser General Public License v3.0

2 stars 4 forks source link

add PMI client plugin to enable flux to be launched by a pmix server #77

Closed garlick closed 1 year ago

garlick commented 1 year ago

flux-framework/flux-core#4865 enables broker bootstrap methods to be provided by dso plugins and drops pmix client support from the broker. This PR re-adds support by providing a dso.

This seems better since it localizes the openpmix dependency here.

codecov[bot] commented 1 year ago

Codecov Report

Merging #77 (6160e4a) into main (36ca4eb) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   79.44%   79.44%           
=======================================
  Files          10       10           
  Lines        1231     1231           
=======================================
  Hits          978      978           
  Misses        253      253

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

garlick commented 1 year ago

Just pushed a small documentation tweak to improve readability slightly.

Note that this PR can be merged before or after flux-framework/flux-core#4865 since it adds a new capability and its test has a prereq on the corresponding capability in flux-core.

garlick commented 1 year ago

Thanks! I'll set MWP.