Skip to content

Commit

Permalink
[ci] Move pip dependencies to docker image
Browse files Browse the repository at this point in the history
Following on from apache#10246, this moves the `pip install`-at-runtime deps to the docker image install so they are baked in.
  • Loading branch information
driazati committed Mar 2, 2022
1 parent a5cb76a commit 989a92e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docker/install/ubuntu_install_core.sh
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ set -o pipefail
apt-get update && apt-get install -y --no-install-recommends \
git make google-mock libgtest-dev cmake wget unzip libtinfo-dev libz-dev \
libcurl4-openssl-dev libssl-dev libopenblas-dev g++ sudo \
apt-transport-https graphviz pkg-config curl
apt-transport-https graphviz pkg-config curl ninja-build parallel

if [[ -d /usr/src/googletest ]]; then
# Single package source (Ubuntu 18.04)
Expand Down
2 changes: 2 additions & 0 deletions docker/install/ubuntu_install_python_package.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,12 @@ pip3 install \
packaging \
Pillow \
pytest \
tlcpack-sphinx-addon==0.2.1 \
pytest-profiling \
pytest-xdist \
requests \
scipy \
synr==0.6.0 \
junitparser==2.4.2 \
six \
tornado
5 changes: 4 additions & 1 deletion tests/scripts/task_ci_setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,10 @@ set -o pipefail
#
echo "Additional setup in ${CI_IMAGE_NAME}"

python3 -m pip install --user tlcpack-sphinx-addon==0.2.1 synr==0.6.0
# TODO: This is redundant with the docker image and can be deleted once
# https://github.com/apache/tvm/pull/10257 is merged and the Docker images are
# updated
python3 -m pip install --user tlcpack-sphinx-addon==0.2.1

# Rebuild standalone_crt in build/ tree. This file is not currently archived by pack_lib() in
# Jenkinsfile. We expect config.cmake to be present from pack_lib().
Expand Down

0 comments on commit 989a92e

Please sign in to comment.