← Back to issue list

Move the build planner logic into craft-platforms

View original Github issue

Metadata

Project
rockcraft
Number
#639
Type
issue
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Build planner logic extracted to craft-platforms in merged PR canonical/craft-platforms#25 (commit d9ff004); rockcraft later simplified in commit 5f898b32. Maintainer mr-cal confirmed completion.

Resolution: closed resolved

Reason: The rock build planner was ported into craft-platforms by PR canonical/craft-platforms#25 ("feat: add rock build planner", commit d9ff004), and rockcraft followed up with commit 5f898b32 "refactor: use default build planner". Maintainer mr-cal closed the issue citing the craft-platforms PR and jugmac00 confirmed.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done Take the core logic from the build planner and move it into craft-platforms ### Why it needs to get done @jugmac00 requested it :-)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Build planner logic extracted to craft-platforms in merged PR canonical/craft-platforms#25 (commit d9ff004); rockcraft later simplified in commit 5f898b32. Maintainer mr-cal confirmed completion.
stealth/space-bunny-alpha — closed resolved Build planner logic was moved out of rockcraft into craft-platforms via merged PR canonical/craft-platforms#25 (commit d9ff004). MR-CAL closed the issue citing that PR; jugmac00 confirmed completion.
stealth/space-bunny-alpha — closed resolved Build planner logic was moved into craft-platforms by merged PR canonical/craft-platforms#25; the maintainer later confirmed completion.
qwen/qwen3.6-35b-a3b — — Closed without implementation or linked pull request. The migration of build planner logic to craft-platforms appears abandoned or superseded, with no further activity or resolution documented.
qwen/qwen3.8-27b
Impact: 35
Quick Win: 19
Staleness: 85
Complexity: 45
Confidence: 85
Support Request: 0
close stale Refactor to move build planner logic into craft-platforms, requested by jugmac00. Work completed: craft-application#391 closed as resolved, rockcraft PR #715 and commit 5f898b32 implemented the change. No BuildPlanner class remains in codebase.
qwen/qwen3.6-35b-a3b
Staleness: 98
Complexity: 25
Confidence: 90
Support Request: 5
close stale Request to migrate build planner core logic to craft-platforms. Created over two years ago, it has zero comments, no labels, and no activity. Currently untriaged and effectively abandoned.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 40
Confidence: 75
Support Request: 5
needs triage Proposal to refactor and move core build planner logic into the craft-platforms library. The issue remains open with no maintainer response, labels, or activity for over two years.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 40
Confidence: 85
Support Request: 10
close stale Task to move build planner logic into craft-platforms. Unlabelled, zero comments, and inactive for over two years with no maintainer engagement.

Update history

Date Change
updated
closed

Related work

  • Likely Fixed By: canonical/craft-platforms#25 (confidence 95%)

    Merged PR that added the rock build planner to craft-platforms, cited in the closing comment.

  • Related To: rockcraft#5f898b32 (confidence 70%)

    rockcraft commit 'refactor: use default build planner' consuming the upstream planner.

  • Related To: canonical/craft-platforms#148 (confidence 45%)

    Later follow-up adding a bases charm build planner, same planner framework.

Related issues

Issue Project State Summary Similarity
#698 Use craft-platforms for build plans rockcraft closed Merged PR #715 (516d29e5) replaced Rockcraft's custom build-plan generation with craft-platforms, resolving #698 and aligning plans with downstream consumers.
81%
#15 Implement a rock build planner craft-platforms closed Rock build planning was implemented and merged in canonical/craft-platforms#25 via commit d9ff004.
78%
#391 Use craft-platforms for build plans craft-application closed Build planning and filtering moved to craft-platforms in merged PR #25 (d9ff004); craft-application adopted the planner through merged PR #666 (621d1a32).
78%
#715 refactor: use craft-platform for the build plans rockcraft merged Merged PR #715 (516d29e5) replaced Rockcraft’s custom build-plan generation with craft-platform, creating one shared source for Rockcraft and remote builds; fixes #698.
78%
#1876 Use craft-platforms for build plans charmcraft closed Closed as done: charmcraft PR #1894 (commit f1755e8f) switched build plan generation to craft-platforms via craft-application's BuildPlanService; rockcraft did the same in #715.
76%
#662 [PROTOTYPE] refactor: use craft-platforms rockcraft closed Prototype PR #662 was closed unmerged in August 2024; the production build-planner migration later landed via rockcraft#715 at commit 516d29e5.
75%
#633 fix: improve `platforms.<platform>.build-on` validation rockcraft closed Closed unmerged as redundant: PR canonical/rockcraft#567 had already improved build-on validation by adopting craft-application's default BuildPlanner.
74%
#5015 Use craft-platforms for build plans snapcraft closed Snapcraft adopted craft-platforms build plans in merged PR #5111 (commit 7969ee8d5), replacing its custom planning logic.
74%
#1894 refactor: use craft-platforms for build plans charmcraft merged Merged PR #1894 (commit f1755e8f, +25/-33) replaced Charmcraft's custom build-plan generation for `platforms` charms with craft-platforms, routing exhaustive build plans through craft-application's BuildPlanService.
73%
#403 [PROTOTYPE] refactor: use craft-platforms craft-application closed Closed unmerged as a prototype; maintainer redirected the craft-platforms build-planner integration to #391, with the build-plan service later landing via #666 (621d1a32).
73%