mirror of
https://github.com/TheGrandWazoo/freenas-proxmox.git
synced 2026-10-20 09:38:29 +02:00
docs: close out ADR-012's write-path verification (zfs.snapshot.* tested)
Every API call the WebSocket transport makes is now live-verified against a real TrueNAS SCALE host with zero param-shape issues remaining. Left as Draft pending Kevin's own sign-off, per his explicit instruction earlier. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 5
parent
c9a8376e2e
commit
27b06d8e62
@@ -344,10 +344,23 @@ targetextents, or datasets on TrueNAS — confirmed by querying `.92` directly
|
||||
before and after, which means `pool.dataset.create` and `pool.dataset.delete`
|
||||
are now confirmed working too (the latter's `[id, {recursive,force}]` shape,
|
||||
sourced from documented API research rather than guesswork, checked out on
|
||||
the first try). This narrows the "write path not yet tested" warning
|
||||
considerably but doesn't close it — still entirely untested: `zfs.snapshot.*`
|
||||
(create/delete/rollback/query, none of it touched by this pass). The file's
|
||||
own header warning has been narrowed to reflect this.
|
||||
the first try).
|
||||
|
||||
**2026-08-31 (same day) — zfs.snapshot.* tested, write path fully closed.**
|
||||
Ran a full `volume_snapshot` → `volume_snapshot_info` → `volume_snapshot_rollback`
|
||||
→ `volume_snapshot_delete` cycle against a disposable volume on `.92`. All four
|
||||
succeeded on the first try, no param-shape bugs found — `zfs.snapshot.create`'s
|
||||
plain-object param, `zfs.snapshot.query`'s dataset filter, `zfs.snapshot.rollback`'s
|
||||
positional `(id, options)`, and `zfs.snapshot.delete`'s single positional id
|
||||
all matched what was implemented. Confirmed zero orphaned snapshots or volumes
|
||||
afterward.
|
||||
|
||||
**Every API call this plugin makes over the WebSocket transport is now
|
||||
live-verified against a real TrueNAS SCALE host.** The "write path not yet
|
||||
tested" warning is fully closed — nothing about `_api_ws`'s method mapping
|
||||
remains unverified. `TrueNAS.pm`'s own header comment has been updated to
|
||||
match. This ADR's live-verification bar (Context, above) is now completely
|
||||
satisfied; whether to mark it Accepted is still Kevin's call to make.
|
||||
|
||||
## References
|
||||
|
||||
|
||||
Reference in New Issue
Block a user