Skip to content

Commit

Permalink
Merge pull request #184 from dalcinl/update-ucx-ucc
Browse files Browse the repository at this point in the history
Make UCX/UCC required dependencies
  • Loading branch information
dalcinl authored Nov 13, 2024
2 parents 3d726bd + 575dead commit f12332c
Show file tree
Hide file tree
Showing 18 changed files with 96 additions and 116 deletions.
2 changes: 1 addition & 1 deletion .azure-pipelines/azure-pipelines-osx.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ c_stdlib:
c_stdlib_version:
- '2.17'
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -34,14 +34,15 @@ mpi_type:
- conda
target_platform:
- linux-64
ucc:
- '1'
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ c_stdlib:
c_stdlib_version:
- '2.17'
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -34,14 +34,15 @@ mpi_type:
- external
target_platform:
- linux-64
ucc:
- '1'
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ c_stdlib_version:
cdt_arch:
- aarch64
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -38,14 +38,15 @@ mpi_type:
- conda
target_platform:
- linux-aarch64
ucc:
- '1'
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ c_stdlib_version:
cdt_arch:
- aarch64
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -38,14 +38,15 @@ mpi_type:
- external
target_platform:
- linux-aarch64
ucc:
- '1'
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ c_stdlib:
c_stdlib_version:
- '2.17'
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -34,14 +34,11 @@ mpi_type:
- conda
target_platform:
- linux-ppc64le
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ c_stdlib:
c_stdlib_version:
- '2.17'
cdt_name:
- cos7
- conda
channel_sources:
- conda-forge
channel_targets:
Expand All @@ -34,14 +34,11 @@ mpi_type:
- external
target_platform:
- linux-ppc64le
ucx:
- '1.17'
zip_keys:
- - c_compiler_version
- cxx_compiler_version
- fortran_compiler_version
- c_stdlib_version
- cdt_name
- cuda_compiler
- cuda_compiler_version
- docker_image
Expand Down
8 changes: 0 additions & 8 deletions .ci_support/migrations/ucx117.yaml

This file was deleted.

4 changes: 2 additions & 2 deletions .ci_support/osx_64_.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ MACOSX_SDK_VERSION:
c_compiler:
- clang
c_compiler_version:
- '17'
- '18'
c_stdlib:
- macosx_deployment_target
c_stdlib_version:
Expand All @@ -21,7 +21,7 @@ cuda_compiler_version:
cxx_compiler:
- clangxx
cxx_compiler_version:
- '17'
- '18'
fortran_compiler:
- gfortran
fortran_compiler_version:
Expand Down
4 changes: 2 additions & 2 deletions .ci_support/osx_arm64_.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ MACOSX_SDK_VERSION:
c_compiler:
- clang
c_compiler_version:
- '17'
- '18'
c_stdlib:
- macosx_deployment_target
c_stdlib_version:
Expand All @@ -21,7 +21,7 @@ cuda_compiler_version:
cxx_compiler:
- clangxx
cxx_compiler_version:
- '17'
- '18'
fortran_compiler:
- gfortran
fortran_compiler_version:
Expand Down
14 changes: 7 additions & 7 deletions .scripts/build_steps.sh

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

45 changes: 28 additions & 17 deletions .scripts/run_osx_build.sh

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

31 changes: 28 additions & 3 deletions azure-pipelines.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 1 addition & 15 deletions recipe/build-mpi.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#!/bin/bash

set -ex

# avoid absolute-paths in compilers
Expand Down Expand Up @@ -83,29 +82,16 @@ fi
--with-hwloc=$PREFIX \
--with-libevent=$PREFIX \
--with-zlib=$PREFIX \
--enable-mca-dso \
--enable-ipv6 \
$build_with_ucx \
$build_with_ucc \
$build_with_cuda \
|| (cat config.log; false)
|| (cat config.log; exit 1)

make -j"${CPU_COUNT:-1}"
make install

POST_LINK=$PREFIX/bin/.openmpi-post-link.sh
if [ -n "$build_with_ucx" ]; then
echo "setting MCA pml to ^ucx..."
echo "pml = ^ucx" >> $PREFIX/etc/openmpi-mca-params.conf
echo "setting MCA osc to ^ucx..."
echo "osc = ^ucx" >> $PREFIX/etc/openmpi-mca-params.conf
cat $RECIPE_DIR/post-link-ucx.sh >> $POST_LINK
fi
if [ -n "$build_with_ucc" ]; then
echo "setting MCA coll_ucc_enable to 0..."
echo "coll_ucc_enable = 0" >> $PREFIX/etc/openmpi-mca-params.conf
cat $RECIPE_DIR/post-link-ucc.sh >> $POST_LINK
fi
if [ -n "$build_with_cuda" ]; then
echo "setting MCA mca_base_component_show_load_errors to 0..."
echo "mca_base_component_show_load_errors = 0" >> $PREFIX/etc/openmpi-mca-params.conf
Expand Down
Loading

0 comments on commit f12332c

Please sign in to comment.