feat: improve error message for checksum mismatch
Metadata
Current evaluation
PR #1683 by cmatsuoka merged (commit 4efd5bd0): checksum mismatch errors now put actual/expected digests in details, add a resolution entry, and name the failing part. Changelog entry added; closes issue #355.
Resolution: closed resolved
Reason: The PR itself was merged into main as commit 4efd5bd0 with 2 approvals and all CI checks passing. It implements the improved checksum error message and closes the referenced issue #355 ('Source checksum mismatch is confusing'), which is now closed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Add a better checksum error message, move actual and expected digest
to details, provide a resolution entry, and show the name of the part
causing the problem. This provides better UX in case of checksum
mismatches and makes it clear where the error originates from.
Fixes #355
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | PR #1683 by cmatsuoka merged (commit 4efd5bd0): checksum mismatch errors now put actual/expected digests in details, add a resolution entry, and name the failing part. Changelog entry added; closes issue #355. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1683 (commit 4efd5bd0) by cmatsuoka improves checksum mismatch errors: expected/actual digests moved to details, resolution added, and the failing part name included. Changelog entry added; closes #355. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1683 as 4efd5bd0, improving checksum mismatch errors with the failing part, digest details, and resolution guidance; fixes #355. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged improvements to checksum mismatch error messages, adding actual and expected digests to details, a resolution entry, and the problematic part name. Approved by two reviewers with all CI checks passing. Resolves #355. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
95
|
needs review | Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Approved by 2 reviewers, all CI checks passing, ready for maintainer merge. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
35
Confidence:
90
|
needs review | Improves checksum mismatch error messages by including actual and expected digests, resolution steps, and the problematic part name. Currently approved by one reviewer, CI passing, and ready for final merge. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Improves checksum mismatch error messages by adding expected/actual digests, resolution steps, and part names. Currently approved by one reviewer, CI passing, awaiting final maintainer review. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Approved by 1 reviewer, CI passing, but lint/test and changelog checkboxes remain unchecked. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
5
Complexity:
20
Confidence:
85
|
needs review | Improves checksum mismatch error messages by adding digests, resolution steps, and part names. Currently awaiting initial maintainer review with CI checks passing or pending. Fixes #355. |
Update history
| Date | Change |
|---|---|
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related work
-
Likely Fixed By:
canonical/craft-parts#355
(confidence 95%)
Issue #355 'Source checksum mismatch is confusing' was closed by this PR, as stated in its body ('Fixes #355').
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #355 Source checksum mismatch is confusing | craft-parts | closed | Closed after checksum mismatch errors gained clearer part/source context in PR #1683, merged as commit 4efd5bd0. |