From 129722db37292fe69eb21a1cd86ef7a4c6b2ec20 Mon Sep 17 00:00:00 2001 From: Changming Sun Date: Tue, 31 Aug 2021 14:13:55 -0700 Subject: [PATCH] Add android binary size monitor back (#8904) --- .../linux-cpu-minimal-build-ci-pipeline.yml | 13 +++++++++++ ...rt_android_baseline_and_report_bin_size.sh | 23 ++----------------- 2 files changed, 15 insertions(+), 21 deletions(-) diff --git a/tools/ci_build/github/azure-pipelines/linux-cpu-minimal-build-ci-pipeline.yml b/tools/ci_build/github/azure-pipelines/linux-cpu-minimal-build-ci-pipeline.yml index faa97f0705..4a70d48f73 100644 --- a/tools/ci_build/github/azure-pipelines/linux-cpu-minimal-build-ci-pipeline.yml +++ b/tools/ci_build/github/azure-pipelines/linux-cpu-minimal-build-ci-pipeline.yml @@ -177,6 +177,19 @@ jobs: /bin/bash /onnxruntime_src/tools/ci_build/github/linux/ort_minimal/build_minimal_ort_android_baseline_and_report_bin_size.sh workingDirectory: $(Build.SourcesDirectory) + + - task: AzureCLI@2 + displayName: 'Azure CLI' + condition: and (succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'), eq(variables['Build.SourceBranch'], 'refs/heads/master')) + inputs: + azureSubscription: AIInfraBuild + scriptLocation: inlineScript + scriptType: bash + inlineScript: | + /usr/bin/python3 $(Build.SourcesDirectory)/tools/ci_build/github/windows/post_binary_sizes_to_dashboard.py --commit_hash=$(Build.SourceVersion) \ + --size_data_file=$(Build.BinariesDirectory)/MinSizeRel/binary_size_data.txt --build_project=onnxruntime --build_id=$(Build.BuildId) + workingDirectory: '$(Build.BinariesDirectory)' + - task: PublishTestResults@2 displayName: 'Publish unit test results' inputs: diff --git a/tools/ci_build/github/linux/ort_minimal/build_minimal_ort_android_baseline_and_report_bin_size.sh b/tools/ci_build/github/linux/ort_minimal/build_minimal_ort_android_baseline_and_report_bin_size.sh index 8da5048e43..e31fc0dad6 100755 --- a/tools/ci_build/github/linux/ort_minimal/build_minimal_ort_android_baseline_and_report_bin_size.sh +++ b/tools/ci_build/github/linux/ort_minimal/build_minimal_ort_android_baseline_and_report_bin_size.sh @@ -35,24 +35,5 @@ python3 /onnxruntime_src/tools/ci_build/github/linux/ort_minimal/check_build_bin --threshold=$BINARY_SIZE_LIMIT_IN_BYTES \ /build/MinSizeRel/libonnxruntime.so -# Post the binary size info to ort mysql DB -# The report script's DB connection failure will not fail the pipeline -# To reduce noise, we only report binary size for Continuous integration (a merge to master) -if [[ $BUILD_REASON == "IndividualCI" || $BUILD_REASON == "BatchedCI" ]] && [[ $BUILD_BRANCH == "refs/heads/master" ]]; then - # Install the mysql connector - python3 -m pip install --user mysql-connector-python - - python3 /onnxruntime_src/tools/ci_build/github/windows/post_binary_sizes_to_dashboard.py \ - --ignore_db_error \ - --commit_hash=$BUILD_SOURCEVERSION \ - --size_data_file=/build/MinSizeRel/binary_size_data.txt \ - --build_project=onnxruntime \ - --build_id=$BUILD_ID -else - echo "No binary size report for build reason: [$BUILD_REASON] and build branch: [$BUILD_BRANCH]" - echo "The content of binary_size_data.txt" - cat /build/MinSizeRel/binary_size_data.txt -fi - -# Clear the build -rm -rf /build/MinSizeRel +echo "The content of binary_size_data.txt" +cat /build/MinSizeRel/binary_size_data.txt