chore(remotebuild): use new build planner
Metadata
Current evaluation
Merged #5390 (cc63a88b8), migrating remote builds to get_launchpad_build_plan() from #5368 and removing remote-build test xfails.
Resolution: closed resolved
Reason: PR #5390 was approved and merged as commit cc63a88b8. It integrated the new build planner added by #5368 and removed the corresponding unit-test xfails.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
* Uses the new `get_launchpad_build_plan()` added in #5368.
* Removes `xfails` from the remote build unit tests.
Passing remote build spread tests: https://github.com/canonical/snapcraft/actions/runs/14386834165/job/40345445458
(SNAPCRAFT-310)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #5390 (cc63a88b8), migrating remote builds to get_launchpad_build_plan() from #5368 and removing remote-build test xfails. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5390 (commit cc63a88b8), switching remote builds to the new Launchpad build planner and removing remote-build test xfails. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #5390 (commit cc63a88b8), updating remote builds to use get_launchpad_build_plan() from #5368 and removing remote-build test xfails. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged remote build updates to adopt the new get_launchpad_build_plan() and remove unit test xfails. Approved by two reviewers, passed CI checks, and integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates remote build to use the new get_launchpad_build_plan() function from #5368 and removes outdated xfails from unit tests. All spread tests pass. Resolves SNAPCRAFT-310. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#5368
(confidence 100%)
Merged as 01185c3bf; added the BuildPlan service and get_launchpad_build_plan() adopted by #5390.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #6010 chore!: remove remote-build service override | snapcraft | merged | Merged #6010 in commit 8bd9cc522, removing the legacy RemoteBuild service override and manual spread tests overlooked by #5994. |