fix(spread): tests must also use newer Chisel
Metadata
Current evaluation
Merged as b3bc4d70 in PR #456: spread tests now install Chisel from the Snap Store instead of building a hardcoded commit, resolving failures exposed by chisel-releases#78.
Resolution: closed resolved
Reason: PR #456 was merged as commit b3bc4d70. It updates the failing spread tasks to use the newer Snap Store Chisel; the later timeout was a test-rerun issue, not a rejection.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
After https://github.com/canonical/chisel-releases/pull/78, the spread tests are now also failing because the building of the Chisel tool was still being done manually, from a hardcoded commit.
This PR changes the failing spread tasks such that Chisel is also installed from the snap store.
Fixes the tests for https://github.com/canonical/rockcraft/pull/455#issuecomment-1890672106.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as b3bc4d70 in PR #456: spread tests now install Chisel from the Snap Store instead of building a hardcoded commit, resolving failures exposed by chisel-releases#78. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as b3bc4d70: spread tests now install Chisel from the Snap Store instead of building a hardcoded commit. The fix was also cherry-picked for Rockcraft 1.1.2. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #456 as commit b3bc4d70, replacing hardcoded Chisel builds in spread tests with snap-store installation. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as b3bc4d70, updating spread tests to install Chisel from the Snap Store instead of a hardcoded commit. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Resolves failing spread tests by switching Chisel installation from a hardcoded commit to the snap store. Addressed review feedback and re-triggered CI after a timeout. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated spread test tasks to install Chisel from the snap store instead of building it from a hardcoded commit, resolving CI failures. The change passed tests after a transient timeout was re-triggered. |
Update history
No update history recorded yet.
Related work
-
Caused By:
chisel-releases#78
(confidence 99%)
The PR identifies this Chisel release as the change that exposed failures in spread tests using a hardcoded build.
-
Related To:
canonical/rockcraft#455
(confidence 98%)
The PR explicitly states that it fixes the spread-test failure reported in the linked #455 comment.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #466 fix(spread): tests must also use newer Chisel (#456) | rockcraft | merged | Merged the 1.1.2 cherry-pick of #456 (main commit b3bc4d70), updating spread tests to install newer Chisel from the Snap Store instead of a hardcoded build. | |
| #469 fix(spread): tests must also use newer Chisel (#456) (#466) | rockcraft | closed | Merged #469, backporting #456 as b3bc4d70 so spread tests use the newer Chisel from the Snap Store in the 1.1.2 release. | |
| #467 fix(spread): tests must also use newer Chisel (#456) (#466) | rockcraft | closed | Merged into the 1.1.2 release branch as a cherry-pick of #466, updating spread tests to use Chisel from the Snap Store; originating fix: #456, commit b3bc4d70. | |
| #162 Use Chisel `main` branch for Spread tests | rockcraft | closed | Chisel PR canonical/chisel#36 merged, enabling Rockcraft Spread tests to use Chisel main after the temporary development branch was removed. | |
| #155 spread: fix project with dotnet chisel slices | rockcraft | merged | Merged PR #155 (commit 1780659a) removed an obsolete .NET prime override and symlink after packages stopped creating version-specific subdirectories, unblocking Spread tests. | |
| #656 test(spread): test real chisel run with updates and security pockets | fetch-service | merged | Merged #656, expanding the Chisel spread test to run curl across base, updates, and security pockets, clean artifacts, verify curl in chroot, and record approved package metadata. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Merged PR #13 added Spread tests for architectures, Chisel slices, craftctl, package repositories, and patchelf (+206/-5 across 15 files). | |
| #18 build: readd chisel for snapcraft 8 | snapcraft-rocks | merged | PR #18 merged, restoring Chisel and its Snapcraft 8.x-only spread test while bumping Snapcraft to 8.0.2. | |
| #152 spread: disable general/chisel | rockcraft | closed | Closed unmerged after chisel-releases#23 fixed the dotnet/Chisel bug; the obsolete Spread handling was addressed separately by rockcraft#155, merged as 1780659a. | |
| #792 ci: install chisel as a snap | craft-parts | closed | Closed unmerged after inactivity; equivalent Chisel-snap CI and home-fixture changes were merged in craft-parts#793, fixing craft-parts#746. |