[PR #18] [MERGED] Add app_update_exclude_list and os_only_lxc_list for LXC updates #18

New issue

Closed

opened 2026-06-08 17:02:13 +08:00 by ivenator1 · 0 comments

ivenator1 commented

2026-06-08 17:02:13 +08:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ivenator1/proxmox-management/pull/18
Author: @ivenator1
Created: 6/8/2026
Status: ✅ Merged
Merged: 6/8/2026
Merged by: @ivenator1

Base: testing ← Head: claude/lxc-os-only-app-exclude

📝 Commits (1)

bc0d7d2 feat: add app_update_exclude_list and os_only_lxc_list for LXC updates

📊 Changes

8 files changed (+130 additions, -32 deletions)

View changed files

📝 CLAUDE.md (+5 -2)
📝 proxmox_fleet/driver.py (+1 -1)
📝 proxmox_fleet/flows/lxc.py (+40 -28)
📝 proxmox_fleet/models/settings.py (+2 -0)
📝 proxmox_fleet/status.py (+3 -0)
📝 tests/unit/test_flow_lxc.py (+51 -1)
📝 tests/unit/test_status_lxc.py (+18 -0)
📝 vars.yml.example (+10 -0)

📄 Description

Summary

app_update_exclude_list: skip the community-script app update for tagged LXCs while still running apt/apk OS updates (mirrors os_update_exclude_list); reports app=SKIPPED
os_only_lxc_list: pull untagged LXC IDs into discovery for OS-only management — they lack /usr/bin/update, so the existing flow naturally reports app="NO SCRIPT" while still updating the OS

Test plan

pytest tests/unit/ — 619 passed
python -m mypy proxmox_fleet/ — clean
--dry-run against the test environment with one ID in each new list, confirm briefing shows app=SKIPPED and app=NO SCRIPT as expected

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ivenator1/proxmox-management/pull/18 **Author:** [@ivenator1](https://github.com/ivenator1) **Created:** 6/8/2026 **Status:** ✅ Merged **Merged:** 6/8/2026 **Merged by:** [@ivenator1](https://github.com/ivenator1) **Base:** `testing` ← **Head:** `claude/lxc-os-only-app-exclude` --- ### 📝 Commits (1) - [`bc0d7d2`](https://github.com/ivenator1/proxmox-management/commit/bc0d7d2c002dc0a9da2ffbcfc911083ef76457f0) feat: add app_update_exclude_list and os_only_lxc_list for LXC updates ### 📊 Changes **8 files changed** (+130 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `CLAUDE.md` (+5 -2) 📝 `proxmox_fleet/driver.py` (+1 -1) 📝 `proxmox_fleet/flows/lxc.py` (+40 -28) 📝 `proxmox_fleet/models/settings.py` (+2 -0) 📝 `proxmox_fleet/status.py` (+3 -0) 📝 `tests/unit/test_flow_lxc.py` (+51 -1) 📝 `tests/unit/test_status_lxc.py` (+18 -0) 📝 `vars.yml.example` (+10 -0) </details> ### 📄 Description ## Summary - `app_update_exclude_list`: skip the community-script app update for tagged LXCs while still running `apt`/`apk` OS updates (mirrors `os_update_exclude_list`); reports `app=SKIPPED` - `os_only_lxc_list`: pull untagged LXC IDs into discovery for OS-only management — they lack `/usr/bin/update`, so the existing flow naturally reports `app="NO SCRIPT"` while still updating the OS ## Test plan - [x] `pytest tests/unit/` — 619 passed - [x] `python -m mypy proxmox_fleet/` — clean - [ ] `--dry-run` against the test environment with one ID in each new list, confirm briefing shows `app=SKIPPED` and `app=NO SCRIPT` as expected 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

ivenator1

2026-06-08 17:02:13 +08:00