Skip to content

add patch to fix support for external PMIx v3.1 in OpenMPI v3.1.3#15566

Merged
boegel merged 1 commit intoeasybuilders:developfrom
wdlingit:ompi313_fix_external_pmix3.1
May 31, 2022
Merged

add patch to fix support for external PMIx v3.1 in OpenMPI v3.1.3#15566
boegel merged 1 commit intoeasybuilders:developfrom
wdlingit:ompi313_fix_external_pmix3.1

Conversation

@wdlingit
Copy link
Copy Markdown
Contributor

@wdlingit wdlingit commented May 31, 2022 •
edited
Loading

Got problem when compiling OpenMPI v3.1.3 (2019a) with external PMIx v3.1.x. The patch is from one commit of openmpi poject: https: open-mpi/ompi#6253

@boegel boegel added the bug fix label May 31, 2022
@boegel
Copy link
Copy Markdown
Member

boegel commented May 31, 2022

@boegelbot please test @ generoso

@boegel boegel added this to the next release (4.5.5?) milestone May 31, 2022
@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented May 31, 2022

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=15566 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_15566 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8588

Test results coming soon (I hope)...

Details

- notification for comment with ID 1141780682 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented May 31, 2022

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/e9a644a42ad6d07ce8fa2fc7e7c3382c for a full test report.

@jfgrimm
Copy link
Copy Markdown
Member

jfgrimm commented May 31, 2022

Test report by @jfgrimm
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
himem06.pri.viking.alces.network - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Platinum 8160 CPU @ 2.10GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/d429ec39530c91cbcd6feca0278f96f5 for a full test report.

boegel
boegel approved these changes May 31, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented May 31, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3539.doduo.os - Linux RHEL 8.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/b582ce680f5c5be4679b348514782e2a for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 31, 2022

Going in, thanks @wdlingit!

@boegel boegel merged commit 38af341 into easybuilders:develop May 31, 2022
@boegel boegel changed the title {mpi}[GCC/8.2.0] OpenMPI v3.1.3 fix support for external PMIx v3.1 add patch to fix support for external PMIx v3.1 in OpenMPI v3.1.3 Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.5

Development

Successfully merging this pull request may close these issues.

4 participants

@wdlingit @boegel @boegelbot @jfgrimm