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

add patch to fix PMIx detection in OpenMPI v4.0.3, v4.0.5, v4.1.0 #14177

Merged

Conversation

akesandgren
Copy link
Contributor

@akesandgren akesandgren commented Oct 20, 2021

(created using eb --new-pr)

cfr. open-mpi/ompi#8823

@akesandgren
Copy link
Contributor Author

@boegelbot Please test @ generoso

@boegelbot
Copy link
Collaborator

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

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

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

Test results coming soon (I hope)...

- notification for comment with ID 947542172 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).

@akesandgren
Copy link
Contributor Author

akesandgren commented Oct 20, 2021

Test report by @akesandgren
FAILED
Build succeeded for 8 out of 10 (10 easyconfigs in total)
b-an02.hpc2n.umu.se - Linux Ubuntu 20.04, x86_64, Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz, Python 3.8.10
See https://gist.github.com/84ea74bd02919fdf7489ed9995c835d5 for a full test report.

Lacked some dependencies hence some failed builds.

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 10 out of 10 (10 easyconfigs in total)
cnx1 - Linux rocky linux 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/befc61f4adbf13107300fc521e0f63d9 for a full test report.

@Micket
Copy link
Contributor

Micket commented Oct 20, 2021

@akesandgren forgot --robot?

@akesandgren
Copy link
Contributor Author

No, intentionally left it out since I didn't want to build the missing toolchains here. I left those for boegelbot...

@boegel boegel changed the title Fix PMIx detection as per OpenMPI issue 8823 add patch to fix PMIx detection in OpenMPI v4.0.3, v4.0.5, v4.1.0 Oct 21, 2021
Copy link
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
Member

boegel commented Oct 21, 2021

Test report by @boegel
SUCCESS
Build succeeded for 10 out of 10 (10 easyconfigs in total)
node2692.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/99f8496c0136bd510c65c31f5bf2e64f for a full test report.

@boegel
Copy link
Member

boegel commented Oct 21, 2021

Going in, thanks @akesandgren!

@boegel boegel merged commit f74f370 into easybuilders:develop Oct 21, 2021
@boegel
Copy link
Member

boegel commented Oct 21, 2021

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

@akesandgren akesandgren deleted the 20211020123721_new_pr_OpenMPI403 branch October 21, 2021 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants