vagrant-persistent-storage 0.0.31 → 0.0.32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0d9d22a1bec8197414b406c3052a2d5116e6dd2e
-  data.tar.gz: 0c67c81eec1e5e7513ccd673ae139d5e719112aa
+  metadata.gz: dbc1d980eb4bca46f4323c4ee32c75cee63dca50
+  data.tar.gz: 9738ed07cd4c02c730bf7d03e1a3df78ffd93490
 SHA512:
-  metadata.gz: 6617e8ead133179367c0eca3d31f8f49dc0271d721f16fa3b80bed8e8d5a4d0e7080fe8c43d4cbe306734836a16944e87218d0fa67cc7db2f0f3d720eaee9b2c
-  data.tar.gz: bbef6c0ab85e08dc39b9fd94c2d30e5887af7af63b5bbd960b7b800d982acdcad8a093a3937d31e3d3e95726f22bde01511e3b7968ec3616443638f2c0e4b02f
+  metadata.gz: ecd1adcf9d29b82d893849c60aa838ac839d937b8d122ee57df7cf83a4bf67cc373bcff7357550481ce3cfa363030f69586df372a8a4d4022684758f2129c2be
+  data.tar.gz: 7c0450d8de1d2a139c6d0e93c06d0f70fb564ed89308a9c4ebc8f4f5f85632e87d87302dfdd8dd153396df98f590d7aa3ccb560579889811e76ea709015ca336

data/README.md

@@ -34,40 +34,29 @@ config.persistent_storage.mountoptions = ['defaults', 'prjquota']
 ```
 
 Device defaults to `/dev/sdb`. For boxes with multiple disks, make sure you increment the drive:
-```ruby
-config.persistent_storage.diskdevice = '/dev/sdc'
 ```
-
-If you are using LVM and you would prefer to use the disk rather than a partition, you can set the following configuration:
-```ruby
-config.persistent_storage.partition = false
+config.persistent_storage.diskdevice = '/dev/sdc'
 ```
 
 Every `vagrant up` will attach this file as hard disk to the guest machine.
-A `vagrant destroy` will detach the storage to avoid deletion of the storage by vagrant.
+An `vagrant destroy` will detach the storage to avoid deletion of the storage by vagrant.
 A `vagrant destroy` generally destroys all attached drives. See [VBoxMange unregistervm --delete option][vboxmanage_delete].
 
 The disk is initialized and added to it's own volume group as specfied in the config; 
 this defaults to 'vagrant'. An ext4 filesystem is created and the disk mounted appropriately,
-with entries added to fstab ... subsequent runs will mount this disk with the options specified.
+with entries added to fstab ... subsequent runs will mount this disk with the options specified
 
 ## Windows Guests
 
 Windows Guests must use the WinRM communicator by setting `vm.communicator = 'winrm'`.  An additional option is provided to 
 allow you to set the drive letter using:
 
-```ruby
+```
 config.persistent_storage.drive_letter = 'Z'
 ```
 
 Options that are irrelevent to Windows are ignored, such as `mountname`, `filesystem`, `mountpoint` and `volgroupname`.
 
-## How Is The Storage Created?
-
-Based on the configuration provided, during a `vagrant up` a bash script is generated and uploaded to `/tmp/disk_operations_#{mnt_name}.sh` (Linux) or `disk_operations_#{mnt_name}.ps1` (Windows).  If the box has not been previously provisioned the script is executed on a `vagrant up`.  To force the scrip to be executed again you can run `vagrant provision` or if you have halted the box, `vagrant up --provision`.
-
-The outcome of the script being run is placed in the home drive of the vagrant user in a file called `disk_operation_log.txt`.
-
 ## Optional settings
 
 ```ruby

data/lib/vagrant-persistent-storage/config.rb

@@ -15,7 +15,6 @@ module VagrantPlugins
       attr_accessor :mountname
       attr_accessor :mountpoint
       attr_accessor :mountoptions
-      attr_accessor :partition
       attr_accessor :diskdevice
       attr_accessor :filesystem
       attr_accessor :volgroupname
@@ -27,7 +26,6 @@ module VagrantPlugins
       alias_method :manage?, :manage
       alias_method :format?, :format
       alias_method :use_lvm?, :use_lvm
-      alias_method :partition?, :partition
       alias_method :enabled?, :enabled
 
       def initialize
@@ -38,7 +36,6 @@ module VagrantPlugins
         @format = true
         @use_lvm = true
         @enabled = false
-        @partition = true
         @location = UNSET_VALUE
         @mountname = UNSET_VALUE
         @mountpoint = UNSET_VALUE
@@ -57,33 +54,31 @@ module VagrantPlugins
         @manage = true if @manage == UNSET_VALUE
         @format = true if @format == UNSET_VALUE
         @use_lvm = true if @use_lvm == UNSET_VALUE
-        @partition = true if @partition == UNSET_VALUE
         @enabled = false if @enabled == UNSET_VALUE
-        @location = "" if @location == UNSET_VALUE
-        @mountname = "" if @mountname == UNSET_VALUE
-        @mountpoint = "" if @mountpoint == UNSET_VALUE
-        @mountoptions = [] if @mountoptions == UNSET_VALUE
-        @diskdevice = "" if @diskdevice == UNSET_VALUE
-        @filesystem = "" if @filesystem == UNSET_VALUE
-        @volgroupname = "" if @volgroupname == UNSET_VALUE
-		    @drive_letter = 0 if @drive_letter == UNSET_VALUE
+        @location = 0 if @location == UNSET_VALUE
+        @mountname = 0 if @mountname == UNSET_VALUE
+        @mountpoint = 0 if @mountpoint == UNSET_VALUE
+        @mountoptions = 0 if @mountoptions == UNSET_VALUE
+        @diskdevice = 0 if @diskdevice == UNSET_VALUE
+        @filesystem = 0 if @filesystem == UNSET_VALUE
+        @volgroupname = 0 if @volgroupname == UNSET_VALUE
+        @drive_letter = 0 if @drive_letter == UNSET_VALUE
         @part_type_code = "8e" if @part_type_code == UNSET_VALUE
       end
 
       def validate(machine)
-        errors = _detected_errors
+        errors = []
 
         errors << validate_bool('persistent_storage.create', @create)
         errors << validate_bool('persistent_storage.mount', @mount)
-        errors << validate_bool('persistent_storage.manage', @manage)
-        errors << validate_bool('persistent_storage.format', @format)
-        errors << validate_bool('persistent_storage.use_lvm', @use_lvm)
-        errors << validate_bool('persistent_storage.enabled', @enabled)
-        errors << validate_bool('persistent_storage.partition', @partition)
+        errors << validate_bool('persistent_storage.mount', @manage)
+        errors << validate_bool('persistent_storage.mount', @format)
+        errors << validate_bool('persistent_storage.mount', @use_lvm)
+        errors << validate_bool('persistent_storage.mount', @enabled)
         errors.compact!
 
-        if !machine.config.persistent_storage.size.kind_of?(Fixnum)
-          errors << I18n.t('vagrant_persistent_storage.config.not_a_number', {
+        if !machine.config.persistent_storage.size.kind_of?(String)
+          errors << I18n.t('vagrant_persistent_storage.config.not_a_string', {
             :config_key => 'persistent_storage.size',
             :is_class   => size.class.to_s,
           })
@@ -126,21 +121,19 @@ module VagrantPlugins
         end
 
         mount_point_path = Pathname.new("#{machine.config.persistent_storage.location}")
-        if ! (mount_point_path.absolute? || mount_point_path.relative?)
+        if ! mount_point_path.absolute?
           errors << I18n.t('vagrant_persistent_storage.config.not_a_path', {
             :config_key => 'persistent_storage.location',
             :is_path   => location.class.to_s,
           })
         end
 
+        { 'Persistent Storage configuration' => errors }
+
         if ! File.exists?@location.to_s and ! @create == "true"
-          errors << I18n.t('vagrant_persistent_storage.config.no_create_and_missing', {
-            :config_key => 'persistent_storage.create',
-            :is_path   => location.class.to_s,
-          })       
+            return { "location" => ["file doesn't exist, and create set to false"] }
         end
-
-        { 'Persistent Storage configuration' => errors }
+        {}
       end
 
       private

data/lib/vagrant-persistent-storage/manage_storage.rb

@@ -14,7 +14,6 @@ module VagrantPlugins
         fs_type = m.config.persistent_storage.filesystem
         manage = m.config.persistent_storage.manage
         use_lvm = m.config.persistent_storage.use_lvm
-        partition = m.config.persistent_storage.partition
         mount = m.config.persistent_storage.mount
         format = m.config.persistent_storage.format
 	part_type_code = m.config.persistent_storage.part_type_code
@@ -47,7 +46,6 @@ module VagrantPlugins
 		if os == "windows"
 			## shell script for windows to create NTFS partition and assign drive letter
 			disk_operations_template = ERB.new <<-EOF
-      	<% if partition == true %>
 			<% if format == true %>
 			foreach ($disk in get-wmiobject Win32_DiskDrive -Filter "Partitions = 0"){
 				$disk.DeviceID
@@ -55,13 +53,11 @@ module VagrantPlugins
 				"select disk "+$disk.Index+"`r clean`r create partition primary`r format fs=ntfs unit=65536 quick`r active`r assign #{drive_letter}" | diskpart >> disk_operation_log.txt
 			}
 			<% end %>
-		<% end %>
 			EOF
 		else
 		## shell script to format disk, create/manage LVM, mount disk
         disk_operations_template = ERB.new <<-EOF
 #!/bin/bash
-<% if partition == true %>
 # fdisk the disk if it's not a block device already:
 re='[0-9][.][0-9.]*[0-9.]*'; [[ $(sfdisk --version) =~ $re ]] && version="${BASH_REMATCH}"
 if ! awk -v ver="$version" 'BEGIN { if (ver < 2.26 ) exit 1; }'; then
@@ -70,15 +66,13 @@ else
 	[ -b #{disk_dev}1 ] || echo ,,#{part_type_code} | sfdisk #{disk_dev}
 fi
 echo "fdisk returned:  $?" >> disk_operation_log.txt
-disk_dev=#{disk_dev}1
-<% end %>
 
 <% if use_lvm == true %>
 # Create the physical volume if it doesn't already exist:
-[[ `pvs #{disk_dev}` ]] || pvcreate #{disk_dev}
+[[ `pvs #{disk_dev}1` ]] || pvcreate #{disk_dev}1
 echo "pvcreate returned:  $?" >> disk_operation_log.txt
 # Create the volume group if it doesn't already exist:
-[[ `vgs #{vg_name}` ]] || vgcreate #{vg_name} #{disk_dev}
+[[ `vgs #{vg_name}` ]] || vgcreate #{vg_name} #{disk_dev}1
 echo "vgcreate returned:  $?" >> disk_operation_log.txt
 # Create the logical volume if it doesn't already exist:
 [[ `lvs #{vg_name} | grep #{mnt_name}` ]] || lvcreate -l 100%FREE -n #{mnt_name} #{vg_name}
@@ -90,7 +84,7 @@ echo "vg activation returned:  $?" >> disk_operation_log.txt
 
 <% if format == true  %>
 # Create the filesytem if it doesn't already exist
-MNT_NAME=#{vg_name}-#{mnt_name}
+MNT_NAME=#{mnt_name}
 [[ `blkid | grep ${MNT_NAME:0:16} | grep #{fs_type}` ]] || mkfs.#{fs_type} -L #{mnt_name} #{device}
 echo "#{fs_type} creation return:  $?" >> disk_operation_log.txt
 <% if mount == true %>

data/lib/vagrant-persistent-storage/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module PersistentStorage
-    VERSION = "0.0.31"
+    VERSION = "0.0.32"
   end
 end

data/locales/en.yml

@@ -9,7 +9,4 @@ en:
       manage_storage: "** Managing persistent storage **"
     config:
       not_a_bool: "A value for %{config_key} can only be true or false, not type '%{value}'"
-      not_an_array_or_string: "A value for %{config_key} must be an Array or String, not type '%{value}'"
-      not_a_string: "A value for %{config_key} must be a String, not type '%{value}'"
-      not_a_path: "A value for %{config_key} must be resolveable as a path"
-      not_a_number: "A value for %{config_key} must be a number, not type '%{value}'"
+      not_an_array_or_string: "A value for %{config_key} must be an Array or String, not type '%{is_class}'"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-persistent-storage
 version: !ruby/object:Gem::Version
-  version: 0.0.31
+  version: 0.0.32
 platform: ruby
 authors:
 - Sebastian Kusnier