sequencer: improve dirty state report message (CRAFT-302)
Metadata
Current evaluation
Merged in craft-parts#83 via 16b77c4b, clarifying dirty-state reports to include the changed dependency step name.
Resolution: closed resolved
Reason: PR craft-parts#83 was merged as commit 16b77c4b. It updates DirtyReport.reason() to name the dependency step and owning part when that dependency is the sole cause.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Clarify the message to include the dependency step name in the reason
why the step is being re-executed.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in craft-parts#83 via 16b77c4b, clarifying dirty-state reports to include the changed dependency step name. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #83 via 16b77c4b, adding the dependency step name to sequencer dirty-state reports so re-execution reasons identify the triggering step (CRAFT-302). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval by two reviewers and passing CI. The change improves the sequencer dirty state report message by including the dependency step name in re-execution reasons, modifying four files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged improvements to the sequencer dirty state report, clarifying the re-execution reason by including the dependency step name. Resolves CRAFT-302. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #192 state_manager: cache dirty reports to reduce recursion | craft-parts | merged | Merged in canonical/craft-parts#192 as commit 88f1090b, caching dirty reports to reduce recursive checks for large dependency chains. | |
| #23 states: add dirty step verification | craft-parts | merged | Merged dirty-step verification: steps rerun when state properties/options change or dependencies require execution (craft-parts#23, ca487581). |