freenas-proxmox

Commit Graph

Author	SHA1	Message	Date
Kevin Adams	3b3cc8d135	fix: ZFSPlugin 8.4.x patch and missing patch dependency (#236 #237 ) Two install-time failures on PVE 8.4.x nodes: 1. 'patch' utility not in Depends — causes silent patch failure on fresh nodes; all three PVE file patches silently skip, plugin appears to install but storage operations fail. Added 'patch' to control.j2. 2. ZFSPlugin.pm routing not applied on PVE 8.4.x — tabs→spaces reformat and get_base($scfg) signature change in 8.4 broke hunks #3 and #4 (the freenas elsif routing branches). Nodes with a prior partial install are doubly affected: grep idempotency check passes on partial freenas strings, so the routing is never applied, causing: "freenas: unknown iscsi provider" on clone/migrate operations. New ZFSPlugin-8.4.14_1.pm.patch generated against stock PVE 8.4.19, bundled as patches/ZFSPlugin/8.4.patch alongside existing 8.patch. Verified clean on pve01-hq (8.4.19) and pve02-hq (8.4.14). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-19 23:16:17 -04:00
Kevin Adams	ac58bf75de	Fix DEBIAN control.j2: use binary-only format for dpkg-deb dpkg-deb --build reads a binary control file and rejects the two-stanza source+binary format. Rewrote control.j2 as a single binary-only stanza. Removed the source stanza (Source, Standards-Version, Build-Depends fields) which are only meaningful in source packages built with dpkg-buildpackage. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-15 15:16:20 -04:00
Kevin Adams	636cd06ff6	Phase 1 project revival + PR fixes (#207 , #209 , #213 ) Project infrastructure: - Replace MIT license with AGPL-3.0 (KSA Technologies, LLC) - Full README rewrite with badges, compatibility table, token auth docs - Add CHANGELOG, CONTRIBUTING, DONORS, SECURITY docs - Add GitHub issue/PR templates, update FUNDING.yml and stale.yml - Replace external packer repo dispatch with self-contained CI (build.yml) - Add packaging/DEBIAN/ with postinst/postrm/triggers (no git clone at install) - Add .perlcriticrc for static analysis - Add .claude/cos/ ADRs, plans, and runbooks Bug fixes from community PRs: - Fix bearer token check in freenas_api_connect: defined() && value instead of defined() alone, so truenas_token_auth=0 no longer activates Bearer Token auth (#207) - Fix LUN 0 falsy bug in ZFSPlugin patch: !$guid -> !defined $guid in both zfs_get_lun_number and zfs_get_wwid_number, fixing VMs on LUN 0 for PVE 9 (#209) - Fix syslog typo "wtih" -> "with" in run_list_extent (#213) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-15 14:22:04 -04:00

Author

SHA1

Message

Date

Kevin Adams

3b3cc8d135

fix: ZFSPlugin 8.4.x patch and missing patch dependency (#236 #237 )

Two install-time failures on PVE 8.4.x nodes:

1. 'patch' utility not in Depends — causes silent patch failure on fresh
   nodes; all three PVE file patches silently skip, plugin appears to
   install but storage operations fail. Added 'patch' to control.j2.

2. ZFSPlugin.pm routing not applied on PVE 8.4.x — tabs→spaces reformat
   and get_base($scfg) signature change in 8.4 broke hunks #3 and #4
   (the freenas elsif routing branches). Nodes with a prior partial
   install are doubly affected: grep idempotency check passes on partial
   freenas strings, so the routing is never applied, causing:
     "freenas: unknown iscsi provider" on clone/migrate operations.

New ZFSPlugin-8.4.14_1.pm.patch generated against stock PVE 8.4.19,
bundled as patches/ZFSPlugin/8.4.patch alongside existing 8.patch.

Verified clean on pve01-hq (8.4.19) and pve02-hq (8.4.14).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-05-19 23:16:17 -04:00

Kevin Adams

ac58bf75de

Fix DEBIAN control.j2: use binary-only format for dpkg-deb

dpkg-deb --build reads a binary control file and rejects the two-stanza
source+binary format. Rewrote control.j2 as a single binary-only stanza.
Removed the source stanza (Source, Standards-Version, Build-Depends fields)
which are only meaningful in source packages built with dpkg-buildpackage.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-05-15 15:16:20 -04:00

Kevin Adams

636cd06ff6

Phase 1 project revival + PR fixes (#207 , #209 , #213 )

Project infrastructure:
- Replace MIT license with AGPL-3.0 (KSA Technologies, LLC)
- Full README rewrite with badges, compatibility table, token auth docs
- Add CHANGELOG, CONTRIBUTING, DONORS, SECURITY docs
- Add GitHub issue/PR templates, update FUNDING.yml and stale.yml
- Replace external packer repo dispatch with self-contained CI (build.yml)
- Add packaging/DEBIAN/ with postinst/postrm/triggers (no git clone at install)
- Add .perlcriticrc for static analysis
- Add .claude/cos/ ADRs, plans, and runbooks

Bug fixes from community PRs:
- Fix bearer token check in freenas_api_connect: defined() && value instead of
  defined() alone, so truenas_token_auth=0 no longer activates Bearer Token auth (#207)
- Fix LUN 0 falsy bug in ZFSPlugin patch: !$guid -> !defined $guid in both
  zfs_get_lun_number and zfs_get_wwid_number, fixing VMs on LUN 0 for PVE 9 (#209)
- Fix syslog typo "wtih" -> "with" in run_list_extent (#213)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-05-15 14:22:04 -04:00

3 Commits