← Back to issue list

lxd integration tests run`sudo lxd init`

View original Github issue

Metadata

Project
craft-providers
Number
#222
Type
issue
State
closed
Author
mr-cal
Labels
good first issue
Created
Updated
Closed

Current evaluation

LXD integration tests requiring sudo were marked and skipped by default, preventing routine runs of `sudo lxd init`; resolved by merged PR #328.

Resolution: closed resolved

Reason: Merged PR #328 added a sudo marker for integration tests requiring sudo and passed the CI environment variable through tox, so sudo-dependent tests no longer run as ordinary tests.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The lxd integration tests call `LXD().init()`, which tries to initialize lxd with `sudo lxd init`: https://github.com/canonical/craft-providers/blob/35bd3c49f3860a74f7f1f424873908581654d968/tests/unit/lxd/test_lxd.py#L32 This call should be removed or mocked out. Reported in https://github.com/canonical/craft-providers/issues/131

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved LXD integration tests requiring sudo were marked and skipped by default, preventing routine runs of `sudo lxd init`; resolved by merged PR #328.
qwen/qwen3.6-35b-a3b — closed stale Issue reported that LXD integration tests call `sudo lxd init` and should be mocked or removed. Closed after inactivity without a linked fix.

Update history

Date Change
created

Related work

  • Likely Fixed By: canonical/craft-providers#328 (confidence 99%)

    Merged PR added the sudo marker and tox environment handling; its result explicitly identifies #222 as fixed.

  • Related To: canonical/craft-providers#131 (confidence 95%)

    #222 was reported as a follow-up to #131, whose resolution also references the sudo-test handling in #328.

Related issues

Issue Project State Summary Similarity
#321 lxd test_init should not run by default since it requires sudo craft-providers closed Closed as a duplicate of #222; sudo-requiring LXD tests are now gated behind a `with_sudo` pytest marker via merged PR #328 (commit 0a90009), so test_init no longer runs by default.
83%