[Tool] calculate fe patch coverage by covercheck (#21802)
calculate fe patch coverage by covercheck Signed-off-by: AndyZiYe <yeziyu@starrocks.com>
This commit is contained in:
parent
643ecf7058
commit
f501d7a74b
|
|
@ -72,3 +72,22 @@ jobs:
|
|||
with:
|
||||
labels: 2.3
|
||||
|
||||
close_msg:
|
||||
runs-on: self-hosted
|
||||
if: github.event.pull_request.merged == true &&
|
||||
!contains(github.event.pull_request.title, 'cherry-pick') &&
|
||||
!contains(github.event.pull_request.title, 'backport')
|
||||
env:
|
||||
PR_NUMBER: ${{ github.event.number }}
|
||||
steps:
|
||||
- name: merge_commit_sha
|
||||
run: |
|
||||
set -x
|
||||
merge_commit_sha=`curl -L \
|
||||
-H "Accept: application/vnd.github+json" \
|
||||
-H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" \
|
||||
-H "X-GitHub-Api-Version: 2022-11-28" \
|
||||
https://api.github.com/repos/${{ github.repository }}/pulls/${PR_NUMBER} 2>/dev/null \
|
||||
| jq .merge_commit_sha`
|
||||
merge_commit_sha=$(echo ${merge_commit_sha//\"})
|
||||
echo "merge_commit_sha=${merge_commit_sha}" >> $GITHUB_OUTPUT
|
||||
|
|
@ -286,37 +286,17 @@ jobs:
|
|||
cd elastic-service && git pull
|
||||
./bin/elastic-ut.sh --pr ${PR_NUMBER} --module fe --branch ${{steps.branch.outputs.branch}}
|
||||
|
||||
- name: Clean ECI
|
||||
if: always()
|
||||
run: |
|
||||
echo ${{ steps.run_ut.outputs.ECI_ID }}
|
||||
eci rm ${{ steps.run_ut.outputs.ECI_ID }}
|
||||
|
||||
- name: Prepare coverage files
|
||||
run: |
|
||||
cd ${{ github.workspace }}/fe/fe-core
|
||||
cp -rf ${{ steps.run_ut.outputs.COV_DIR }} target/
|
||||
|
||||
- name: MERGE_COMMIT_SHA
|
||||
id: merge_commit_sha
|
||||
run: |
|
||||
set -x
|
||||
merge_commit_sha=`curl -L \
|
||||
-H "Accept: application/vnd.github+json" \
|
||||
-H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" \
|
||||
-H "X-GitHub-Api-Version: 2022-11-28" \
|
||||
https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.number }} 2>/dev/null \
|
||||
| jq .merge_commit_sha`
|
||||
merge_commit_sha=$(echo ${merge_commit_sha//\"})
|
||||
echo "merge_commit_sha=${merge_commit_sha}" >> $GITHUB_OUTPUT
|
||||
|
||||
- name: Upload cov with merge_commit_sha
|
||||
uses: codecov/codecov-action@v3
|
||||
with:
|
||||
override_branch: ${{ github.base_ref }}
|
||||
override_commit: ${{ steps.merge_commit_sha.outputs.merge_commit_sha }}
|
||||
token: ${{ secrets.CODECOV_TOKEN }}
|
||||
files: ${{ github.workspace }}/fe/fe-core/target/${{ steps.run_ut.outputs.COV_FILE }}
|
||||
dry_run: false
|
||||
override_pr: 21756
|
||||
name: main-fe-ut
|
||||
fail_ci_if_error: true
|
||||
verbose: true
|
||||
|
||||
- name: Coverage Report
|
||||
uses: codecov/codecov-action@v3
|
||||
with:
|
||||
|
|
@ -327,6 +307,26 @@ jobs:
|
|||
fail_ci_if_error: false
|
||||
verbose: true
|
||||
|
||||
- name: Incremental Coverage Report
|
||||
run: |
|
||||
set -ex
|
||||
rm -rf ./coverchecker
|
||||
ln -s /var/local/env/coverchecker ./coverchecker && cd coverchecker && git pull
|
||||
time_count=0
|
||||
pull_status=1
|
||||
export JAVA_HOME=/var/local/env/jdk1.8.0_202;
|
||||
export PATH=$JAVA_HOME/bin:$PATH;
|
||||
while (( $pull_status != 0 )); do
|
||||
if (( $time_count == 3 )); then
|
||||
exit 1
|
||||
fi
|
||||
timeout 180 java -jar cover-checker-console/target/cover-checker-console-1.4.0-jar-with-dependencies.jar \
|
||||
--cover ${{ github.workspace }}/fe/fe-core/target/result/ --github-token ${{ secrets.GITHUB_TOKEN }} \
|
||||
--repo ${{ github.repository }} --threshold 80 --github-url api.github.com --pr ${PR_NUMBER} -type jacoco
|
||||
pull_status=$?
|
||||
time_count=`expr $time_count + 1`
|
||||
done
|
||||
|
||||
- name: UT Report
|
||||
if: always()
|
||||
uses: mikepenz/action-junit-report@v3
|
||||
|
|
@ -336,12 +336,6 @@ jobs:
|
|||
token: ${{ secrets.GITHUB_TOKEN }}
|
||||
report_paths: ${{ steps.run_ut.outputs.FE_REPORT_DIR }}/*.xml
|
||||
|
||||
- name: Clean ECI
|
||||
if: always()
|
||||
run: |
|
||||
echo ${{ steps.run_ut.outputs.ECI_ID }}
|
||||
eci rm ${{ steps.run_ut.outputs.ECI_ID }}
|
||||
|
||||
- name: Upload log
|
||||
if: always()
|
||||
uses: actions/upload-artifact@v3
|
||||
|
|
@ -357,7 +351,6 @@ jobs:
|
|||
rm -f ${{ steps.run_ut.outputs.RES_FILE }}
|
||||
rm -f ${{ steps.run_ut.outputs.RES_LOG }}
|
||||
rm -rf ${{ steps.run_ut.outputs.COV_DIR }}
|
||||
|
||||
build:
|
||||
runs-on: self-hosted
|
||||
needs: [be-checker, fe-checker, be-ut, fe-ut]
|
||||
|
|
|
|||
10
codecov.yml
10
codecov.yml
|
|
@ -6,10 +6,10 @@ coverage:
|
|||
status:
|
||||
project:
|
||||
fe-ut:
|
||||
target: 60%
|
||||
target: 80%
|
||||
threshold: 5%
|
||||
only_pulls: true
|
||||
patch:
|
||||
fe-ut:
|
||||
target: 80%
|
||||
only_pulls: true
|
||||
# patch:
|
||||
# fe-ut:
|
||||
# target: 80%
|
||||
# only_pulls: true
|
||||
Loading…
Reference in New Issue