Remove String interpolation from mount options. Remove the `join(',')…

…` too. The `options` declaration supports both Array and Strings. Update README accordingly.

README.md

@@ -20,6 +20,7 @@ See `attributes/default.rb` for default vaules.
 * `node['backup']['log_path']` - Where backup logs will be stored. Defaults is `/var/log`
 * `node['backup']['addl_flags']` - Additional flags to pass on to the backup executable, such as `--tmp-path`
 * `node['backup']['model_path']` - Where backup models (definitions) are stored. Default is `node['backup']['config_path']/models`
+* `node['backup']['mount_options']` String or Array of [mount options](https://docs.getchef.com/resource_mount.html#attributes). (example: `["rw", "nfsvers=3"]` would become the comma-delimited string, `rw,nfsvers=3`, passed to the `-o` argument of the `mount` command and inserted into the `<options>` column of `/etc/fstab`).
 * `node['backup']['dependencies']` - An array of arrays of additional dependencies and optional versions needed for backups. The backup gem will inform you about these when the backup runs. (examples: `['fog']`, `[['fog', '1.4.0'], ['s3']]`)
 * `node['backup']['user']` - User that performs backups. Default is root
 * `node['backup']['group']` - Group that performs backups. Default is root

providers/mount.rb

@@ -17,7 +17,7 @@ def whyrun_supported?
     mount new_resource.path do
       fstype 'nfs'
       device device_name
-      options "#{node['backup']['mount_options'].join(',')}"
+      options node['backup']['mount_options']
       action [:mount, :enable]
     end
   end