fix: plumb skiplock parameter for VMs, force for containers

VM removal: add skiplock=1 parameter when -Force is specified. PVE honors it
for root@pam only; non-root callers receive 403 errors. Updated help text.

Container removal: map -Force to force=1 (LXC-specific parameter for forcing
removal of running containers). Containers do not support skiplock.

Also: clarified class and parameter documentation to remove false claims about
-Force suppressing confirmation (it does not).

Fixes #136. Addresses correctness reviewer findings.
This commit is contained in:
goodolclint-claude[bot]
2026-09-02 16:41:30 +00:00
committed by GitHub
parent c12bfe9183
commit 75f4bbfeed
5 changed files with 16 additions and 16 deletions
@@ -56,7 +56,7 @@ namespace PSProxmoxVE.Core.Tests.Services
"powershell.exe", new[] { "-NoProfile", "-Command", "echo hi" });
Assert.NotNull(captured);
// Args are argv, not STDIN "input-data" must never be emitted.
// Args are argv, not STDIN - "input-data" must never be emitted.
Assert.DoesNotContain(captured!, kvp => kvp.Key == "input-data");
}