fix(TestingService): provide a better error if spread.yaml is missing
Metadata
Current evaluation
Merged update that improves error messaging when spread.yaml is missing in the TestingService. Approved by two reviewers, passed CI checks, and includes a changelog entry. Resolves #703.
Suggested action: —
No scores available.
Issue body
Fixes #703
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
- [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged update that improves error messaging when spread.yaml is missing in the TestingService. Approved by two reviewers, passed CI checks, and includes a changelog entry. Resolves #703. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged improvements to TestingService error handling when spread.yaml is missing, resolving issue #703. Changes passed linting and testing, and were documented in the changelog. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #701 fix(TestingService): error better when we can't find spread | craft-application | merged | Merged improvement to TestingService error handling when spreads are missing, resolving #697. Approved by two reviewers and passed CI checks. | |
| #1029 fix(testing): error early if spread.yaml is invalid. | craft-application | merged | Merged after two approvals. Adds early validation to fail tests immediately if spread.yaml is invalid. CI passed except for an unrelated snap-test failure, which was overridden. | |
| #723 fix(testing): more intelligent test handling | craft-application | merged | Merged after approval. Updates test handling to run spread tests as root and strictly parse top-level spread.yaml keys. Approved by two reviewers and passed CI checks before integration. | |
| #3640 spread: update error when local snap is missing | snapcraft | merged | Merged a one-line change to improve error reporting when a local snap is missing during spread updates. Approved by a reviewer, passed CI, and integrated into the repository. | |
| #5280 test: fix spread.yaml multipass backend | snapcraft | merged | Merged fix for the spread.yaml multipass backend that prevented automatic image upgrades, forcing it to use only explicitly requested task.yaml images. Approved by reviewers and passed CI. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged a workaround for a flaky spread test (issue #533). The change adjusts one file (+2/-1 lines) to skip a no-op name query check while retaining other validations. Approved by two reviewers and passed all CI checks. | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #934 fix(templates): typo in spread .extension | rockcraft | merged | Merged a single-line correction for a typo in the spread .extension template. The fix passed all CI checks and received approval from two reviewers before integration. |