Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update for pr_coverage.out #112

Merged
merged 5 commits into from
Aug 26, 2024
Merged

update for pr_coverage.out #112

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
42dab9f
update for pr_coverage.out
Rosyrain Aug 26, 2024
c32bc95
update runner to use amd64-mo-guangzhou-2xlarge16
Rosyrain Aug 26, 2024
6ab5e90
update
Rosyrain Aug 26, 2024
33aa342
update exit 0
Rosyrain Aug 26, 2024
4f2619e
update python fetch-depth
Rosyrain Aug 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 18 additions & 4 deletions .github/workflows/utils.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ jobs:
if: ${{ needs.check_organization_user.outputs.safe_label == '1' || needs.check_organization_user.outputs.in_org == '1' }}
name: PR Unit and BVT Test Coverage
needs: [check_organization_user]
runs-on: ubuntu-22.04
runs-on: amd64-mo-guangzhou-2xlarge16
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -104,7 +104,7 @@ jobs:
- name: Set up Go and JAVA
uses: matrixorigin/CI/actions/setup-env@main
with:
setup-java: true
setup-java: false
- name: Set Variables
run: |
echo "ut_report='UT-Report.out'" >> $GITHUB_ENV
Expand All @@ -117,6 +117,10 @@ jobs:
echo "ut_excluded_pkg='pkg/pb\|pkg/sql/parsers/goyacc\|yaccpar'" >> $GITHUB_ENV
echo "bvt_excluded_pkg='pkg/pb\|yaccpar'" >> $GITHUB_ENV

- name: Set JAVA PATH
run: |
echo "$JAVA_HOME/bin" >> $GITHUB_PATH

- name: Clone test-tool repository
uses: actions/checkout@v4
with:
Expand Down Expand Up @@ -204,7 +208,7 @@ jobs:
uses: actions/checkout@v3
with:
repository: matrixorigin/CI
fetch-depth: "0"
fetch-depth: "1"
path: CI
- name: Get Python file
run: |
Expand All @@ -216,8 +220,18 @@ jobs:
python $GITHUB_WORKSPACE/parse_coverage.py -coverage_files $GITHUB_WORKSPACE/matrixone/ut_coverage.out $GITHUB_WORKSPACE/matrixone/bvt_coverage.out -diff_path $GITHUB_WORKSPACE/matrixone/diff.patch

- name: Compress final result files
id: compress_result
if: ${{ always() && !cancelled() }}
run: |
cd $GITHUB_WORKSPACE
if [ ! -f "$GITHUB_WORKSPACE/pr_coverage.out" ]; then
echo "pr_coverage.out does not exist.The code block has not changed,skip the remaining steps."
echo "file_exists=false" >> $GITHUB_OUTPUT
exit 0
fi

echo "file_exists=true" >> $GITHUB_OUTPUT

cd $GITHUB_WORKSPACE/matrixone
cp $GITHUB_WORKSPACE/pr_coverage.out $GITHUB_WORKSPACE/matrixone/pr_coverage.out
Rosyrain marked this conversation as resolved.
Show resolved Hide resolved
go tool cover -o pr_coverage.html -html=pr_coverage.out
Rosyrain marked this conversation as resolved.
Show resolved Hide resolved
Expand All @@ -231,7 +245,7 @@ jobs:
zip -r final_result_files.zip pr_coverage.html pr_coverage.out merged_coverage.out
Rosyrain marked this conversation as resolved.
Show resolved Hide resolved
- name: Upload final result files
uses: actions/upload-artifact@v4
if: ${{ always() && !cancelled() }}
if: ${{ always() && !cancelled() && steps.compress_result.outputs.file_exists == 'true' }}
continue-on-error: true
with:
name: final-result-files
Expand Down