feat: add hooks to the remote-build command
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Different application recipe specs have different requirements for building, with many potential extra arguments and validation parsing that the general command currently cannot support. This PR adds two likely common hooks for applications to override as needed.
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
- [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged implementation adding two configurable hooks to the remote-build command. Allows applications to override build requirements, handle extra arguments, and perform custom validation parsing. Integrated by maintainer bepri. |
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. | |
| #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. | |
| #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. | |
| #196 feat: remote build service | craft-application | merged | Merged the remote build service feature, migrating code from snapcraft. Approved by two reviewers, passed CI checks, and introduced 2316 lines across 26 files. | |
| #2642 remote-build: detect early build errors | snapcraft | merged | Merged. Improved remote build error detection to distinguish between pending builders and actual failures like missing config or timeouts. Approved by reviewer, passed CI, and modified 3 files (+131/-47 lines). Minor import fix applied. | |
| #4432 fix(remote-build): improve logging and catch exceptions | snapcraft | merged | Merged pull request improving remote build logging and exception handling. Approved by reviewers, passed CI checks, and resolves issue #4399. | |
| #4395 feat(remote-build): new command logic | snapcraft | merged | Merged into main. Introduces a RemoteBuild class to orchestrate remote builds, adds local project directory caching, and implements post-build cache cleanup. Resolves #4324. Approved by three reviewers with passing CI and 97.88% diff coverage. | |
| #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. | |
| #1531 feat: add remote build command | charmcraft | closed | Abandoned. The author closed the pull request to create a new one for the rebased remote-build feature. This version remained open with unresolved review comments and pending external dependencies. | |
| #1441 feat: implement mid-step callbacks | craft-parts | merged | Merged after approval by two reviewers and passing CI checks. The change generalizes pre- and post-step hooks to enable mid-step callbacks and adds a pre-organize hook point for the build step. |