← Back to issue list

remote-build: architecture handling

View original Github issue

Metadata

Project
snapcraft
Number
#2774
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2774 via commit 8300281d6, cleaning up and fixing remote-build architecture handling. Missing CLI unit tests were acknowledged as follow-up work.

Resolution: closed resolved

Reason: PR #2774 was approved and merged as commit 8300281d6, which fixed and cleaned up remote-build architecture handling.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2774 via commit 8300281d6, cleaning up and fixing remote-build architecture handling. Missing CLI unit tests were acknowledged as follow-up work.
stealth/space-bunny-alpha — closed resolved Merged remote-build architecture handling and cleanup in commit 8300281d6. The maintainer noted missing CLI unit tests and planned follow-up coverage.
qwen/qwen3.6-35b-a3b — — Merged architecture handling updates for remote-build. Modified one file with 36 additions and 30 deletions. Approved by one reviewer. Author noted missing CLI unit tests and plans a follow-up PR to address them.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Implements architecture handling for remote-build. Author acknowledged missing CLI unit tests and committed to a follow-up PR to add coverage once current changes stabilize.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4495 remote build: odd errors for certain `architectures:` setups snapcraft closed Remote-build architecture handling was improved; #4994 (5cd480725) fixed CLI validation and #5005 (ad3b54bbd) rejected unsupported duplicate build-on artifacts.
78%
#4324 remote-build: new command logic snapcraft closed Remote-build logic was replaced by merged PR #4395 (commit 2e0647594), adding isolated per-build cache worktrees, cleanup, and generated build IDs.
75%
#2785 remote-build: add initial command unit tests snapcraft merged PR #2785 by cjp256 merged into snapcraft master, adding initial unit tests for the remote-build command covering build-on arch handling; reviewer comments were addressed and the branch rebased.
74%
#2042167 remote-build doesn't support shorthand architecture format snapcraft (launchpad) closed Remote-build now handles shorthand architectures consistently; PR #4793 fixed the underlying bug in commit 53ec55cef.
73%
#4793 fix(remote-build): don't fail with certain archs snapcraft merged Merged into the hotfix branch as commit 53ec55cef, fixing architecture-specific remote-build failures from #4780, with companion support in craft-application#343.
72%
#4842 remote build: document nuances of current and legacy remote builders snapcraft closed Remote-builder nuances were documented by PR #4882 (commit bd49f8db0), completing the documentation subtask split from #4842.
72%
#4995 remote build: error if multiple artefacts can be built on the same architecture snapcraft closed Fixed by merged PR #5005 (commit ad3b54bbd): remote builds now fail early when multiple artifacts share a build-on architecture.
72%
#4384 refactor(remote-build): reorganize functions snapcraft merged Merged #4384 via commit 23e2601c9, reorganizing remote-build functions in preparation for #4322; follow-up #4385 added build-ID generation.
72%
#4780 remote build: fails with some architecture definitions snapcraft closed Remote-build architecture handling was fixed by merged PR canonical/snapcraft#4793 (commit 53ec55cef), with companion support in canonical/craft-application#343.
72%
#4660 refactor(test): versioned remote-build tests snapcraft merged Merged PR #4660 (commit 2423be37e), refactoring versioned remote-build tests without logic changes.
71%