remote-build: refactor onto craft-application's command
Metadata
Current evaluation
Remote-build was refactored onto craft-application's RemoteBuild in merged PR #5239 (commit f31d99498), bringing the requested upstream improvements.
Resolution: closed resolved
Reason: Merged PR #5239 implemented the refactor in commit f31d99498. RemoteBuildCommand now subclasses craft_application.commands.RemoteBuild, directly satisfying the issue's requirement.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
The snapcraft remote-build command should be a child class of `craft_application.commands.remote`
### Why it needs to get done
To bring in remote-build improvements and bug fixes, such as:
* https://github.com/canonical/craft-application/issues/618
* https://github.com/canonical/craft-application/issues/619
* https://github.com/canonical/craft-application/issues/620
* https://github.com/canonical/craft-application/issues/621
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Remote-build was refactored onto craft-application's RemoteBuild in merged PR #5239 (commit f31d99498), bringing the requested upstream improvements. | |
| stealth/space-bunny-alpha | — | closed resolved | Remote-build was refactored onto craft-application's command in merged PR #5239 (commit f31d99498), bringing in the referenced improvements and fixes. | |
| stealth/space-bunny-alpha | — | closed resolved | Remote-build was refactored onto craft-application's command in merged PR #5239 (commit f31d99498), delivering the requested improvements. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The refactoring to inherit from craft_application.commands.remote was abandoned and tracked internally via Jira ticket CRAFT-4044. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The task to refactor snapcraft's remote-build command onto craft-application's command was not implemented, suggesting it was abandoned or superseded. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Refactoring the remote-build command to inherit from craft-application was closed without implementation. No pull request was created, and the request appears abandoned following an automated Jira sync. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/snapcraft#5239
(confidence 100%)
The merged PR introduced the requested craft-application-based remote-build command.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5239 refactor: use craft-application's remote-build | snapcraft | merged | Merged PR #5239 as f31d99498, replacing Snapcraft’s legacy remote-build path with craft-application’s command for CRAFT-4044. | |
| #482 refactor: subclass remote-build errors from CraftError | craft-application | merged | Merged craft-application#482 as be2750d8, making RemoteBuild errors CraftError subclasses, moving details into brief messages, and documenting release 4.2.4. | |
| #586 feat: add a 'remote-build' command | craft-application | merged | Merged PR #586 as commit 6e49d794, adding a remote-build command adapted from Charmcraft and generalized to be craft-tool-agnostic. | |
| #4653 feat(craft-application): support remote build | snapcraft | closed | Closed unmerged; superseded by canonical/snapcraft#4664 (commit 57d184af7), which landed the same craft-application remote build support for core24. | |
| #4664 feat(craft-application): support remote build | snapcraft | merged | PR #4664 by syu-w merged (commit 57d184af7), adding remote build support to craft-application in Snapcraft, including --platform and --build-for handling. Shipped in the 8.0.0 release notes as the new core24 remote-builder. | |
| #4395 feat(remote-build): new command logic | snapcraft | merged | Merged as commit 2e0647594; adds RemoteBuild class and new remote-build command orchestration, per-build project cache in the local cache dir with post-build cleanup, plus typing/lint updates (CRAFT-1979). Fixes #4324. | |
| #4837 build(deps): bump craft-application to 2.8.0 | snapcraft | merged | Merged in commit 00fa0ac74, upgrading craft-application to 2.8.0 for multi-architecture remote builds, Launchpad API retries with backoff, and correct remote-build log names. | |
| #4908 remote build: stop raising internal errors | snapcraft | closed | Remote-build errors now produce actionable messages and documentation links via merged PR #5052 (741e36eb9), updating craft-application to 4.2.4. | |
| #300 Remote-build: use craft-application for core22 | craft-application | closed | Core22 remote builds moved to craft-application in merged Snapcraft PR #4724 (commit 9905f2831), removing the legacy core22 build path. | |
| #4710 Remove core22 remote-builder in favour of craft-application remote-builder | snapcraft | closed | Core22 remote builds now use craft-application’s remote builder via merged PR #4724 (9905f2831); legacy remote-builder code was later removed by PR #5994 (b08f1a66e). |