lxd: use SUDO_UID for ID mapping
Metadata
Current evaluation
Merged a one-file change implementing SUDO_UID for LXD ID mapping, resolving bug 1719996. The pull request passed CI, received approval from one reviewer with no unresolved comments, and was successfully integrated.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] 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)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Fixes: [bug 1719996](https://bugs.launchpad.net/snapcraft/+bug/1719996)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-file change implementing SUDO_UID for LXD ID mapping, resolving bug 1719996. The pull request passed CI, received approval from one reviewer with no unresolved comments, and was successfully integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates LXD ID mapping to use SUDO_UID, resolving bug 1719996. The change ensures correct user ID translation when snapcraft runs under sudo. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Resolves bug 1719996 by using SUDO_UID for LXD ID mapping. Authored by external contributor kalikiana, the change passed static and unit tests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1629 lxd: fix the unit test for the user id map | snapcraft | merged | Merged a fix for an LXD unit test failure when run as root, introduced in #1588. Adjusts the user ID map test logic. Approved by two reviewers and passed CI. One remaining test error was noted but did not block the merge. | |
| #1385 lxd: Don't assume user ID to 1000 for raw.idmap | snapcraft | merged | Merged fix for LXD raw.idmap that incorrectly assumed user ID 1000. Resolves bug 1700703. Approved by reviewers and validated across environments. | |
| #18 providers: fix id mapping in lxd project mount | rockcraft | merged | Merged fix for LXD project mount ID mapping. The change maps the directory owner instead of the current user's UID, preventing file creation failures when IDs differ. This resolves OCI image generation failures in GCE environments. | |
| #1904 lxd: raw.idmap expects host and container id respectively | snapcraft | merged | Merged a fix correcting raw.idmap syntax in LXD configuration. The change maps host and container IDs correctly, preventing container files from being owned by root. Resolves LP: #1741753 with a four-line adjustment across two files. |