-
-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Let autofs::mount
mount option default to title.
#66
Conversation
manifests/mount.pp
Outdated
@@ -20,7 +20,7 @@ | |||
# order => 01 | |||
# } | |||
# | |||
# @param mount Location where you will mount the remote NFS Share. | |||
# @param mount Location where you will mount the remote NFS Share. Defaults to the title. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A default doesn't need to specified in a Puppet Strings @param
tag as string is able to look up the defaults on its own
A little nitpicky, but it's nice to keep things consistent across module docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected, I should actually read the puppet-doc docs one day.
@traylenator can you rebase this? |
I'll rebase after #65 if that's okay as it will need redoing again then. |
rebased |
```puppet autofs::mount{'data': mount => '/data', mapfile => '/etc/auto.data', ... } ``` can now be specified as ```puppet autofs::mount{'/data': mapfile => '/etc/auto.data', ... } ``` Given the mount point has to be a unique key in the `/etc/auto.master` anyway this makes sense. Backwards compatible anyway of course.
can now be specified as
Given the mount point has to be a unique key in the
/etc/auto.master
anyway this makes sense.
Backwards compatible anyway of course.