Issue #6793 has been updated by Ben Hughes. Status changed from Unreviewed to Needs Decision Assignee set to Nigel Kersten
Thank you for doing the ground work with the problem, very helpful. While it's only a small change to do the right thing, mounts are the kind of things you really don't want to break! I guess the options are: * warn/error in the parser about a "/" on the end. * Make the regexp take in to account a trailing "/" in lib/puppet/provider/mount.rb * Keep the current, safer mechanism and document it accordingly. The reason I say the latter is safe as there's less "magic" involved. Perhaps even taking this to puppet-users? ---------------------------------------- Bug #6793: mount provider fails when paths have a trailing slash https://projects.puppetlabs.com/issues/6793 Author: John Spray Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: mount Target version: Affected Puppet version: 2.6.6 Keywords: Branch: A resource like "/mnt/foo" works fine, but "/mnt/foo/" only works on the first mount. On subsequent runs, puppet incorrectly determines the state as unmounted, tries to mount it, and fails. This is because of puppet/provider/mount.rb function "mounted?" which searches for resource[:name] in the output of mount. Mount never prints the trailing slash, so puppet things the mount isn't there. It should either reject as invalid resources with a trailing slash, or make sure that they are correctly recognised when mounted. -- 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.
