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

only add --with-ucc for OpenMPI 4.1.4+ #3223

Conversation

Flamefire
Copy link
Contributor

@Flamefire Flamefire commented Feb 12, 2024

(created using eb --new-pr)

--with-ucc support was added to 4.1.4 with open-mpi/ompi#10171 as far as I can tell

I checked by checking the existance of the file, e.g. https://github.com/open-mpi/ompi/blob/v4.1.4/config/ompi_check_ucc.m4

Fixes #3222

@branfosj branfosj added this to the release after 4.9.0 milestone Feb 12, 2024
@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles

Overview of tested easyconfigs (in order)

  • SUCCESS OpenMPI-4.1.1-GCC-10.3.0.eb
  • SUCCESS OpenMPI-4.1.2-GCC-11.2.0.eb
  • SUCCESS OpenMPI-4.1.4-GCC-11.3.0.eb
  • SUCCESS OpenMPI-4.1.5-GCC-12.2.0.eb
  • SUCCESS OpenMPI-4.1.6-GCC-13.2.0.eb

Build succeeded for 5 out of 5 (5 easyconfigs in total)
zen2-rockylinux-89 - Linux Rocky Linux 8.9, x86_64, AMD EPYC 7452 32-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/SebastianAchilles/f20320bd8f39714ae8c1c8b498f9d18f for a full test report.

@Flamefire
Copy link
Contributor Author

Flamefire commented Feb 12, 2024

Test report by @Flamefire

Overview of tested easyconfigs (in order)

Build succeeded for 5 out of 7 (7 easyconfigs in total)
n1577 - Linux RHEL 8.7 (Ootpa), x86_64, Intel(R) Xeon(R) Platinum 8470 (icelake), Python 3.8.13
See https://gist.github.com/Flamefire/aaa55c819b9b193d7b77bf6b0e944dc2 for a full test report.

Copy link
Member

@SebastianAchilles SebastianAchilles left a comment

Choose a reason for hiding this comment

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

lgtm

@SebastianAchilles
Copy link
Member

Going in, thanks @Flamefire!

@SebastianAchilles SebastianAchilles merged commit 3a164a5 into easybuilders:develop Feb 12, 2024
47 checks passed
@Flamefire Flamefire deleted the 20240212103005_new_pr_openmpi branch February 12, 2024 19:51
@boegel boegel changed the title OpenMPI: Add --with-ucc only for 4.1.4+ only add --with-ucc for OpenMPI 4.1.4+ Feb 14, 2024
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.

OpenMPI: Unknown UCC configure option
4 participants