reduce ci performance time #599
Conversation
|
|
… upload them to the cache
There was a problem hiding this comment.
Pull Request Overview
This PR consolidates multiple GitHub Actions workflows to reduce CI performance time by merging initialization environments and compilation steps. The main optimization involves eliminating duplicate setup work across test jobs and removing redundant sysbench tests from memtracer testing.
- Merged separate
test.ymlandbuild.ymlworkflows into a singlebuild-test.ymlworkflow - Introduced artifact caching and sharing between jobs to avoid rebuilding dependencies
- Consolidated memtracer sysbench testing into the main sysbench test matrix
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
.github/workflows/test.yml |
Removed standalone test workflow file |
.github/workflows/build.yml |
Removed standalone build workflow file |
.github/workflows/build-test.yml |
New consolidated workflow with optimized job dependencies and artifact sharing |
| strategy: | ||
| matrix: | ||
| include: | ||
| # 基础sysbench测试 |
There was a problem hiding this comment.
[nitpick] The comment is in Chinese. Consider using English comments for better international collaboration and consistency with the rest of the codebase.
| # 基础sysbench测试 | |
| # Basic sysbench test |
Copilot uses AI. Check for mistakes.
| test_case: 'miniob_select' | ||
| build_type: 'release' | ||
| memtracer: '' | ||
| # memtracer sysbench测试 |
There was a problem hiding this comment.
[nitpick] The comment is in Chinese. Consider using English comments for better international collaboration and consistency with the rest of the codebase.
| # memtracer sysbench测试 | |
| # memtracer sysbench test |
Copilot uses AI. Check for mistakes.
| - name: Download build artifacts | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: ${{ matrix.build_type == 'debug' && 'build-debug-artifacts' || 'build-release-artifacts' }} |
There was a problem hiding this comment.
The workflow references 'build-debug-artifacts' but no job creates this artifact. Only 'build-release-artifacts' is uploaded by the build-release job, which could cause the sysbench-test job to fail when matrix.build_type is 'debug'.
Copilot uses AI. Check for mistakes.
There was a problem hiding this comment.
At present, the system bench does not conduct testing based on debug mode, so no debug artifacts have been uploaded.
3 participants
What problem were solved in this pull request?
Issue Number: close #538
Problem:
What is changed and how it works?
Merge initialization environments and compilation steps for multiple tests.
Remove a duplicate test from memtracer test and sysbench test.
Other information
This is just a process optimization, a more effective method is to reduce the time of sysbench testing, and this PR has not completely solved the problem.