Issue #6409 has been updated by Daniel Pittman. Description updated Category set to mount Status changed from Unreviewed to Needs Decision Assignee set to Nigel Kersten
I would like to drop the in-core mount provider, in favour of the split module version, to fix this. Can we do that? ---------------------------------------- Bug #6409: Mount provider should reject spaces in fstab fields https://projects.puppetlabs.com/issues/6409 Author: Paul Berry Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: mount Target version: Affected Puppet version: Keywords: mount fstab Branch: This is accepted by Puppet: mount { '/Volumes/NIKON D40X': ensure => unmounted, device => "/dev/disk1s1", fstype => msdos } But (at least on Mac OS X) it causes problems, because it causes an fstab entry to be created that has a space in the mount point: /dev/disk1s1 /Volumes/NIKON D40X msdos 0 0 According to mac's fstab man page, fstab fields are separated by whitespace, so the "D40X" gets interpreted as the fstype, which obviously doesn't work. Additionally, the next time Puppet needs to change the fstab file, it will change the space to a hard tab, leaving a new entry behind for every single puppet run. For example: /dev/disk1s1 /Volumes/NIKON D40X msdos 0 0 /dev/disk1s1 /Volumes/NIKON D40X msdos 0 0 /dev/disk1s1 /Volumes/NIKON D40X msdos 0 0 It seems likely that similar problems exist on most OSes. It seems like an easy and sufficient solution to this problem is to prohibit spaces in the properties of mount resources. -- 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.
