← Back to issue list

lxd: update logging semantics

View original Github issue

Metadata

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

Current evaluation

Merged as commit ed66930, #378 lowered most LXD warnings to debug, added limited info logs, and fixed #377; Multipass remained pending.

Resolution: closed resolved

Reason: PR #378 was merged into main as commit ed66930, implementing the logging changes and explicitly fixing issue #377.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This commit does two things: - Update almost all instances of log.warning() to log.debug() in lxd- related code. The rationale is that the majority of those messages are meant to assist developers in debugging/troubleshooting, and not really meant for end-users. - Add a couple of log.info() messages. These are only a couple on purpose, as the intent is to provide a _bit_ more information during instance fetching/setup without overwhelming the end-user with info that is ultimately not relevant to them ("implementation details"). For example, we say that we're "Starting the instance" but not that we are "Configuring networking". Since the debug messages are preserved, the information is ultimately still available in logfiles. Note that multipass code still needs to be updated. Fixes #377 - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit ed66930, #378 lowered most LXD warnings to debug, added limited info logs, and fixed #377; Multipass remained pending.
qwen/qwen3.6-35b-a3b — closed resolved Resolved #377 by merging PR #378, which updates LXD logging from warnings to debug/info levels to reduce end-user noise and aid developer troubleshooting.

Update history

Date Change
created

Related work

  • Related To: canonical/craft-providers#377 (confidence 100%)

    The PR explicitly states “Fixes #377” and its merged changes implement the requested logging semantics.

Related issues

Issue Project State Summary Similarity
#1866 lxd: setup LXD remote for multipass snapcraft closed Closed unmerged; its Multipass/LXD remote setup was replaced by #1916 (commit dd4404843), which reworked remote initialization to occur lazily.
72%
#161 lxd: improve error messages craft-providers merged Merged PR #161 consolidated duplicate LXD error messages and added help text to other errors in commit 18d4470.
72%