lxd: pass original CLI arguments down to container
Metadata
Current evaluation
Changes to pass original CLI arguments to LXD containers were closed without merging. The fix for architecture defaults and ignored flags was abandoned as an alternative to #1493 after a discussion on sys.argv handling.
Suggested action: —
No scores available.
Issue body
Originally trying to fix the --target-arch default as [discussed in the forum](https://forum.snapcraft.io/t/cleanbuild-remote-on-pi-grabs-wrong-arch-lxc-image/691/7) so container builds default to the architecture of the container rather than the client, I followed up [on a suggestion](https://github.com/snapcore/snapcraft/pull/1493) to pass the original commands instead of picking options we need by hand.
This also addresses [`--no-parallel-build` being ignored when building in a container](https://forum.snapcraft.io/t/cleanbuild-no-parallel-build-exclusive/614/4).
**Alternative approach to #1493**.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Changes to pass original CLI arguments to LXD containers were closed without merging. The fix for architecture defaults and ignored flags was abandoned as an alternative to #1493 after a discussion on sys.argv handling. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without merging. The PR proposed passing original CLI arguments to LXD containers to fix architecture and parallel build flags, but the approach was rejected after discussing sys.argv compatibility. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without merging. The change aimed to pass original CLI arguments to LXD containers to fix architecture and parallel build flags. Discussion centered on sys.argv versus manual parsing, but no consensus was reached and the change was abandoned. |
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 after two approvals and CI. It fixes container build architecture handling by only passing the target arch when explicitly specified, defaulting to the container's arch. PR #1498 context and description clarity were resolved before merge. |