Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Commit

Permalink
ci: install docker 19.03 for arm64 to let build image go
Browse files Browse the repository at this point in the history
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: #861
Signed-off-by: Jianyong Wu <[email protected]>
  • Loading branch information
jongwu committed Nov 12, 2020
1 parent 5cfb8ec commit 41522ed
Show file tree
Hide file tree
Showing 2 changed files with 46 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .ci/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ pushd "${tests_repo_dir}"
.ci/run.sh
testcidir=$(dirname "$0")

# docker version may need to be upgraded to let make proto go on arm64
arch=$(go env GOARCH)
if [ "$arch" == "arm64" ]; then
"../agent/.ci/run_arm64.sh"
fi

echo "Starting docker service before making proto"
sudo systemctl start docker

Expand Down
40 changes: 40 additions & 0 deletions .ci/run_arm64.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
#!/bin/bash
#
# Copyright (c) 2020 Arm Ltd Corporation
#
# SPDX-License-Identifier: Apache-2.0

source "${tests_repo_dir}/.ci/lib.sh"

install_docker_ubuntu() {
pkg_name="docker-ce"
repo_url="https://download.docker.com/linux/ubuntu"
curl -L "${repo_url}/gpg" | sudo apt-key add -
sudo -E add-apt-repository "deb [arch=${arch}] ${repo_url} $(lsb_release -cs) stable"
sudo -E apt-get update
docker_version_full=$(apt-cache madison $pkg_name | grep "$docker_version" | awk '{print $3}' | head -1)
sudo -E apt-get -y install "${pkg_name}=${docker_version_full}"
}

# make proto will fail on arm64 when use docker 18.06 and using docker 19.03
# can avoid this failure. For now this change is only for ubuntu as we know little
# about the other cases.
main() {
ID=$(cat /etc/os-release | grep "^ID=" | cut -f 2 -d "=")
if [ "$ID" != "ubuntu" ]; then
echo "docker upgrade is only done for ubuntu"
exit 0
fi
current_docker_version=$(docker version | awk '/Engine/{getline; print $2 }')
current_docker_version=${current_docker_version%.*}
docker_version=$(get_version "externals.docker.architecture.aarch64.agent.version")
docker_version=${docker_version/v}
docker_version=${docker_version/-*}
if [[ `echo "$current_docker_version < $docker_version" | bc` -eq 1 ]]; then
command -v docker >/dev/null 2>&1 && "${testcidir}/../cmd/container-manager/manage_ctr_mgr.sh" docker remove
echo "reinstall docker $docker_version for arm64"
install_docker_ubuntu $docker_version
fi
}

main

0 comments on commit 41522ed

Please sign in to comment.