ci: fix spread test missing PATH
Metadata
Current evaluation
Merged PR #1083 as commit 13c96b83, fixing the missing PATH in Spread tests; configure and remove tests then passed.
Resolution: closed resolved
Reason: PR #1083 was merged as commit 13c96b83. Its author confirmed that the configure and remove Spread tests passed after the PATH fix.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1083 as commit 13c96b83, fixing the missing PATH in Spread tests; configure and remove tests then passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/charmcraft#1083 to fix a missing PATH in CI spread tests; the configure and remove spread tests then passed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix for a CI spread test missing the PATH variable. The change updated four files and received reviewer approval. Post-merge validation confirmed the spread tests for configure and remove passed successfully. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for CI spread test failure caused by a missing PATH variable. The maintainer confirmed the spread tests for configure and remove passed after the change. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3662 tests: update spread url | snapcraft | merged | Merged PR #3662 updated Spread tarball URLs in two test files via commit 507959d70. | |
| #5786 ci: run spread on push to main/hotfix | snapcraft | merged | Merged PR #5786 as commit db2b03097, adding Spread test runs for pushes to main and hotfix/** branches. | |
| #573 ci: run spread tests on 'spread-installed' runners | rockcraft | merged | Merged PR #573 as commit c8ff5823, updating CI to run Spread tests on runners labeled "spread-installed." | |
| #701 fix(TestingService): error better when we can't find spread | craft-application | merged | Merged PR #701 as bb2bda9c, improving the TestingService error when spread is missing from PATH; fixes #697. | |
| #127 Fix spread tests | imagecraft | merged | Fixed and merged as commit cd43de1, updating 11 files (+49/-38); all listed CI checks, including spread, passed. | |
| #1133 tests: fix the test that was modifying the environment variables | snapcraft | merged | Merged PR #1133 (commit c0b0f9752), fixing a test that leaked PATH/environment-variable changes and guarding against an empty PATH. | |
| #3663 tests: update spread url | snapcraft | merged | Merged via commit bcea80568, updating two Spread test tarball URLs (+2/-2 lines). | |
| #6350 test: fix spread tests | snapcraft | merged | Merged PR #6350 as a18c7f485, fixing all non-store Spread tests and OSV scanning, including pinning Black 26.3.1 to avoid patchelf failures. | |
| #1091 ci: use openstack for spread | craft-application | merged | Merged PR #1091 in commit 826edb3f, switching Spread CI to OpenStack and fixing #1096. | |
| #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). |