From 7c5eb76113912cbc940c535993ff38218ec1f1cf Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Tue, 10 Nov 2020 17:49:10 +0800 Subject: [PATCH 1/4] delete unused variable --- tools/ci/config/target-test.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/ci/config/target-test.yml b/tools/ci/config/target-test.yml index c5f17599a2..cfdf5afc60 100644 --- a/tools/ci/config/target-test.yml +++ b/tools/ci/config/target-test.yml @@ -58,7 +58,6 @@ - .target_test_job_template - .rules:tests:unit_test variables: - TEST_CASE_PATH: "$COMPONENT_UT_DIRS" CONFIG_FILE_PATH: "${CI_PROJECT_DIR}/component_ut/test_configs" script: - *define_config_file_name From 9e4f5dd01ffb4c9c880425444e50e80f8db097f8 Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Tue, 10 Nov 2020 17:48:35 +0800 Subject: [PATCH 2/4] CI: Add dependency "fast_template_app" to build jobs --- tools/ci/config/build.yml | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/tools/ci/config/build.yml b/tools/ci/config/build.yml index 1b209c7c2f..1c968b3cb5 100644 --- a/tools/ci/config/build.yml +++ b/tools/ci/config/build.yml @@ -43,7 +43,9 @@ # build-related-pre-check-jobs ------------------------------------------------ # Build at least one project for each target at earliest stage to reduce build cost for obvious failing commits fast_template_app: - extends: .build_template_app_template + extends: + - .build_template_app_template + - .rules:build_tests:target_test-weekend_test stage: pre_check variables: BUILD_COMMAND_ARGS: "-p" @@ -89,6 +91,10 @@ build_ssc_esp32s2: extends: - .build_template - .rules:build_tests:unit_test + needs: + - job: fast_template_app + artifacts: false + - scan_tests artifacts: paths: - tools/unit-test-app/output/${IDF_TARGET} @@ -132,6 +138,10 @@ build_esp_idf_tests_cmake_esp32s3: extends: - .build_template - .rules:build_tests:example_test-weekend_test + needs: + - job: fast_template_app + artifacts: false + - scan_tests variables: TEST_PREFIX: examples TEST_RELATIVE_DIR: examples @@ -168,8 +178,6 @@ build_examples_make: # same as above, but for CMake .build_examples_cmake: extends: .build_examples_template - needs: - - scan_tests artifacts: paths: - build_${TEST_PREFIX}/list.json @@ -291,6 +299,9 @@ build_docs_pdf: extends: - .build_template - .rules:build_tests:weekend_test + needs: + - job: fast_template_app + artifacts: false script: - ${IDF_PATH}/tools/ci/test_configure_ci_environment.sh - rm -rf test_build_system @@ -390,6 +401,9 @@ build_installer: # This job builds template app with permutations of targets and optimization levels build_template_app: + needs: + - job: fast_template_app + artifacts: false extends: .build_template_app_template # Sonarqube related jobs put here for this reason: From 22168a791034343f76df60f98025dfdc5af4c71a Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Thu, 19 Nov 2020 15:08:53 +0800 Subject: [PATCH 3/4] remove all pre-stage artifacts in build stage jobs --- tools/ci/config/build.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/ci/config/build.yml b/tools/ci/config/build.yml index 1c968b3cb5..90c36c004f 100644 --- a/tools/ci/config/build.yml +++ b/tools/ci/config/build.yml @@ -5,6 +5,7 @@ - build variables: SIZE_INFO_LOCATION: "$CI_PROJECT_DIR/size_info.txt" + dependencies: [] .build_template_app_template: extends: From 4bd2130eab8444dad51cf6676d7985c4dd1d3887 Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Fri, 27 Nov 2020 19:11:59 +0800 Subject: [PATCH 4/4] set dependencies to null to avoid missing artifacts issue --- tools/ci/config/build.yml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tools/ci/config/build.yml b/tools/ci/config/build.yml index 90c36c004f..6410b8081b 100644 --- a/tools/ci/config/build.yml +++ b/tools/ci/config/build.yml @@ -92,6 +92,7 @@ build_ssc_esp32s2: extends: - .build_template - .rules:build_tests:unit_test + dependencies: # set dependencies to null to avoid missing artifacts issue needs: - job: fast_template_app artifacts: false @@ -139,6 +140,7 @@ build_esp_idf_tests_cmake_esp32s3: extends: - .build_template - .rules:build_tests:example_test-weekend_test + dependencies: # set dependencies to null to avoid missing artifacts issue needs: - job: fast_template_app artifacts: false @@ -300,6 +302,7 @@ build_docs_pdf: extends: - .build_template - .rules:build_tests:weekend_test + dependencies: # set dependencies to null to avoid missing artifacts issue needs: - job: fast_template_app artifacts: false @@ -394,6 +397,7 @@ build_installer: image: $CI_DOCKER_REGISTRY/wine-innosetup:1 tags: - build + dependencies: # set dependencies to null to avoid missing artifacts issue needs: - build_cmdlinerunner script: @@ -440,6 +444,7 @@ build_template_app: REPORT_DIR: examples/get-started/hello_world/tidybuild/report tags: - host_test + dependencies: # set dependencies to null to avoid missing artifacts issue needs: - clang_tidy_check_regular