[pve-devel] [PATCH v6 pve-storage 0/6] FreeNAS storage plugin

mir at datanom.net mir at datanom.net
Mon Jun 19 17:13:18 CEST 2017


From: Michael Rasmussen <mir at datanom.net>

(Resending, Diregard previous)

This patch series adds a storage plugin for FreeNAS using the FreeNAS API.
The plugin supports both Qemu and LXC VM. Features supported, see below:
    
                    VM          CT
create              YES         YES
delete              YES         YES
resize              (YES)       (YES) Note 1
snapshot
   offline          YES         YES
   live             YES         "YES" (state is not saved)
backup
   snapshot         YES         YES
   standby          YES         YES
   offline          YES         YES
clone
   full             YES         YES (Unavailable in GUI)
   linked           YES         YES (Unavailable in GUI)
    
Note 1: Due to a bug in the FreeNAS API live resizing has been disabled
in the plugin. See https://bugs.freenas.org/issues/24432

Since the plugin attaches disks through the local scsi subsystem via
openiscsi and therefore not uses libiscsi there should be support for
MPIO and authentication. MPIO is not tested since I don't have the
required hardware to do so. Authentication is not enabled but should be
easy to do if the requirements exists.

Changes since v5:
    * if ! to unless
    * all helper methods now private
    * remove unused includes
    * remove unused code
    * more descriptive variable names
    * change api timeout to be in sync with the PVE API
    * add loop over limit until empty resultset is returned
    * fix API version check
    * fix handling 409 code in create_target and create_target_group
    * Remove unnecessary error handling
    * Replace hardcoded max luns number with variable
    * Fix error handling in freenas_create_lun method
    * Write directly to file instead of using a shell to echo to file
    * Fix error handling in deactivate_lun and get_active_luns
    * Improve code readability and error handling in rescan_session
    * Declaring variables where used in create_base
    * Check if base already exists when creating base
    * Improve error handling in create_base
    * Fix clone_image error handling and remove unnecessary check for running base
    * Remove unused variable in alloc_image
    * More robust error handling in free_image
    * Remove HTML code from volume_resize
    * Fix error handling in shapshot rollback
    * Improve code readability and error handling in activate_lun
    * Improve code readability and error handling in deactivate_lun
    * Replace sleep 1 with a combination of udevadm trigger udevadm settle
    * Remove check of running VM and CT. relaying on parsed option running for
      Qemu and handle running LXC in PVE::API2::LXC now sends running status
      as part of call to volume_resize when storage id is freenas.

Changes since v4:
    * regression and indentation fix

Changes since v3:
    * Adds support for creating snapshot backups of LXC containers.
      This more or less makes the plugin feature complete.
      Only outstanding feature depends on a bug fix in the FreeNAS API
      which is first scheduled for next stable release:
      https://bugs.freenas.org/issues/23881 (Targeted for 11.1)

Changes since v2:
    * These are internal revisions not published


Michael Rasmussen (6):
  First beta of FreeNAS storage plugin. Missing snapshot backup for LXC
  Proper check if VM/CT is running
  Fix missing $vmid
  Add support for creating LXC snapshot backup
  Fix regression and indentation
  Bug fixes and clean-ups in response to review from Fabian.

 PVE/Storage.pm               |    2 +
 PVE/Storage/FreeNASPlugin.pm | 1362 ++++++++++++++++++++++++++++++++++++++++++
 PVE/Storage/Makefile         |    2 +-
 PVE/Storage/Plugin.pm        |    2 +-
 4 files changed, 1366 insertions(+), 2 deletions(-)
 create mode 100644 PVE/Storage/FreeNASPlugin.pm

-- 
2.11.0


----

This mail was virus scanned and spam checked before delivery.
This mail is also DKIM signed. See header dkim-signature.




More information about the pve-devel mailing list