From e5509b0db08a29b413b85ecfd7fd1f9a603144fb Mon Sep 17 00:00:00 2001 From: Mengwei Liu Date: Fri, 13 Sep 2024 13:41:14 -0700 Subject: [PATCH] [CI] Try to fix test_model.sh Trying to fix failures such as https://github.com/pytorch/executorch/actions/runs/10855583772/job/30128512970 For the time missing operator error is repro'ed on my Mac, I see cmake-out/ was not cleaned up and merged.yaml was missing linear.out because it was super old. Now without deep understanding of how CI job workers cache previous runs, I'm refactoring `build_cmake_executor_runner` in test_model.sh to make sure it always build clean. --- .ci/scripts/test_model.sh | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/.ci/scripts/test_model.sh b/.ci/scripts/test_model.sh index f63760ef1e3..f558a508c93 100755 --- a/.ci/scripts/test_model.sh +++ b/.ci/scripts/test_model.sh @@ -50,14 +50,13 @@ prepare_artifacts_upload() { build_cmake_executor_runner() { echo "Building executor_runner" - (rm -rf ${CMAKE_OUTPUT_DIR} \ - && mkdir ${CMAKE_OUTPUT_DIR} \ - && cd ${CMAKE_OUTPUT_DIR} \ - && retry cmake -DCMAKE_BUILD_TYPE=Release \ + rm -rf ${CMAKE_OUTPUT_DIR} + cmake -DCMAKE_BUILD_TYPE=Debug \ -DEXECUTORCH_BUILD_KERNELS_OPTIMIZED=ON \ - -DPYTHON_EXECUTABLE="$PYTHON_EXECUTABLE" ..) + -DPYTHON_EXECUTABLE="$PYTHON_EXECUTABLE" \ + -B${CMAKE_OUTPUT_DIR} . - cmake --build ${CMAKE_OUTPUT_DIR} -j4 + cmake --build ${CMAKE_OUTPUT_DIR} -j4 --config Debug } run_portable_executor_runner() { @@ -65,9 +64,7 @@ run_portable_executor_runner() { if [[ "${BUILD_TOOL}" == "buck2" ]]; then buck2 run //examples/portable/executor_runner:executor_runner -- --model_path "./${MODEL_NAME}.pte" elif [[ "${BUILD_TOOL}" == "cmake" ]]; then - if [[ ! -f ${CMAKE_OUTPUT_DIR}/executor_runner ]]; then - build_cmake_executor_runner - fi + build_cmake_executor_runner ./${CMAKE_OUTPUT_DIR}/executor_runner --model_path "./${MODEL_NAME}.pte" else echo "Invalid build tool ${BUILD_TOOL}. Only buck2 and cmake are supported atm"