Issue #7188 has been updated by Stefan Schulte.

I don't own a Solaris box so unfortunately I cannot test your code but 
nevertheless I have two questions about it ;-)

* Is there a reason why the mountpoint *type* apparently supports a fstype 
property while the *provider* does not (does this even work?)
* In the remount method you do
<pre>
  def remount
    if resource[:remounts] == :true
      mount_with_options "-o", "remount", resource[:name]
[...]
</pre>
If I had a resource with `options => 'ro'`, wouldn't you then execute `mount -o 
ro -o remount /foo` in this case? Is that valid mount syntax?
----------------------------------------
Feature #7188: Mount provider/type split for Solaris
https://projects.puppetlabs.com/issues/7188

Author: Nigel Kersten
Status: Accepted
Priority: High
Assignee: 
Category: 
Target version: 
Affected Puppet version: 
Keywords: 
Branch: https://github.com/stschulte/puppet/commits/feature%2Fnext%2Fmounttab


Splitting the mount type/provider into two, one to manage  "fstab" entries, the 
other to manage actual mounts.

This work will be done as an open sourced module that should be compatible with 
2.6.x and 2.7.x, with a view to merge this into Telly before release.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to