feat: remote build service
Metadata
Current evaluation
Merged the remote build service feature, migrating code from snapcraft. Approved by two reviewers, passed CI checks, and introduced 2316 lines across 26 files.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Recommended review technique:
0. Everything before the "bring remote-build code from snapcraft" is https://github.com/canonical/craft-application/pull/194
1. Start at `feat(remote): bring remote-build code from snapcraft`. This is a copy of the newest remote build code from snapcraft.
2. Go commit by commit
Sorry this is so big! I tried my best to break it into reasonably reviewable chunks, but I still ended up with a "draw the rest of the owl" PR
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged the remote build service feature, migrating code from snapcraft. Approved by two reviewers, passed CI checks, and introduced 2316 lines across 26 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the remote build service feature, integrating code from snapcraft and upstream changes from craft-application PR #194. The maintainer successfully implemented the functionality after passing standard checks. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #769 feat: add the remote-build command | rockcraft | merged | Merged the remote-build command. Approved by three reviewers, the change adds 86 lines and removes 7 across nine files. Integrated by the maintainer without further comments or CI issues. | |
| #1546 feat: remote build command | charmcraft | merged | Merged the remote build command feature. Approved by two reviewers and passing CI, the PR rebases prior work and adds remote build functionality. Follow-up issues were created to resolve related build-id recovery and dependency bugs. | |
| #636 feat: add a "pending" status to remote builds | craft-application | merged | Merged the feature adding a pending status to remote builds. The change upstreams code from canonical/snapcraft#4956, received two approvals, and modified four files with 43 additions and 15 deletions. | |
| #299 fix(remote-build): minor edge cases found with snapcraft | craft-application | merged | Fixed minor snapcraft edge cases in remote builds. Merged following approval by two reviewers and passing CI checks. The update modifies three files, adding 43 lines and removing 13. | |
| #3648 cli: reintroduce remote-build and promote to snapcraft help | snapcraft | merged | Merged the reintroduction of the remote-build CLI command and promoted it to snapcraft help. Approved by one reviewer, passed CI checks, and resolved Launchpad bug #1943244 via a single-file, four-line deletion. | |
| #586 feat: add a 'remote-build' command | craft-application | merged | Merged a 'remote-build' command adapted from Charmcraft to be craft-tool-agnostic. Approved by three reviewers and implemented across six files with 331 additions. | |
| #2765 remote-build: initial Windows support | snapcraft | merged | Merged initial Windows support for remote-build. Approved by one reviewer, adding 72 lines and removing 26 across six files. Requires snapcraft PR #2759. | |
| #4382 tests: use less mocks in remote-build | snapcraft | merged | Merged changes replace mocked snapcraft.yaml loading in remote-build tests with direct file creation. Approved by two reviewers, passed CI, and serves as preparatory work for issue #4322. | |
| #2500 cli: add remote build | snapcraft | merged | Merged. Adds remote build capability to Launchpad, dispatching source trees for remote compilation and retrieving resulting snap packages or logs. Approved by reviewers, passed CI, and integrated despite a minor test coverage decrease. | |
| #632 feat: add hooks to the remote-build command | craft-application | merged | Merged. Implemented two configurable hooks for the remote-build command to support application-specific build arguments and validation overrides. Approved by two reviewers, passed CI checks, and updated the changelog. |