[CI] Pack release build #17781
[CI] Pack release build #17781
Conversation
This patch adds a step to conditionally pack pure sycl-toolchain for release.
KornevNikita
temporarily deployed
to
WindowsCILock
GitHub Actions
Inactive
|
@AlexeySachkov FYI |
KornevNikita
temporarily deployed
to
WindowsCILock
GitHub Actions
Inactive
| - name: Pack toolchain release | ||
| if: ${{ always() && !cancelled() && steps.build.conclusion == 'success' && inputs.pack_release == 'true' }} | ||
| run: tar -I '${{ steps.artifact_info.outputs.COMPRESS }}' -cf ${{ steps.artifact_info.outputs.ARCHIVE_NAME }} -C $GITHUB_WORKSPACE/build/install . |
There was a problem hiding this comment.
how is this different from the archive created in Pack toolchain on line 238/261?
There was a problem hiding this comment.
The main difference is lack of extra tools that are only used in testing - they are being built and deployed on lines 246-253
Going forward, we would like to release binaries and for that we would like to minimize our release package by at least dropping internal testing tools.
Of course, the current approach isn't exactly the most efficient, because it effectively bundles the SYCL RT twice into artifacts, but that's a start, it will be improved later
There was a problem hiding this comment.
is this only for sycl-rel or will we be releasing the new no-tools package from the sycl branch too?
There was a problem hiding this comment.
Yes it's only for sycl-rel. There is inputs.pach_release to trigger this step
There was a problem hiding this comment.
I just preserved it during the refactoring, it was there long before me :D
There was a problem hiding this comment.
Then it's time to eliminate it:)) I'll create a PR.
Anyway, @sarnex @aelovikov-intel, can we merge this one and then refactor the workflow file?
There was a problem hiding this comment.
@sarnex @aelovikov-intel gentle friendly soft ping
There was a problem hiding this comment.
I'm fine, but I think @sarnex was the one objecting, so let's wait for him to confirm he's ok.
KornevNikita
temporarily deployed
to
WindowsCILock
GitHub Actions
Inactive
KornevNikita
temporarily deployed
to
WindowsCILock
GitHub Actions
Inactive
This patch adds a step to conditionally pack pure sycl-toolchain for release.