LotusGuild/hwmonDaemon
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: show LXC storage in --dry-run summary #25

Merged
jared merged 1 commits from fix/dry-run-lxc-storage-summary into main 2026-07-15 17:03:26 -04:00
Conversation 0 Commits 1 Files Changed 2
+62
jared commented 2026-07-15 17:02:17 -04:00
Owner
Copy Link
Copy Source

The LXC storage check runs in every mode, but its results were never
printed in the --dry-run summary, so a --dry-run appeared to skip LXC
storage entirely. Add an "LXC Storage:" section to the summary
(per-container usage %, a warning flag over the LXC_WARNING threshold, and
the issue count) via a small, testable helper _format_lxc_dry_run().
Adds 4 regression tests.

Closes #23
Ref: #23

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

The LXC storage check runs in every mode, but its results were never printed in the --dry-run summary, so a --dry-run appeared to skip LXC storage entirely. Add an "LXC Storage:" section to the summary (per-container usage %, a warning flag over the LXC_WARNING threshold, and the issue count) via a small, testable helper `_format_lxc_dry_run()`. Adds 4 regression tests. Closes #23 Ref: https://code.lotusguild.org/LotusGuild/hwmonDaemon/issues/23 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
jared added 1 commit 2026-07-15 17:02:17 -04:00
fix: show LXC storage in --dry-run summary
Lint / Python (flake8) (push) Successful in 57s
Details
Security / Python Security (bandit) (push) Successful in 59s
Details
Test / Python Tests (pytest) (push) Successful in 46s
Details
Lint / Notify on failure (push) Has been skipped
Details
Lint / Python (flake8) (pull_request) Successful in 34s
Details
Security / Python Security (bandit) (pull_request) Successful in 34s
Details
Test / Python Tests (pytest) (pull_request) Successful in 1m33s
Details
Lint / Notify on failure (pull_request) Has been skipped
Details
0feac1746d 
The LXC storage check runs in every mode, but its results were never
printed in the --dry-run summary, so a --dry-run appeared to skip LXC
storage entirely. Add an "LXC Storage:" section to the summary
(per-container usage %, a warning flag over the LXC_WARNING threshold, and
the issue count) via a small, testable helper `_format_lxc_dry_run()`.
Adds 4 regression tests.

Closes #23
Ref: #23

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
jared merged commit 26aef22b47 into main 2026-07-15 17:03:26 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
cole (Cole Keller) jared (Jared Vititoe) nathan (Nathan Vititoe)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: LotusGuild/hwmonDaemon#25