← Back to issue list

fix: refresh build snaps if already installed

View original Github issue

Metadata

Project
craft-parts
Number
#1692
Type
pull request
State
open
Author
cmatsuoka
Labels
—
Created
Updated
Closed
—

Current evaluation

⚠️ This summary may be outdated — the issue content or evaluation prompts have changed since this evaluation was made.

Refreshes build snaps when already installed to reflect channel changes and triggers rebuilds. Approved by 2 reviewers with 4 unresolved comments and some flaky CI failures.

Suggested action: needs review

Reason: The PR has 2 approvals but 4 unresolved review comments remain, and several CI checks are failing (test-flaky and test-common slow tests, OSV-scanner). The author should address the unresolved review comments and resolve CI failures before merge.

Evaluated by: mr-cal-bot

Impact: 65 Quick Win: 18 Complexity: 55 Confidence: 75 Actionability: 60

Issue body

If a build snap is already installed, refresh it to the current channel to reflect changes in the project. Also add it to the build state so the part will be rebuilt in case of channel changes. Fixes #496 - [ ] Have you followed the guidelines for contributing? - [ ] 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
qwen/qwen3.6-35b-a3b
Impact: 65
Quick Win: 18
Complexity: 55
Confidence: 75
Actionability: 60
needs review Refreshes build snaps when already installed to reflect channel changes and triggers rebuilds. Approved by 2 reviewers with 4 unresolved comments and some flaky CI failures.
stealth/space-bunny-alpha
Impact: 56
Quick Win: 14
Complexity: 52
Confidence: 88
Actionability: 51
needs review Refreshes installed build snaps to their requested channels and records them in build state to trigger rebuilds; approved twice, but one review thread and four CI jobs remain unresolved.
stealth/space-bunny-alpha
Impact: 55
Quick Win: 18
Complexity: 42
Confidence: 87
Actionability: 55
needs review Refreshes installed build snaps to the current channel and records them in build state so channel changes trigger rebuilds; approved, but one review comment and four CI jobs remain unresolved.
qwen3.6-35b-a3b-mtp-q6
Impact: 55
Quick Win: 33
Staleness: 15
Complexity: 40
Confidence: 75
needs review Refreshes build snaps when already installed to reflect channel changes, fixing #496. Has 2 approvals, 1 unresolved review comment, and mostly passing CI.
qwen/qwen3.8-27b
Impact: 50
Quick Win: 20
Staleness: 5
Complexity: 60
Confidence: 75
needs review Fixes build-snaps channel changes not taking effect without a clean by refreshing already-installed build snaps to the current channel and tracking them in build state. Approved by 2 reviewers; 1 unresolved comment, CI pending. Fixes #496.
qwen/qwen3.8-27b
Impact: 55
Quick Win: 30
Staleness: 5
Complexity: 45
Confidence: 80
needs review Fixes craft-parts #496: refresh already-installed build snaps to the configured channel and record them in build state so channel changes trigger rebuilds without clean. 2 approvals, all CI green, 1 unresolved review comment.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and tracking them in build state. Currently approved by two reviewers with CI tests pending.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 45
Confidence: 85
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by two reviewers with one unresolved comment. CI mostly passes except for isolated Java plugin test failures.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 35
Confidence: 95
needs review Fixes #496 by refreshing installed build snaps to the current channel and adding them to build state. Approved by 2 reviewers, CI passing, ready to merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 45
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and tracking them in build state. Approved by two reviewers with zero unresolved comments, but CI is still pending and one lint check is failing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 45
Confidence: 90
needs review Updates build snap handling to refresh already installed snaps to the current channel and tracks them in build state. Approved by two reviewers with all CI checks passing, ready for maintainer merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 40
Confidence: 90
needs review Fixes #496 by refreshing build snaps when already installed to reflect project changes and adding them to build state. Approved by two reviewers, CI passing, ready to merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 45
Confidence: 95
needs review Fixes #496 by refreshing installed build snaps to the current channel and adding them to build state for rebuilds. Approved by two reviewers, CI passing, ready to merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 40
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Awaiting pending CI tests; approved by two reviewers.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 45
Confidence: 95
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by two reviewers, CI passes except for known flaky tests, ready for merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by two reviewers, CI passing except for known flaky tests, ready for merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 95
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by 2 reviewers with 0 unresolved comments. CI shows flaky test failures but is otherwise passing.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 15
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by 2 reviewers, CI mostly passing with known flaky test failures.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and tracking them in build state. Approved by one reviewer, CI passing, ready for merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Approved by one reviewer, CI mostly passing with expected flaky failures, awaiting final merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and tracking them in build state. Currently approved by one reviewer with one unresolved comment and pending slow-test CI jobs.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 90
needs review Fixes #496 by refreshing installed build snaps to the current channel and updating build state. Currently awaiting maintainer review with some flaky and slow tests failing or pending.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 90
needs review Fixes build snap refresh behavior when already installed and updates build state for channel changes. Addresses issue #496. Currently awaiting maintainer review with CI checks pending.

Update history

Date Change
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
created

Related work

  • Related To: craft-parts#496 (confidence 95%)

    This PR directly fixes issue #496 where build-snaps channel changes don't take effect without a clean.

  • Related To: snapcraft#3617 (confidence 70%)

    Related snapcraft issue about not refreshing already installed snaps, possibly overlapping scope.

Related issues

No related issues found above the similarity threshold.