Cgroup teardown logs expected rule misses as warnings #161

New issue

Closed

opened 2026-08-24 18:34:29 -04:00 by mysticalsoap · 0 comments

mysticalsoap commented

2026-08-24 18:34:29 -04:00

Owner

delete_cgroup unconditionally attempts its whole v4+v6 deletion list, and every rule that isn't present (fresh boot, repeated rebuilds, v6 rules on a v6-less network where the inserts were skipped) logs

WARNING - iptables: failed to apply: Command '['ip6tables-legacy', '--wait', '-D', 'OUTPUT', '-m', 'cgroup', ...]' returned non-zero exit status 1.

Delete-before-insert idempotency is fine; logging each expected miss at WARNING is not — it fires in batches on every service start, network change and settings apply, and buries real failures. add_rule already knows how to probe with -C before inserting; deletions could do the same (probe, skip quietly when absent), or expected-miss deletions could log at debug.

Same noise family as the import-side #158 (fixed in #159), remaining half of what a clean log needs.

🤖 Generated with Claude Code

`delete_cgroup` unconditionally attempts its whole v4+v6 deletion list, and every rule that isn't present (fresh boot, repeated rebuilds, v6 rules on a v6-less network where the inserts were skipped) logs ``` WARNING - iptables: failed to apply: Command '['ip6tables-legacy', '--wait', '-D', 'OUTPUT', '-m', 'cgroup', ...]' returned non-zero exit status 1. ``` Delete-before-insert idempotency is fine; logging each expected miss at WARNING is not — it fires in batches on every service start, network change and settings apply, and buries real failures. `add_rule` already knows how to probe with `-C` before inserting; deletions could do the same (probe, skip quietly when absent), or expected-miss deletions could log at debug. Same noise family as the import-side #158 (fixed in #159), remaining half of what a clean log needs. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

mysticalsoap added the

cleanup

label

2026-08-24 18:34:29 -04:00

mysticalsoap referenced this issue from a pull request that will close it,

2026-08-24 19:56:07 -04:00

Quiet the log: expected states stop reporting as failures #163

mysticalsoap closed this issue

2026-08-24 20:16:15 -04:00