Use Chisel `main` branch for Spread tests
Metadata
Current evaluation
Chisel PR canonical/chisel#36 merged, enabling Rockcraft Spread tests to use Chisel main after the temporary development branch was removed.
Resolution: closed resolved
Reason: The closing maintainer comment confirmed that canonical/chisel#36 had merged and that Spread tests now cloned Chisel's main branch.
Evaluated by: mr-cal-bot
No scores available.
Issue body
We're temporarily using a development branch in [docs/how-to/code/create-slice/task.yaml](https://github.com/canonical/rockcraft/pull/153/files/123cf4aa31d908f55c958d18ef6c95dabf391a23#diff-11e0033250351fc520ac0970830ea8ca98ed9e6485e59cb0bd680ac3d344d7b3), due to an existing bug.
Once https://github.com/canonical/chisel/pull/36 is merged, we can switch back to Chisel's main branch
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Chisel PR canonical/chisel#36 merged, enabling Rockcraft Spread tests to use Chisel main after the temporary development branch was removed. | |
| stealth/space-bunny-alpha | — | closed resolved | Spread tests returned to Chisel main after canonical/chisel#36 merged; later commit b3bc4d70 (rockcraft#456) updated the same test for a newer Chisel release. | |
| stealth/space-bunny-alpha | — | closed resolved | Chisel#36 merged, allowing Rockcraft's Spread tests to clone Chisel main instead of the temporary development branch. | |
| stealth/space-bunny-alpha | — | closed resolved | Closed after canonical/chisel#36 merged and the temporary development branch was removed; a maintainer verified that Spread tests cloned Chisel's main branch. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved by merging upstream Chisel PR #36. Repository configuration updated to clone the Chisel main branch for Spread tests, replacing the temporary development branch workaround. Issue closed as fixed. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed as resolved after upstream canonical/chisel#36 merged, enabling the switch from the temporary development branch back to Chisel's main branch for Spread tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed after Chisel PR #36 merged, allowing the switch from the temporary development branch back to the main branch for Spread tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved and closed after upstream canonical/chisel#36 merged, eliminating the need for a temporary development branch. The codebase now uses Chisel's main branch for Spread tests. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Switching Spread tests to Chisel's main branch was completed after PR #36 merged. The codebase now uses the main branch, and the request was closed as resolved. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
chisel#36
(confidence 99%)
Merging this Chisel fix removed the blocker that required Rockcraft's temporary development branch.
-
Caused By:
canonical/rockcraft#153
(confidence 98%)
This PR introduced the temporary Chisel development-branch workaround cited in #162.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #456 fix(spread): tests must also use newer Chisel | rockcraft | merged | 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. | |
| #1304 test(spread): fix cross-test contamination | rockcraft | merged | Merged PR #1304: spread variants shared the python-from-chisel-bare dir, causing cross-test contamination; each test now copies a clean rockcraft.yaml template in first (+6 lines, 2 files). | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #1267 docs: Remove Chisel-related duplicated content | rockcraft | merged | Merged as commit 888acff0: removed outdated, duplicated Chisel how-to docs from Rockcraft, deferring to Chisel's own docs; explanation page now covers Chisel's relation to Rockcraft, and a flaky custom-slice Spread test was dropped. | |
| #84 Add chisel to rockcraft | rockcraft | merged | Merged PR #84 as commit ddf018ce, adding Chisel to Rockcraft’s snap and enabling Chisel slices in stage-packages. |