Use Chisel `main` branch for Spread tests
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed after Chisel PR 36 merged, allowing a switch from a temporary development branch back to the Chisel main branch for Spread tests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #456 fix(spread): tests must also use newer Chisel | rockcraft | merged | 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. | |
| #467 fix(spread): tests must also use newer Chisel (#456) (#466) | rockcraft | closed | Merged and closed as a cherry-pick from main to update test dependencies to a newer Chisel version for the 1.1.2 release. | |
| #469 fix(spread): tests must also use newer Chisel (#456) (#466) | rockcraft | closed | Merged via cherry-pick into main to prepare the 1.1.2 release. Updates spread tests to use the newer Chisel version. Passed CI checks and required no reviews. | |
| #466 fix(spread): tests must also use newer Chisel (#456) | rockcraft | merged | Merged a cherry-picked commit from main to align tests with the newer Chisel version, facilitating the 1.1.2 release. Approved by one reviewer. | |
| #155 spread: fix project with dotnet chisel slices | rockcraft | merged | Merged to remove an obsolete prime override and symlink in the spread configuration, as dotnet packages no longer create version-specific subdirectories. The change resolved CI failures blocking other pull requests. | |
| #152 spread: disable general/chisel | rockcraft | closed | Closed as superseded. Upstream chisel-releases fixed the dotnet bug breaking the chisel cut operation, making the temporary test disable unnecessary. Configuration updates were moved to a separate pull request. | |
| #4795 tests: update chisel-base test to use core24 | snapcraft | merged | Merged a single-line test update to switch the chisel-base spread test to core24, aligning with the devel branch targeting Ubuntu 24.10. Changes were approved and passed CI. |