← Back to issue list

lxd instance: drop sudo usage (CRAFT-289)

View original Github issue

Metadata

Project
craft-providers
Number
#54
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #54 via fa985d0, removing sudo from LXD instance operations so builds no longer require sudo in buildd images.

Resolution: closed resolved

Reason: PR #54 was merged in commit 386c44c, with implementation commit fa985d0 removing the sudo dependency from LXD instance command handling. Follow-up PR #61 later improved related environment handling.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This will remove the sudo dependency for LXD, which is not in the LXD buildd images. As using sudo populates the processes with /etc/environment, a subsequent PR will will improve environment handling to ensure consistency across providers. Signed-off-by: Chris Patterson <chris.patterson@canonical.com> - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #54 via fa985d0, removing sudo from LXD instance operations so builds no longer require sudo in buildd images.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR dropping `sudo` usage for LXD instances to resolve missing dependency in buildd images. Environment handling improvements planned for a follow-up.

Update history

Date Change
created

Related work

  • Related To: craft-providers#61 (confidence 98%)

    Merged follow-up implementing CRAFT-309 environment consistency improvements referenced in PR #54.

Related issues

Issue Project State Summary Similarity
#69 lxd: ensure user is permitted to use lxd (CRAFT-374) craft-providers merged Merged PR #69 added LXD permission and readiness checks, resolving CRAFT-374 in commit 5478f63.
71%
#3150 cli: error/warn when using sudo snapcraft merged Merged PR #3150 as commit 84891da30, adding sudo errors for LXD/Multipass/remote-build, a warning for host cleanup, and sudo test-environment updates to reduce Sentry permission-error reports.
71%