Skip to content

Commit

Permalink
CI fixes in OpenBLAS builds (#2762)
Browse files Browse the repository at this point in the history
The riscv64 and x86_64 OpenBLAS builds have been failing. For the riscv64
build, the location of the qemu debian package changed. We update the
URL to obtain, as well as making sure that failing to obtain the right
QEMU will cause an error in CI.

For the x86_64 build, illegal instructions are being reported, which I
can't reproduce locally. My intuition is that there is something wrong
with the cached build, so we update the key for the build. The naming in
the key now matches that of the aarch64 and riscv64 builds.
  • Loading branch information
keeranroth authored May 2, 2024
1 parent e1f633d commit 38d1ebf
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions .ci/env/apt.sh
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,12 @@ function install_qemu_emulation_apt {
}

function install_qemu_emulation_deb {
qemu_deb=qemu-user-static_8.2.2+ds-2+b1_amd64.deb
wget http://ftp.de.debian.org/debian/pool/main/q/qemu/${qemu_deb}
qemu_deb=qemu-user-static_8.2.3+ds-2_amd64.deb
set -eo pipefail
wget http://ftp.debian.org/debian/pool/main/q/qemu/${qemu_deb}
sudo dpkg -i ${qemu_deb}
sudo systemctl restart systemd-binfmt.service
set +eo pipefail
}

function install_llvm_version {
Expand Down
2 changes: 1 addition & 1 deletion .ci/pipeline/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ jobs:
displayName: 'System info'
- task: Cache@2
inputs:
key: '"gnu" | "x86_64" | "openblas" | "$(OPENBLAS_VERSION)" | "$(VM_IMAGE)"'
key: '"gcc" | "x86_64" | "openblas" | "$(OPENBLAS_VERSION)" | "$(VM_IMAGE)"'
path: $(OPENBLAS_CACHE_DIR)
cacheHitVar: OPENBLAS_RESTORED
- script: |
Expand Down

0 comments on commit 38d1ebf

Please sign in to comment.