lxd: use SUDO_UID for ID mapping
Metadata
Current evaluation
Merged PR #1588 as commit 695794f4c, changing LXD ID mapping to use SUDO_UID and fixing Launchpad bug 1719996.
Resolution: closed resolved
Reason: PR #1588 was merged as commit 695794f4c. The change updates LXD ID mapping to use SUDO_UID, resolving the linked bug.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1588 as commit 695794f4c, changing LXD ID mapping to use SUDO_UID and fixing Launchpad bug 1719996. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 695794f4c, fixing LXD ID mapping to use SUDO_UID for the invoking user and addressing Launchpad bug 1719996. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1588 as commit 695794f4c, updating LXD ID mapping to use SUDO_UID and fix Launchpad bug 1719996. | |
| 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. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1385 lxd: Don't assume user ID to 1000 for raw.idmap | snapcraft | merged | Merged PR #1385 as commit e85e0779b, removing LXD raw.idmap's hard-coded UID 1000 assumption for Launchpad bug 1700703; PR #1904 later corrected mapping order. | |
| #1629 lxd: fix the unit test for the user id map | snapcraft | merged | Merged as commit e820e119e (#1629), fixing the LXD user ID-map unit test under root execution after #1588 changed ID mapping to use SUDO_UID. | |
| #780 fix(lxd): set uid map and gid map separately | craft-providers | merged | Merged PR #780 as commit 5717e2c, fixing #769 by configuring LXD UID and GID maps separately; approved by two reviewers. | |
| #98 lxd: pass id for host mount uid mapping | craft-providers | merged | Merged #98 added an application-provided ID for LXD host-mount UID mapping while retaining the running user's UID as the default, resolving #97. | |
| #18 providers: fix id mapping in lxd project mount | rockcraft | merged | Merged PR #18 fixed LXD project mounts to map the host directory owner rather than the current UID, preventing file-creation failures when their IDs differ. | |
| #769 LXD idmap improperly set for high uids and gids | craft-providers | closed | Fixed by merged PR #98 (76a2d3a): LXD raw.idmap now maps the host UID and GID separately, correcting ownership for high, mismatched IDs. | |
| #97 Filesystem uid mapping incorrect in lxd instance | craft-providers | closed | LXD host-mount ID mapping was fixed by #98 (05d4834), then refined by merged #780 (5717e2c), which sets UID and GID maps separately. |