← Back to issue list

spread: remove dead code for lxd setup and add debug prints

View original Github issue

Metadata

Project
snapcraft
Number
#3112
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3112 in commit cf2fec383, removing unreachable LXD setup code and adding debug output for intermittent LXD snap-install failures.

Resolution: closed resolved

Reason: PR #3112 was merged as commit cf2fec383, delivering the dead-code removal and LXD installation debug logging.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The elif was effectively evaluated by the prior if. Add some debug logic too to help solve some random lxd snap install failures. Signed-off-by: Chris Patterson <chris.patterson@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #3112 in commit cf2fec383, removing unreachable LXD setup code and adding debug output for intermittent LXD snap-install failures.
stealth/space-bunny-alpha — closed resolved Merged #3112 (commit cf2fec383), removing unreachable LXD setup code and adding debug output to diagnose intermittent LXD snap installation failures.
qwen/qwen3.6-35b-a3b — — Merged removal of redundant elif dead code in LXD setup logic and addition of debug prints for LXD snap install failures. Approved by one reviewer, passed CI checks, and integrated.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes removing dead code in the LXD setup logic and adding debug prints to diagnose LXD snap installation failures.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1677 ci: don't install lxd craft-parts merged Merged PR #1677 removed unused LXD installation from CI; commits 2b568926 and 5282b96f record the change.
72%
#1647 lxd: better surfacing of errors snapcraft merged Merged in snapcraft#1647 as commit 71f48bdc9, improving LXD error handling so callers return silent, correctly coded errors without printing messages twice.
71%
#3281 build providers: hide systemd setup for LXD snapcraft merged Merged in PR #3281 (commit 8029036c6), hiding verbose LXD systemd setup output by default while preserving it in internal debug logs.
71%
#2936 build providers: clean up LXD startup message snapcraft merged Merged PR snapcraft#2936 (commit bc605d706) trimmed the misleading LXD cloud-init startup log message, removing an implementation detail that users misread as data being sent to the cloud.
70%
#36 lxd: introduce installation support craft-providers merged Merged PR #36 added LXD installation support, including installer logic and tests; implementation commit aab8fd0 was merged via 1a1212c.
70%
#2983 tests: add LXD spread test snapcraft merged Merged PR #2983 as 4b8522a14, adding spread coverage that builds the LXD package with LXD and destructive/host providers; installation checks were deferred.
70%