Skip to content

Commit

Permalink
[ci][release-automation] Fix linux wheel sanity check step (ray-proje…
Browse files Browse the repository at this point in the history
…ct#44277)

- Install `miniconda3` in `/usr/local/bin` instead of `/root` because we are now using non-root user.
- Fix path for miniconda3 to be `/usr/local/bin/....` instead of `/root/....` in the linux wheel check script
- Run `/etc/profile.d/conda.sh` straight from script so we don't have to call conda init again. This was done for macos wheel check script but not yet for Linux.

Signed-off-by: khluu <[email protected]>
  • Loading branch information
khluu authored Mar 25, 2024
1 parent 31a4571 commit 99cb040
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .buildkite/release-automation/forge_x86_64.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ ln -s /usr/bin/clang-12 /usr/bin/clang

# Install miniconda
curl -sfL https://repo.anaconda.com/miniconda/Miniconda3-py38_23.1.0-1-Linux-x86_64.sh > /tmp/miniconda.sh
bash /tmp/miniconda.sh -b -u -p /root/miniconda3
bash /tmp/miniconda.sh -b -u -p /usr/local/bin/miniconda3
rm /tmp/miniconda.sh
/root/miniconda3/bin/conda init bash
/usr/local/bin/miniconda3/bin/conda init bash

# Install Bazelisk
curl -L https://github.com/bazelbuild/bazelisk/releases/download/v1.19.0/bazelisk-linux-amd64 --output /usr/local/bin/bazelisk
Expand Down
3 changes: 2 additions & 1 deletion .buildkite/release-automation/verify-linux-wheels.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ if [[ -z "$RAY_HASH" ]]; then
exit 1
fi

export PATH="/root/miniconda3/bin:$PATH"
export PATH="/usr/local/bin/miniconda3/bin:$PATH"
source "/usr/local/bin/miniconda3/etc/profile.d/conda.sh"

conda create -n rayio python="${PYTHON_VERSION}" -y

Expand Down

0 comments on commit 99cb040

Please sign in to comment.