dist,copy-drivers: exit 1 if download driver fail

For easy to debug "make dist-rpm" error or koji task error, exit 1 if
download driver fail. BTW, try to download driver 3 times.

Signed-off-by: Jianping Liu <frankjpliu@tencent.com>
Reviewed-by: Yongliang Gao <leonylgao@tencent.com>
This commit is contained in:
Jianping Liu 2025-02-25 20:04:01 +08:00
parent 9f56ac480a
commit 8f66b613aa
2 changed files with 15 additions and 7 deletions

View File

@ -17,7 +17,7 @@ check_url_reachable()
mlnx_tgz_url=$(../../drivers/thirdparty/release-drivers/mlnx/get_mlnx_info.sh backup_url) mlnx_tgz_url=$(../../drivers/thirdparty/release-drivers/mlnx/get_mlnx_info.sh backup_url)
curl -I $mlnx_tgz_url 1>/dev/null 2>&1 curl -I $mlnx_tgz_url 1>/dev/null 2>&1
if (( $? != 0 )); then if (( $? != 0 )); then
echo "Could not download ${mlnx_tgz_name} !" echo "Could not download ${mlnx_tgz_name} ! URL can't reachable!"
exit 1 exit 1
fi fi
} }
@ -25,13 +25,13 @@ check_url_reachable()
thirdparty_mlnx(){ thirdparty_mlnx(){
get_mlnx_tgz_ok=1 get_mlnx_tgz_ok=1
# Real MLNX_OFED_LINUX-*.tgz will more than 1024 bytes. sha256_tmp=$(sha256sum ${mlnx_tgz_name} | awk '{printf $1}')
# Dummy MLNX_OFED_LINUX-*.tgz will less than 1024 bytes. if [[ $sha256_tmp != $mlnx_tgz_sha256 ]]; then
if [ $(stat -c%s ${mlnx_tgz_name}) -lt 1024 ]; then
rm -f ${mlnx_tgz_name} rm -f ${mlnx_tgz_name}
timeout 900 wget -q $mlnx_tgz_url || get_mlnx_tgz_ok=0 timeout 900 wget -q $mlnx_tgz_url || get_mlnx_tgz_ok=0
fi fi
## Check the mlnx_tgz which just downloaded.
sha256_tmp=$(sha256sum ${mlnx_tgz_name} | awk '{printf $1}') sha256_tmp=$(sha256sum ${mlnx_tgz_name} | awk '{printf $1}')
if [[ $sha256_tmp != $mlnx_tgz_sha256 ]]; then get_mlnx_tgz_ok=0; fi if [[ $sha256_tmp != $mlnx_tgz_sha256 ]]; then get_mlnx_tgz_ok=0; fi
@ -44,8 +44,14 @@ thirdparty_mlnx(){
## ##
## main , script start run at here. ## main , script start run at here.
## ##
check_url_reachable max_attempts=3
thirdparty_mlnx for attempt in {1..$max_attempts}; do
check_url_reachable
thirdparty_mlnx
echo "Having downloaded thirdparty drivers." if (( $? == 0 )); then
echo "Having downloaded thirdparty drivers."
break
fi
done

View File

@ -6,12 +6,14 @@ thirdparty_mlnx(){
if [ ! -e release-drivers/mlnx/${mlnx_tgz_name} ] ; then if [ ! -e release-drivers/mlnx/${mlnx_tgz_name} ] ; then
./download-and-copy-drivers.sh ./download-and-copy-drivers.sh
if (( $? != 0 )) ; then exit 1 ; fi
mv ${mlnx_tgz_name} release-drivers/mlnx/ mv ${mlnx_tgz_name} release-drivers/mlnx/
else else
sha256_tmp=$(sha256sum release-drivers/mlnx/${mlnx_tgz_name} | awk '{printf $1}') sha256_tmp=$(sha256sum release-drivers/mlnx/${mlnx_tgz_name} | awk '{printf $1}')
if [[ $sha256_tmp != $mlnx_tgz_sha256 ]]; then if [[ $sha256_tmp != $mlnx_tgz_sha256 ]]; then
echo "Warning: release-drivers/mlnx/${mlnx_tgz_name} is exist, but sha256sum is not correct!" echo "Warning: release-drivers/mlnx/${mlnx_tgz_name} is exist, but sha256sum is not correct!"
./download-and-copy-drivers.sh ./download-and-copy-drivers.sh
if (( $? != 0 )) ; then exit 1 ; fi
mv ${mlnx_tgz_name} release-drivers/mlnx/ mv ${mlnx_tgz_name} release-drivers/mlnx/
fi fi
fi fi