Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

incremeting pmix to latest release (4.2.4) and enabled pmix in openmpi #1848

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

mghpcsim
Copy link
Collaborator

This is a simplified PR for re-enabling PMIX for OpenMPI as well as updating pmix to the latest release.

Signed-off-by: Christopher S. Simmons <csim@mghpcc.org>
@github-actions
Copy link

Test Results

  4 files  ±0    4 suites  ±0   0s ⏱️ ±0s
17 tests ±0  16 ✔️  - 1  0 💤 ±0  1 +1 
18 runs  ±0  17 ✔️  - 1  0 💤 ±0  1 +1 

For more details on these failures, see this check.

Results for commit dfe607d. ± Comparison against base commit 6a9f7ea.

@adrianreber
Copy link
Member

The slurm tests are failing with:

# srun: error: Couldn't find the specified plugin name for mpi/pmix looking at all files
# srun: error: cannot find mpi plugin for mpi/pmix
# srun: error: MPI: Cannot create context for mpi/pmix
# srun: error: MPI: Unable to load any plugin
# srun: error: Invalid MPI type 'pmix', --mpi=list for acceptable types

Does this change also require a rebuild of slurm? Do we need any changes in slurm or just a rebuild?

@mghpcsim
Copy link
Collaborator Author

Yeah this has a circular dependency (as we discussed in Slack) that won't work on Github actions. On my local test system, I have to load the pmix module and then rebuild SLURM for it to pick it up.

I think we will need to merge and send this to OBS to have SLURM build updated before OpenMPI with pmix will pass any tests. This goes for this PR and the duplicate one for the 2.x branch

@adrianreber adrianreber merged commit 1783619 into openhpc:3.x Jul 24, 2023
13 of 19 checks passed
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.

2 participants