Skip to content

Conversation

@bartoldeman
Copy link
Contributor

(created using eb --new-pr)

@bartoldeman
Copy link
Contributor Author

bartoldeman commented May 21, 2022

UCC is newly supported by Open MPI 4.1.4

Note that UCC like UCX can optionally depend on CUDA (and NCCL for UCC only), and uses dynamically loaded components for that.

But fortunately unlike for UCX no special patch is needed, since UCC_COMPONENT_PATH is sufficient and glob(3) does the rest.

@bartoldeman
Copy link
Contributor Author

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

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

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

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

Test results coming soon (I hope)...

- notification for comment with ID 1133778482 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
Collaborator

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/afdc09a7a779a3710dd92443df60c9cd for a full test report.

@bartoldeman bartoldeman added this to the next release (4.5.5?) milestone May 21, 2022
@Micket
Copy link
Contributor

Micket commented May 22, 2022

Note that UCC like UCX can optionally depend on CUDA (and NCCL for UCC only), and uses dynamically loaded components for that.

so we'll make a UCC-CUDA much like UCX-CUDA later (once we decide on CUDA version)?

@Micket
Copy link
Contributor

Micket commented May 23, 2022

Test report by @Micket
SUCCESS
Build succeeded for 8 out of 8 (1 easyconfigs in total)
alvis-c1 - Linux Rocky Linux 8.5, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/4cbd0e63415253f25ee9ee08f0142ab4 for a full test report.

@jfgrimm
Copy link
Member

jfgrimm commented May 23, 2022

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

@jfgrimm
Copy link
Member

jfgrimm commented May 23, 2022

@boegelbot please test @ jsc-zen2

@jfgrimm jfgrimm mentioned this pull request May 23, 2022
2 tasks
@boegelbot
Copy link
Collaborator

@jfgrimm: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=15527 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_15527 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1134634495 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
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen2c1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/8c5141e812e53be76ab88f5350cdba38 for a full test report.

Copy link
Member

@jfgrimm jfgrimm left a comment

Choose a reason for hiding this comment

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

LGTM

@jfgrimm
Copy link
Member

jfgrimm commented May 23, 2022

Going in, thanks @bartoldeman!

@jfgrimm jfgrimm merged commit fa89ee3 into easybuilders:develop May 23, 2022
@bartoldeman
Copy link
Contributor Author

@Micket

so we'll make a UCC-CUDA much like UCX-CUDA later (once we decide on CUDA version)?

yes exactly!

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