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

small tweak to get intel MPI working on RHEL 7 #743

Merged
merged 3 commits into from Aug 4, 2016

Conversation

Projects
None yet
3 participants
@garlick
Copy link
Member

garlick commented Aug 4, 2016

Drop SLURM environment variables from intel_mpi.lua plugin.
Set I_MPI_PMI_LIBRARY to path to libpmi.so, only if already set.
Correct path to PMI library for in-tree testing.

garlick added some commits Aug 4, 2016

build: update PMI library path
Update the settings for
  INTREE_PROGRAM_LIBRARY_PATH
  INTREE_PMI_LIBRARY_PATH
to reflect move of libpmi in the source tree.
lua.d/intel_mpi.lua: revert SLURM_ env settings
Now that wrexecd provides PMI-1 wire protocol, these SLURM_
environment variables (discussed in #630) are no longer required.
lua.d/intel_mpi.lua: override I_MPI_PMI_LIBRARY
Set I_MPI_PMI_LIBRARY to point the the Flux PMI library only
if it is set.  It works either way:  if set, the Flux PMI library
PMI-1 wire protocol client speaks with PMI-1 process manager service
in wrexecd.  If unset, Intel MPI's built-in PMI wire protocol client
speaks directly with wrexecd.

Fixes #630

@garlick garlick added the review label Aug 4, 2016

@coveralls

This comment has been minimized.

Copy link

coveralls commented Aug 4, 2016

Coverage Status

Coverage increased (+0.05%) to 75.247% when pulling f7868c4 on garlick:intel_mpi into ea77756 on flux-framework:master.

@grondo

This comment has been minimized.

Copy link
Contributor

grondo commented Aug 4, 2016

Thanks! I'll press the button once Travis completes.

@grondo grondo merged commit 2556295 into flux-framework:master Aug 4, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.05%) to 75.247%
Details

@grondo grondo removed the review label Aug 4, 2016

@garlick garlick referenced this pull request Aug 10, 2016

Closed

0.4.0 release notes #739

@garlick garlick deleted the garlick:intel_mpi branch Aug 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.