fix(lxd): set uid map and gid map separately
Metadata
Current evaluation
Merged PR #780 as commit 5717e2c, fixing #769 by configuring LXD UID and GID maps separately; approved by two reviewers.
Resolution: closed resolved
Reason: PR #780 was merged as commit 5717e2c and explicitly states that it fixes #769. The change correctly sets the LXD UID map and GID map separately, resolving high-ID mapping.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #769
A Snapcraft build to test this is available from https://github.com/canonical/snapcraft/pull/5641
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #780 as commit 5717e2c, fixing #769 by configuring LXD UID and GID maps separately; approved by two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged fix to set LXD uid map and gid map separately, resolving #769 in craft-providers where high uids/gids were improperly mapped. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#769
(confidence 100%)
The PR explicitly identifies #769 as the issue it fixes.
-
Related To:
canonical/snapcraft#5641
(confidence 90%)
The PR links this Snapcraft build for testing the LXD fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1588 lxd: use SUDO_UID for ID mapping | snapcraft | merged | Merged PR #1588 as commit 695794f4c, changing LXD ID mapping to use SUDO_UID and fixing Launchpad bug 1719996. | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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. |