lxd: proper filename set when using architectures
Metadata
Current evaluation
Merged into master, fixing LP #1789882 by ensuring proper filename handling when using architectures in LXD. Approved by reviewer, passed CI checks, and modified 3 files (+14/-6 lines). Code coverage impact was negligible.
Suggested action: —
No scores available.
Issue body
LP: #1789882
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into master, fixing LP #1789882 by ensuring proper filename handling when using architectures in LXD. Approved by reviewer, passed CI checks, and modified 3 files (+14/-6 lines). Code coverage impact was negligible. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Resolves Launchpad bug #1789882 by correctly setting filenames when using architectures in LXD. Code coverage decreased by less than 0.01%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Resolves Launchpad bug #1789882 by ensuring LXD generates the correct filename when architectures are specified. Code coverage decreased by less than 0.01% upon integration. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #552 Make the lxd unit test pass in architectures other than amd64. | snapcraft | merged | Merged a +3/-1 change to fix lxd unit tests for non-amd64 architectures. Resolved LP #1589747 after a retest request. | |
| #1286 lxd: Setup image and target arch for cross-compilation | snapcraft | merged | Merged into master. Configures LXD container image and target architecture for cross-compilation, using the host architecture image to avoid slow QEMU emulation. Approved by two reviewers and passed CI. | |
| #1074 ci: fix lxd image name too long | charmcraft | merged | Merged a fix for a CI configuration error where the LXD image name exceeded the maximum length. Approved by one reviewer, the change modified one file (+3/-2 lines) and was successfully integrated. | |
| #817 lxd: use built-in image streams. | snapcraft | merged | Merged. Replaced custom remote setup for cleanbuilds with LXD built-in image streams, streamlining configuration. Approved by one reviewer with zero unresolved comments. The two-file change (+4/-13 lines) resolves LP #1625787. | |
| #1488 lxd: always remove existing device for project folder | snapcraft | merged | Merged as a follow-up to #1483, ensuring existing devices are always removed for the LXD project folder. Approved by two reviewers with zero unresolved comments. The change modifies three files, adds 64 lines, removes one, and passed all CI checks. |