From fd0c0e3c972b1ad67e2cddc0881e7fe1537c1937 Mon Sep 17 00:00:00 2001 From: tianshuo78520a <707759223@qq.com> Date: Fri, 25 Mar 2022 14:32:26 +0800 Subject: [PATCH] Add Coverage build size check (#40749) --- paddle/scripts/paddle_build.sh | 36 ++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/paddle/scripts/paddle_build.sh b/paddle/scripts/paddle_build.sh index 2f437a7d66..407cf08c95 100755 --- a/paddle/scripts/paddle_build.sh +++ b/paddle/scripts/paddle_build.sh @@ -2927,6 +2927,41 @@ function build_develop() { cmake_gen_and_build ${PYTHON_ABI:-""} ${parallel_number} } +function check_coverage_build() { + if [ ! "${buildSize}" ];then + echo "build size not found" + exit 1 + fi + + if [ ${WITH_COVERAGE} != "ON" ];then + echo "WARNING: check_coverage need to compile with WITH_COVERAGE=ON, but got WITH_COVERAGE=OFF" + exit 1 + fi + + rm -f build_size + curl -O https://paddle-docker-tar.bj.bcebos.com/paddle_ci_index/build_size + dev_coverage_build_size=`cat build_size|sed 's#G##g'` + pr_coverage_build_size=`echo $buildSize|sed 's#G##g'` + + diff_coverage_build_size=`echo $(($pr_coverage_build_size - $dev_coverage_build_size))` + + set +x + if [ ${diff_coverage_build_size} -gt 3 ]; then + approval_line=`curl -H "Authorization: token ${GITHUB_API_TOKEN}" https://api.github.com/repos/PaddlePaddle/Paddle/pulls/${GIT_PR_ID}/reviews?per_page=10000` + APPROVALS=`echo ${approval_line}|python ${PADDLE_ROOT}/tools/check_pr_approval.py 1 29832297 6836917 43953930` + echo "current pr ${GIT_PR_ID} got approvals: ${APPROVALS}" + if [ "${APPROVALS}" == "FALSE" ]; then + echo "==========================================================================================" + echo "This PR make the release paddlepaddle coverage build size growth exceeds 3 G, please explain why your PR exceeds 3G to ext_ppee@baidu.com." + echo "Then you must have one RD (tianshuo78520a (Recommend) or luotao1 or phlrain) approval for this PR\n" + echo "==========================================================================================" + exit 6 + fi + fi + set -x +} + + function main() { local CMD=$1 local parallel_number=$2 @@ -3053,6 +3088,7 @@ function main() { check_diff_file_for_coverage cmake_gen_and_build ${PYTHON_ABI:-""} ${parallel_number} enable_unused_var_check + check_coverage_build ;; gpu_cicheck_coverage) check_approvals_of_unittest 1 -- GitLab