-
Notifications
You must be signed in to change notification settings - Fork 302
[CI] [GHA] Split wheel building job into several parallel ones #3032
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.
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
base: master
Are you sure you want to change the base?
[CI] [GHA] Split wheel building job into several parallel ones #3032
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enables sccache (Shared Compiler Cache) across multiple build configurations in the Linux GitHub Actions workflow to improve build performance through compiler output caching.
- Adds
CMAKE_CXX_COMPILER_LAUNCHERandCMAKE_C_COMPILER_LAUNCHERflags pointing tosccachein all CMake invocations - Configures
sccachefor both direct CMake builds and Python wheel builds (via--config-settings) - Applies caching uniformly across different build scenarios: library builds, Python bindings, samples, JS bindings, and GenAI components
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.github/workflows/linux.yml
Outdated
| --config-settings=override=cross.arch="manylinux_2_31_x86_64" \ | ||
| --config-settings=cmake.args="-DCMAKE_CXX_COMPILER_LAUNCHER=sccache -DCMAKE_C_COMPILER_LAUNCHER=sccache" \ | ||
| ${{ needs.openvino_download.outputs.ov_wheel_source }} \ | ||
| ${{ env.SRC_DIR }} 2>&1 | tee ${{ env.WHEELS_DIR }}/build_py${py_version}.log |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't GenAI compilation already parallelized inside of each python case?
This reverts commit 933df05.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sccache in the wheel build job There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.