public_git/freenas-proxmox
1
0
Fork 0
mirror of https://github.com/TheGrandWazoo/freenas-proxmox.git synced 2026-09-30 03:51:27 +02:00
Code Issues Projects Releases Wiki Activity

docs: narrow TrueNAS.pm's WS write-path warning to zfs.snapshot.* only

Browse Source 
Everything else in the write path (target/extent/targetextent/dataset
create+delete) is now live-confirmed, see ADR-012.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Kevin Adams
2026-08-31 10:13:35 -04:00
co-authored by Claude Sonnet 5
parent c05de59396
commit 95ec8e1fa8
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
perl5/PVE/Storage/Custom/TrueNAS.pm
+9 -5
View File
@@ -18,11 +18,15 @@ package PVE::Storage::Custom::TrueNAS;
#
# WARNING: the WebSocket transport (_api_ws, _ws_connect, _ws_call) has been
# live-verified for every *read* (query) call this plugin makes (ADR-012,
# four runs across three TrueNAS SCALE versions incl. 25.04.2.6). The
# *write* path (create/delete/update param shapes below) follows TrueNAS's
# documented CRUDService conventions but has NOT yet been independently
# live-tested against a real host — verify against .91/.92 before trusting
# alloc_image/free_image/snapshot operations in production on SCALE 25.04+.
# four runs across three TrueNAS SCALE versions incl. 25.04.2.6), and for a
# full alloc_image/path/volume_size_info/free_image cycle against .92
# (2026-08-31) — zvol, extent, target, targetextent create AND delete all
# confirmed leaving zero orphans. Two real bugs were caught and fixed doing
# that (see ADR-012): missing int() on regex-captured ids, and
# iscsi.extent.delete's positional (id, remove, force) signature, which is
# NOT (id, {force=>bool}). Still entirely untested against the WS transport:
# zfs.snapshot.* (create/delete/rollback/query) — verify before trusting
# volume_snapshot* operations in production on SCALE 25.04+.
#
# Per-VM target architecture:
# Each VM gets its own iSCSI target (proxmox-vm-<vmid>).