-
Notifications
You must be signed in to change notification settings - Fork 27.4k
Unify nccl versions for x86 and aarch64 builds #149554
Copy link
Copy link
Closed
Labels
module: binariesAnything related to official binaries that we release to usersAnything related to official binaries that we release to usersmodule: ciRelated to continuous integrationRelated to continuous integrationtopic: binariestriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Description
🐛 Describe the bug
Please see:
#149540
After landing of: #149351
CUDA aarch64 builds where broken, since nccl was defined in .ci/docker/common/install_cuda_aarch64.sh as well as in the matrix.
Consider removing the installation of nccl in the docker.
Change to have only 1 source of truth for both builds.
Versions
2.8.0
cc @seemethere @malfet @osalpekar @pytorch/pytorch-dev-infra
Reactions are currently unavailable
Metadata
Metadata
Labels
module: binariesAnything related to official binaries that we release to usersAnything related to official binaries that we release to usersmodule: ciRelated to continuous integrationRelated to continuous integrationtopic: binariestriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Type
Projects
Status
Done