Repository navigation
feat(ci): validate packages - #57
Conversation
- Added a new verification job to ensure the published package matches the built artifact. - Running minimal examples to validate the package functionality. Signed-off-by: Patryk Kamiński <patryk.kaminski@intel.com>
- Introduced a new Python script to check that the installed torch-tweak release can run on an Intel XPU. - Updated the release workflow to utilize this script for verifying package integrity and compatibility during CI. Signed-off-by: Patryk Kamiński <patryk.kaminski@intel.com>
- Replaced the inline verification steps in the release workflow with a dedicated `verify-release.yml` workflow. - Possible to rerun the verification without publishing a package Signed-off-by: Patryk Kamiński <patryk.kaminski@intel.com>
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Copilot review overview
2 open issues
What changed in this PR
Adds a post-publish validation workflow that installs the newly published torch-tweak wheel and verifies basic XPU functionality, and wires it into the release pipeline.
Changes:
- Introduces a reusable
verify-releaseworkflow to download, install, and run a minimal XPU smoke test against a published wheel. - Updates
release.ymlto call the new verification workflow after publishing and to pass the resolved version via job outputs. - Adds a Python script to validate that the installed package matches the expected release and can run a simple XPU computation.
| File | Description |
|---|---|
| .github/workflows/verify-release.yml | New workflow to download published wheel, optionally compare with build artifact, install, and run smoke checks on Intel GPU. |
| .github/workflows/release.yml | Wires verification into the release pipeline and exposes the published version as a job output. |
| .github/scripts/check_installed_package.py | New script that validates installed distribution metadata and runs a minimal XPU computation. |
Review effort: Lite
| runs-on: ubuntu-latest | ||
| environment: | ||
| name: ${{ inputs.target }} | ||
| url: >- |
There was a problem hiding this comment.
not related, but I just looked at it - do we need this url?
There was a problem hiding this comment.
This will automatically place these urls in the Deployments tab
| - name: Set up Python | ||
| uses: actions/setup-python@6d4fd22b20465ccc0f7354476588e8df7c9ca0e0 # v7.1.0 | ||
| with: | ||
| python-version: "3.14" |
There was a problem hiding this comment.
nit: you can add the same comment as in release.yml (and maybe more, to keep it the same value).
There was a problem hiding this comment.
python is not used in release.yml anymore
No description provided.