← Back to issue list

lxd: let lxd choose the architecture

View original Github issue

Metadata

Project
snapcraft
Number
#1718
Type
pull request
State
merged
Author
mwhudson
Labels
—
Created
Updated
Closed

Current evaluation

Merged in #1718 as 834cfce4c, delegating default container architecture selection to LXD while recording the container architecture after startup.

Resolution: closed resolved

Reason: PR #1718 merged as commit 834cfce4c after two approvals. It removes Snapcraft's duplicate architecture-selection logic and obtains the architecture from the started LXD container.

Evaluated by: mr-cal-bot

No scores available.

Issue body

lxd implements exactly the same logic as snapcraft does to choose an architecture when one is not specified, so remove snapcraft's copy. It is still necessary to record the architecture of the created container to detect whether snaps can just be copied across, but this is now done after the container is started, which will still be correct when/if support for building on a non-default image is merged.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in #1718 as 834cfce4c, delegating default container architecture selection to LXD while recording the container architecture after startup.
stealth/space-bunny-alpha — closed resolved Merged in #1718 as 834cfce4c: Snapcraft now lets LXD select the container architecture and records it after startup.
qwen/qwen3.6-35b-a3b — — Merged after two approvals and passing CI. Removes snapcraft's redundant LXD architecture selection logic, delegating it to LXD. Architecture recording now occurs post-container startup. Released with version 2.35.
qwen3.6-35b-a3b-mtp-q6 — — Merged after approvals and a passing CI run. Removes snapcraft's duplicate LXD architecture selection logic, deferring to LXD. Integration was delayed until v2.35 to prioritize autopkgtest fixes, then applied to master.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1493 lxd: only pass target arch if specified explicitly snapcraft merged Merged via commit 0a7de4515, passing --target-arch to LXD only when explicitly set so builds default to the container architecture.
77%
#1498 lxd: pass original CLI arguments down to container snapcraft closed Closed unmerged in favor of #1493, which forwarded --target-arch only when explicitly set; commit 0a7de4515 preserved the container architecture default.
77%
#1439 target arch default for containers snapcraft closed Fixed by merged PR #1493 (commit 0a7de4515): LXD now defaults to the container architecture. Competing PR #1498 was closed unmerged.
76%
#2046 lxd: specify arch in lxc image list command snapcraft merged Merged PR #2046 fixed incorrect LXD image metadata by specifying the architecture in lxc image list; merged as commit 828ea5120.
71%